Skip to content

Improve Deep Review completion and companion routing#707

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/deep-review-completion-expands
May 13, 2026
Merged

Improve Deep Review completion and companion routing#707
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/deep-review-completion-expands

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

@limityan limityan commented May 13, 2026

Summary

  • Keep the Deep Review floating action bar expanded when a resumed review completes successfully.
  • Route Agent Companion review-session notifications through the parent session and aux pane so Deep Review opens in the expected side panel.
  • Reuse an existing Deep Review aux tab without forcing the right panel width back to the default.

Verification

  • git diff --check gcwing/main..HEAD
  • pnpm --dir src/web-ui exec vitest run src/flow_chat/services/openBtwSession.test.ts src/app/services/openAgentCompanionSession.test.ts src/flow_chat/components/btw/BtwSessionPanel.review-action.test.tsx
  • pnpm run lint:web
  • pnpm run type-check:web

Expand the Deep Review action bar when a resumed review completes successfully.

Route Agent Companion review-session notifications through the parent session and aux pane so Deep Review opens in the expected side panel. Reuse an existing aux pane tab without forcing the right panel width back to the default.
@limityan limityan force-pushed the yanzhn/deep-review-completion-expands branch from 3c2e972 to d2ba4f8 Compare May 13, 2026 10:21
@limityan limityan changed the title Expand Deep Review completion action bar after resume Improve Deep Review completion and companion routing May 13, 2026
@limityan limityan force-pushed the yanzhn/deep-review-completion-expands branch 2 times, most recently from 85f1261 to 968bf95 Compare May 13, 2026 10:29
@limityan limityan merged commit 1e002ac into GCWing:main May 13, 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