Deep Review UX Improvements & Session Stability Fixes#547
Merged
limityan merged 5 commits intoGCWing:mainfrom Apr 26, 2026
Merged
Deep Review UX Improvements & Session Stability Fixes#547limityan merged 5 commits intoGCWing:mainfrom
limityan merged 5 commits intoGCWing:mainfrom
Conversation
limityan
force-pushed
the
yanzhn/session
branch
3 times, most recently
from
cb297c1 to
f7536d7
Compare
Add confirmation before replacing chat input, fix overlay/scroll issues, remove minimize button and hint text, add i18n
The buildSessionMetadata function used ?? to fall back to existingMetadata for unreadCompletion and needsUserAttention. When the store cleared these fields (setting them to undefined), undefined ?? existingMetadata.unreadCompletion would restore the old persisted value, preventing the clear from reaching disk. On next app launch, initializeFromDisk would restore the stale value, causing green dots to reappear on old sessions. Now the in-memory session value is used directly as the source of truth, so clears propagate correctly through JSON.stringify (which omits undefined properties) to the backend. Generated with BitFun Co-Authored-By: BitFun
…rectives to reviewer prompts
limityan
force-pushed
the
yanzhn/session
branch
from
f7536d7 to
e0b7d79
Compare
limityan
changed the title
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
1 participant
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.
Deep Review UX Improvements & Session Stability Fixes
Summary
Improves Deep Review UX and prompt efficiency, fixes session list stability bugs (unread indicators, ordering).
Changes
perf(deep-review): add efficiency rules and role-specific strategy directives to reviewer prompts (
e0b7d79)fix: internationalize reviewer role description in deep review subagent pages (
b4d389c)fix: prevent unread session dots from reappearing after restart (
f68866a)buildSessionMetadataused??fallback that restored stale persisted values after clear; now uses in-memory value as source of truthunread_completionfrom persisted runtime state during restorefix(web-ui): improve deep review action bar UX (
d155dbd)fix(web-ui): keep session ordering stable (
b01b71d)lastActiveAtfrom sort key; useslastFinishedAt/createdAtto prevent sessions from jumping on click