Skip to content

fix(review-ui): set default window size#2

Merged
lynnswap merged 1 commit into
mainfrom
codex/default-window-size
May 21, 2026
Merged

fix(review-ui): set default window size#2
lynnswap merged 1 commit into
mainfrom
codex/default-window-size

Conversation

@lynnswap
Copy link
Copy Markdown
Owner

Summary

  • Set ReviewMonitor's fallback main window content size to 600 x 400 when no saved frame exists.
  • Keep AppKit frame autosave restoration intact by applying the autosave name after the fallback size is configured.
  • Add a shell test that uses an isolated frame autosave name to verify the no-saved-frame default size path.

Testing

  • swift test --build-system swiftbuild --no-parallel
  • xcodebuild test -project Tools/ReviewMonitor/ReviewMonitor.xcodeproj -scheme ReviewMonitor -destination 'platform=macOS,arch=arm64' CODE_SIGNING_ALLOWED=NO CODE_SIGNING_REQUIRED=NO

@lynnswap lynnswap marked this pull request as ready for review May 21, 2026 02:53
@lynnswap lynnswap merged commit 4d8e915 into main May 21, 2026
1 check passed
@lynnswap lynnswap deleted the codex/default-window-size branch May 21, 2026 03:03
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