Skip to content

debrief: route split-root debriefs to the state checkout, not the definition dir#449

Merged
clkao merged 1 commit into
mainfrom
spacedock-ensign/debrief-split-root-state-home
Jun 30, 2026
Merged

debrief: route split-root debriefs to the state checkout, not the definition dir#449
clkao merged 1 commit into
mainfrom
spacedock-ensign/debrief-split-root-state-home

Conversation

@clkao

@clkao clkao commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Route split-root debriefs to the state checkout on its own branch — keeping session churn off the code trunk and ending cross-home numbering collisions.

What changed

  • Resolve the debrief home from status --boot --json (entity_dir) and thread it through all four _debriefs/ touch-points.
  • Commit split-root debriefs path-scoped on the state branch (rebase-conflict halt); leave the single-root path unchanged.
  • Halt-gate a declared-but-absent state checkout ("state not initialized") before any write.
  • Add a four-case split-root regression guard.

Evidence

  • go test ./skills/integration/ -run Debrief — 4/4 passed; the guard bites under a pre-fix revert (AC-1/3/4 fail, AC-2 holds).
  • Live /debrief drive on a real split-root fixture: new …-04 lands in {state_checkout}/_debriefs/ on spacedock-state/dev; trunk main byte-identical before/after.

7d4

…inition dir

Resolve {debrief_root} from status --boot --json (entity_dir) in a new Phase-1
Step 1b and thread it through all four _debriefs/ touch-points. Split-root
commits become path-scoped state-checkout commit+push with the rebase-conflict
halt; an entity_dir_present==false halt-gate stops a declared-but-absent checkout
before any git -C {state_checkout} op. Single-root path unchanged.

Add a deterministic fixture regression guard (skills/integration) covering AC-1
(split-root: file in checkout, trunk HEAD byte-identical), AC-2 (single-root
negative control), AC-3 (numbering from checkout only -> -04, orphan untouched),
and AC-4 (absent-checkout halt). The guard drives the real boot record and
transcribes the skill's git sequence; it is a regression guard / live-drive seed,
not a substitute for the validation-stage live drive.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@clkao
clkao merged commit d97d30e into main Jun 30, 2026
4 checks passed
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