Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,11 @@ All notable changes to the claude-plugins project will be documented in this fil

The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). Entries are listed newest-first; each plugin section is treated as released when merged to `main`.

### code-review v3.4.1

#### Fixed
- Corrected stale documentation that described the deterministic-prefix golden harness as unbuilt future work. The `golden_fixture_harness.py` and `cmd_prepare_run` docstrings and the `SCHEMA.md` golden-fixture section now point at the delivered prefix harness (`prefix_golden_harness.py`) and its subprocess A/B parity oracle, and note that the orchestrator runs the deterministic prefix in-process via `run-prefix` rather than walking those stages one at a time.

### code-review v3.4.0

#### Changed
Expand Down
2 changes: 1 addition & 1 deletion plugins/code-review/.claude-plugin/plugin.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "code-review",
"description": "Code review plugin",
"version": "3.4.0",
"version": "3.4.1",
"author": {
"name": "ClosedLoop",
"email": "support@closedloop.ai"
Expand Down
6 changes: 4 additions & 2 deletions plugins/code-review/SCHEMA.md
Original file line number Diff line number Diff line change
Expand Up @@ -719,8 +719,10 @@ The remaining 3 fixtures requiring plans 03/05/06
(`golden_impact_with_callsites`, `golden_coverage_gap`,
`golden_budget_exceeded`) have reserved directories with READMEs and
are skipped via a `_DEFERRED_FIXTURES` map in the test module until
their dependent plans land. Phase 4b will extend the harness to walk
`run_plan.json` end-to-end through a declarative stage runner.
their dependent plans land. The deterministic prefix (stages `01` through
Gate B) is pinned separately by `prefix_golden_harness.py` (PLN-1229
Phase 0); its subprocess A/B parity oracle guards the in-process
`run-prefix` batch runner byte-for-byte.

---

Expand Down
3 changes: 2 additions & 1 deletion plugins/code-review/tools/python/code_review_helpers.py
Original file line number Diff line number Diff line change
Expand Up @@ -11013,7 +11013,8 @@ def cmd_prepare_run(args: argparse.Namespace) -> int:
"""Emit ``run_plan.json`` describing the full review pipeline.

PLN-719 Section 6. The output is consumed by the ``/start`` orchestrator,
which walks the plan stage-by-stage (Phase 4b).
which runs the deterministic prefix in-process via ``run-prefix``
(PLN-1229) and walks the reviewer tail stage-by-stage.

Determinism: same inputs produce byte-identical output **except for the
``review_id`` field**, which is a fresh ``uuid.uuid4()`` per invocation.
Expand Down
7 changes: 5 additions & 2 deletions plugins/code-review/tools/python/golden_fixture_harness.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,11 @@
the wall-clock telemetry block) are normalized before diff/write so the
expected files stay stable across runs.

Phase 8 ships the post-collection harness only. Phase 4b will extend it to
walk ``run_plan.json`` end-to-end through a declarative stage runner.
This module pins the post-collection half only (``collect-findings`` →
``validate`` → ``finalize-result``). The deterministic prefix — stages ``01``
through Gate B — is pinned separately by ``prefix_golden_harness.py``
(PLN-1229 Phase 0), whose subprocess A/B parity oracle guards the in-process
``run-prefix`` batch runner byte-for-byte.
"""

from __future__ import annotations
Expand Down
Loading