Post-merge-review: Fix template-no-action-modifiers autofix: skip when hash pairs are present

lib/rules/template-no-action-modifiers.js

@@ -50,8 +50,10 @@ module.exports = {
         node.path.head?.type !== 'ThisHead'
       ) {
         // Only offer autofix when the first param is a path expression
+        // and there are no hash pairs (e.g. on="submit") which would be left behind
         const maybePath = node.params?.[0];
-        const canFix = maybePath && maybePath.type === 'GlimmerPathExpression';
+        const hasHashPairs = node.hash?.pairs?.length > 0;
+        const canFix = maybePath && maybePath.type === 'GlimmerPathExpression' && !hasHashPairs;
 
         context.report({
           node,

tests/lib/rules/template-no-action-modifiers.js

@@ -61,5 +61,17 @@ ruleTester.run('template-no-action-modifiers', rule, {
         '<template><button {{on "click" (fn this.handleClick "arg1" "arg2")}}>Save</button></template>',
       errors: [{ messageId: 'noActionModifier' }],
     },
+    {
+      // Path expression with hash pair (on="click") — no autofix to avoid leaving behind stale hash
+      code: '<template><button {{action this.handleClick on="click"}}>Save</button></template>',
+      output: null,
+      errors: [{ messageId: 'noActionModifier' }],
+    },
+    {
+      // Path expression with hash pair (on="submit") — no autofix
+      code: '<template><form {{action this.handleSubmit on="submit"}}>Submit</form></template>',
+      output: null,
+      errors: [{ messageId: 'noActionModifier' }],
+    },
   ],
 });