Skip to content

Conversation

@ematipico
Copy link
Member

Changes

  • fixes a bug where the feature experimental.chromeDevtoolsWorkspace wasn't supported anymore by the Cloudflare adapter because the handling was done inside the astro dev server code path. I fixed it by moving the implementation inside a vite plugin, which is added to createVite.
  • Reduced the code duplication of the patch overlay logic. It was added twice: once as a plugin inside createVite and again in the astro server vite plugin. Now we have only one instance. I removed the use of opts.ssr because we already use applyToEnvironment
  • Removed the use of the module loader in some places. This is part of removing AstroServerApp
  • Added the same logic for CSS we use in DevApp inside AstroServerApp. This is part of removing AstroServerApp from the codebase

Testing

Manually tested the cloudflare fixture. Manually tested examples/blog.

CI should stay green

Docs

N/A

@changeset-bot
Copy link

changeset-bot bot commented Dec 16, 2025

🦋 Changeset detected

Latest commit: 0c42b20

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ematipico ematipico changed the title chore: changeset fix(cloudflare): experimental chromeDevtoolsWorkspace Dec 16, 2025
@github-actions github-actions bot added pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) labels Dec 16, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 16, 2025

⚠️ Package Trust Level Decreased

Caution

Decreased trust levels may indicate a higher risk of supply chain attacks. Please review these changes carefully.

📦 Package 🔒 Before 🔓 After
@cloudflare/unenv-preset trusted-with-provenance none
workerd trusted-with-provenance none
undici provenance none
miniflare trusted-with-provenance none
youch provenance none
@cloudflare/workerd-darwin-64 trusted-with-provenance none
@cloudflare/workerd-darwin-arm64 trusted-with-provenance none
@cloudflare/workerd-linux-64 trusted-with-provenance none
@cloudflare/workerd-linux-arm64 trusted-with-provenance none
@cloudflare/workerd-windows-64 trusted-with-provenance none
wrangler trusted-with-provenance none

@ematipico ematipico force-pushed the refactor/reduce-code-duplication branch from df4e97e to 0c42b20 Compare December 16, 2025 15:43
@ematipico ematipico requested a review from matthewp December 16, 2025 15:43
@ematipico ematipico merged commit b5aa52b into next Dec 16, 2025
21 of 22 checks passed
@ematipico ematipico deleted the refactor/reduce-code-duplication branch December 16, 2025 16:18
@florian-lefebvre
Copy link
Member

@ematipico
Copy link
Member Author

Cheers, moved it to done

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

Labels

pkg: astro Related to the core `astro` package (scope) pkg: integration Related to any renderer integration (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants