Do not add tree-shaken variables to namespaces when inlining dynamic …

…imports (#5047)

src/Chunk.ts

@@ -1367,7 +1367,9 @@ export default class Chunk {
 		if (!this.outputOptions.preserveModules && this.includedNamespaces.has(module)) {
 			const memberVariables = module.namespace.getMemberVariables();
 			for (const variable of Object.values(memberVariables)) {
-				moduleImports.add(variable);
+				if (variable.included) {
+					moduleImports.add(variable);
+				}
 			}
 		}
 		for (let variable of moduleImports) {

src/Module.ts

@@ -452,7 +452,7 @@ export default class Module {
 		) {
 			for (const exportName of [...this.getReexports(), ...this.getExports()]) {
 				const [exportedVariable] = this.getVariableForExportName(exportName);
-				if (exportedVariable) {
+				if (exportedVariable?.included) {
 					dependencyVariables.add(exportedVariable);
 				}
 			}

test/form/samples/inline-with-reexport/_config.js

@@ -0,0 +1,8 @@
+module.exports = defineTest({
+	description: 'handles inlining dynamic imports when the imported module contains reexports',
+	options: {
+		output: {
+			inlineDynamicImports: true
+		}
+	}
+});

test/form/samples/inline-with-reexport/_expected.js

@@ -0,0 +1,5 @@
+Promise.resolve().then(function () { return a; });
+
+var a = /*#__PURE__*/Object.freeze({
+	__proto__: null
+});

test/form/samples/inline-with-reexport/a.js

@@ -0,0 +1 @@
+export { b } from './b.js';

test/form/samples/inline-with-reexport/b.js

@@ -0,0 +1 @@
+export const b = 42;

test/form/samples/inline-with-reexport/main.js

@@ -0,0 +1 @@
+import('./a.js');