normalize emitted lazy entries in the raw output bundle

Downstream plugins inspecting the client bundle in generateBundle (and
Vite's manifest plugin serializing it) saw lazy facade chunks marked
isEntry, mistaking them for application entries. Apply the existing
reclassification to the Rollup bundle itself, ahead of the client asset
map build. Ports the fix from #266.

Co-authored-by: Brenley Dueck <brenleydueck@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Author: 3 people

.changeset/clean-lazy-entries.md

@@ -0,0 +1,5 @@
+---
+'vite-plugin-solid': patch
+---
+
+Reclassify emitted lazy facade chunks as dynamic entries in the raw output bundle so downstream plugins do not mistake them for application entries.

examples/css-matrix/test/run.mjs

@@ -292,6 +292,17 @@ async function runMode(mode) {
         sharedChunks.size === 1,
         `expected 1 shared chunk, saw ${sharedChunks.size} (${[...sharedChunks].join(', ')})`,
       );
+      // Emitted lazy facade chunks must be reclassified as dynamic entries in
+      // the output bundle itself — downstream plugins (and this manifest,
+      // which Vite generates from the bundle) must see exactly one real entry.
+      const entries = Object.keys(manifest).filter((k) => manifest[k].isEntry);
+      record(
+        mode,
+        'ssr',
+        'single application entry in client manifest',
+        entries.length === 1 && entries[0] === 'src/entry-client.tsx',
+        `entries: ${entries.join(', ')}`,
+      );
     }
 
     // ---- Phase 2: browser ------------------------------------------------

src/index.ts

@@ -240,7 +240,9 @@ function normalizeSourceMap(map: string | babel.TransformOptions['inputSourceMap
  * though they are semantically dynamic entries. Reclassify any entry that is
  * dynamically imported by another chunk so the runtime's entry-asset
  * detection (which keys off `isEntry`) can't pick a lazy facade instead of
- * the real client entry.
+ * the real client entry. Works on both the Vite manifest.json shape and the
+ * raw Rollup output bundle — both key entries by name and expose
+ * `dynamicImports` / `isEntry` with the same meaning.
  */
 function normalizeEmittedLazyEntries(manifest: Record<string, any>) {
   const dynamicKeys = new Set<string>();
@@ -585,6 +587,11 @@ export default function solidPlugin(options: Partial<Options> = {}): Plugin {
 
     generateBundle(_options, bundle) {
       if (!isBuild || !isClientBuild(this)) return;
+      // Reclassify emitted lazy facade chunks in the raw bundle (not just the
+      // serialized manifest read back later) so downstream plugins inspecting
+      // the bundle don't mistake them for application entries. Must precede
+      // the client asset map build, which keys off dynamic entries.
+      if (options.ssr) normalizeEmittedLazyEntries(bundle);
       substituteClientManifest(bundle, buildClientAssetMap(bundle, projectRoot, base));
     },