The harness now pushes the phase commit right after committing it
(PHASE_PUSH=1 default, new push_phase in lib.sh): upstream when set,
else 'git push -u <first remote> <branch>'; no remote = skip with a
notice. A push failure keeps the commit local, prints the same loud
ERROR contract as a commit failure, and stops the run — the next
phase's push sweeps the unpushed commit in. PHASE_PUSH=0 opts out with
a loud notice. SKILL.md (Commits + config table + exit codes) and the
executor prompts document the new behavior.
Children never commit (their prompts forbid git add/commit, overriding
project instructions) so the commit is deterministic. At the phase commit
point (00_phase.md final pass) the harness makes ONE atomic commit: the
phase's code changes, the todo→complete file move, and the executor
reports, together.
- Scoped staging: a worktree snapshot taken at the phase's first unit
(phase-sessions/dirty-<phase>, always kept) is subtracted, so the
owner's pre-existing uncommitted work is left alone; runtime artifacts
are never staged. The commit prints exactly what went in.
- Verified: the moved phase file (and report, unless gitignored) must be
in the index before committing.
- Loud on failure: a commit failure prints the git error + hand-fix and
stops the run; the phase stays complete, the miss is never swept into a
later phase.
- PHASE_COMMIT_SUBJECT (default 'phase: <phase>'); the executor's final
report becomes the commit body; --no-gpg-sign always passed.
- PHASE_COMMIT defaults to 1: completed phases are committed by default.
Explicit PHASE_COMMIT=0 opts out with a loud warning that the phase is
complete but uncommitted — a completed phase is never left uncommitted
silently (that gap let four phases pile up uncommitted in
brain_of_reese).
Standardize on the .agents/ directory across all phased-execution
skills (phase state, reports, sessions, validate.sh, PLAN.md, and
per-story/feature/workflow trees). Legacy dot-less agent/ fallbacks
in migration scripts are untouched.
- A phase is now a directory: 00_phase.md (overview + task index) plus
small, quick NN_task.md task files; complete/ mirrors todo/
- Task is the unit of execution: run-task.sh (one unit), run-phase.sh
(phase to completion incl. 00_phase.md final pass), auto-phase.sh
(all units in order); validate.sh gate runs after every task
- PHASE_COMMIT commits at phase boundaries (00_phase.md / legacy file moves)
- Legacy flat todo/NN_name.md files still execute as a single unit;
new migrate-phases-to-tasks.sh converts them to the directory layout
- phase-status.sh reports per-task state and the next unit
- New task-template.md; phase templates now carry a Tasks index
- retries now resume the failed executor's session (pre/post session
tracking replaces the broken mtime-vs-ref check)
- report recovery from the session file when the JSON stream loses its
tail (child signaled mid-flush)
- progress.mjs: thinking indicators (◐ thinking… / ◑ thought for Ns),
compaction (⧉ …) and provider auto-retry lines; trims trailing
whitespace so no blank lines after LLM text; exits 1 on truncated
stream, model error, or missing final text (pi --mode json always
exits 0 even on errors)
- explicit ✗ ERROR lines + exit codes: 0 ok, 1 phase failed, 130/143
interrupted (INT/TERM traps; post-pipeline check covers the case
where bash suppresses the INT trap after a job dies from SIGINT)
- PIPESTATUS captured in a single statement (any following command
resets it)
- skills dir: .gitignore README.md so pi's skill scanner (which honors
.gitignore) stops warning 'description is required'
ducoterra