Fix handling of namespace reexports when preserving modules

src/Chunk.ts

@@ -782,6 +782,9 @@ export default class Chunk {
 							continue;
 						}
 						const chunkDep = this.renderedDependencies!.get(chunk)!;
+						if (!chunkDep) {
+							continue;
+						}
 						const { imports, reexports } = chunkDep;
 						const importedByReexported = reexports?.find(
 							({ reexported }) => reexported === exportName

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_config.js

@@ -0,0 +1,8 @@
+module.exports = {
+	description: 'handles doing a namespace reexport of a reexport',
+	options: {
+		output: {
+			preserveModules: true
+		}
+	}
+};

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/amd/dep.js

@@ -0,0 +1,7 @@
+define(['exports'], (function (exports) { 'use strict';
+
+	const value = 'foo';
+
+	exports.value = value;
+
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/amd/main.js

@@ -0,0 +1,5 @@
+define(['./reexport', './dep'], (function (reexport, dep) { 'use strict';
+
+	console.log(dep.value);
+
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/amd/reexport.js

@@ -0,0 +1,5 @@
+define((function () { 'use strict';
+
+	console.log('reexport');
+
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/cjs/dep.js

@@ -0,0 +1,5 @@
+'use strict';
+
+const value = 'foo';
+
+exports.value = value;

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/cjs/main.js

@@ -0,0 +1,6 @@
+'use strict';
+
+require('./reexport.js');
+var dep = require('./dep.js');
+
+console.log(dep.value);

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/cjs/reexport.js

@@ -0,0 +1,3 @@
+'use strict';
+
+console.log('reexport');

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/es/dep.js

@@ -0,0 +1,3 @@
+const value = 'foo';
+
+export { value };

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/es/main.js

@@ -0,0 +1,4 @@
+import './reexport.js';
+import { value } from './dep.js';
+
+console.log(value);

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/es/reexport.js

@@ -0,0 +1 @@
+console.log('reexport');

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/system/dep.js

@@ -0,0 +1,10 @@
+System.register([], (function (exports) {
+	'use strict';
+	return {
+		execute: (function () {
+
+			const value = exports('value', 'foo');
+
+		})
+	};
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/system/main.js

@@ -0,0 +1,14 @@
+System.register(['./reexport.js', './dep.js'], (function () {
+	'use strict';
+	var value;
+	return {
+		setters: [null, function (module) {
+			value = module.value;
+		}],
+		execute: (function () {
+
+			console.log(value);
+
+		})
+	};
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/_expected/system/reexport.js

@@ -0,0 +1,10 @@
+System.register([], (function () {
+	'use strict';
+	return {
+		execute: (function () {
+
+			console.log('reexport');
+
+		})
+	};
+}));

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/dep.js

@@ -0,0 +1 @@
+export const value = 'foo';

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/main.js

@@ -0,0 +1,3 @@
+import { value } from './reexport2';
+
+console.log(value);

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/reexport.js

@@ -0,0 +1,2 @@
+export { value } from './dep.js';
+console.log('reexport');

test/chunking-form/samples/preserve-modules-namespace-reexport-reexport/reexport2.js

@@ -0,0 +1 @@
+export * from './reexport.js';