Skip to content

improvements to Deep Review UX, desktop devtools, and session management fixes.#537

Merged
limityan merged 4 commits intoGCWing:mainfrom
limityan:yanzhn/fix
Apr 26, 2026
Merged

improvements to Deep Review UX, desktop devtools, and session management fixes.#537
limityan merged 4 commits intoGCWing:mainfrom
limityan:yanzhn/fix

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

This PR includes improvements to Deep Review UX, desktop devtools, and session management fixes.

Changes

Deep Review Improvements

  • Task card/panel display: Show reviewer role name, description, and responsibilities instead of raw prompts for built-in review-team subagents
  • Action bar UX: Group remediation items by priority (must_fix/should_improve/needs_decision/verification) with per-group select-all toggles and colored headers
  • Dynamic layout: Use ResizeObserver to prevent content overlap

Desktop Devtools

  • Add devtools Cargo feature enabling:
    • Native webview DevTools (Ctrl/Cmd+Shift+J)
    • Interactive element inspector (Ctrl/Cmd+Shift+I) with hover highlight, tooltip, and computed styles/CSS variables/box model capture
  • Feature auto-enabled in dev and release-fast builds; disabled in production release builds

Session Management Fixes

  • Mark interrupted sessions as unread on app close and abnormal exit detection on restore
  • Sort sessions by lastActiveAt to auto-promote when user sends a message

…essage

Include lastActiveAt in session sort priority so that sending a message
moves the session to the top of the workspace session list.

Priority: lastActiveAt > lastFinishedAt > createdAt
Add a `devtools` Cargo feature to the desktop app that enables:
- Native webview DevTools (Ctrl/Cmd+Shift+J)
- Interactive element inspector (Ctrl/Cmd+Shift+I) with hover highlight,
  tooltip, and click-to-capture for computed styles, CSS variables,
  box model, colors, and attributes

The feature is automatically enabled in dev builds and release-fast
profile builds, but never in release profile builds intended for end
users. All debug code is conditionally compiled via `#[cfg]` to ensure
zero overhead in production.

New files:
- src/apps/desktop/src/api/debug_api.rs
- src/web-ui/src/infrastructure/debug/mainWindowInspector.ts
- src/web-ui/src/infrastructure/debug/useDebugInspector.ts

Updated:
- Cargo.toml, package.json, theme.rs, lib.rs, App.tsx, AGENTS.md
…ynamic layout

- Group remediation items by priority (must_fix/should_improve/needs_decision/verification)
- Add per-group select-all toggle with colored headers
- Add hint text explaining read-only default behavior
- Increase overall size for better high-DPI visibility
- Use ResizeObserver to dynamically adjust body padding and prevent content overlap
- Add ungrouped locale key for fallback items

Generated with BitFun

Co-Authored-By: BitFun
…ards and detail panel

- In TaskToolDisplay, show reviewer role name, description, and responsibilities
  instead of the raw prompt for built-in review-team subagents.
- In TaskDetailPanel, add a reviewer context section with role info, falling back
  to the raw prompt only for non-reviewer tasks.
- Add ReviewerContext interface and getReviewerContextBySubagentId() helper in
  reviewTeamService (already committed in previous change).
- Add locale keys: reviewerContextLabel (en/zh-CN/zh-TW).
- Add corresponding SCSS styles for both card and panel views.

Generated with BitFun

Co-Authored-By: BitFun
@limityan limityan merged commit 0ea967e into GCWing:main Apr 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant