revert: PR 16708

[ematipico]

Changes

It seems that emdash-cms/emdash#1116 was caused by #16708

This PR reverts it

Testing

Preview release

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: ac8c970

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

This PR includes changesets to release 36 packages

Name	Type
@astrojs/cloudflare	Patch
@test/astro-cloudflare-node-prerender-mdx	Patch
@test/astro-cloudflare	Patch
@test/astro-cloudflare-allowed-hosts	Patch
@test/astro-cloudflare-astro-dev-platform	Patch
@test/astro-cloudflare-astro-env	Patch
@test/astro-cloudflare-binding-image-service	Patch
@test/astro-cloudflare-cache-provider-wait-until	Patch
@test/astro-cloudflare-client-address	Patch
@test/astro-cloudflare-compile-image-service	Patch
@test/astro-cloudflare-custom-entryfile	Patch
@test/astro-cloudflare-dev-image-endpoint	Patch
@test/astro-cloudflare-external-image-service	Patch
@test/astro-cloudflare-external-redirects	Patch
@test/astro-cloudflare-internal-redirects	Patch
@test/astro-cloudflare-no-output	Patch
@test/astro-cloudflare-prerender-node-env	Patch
@test/astro-cloudflare-prerender-queue-consumers	Patch
@test/astro-cloudflare-prerender-styles	Patch
@test/astro-cloudflare-prerenderer-errors	Patch
@test/routing-priority-cloudflare	Patch
@test/cf-server-entry	Patch
@test/astro-cloudflare-server-island-prerender-framework	Patch
@test/astro-cloudflare-sessions	Patch
@test/astro-cloudflare-sql-import	Patch
@test/cf-ssr-deps	Patch
@test/astro-cloudflare-static	Patch
@test/astro-cloudflare-svelte-rune-deps	Patch
@test/astro-cloudflare-top-level-return	Patch
@test/astro-cloudflare-vite-plugin	Patch
@test/astro-cloudflare-with-base	Patch
@test/astro-cloudflare-with-react	Patch
@test/astro-cloudflare-with-solid-js	Patch
@test/astro-cloudflare-with-svelte	Patch
@test/astro-cloudflare-with-vue	Patch
@test/astro-cloudflare-wrangler-preview-platform	Patch

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

[github-actions]

⚠️ Dependency Count

This PR adds 19 new dependencies (2184 → 2203), which exceeds the threshold of 10.

📊 Dependency Size Changes

Warning

This PR adds 15.2 MB of new dependencies, which exceeds the threshold of 100 kB.

📦 Package	📏 Size
@opentelemetry/semantic-conventions@1.28.0	6.2 MB
@types/node@22.19.19	-2.4 MB
@types/node@22.19.11	2.4 MB
@types/node@25.2.3	2.4 MB
@types/node@20.19.33	2.3 MB
@opentelemetry/core@1.30.1	908.7 kB
@opentelemetry/sdk-trace-base@1.30.1	796.3 kB
@opentelemetry/instrumentation@0.203.0	655.2 kB
@opentelemetry/resources@1.30.1	557.7 kB
@opentelemetry/api-logs@0.203.0	200.1 kB
@typescript-eslint/types@8.57.1	162.4 kB
resolve@1.22.11	154.2 kB
@opentelemetry/propagator-b3@1.30.1	145 kB
cjs-module-lexer@1.4.3	137.7 kB
@netlify/blobs@10.7.4	134.1 kB
import-in-the-middle@1.15.0	127.7 kB
@netlify/otel@5.1.5	82.8 kB
@opentelemetry/propagator-jaeger@1.30.1	81.4 kB
@opentelemetry/context-async-hooks@1.30.1	59.3 kB
@opentelemetry/sdk-trace-node@1.30.1	35.6 kB
require-in-the-middle@7.5.2	20 kB

Total size change: 15.2 MB

⚠️ 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
@opentelemetry/core	trusted-with-provenance	provenance
@opentelemetry/instrumentation	trusted-with-provenance	provenance
@opentelemetry/resources	trusted-with-provenance	provenance
@opentelemetry/sdk-trace-node	trusted-with-provenance	provenance
@opentelemetry/api-logs	trusted-with-provenance	provenance
@opentelemetry/context-async-hooks	trusted-with-provenance	provenance
@opentelemetry/semantic-conventions	trusted-with-provenance	provenance
require-in-the-middle	trusted-with-provenance	none
@opentelemetry/sdk-trace-base	trusted-with-provenance	provenance

[pkg-pr-new]

npm i https://pkg.pr.new/astro@16801

npm i https://pkg.pr.new/@astrojs/cloudflare@16801

commit: ac8c970

[Princesseuh]

We should maybe reopen #16248 as well

[ascorbic]

Hopefully we can land a new variant of the fix. We ran into two separate symptoms with this: "Astro is not defined", and a failure to resolve virtual modules similar to the one in nicia-ai/typegraph#146

[ascorbic]

Here's the stack trace from the user report, which I was able to repro on the default EmDash blog template:

20:49:25 [ERROR] [vite] Internal server error: Astro is not defined
      at runInRunnerObject (workers/runner-worker/index.js:107:3)
      at _NonRunnablePipeline.getComponentByRoute (C:/Users/edwar/Documents/GitHub/dev-sidequest-diy/node_modules/.vite/deps_ssr/astro_app_entrypoint_dev.js?v=997697c1:300:20)
      at matchRoute (C:/Users/edwar/Documents/GitHub/dev-sidequest-diy/node_modules/.vite/deps_ssr/astro_app_entrypoint_dev.js?v=997697c1:370:14)
      at DevApp.devMatch (C:/Users/edwar/Documents/GitHub/dev-sidequest-diy/node_modules/.vite/deps_ssr/astro_app_entrypoint_dev.js?v=997697c1:479:26)
      at Object.handle [as fetch] (C:/Users/edwar/Documents/GitHub/dev-sidequest-diy/node_modules/.vite/deps_ssr/@astrojs_cloudflare_entrypoints_server.js?v=997697c1:174:20)
      at maybeCaptureError (workers/runner-worker/index.js:51:10)