Skip to content

fix(desktop): harden Windows WebView startup#3830

Merged
SivanCola merged 1 commit into
esengine:main-v2from
SivanCola:codex/windows-webview2-blank-window
Jun 10, 2026
Merged

fix(desktop): harden Windows WebView startup#3830
SivanCola merged 1 commit into
esengine:main-v2from
SivanCola:codex/windows-webview2-blank-window

Conversation

@SivanCola

Copy link
Copy Markdown
Collaborator

Summary

  • embed the WebView2 bootstrapper in Windows desktop installer builds instead of relying on the default strategy
  • add a Windows WebView2 GPU fallback, enabled by default for canary builds and overrideable with REASONIX_DESKTOP_DISABLE_WEBVIEW2_GPU
  • install frontend crash handlers before early Wails runtime/theme calls so startup failures render an actionable overlay
  • document the Windows fallback and add regression coverage for the GPU toggle

Refs #3820

Tests

  • cd desktop && go test ./...
  • cd desktop/frontend && pnpm typecheck
  • bash -n scripts/desktop-build.sh
  • cd desktop/frontend && pnpm build

@SivanCola SivanCola requested a review from esengine as a code owner June 10, 2026 09:42
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) updater Auto-update / installer / release packaging labels Jun 10, 2026
@SivanCola SivanCola enabled auto-merge June 10, 2026 09:45

@esengine esengine left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean, well-scoped, and the test matrix on the GPU toggle is a nice touch. -webview2 embed is the right fix for the missing-runtime blank window, and installing the crash handlers before the early theme/runtime calls means we actually get a diagnosable overlay instead of a featureless webview. CI is green across all three platforms. Merging.

@SivanCola SivanCola merged commit b318985 into esengine:main-v2 Jun 10, 2026
14 checks passed
@coderpang

Copy link
Copy Markdown
image It still dosen't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) updater Auto-update / installer / release packaging v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants