feat(rig-epic,rig-task): front-loaded spec review, advisor gate, squash child-merge - #29
Merged
Merged
Conversation
…thers-loop learnings) Port two learnings from the paired Smithers durable workflow back into the source-of-truth skills: - rig-epic `run`: **front-loaded spec review** — review ALL children's specs up front (architect+qa over every child vs the integration branch), resolve blockers once, then run children `--spec-cleared` so none re-pauses mid-run. A child that pauses on a spec question stalls the whole epic (fatal under parallel/durable execution); front-loading makes it the single spec decision. - rig-epic `run --advisor`: **unattended mode** — decide that front-loaded gate with a delegated architect advisor pass (proceed + synthesized per-child direction, or halt with a blocked report) instead of a human, so kicked-off epics run hands-off to child PRs. The Smithers workflow implements this with a cheap model (the "Fable advisor"). - rig-task `--spec-cleared`: skip Step 2's blocking pause (keep the notes, don't gate) when the caller already front-loaded the spec — the enabler for the above. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ENct54EpFjBkVMecSGh4z
Child auto-merge said `gh pr merge --rebase --auto`, but many repos disallow rebase merges (and a repo with no required checks can't arm `--auto` at all). Switch to `--squash` for child PRs into the integration branch — into a stacked branch or the trunk — and merge directly when there are no checks for `--auto` to wait on. Matches the paired Smithers workflow. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ENct54EpFjBkVMecSGh4z
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.
Backports the skill learnings from the Smithers durable-workflow experiment (umbrella PR #28) onto
main— skills-only, nosmithers/files.run: front-loaded spec review — review ALL children's specs up front, resolve blockers once, run children--spec-clearedso none re-pauses mid-run (a paused child stalls the whole epic — fatal under parallel/durable execution).run --advisor— decide that gate with a delegated architect advisor pass (proceed + per-child direction, or halt) instead of a human, so kicked-off epics run hands-off to child PRs.--spec-cleared— the enabler: run spec review for notes but skip the blocking pause when the caller front-loaded it.Prompt-only, additive. Validated live: drove a full multi-epic slice to completion unattended.
🤖 Generated with Claude Code
https://claude.ai/code/session_015ENct54EpFjBkVMecSGh4z