Skip to content

Commit 8e67cf3

Browse files
authored
chore(desktop): delete dead persona catalog UI cluster (#2886)
## Summary - Deletes 6 orphaned files in `desktop/src/features/agents/ui/` (556 lines) that formed a closed cluster with zero imports from the reachable module graph — orphaned by the 1B dialog consolidation (`PersonaCatalogSurface`, `PersonaCatalogSection`, `PersonaCatalogDetailsSheet`, `PersonaCatalogSelectionBadge`, `PersonaIdentity`, `PersonaLibraryEntryPoints`). - Removes a stale `check-file-sizes.mjs` override entry for the no-longer-existent `PersonaDialog.tsx`. Verified dead two ways: import-graph reachability walk from all entry points puts all six outside the reachable set, and `tsc --noEmit` passes clean after deletion. Their `data-testid`s have zero references outside the cluster. `PersonaCatalogDialog.tsx` is alive (`AgentsView` imports it) and stays. Independent of #1968 — pure dead code removal, no behavioral change.
1 parent 166c665 commit 8e67cf3

7 files changed

Lines changed: 0 additions & 570 deletions

desktop/scripts/check-file-sizes.mjs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -219,20 +219,6 @@ const overrides = new Map([
219219
// (+3) — the new team-id spawn link and the runtime-layered instructions
220220
// field.
221221
["src/shared/api/types.ts", 1047],
222-
// readiness-gate: PersonaDialog.tsx threads computeLocalModeGate +
223-
// requiredCredentialEnvKeys + RequiredFieldLabel so the "New agent" dialog
224-
// shows required markers and credential amber rows (parity with
225-
// CreateAgentDialog). +23 lines of gate wiring. Queued to split.
226-
// config-bridge-aware requirements: useRuntimeFileConfigQuery wiring adds
227-
// ~16 lines. Queued to split.
228-
// baked-env-required-badge: useBakedBuildEnvKeysQuery + bakedEnvKeys wiring
229-
// + correct exclusion-semantics for requiredEnvKeys adds ~14 lines.
230-
// +2 lines: filter managed provider key from requiredEnvKeys (suppress dead-input locked row).
231-
// global-agent-config parity: wire useGlobalAgentConfig into PersonaDialog
232-
// (Gap A: global-aware computeLocalModeGate + drop bare requiredCredentialEnvKeys;
233-
// Gap B: hasAutoOpenedAdvancedRef auto-expand effect) + effective-provider
234-
// save gate + Inherit/Select-a-provider label. Queued to split.
235-
["src/features/agents/ui/PersonaDialog.tsx", 1080],
236222
// harness-persona-sync feature growth, queued to split in the resolver-unify
237223
// refactor followup. discovery.rs is dominated by the new test module
238224
// (the effective_agent_command / divergent / create-time override matrix);

desktop/src/features/agents/ui/PersonaCatalogDetailsSheet.tsx

Lines changed: 0 additions & 163 deletions
This file was deleted.

desktop/src/features/agents/ui/PersonaCatalogSection.tsx

Lines changed: 0 additions & 166 deletions
This file was deleted.

desktop/src/features/agents/ui/PersonaCatalogSelectionBadge.tsx

Lines changed: 0 additions & 29 deletions
This file was deleted.

0 commit comments

Comments
 (0)