fix(application): Complete PR #9247 compliance — add CHANGELOG and CONTRIBUTORS entries
CI / benchmark-regression (push) Has been skipped
CI / helm (push) Successful in 1m0s
CI / push-validation (push) Successful in 1m5s
CI / typecheck (push) Successful in 2m14s
CI / quality (push) Successful in 2m14s
CI / security (push) Successful in 2m18s
CI / lint (push) Successful in 1m51s
CI / integration_tests (push) Successful in 4m27s
CI / unit_tests (push) Successful in 5m49s
CI / docker (push) Successful in 1m40s
CI / coverage (push) Successful in 13m42s
CI / build (push) Successful in 29s
CI / e2e_tests (push) Successful in 3m22s
CI / status-check (push) Successful in 3s
CI / benchmark-publish (push) Successful in 1h17m45s
CI / benchmark-publish (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 4m24s
CI / push-validation (pull_request) Successful in 35s
CI / helm (pull_request) Successful in 37s
CI / build (pull_request) Successful in 1m5s
CI / lint (pull_request) Successful in 1m22s
CI / quality (pull_request) Successful in 1m42s
CI / security (pull_request) Successful in 1m43s
CI / typecheck (pull_request) Successful in 1m50s
CI / integration_tests (pull_request) Successful in 4m37s
CI / unit_tests (pull_request) Successful in 4m58s
CI / docker (pull_request) Successful in 1m28s
CI / benchmark-regression (pull_request) Failing after 35s
CI / coverage (pull_request) Successful in 10m46s
CI / status-check (pull_request) Successful in 5s

- Add CHANGELOG.md entry under [Unreleased] Fixed section for error suppression removal (issue #9060)
- Update CONTRIBUTORS.md with HAL 9000 contribution note for this fix
- Branch: bugfix/m-error-suppression-reactive-registry-adapter-v2

ISSUES CLOSED: #9060
This commit was merged in pull request #9247.
This commit is contained in:
2026-05-07 10:24:22 +00:00
committed by Forgejo
parent 2778bde95a
commit 94dd77fbcd
2 changed files with 4 additions and 0 deletions
+2
View File
@@ -47,6 +47,8 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
### Fixed
- **Error suppression removed from `reactive_registry_adapter.py`** (#9060): Removed two `try...except Exception:` blocks in `register_registry_agents()` that silently suppressed errors, violating the CONTRIBUTING.md fail-fast policy. Exceptions from `actor_registry.list_actors()` and the route bridge refresh now propagate to the caller instead of being swallowed. Added Behave scenarios verifying RuntimeError, AttributeError, and TypeError propagation.
- **Implementation Supervisor PR Compliance Checklist** (#9824): Added a mandatory
8-item PR Compliance Checklist to the worker prompt body in `implementation-supervisor.md`
that every implementation worker must complete before creating a PR. Checklist covers:
+2
View File
@@ -32,3 +32,5 @@ Below are some of the specific details of various contributions.
* HAL 9000 has contributed comprehensive milestone documentation for v3.6.0 (Advanced Concepts & Deferred Features) and v3.7.0 (TUI Implementation) (PR #9903): split into sub-documents covering context strategies, LLM backends, resource types, A2A rename, container tool execution, scope chain resolution, cost/safety budgets, E2E workflow tests, code review examples, plugin architecture, TUI layout, persona system, reference/command input, session management, configuration, and TuiMaterializer integration.
* HAL 9000 has contributed the LLMTraceRepository data-integrity fix (PR #8185 / issue #7505): replaced the unconditional `session.commit()` in `LLMTraceRepository.save()` with a dual-path implementation that respects the UnitOfWork pattern — flushing only when an external session is provided, and flushing + committing + closing when operating standalone. This eliminates premature transaction commits, loss of rollback capability, and a docstring/implementation mismatch.
* HAL 9000 has contributed the ACMS Index Data Model and File Traversal Engine (PR #9664 / issue #9579): foundational data structures for indexed context entries with hot/warm/cold/archive storage tier classification, tag system, and a timeout-safe chunked file traversal engine for large projects with 10,000+ files.
* HAL 9000 has contributed the error-suppression removal fix (PR #9247 / issue #9060): removed both `try...except Exception:` blocks in `register_registry_agents()` that silently suppressed errors from `actor_registry.list_actors()` and the route bridge refresh, enabling exceptions to propagate per CONTRIBUTING.md fail-fast policy. Added three Behave scenarios verifying RuntimeError, AttributeError, and TypeError propagation.