fix(desktop): persist window size and restore macOS fullscreen exits cleanly

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597

[rushilrai]

What Changed

This PR makes the desktop app remember its last window state across relaunches.

It restores:

• normal window size and position
• maximized/fill-screen state
• windows closed from macOS true fullscreen back onto the current desktop without the white startup flash

The restore logic keeps normal restore bounds separate from the fullscreen-origin visible frame so standard maximize behaviour stays intact.

Why

Right now the desktop app always comes back at the default size, which is frustrating if you keep it in a specific layout.
This change makes relaunch behavior feel much closer to what users expect from native desktop apps.

UI Changes

Desktop window restore behaviour only.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

Note

Persist window size and restore macOS fullscreen state across sessions

• Adds windowState.ts with loadWindowState and attachWindowStatePersistence to save/restore window bounds and mode to userData/window-state.json.
• On launch, restores the previous position, size, and restore mode (normal, maximized, or fullscreen-origin); falls back to a centered default (1100×780) if the saved state is missing, invalid, or off-screen.
• Debounces resize/move persistence by 250ms; flushes immediately on maximize, fullscreen toggle, and close events.
• On macOS, windows that were fullscreen before close are shown without re-entering fullscreen, using restoreMode: 'fullscreen-origin' to track this case.
• Risk: first run or corrupted/off-screen state silently falls back to defaults with a console.warn.

^{Macroscope summarized 1a0662d.}

---

Note

Medium Risk
Changes Electron window creation/startup behavior and introduces new disk persistence logic; mistakes could cause bad restore geometry or startup regressions, though it falls back safely when state is invalid/off-screen.

Overview
The desktop app now persists window bounds and restore mode to userData/window-state.json and restores them on launch, instead of always using fixed default dimensions.

Adds windowState.ts with loadWindowState (validates/clamps bounds, ensures the window is sufficiently on-screen, and centers defaults when missing/invalid) and attachWindowStatePersistence (debounced move/resize persistence, immediate flush on close/maximize/fullscreen toggles, with a special fullscreen-origin mode to avoid macOS fullscreen re-entry/white flash). Includes a comprehensive vitest suite covering restore fallbacks, visibility checks, and persistence behavior.

^{Written by Cursor Bugbot for commit 1a0662d. This will update automatically on new commits. Configure here.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2a5c22f3-9540-4c3b-8ddd-170b7bb6c8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[juliusmarminge]

Closing because this window-state persistence implementation targets the pre-Effect desktop architecture and most touched files are gone.

This was closed as part of a large repo-wide maintenance sweep. If you think it's still relevant, please reopen.

[rushilrai]

Thanks for the review.
Still think this would be a nice addition, so will be opening a new PR using the current Effect implementation as base.
Will link it here as well.

[rushilrai]

Superseded by the Effect based implementation at feat (desktop): persist window size and restore macOS fullscreen exits cleanly
#3597