@open-xchange/vite-plugin-ox-manifests 1.0.4 → 1.0.5

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+## [1.0.5] - 2026-05-08
+
+### Fixed
+
+- Drop externalised modules from `imports`/`dynamicImports` in `manifest.json`. Under Rolldown these were leaking through (Rollup/Vite 7 silently filtered them), causing downstream consumers like `core-ui-middleware` to crash with `Cannot read properties of undefined (reading 'file')` when walking transitive deps.
+
 ## [1.0.4] - 2026-04-09
 
 ### Fixed

package/dist/plugins/manifests.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"manifests.d.ts","sourceRoot":"","sources":["../../src/plugins/manifests.ts"],"names":[],"mappings":";AAyHA,wBAgGE"}
+{"version":3,"file":"manifests.d.ts","sourceRoot":"","sources":["../../src/plugins/manifests.ts"],"names":[],"mappings":";AAkIA,wBAgGE"}

package/dist/plugins/manifests.js

@@ -79,14 +79,23 @@ function buildManifestFromBundle(bundle, root) {
         manifest[src] = entry;
         fileNameToKey[output.fileName] = src;
     }
-    // Remap imports/dynamicImports from output filenames to manifest keys
+    // Remap imports/dynamicImports from output filenames to manifest keys.
+    // Drop entries that don't resolve to a manifest key — under Rolldown,
+    // chunk.imports/dynamicImports include externalized modules that Rollup
+    // (Vite 7) silently filtered out. Leaving them in causes downstream
+    // consumers (e.g. core-ui-middleware's viteManifestToDeps) to crash on
+    // undefined lookups when walking imports.
     for (const key in manifest) {
         const entry = manifest[key];
         if (entry.imports) {
-            entry.imports = entry.imports.map(imp => fileNameToKey[imp] ?? imp);
+            entry.imports = entry.imports
+                .map(imp => fileNameToKey[imp] ?? imp)
+                .filter(imp => imp in manifest);
         }
         if (entry.dynamicImports) {
-            entry.dynamicImports = entry.dynamicImports.map(imp => fileNameToKey[imp] ?? imp);
+            entry.dynamicImports = entry.dynamicImports
+                .map(imp => fileNameToKey[imp] ?? imp)
+                .filter(imp => imp in manifest);
         }
     }
     // Also include assets