Skip to content

feat(run-wave): carry-ins to the planner and late carry-ins to the implementer [a3d6a64a] - #624

Merged
jpicklyk merged 3 commits into
release/4.0from
4.0/carry-in-plumbing
Oct 22, 2026
Merged

jpicklyk merged 3 commits into
release/4.0from
4.0/carry-in-plumbing

Conversation

@jpicklyk

@jpicklyk jpicklyk commented Oct 22, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • 4.0 twin of main feat(plugin): planner and reviewer enforce carry-in dispositions [f32d5442] #574 (d8f23c8): carry-in plumbing that the 4.0 planner/reviewer agent text already expects.
  • run-planner-lib.mjs: carryInKeys / isCarryInDisposed, snapshot carryIns validation, per-item carryIns / lateCarryIns, meta.lateCarryIns, and a Late carry-ins: explain line.
  • implement-wave.js: promptCarryIns in seatPrompt (a CARRY-INS block for planner-v1, a LATE CARRY-INS block for implementer-v1). It uses 4.0 notes_read(select:"get") per key, and adds no new core export.
  • run-wave SKILL.md Steps 2 and 3, references/snapshot.md, and the args-v1 schema fixture.

Test Results

  • New blind test file carry-in-dispositions.test.mjs: 28/28.
  • scripts and hooks globs after the review fixes: 2528 pass, 4 skip, 1 timing flake in the untouched phase-guard.test.mjs parallel-fetch test (667ms against a 600ms limit under the concurrent run). That file then passed 47/47 alone. gen-call-shapes --check: clean.

Review

Independent opus review: Pass with observations, 0 blocking, 7 observations. Test independence: independent.

  • O1 (the keyless list is paged): fixed in the docs commit, which tells Step 2 to follow page.next.
  • O7 (process wording in the CHANGELOG bullet): fixed.
  • O2, O3 and O4 accepted: O2 is wording sanctioned by A1, O3 is a minor test-strength gap, and O4 is behaviour inherited from main.
  • O5 is a dispatch-process note. O6 is follow-up c52b5181.
  • 7 of 7 disposed.

Docs

claude-plugins/task-orchestrator/skills/run-wave/SKILL.md, claude-plugins/task-orchestrator/skills/run-wave/references/snapshot.md, CHANGELOG.md

Changelog

One bullet under ## [4.0.0] -> ### Plugin: "Changed: run-wave carries carry-in note keys through the plan." 4.0.0 is cut but not tagged.

MCP

a3d6a64a-4d4f-40b0-b91a-a1f5d9e2506f (follow-up: c52b5181, review-wave CARRY-INS)

🤖 Generated with Claude Code

jpicklyk and others added 3 commits October 22, 2026 13:52
…planner and late carry-ins to the implementer [a3d6a64a]

4.0 twin of main #574 (d8f23c8).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…y CHANGELOG wording [a3d6a64a]

Review O1 and O7.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jpicklyk
jpicklyk enabled auto-merge (squash) October 22, 2026 18:03
@jpicklyk
jpicklyk merged commit 75df163 into release/4.0 Oct 22, 2026
5 checks passed
@jpicklyk
jpicklyk deleted the 4.0/carry-in-plumbing branch October 22, 2026 18:13
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