From 423ab7f2a9e9317bb33fdd221607b7926f99e962 Mon Sep 17 00:00:00 2001 From: mika kuns Date: Thu, 6 Aug 2026 11:09:32 +0200 Subject: [PATCH] chore(ui): remove dead AgentStripView fragment AgentStripView was superseded by WorkConsole during the task-detail redesign but never wired into any view and never deleted. Drops the unused Border.agent-strip / diff-meter styles and the 14 agent.* locale keys that only it referenced, and marks A4 in the feature-unification spec as resolved by deletion. --- .../2026-06-19-feature-unification-design.md | 2 +- src/ClaudeDo.Localization/locales/de.json | 16 +- src/ClaudeDo.Localization/locales/en.json | 16 +- src/ClaudeDo.Ui/CLAUDE.md | 2 +- src/ClaudeDo.Ui/Design/IslandStyles.axaml | 57 ------ .../Views/Islands/AgentStripView.axaml | 167 ------------------ .../Views/Islands/AgentStripView.axaml.cs | 19 -- 7 files changed, 4 insertions(+), 275 deletions(-) delete mode 100644 src/ClaudeDo.Ui/Views/Islands/AgentStripView.axaml delete mode 100644 src/ClaudeDo.Ui/Views/Islands/AgentStripView.axaml.cs diff --git a/docs/superpowers/specs/2026-06-19-feature-unification-design.md b/docs/superpowers/specs/2026-06-19-feature-unification-design.md index 11bcce98..b78d36b4 100644 --- a/docs/superpowers/specs/2026-06-19-feature-unification-design.md +++ b/docs/superpowers/specs/2026-06-19-feature-unification-design.md @@ -35,7 +35,7 @@ the engine. | A1 | Diff viewing | `DiffModalViewModel` (worktree + commit-range), `WorktreeModalViewModel` (file-tree + per-file), `PlanningDiffViewModel` (per-subtask + integration) | `UnifiedDiffParser`, `DiffLinesView` (good) | | A2 | Agent-config editing | `ListSettingsModalViewModel` (list scope), `AgentSettingsSectionViewModel` (task scope); global lives in `SettingsModalViewModel` | `InheritanceResolver`, `InheritedBadge` (good) | | A3 | Worktree actions | `WorktreesOverviewModalViewModel` per-row cmds (Merge/Discard/Keep/ForceRemove/ShowDiff/Jump) vs `MergeSectionViewModel` (Merge/OpenDiff) | same `IWorkerClient` calls | -| A4 | Merge display | `AgentStripView` re-displays `MergeSectionViewModel` state | — | +| A4 | Merge display | ~~`AgentStripView` re-displays `MergeSectionViewModel` state~~ — resolved 2026-08-06: `AgentStripView` was dead (never bound after the task-detail redesign); deleted instead of unified | — | ### Bucket B — entry-point sprawl (one backend, many hand-wired doors) diff --git a/src/ClaudeDo.Localization/locales/de.json b/src/ClaudeDo.Localization/locales/de.json index 20ad9248..6122473e 100644 --- a/src/ClaudeDo.Localization/locales/de.json +++ b/src/ClaudeDo.Localization/locales/de.json @@ -250,21 +250,7 @@ } }, "agent": { - "stopTip": "Agent stoppen", - "sendToQueue": "In Warteschlange einreihen", - "sendToQueueTip": "Diese Aufgabe einreihen, damit der Worker sie übernimmt", - "removeFromQueue": "Aus Warteschlange entfernen", - "removeFromQueueTip": "Diese Aufgabe wieder aus der Warteschlange nehmen", - "worktreeLabel": "WORKTREE", - "copyPathTip": "Pfad kopieren", - "diffLabel": "DIFF", - "openDiff": "Diff öffnen", - "worktreeBtn": "Worktree", - "openWorktreeTip": "Worktree im Datei-Explorer öffnen", - "continue": "Fortsetzen", - "continueTip": "Die letzte Sitzung fortsetzen und weitermachen", - "resetAndRetry": "Zurücksetzen & erneut versuchen", - "resetAndRetryTip": "Den Worktree verwerfen und die Aufgabe erneut einreihen, um von vorn zu beginnen" + "openWorktreeTip": "Worktree im Datei-Explorer öffnen" }, "notes": { "today": "Heute", diff --git a/src/ClaudeDo.Localization/locales/en.json b/src/ClaudeDo.Localization/locales/en.json index 5cc91a0d..d895de0c 100644 --- a/src/ClaudeDo.Localization/locales/en.json +++ b/src/ClaudeDo.Localization/locales/en.json @@ -250,21 +250,7 @@ } }, "agent": { - "stopTip": "Stop agent", - "sendToQueue": "Send to queue", - "sendToQueueTip": "Queue this task for the worker to pick up", - "removeFromQueue": "Remove from queue", - "removeFromQueueTip": "Take this task back out of the queue", - "worktreeLabel": "WORKTREE", - "copyPathTip": "Copy path", - "diffLabel": "DIFF", - "openDiff": "Open diff", - "worktreeBtn": "Worktree", - "openWorktreeTip": "Open worktree in file explorer", - "continue": "Continue", - "continueTip": "Resume the last session and keep going", - "resetAndRetry": "Reset & retry", - "resetAndRetryTip": "Discard the worktree and re-queue the task to run from scratch" + "openWorktreeTip": "Open worktree in file explorer" }, "notes": { "today": "Today", diff --git a/src/ClaudeDo.Ui/CLAUDE.md b/src/ClaudeDo.Ui/CLAUDE.md index 8213da06..ad14e0d6 100644 --- a/src/ClaudeDo.Ui/CLAUDE.md +++ b/src/ClaudeDo.Ui/CLAUDE.md @@ -30,7 +30,7 @@ ViewModels/ WorkerConnection, WorktreesOverview, UnifiedDiffParser Conflicts/ — ConflictResolverViewModel + ConflictModels Views/ — mirrors the VM layout; Islands/Detail/ holds TaskHeaderBar, - DescriptionStepsCard, WorkConsole; plus AgentStripView, SessionTerminalView + DescriptionStepsCard, WorkConsole; plus SessionTerminalView Views/Controls/ — MarkdownView, ModalShell, ThemedDatePicker, DiffLinesView, InheritedBadge, AgentConfigEditor Design/ — Tokens.axaml (design tokens; merged before styles) diff --git a/src/ClaudeDo.Ui/Design/IslandStyles.axaml b/src/ClaudeDo.Ui/Design/IslandStyles.axaml index da7c8242..fbcd8e8d 100644 --- a/src/ClaudeDo.Ui/Design/IslandStyles.axaml +++ b/src/ClaudeDo.Ui/Design/IslandStyles.axaml @@ -8,7 +8,6 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -