Co-authored-by: Princesseuh <[email protected]>

Co-authored-by: Emanuele Stoppa <[email protected]>

* Consolidate inline script escaping into shared stringifyForScript utility

Unify the two separate script-embedding escape approaches (defineScriptVars
and safeJsonStringify) into a single stringifyForScript function in escape.ts.
Uses a comprehensive < escape strategy rather than pattern-matching specific
tag sequences, covering all ETAGO variants in one pass.

* Add changeset

* Update changeset description

* Fix stringifyForScript to handle undefined values

* Update astro-directives test assertion for new escape sequence

…16202)

…t: node (#16210)

* fix(cloudflare,svelte): resolve .svelte files in node_modules with prerenderEnvironment: 'node'

Packages that ship .svelte files (e.g. bits-ui) failed with 'Unknown file
extension .svelte' when using the Cloudflare adapter with
prerenderEnvironment: 'node'. Two issues caused this:

1. The cf-externals plugin set top-level ssr.noExternal = true, which
   caused vitefu's crawlFrameworkPkgs to return an empty noExternal list
   (it assumed everything was already noExternal). This only applied to
   the ssr environment in Vite 6, leaving the prerender environment
   without the needed noExternal entries.

2. The createNodePrerenderPlugin disabled dep optimization entirely for
   the prerender environment (noDiscovery: true, include: []).

The fix removes ssr.noExternal = true from cf-externals, removes the dep
optimization override, and has @astrojs/svelte use crawlFrameworkPkgs to
discover svelte packages and add them to resolve.noExternal for all
server environments via configEnvironment.

* add changeset

* fix: use fileURLToPath for cross-platform root path

* test: add response body to assertion error for CI debugging

* fix: rename fake-svelte-pkg/dist to src to avoid .gitignore exclusion

* fix: avoid full-reload in scss modules

* fix: improve regex

* fix: ci

* test: add test for modules

* fix(hmr): prevent full-reload for SCSS/CSS module changes

SCSS and CSS module file changes were triggering unnecessary full page
reloads during development instead of applying HMR updates. This broke
the developer experience by losing component state and scroll position
on every style edit.

Root cause: In the `astro:hmr-reload` Vite plugin, style files (CSS,
SCSS, SASS, LESS, etc.) in the SSR module graph were correctly skipped
via a regex check, but the handler returned `undefined` instead of an
empty array. In Vite 6, returning `undefined` from a `hotUpdate` hook
means "I didn't handle this", causing Vite to propagate through the SSR
module graph to `.astro` importers. Since `.astro` pages have no HMR
boundary, this triggered a full page reload.

Changes to `vite-plugin-hmr-reload`:

- Extract `isStyleModule()` helper that checks both `mod.file` and
  `mod.id` (stripping query params like `?inline`, `?used`) against
  the style extension regex. This correctly identifies all style-related
  modules including CSS module variants.

- Return `[]` (empty array) when only style modules were encountered
  in the SSR environment. This tells Vite "handled, nothing to update
  in SSR", preventing the propagation chain that caused full reloads.
  The client environment handles CSS HMR natively through
  framework-specific HMR boundaries (Preact, React, Vue, etc.).

Changes to e2e test fixtures:

- Update the SCSS module HMR test to use a Preact component with
  `client:load` instead of a pure server-rendered Astro page. This
  matches the real-world scenario from the original bug report (issue
  #14869) where Preact + SCSS modules triggered full reloads. CSS
  module HMR requires a client-side framework to re-render components
  with updated class name hashes — pure SSR pages cannot hot-update
  CSS module class names without a full reload.

Closes #14869

* fix(hmr): clarify comment about CSS HMR working for all pages

The previous comment suggested CSS HMR only worked through
framework-specific boundaries. Vite's built-in style update
mechanism handles it for all pages, with or without framework
components (covered by the scss-external test).

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

* update pnpm-lock.yaml

* chore: add changeset for SCSS/CSS module HMR fix

---------

Co-authored-by: Erika <[email protected]>
Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>