Skip to content

fix(orchestration): bind review and acceptance repositories - #13

Merged
Ldsystem merged 5 commits into
mainfrom
codex/fix-bound-review-package-root
Aug 26, 2026
Merged

fix(orchestration): bind review and acceptance repositories#13
Ldsystem merged 5 commits into
mainfrom
codex/fix-bound-review-package-root

Conversation

@Ldsystem

@Ldsystem Ldsystem commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • resolve review-package Git refs and diffs from the validated task execution binding
  • evaluate plan-level acceptance in the material execution repository, not the workspace control repository
  • select the fresh repository recorded by the terminal passing integration handoff when task worktrees have non-linear squash histories
  • keep generated review-package output under the workspace control root
  • add regressions for separate control/execution repositories and multi-worktree terminal acceptance

Validation

  • canonical full suite before the terminal-root refinement: 527 passed
  • affected workflow-contract suite after the final refinement: 53 passed
  • real archived plan knowledge and acceptance gates: 7 validated handoffs, passed
  • git diff --check: passed
  • independent local Cursor/Grok review: accepted exact head c8a95e0a967b614146d4efc1dcfefe8a3dab49ce, no findings

Safety

  • no public CLI change
  • ambiguous repositories still fail closed when no unique fresh acceptance root exists
  • remains open and unmerged pending review

@Ldsystem Ldsystem changed the title fix(orchestration): diff bound execution repository fix(orchestration): bind review and acceptance repositories Aug 26, 2026
@Ldsystem
Ldsystem merged commit 088ccfb into main Aug 26, 2026
2 checks passed
@Ldsystem
Ldsystem deleted the codex/fix-bound-review-package-root branch August 26, 2026 14:29
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