Merge pull request #2497 from johanrd/fix/2477

 Post-merge review of #2477 (`template-no-link-to-positional-params`)

Author: NullVoxPopuli

lib/rules/template-no-link-to-positional-params.js

@@ -1,15 +1,3 @@
-/**
- * Helper function to check if a node is a LinkTo component
- * @param {Object} node - The AST node to check
- * @returns {boolean} - True if node is a LinkTo component
- */
-function isLinkToComponent(node) {
-  if (node.type === 'GlimmerElementNode') {
-    return node.tag === 'LinkTo' || node.tag === 'link-to';
-  }
-  return false;
-}
-
 /** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
@@ -49,29 +37,6 @@ module.exports = {
     }
 
     return {
-      GlimmerElementNode(node) {
-        if (!isLinkToComponent(node)) {
-          return;
-        }
-
-        // Check if there are positional params (non-@ attributes without = sign, which are params)
-        const hasPositionalParams = node.attributes.some(
-          (attr) =>
-            attr.type === 'GlimmerAttrNode' &&
-            !attr.name.startsWith('@') &&
-            attr.value &&
-            attr.value.type !== 'GlimmerTextNode' &&
-            !node.modifiers.some((mod) => mod.path.original === attr.name)
-        );
-
-        if (hasPositionalParams) {
-          context.report({
-            node,
-            messageId: 'noLinkToPositionalParams',
-          });
-        }
-      },
-
       GlimmerMustacheStatement(node) {
         checkForPositionalParams(node);
       },