Make macro configs (except fastboot) be considered not having a runti…

…me implementation
embroider-build · Feb 26, 2024 · 21ab0eb · 21ab0eb
1 parent e98e401
commit 21ab0eb
Show file tree

Hide file tree

Showing 3 changed files with 72 additions and 19 deletions.
diff --git a/packages/macros/src/babel/evaluate-json.ts b/packages/macros/src/babel/evaluate-json.ts
@@ -388,13 +388,43 @@ export class Evaluator {
       return { confident: true, value: moduleExists(path, this.state), hasRuntimeImplementation: false };
     }
     if (callee.referencesImport('@embroider/macros', 'getConfig')) {
-      return { confident: true, value: getConfig(path, this.state, 'package'), hasRuntimeImplementation: true };
+      return { confident: true, value: getConfig(path, this.state, 'package'), hasRuntimeImplementation: false };
     }
     if (callee.referencesImport('@embroider/macros', 'getOwnConfig')) {
-      return { confident: true, value: getConfig(path, this.state, 'own'), hasRuntimeImplementation: true };
+      return { confident: true, value: getConfig(path, this.state, 'own'), hasRuntimeImplementation: false };
     }
     if (callee.referencesImport('@embroider/macros', 'getGlobalConfig')) {
-      return { confident: true, value: getConfig(path, this.state, 'getGlobalConfig'), hasRuntimeImplementation: true };
+      // Check for getGlobalConfig().fastboot.isRunning, which is the only pattern in use where config actually needs to have a runtime implementation.
+      // For compatibility reasons we will continue to support that. All other cases of macro configs are static now.
+      let maybeFastbootMemberExpression = path.parentPath;
+      if (
+        maybeFastbootMemberExpression.isMemberExpression() ||
+        maybeFastbootMemberExpression.isOptionalMemberExpression()
+      ) {
+        let maybeFastbootProperty = maybeFastbootMemberExpression.get('property');
+        if (maybeFastbootProperty.isIdentifier() && maybeFastbootProperty.node.name === 'fastboot') {
+          let maybeIsRunningMemberExpression = maybeFastbootMemberExpression.parentPath;
+          if (
+            maybeIsRunningMemberExpression.isMemberExpression() ||
+            maybeIsRunningMemberExpression.isOptionalMemberExpression()
+          ) {
+            let maybeIsRunningProperty = maybeIsRunningMemberExpression.get('property');
+            if (maybeIsRunningProperty.isIdentifier() && maybeIsRunningProperty.node.name === 'isRunning') {
+              return {
+                confident: true,
+                value: getConfig(path, this.state, 'getGlobalConfig'),
+                hasRuntimeImplementation: true,
+              };
+            }
+          }
+        }
+      }
+
+      return {
+        confident: true,
+        value: getConfig(path, this.state, 'getGlobalConfig'),
+        hasRuntimeImplementation: false,
+      };
     }
     if (callee.referencesImport('@embroider/macros', 'isDevelopingApp')) {
       return {

diff --git a/packages/macros/tests/babel/eval.test.ts b/packages/macros/tests/babel/eval.test.ts
@@ -164,21 +164,57 @@ describe('hasRuntimeImplementation', function () {
         let code = transform(`
         import { getConfig } from '@embroider/macros';
         const result = getConfig('foo')`);
-        expect(code).toMatch(`result = true`);
+        expect(code).toMatch(`result = false`);
+      });
+
+      test('getConfig property access', () => {
+        let code = transform(`
+        import { getConfig } from '@embroider/macros';
+        const result = getConfig('foo').bar`);
+        expect(code).toMatch(`result = false`);
       });
 
       // this is throwing internally, not sure how to fix
       test.skip('getOwnConfig', () => {
         let code = transform(`
         import { getOwnConfig } from '@embroider/macros';
         const result = getOwnConfig()`);
-        expect(code).toMatch(`result = true`);
+        expect(code).toMatch(`result = false`);
       });
 
       test('getGlobalConfig', () => {
         let code = transform(`
         import { getGlobalConfig } from '@embroider/macros';
         const result = getGlobalConfig()`);
+        expect(code).toMatch(`result = false`);
+      });
+
+      test('getGlobalConfig property access', () => {
+        let code = transform(`
+        import { getGlobalConfig } from '@embroider/macros';
+        const result = getGlobalConfig().foo`);
+        expect(code).toMatch(`result = false`);
+      });
+
+      test('getGlobalConfig fastboot access', () => {
+        let code = transform(`
+        import { getGlobalConfig } from '@embroider/macros';
+        const result = getGlobalConfig().fastboot`);
+        expect(code).toMatch(`result = false`);
+      });
+
+      // fastboot.isRunning relies on dynamic evaluation at runtime. For backwards compatibility we keep it working. See https://github.com/embroider-build/embroider/issues/1804
+      test('getGlobalConfig fastboot.isRunning access', () => {
+        let code = transform(`
+        import { getGlobalConfig } from '@embroider/macros';
+        const result = getGlobalConfig().fastboot.isRunning`);
+        expect(code).toMatch(`result = true`);
+      });
+
+      test('getGlobalConfig fastboot?.isRunning access', () => {
+        let code = transform(`
+        import { getGlobalConfig } from '@embroider/macros';
+        const result = getGlobalConfig().fastboot?.isRunning`);
         expect(code).toMatch(`result = true`);
       });
 

diff --git a/packages/macros/tests/babel/macro-condition.test.ts b/packages/macros/tests/babel/macro-condition.test.ts
@@ -360,7 +360,7 @@ describe('macroCondition', function () {
         expect(code).not.toMatch(/beta/);
       });
 
-      buildTimeTest('can see booleans inside getConfig', () => {
+      test('can see booleans inside getConfig', () => {
         let code = transform(`
       import { macroCondition, getConfig } from '@embroider/macros';
       export default function() {
@@ -373,19 +373,6 @@ describe('macroCondition', function () {
         expect(code).not.toMatch(/alpha/);
       });
 
-      runTimeTest('can see booleans inside getConfig', () => {
-        let code = transform(`
-      import { macroCondition, getConfig } from '@embroider/macros';
-      export default function() {
-        // this deliberately chains three kinds of property access syntax: by
-        // identifier, by numeric index, and by string literal.
-        return macroCondition(getConfig('qunit').items[0]["other"]) ? 'alpha' : 'beta';
-      }
-      `);
-        expect(run(code, { filename })).toBe('beta');
-        expect(code).toMatch(/alpha/);
-      });
-
       if (transform.babelMajorVersion === 7) {
         buildTimeTest('can be used as class field initializer', () => {
           let code = transform(`