Prefer variables in scope over global variables

packages/core/src/transform/template/template-to-typescript.ts

@@ -4,7 +4,7 @@
 import ScopeStack from './scope-stack.js';
 import { GlintEmitMetadata, GlintSpecialForm } from '@glint/core/config-types';
 import { TextContent } from './glimmer-ast-mapping-tree.js';
 import { Directive } from './transformed-module.js';
 import { DirectiveKind } from './transformed-module.js';
 
 const SPLATTRIBUTES = '...attributes';
@@ -574,10 +574,10 @@
     function treatAsGlobal(name: string): boolean {
       if (globals) {
         // If we have a known set of global identifiers, we should only treat
-        // members of that set as global and assume everything else is local.
-        // This is typically true in environments that capture scope, like
-        // strict-mode Ember.
-        return globals.includes(name);
+        // members of that set as global, unless the identifier is in scope,
+        // and assume everything else is local. This is typically true in
+        // environments that capture scope, like strict-mode Ember.
+        return globals.includes(name) && !scope.hasBinding(name);
       } else {
         // Otherwise, we assume everything is global unless we can see it
         // in scope as a block variable. This is the case in resolver-based
@@ -926,7 +926,7 @@
 
     function emitPlainAttributes(
       node: AST.ElementNode,
       directivesWeakMap: WeakMap<AST.Node, DirectiveKind>,
     ): void {
       let attributes = node.attributes.filter(
         (attr) => !attr.name.startsWith('@') && attr.name !== SPLATTRIBUTES,
@@ -994,7 +994,7 @@
 
     function emitModifiers(
       node: AST.ElementNode,
       directivesWeakMap: WeakMap<AST.Node, DirectiveKind>,
     ): void {
       for (let modifier of node.modifiers) {
         mapper.forNode(modifier, () => {

test-packages/package-test-core/__tests__/transform/template-to-typescript.test.ts

@@ -1491,4 +1491,26 @@ describe('Transform: rewriteTemplate', () => {
       ]);
     });
   });
+
+  describe('global variables', () => {
+    test('uses vaariable in scope over global variable', () => {
+      let template = `
+        {{action "action"}}
+        {{#each actions as |action|}}
+          {{action}}
+        {{/each}}`;
+
+      expect(templateBody(template, { globals: ['action'] })).toMatchInlineSnapshot(`
+        "__glintDSL__.emitContent(__glintDSL__.resolve(__glintDSL__.Globals["action"])("action"));
+        {
+        const __glintY__ = __glintDSL__.emitComponent(__glintDSL__.resolve(each)(actions));
+        {
+        const [action] = __glintY__.blockParams["default"];
+        __glintDSL__.emitContent(__glintDSL__.resolveOrReturn(action)());
+        }
+        each;
+        }"
+      `);
+    });
+  });
 });