Update on "compiler: getGlobalDeclaration() takes a NonLocalBinding"

No-op refactor to make Environment#getGlobalDeclaration() take a NonLocalBinding instead of just a name. The idea is that in subsequent PRs we can use information about the binding to resolve a type more accurately. For example, we can resolve `Array` differently if its an import or local and not the global Array. Similar for resolving local `useState` differently than the one from React.

[ghstack-poisoned]

compiler/packages/babel-plugin-react-compiler/src/HIR/HIRBuilder.ts

@@ -235,7 +235,7 @@ export default class HIRBuilder {
       return { kind: "Global", name: originalName };
     }
 
-    // Check if the binding is from module scope, if so return null
+    // Check if the binding is from module scope
     const outerBinding =
       this.parentFunction.scope.parent.getBinding(originalName);
     if (babelBinding === outerBinding) {
@@ -286,6 +286,13 @@ export default class HIRBuilder {
   isContextIdentifier(path: NodePath<t.Identifier | t.JSXIdentifier>): boolean {
     const binding = this.#resolveBabelBinding(path);
     if (binding) {
+      // Check if the binding is from module scope, if so return null
+      const outerBinding = this.parentFunction.scope.parent.getBinding(
+        path.node.name
+      );
+      if (binding === outerBinding) {
+        return false;
+      }
       return this.#env.isContextIdentifier(binding.identifier);
     } else {
       return false;