fix(desktop): streamline task controls and toolbar / 精简任务控制与顶栏 - #8392
Merged
SivanCola merged 1 commit intoAug 11, 2026
Merged
Conversation
Problem: The task monitor exposed duplicate stop and cancel actions with a cramped confirmation flow, while the topic toolbar repeated command and shortcut entry points and placed the workspace control inconsistently. Root cause: The desktop UI surfaced backend control aliases as separate primary actions and accumulated utility controls without a clear priority order or complete localization. Fix: Merge active-task termination into a single accessible stop flow, localize task state and session-summary labels, remove redundant toolbar entry points, and place session summary before the workspace toggle. Keep the underlying cancel APIs and keyboard/settings access paths intact. Verification: - pnpm exec tsx src/components/TaskMonitorPanel.test.tsx - pnpm exec tsx src/__tests__/app-chrome-tabs.test.ts - pnpm check:css - pnpm typecheck - Browser verification at 127.0.0.1:5174
SivanCola
force-pushed
the
fix/desktop-task-monitor-toolbar
branch
from
August 11, 2026 10:33
90f83ee to
f57e1bc
Compare
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.
Summary
Issues
Verification
pnpm buildpnpm testpnpm test:task-monitor(22 panel tests + 3 navigation tests)pnpm exec tsx src/__tests__/app-chrome-tabs.test.ts(128 tests)pnpm exec tsx src/__tests__/topicbar-controls.test.ts(2 contracts)go run ./tools/repolintDocumentation impact
Documentation-impact: none - this is a self-explanatory desktop UI simplification with localized labels and no public API or configuration change.
Cache impact
Cache-impact: none - frontend presentation and interaction only; no cache keys, providers, model catalogs, or request payload schemas change.
Cache-guard: provider-visible prompts and tool schemas are untouched; full frontend build, bundle budgets, and test suite pass.
System-prompt-review: N/A