🐛 fix: undefined deoptimizePath when the first element is empty str…

…ing (#4070)

* 🐛 fix: undefined `deoptimizePath` when the first element is empty string

* Add test

Co-authored-by: Lukas Taegert-Atkinson <lukas.taegert-atkinson@tngtech.com>
Co-authored-by: Lukas Taegert-Atkinson <lukastaegert@users.noreply.github.com>

src/ast/variables/NamespaceVariable.ts

@@ -33,12 +33,12 @@ export default class NamespaceVariable extends Variable {
 		const memberVariables = this.getMemberVariables();
 		const memberPath = path.length <= 1 ? UNKNOWN_PATH : path.slice(1);
 		const key = path[0];
-		if (typeof key !== 'string') {
+		if (typeof key === 'string') {
+			memberVariables[key]?.deoptimizePath(memberPath);
+		} else {
 			for (const key of Object.keys(memberVariables)) {
 				memberVariables[key].deoptimizePath(memberPath);
 			}
-		} else {
-			memberVariables[key].deoptimizePath(memberPath);
 		}
 	}
 

test/function/samples/deoptimize-missing-namespace-members/_config.js

@@ -0,0 +1,8 @@
+const assert = require('assert');
+
+module.exports = {
+	description: 'handles deoptimization of non-existing namespace members',
+	exports(exports) {
+		assert.strictEqual(exports.bar, undefined);
+	}
+};

test/function/samples/deoptimize-missing-namespace-members/main.js

@@ -0,0 +1,4 @@
+import * as ns from './other.js';
+
+const foo = 'foo';
+export const bar = ns[foo];