Skip to content

Centralize desktop renderer URL resolution#383

Merged
BunsDev merged 1 commit intomainfrom
okcode/fix-popout-browser-preview
Apr 9, 2026
Merged

Centralize desktop renderer URL resolution#383
BunsDev merged 1 commit intomainfrom
okcode/fix-popout-browser-preview

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 9, 2026

Summary

  • Centralize desktop renderer URL construction in a shared resolveDesktopRendererUrl helper.
  • Use the helper for both the main window and pop-out preview windows so dev and packaged behavior stay consistent.
  • Add unit coverage for packaged URLs, dev URLs, query param merging, and the missing-dev-server failure case.
  • Update the Electron routing comment to reflect the desktop shell origin.

Testing

  • bun fmt
  • bun lint
  • bun typecheck
  • Not run: bun run test

- Add a shared helper for dev and packaged renderer URLs
- Use it for main and pop-out windows to preserve query params
- Update the Electron history comment for the desktop shell origin
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 9, 2026 9:50pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 9, 2026 9:50pm

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 9, 2026
@BunsDev BunsDev merged commit af67e12 into main Apr 9, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant