docs(improve): headless-CLI executor variant for execute - #22
Open
funkiirabu wants to merge 1 commit into
Open
Conversation
execute currently requires a host that can spawn worktree-isolated subagents; hosts that can't are told to hand the plan over for manual execution. But the dispatch contract (inlined plan + preamble + report format) works just as well through any headless coding CLI run in a worktree the advisor creates itself. Documents that variant in closing-the-loop.md (worktree creation, prompt via temp file, claude -p / codex exec / t2code exec as examples, stateless REVISE loop) and softens the SKILL.md requirement to match. Review protocol, verdicts, and the never-merge rule are unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
execute <plan>currently requires a host agent that can spawn subagents with worktree isolation — hosts that can't are told to hand the plan over for manual execution. This PR documents a middle path that keeps the advisor/executor split working on those hosts: dispatch any headless coding CLI (e.g.claude -p,codex exec,t2code exec) in a worktree the advisor creates itself.Changes
references/closing-the-loop.md— new "Dispatch variant — headless coding CLI" subsection after the report format: worktree creation, dispatch prompt via temp file (same inlined plan + preamble + report format as the subagent path), capturing stdout as the executor report, and how REVISE works when the executor is stateless across invocations.SKILL.md— theexecute <plan>bullet now names both executor shapes instead of hard-requiring subagent support; manual handover remains the final fallback.Why
The plan format is already executor-agnostic — it's the skill's biggest strength. This makes the execute flow match: the advisor plans on an expensive model, a cheap CLI executes, and the review protocol (untrusted diff, criteria re-runs, max 2 revision rounds, never merging) applies unchanged. No behavior changes for hosts that can spawn subagents.
Made with Cursor