Repository navigation
Conversation
There was a problem hiding this comment.
The change works as described. I ran the whole review.spec.ts suite against the mocked server: all 22 tests pass, including the new refresh test. Before/after screenshots of the review pane (one Git change) are below. Type checks for gui-extensions and session-ui pass. The four typecheck:e2e errors in app are all in timeline-stream.spec.ts, which this change doesn't touch.
The refresh queries are worked out from the session open at click time, and the spinner state is stored per session. Refreshing the last-turn query while Git mode is shown only marks it out of date; it doesn't fetch it. The new SessionReviewV2 props are optional, so other callers are unaffected.
Public API unchanged: the only additions are internal UI component props and one English i18n key.
No concerns.
Related: #52941
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |


Adds a Refresh button to the review pane toolbar, after the previous and next file buttons. It uses the browser pane's reload glyph, and shows the shared
Loaderwhile it is busy.SessionReviewV2takes optionalonRefresh,refreshing, andrefreshDisabledprops. WithoutonRefresh, the toolbar is unchanged.IconButtonghost/small, the samesession-review-v2-file-nav-buttonbox (36×28), and no gap.rotate-clockwiseicon: theoutline-rotate-clockwiseglyph redrawn on the 20-unit grid, likearrow-leftandoutline-arrow-left. At 16px it draws a 0.8px stroke with a square cap and a miter join, centered, the same as the arrows.refresh(),refreshing(), andcanRefresh():refresh()invalidates all three diff queries (Git/branch, last turn, session details) at once, without the debounce.refreshis renamedrefreshSoon. Its behavior does not change.ui.sessionReviewV2.refreshkey (English only).review.spec.tscase: change the mocked diff with no filesystem event, click Refresh, checkaria-busywhile the request is held, then check that the new file shows.