Skip to content

Release 1.6.3: review carry-over - #29

Merged
noderaven merged 10 commits into
mainfrom
release-1.6.3
Oct 15, 2026
Merged

noderaven merged 10 commits into
mainfrom
release-1.6.3

Conversation

@noderaven

Copy link
Copy Markdown
Owner

Release 1.6.3: review carry-over. What earlier agents decided or found now reaches the agents who come after them, once and correctly.

Built directly (no parallel-lanes run). Design: revision 3, checked by a builder review and an adversarial claim check; the build then had a whole-branch code review and an adversarial end-to-end review on copies of three real run archives, with one fix pass.

Changes

  • Ledger: ruling roles, ids, scope, overrides and permitted files; ledger rulings, carry-launch, minors, owners, undeclared, write-owners; status gains minors and overrides, and undeclared becomes owner-annotated.
  • carry-launch records a launch's pre-flight rulings and the task minors still open after it, from its Workflow result file, once per launch and in launch order.
  • Engine: every task agent starts with the rulings in force; the adjudicator, final agents and a relaunch's pre-flight read them; only the security lens owns the task-minor checklist; carried minors reach it on a resume (final.carried_minors_open); a relaunch with nothing left to run skips pre-flight; run rulings are numbered PF-n.
  • Setup writes an owner map; reviewers see who owns each file changed outside a task's Files list; implementers need a ruling before touching another lane's or a later task's file; failures outside a task are recorded as run-wide observed rulings.
  • Docs: reference.md "Carry-over", SKILL.md Launch step 5 and Resume step 2, README.

Testing

  • Full suite on Windows: 1497 tests, 1481 pass, 16 skipped, 0 fail.
  • Python 3.8.20 (uv): full suite 1487 tests, 0 fail before the fix pass; the Python-backed files again after it, 176 tests, 0 fail.
  • carry-launch exercised on copies of the sc1, f1a and hx1 archives.

- Ruling ledger events record the role: by is implementer for implement
  and fix agents, reviewer for review and re-review agents.
- The adjudicator's prompt shows the run rulings (pre-flight's, and its
  own pre-flight ruling), so it decides with them in force.
- The run result keeps tasks.<id>.minor_findings (the approving review's,
  done tasks only) and tasks.<id>.rulings, so a later launch can read them.
- A relaunch with every task done and reviewed runs no pre-flight and the
  dry run plans none (preflight.skipped: "no task left to run").
- reference.md: quitting Claude Code (2.1.300+) during a check kills it;
  the wait reports stopped, reason died.
@noderaven
noderaven merged commit 911b4aa into main Oct 15, 2026
4 checks passed
@noderaven
noderaven deleted the release-1.6.3 branch October 15, 2026 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant