Skip to content

Document development-server and mounting caveats - #667

Merged
tricknotes merged 1 commit into
mainfrom
document-vite-notes
Sep 4, 2026
Merged

Document development-server and mounting caveats#667
tricknotes merged 1 commit into
mainfrom
document-vite-notes

Conversation

@tricknotes

@tricknotes tricknotes commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

Four short notes, verified against a real setup:

  • Rewrite coverage (README, Vite section): the index.html rewriting covers every tag — Vite externalizes even the document's inline bootstrapping script
  • CORS (README + UPGRADING, container note): Vite's default CORS allows localhost origins only; pages opened on a LAN IP or a custom domain need server.cors configured in vite.config.*
  • Non-HTML requests (README, routing options): requests without Accept: text/htmlcurl's default, most health checkers — skip the mount_ember_app route and receive the raw index.html from the asset handler
  • Layout (README, render_ember_app): point at the existing "disable the layout" note from the first example, where it is easy to miss

Documentation only, no behavior changes.

Verified against a real setup:

* The index.html rewriting covers every tag —
  Vite externalizes even the document's inline bootstrapping script
* Vite's default CORS allows localhost origins only,
  so pages on a LAN IP or a custom domain need server.cors configured
* Requests without Accept: text/html skip the mount_ember_app route
  and receive the raw index.html from the asset handler
* render_ember_app emits a complete HTML document —
  point at the existing layout note from the first example

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@tricknotes
tricknotes merged commit 99228fe into main Sep 4, 2026
12 checks passed
@tricknotes
tricknotes deleted the document-vite-notes branch September 4, 2026 12:23
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.

2 participants