Skip to content

feat(implement-spec): implementers run as delegate_task worktree children - #30

Merged
leomontigatti merged 8 commits into
mainfrom
feat/implement-spec-delegate-task
Oct 26, 2026
Merged

leomontigatti merged 8 commits into
mainfrom
feat/implement-spec-delegate-task

Conversation

@leomontigatti

Copy link
Copy Markdown
Owner

Layer 3 of 3, on #29.

Every project that runs implement-spec: the implementer's checkout now gets its setup only from T3, so the project's t3.json must be imported in T3 (project settings → Actions → Import from t3.json). The brief no longer runs runOnWorktreeCreate by hand. Nothing to rename.

Each implementer is now a delegate_task child with isolation: "worktree" and mode: "async", replacing the background Agent-tool worktree that was set up by hand. T3 prepares its checkout, and the child's completion wakes the orchestrator. A delegate child can't be messaged, so a retry or an answered question starts a new child that merges the earlier branch first. A blocked child commits wip: first, so nothing is lost. The implementer's harness wrappers are removed, THREADS.md drops its exception, implementer-worktrees is resolved, and the new active entry delegate-task-worktrees records what this relies on. This follows the 2026-10-26 trial recorded in #29.

Review

Tier: readback+correctness, reviewed 1729be7

  • readback (Luna): 2 findings, fixed (record each attempt's branch and worktree; the trial's reference in delegate-task-worktrees)
  • correctness (Sol): 3 findings, fixed (a blocked child's uncommitted work; the ancestry check racing another ticket's merge; older retry attempts left behind); fix round no findings

🤖 Generated with Claude Code

leomontigatti and others added 7 commits October 26, 2026 09:07
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…and background-agent exceptions

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…dren

Each implementer is now a T3 delegate_task child with isolation: worktree and
mode: async, instead of a background Agent-tool worktree set up by hand. A
retry or an answered question is a new child carrying the earlier branch,
since a delegate child takes no SendMessage. The brief drops its setup step
and commits on the branch T3 gave it; the harness wrappers go, and THREADS.md
loses its exception. implementer-worktrees is resolved; delegate-task-worktrees
records what we now rely on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…try check, clean every attempt

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts:
#	skills/implement-spec/SKILL.md
#	t3-dependencies.md
@leomontigatti
leomontigatti merged commit 271afd4 into main Oct 26, 2026
3 checks passed
@leomontigatti
leomontigatti deleted the feat/implement-spec-delegate-task branch October 26, 2026 15:23
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