Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: ultra dev #230

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

feat: ultra dev #230

wants to merge 8 commits into from

Conversation

deckchairlabs
Copy link
Member

@deckchairlabs deckchairlabs commented Dec 15, 2022

This is very much a WIP and not HMR (would only support basic window.reload)

Theres a new example basic-dev utilising the new createDev. Basically it creates a temp directory and moves everything into that, we need to properly filter out ignored files etc.

What works

  • The server starts (yay) and serves requests

What isn't working

  • The paths are all screwed up, the client hydration script attempts to be loaded from /_ultra/compiler/[temp path]/client.tsx which then throws an error.
  • Need to inject a websocket script so we can trigger a window reload
  • Need to handle removing files without causing an error
  • Need to test that appending a timestamp to the main entrypoint server.tsx works as intended for it's dependencies ie. changing something in app.tsx for instance.

@talentlessguy
Copy link

here's a relatively simple implementation of HMR for ESM: https://github.com/FredKSchott/esm-hmr

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