sync(is-native-element): canonical update — use binding resolution for scope-shadowing (Copilot review)

Author: johanrd

lib/utils/is-native-element.js

@@ -60,16 +60,31 @@ function isNativeElement(node, sourceCode) {
   }
   const scope = sourceCode.getScope(node.parent);
   const firstPart = node.parts && node.parts[0];
-  // Compare by identifier name rather than AST node object identity — object
-  // identity isn't guaranteed across parser versions (ember-eslint-parser can
-  // produce distinct node objects for the same token depending on how the
-  // scope manager walks the tree), but the resolved `.name` is stable.
-  if (firstPart && scope.references.some((ref) => ref.identifier?.name === firstPart?.name)) {
+  // Scope-shadowing detection: treat the tag as a component invocation only
+  // when its name resolves to an actual BINDING (const / let / import / block-
+  // param) in the enclosing scope chain — not just any reference. A bare
+  // reference like `{{div}}` (helper invocation elsewhere in the template)
+  // creates an entry in `scope.references` without binding a local
+  // identifier; it must not shadow the native `<div>` tag. Walking upward
+  // through `scope.upper` catches bindings declared in outer scopes — e.g.
+  // a module-level `const div = ...` shadowing templates deeper in the file.
+  if (firstPart && hasBindingInScopeChain(scope, firstPart.name)) {
     return false;
   }
   return true;
 }
 
+function hasBindingInScopeChain(scope, name) {
+  let current = scope;
+  while (current) {
+    if (current.variables && current.variables.some((v) => v.name === name)) {
+      return true;
+    }
+    current = current.upper;
+  }
+  return false;
+}
+
 /**
  * Inverse of {@link isNativeElement}. Returns true when the node should NOT
  * be treated as a native HTML element — either because it's a component