Tree-shaking the namespace if it is passed to function

src/ast/nodes/MemberExpression.ts

@@ -482,7 +482,7 @@ export default class MemberExpression
 			}
 		}
 		this.object.includePath(path, context, includeChildrenRecursively);
-		this.property.includePath(path, context, includeChildrenRecursively);
+		this.property.includePath(EMPTY_PATH, context, includeChildrenRecursively);
 	}
 }
 

src/ast/variables/LocalVariable.ts

@@ -184,7 +184,7 @@ export default class LocalVariable extends Variable {
 
 	includePath(path?: ObjectPath): void {
 		if (!this.included) {
-			super.includePath();
+			super.includePath(path);
 			for (const declaration of this.declarations) {
 				// If node is a default export, it can save a tree-shaking run to include the full declaration now
 				if (!declaration.included)

src/ast/variables/ParameterVariable.ts

@@ -1,7 +1,7 @@
 import type { AstContext } from '../../Module';
 import { EMPTY_ARRAY } from '../../utils/blank';
 import type { DeoptimizableEntity } from '../DeoptimizableEntity';
-import type { HasEffectsContext } from '../ExecutionContext';
+import { createInclusionContext, type HasEffectsContext } from '../ExecutionContext';
 import type { NodeInteraction } from '../NodeInteractions';
 import { INTERACTION_ASSIGNED, INTERACTION_CALLED } from '../NodeInteractions';
 import type ExportDefaultDeclaration from '../nodes/ExportDefaultDeclaration';
@@ -180,6 +180,13 @@ export default class ParameterVariable extends LocalVariable {
 		return knownValue.hasEffectsOnInteractionAtPath(path, interaction, context);
 	}
 
+	includePath(path?: ObjectPath): void {
+		super.includePath(path);
+		if (this.knownValue && path) {
+			this.knownValue.includePath(path, createInclusionContext(), false);
+		}
+	}
+
 	deoptimizeArgumentsOnInteractionAtPath(interaction: NodeInteraction, path: ObjectPath): void {
 		// For performance reasons, we fully deoptimize all deeper interactions
 		if (

src/ast/variables/Variable.ts

@@ -108,9 +108,9 @@ export default class Variable extends ExpressionEntity {
 	 * previously.
 	 * Once a variable is included, it should take care all its declarations are included.
 	 */
-	includePath(_path?: ObjectPath): void {
+	includePath(path?: ObjectPath): void {
 		this.included = true;
-		this.renderedLikeHoisted?.includePath();
+		this.renderedLikeHoisted?.includePath(path);
 	}
 
 	/**

test/chunking-form/samples/dynamic-import-with-namespace-5/_config.js

@@ -1,4 +1,3 @@
 module.exports = defineTest({
-	description:
-		'Passing the namespace to a function will disable the tree-shaking of namespace (Ideally, it should not disable the tree-shaking of namespace)'
+	description: 'Passing the namespace to a function does not disable the tree-shaking of namespace'
 });

test/chunking-form/samples/dynamic-import-with-namespace-5/_expected/amd/generated-module.js

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

test/chunking-form/samples/dynamic-import-with-namespace-5/_expected/cjs/generated-module.js

@@ -1,7 +1,5 @@
 'use strict';
 
 const foo = () => {};
-const bar = () => {};
 
-exports.bar = bar;
 exports.foo = foo;

test/chunking-form/samples/dynamic-import-with-namespace-5/_expected/es/generated-module.js

@@ -1,4 +1,3 @@
 const foo = () => {};
-const bar = () => {};
 
-export { bar, foo };
+export { foo };

test/chunking-form/samples/dynamic-import-with-namespace-5/_expected/system/generated-module.js

@@ -4,7 +4,6 @@ System.register([], (function (exports) {
 		execute: (function () {
 
 			const foo = exports("foo", () => {});
-			const bar = exports("bar", () => {});
 
 		})
 	};