Skip to content

fix(settings): recover from a WebView2 crash and keep unsaved edits (#297) - #298

Merged
Maxaubert merged 4 commits into
mainfrom
fix/297-settings-crash-recovery
Sep 30, 2026
Merged

Maxaubert merged 4 commits into
mainfrom
fix/297-settings-crash-recovery

Conversation

@Maxaubert

Copy link
Copy Markdown
Owner

Closes #297.

When the Settings page's WebView2 process fails (seen with RTSS hooking a dxgi.dll that is being unloaded), the host recreates the environment and controller, at most 3 times a minute. Unapplied edits are mirrored to the host and handed back to the fresh page (restoreDraft). After the budget runs out, Settings shows one message and closes cleanly. Controller-creation failures are logged and recovered the same way.

Split out of the colour-filter branch (#288), where it was built and reviewed.

  • Pure decision logic in src/config_ui/webview_recover.h, unit-tested.
  • Playwright test: an unapplied edit survives a simulated recovery.
  • Verified on the owner's PC earlier (forced crash: page recovered, draft kept; give-up closed properly), and running in the owner's tested build since.
  • Version 0.13.2.

🤖 Generated with Claude Code

https://claude.ai/code/session_01KPUNAWcwghXdHCApcKKjSG

Maxaubert and others added 4 commits September 30, 2026 17:02
…bout to get back

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KPUNAWcwghXdHCApcKKjSG
…ges left alone, no onboarding re-entry, controller failure recovered

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KPUNAWcwghXdHCApcKKjSG
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KPUNAWcwghXdHCApcKKjSG
@Maxaubert
Maxaubert merged commit ec17f74 into main Sep 30, 2026
1 check passed
@Maxaubert
Maxaubert deleted the fix/297-settings-crash-recovery branch September 30, 2026 15:37
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.

Settings recovers from a WebView2 crash and keeps unsaved edits

1 participant