From 94dd77fbcd7ed2226cd1f9d11b90c43bf0a3eda1 Mon Sep 17 00:00:00 2001 From: HAL9000 Date: Thu, 7 May 2026 10:24:22 +0000 Subject: [PATCH] =?UTF-8?q?fix(application):=20Complete=20PR=20#9247=20com?= =?UTF-8?q?pliance=20=E2=80=94=20add=20CHANGELOG=20and=20CONTRIBUTORS=20en?= =?UTF-8?q?tries=20-=20Add=20CHANGELOG.md=20entry=20under=20[Unreleased]?= =?UTF-8?q?=20Fixed=20section=20for=20error=20suppression=20removal=20(iss?= =?UTF-8?q?ue=20#9060)=20-=20Update=20CONTRIBUTORS.md=20with=20HAL=209000?= =?UTF-8?q?=20contribution=20note=20for=20this=20fix=20-=20Branch:=20bugfi?= =?UTF-8?q?x/m-error-suppression-reactive-registry-adapter-v2?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ISSUES CLOSED: #9060 --- CHANGELOG.md | 2 ++ CONTRIBUTORS.md | 2 ++ 2 files changed, 4 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index d041f84d2..c16904dc4 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -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: diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md index eea7976f0..51815111f 100644 --- a/CONTRIBUTORS.md +++ b/CONTRIBUTORS.md @@ -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. \ No newline at end of file