feat(orchestration): starter recipes + getRecipes registry (#10) - #24
Merged
Conversation
Add two built-in recipes to the token-free recipe director, alongside #9's implement_then_review: - single_worker_pr: one task, no track/deps → its own worktree → one PR. - repro_fix_verify: three tasks on one shared track, chained repro→fix→verify via dependsOn so the deps form a total order on the track (satisfying the coordinator's same-track ordering guard). Each spec instructs the worker to commit its artifact so the next task in the shared worktree sees it. Add getRecipes(): Recipe[] returning all three built-ins in display order (fresh array so callers can't mutate the registry) for the #11 picker to list. Tests cover both new recipes compiling to the right tasks/tracks/deps, the repro_fix_verify deps forming a strict total order, and getRecipes returning all three. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
9 tasks
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.
Closes #10.
Adds two built-in recipes to the token-free recipe director (alongside #9's
implement_then_review) and a registry the #11 picker can list. Edits onlyrecipe-director-recipes.ts+ its test — no picker/launch/UI changes (#11 owns those and only readsgetRecipes()).What's added
single_worker_pr— one task (deliver), no track/deps. Its track defaults to the task key → its own worktree/branch → exactly one PR. A generic "do the whole job and open a PR" spec.repro_fix_verify— three tasks on one shared track (repro-fix-verify), chainedrepro → fix → verifyviadependsOn. The chain is a total order on the track, which satisfies the coordinator's same-track ordering guard (it refuses same-track tasks not totally ordered by deps, since they'd race one checkout). Each spec instructs the worker to commit its artifact (the failing test, the fix) so the next task in the shared worktree sees it.getRecipes(): Recipe[]— returns all three built-ins in display order, as a fresh array so callers can't mutate the registry.Tests
single_worker_pr= 1 task, own track;repro_fix_verify= 3 same-track, dep-chained).repro_fix_verify's deps form a strict total order on its track.getRecipes()returns all three; exposes{ name, description }shape; returns a fresh (non-mutable) array.Verification
vitest (13 pass), typecheck, oxlint, and electron-vite build all green.
Merge note
#11's picker depends on
getRecipes(), so this should merge first. Do not merge yet.🤖 Generated with Claude Code