chore(orchestration): remove the recipe-director layer - #36
Merged
Conversation
Removes the experimental recipe-Orcastrator layer (PRs #9/#10/#11) while leaving the Smart/LLM director, the F1–F4 orchestration bridge, and the Control Panel / Mission Control fully intact and working. Deleted (recipe-specific files + tests): - lib/recipe-director-launch.ts - lib/recipe-director-recipes.ts - lib/director-backend.ts (the DirectorBackend abstraction) - components/director/DirectorTypePicker.tsx Reverted recipe wiring in shared files (recipe parts only): - OrchestratorLaunchModal.tsx — the "New Orcastrator" flow launches the Smart/LLM director directly via launchOrchestratorForProject again; no director-type picker, no recipe option. - rpc/methods/orchestration.ts + shared/orchestration-binding.ts — dropped the recipe-only `targetWorktree` taskCreate param; taskCreate again stamps target_key from the caller terminal's worktree. The bridge/run/taskCreate plumbing the Smart director + Control Panel use is unchanged. - i18n/locales/*.json — removed the DirectorTypePicker keys (all locales). Kept (shared infra the Smart director depends on): - lib/director-worktree-shell.ts — extracted out of orchestrator-launch.ts by #9 and still used by the surviving Smart-director launch path. - orchestrator-launch.ts (launchOrchestratorForProject), the F1–F4 bridge (per-run isolation, coordinator worktrees, preload run/taskCreate), the Mission Control live activity line, and the shipped-scope + focus-steal fixes. - The OrchestratorsSidebarSection "+" styling/alignment fixes (#33) — not recipe code; the "+" already opens the launch modal directly. Recoverability: the pre-removal state is parked on origin/parked/recipe-director, and this PR can be reverted to restore recipes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
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 this does
Removes the experimental recipe-Orcastrator layer (added by PRs #9/#10/#11 + a layout fix) as a forward removal — a normal PR that deletes the recipe code. The Smart/LLM director, the F1–F4 orchestration bridge, and the Control Panel / Mission Control remain fully intact and working.
Removed (recipe-specific files + their tests)
src/renderer/src/lib/recipe-director-launch.tssrc/renderer/src/lib/recipe-director-recipes.tssrc/renderer/src/lib/director-backend.ts(theDirectorBackendabstraction)src/renderer/src/components/director/DirectorTypePicker.tsxReverted recipe wiring in shared files (recipe parts only)
OrchestratorLaunchModal.tsx— the "New Orcastrator" flow launches the Smart/LLM director directly vialaunchOrchestratorForProjectagain; no director-type picker, no recipe option.rpc/methods/orchestration.ts+shared/orchestration-binding.ts— dropped the recipe-onlytargetWorktreetaskCreateparam;taskCreateagain stampstarget_keyfrom the caller terminal's worktree. The bridge/run/taskCreate plumbing the Smart director + Control Panel use is unchanged.i18n/locales/*.json— removed theDirectorTypePickerkeys across all locales (en/es/ja/ko/zh).Kept — shared infra the Smart director depends on
lib/director-worktree-shell.ts— extracted out oforchestrator-launch.tsby RecipeDirectorBackend: compile recipe -> task DAG -> orchestration.run, no LLM in worktree #9 and still used by the surviving Smart-director launch path, so it stays (not inlined back).orchestrator-launch.ts(launchOrchestratorForProject), the F1–F4 bridge (per-run isolation, coordinator worktrees, preload run/taskCreate), the Mission Control live activity line, and the shipped-scope + focus-steal fixes.OrchestratorsSidebarSection"+" styling/alignment fixes (fix(orchestration): match Orcastrators "+" weight to the Projects "+" #33) — not recipe code; the "+" already opens the launch modal directly.Verification
rgforrecipe-director|RecipeDirector|DirectorBackend|DirectorTypePicker|getRecipes|director-backendoversrc/→ zero hits.oxlint✓, localization catalog + coverage ✓, electron-vite (build:desktop) ✓.Recoverability
origin/parked/recipe-director.🤖 Generated with Claude Code