fix(orchestration): match Orcastrators "+" weight to the Projects "+" - #33
Merged
Merged
Conversation
The New-Orcastrator "+" was a raw <button> at 40% foreground opacity with
strokeWidth={2}, rendering noticeably fainter than the Projects "+". Swap it
for the same shadcn <Button variant="ghost" size="icon-xs"> the Projects "+"
uses, so it inherits full-strength sidebar foreground and matches stroke weight.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
zaridan
added a commit
that referenced
this pull request
Jun 24, 2026
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
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.
Problem
The ORCASTRATORS sidebar section's New-Orcastrator "+" rendered noticeably fainter than the Projects "+". The Orcastrators "+" was a raw
<button>styled at 40% foreground opacity (text-worktree-sidebar-foreground/40) withstrokeWidth={2}, while the Projects "+" inSidebarHeader.tsxis a full-strength shadcnButton.Fix
In
OrchestratorsSidebarSection.tsx, replace the raw faint<button>with the same shadcnButtonmarkup the Projects "+" uses:It now inherits the full-strength sidebar foreground (no
/40opacity) and matches the Projects "+" stroke weight (2.25). ExistingonClickandaria-labelare unchanged;Buttonis imported from@/components/ui/button.Scope
SidebarHeadertouched.aria-labelpreserved.Checks
src/relay/subprocess.test.ts, is a pre-existing flaky timing test — passes in isolation; unrelated to this UI change)🤖 Generated with Claude Code