From 365138c12de76fd75123b47b6ee82777586b572b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Thu, 12 Mar 2026 23:13:03 +0100
Subject: [PATCH 1/6] Fix template-no-autofocus-attribute: add
 MustacheStatement, remove autofix

- Add GlimmerMustacheStatement visitor for {{input autofocus=true}}
- Change fixable to null (matching original's deliberate design choice)
- Add test cases for mustache form, div, h1, input autofocus="autofocus"
---
 lib/rules/template-no-autofocus-attribute.js  | 27 ++++----
 .../rules/template-no-autofocus-attribute.js  | 61 +++++++++++++++++--
 2 files changed, 69 insertions(+), 19 deletions(-)

diff --git a/lib/rules/template-no-autofocus-attribute.js b/lib/rules/template-no-autofocus-attribute.js
index b0d775d14a..db3465c1be 100644
--- a/lib/rules/template-no-autofocus-attribute.js
+++ b/lib/rules/template-no-autofocus-attribute.js
@@ -9,7 +9,7 @@ module.exports = {
       strictGts: true,
       url: 'https://github.com/ember-cli/eslint-plugin-ember/tree/master/docs/rules/template-no-autofocus-attribute.md',
     },
-    fixable: 'code',
+    fixable: null,
     schema: [],
     messages: {
       noAutofocus:
@@ -26,20 +26,19 @@ module.exports = {
           context.report({
             node: autofocusAttr,
             messageId: 'noAutofocus',
-            fix(fixer) {
-              // Remove the attribute including preceding whitespace
-              const sourceCode = context.sourceCode;
-              const text = sourceCode.getText();
-              const attrStart = autofocusAttr.range[0];
-              const attrEnd = autofocusAttr.range[1];
-
-              let removeStart = attrStart;
-              while (removeStart > 0 && /\s/.test(text[removeStart - 1])) {
-                removeStart--;
-              }
+          });
+        }
+      },
 
-              return fixer.removeRange([removeStart, attrEnd]);
-            },
+      GlimmerMustacheStatement(node) {
+        if (!node.hash || !node.hash.pairs) {
+          return;
+        }
+        const autofocusPair = node.hash.pairs.find((pair) => pair.key === 'autofocus');
+        if (autofocusPair) {
+          context.report({
+            node: autofocusPair,
+            messageId: 'noAutofocus',
           });
         }
       },
diff --git a/tests/lib/rules/template-no-autofocus-attribute.js b/tests/lib/rules/template-no-autofocus-attribute.js
index afee5842d2..d3e6b38dac 100644
--- a/tests/lib/rules/template-no-autofocus-attribute.js
+++ b/tests/lib/rules/template-no-autofocus-attribute.js
@@ -29,9 +29,6 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         <input type="text" autofocus />
       </template>`,
-      output: `<template>
-        <input type="text"/>
-      </template>`,
       errors: [
         {
           message:
@@ -44,11 +41,65 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         <textarea autofocus></textarea>
       </template>`,
-      output: `<template>
-        <textarea></textarea>
+      errors: [
+        {
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        {{input type="text" autofocus=true}}
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerHashPair',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        {{component "input" type="text" autofocus=true}}
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerHashPair',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <div autofocus>
+        </div>
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <h1 autofocus>
+        </h1>
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <input autofocus="autofocus" />
       </template>`,
       errors: [
         {
+          messageId: 'noAutofocus',
           type: 'GlimmerAttrNode',
         },
       ],

From 212c011c8eeb1e0dae758d59c5500b79ccac66f8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Fri, 13 Mar 2026 01:29:20 +0100
Subject: [PATCH 2/6] Restore autofix for element attributes, keep
 MustacheStatement handler

The original isFixable:false was just a documentation tag, not a design
choice. Restore the fixer for element attributes while keeping the new
MustacheStatement handler for {{input autofocus=true}} patterns.
---
 lib/rules/template-no-autofocus-attribute.js      | 15 ++++++++++++++-
 .../lib/rules/template-no-autofocus-attribute.js  |  6 ++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/lib/rules/template-no-autofocus-attribute.js b/lib/rules/template-no-autofocus-attribute.js
index db3465c1be..34acbbe2ed 100644
--- a/lib/rules/template-no-autofocus-attribute.js
+++ b/lib/rules/template-no-autofocus-attribute.js
@@ -9,7 +9,7 @@ module.exports = {
       strictGts: true,
       url: 'https://github.com/ember-cli/eslint-plugin-ember/tree/master/docs/rules/template-no-autofocus-attribute.md',
     },
-    fixable: null,
+    fixable: 'code',
     schema: [],
     messages: {
       noAutofocus:
@@ -26,6 +26,19 @@ module.exports = {
           context.report({
             node: autofocusAttr,
             messageId: 'noAutofocus',
+            fix(fixer) {
+              const sourceCode = context.sourceCode;
+              const text = sourceCode.getText();
+              const attrStart = autofocusAttr.range[0];
+              const attrEnd = autofocusAttr.range[1];
+
+              let removeStart = attrStart;
+              while (removeStart > 0 && /\s/.test(text[removeStart - 1])) {
+                removeStart--;
+              }
+
+              return fixer.removeRange([removeStart, attrEnd]);
+            },
           });
         }
       },
diff --git a/tests/lib/rules/template-no-autofocus-attribute.js b/tests/lib/rules/template-no-autofocus-attribute.js
index d3e6b38dac..c5ba4f9656 100644
--- a/tests/lib/rules/template-no-autofocus-attribute.js
+++ b/tests/lib/rules/template-no-autofocus-attribute.js
@@ -29,6 +29,9 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         <input type="text" autofocus />
       </template>`,
+      output: `<template>
+        <input type="text"/>
+      </template>`,
       errors: [
         {
           message:
@@ -41,6 +44,9 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         <textarea autofocus></textarea>
       </template>`,
+      output: `<template>
+        <textarea></textarea>
+      </template>`,
       errors: [
         {
           type: 'GlimmerAttrNode',

From 1a4fb5aa33210d3f61c00dfb5cadb23e9cc39daf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Fri, 13 Mar 2026 01:40:47 +0100
Subject: [PATCH 3/6] Add output assertions for all autofixable test cases

---
 tests/lib/rules/template-no-autofocus-attribute.js | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/tests/lib/rules/template-no-autofocus-attribute.js b/tests/lib/rules/template-no-autofocus-attribute.js
index c5ba4f9656..06bf02229d 100644
--- a/tests/lib/rules/template-no-autofocus-attribute.js
+++ b/tests/lib/rules/template-no-autofocus-attribute.js
@@ -80,6 +80,10 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
         <div autofocus>
         </div>
       </template>`,
+      output: `<template>
+        <div>
+        </div>
+      </template>`,
       errors: [
         {
           messageId: 'noAutofocus',
@@ -92,6 +96,10 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
         <h1 autofocus>
         </h1>
       </template>`,
+      output: `<template>
+        <h1>
+        </h1>
+      </template>`,
       errors: [
         {
           messageId: 'noAutofocus',
@@ -103,6 +111,9 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         <input autofocus="autofocus" />
       </template>`,
+      output: `<template>
+        <input />
+      </template>`,
       errors: [
         {
           messageId: 'noAutofocus',

From 465571f9b6155def836c42a0b86414817f744f34 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Fri, 13 Mar 2026 07:58:41 +0100
Subject: [PATCH 4/6] Add output: null to test cases for consistent-output lint
 rule

---
 .claude/settings.local.json                   |  34 ++
 FIX.md                                        | 102 ++++
 REVIEW.md                                     | 470 ++++++++++++++++++
 .../rules/template-no-autofocus-attribute.js  |   2 +
 4 files changed, 608 insertions(+)
 create mode 100644 .claude/settings.local.json
 create mode 100644 FIX.md
 create mode 100644 REVIEW.md

diff --git a/.claude/settings.local.json b/.claude/settings.local.json
new file mode 100644
index 0000000000..9138fee884
--- /dev/null
+++ b/.claude/settings.local.json
@@ -0,0 +1,34 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(for branch:*)",
+      "Read(//Users/johanrd/fremby/ember-template-lint-main/.git/**)",
+      "Read(//Users/johanrd/fremby/ember-template-lint-main/**)",
+      "Bash(gh api:*)",
+      "Bash(git checkout:*)",
+      "Bash(npx jest:*)",
+      "Bash(cat jest.config.js 2>/dev/null || cat jest.config.cjs 2>/dev/null || echo \"no jest config\" ; ls package.json && node -e \"const p=require\\('./package.json'\\); console.log\\(JSON.stringify\\(p.scripts?.test || 'no test script'\\)\\)\")",
+      "Bash(npx vitest:*)",
+      "Bash(git add:*)",
+      "WebFetch(domain:www.w3.org)",
+      "Bash(git diff:*)",
+      "Bash(node -e \"\nconst regexp = /\\(.*\\\\s\\)?noopener\\\\s\\(.*\\\\s\\)?noreferrer\\(\\\\s.*\\)?|\\(.*\\\\s\\)?noreferrer\\\\s\\(.*\\\\s\\)?noopener\\(\\\\s.*\\)?/;\nconsole.log\\('noopener noreferrer:', regexp.test\\('noopener noreferrer'\\)\\);\nconsole.log\\('noreferrer noopener:', regexp.test\\('noreferrer noopener'\\)\\);\nconsole.log\\('noopener', regexp.test\\('noopener'\\)\\);\nconsole.log\\('noreferrer', regexp.test\\('noreferrer'\\)\\);\nconsole.log\\('xnoopener noreferrer', regexp.test\\('xnoopener noreferrer'\\)\\);\nconsole.log\\('noopenerx noreferrer', regexp.test\\('noopenerx noreferrer'\\)\\);\nconsole.log\\('nofollow noopener noreferrer:', regexp.test\\('nofollow noopener noreferrer'\\)\\);\n\")",
+      "Bash(node -e \"\nconst regexp = /\\(.*\\\\s\\)?noopener\\\\s\\(.*\\\\s\\)?noreferrer\\(\\\\s.*\\)?|\\(.*\\\\s\\)?noreferrer\\\\s\\(.*\\\\s\\)?noopener\\(\\\\s.*\\)?/;\n// Test edge cases\nconsole.log\\('foonoopener noreferrer:', regexp.test\\('foonoopener noreferrer'\\)\\);  \nconsole.log\\('noopener foonoreferrer:', regexp.test\\('noopener foonoreferrer'\\)\\);  \nconsole.log\\('foonoreferrer noopener:', regexp.test\\('foonoreferrer noopener'\\)\\);  \nconsole.log\\('noreferrer foonoopener:', regexp.test\\('noreferrer foonoopener'\\)\\);  \n\")",
+      "Bash(node -e \"\nconst AllowedPrefix = /[a-z]/;\n// Original uses: !AllowedPrefix.test\\(firstChar\\) || isReserved\n// So if first char is lowercase AND not reserved => allowed\n// If first char is NOT lowercase OR is reserved => flagged\n\n// Check _ prefix\nconsole.log\\('_ matches:', AllowedPrefix.test\\('_'\\)\\);  // false => flagged\nconsole.log\\('A matches:', AllowedPrefix.test\\('A'\\)\\);  // false => flagged\nconsole.log\\('a matches:', AllowedPrefix.test\\('a'\\)\\);  // true => allowed\nconsole.log\\('1 matches:', AllowedPrefix.test\\('1'\\)\\);  // false => flagged\n\")",
+      "Bash(node -e \"\n// The 'token' check does: convertToStrings\\(permittedValues\\).includes\\(value\\)\n// This is case-sensitive!\n// But 'tokenlist' does: token.toLowerCase\\(\\)\n// So tokenlist is case-insensitive but token is case-sensitive\n// Let's check aria-checked token values\nconsole.log\\('aria-checked values typically: true, false, mixed'\\);\n\n// The 'token' type does NOT lowercase the value before checking\n// So aria-checked='True' would fail, but probably should pass per WAI-ARIA spec\n// WAI-ARIA 1.2 says values are case-insensitive for 'token' type too\n\")",
+      "Bash(node -e \"\n// Check the original: token type doesn't do case-insensitive matching\n// but tokenlist does .toLowerCase\\(\\) on tokens\n\n// Token type check \\(line 90\\):\n// return typeof value === 'string' && convertToStrings\\(permittedValues\\).includes\\(value\\);\n// ^ case-sensitive\n\n// Tokenlist type check \\(line 95\\):\n// value.split\\(' '\\).every\\(\\(token\\) => permittedValues.includes\\(token.toLowerCase\\(\\)\\)\\)\n// ^ case-insensitive via toLowerCase\n\n// This is inconsistent! aria-live='POLITE' would fail \\(token type\\)\n// but aria-relevant='ADDITIONS REMOVALS' might pass \\(tokenlist type\\)\n\n// However... WAI-ARIA says attribute values are case-sensitive for token types\n// Actually the HTML spec says attribute values are generally case-insensitive for enumerated types\n// but ARIA values in practice must be lowercase. So this might actually be correct.\nconsole.log\\('tokenlist is case-insensitive, token is case-sensitive'\\);\nconsole.log\\('This is a potential inconsistency in the original, but arguable'\\);\n\")",
+      "Bash(cd /Users/johanrd/fremby/ember-template-lint-main && node -e \"\nimport\\('aria-query'\\).then\\(\\({aria}\\) => {\n  const checked = aria.get\\('aria-checked'\\);\n  console.log\\('aria-checked:', JSON.stringify\\(checked\\)\\);\n  const relevant = aria.get\\('aria-relevant'\\);\n  console.log\\('aria-relevant:', JSON.stringify\\(relevant\\)\\);\n  const live = aria.get\\('aria-live'\\);\n  console.log\\('aria-live:', JSON.stringify\\(live\\)\\);\n}\\);\n\")",
+      "Bash(node -e \"\nconst { aria } = require\\('aria-query'\\);\nconst checked = aria.get\\('aria-checked'\\);\nconsole.log\\('aria-checked:', JSON.stringify\\(checked\\)\\);\nconst relevant = aria.get\\('aria-relevant'\\);\nconsole.log\\('aria-relevant:', JSON.stringify\\(relevant\\)\\);\nconst live = aria.get\\('aria-live'\\);\nconsole.log\\('aria-live:', JSON.stringify\\(live\\)\\);\n\")",
+      "Bash(node -e \"\n// token: convertToStrings\\(permittedValues\\).includes\\(value\\) => case-SENSITIVE\n// tokenlist: permittedValues.includes\\(token.toLowerCase\\(\\)\\) => case-INSENSITIVE\n\n// So aria-live='POLITE' would fail \\(token, case-sensitive\\) -- strict but reasonable\n// But aria-relevant='ADDITIONS' would pass \\(tokenlist, case-insensitive\\) -- lenient\n\n// Per WAI-ARIA spec: attribute values for tokens should be compared case-insensitively\n// in HTML context. See: https://www.w3.org/TR/wai-aria-1.2/#propcharacteristic_value\n// 'Value type: token' - The token value is case-sensitive when specified in the\n// host language. But HTML is case-insensitive for attribute values.\n\n// This is an inconsistency in the original but subtle enough that it's arguable.\n// The main concern is: token is strict \\(case-sensitive\\), tokenlist is lenient.\nconsole.log\\('Inconsistency confirmed but arguable'\\);\n\")",
+      "Bash(node -e \"\n// ORIGINAL no-invalid-role.js:\n// Line 263: ['presentation', 'none'].includes\\(roleValue\\)    <-- CASE-SENSITIVE  \n// Line 272: !VALID_ROLES.has\\(roleValue.toLowerCase\\(\\)\\)        <-- CASE-INSENSITIVE\n\n// So 'Presentation' would:\n// 1. NOT match the presentation/none check \\(line 263\\) -- allowed through\n// 2. .toLowerCase\\(\\) => 'presentation' => VALID_ROLES.has\\('presentation'\\) => true => not flagged\n\n// Result: 'Presentation' is silently accepted without checking against SEMANTIC_ELEMENTS\n// This is the confirmed case-sensitivity bug\n\n// BRANCH fix/2475:\n// Uses roleLower = role.toLowerCase\\(\\) for BOTH checks\n// So 'Presentation' on <button> would correctly be flagged\n\nconsole.log\\('Confirmed: original has case-sensitivity bug in no-invalid-role'\\);\nconsole.log\\('presentation/none check is case-sensitive, validity check is case-insensitive'\\);\nconsole.log\\('Role=Presentation on <button> silently passes both checks in the original'\\);\n\")",
+      "Bash(node -e \"\n// From the original no-invalid-aria-attributes.js line 93-96:\n// case 'tokenlist': {\n//   return \\(\n//     typeof value === 'string' &&\n//     value.split\\(' '\\).every\\(\\(token\\) => permittedValues.includes\\(token.toLowerCase\\(\\)\\)\\)\n//   \\);\n// }\n\n// aria-relevant has permitted values: ['additions', 'all', 'removals', 'text']\n// So aria-relevant='ADDITIONS' => 'additions'.toLowerCase\\(\\) => 'additions' => found => valid\n// This means the original accepts 'ADDITIONS' which per WAI-ARIA spec should be case-sensitive\n\n// But this is arguable since HTML attribute values are generally case-insensitive\n// And the spec says 'unless otherwise specified'\n\nconsole.log\\('tokenlist lowercasing in original: potential spec violation'\\);\nconsole.log\\('token NOT lowercasing in original: matches spec'\\);\nconsole.log\\('The inconsistency EXISTS but may be intentional for HTML tolerance'\\);\n\")",
+      "Bash(node -e \"\n// Original no-autofocus-attribute.js:\n// MustacheStatement\\(node\\) {\n//   const autofocusAttribute = node.hash.pairs.find\\(\\(pair\\) => pair.key === 'autofocus'\\);\n// }\n\n// Note: no null check on node.hash or node.hash.pairs\n// This could crash if node.hash is null/undefined\n\n// The branch fix/2408 adds this guard:\n// if \\(!node.hash || !node.hash.pairs\\) { return; }\n\n// This is a defensive fix, but does the original actually have this bug?\n// In ember-template-lint, MustacheStatement always has .hash with .pairs\n// So the original doesn't crash in practice, but the branch is more defensive.\n\nconsole.log\\('fix/2408 adds defensive null check - not a bug in original context'\\);\n\")",
+      "Bash(node -e \"\n// Original line 25:\n// if \\(!AllowedPrefix.test\\(firstChar\\) || isReserved\\) {\n//   this.log\\({\n//     message: isReserved ? ERROR_MESSAGE_RESERVED\\('@' + part\\) : ERROR_MESSAGE_CAPITAL,\n//   }\\);\n// }\n\n// Note: The original uses OR: !AllowedPrefix || isReserved\n// If isReserved is true, it logs the reserved message regardless of the first char\n// If !AllowedPrefix \\(first char not lowercase\\), it logs the capital message\n\n// The branch uses:\n// if \\(RESERVED.has\\(name\\)\\) { report reservedArgument }\n// else if \\(!ALLOWED_PREFIX.test\\(firstChar\\)\\) { report noCapitalArguments }\n\n// Original: @arguments => isReserved=true, !AllowedPrefix\\('a'\\)=false \n//   => condition: false || true => true, message: reserved ✓\n// Branch: RESERVED.has\\('@arguments'\\) => true => reservedArgument ✓\n\n// Original: @_foo => isReserved=false, !AllowedPrefix\\('_'\\)=true\n//   => condition: true || false => true, message: capital ✓  \n// Branch: !RESERVED => !ALLOWED_PREFIX\\('_'\\)=true => noCapitalArguments ✓\n\n// Edge case: what if something is BOTH reserved AND starts with non-lowercase?\n// In the original, @arguments starts with 'a' which IS lowercase, so this doesn't apply.\n// @args starts with 'a' too. @block starts with 'b'. @else starts with 'e'.\n// So all reserved names start with lowercase -- no overlap issue.\n\nconsole.log\\('Branch matches original behavior exactly'\\);\n\")",
+      "WebFetch(domain:raw.githubusercontent.com)",
+      "Bash(git stash:*)",
+      "Bash(git worktree:*)",
+      "Bash(npx eslint:*)",
+      "Bash(npx prettier:*)"
+    ]
+  }
+}
diff --git a/FIX.md b/FIX.md
new file mode 100644
index 0000000000..1f07d95435
--- /dev/null
+++ b/FIX.md
@@ -0,0 +1,102 @@
+# Post-merge review of extracted template-lint rules
+
+Line-by-line comparison against the original [ember-template-lint](https://github.com/ember-template-lint/ember-template-lint) source at commit [`f43c6f1`](https://github.com/ember-template-lint/ember-template-lint/tree/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b) (v7.9.3). Each section documents a discrepancy found in the extraction and the fix applied. All branches have passing tests.
+
+---
+
+## Post-merge review of #2381 (`template-no-log`)
+
+Missing scope tracking for block parameters causes false positives when `log` is a block param name (e.g., `{{#each this.logs as |log|}}{{log}}{{/each}}`). The original checks [`this.isLocal(node)`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-log.js#L7) before reporting. Fix adds a `localScopes` stack with `GlimmerBlockStatement`/`GlimmerElementNode` enter/exit pairs, matching the pattern already used in `template-no-dynamic-subexpression-invocations`.
+
+---
+
+## Post-merge review of #2395 (`template-link-href-attributes`)
+
+Missing exception for `<a>` elements with both `role` and `aria-disabled` attributes, causing false positives on accessible patterns like `<a role="link" aria-disabled="true">`. The original skips reporting when both are present — see [`link-href-attributes.js` lines 27–29](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-href-attributes.js#L27-L29).
+
+---
+
+## Post-merge review of #2396 (`template-link-rel-noopener`)
+
+Two issues. (1) Detection regex uses bare `/noopener/` which matches substrings. The original uses a [whitespace-bounded regex](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-rel-noopener.js#L24). Fix uses `/(?:^|\s)noopener(?:\s|$)/`. (2) Fixer always inserts a new `rel` attribute even when one already exists, producing duplicate attributes. The [original's fixer](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-rel-noopener.js#L99-L102) strips existing `noopener`/`noreferrer` and re-adds them in canonical order. Fix now does the same.
+
+---
+
+## Post-merge review of #2408 (`template-no-autofocus-attribute`)
+
+Missing `MustacheStatement` visitor. The original handles both [`ElementNode`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-autofocus-attribute.js#L21-L26) and [`MustacheStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-autofocus-attribute.js#L28-L32) (for `{{input autofocus=true}}`). The extraction only had `ElementNode`, so `{{input autofocus=true}}` and `{{component "input" autofocus=true}}` silently passed.
+
+---
+
+## Post-merge review of #2414 (`template-no-capital-arguments`)
+
+Three missing features. The original has both a [`PathExpression`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L19-L33) and an [`AttrNode`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L34-L46) visitor, plus a [`RESERVED` set](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L9) and [`AllowedPrefix` regex](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L8). The extraction was missing: (1) `AttrNode` visitor — `<Foo @Name="bar" />` wasn't caught, (2) reserved argument detection (`@arguments`, `@args`, `@block`, `@else`), (3) underscore-prefix detection (`@_Name`).
+
+---
+
+## Post-merge review of #2423 (`template-no-action`)
+
+Two bugs. (1) Missing [`ElementModifierStatement` visitor](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action.js#L46-L48) — `<button {{action "submit"}}>` silently passed. (2) The original's [`MustacheStatement` visitor](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action.js#L40-L42) reports `{{action}}` everywhere including attribute position (`<input onclick={{action "foo"}}>`), but the extraction incorrectly walked parents to skip `GlimmerAttrNode`.
+
+---
+
+## Post-merge review of #2424 (`template-no-action-modifiers`)
+
+Missing allowlist configuration. The original supports an [`allowlist` config](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action-modifiers.js#L17-L19) (passed as an array, e.g., `['button']`) that skips reporting when the parent element's tag is in the list — see the [allowlist guard](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action-modifiers.js#L48). The extraction had `schema: []` with no config support.
+
+---
+
+## Post-merge review of #2429 (`template-no-element-event-actions`)
+
+Over-reporting due to erroneous `ConcatStatement` check. The original only checks [`node.value.type === 'MustacheStatement'`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-element-event-actions.js#L61). The extraction also checked `GlimmerConcatStatement`.
+
+---
+
+## Post-merge review of #2442 (`template-no-inline-styles`)
+
+The original's `allowDynamicStyles` exemption only checks [`style.value.type === 'MustacheStatement'`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-inline-styles.js#L62). The extraction incorrectly also exempted `GlimmerConcatStatement` (e.g., `style="{{foo}} bar"`), which the original flags as invalid. Also fixes the same ConcatStatement issue in `template-no-element-event-actions`.
+
+---
+
+## Post-merge review of #2469 (`template-no-implicit-this`)
+
+Missing `welcome-page` and `rootURL` in built-ins. The original includes [`ARGLESS_DEFAULT_BLUEPRINT`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-implicit-this.js#L48-L51) containing these in its allow list. Also fixes the ConcatStatement issue in `template-no-element-event-actions`.
+
+---
+
+## Post-merge review of #2471 (`template-no-invalid-aria-attributes`)
+
+Error messages don't match original's [`createInvalidAttributeTypeErrorMessage`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L11-L38). Specifically: (1) `boolean` type produced `"a boolean (true or false)"` but the original's [default case](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L36) produces `"a boolean."`. (2) [`id`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L30) and [`idlist`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L27) have no trailing period in the original, but the extraction's message template added one.
+
+---
+
+## Post-merge review of #2475 (`template-no-invalid-role`)
+
+Three issues. (1) Missing roles (`associationlistitemkey`, `associationlistitemvalue`, `cell`) in `VALID_ROLES`. (2) `SEMANTIC_ELEMENTS` set doesn't match the original's [`ELEMENTS_DISALLOWING_PRESENTATION_OR_NONE_ROLE`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L16-L102). (3) Missing `.toLowerCase()` for the [`VALID_ROLES.has()` check](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L272).
+
+**Intentional improvement over original:** The `presentation`/`none` check also uses `.toLowerCase()`, while the [original is case-sensitive there](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L263). The original has an internal inconsistency: the validity check uses `.toLowerCase()` but the presentation/none check does not, meaning `role="NONE"` on a `<button>` silently passes. Per the [WAI-ARIA 1.2 spec](https://www.w3.org/TR/wai-aria-1.2/#document-handling_author-errors_roles): *"Case-sensitivity of the comparison inherits from the case-sensitivity of the host language"* — and HTML is case-insensitive.
+
+---
+
+## Post-merge review of #2477 (`template-no-link-to-positional-params`)
+
+Incorrect `GlimmerElementNode` handler. The original only handles [`MustacheStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-link-to-positional-params.js#L9) and [`BlockStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-link-to-positional-params.js#L13) (curly-brace `{{link-to}}`). The extraction added a `GlimmerElementNode` handler that tries to detect positional params on `<LinkTo>` angle-bracket invocations using a heuristic with no basis in the original.
+
+---
+
+## Post-merge review of #2484 (`template-no-nested-interactive`)
+
+Config option name mismatch. The original uses [`ignoreUsemapAttribute`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-nested-interactive.js#L166). The extraction renamed it to `ignoreUsemap`. Fix accepts both names for migration compatibility.
+
+---
+
+## Post-merge review of #2490 (`template-no-outlet-outside-routes`)
+
+Missing `BlockStatement` handler. The original registers both [`MustacheStatement` and `BlockStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-outlet-outside-routes.js#L26-L28) for `_checkForOutlet`. The extraction only had `GlimmerMustacheStatement`, so `{{#outlet}}content{{/outlet}}` in non-route templates silently passed.
+
+---
+
+## Notes
+
+- **Overlapping fix:** #2429, #2442, and #2469 all contain the same `template-no-element-event-actions` ConcatStatement removal. They will conflict — merge #2442 first (bundles both rules' fixes + tests) and rebase the others.
+- 14 branches faithfully reproduce the original ember-template-lint behavior; #2475 intentionally improves one case-sensitivity inconsistency backed by the WAI-ARIA spec.
diff --git a/REVIEW.md b/REVIEW.md
new file mode 100644
index 0000000000..b2fd2b4113
--- /dev/null
+++ b/REVIEW.md
@@ -0,0 +1,470 @@
+# Extract Rule PRs - Systematic Review
+
+> Comparing each PR against its ember-template-lint counterpart in `/Users/johanrd/fremby/ember-template-lint-main`.
+> Goal: in gjs/gts projects, these rules must behave equivalently so ember-template-lint is no longer needed.
+
+## Overview
+
+| PR | Rule | Status | Key Issues |
+|----|------|--------|------------|
+| **Batch 1** | | | |
+| #2477 | template-no-link-to-positional-params | **NEEDS WORK** | Unnecessary `<LinkTo>` handler (can false-positive on `class={{...}}`); static error message loses migration advice |
+| #2475 | template-no-invalid-role | **NEEDS WORK** | **Bug:** missing `.toLowerCase()` on role values — `role="Button"` falsely flagged; ~19 elements missing from semantic set |
+| #2474 | template-no-invalid-meta | **NEEDS WORK** | New `charset` validation not in original (enhancement, not extraction); test gaps around charset logic |
+| #2473 | template-no-invalid-link-title | **NEEDS WORK** | Single generic message loses "both attribute and argument" context; missing strict-mode guard for `LinkTo` |
+| #2472 | template-no-invalid-link-text | **NEEDS WORK** | Nested element traversal differs from original; doc self-reference bug |
+| #2471 | template-no-invalid-aria-attributes | **NEEDS WORK** | Generic error messages lose type-specific guidance; stricter numeric validation (improvement) |
+| **Batch 2** | | | |
+| #2470 | template-no-input-block | **PASS** | Minor message text difference (cosmetic) |
+| #2469 | template-no-implicit-this | **NEEDS WORK** | `isControlFlowParam` wrongly skips flagging ambiguous params; missing RegExp allow; incomplete block param detection; missing default allow entries |
+| #2468 | template-no-extra-mut-helper-argument | **PASS** | Clean extraction, identical behavior |
+| #2467 | template-no-dynamic-subexpression-invocations | **NEEDS WORK** | Flags body-context `this.*` mustaches not flagged by original; generic error message loses context-specific guidance |
+| #2465 | template-linebreak-style | **PASS** | Minor: missing `true` (consistency) mode and editorconfig integration; adds fixer (improvement) |
+| #2464 | template-eol-last | **PASS** | Minor: missing `editorconfig` config option |
+| **Batch 3** | | | |
+| #2462 | template-block-indentation | **NEEDS WORK** | Missing `validateBlockStart`; no `columnOffset` handling |
+| #2461 | template-attribute-indentation | **NEEDS WORK** | Missing `validateClosingTag`; `processElements` defaults differ |
+| #2460 | template-no-curly-component-invocation | **NEEDS WORK** | `noImplicitThis` accepted in schema but never used; multi-part paths not handled |
+| #2459 | template-no-class-bindings | **PASS** | Slightly stricter (catches non-`@` attrs too) — reasonable enhancement |
+| #2457 | template-no-bare-yield | **NEEDS WORK** | Fundamentally different semantics from `no-yield-only` |
+| #2456 | template-no-at-ember-render-modifiers | **NEEDS WORK** (minor) | Misleading error message recommends the very modifiers being banned |
+| **Batch 4** | | | |
+| #2455 | template-no-args-paths | **NEEDS WORK** | No `isLocal` check (false positives on block params named `args`); generic error message |
+| #2454 | template-no-ambiguous-glimmer-paths | **NEEDS WORK** | Dramatically narrower than `no-implicit-this`; only dotted paths in mustaches; no `allow` config |
+| #2453 | template-no-action-on-submit-button | **PASS** | Solid implementation, correct logic, good test coverage |
+| #2452 | template-modifier-name-case | **NEEDS WORK** | `dasherize` produces leading dashes for capitalized names; unused broken `messageId` |
+| #2451 | template-deprecated-render-helper | **NEEDS WORK** (minor) | No fix/suggestion behavior (original auto-fixes); slight message differences |
+| #2450 | template-deprecated-inline-view-helper | **NEEDS WORK** | No `isLocal` check for block params named `view`; no fix; possible double-reporting |
+| **Batch 5** | | | |
+| #2441 | template-no-inline-linkto | **NEEDS WORK** | Scope expanded beyond original (adds `<LinkTo>` angle bracket checks); missing autofix |
+| #2439 | template-no-index-component-invocation | **PASS** | Minor: verify `.value` vs `.original` for string literals |
+| #2438 | template-no-html-comments | **NEEDS WORK** | `originallyFrom.name` bug (says `eslint-plugin-ember` instead of `ember-template-lint`); different detection approach |
+| #2436 | template-no-form-action | **NEEDS WORK** | **No ember-template-lint counterpart exists**; questionable rule semantics |
+| #2435 | template-no-forbidden-elements | **NEEDS WORK** | Missing object config form `{ forbidden: [...] }`; missing `head.hbs` exception |
+| #2432 | template-no-extra-mut-helpers (Draft) | **CRITICAL** | **Rule logic is inverted** — flags correct `(mut value)` usage instead of `(mut a b)` |
+| **Batch 6** | | | |
+| #2427 | template-no-duplicate-landmark-elements | **NEEDS WORK** | Missing dialog/popover scoping; incomplete block scoping (only `if`/`unless`) |
+| #2426 | template-no-duplicate-id | **NEEDS WORK** | Missing ancestor yield block resolution; potential false positives for block-param IDs |
+| #2418 | template-no-down-event-binding (Draft) | **CRITICAL** | Wrong event set (missing `pointerdown`, incorrectly added `touchstart`); missing `action` modifier handling |
+| #2413 | template-no-builtin-form-components (Draft) | **PASS** | Simple, correct port |
+| #2409 | template-no-bare-strings | **NEEDS WORK** | Complex rule (371 lines in original); needs manual verification of full implementation |
+
+---
+
+## Critical Issues (must fix before merge)
+
+### #2432 — template-no-extra-mut-helpers: Inverted logic
+The ember-template-lint `no-extra-mut-helper-argument` flags `(mut attr extraArg)` (too many arguments). The PR flags `(mut this.value)` (one argument = correct usage). **The condition is backwards.** Should check `params.length > 1`, not `params.length === 1`.
+
+### #2418 — template-no-down-event-binding: Wrong event set
+- **Missing:** `pointerdown`, `onpointerdown` (the whole point of the original rule name `no-pointer-down-event-binding`)
+- **Incorrectly added:** `touchstart` (not in original)
+- Missing `{{action}}` modifier handling
+- Missing case-insensitive matching
+- Error message recommends wrong alternatives
+
+### #2457 — template-no-bare-yield: Different rule entirely
+- ember-template-lint `no-yield-only` only flags when `{{yield}}` is the **only content** in the entire template
+- PR flags **any** `{{yield}}` without parameters regardless of context
+- `<div>{{yield}}</div>` would pass in ember-template-lint but fail in the PR
+
+### #2475 — template-no-invalid-role: Missing case-insensitive role matching
+Role values are case-insensitive per the ARIA spec. The PR checks `VALID_ROLES.has(role)` without `.toLowerCase()`. This means `role="Button"` or `role="NAVIGATION"` will be falsely flagged as invalid. The original correctly lowercases before lookup.
+
+### #2477 — template-no-link-to-positional-params: Unnecessary `<LinkTo>` handler
+Angle-bracket `<LinkTo>` never supported positional params — you write `<LinkTo @route="about">`. The original deliberately does NOT check `<LinkTo>`. The PR's `GlimmerElementNode` handler checks non-`@` attrs with dynamic values, which could false-positive on `class={{...}}`, `id={{...}}`, `data-test={{...}}`, etc.
+
+### #2436 — template-no-form-action: No counterpart
+No corresponding rule exists in ember-template-lint. The HTML `action` attribute on `<form>` is standard HTML, not an Ember concern. Should not be presented as an "extraction."
+
+---
+
+## Detailed Reviews
+
+---
+
+### PR #2477 — template-no-link-to-positional-params
+**ember-template-lint rule:** `no-link-to-positional-params` | **Status: NEEDS WORK**
+
+Detects deprecated positional parameter usage in `{{link-to}}` / `{{#link-to}}`.
+
+**Issues:**
+1. **Unnecessary `<LinkTo>` handler:** Angle-bracket `<LinkTo>` never supported positional params. The original deliberately omits this check. The PR's heuristic (`!attr.name.startsWith('@') && attr.value.type !== 'GlimmerTextNode'`) could false-positive on legitimate HTML attributes like `class={{dynamicClass}}`. **Remove the `GlimmerElementNode` handler entirely.**
+2. **Static error message:** Original dynamically includes which named arguments to use (`@route`, `@model`/`@models`, `@query`) and advises using block form. PR uses a single generic string.
+3. **Misleading docs:** Example shows `<LinkTo "posts.post" @model={{this.post}}>` as incorrect — but this isn't valid Ember syntax. Angle-bracket components can't receive positional string params.
+4. **Zero tests for `<LinkTo>` code path** — ~30 lines of rule code are untested.
+
+---
+
+### PR #2475 — template-no-invalid-role
+**ember-template-lint rule:** `no-invalid-role` | **Status: NEEDS WORK**
+
+Validates ARIA roles and flags `presentation`/`none` on semantic elements.
+
+**Issues:**
+1. **Bug: Missing `.toLowerCase()`:** ARIA role values are case-insensitive per spec. PR checks `VALID_ROLES.has(role)` without lowercasing. `role="Button"` is falsely flagged.
+2. **~19 elements missing from `SEMANTIC_ELEMENTS`:** `audio`, `video`, `iframe`, `embed`, `area`, `br`, `source`, `track`, `wbr`, `object`, `param`, `map`, `applet`, `dir`, `menuitem`, `noembed`, `rb`, `rtc`, `tt` — all present in original but absent from PR.
+3. **Elements added not in original:** `address`, `article`, `aside`, `footer`, `h1`-`h6`, `header`, `hgroup`, `nav`, `section` — reasonable additions but behavioral divergence.
+4. **Empty/whitespace role silently ignored:** PR trims + skips empty roles; original would flag `role="  "` as invalid.
+5. **Check order reversed with early return:** Invalid-role reports first then returns, preventing the presentation check from ever firing on the same node. Original runs both checks independently.
+
+---
+
+### PR #2474 — template-no-invalid-meta
+**ember-template-lint rule:** `no-invalid-meta` | **Status: NEEDS WORK**
+
+Validates `<meta>` elements (content requirements, viewport, refresh).
+
+**Issues:**
+1. **New `invalidCharset` check:** Validates `<meta charset="...">` must be `utf-8`. This does not exist in ember-template-lint. Should be documented as an enhancement, not a straight extraction.
+2. **`hasIdentifier` includes `hasHttpEquiv`:** Original keeps these separate (`hasIdentifier || hasHttpEquiv`). Functionally equivalent but semantically misleading.
+3. **Edge case: `parseInt("", 10)` → `NaN`:** Empty `content=""` with `http-equiv="refresh"` silently passes. Should probably be flagged.
+4. **Test gaps:** Missing tests for `charset` + other attrs combos, multiple errors on single element, boundary value `delay=72000`, charset in HBS mode.
+
+---
+
+### PR #2473 — template-no-invalid-link-title
+**ember-template-lint rule:** `no-invalid-link-title` | **Status: NEEDS WORK**
+
+Disallows title attributes that duplicate link text.
+
+**Issues:**
+1. **Single generic message:** Original has distinct messages for "both attribute and argument" vs "redundant title". PR uses one message for all cases — confusing for `<LinkTo @title="foo" title="bar">`.
+2. **Missing strict-mode guard:** Original skips `LinkTo` in strict mode (`this.isStrictMode`). PR always checks it. Could produce false positives in GJS `<template>` context.
+3. **Dual-title early return:** PR returns after reporting dual-title, never checking if the title also duplicates link text. Original can report both errors.
+4. **Missing tests:** Empty `title` on `<LinkTo>`, empty `@title`, title-as-substring-of-text.
+
+---
+
+### PR #2472 — template-no-invalid-link-text
+**ember-template-lint rule:** `no-invalid-link-text` | **Status: NEEDS WORK**
+
+Disallows generic link text ("click here", "more info", "read more", "more").
+
+**Issues:**
+1. **Nested element traversal:** PR recursively traverses child elements (e.g., `<a><span>click here</span></a>`). Original only checks direct `TextNode` children. This is a behavioral difference — arguably better but should be tested and documented.
+2. **Doc self-reference bug:** References section links to itself instead of the ember-template-lint rule.
+3. **Missing tests:** `allowEmptyLinks` in GJS mode, standalone `"more"` test, `aria-label` with disallowed value on `<LinkTo>`, nested element content.
+
+---
+
+### PR #2471 — template-no-invalid-aria-attributes
+**ember-template-lint rule:** `no-invalid-aria-attributes` | **Status: NEEDS WORK**
+
+Validates ARIA attribute names and value types using `aria-query`.
+
+**Issues:**
+1. **Generic error messages:** Original provides type-specific messages (e.g., "must be a boolean or the string 'mixed'", "must be an integer"). PR uses `"Invalid value for ARIA attribute aria-checked."` — less actionable.
+2. **Stricter numeric validation (improvement):** PR uses `Number()` instead of `parseInt` — `"50px"` correctly rejected. Also distinguishes `integer` vs `number` types. These are improvements over the original.
+3. **`string` type accepts booleans:** PR accepts `aria-label="true"`, original rejects it. Minor difference.
+4. **Reports all errors per element:** Original stops after first invalid attribute. PR reports all. Better for ESLint UX.
+5. **Missing tests:** `aria-hidden` without value, empty string values, decimal for `number` type, negative integers.
+
+---
+
+### PR #2453 — template-no-action-on-submit-button
+**ember-template-lint rule:** `no-input-block` | **Status: PASS**
+
+Disallows block usage of `{{input}}` helper. Simple rule, correctly ported.
+
+- **Error message:** Minor cosmetic difference — PR omits `{{}}` around "input" in message text. Acceptable.
+- **Schema/fix/test coverage:** All match. No configuration, no fix, good coverage.
+
+---
+
+### PR #2469 — template-no-implicit-this
+**ember-template-lint rule:** `no-implicit-this` | **Status: NEEDS WORK**
+
+Requires explicit `this.` or `@` prefix for property access in templates.
+
+**Issues:**
+1. **`isControlFlowParam` wrongly skips flagging:** `{{if ambiguousPath "yes"}}` — `ambiguousPath` should be flagged (ember-template-lint flags it) but the PR skips it via `isControlFlowParam`.
+2. **Missing default allow entries:** ember-template-lint always includes `welcome-page` and `rootURL` in the allow list.
+3. **No RegExp support in `allow`:** ember-template-lint supports both strings and RegExp values. PR only allows strings.
+4. **Incomplete block param detection:** `isBlockParamPath` only checks direct parent, not ancestor scopes.
+5. **Missing SubExpression/ElementModifierStatement callee handling:** Callees of sub-expressions like `(some-helper ...)` could be incorrectly flagged.
+6. **Shorter error message:** Omits guidance about angle bracket invocation and rule configuration.
+
+---
+
+### PR #2468 — template-no-extra-mut-helper-argument
+**ember-template-lint rule:** `no-extra-mut-helper-argument` | **Status: PASS**
+
+Disallows passing more than one argument to `mut`. Simple rule, correctly ported.
+
+- Error message identical.
+- Minor style note: uses inline `message` instead of `messageId`. Functional but inconsistent with other PRs.
+
+---
+
+### PR #2467 — template-no-dynamic-subexpression-invocations
+**ember-template-lint rule:** `no-dynamic-subexpression-invocations` | **Status: NEEDS WORK**
+
+Disallows dynamic helper/modifier invocations with dynamic callee paths.
+
+**Issues:**
+1. **Behavioral extension:** `{{this.formatter this.data}}` in body context is flagged by PR but NOT by ember-template-lint (original only flags attr-position mustaches with dynamic callees).
+2. **Generic error message:** Original gives context-specific messages (e.g., "You must use the `fn` helper" for attr-position). PR uses one generic message.
+3. **Local scope tracking diverges:** PR implements own `localScopes` stack vs ember-template-lint's `this.scope.isLocal()`.
+
+---
+
+### PR #2465 — template-linebreak-style
+**ember-template-lint rule:** `linebreak-style` | **Status: PASS**
+
+Enforces consistent linebreaks (LF vs CRLF).
+
+- **Improvement:** PR adds a fixer (ember-template-lint has none).
+- **Minor gaps:** Missing `true` (auto-detect consistency) mode. Missing editorconfig `end_of_line` integration.
+
+---
+
+### PR #2464 — template-eol-last
+**ember-template-lint rule:** `eol-last` | **Status: PASS**
+
+Requires or disallows newline at end of template.
+
+- Error messages identical.
+- Fix behavior equivalent.
+- Correctly skips gjs/gts embedded templates.
+- **Minor gap:** Missing `editorconfig` option (`insert_final_newline`).
+
+---
+
+### PR #2462 — template-block-indentation
+**ember-template-lint rule:** `block-indentation` | **Status: NEEDS WORK**
+
+Enforces consistent block indentation.
+
+**Issues:**
+1. **Missing `validateBlockStart`:** Top-level block/element indentation relative to template root is not validated.
+2. **No `columnOffset` handling:** ember-template-lint adjusts for embedded template offsets. PR does not, which could cause issues in certain nesting scenarios.
+3. **Missing `sourceDoesNotMatchNode` guard:** Safety check against false positives is absent.
+
+Configuration, error messages, and test coverage are otherwise equivalent.
+
+---
+
+### PR #2461 — template-attribute-indentation
+**ember-template-lint rule:** `attribute-indentation` | **Status: NEEDS WORK**
+
+Enforces proper attribute indentation on multi-line invocations.
+
+**Issues:**
+1. **Missing `validateClosingTag`:** Misaligned `</div>` closing tags won't be caught.
+2. **`processElements` defaults to `true`:** ember-template-lint requires explicit opt-in. PR default could cause unexpected errors during migration.
+3. **Missing close-brace validation for paramless blocks with block params.**
+
+
+
+### PR #2459 — template-no-class-bindings
+**ember-template-lint rule:** `no-class-bindings` | **Status: PASS**
+
+Disallows `classBinding`/`classNameBindings` arguments.
+
+- Slightly stricter than original (also catches non-`@` prefixed attrs on angle bracket components). Reasonable enhancement.
+- Good test coverage of all 4 forbidden patterns.
+
+---
+
+### PR #2457 — template-no-bare-yield
+**ember-template-lint rule:** `no-yield-only` | **Status: NEEDS WORK** (critical semantic mismatch)
+
+See [Critical Issues](#critical-issues-must-fix-before-merge) above. The PR flags any `{{yield}}` without params. The original only flags templates whose **entire content** is just `{{yield}}`.
+
+---
+
+### PR #2456 — template-no-at-ember-render-modifiers
+**ember-template-lint rule:** `no-at-ember-render-modifiers` | **Status: NEEDS WORK** (minor)
+
+Disallows `did-insert`, `did-update`, `will-destroy` modifiers.
+
+**Issue:** Error message says *"Use (did-insert), (did-update), or (will-destroy) from ember-render-helpers instead"* — recommending the very modifiers being banned. ember-template-lint correctly says to use a custom modifier via `ember-modifier`.
+
+
+
+### PR #2450 — template-deprecated-inline-view-helper
+**ember-template-lint rule:** `deprecated-inline-view-helper` | **Status: NEEDS WORK**
+
+Disallows deprecated inline `{{view}}` helper and `view.*` paths.
+
+**Issues:**
+1. **No `isLocal` check:** `{{#each items as |view|}}{{view.name}}{{/each}}` would be falsely flagged.
+2. **Missing dedup guard:** No `_processedByInlineViewHelper` equivalent — same node could be reported twice (once from element visitor, once from mustache visitor).
+3. **No autofix:** Original provides fix suggestions (removing `view.` prefix).
+
+---
+
+### PR #2441 — template-no-inline-linkto
+**ember-template-lint rule:** `inline-link-to` | **Status: NEEDS WORK**
+
+**Issues:**
+1. **Scope expanded beyond original:** PR also checks angle bracket `<LinkTo />` with empty children — ember-template-lint only checks curly `{{link-to ...}}`.
+2. **Missing autofix:** Original converts inline to block form.
+3. **Message differs.**
+
+---
+
+### PR #2439 — template-no-index-component-invocation
+**ember-template-lint rule:** `no-index-component-invocation` | **Status: PASS**
+
+Disallows explicit `/index` or `::Index` suffix on component invocations.
+
+- All 4 node types covered (element, mustache, block, sub-expression).
+- Dynamic error messages match.
+- Good test coverage including nested namespaces and false positives.
+- **Minor:** Verify `.value` vs `.original` property for string literal params.
+
+---
+
+### PR #2438 — template-no-html-comments
+**ember-template-lint rule:** `no-html-comments` | **Status: NEEDS WORK**
+
+**Issues:**
+1. **`originallyFrom.name` bug:** Set to `'eslint-plugin-ember'` instead of `'ember-template-lint'`.
+2. **Different detection mechanism:** Uses `sourceCode.getAllComments()` + raw source check instead of `CommentStatement` visitor. Reasonable for ESLint but different approach.
+3. **Missing config comment filtering:** ember-template-lint skips `<!-- template-lint-disable -->` comments. Less relevant for ESLint context but worth noting.
+4. **Message differs:** `"HTML comment detected"` → `"HTML comments should not be used in templates..."`.
+
+---
+
+### PR #2436 — template-no-form-action
+**No ember-template-lint counterpart** | **Status: NEEDS WORK**
+
+See [Critical Issues](#critical-issues-must-fix-before-merge). No corresponding rule exists. The HTML `action` attribute on `<form>` is standard HTML. This is a new rule, not an extraction.
+
+---
+
+### PR #2432 — template-no-extra-mut-helpers (Draft)
+**ember-template-lint rule:** `no-extra-mut-helper-argument` | **Status: CRITICAL**
+
+See [Critical Issues](#critical-issues-must-fix-before-merge). Logic is inverted — flags correct single-argument usage instead of incorrect multi-argument usage.
+
+---
+
+### PR #2427 — template-no-duplicate-landmark-elements
+**ember-template-lint rule:** `no-duplicate-landmark-elements` | **Status: NEEDS WORK**
+
+**Issues:**
+1. **Missing dialog/popover scoping:** Landmarks inside `<dialog>` or `[popover]` elements should be treated as separate scope.
+2. **Incomplete block scoping:** Only `if`/`unless` blocks get scope treatment; `each`, `let`, component blocks do not.
+3. **Added `section` element** (not in original — `<section>` → `region` role).
+4. **Different reporting approach:** Batch analysis at `Program:exit` vs eager per-element reporting.
+
+---
+
+### PR #2426 — template-no-duplicate-id
+**ember-template-lint rule:** `no-duplicate-id` | **Status: NEEDS WORK**
+
+**Issues:**
+1. **Missing ancestor yield block resolution:** ember-template-lint's `getMustacheValue` differentiates `{{inputProperties.id}}` across different `<MyComponent>` invocations. PR uses `sourceCode.getText()` which would produce identical strings → false positives.
+2. **Extra `conditionalReportedDuplicates` logic** not in original — may suppress valid reports.
+
+---
+
+
+### PR #2418 — template-no-down-event-binding (Draft)
+**ember-template-lint rule:** `no-pointer-down-event-binding` | **Status: CRITICAL**
+
+See [Critical Issues](#critical-issues-must-fix-before-merge). Wrong event set, missing modifier handling, wrong error message.
+
+---
+
+### PR #2413 — template-no-builtin-form-components (Draft)
+**ember-template-lint rule:** `no-builtin-form-components` | **Status: PASS**
+
+Simple rule checking `<Input>` and `<Textarea>`. Error messages identical. No configuration. Both gjs and hbs tested.
+
+---
+
+### PR #2419 — template-no-duplicate-attributes
+**ember-template-lint rule:** `no-duplicate-attributes` | **Status: PASS**
+
+Clean port. All 4 node types covered (element, mustache, block, sub-expression). More efficient `Map`-based algorithm vs original's O(n^2). Fix uses ESLint's `fixer.removeRange` correctly. Good test coverage.
+
+---
+
+### PR #2409 — template-no-bare-strings
+**ember-template-lint rule:** `no-bare-strings` | **Status: NEEDS WORK**
+
+Complex rule (371 lines in ember-template-lint). Areas requiring manual verification:
+- `isStrictMode` handling for `Input`/`Textarea` built-in component attributes
+- `page-title`/`if`/`unless`/`concat` helper string checking
+- Default allowlist completeness (67 entries in original)
+- `elementAttributes` and `globalAttributes` handling
+- Object and array config forms
+
+---
+
+## Statistics
+
+| Status | Count |
+|--------|-------|
+| **PASS** | 9 |
+| **NEEDS WORK** | 23 |
+| **CRITICAL** | 4 |
+| **Total** | 36 |
+
+## Loose-Mode-Only Rules (irrelevant for .gts/.gjs)
+
+Some of the 36 PRs extract rules that only apply to loose-mode (classic `.hbs`) templates and are **irrelevant for strict-mode `.gjs`/`.gts` files**. In strict mode, the Glimmer compiler enforces these constraints at compile time, or the patterns simply cannot occur.
+
+### Explicitly `templateMode: 'loose'` in the PR branches
+
+Verified by checking the actual PR branch code:
+
+| PR | Rule | Why loose-only |
+|----|------|---------------|
+| #2469 | template-no-implicit-this | Strict mode enforces explicit `this`/`@` at parser level. ember-template-lint also returns early when `isStrictMode`. |
+| #2460 | template-no-curly-component-invocation | Curly component invocation is impossible in strict mode |
+| #2459 | template-no-class-bindings | `classBinding`/`classNameBindings` are a loose-mode-only API |
+| #2470 | template-no-input-block | Block `{{input}}` is a loose-mode-only built-in |
+| #2468 | template-no-extra-mut-helper-argument | `mut` helper is a loose-mode-only helper |
+
+### Explicitly `templateMode: 'both'` in the PR branches
+
+These are confirmed as applying to both loose and strict mode:
+
+`template-no-at-ember-render-modifiers`, `template-no-bare-yield`, `template-no-dynamic-subexpression-invocations`, `template-eol-last`, `template-linebreak-style`, `template-block-indentation`, `template-attribute-indentation`
+
+### Conceptually loose-only (not explicitly marked but pattern can't occur in strict mode)
+
+| PR | Rule | Why irrelevant in strict mode |
+|----|------|-------------------------------|
+| #2451 | template-deprecated-render-helper | `{{render}}` helper does not exist in strict mode |
+| #2450 | template-deprecated-inline-view-helper | `{{view}}` helper does not exist in strict mode |
+| #2441 | template-no-inline-linkto | Curly `{{link-to}}` syntax — in GJS you use `<LinkTo>` |
+| #2477 | template-no-link-to-positional-params | Positional params only possible with curly `{{link-to}}` |
+| #2454 | template-no-ambiguous-glimmer-paths | Docs say HBS only; strict mode enforces explicit paths |
+| #2432 | template-no-extra-mut-helpers (Draft) | Docs say HBS Only; `mut` doesn't exist in strict mode |
+
+### Rules with `isStrictMode` guards in ember-template-lint
+
+These ember-template-lint rules have `isStrictMode` guards that affect behavior in GJS/GTS context. The extract PRs should replicate these guards:
+
+| Rule | Guard behavior |
+|------|---------------|
+| `no-implicit-this` | Returns early (disables itself) when `isStrictMode` |
+| `no-invalid-link-title` | Skips `LinkTo` checks when `isStrictMode` |
+| `no-invalid-link-text` | Skips `LinkTo` checks when `isStrictMode` |
+| `no-bare-strings` | Excludes `Input`/`Textarea` built-in component attributes when `isStrictMode` |
+| `block-indentation` | Adjusts standalone validation when `isStrictMode && isEmbedded` |
+| `eol-last` | Disables when `isEmbedded` (GJS templates are embedded) |
+
+**Impact on the extract PRs:** PRs #2473 and #2472 are missing these `isStrictMode` guards, which could cause false positives in GJS/GTS files. PR #2464 (eol-last) correctly handles this by skipping templates starting with `<template>`. PR #2409 (no-bare-strings) needs verification that strict-mode attribute handling is correct.
+
+### Summary
+
+Of the 36 extract rule PRs, **~11 are loose-mode only** (5 explicitly + 6 conceptually) and do not contribute to the goal of replacing ember-template-lint for GJS/GTS projects. The remaining **~25 rules are relevant for GJS/GTS** and are the ones that matter for eliminating the ember-template-lint dependency.
+
+---
+
+## Common Patterns Across PRs
+
+1. **Missing `isLocal` / block param checks** — PRs #2469, #2455, #2450 all fail to detect when a path refers to a block parameter, causing false positives.
+2. **Missing autofix** — Several ember-template-lint rules provide autofixes that are not ported: #2441, #2450, #2451, #2452 (partial).
+3. **Error message divergence** — Many PRs use shorter/generic messages losing actionable guidance from the originals (#2469, #2467, #2471, #2473, #2477).
+4. **Configuration gaps** — Several PRs don't support all config forms (RegExp in allow lists, object configs, editorconfig integration).
+5. **Scope expansion** — Some PRs flag cases the original doesn't (#2441, #2457, #2467, #2477), which would cause unexpected errors for projects migrating from ember-template-lint.
+6. **Case sensitivity bugs** — #2475 missing `.toLowerCase()` on role values; #2418 missing case-insensitive event matching.
+7. **Strict-mode guards missing** — #2473, #2472 don't implement `isStrictMode` checks that the originals use to skip `LinkTo` in strict (GJS) context.
diff --git a/tests/lib/rules/template-no-autofocus-attribute.js b/tests/lib/rules/template-no-autofocus-attribute.js
index 06bf02229d..ad67f03943 100644
--- a/tests/lib/rules/template-no-autofocus-attribute.js
+++ b/tests/lib/rules/template-no-autofocus-attribute.js
@@ -57,6 +57,7 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         {{input type="text" autofocus=true}}
       </template>`,
+      output: null,
       errors: [
         {
           messageId: 'noAutofocus',
@@ -68,6 +69,7 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
       code: `<template>
         {{component "input" type="text" autofocus=true}}
       </template>`,
+      output: null,
       errors: [
         {
           messageId: 'noAutofocus',

From c95d0c84fc5a7af785d6214a24197112188f4ce9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Fri, 13 Mar 2026 13:11:47 +0100
Subject: [PATCH 5/6] Remove .claude/settings.local.json

---
 .claude/settings.local.json | 34 ----------------------------------
 1 file changed, 34 deletions(-)
 delete mode 100644 .claude/settings.local.json

diff --git a/.claude/settings.local.json b/.claude/settings.local.json
deleted file mode 100644
index 9138fee884..0000000000
--- a/.claude/settings.local.json
+++ /dev/null
@@ -1,34 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(for branch:*)",
-      "Read(//Users/johanrd/fremby/ember-template-lint-main/.git/**)",
-      "Read(//Users/johanrd/fremby/ember-template-lint-main/**)",
-      "Bash(gh api:*)",
-      "Bash(git checkout:*)",
-      "Bash(npx jest:*)",
-      "Bash(cat jest.config.js 2>/dev/null || cat jest.config.cjs 2>/dev/null || echo \"no jest config\" ; ls package.json && node -e \"const p=require\\('./package.json'\\); console.log\\(JSON.stringify\\(p.scripts?.test || 'no test script'\\)\\)\")",
-      "Bash(npx vitest:*)",
-      "Bash(git add:*)",
-      "WebFetch(domain:www.w3.org)",
-      "Bash(git diff:*)",
-      "Bash(node -e \"\nconst regexp = /\\(.*\\\\s\\)?noopener\\\\s\\(.*\\\\s\\)?noreferrer\\(\\\\s.*\\)?|\\(.*\\\\s\\)?noreferrer\\\\s\\(.*\\\\s\\)?noopener\\(\\\\s.*\\)?/;\nconsole.log\\('noopener noreferrer:', regexp.test\\('noopener noreferrer'\\)\\);\nconsole.log\\('noreferrer noopener:', regexp.test\\('noreferrer noopener'\\)\\);\nconsole.log\\('noopener', regexp.test\\('noopener'\\)\\);\nconsole.log\\('noreferrer', regexp.test\\('noreferrer'\\)\\);\nconsole.log\\('xnoopener noreferrer', regexp.test\\('xnoopener noreferrer'\\)\\);\nconsole.log\\('noopenerx noreferrer', regexp.test\\('noopenerx noreferrer'\\)\\);\nconsole.log\\('nofollow noopener noreferrer:', regexp.test\\('nofollow noopener noreferrer'\\)\\);\n\")",
-      "Bash(node -e \"\nconst regexp = /\\(.*\\\\s\\)?noopener\\\\s\\(.*\\\\s\\)?noreferrer\\(\\\\s.*\\)?|\\(.*\\\\s\\)?noreferrer\\\\s\\(.*\\\\s\\)?noopener\\(\\\\s.*\\)?/;\n// Test edge cases\nconsole.log\\('foonoopener noreferrer:', regexp.test\\('foonoopener noreferrer'\\)\\);  \nconsole.log\\('noopener foonoreferrer:', regexp.test\\('noopener foonoreferrer'\\)\\);  \nconsole.log\\('foonoreferrer noopener:', regexp.test\\('foonoreferrer noopener'\\)\\);  \nconsole.log\\('noreferrer foonoopener:', regexp.test\\('noreferrer foonoopener'\\)\\);  \n\")",
-      "Bash(node -e \"\nconst AllowedPrefix = /[a-z]/;\n// Original uses: !AllowedPrefix.test\\(firstChar\\) || isReserved\n// So if first char is lowercase AND not reserved => allowed\n// If first char is NOT lowercase OR is reserved => flagged\n\n// Check _ prefix\nconsole.log\\('_ matches:', AllowedPrefix.test\\('_'\\)\\);  // false => flagged\nconsole.log\\('A matches:', AllowedPrefix.test\\('A'\\)\\);  // false => flagged\nconsole.log\\('a matches:', AllowedPrefix.test\\('a'\\)\\);  // true => allowed\nconsole.log\\('1 matches:', AllowedPrefix.test\\('1'\\)\\);  // false => flagged\n\")",
-      "Bash(node -e \"\n// The 'token' check does: convertToStrings\\(permittedValues\\).includes\\(value\\)\n// This is case-sensitive!\n// But 'tokenlist' does: token.toLowerCase\\(\\)\n// So tokenlist is case-insensitive but token is case-sensitive\n// Let's check aria-checked token values\nconsole.log\\('aria-checked values typically: true, false, mixed'\\);\n\n// The 'token' type does NOT lowercase the value before checking\n// So aria-checked='True' would fail, but probably should pass per WAI-ARIA spec\n// WAI-ARIA 1.2 says values are case-insensitive for 'token' type too\n\")",
-      "Bash(node -e \"\n// Check the original: token type doesn't do case-insensitive matching\n// but tokenlist does .toLowerCase\\(\\) on tokens\n\n// Token type check \\(line 90\\):\n// return typeof value === 'string' && convertToStrings\\(permittedValues\\).includes\\(value\\);\n// ^ case-sensitive\n\n// Tokenlist type check \\(line 95\\):\n// value.split\\(' '\\).every\\(\\(token\\) => permittedValues.includes\\(token.toLowerCase\\(\\)\\)\\)\n// ^ case-insensitive via toLowerCase\n\n// This is inconsistent! aria-live='POLITE' would fail \\(token type\\)\n// but aria-relevant='ADDITIONS REMOVALS' might pass \\(tokenlist type\\)\n\n// However... WAI-ARIA says attribute values are case-sensitive for token types\n// Actually the HTML spec says attribute values are generally case-insensitive for enumerated types\n// but ARIA values in practice must be lowercase. So this might actually be correct.\nconsole.log\\('tokenlist is case-insensitive, token is case-sensitive'\\);\nconsole.log\\('This is a potential inconsistency in the original, but arguable'\\);\n\")",
-      "Bash(cd /Users/johanrd/fremby/ember-template-lint-main && node -e \"\nimport\\('aria-query'\\).then\\(\\({aria}\\) => {\n  const checked = aria.get\\('aria-checked'\\);\n  console.log\\('aria-checked:', JSON.stringify\\(checked\\)\\);\n  const relevant = aria.get\\('aria-relevant'\\);\n  console.log\\('aria-relevant:', JSON.stringify\\(relevant\\)\\);\n  const live = aria.get\\('aria-live'\\);\n  console.log\\('aria-live:', JSON.stringify\\(live\\)\\);\n}\\);\n\")",
-      "Bash(node -e \"\nconst { aria } = require\\('aria-query'\\);\nconst checked = aria.get\\('aria-checked'\\);\nconsole.log\\('aria-checked:', JSON.stringify\\(checked\\)\\);\nconst relevant = aria.get\\('aria-relevant'\\);\nconsole.log\\('aria-relevant:', JSON.stringify\\(relevant\\)\\);\nconst live = aria.get\\('aria-live'\\);\nconsole.log\\('aria-live:', JSON.stringify\\(live\\)\\);\n\")",
-      "Bash(node -e \"\n// token: convertToStrings\\(permittedValues\\).includes\\(value\\) => case-SENSITIVE\n// tokenlist: permittedValues.includes\\(token.toLowerCase\\(\\)\\) => case-INSENSITIVE\n\n// So aria-live='POLITE' would fail \\(token, case-sensitive\\) -- strict but reasonable\n// But aria-relevant='ADDITIONS' would pass \\(tokenlist, case-insensitive\\) -- lenient\n\n// Per WAI-ARIA spec: attribute values for tokens should be compared case-insensitively\n// in HTML context. See: https://www.w3.org/TR/wai-aria-1.2/#propcharacteristic_value\n// 'Value type: token' - The token value is case-sensitive when specified in the\n// host language. But HTML is case-insensitive for attribute values.\n\n// This is an inconsistency in the original but subtle enough that it's arguable.\n// The main concern is: token is strict \\(case-sensitive\\), tokenlist is lenient.\nconsole.log\\('Inconsistency confirmed but arguable'\\);\n\")",
-      "Bash(node -e \"\n// ORIGINAL no-invalid-role.js:\n// Line 263: ['presentation', 'none'].includes\\(roleValue\\)    <-- CASE-SENSITIVE  \n// Line 272: !VALID_ROLES.has\\(roleValue.toLowerCase\\(\\)\\)        <-- CASE-INSENSITIVE\n\n// So 'Presentation' would:\n// 1. NOT match the presentation/none check \\(line 263\\) -- allowed through\n// 2. .toLowerCase\\(\\) => 'presentation' => VALID_ROLES.has\\('presentation'\\) => true => not flagged\n\n// Result: 'Presentation' is silently accepted without checking against SEMANTIC_ELEMENTS\n// This is the confirmed case-sensitivity bug\n\n// BRANCH fix/2475:\n// Uses roleLower = role.toLowerCase\\(\\) for BOTH checks\n// So 'Presentation' on <button> would correctly be flagged\n\nconsole.log\\('Confirmed: original has case-sensitivity bug in no-invalid-role'\\);\nconsole.log\\('presentation/none check is case-sensitive, validity check is case-insensitive'\\);\nconsole.log\\('Role=Presentation on <button> silently passes both checks in the original'\\);\n\")",
-      "Bash(node -e \"\n// From the original no-invalid-aria-attributes.js line 93-96:\n// case 'tokenlist': {\n//   return \\(\n//     typeof value === 'string' &&\n//     value.split\\(' '\\).every\\(\\(token\\) => permittedValues.includes\\(token.toLowerCase\\(\\)\\)\\)\n//   \\);\n// }\n\n// aria-relevant has permitted values: ['additions', 'all', 'removals', 'text']\n// So aria-relevant='ADDITIONS' => 'additions'.toLowerCase\\(\\) => 'additions' => found => valid\n// This means the original accepts 'ADDITIONS' which per WAI-ARIA spec should be case-sensitive\n\n// But this is arguable since HTML attribute values are generally case-insensitive\n// And the spec says 'unless otherwise specified'\n\nconsole.log\\('tokenlist lowercasing in original: potential spec violation'\\);\nconsole.log\\('token NOT lowercasing in original: matches spec'\\);\nconsole.log\\('The inconsistency EXISTS but may be intentional for HTML tolerance'\\);\n\")",
-      "Bash(node -e \"\n// Original no-autofocus-attribute.js:\n// MustacheStatement\\(node\\) {\n//   const autofocusAttribute = node.hash.pairs.find\\(\\(pair\\) => pair.key === 'autofocus'\\);\n// }\n\n// Note: no null check on node.hash or node.hash.pairs\n// This could crash if node.hash is null/undefined\n\n// The branch fix/2408 adds this guard:\n// if \\(!node.hash || !node.hash.pairs\\) { return; }\n\n// This is a defensive fix, but does the original actually have this bug?\n// In ember-template-lint, MustacheStatement always has .hash with .pairs\n// So the original doesn't crash in practice, but the branch is more defensive.\n\nconsole.log\\('fix/2408 adds defensive null check - not a bug in original context'\\);\n\")",
-      "Bash(node -e \"\n// Original line 25:\n// if \\(!AllowedPrefix.test\\(firstChar\\) || isReserved\\) {\n//   this.log\\({\n//     message: isReserved ? ERROR_MESSAGE_RESERVED\\('@' + part\\) : ERROR_MESSAGE_CAPITAL,\n//   }\\);\n// }\n\n// Note: The original uses OR: !AllowedPrefix || isReserved\n// If isReserved is true, it logs the reserved message regardless of the first char\n// If !AllowedPrefix \\(first char not lowercase\\), it logs the capital message\n\n// The branch uses:\n// if \\(RESERVED.has\\(name\\)\\) { report reservedArgument }\n// else if \\(!ALLOWED_PREFIX.test\\(firstChar\\)\\) { report noCapitalArguments }\n\n// Original: @arguments => isReserved=true, !AllowedPrefix\\('a'\\)=false \n//   => condition: false || true => true, message: reserved ✓\n// Branch: RESERVED.has\\('@arguments'\\) => true => reservedArgument ✓\n\n// Original: @_foo => isReserved=false, !AllowedPrefix\\('_'\\)=true\n//   => condition: true || false => true, message: capital ✓  \n// Branch: !RESERVED => !ALLOWED_PREFIX\\('_'\\)=true => noCapitalArguments ✓\n\n// Edge case: what if something is BOTH reserved AND starts with non-lowercase?\n// In the original, @arguments starts with 'a' which IS lowercase, so this doesn't apply.\n// @args starts with 'a' too. @block starts with 'b'. @else starts with 'e'.\n// So all reserved names start with lowercase -- no overlap issue.\n\nconsole.log\\('Branch matches original behavior exactly'\\);\n\")",
-      "WebFetch(domain:raw.githubusercontent.com)",
-      "Bash(git stash:*)",
-      "Bash(git worktree:*)",
-      "Bash(npx eslint:*)",
-      "Bash(npx prettier:*)"
-    ]
-  }
-}

From 210d8eca11429bb47e65eba14590454f62d34c36 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Fri, 13 Mar 2026 13:43:51 +0100
Subject: [PATCH 6/6] cleanup

---
 FIX.md    | 102 ------------
 REVIEW.md | 470 ------------------------------------------------------
 2 files changed, 572 deletions(-)
 delete mode 100644 FIX.md
 delete mode 100644 REVIEW.md

diff --git a/FIX.md b/FIX.md
deleted file mode 100644
index 1f07d95435..0000000000
--- a/FIX.md
+++ /dev/null
@@ -1,102 +0,0 @@
-# Post-merge review of extracted template-lint rules
-
-Line-by-line comparison against the original [ember-template-lint](https://github.com/ember-template-lint/ember-template-lint) source at commit [`f43c6f1`](https://github.com/ember-template-lint/ember-template-lint/tree/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b) (v7.9.3). Each section documents a discrepancy found in the extraction and the fix applied. All branches have passing tests.
-
----
-
-## Post-merge review of #2381 (`template-no-log`)
-
-Missing scope tracking for block parameters causes false positives when `log` is a block param name (e.g., `{{#each this.logs as |log|}}{{log}}{{/each}}`). The original checks [`this.isLocal(node)`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-log.js#L7) before reporting. Fix adds a `localScopes` stack with `GlimmerBlockStatement`/`GlimmerElementNode` enter/exit pairs, matching the pattern already used in `template-no-dynamic-subexpression-invocations`.
-
----
-
-## Post-merge review of #2395 (`template-link-href-attributes`)
-
-Missing exception for `<a>` elements with both `role` and `aria-disabled` attributes, causing false positives on accessible patterns like `<a role="link" aria-disabled="true">`. The original skips reporting when both are present — see [`link-href-attributes.js` lines 27–29](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-href-attributes.js#L27-L29).
-
----
-
-## Post-merge review of #2396 (`template-link-rel-noopener`)
-
-Two issues. (1) Detection regex uses bare `/noopener/` which matches substrings. The original uses a [whitespace-bounded regex](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-rel-noopener.js#L24). Fix uses `/(?:^|\s)noopener(?:\s|$)/`. (2) Fixer always inserts a new `rel` attribute even when one already exists, producing duplicate attributes. The [original's fixer](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/link-rel-noopener.js#L99-L102) strips existing `noopener`/`noreferrer` and re-adds them in canonical order. Fix now does the same.
-
----
-
-## Post-merge review of #2408 (`template-no-autofocus-attribute`)
-
-Missing `MustacheStatement` visitor. The original handles both [`ElementNode`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-autofocus-attribute.js#L21-L26) and [`MustacheStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-autofocus-attribute.js#L28-L32) (for `{{input autofocus=true}}`). The extraction only had `ElementNode`, so `{{input autofocus=true}}` and `{{component "input" autofocus=true}}` silently passed.
-
----
-
-## Post-merge review of #2414 (`template-no-capital-arguments`)
-
-Three missing features. The original has both a [`PathExpression`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L19-L33) and an [`AttrNode`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L34-L46) visitor, plus a [`RESERVED` set](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L9) and [`AllowedPrefix` regex](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-capital-arguments.js#L8). The extraction was missing: (1) `AttrNode` visitor — `<Foo @Name="bar" />` wasn't caught, (2) reserved argument detection (`@arguments`, `@args`, `@block`, `@else`), (3) underscore-prefix detection (`@_Name`).
-
----
-
-## Post-merge review of #2423 (`template-no-action`)
-
-Two bugs. (1) Missing [`ElementModifierStatement` visitor](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action.js#L46-L48) — `<button {{action "submit"}}>` silently passed. (2) The original's [`MustacheStatement` visitor](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action.js#L40-L42) reports `{{action}}` everywhere including attribute position (`<input onclick={{action "foo"}}>`), but the extraction incorrectly walked parents to skip `GlimmerAttrNode`.
-
----
-
-## Post-merge review of #2424 (`template-no-action-modifiers`)
-
-Missing allowlist configuration. The original supports an [`allowlist` config](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action-modifiers.js#L17-L19) (passed as an array, e.g., `['button']`) that skips reporting when the parent element's tag is in the list — see the [allowlist guard](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-action-modifiers.js#L48). The extraction had `schema: []` with no config support.
-
----
-
-## Post-merge review of #2429 (`template-no-element-event-actions`)
-
-Over-reporting due to erroneous `ConcatStatement` check. The original only checks [`node.value.type === 'MustacheStatement'`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-element-event-actions.js#L61). The extraction also checked `GlimmerConcatStatement`.
-
----
-
-## Post-merge review of #2442 (`template-no-inline-styles`)
-
-The original's `allowDynamicStyles` exemption only checks [`style.value.type === 'MustacheStatement'`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-inline-styles.js#L62). The extraction incorrectly also exempted `GlimmerConcatStatement` (e.g., `style="{{foo}} bar"`), which the original flags as invalid. Also fixes the same ConcatStatement issue in `template-no-element-event-actions`.
-
----
-
-## Post-merge review of #2469 (`template-no-implicit-this`)
-
-Missing `welcome-page` and `rootURL` in built-ins. The original includes [`ARGLESS_DEFAULT_BLUEPRINT`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-implicit-this.js#L48-L51) containing these in its allow list. Also fixes the ConcatStatement issue in `template-no-element-event-actions`.
-
----
-
-## Post-merge review of #2471 (`template-no-invalid-aria-attributes`)
-
-Error messages don't match original's [`createInvalidAttributeTypeErrorMessage`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L11-L38). Specifically: (1) `boolean` type produced `"a boolean (true or false)"` but the original's [default case](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L36) produces `"a boolean."`. (2) [`id`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L30) and [`idlist`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-aria-attributes.js#L27) have no trailing period in the original, but the extraction's message template added one.
-
----
-
-## Post-merge review of #2475 (`template-no-invalid-role`)
-
-Three issues. (1) Missing roles (`associationlistitemkey`, `associationlistitemvalue`, `cell`) in `VALID_ROLES`. (2) `SEMANTIC_ELEMENTS` set doesn't match the original's [`ELEMENTS_DISALLOWING_PRESENTATION_OR_NONE_ROLE`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L16-L102). (3) Missing `.toLowerCase()` for the [`VALID_ROLES.has()` check](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L272).
-
-**Intentional improvement over original:** The `presentation`/`none` check also uses `.toLowerCase()`, while the [original is case-sensitive there](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-invalid-role.js#L263). The original has an internal inconsistency: the validity check uses `.toLowerCase()` but the presentation/none check does not, meaning `role="NONE"` on a `<button>` silently passes. Per the [WAI-ARIA 1.2 spec](https://www.w3.org/TR/wai-aria-1.2/#document-handling_author-errors_roles): *"Case-sensitivity of the comparison inherits from the case-sensitivity of the host language"* — and HTML is case-insensitive.
-
----
-
-## Post-merge review of #2477 (`template-no-link-to-positional-params`)
-
-Incorrect `GlimmerElementNode` handler. The original only handles [`MustacheStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-link-to-positional-params.js#L9) and [`BlockStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-link-to-positional-params.js#L13) (curly-brace `{{link-to}}`). The extraction added a `GlimmerElementNode` handler that tries to detect positional params on `<LinkTo>` angle-bracket invocations using a heuristic with no basis in the original.
-
----
-
-## Post-merge review of #2484 (`template-no-nested-interactive`)
-
-Config option name mismatch. The original uses [`ignoreUsemapAttribute`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-nested-interactive.js#L166). The extraction renamed it to `ignoreUsemap`. Fix accepts both names for migration compatibility.
-
----
-
-## Post-merge review of #2490 (`template-no-outlet-outside-routes`)
-
-Missing `BlockStatement` handler. The original registers both [`MustacheStatement` and `BlockStatement`](https://github.com/ember-template-lint/ember-template-lint/blob/f43c6f11fdf8fc8ecb51ba04cea0f367b1af544b/lib/rules/no-outlet-outside-routes.js#L26-L28) for `_checkForOutlet`. The extraction only had `GlimmerMustacheStatement`, so `{{#outlet}}content{{/outlet}}` in non-route templates silently passed.
-
----
-
-## Notes
-
-- **Overlapping fix:** #2429, #2442, and #2469 all contain the same `template-no-element-event-actions` ConcatStatement removal. They will conflict — merge #2442 first (bundles both rules' fixes + tests) and rebase the others.
-- 14 branches faithfully reproduce the original ember-template-lint behavior; #2475 intentionally improves one case-sensitivity inconsistency backed by the WAI-ARIA spec.
diff --git a/REVIEW.md b/REVIEW.md
deleted file mode 100644
index b2fd2b4113..0000000000
--- a/REVIEW.md
+++ /dev/null
@@ -1,470 +0,0 @@
-# Extract Rule PRs - Systematic Review
-
-> Comparing each PR against its ember-template-lint counterpart in `/Users/johanrd/fremby/ember-template-lint-main`.
-> Goal: in gjs/gts projects, these rules must behave equivalently so ember-template-lint is no longer needed.
-
-## Overview
-
-| PR | Rule | Status | Key Issues |
-|----|------|--------|------------|
-| **Batch 1** | | | |
-| #2477 | template-no-link-to-positional-params | **NEEDS WORK** | Unnecessary `<LinkTo>` handler (can false-positive on `class={{...}}`); static error message loses migration advice |
-| #2475 | template-no-invalid-role | **NEEDS WORK** | **Bug:** missing `.toLowerCase()` on role values — `role="Button"` falsely flagged; ~19 elements missing from semantic set |
-| #2474 | template-no-invalid-meta | **NEEDS WORK** | New `charset` validation not in original (enhancement, not extraction); test gaps around charset logic |
-| #2473 | template-no-invalid-link-title | **NEEDS WORK** | Single generic message loses "both attribute and argument" context; missing strict-mode guard for `LinkTo` |
-| #2472 | template-no-invalid-link-text | **NEEDS WORK** | Nested element traversal differs from original; doc self-reference bug |
-| #2471 | template-no-invalid-aria-attributes | **NEEDS WORK** | Generic error messages lose type-specific guidance; stricter numeric validation (improvement) |
-| **Batch 2** | | | |
-| #2470 | template-no-input-block | **PASS** | Minor message text difference (cosmetic) |
-| #2469 | template-no-implicit-this | **NEEDS WORK** | `isControlFlowParam` wrongly skips flagging ambiguous params; missing RegExp allow; incomplete block param detection; missing default allow entries |
-| #2468 | template-no-extra-mut-helper-argument | **PASS** | Clean extraction, identical behavior |
-| #2467 | template-no-dynamic-subexpression-invocations | **NEEDS WORK** | Flags body-context `this.*` mustaches not flagged by original; generic error message loses context-specific guidance |
-| #2465 | template-linebreak-style | **PASS** | Minor: missing `true` (consistency) mode and editorconfig integration; adds fixer (improvement) |
-| #2464 | template-eol-last | **PASS** | Minor: missing `editorconfig` config option |
-| **Batch 3** | | | |
-| #2462 | template-block-indentation | **NEEDS WORK** | Missing `validateBlockStart`; no `columnOffset` handling |
-| #2461 | template-attribute-indentation | **NEEDS WORK** | Missing `validateClosingTag`; `processElements` defaults differ |
-| #2460 | template-no-curly-component-invocation | **NEEDS WORK** | `noImplicitThis` accepted in schema but never used; multi-part paths not handled |
-| #2459 | template-no-class-bindings | **PASS** | Slightly stricter (catches non-`@` attrs too) — reasonable enhancement |
-| #2457 | template-no-bare-yield | **NEEDS WORK** | Fundamentally different semantics from `no-yield-only` |
-| #2456 | template-no-at-ember-render-modifiers | **NEEDS WORK** (minor) | Misleading error message recommends the very modifiers being banned |
-| **Batch 4** | | | |
-| #2455 | template-no-args-paths | **NEEDS WORK** | No `isLocal` check (false positives on block params named `args`); generic error message |
-| #2454 | template-no-ambiguous-glimmer-paths | **NEEDS WORK** | Dramatically narrower than `no-implicit-this`; only dotted paths in mustaches; no `allow` config |
-| #2453 | template-no-action-on-submit-button | **PASS** | Solid implementation, correct logic, good test coverage |
-| #2452 | template-modifier-name-case | **NEEDS WORK** | `dasherize` produces leading dashes for capitalized names; unused broken `messageId` |
-| #2451 | template-deprecated-render-helper | **NEEDS WORK** (minor) | No fix/suggestion behavior (original auto-fixes); slight message differences |
-| #2450 | template-deprecated-inline-view-helper | **NEEDS WORK** | No `isLocal` check for block params named `view`; no fix; possible double-reporting |
-| **Batch 5** | | | |
-| #2441 | template-no-inline-linkto | **NEEDS WORK** | Scope expanded beyond original (adds `<LinkTo>` angle bracket checks); missing autofix |
-| #2439 | template-no-index-component-invocation | **PASS** | Minor: verify `.value` vs `.original` for string literals |
-| #2438 | template-no-html-comments | **NEEDS WORK** | `originallyFrom.name` bug (says `eslint-plugin-ember` instead of `ember-template-lint`); different detection approach |
-| #2436 | template-no-form-action | **NEEDS WORK** | **No ember-template-lint counterpart exists**; questionable rule semantics |
-| #2435 | template-no-forbidden-elements | **NEEDS WORK** | Missing object config form `{ forbidden: [...] }`; missing `head.hbs` exception |
-| #2432 | template-no-extra-mut-helpers (Draft) | **CRITICAL** | **Rule logic is inverted** — flags correct `(mut value)` usage instead of `(mut a b)` |
-| **Batch 6** | | | |
-| #2427 | template-no-duplicate-landmark-elements | **NEEDS WORK** | Missing dialog/popover scoping; incomplete block scoping (only `if`/`unless`) |
-| #2426 | template-no-duplicate-id | **NEEDS WORK** | Missing ancestor yield block resolution; potential false positives for block-param IDs |
-| #2418 | template-no-down-event-binding (Draft) | **CRITICAL** | Wrong event set (missing `pointerdown`, incorrectly added `touchstart`); missing `action` modifier handling |
-| #2413 | template-no-builtin-form-components (Draft) | **PASS** | Simple, correct port |
-| #2409 | template-no-bare-strings | **NEEDS WORK** | Complex rule (371 lines in original); needs manual verification of full implementation |
-
----
-
-## Critical Issues (must fix before merge)
-
-### #2432 — template-no-extra-mut-helpers: Inverted logic
-The ember-template-lint `no-extra-mut-helper-argument` flags `(mut attr extraArg)` (too many arguments). The PR flags `(mut this.value)` (one argument = correct usage). **The condition is backwards.** Should check `params.length > 1`, not `params.length === 1`.
-
-### #2418 — template-no-down-event-binding: Wrong event set
-- **Missing:** `pointerdown`, `onpointerdown` (the whole point of the original rule name `no-pointer-down-event-binding`)
-- **Incorrectly added:** `touchstart` (not in original)
-- Missing `{{action}}` modifier handling
-- Missing case-insensitive matching
-- Error message recommends wrong alternatives
-
-### #2457 — template-no-bare-yield: Different rule entirely
-- ember-template-lint `no-yield-only` only flags when `{{yield}}` is the **only content** in the entire template
-- PR flags **any** `{{yield}}` without parameters regardless of context
-- `<div>{{yield}}</div>` would pass in ember-template-lint but fail in the PR
-
-### #2475 — template-no-invalid-role: Missing case-insensitive role matching
-Role values are case-insensitive per the ARIA spec. The PR checks `VALID_ROLES.has(role)` without `.toLowerCase()`. This means `role="Button"` or `role="NAVIGATION"` will be falsely flagged as invalid. The original correctly lowercases before lookup.
-
-### #2477 — template-no-link-to-positional-params: Unnecessary `<LinkTo>` handler
-Angle-bracket `<LinkTo>` never supported positional params — you write `<LinkTo @route="about">`. The original deliberately does NOT check `<LinkTo>`. The PR's `GlimmerElementNode` handler checks non-`@` attrs with dynamic values, which could false-positive on `class={{...}}`, `id={{...}}`, `data-test={{...}}`, etc.
-
-### #2436 — template-no-form-action: No counterpart
-No corresponding rule exists in ember-template-lint. The HTML `action` attribute on `<form>` is standard HTML, not an Ember concern. Should not be presented as an "extraction."
-
----
-
-## Detailed Reviews
-
----
-
-### PR #2477 — template-no-link-to-positional-params
-**ember-template-lint rule:** `no-link-to-positional-params` | **Status: NEEDS WORK**
-
-Detects deprecated positional parameter usage in `{{link-to}}` / `{{#link-to}}`.
-
-**Issues:**
-1. **Unnecessary `<LinkTo>` handler:** Angle-bracket `<LinkTo>` never supported positional params. The original deliberately omits this check. The PR's heuristic (`!attr.name.startsWith('@') && attr.value.type !== 'GlimmerTextNode'`) could false-positive on legitimate HTML attributes like `class={{dynamicClass}}`. **Remove the `GlimmerElementNode` handler entirely.**
-2. **Static error message:** Original dynamically includes which named arguments to use (`@route`, `@model`/`@models`, `@query`) and advises using block form. PR uses a single generic string.
-3. **Misleading docs:** Example shows `<LinkTo "posts.post" @model={{this.post}}>` as incorrect — but this isn't valid Ember syntax. Angle-bracket components can't receive positional string params.
-4. **Zero tests for `<LinkTo>` code path** — ~30 lines of rule code are untested.
-
----
-
-### PR #2475 — template-no-invalid-role
-**ember-template-lint rule:** `no-invalid-role` | **Status: NEEDS WORK**
-
-Validates ARIA roles and flags `presentation`/`none` on semantic elements.
-
-**Issues:**
-1. **Bug: Missing `.toLowerCase()`:** ARIA role values are case-insensitive per spec. PR checks `VALID_ROLES.has(role)` without lowercasing. `role="Button"` is falsely flagged.
-2. **~19 elements missing from `SEMANTIC_ELEMENTS`:** `audio`, `video`, `iframe`, `embed`, `area`, `br`, `source`, `track`, `wbr`, `object`, `param`, `map`, `applet`, `dir`, `menuitem`, `noembed`, `rb`, `rtc`, `tt` — all present in original but absent from PR.
-3. **Elements added not in original:** `address`, `article`, `aside`, `footer`, `h1`-`h6`, `header`, `hgroup`, `nav`, `section` — reasonable additions but behavioral divergence.
-4. **Empty/whitespace role silently ignored:** PR trims + skips empty roles; original would flag `role="  "` as invalid.
-5. **Check order reversed with early return:** Invalid-role reports first then returns, preventing the presentation check from ever firing on the same node. Original runs both checks independently.
-
----
-
-### PR #2474 — template-no-invalid-meta
-**ember-template-lint rule:** `no-invalid-meta` | **Status: NEEDS WORK**
-
-Validates `<meta>` elements (content requirements, viewport, refresh).
-
-**Issues:**
-1. **New `invalidCharset` check:** Validates `<meta charset="...">` must be `utf-8`. This does not exist in ember-template-lint. Should be documented as an enhancement, not a straight extraction.
-2. **`hasIdentifier` includes `hasHttpEquiv`:** Original keeps these separate (`hasIdentifier || hasHttpEquiv`). Functionally equivalent but semantically misleading.
-3. **Edge case: `parseInt("", 10)` → `NaN`:** Empty `content=""` with `http-equiv="refresh"` silently passes. Should probably be flagged.
-4. **Test gaps:** Missing tests for `charset` + other attrs combos, multiple errors on single element, boundary value `delay=72000`, charset in HBS mode.
-
----
-
-### PR #2473 — template-no-invalid-link-title
-**ember-template-lint rule:** `no-invalid-link-title` | **Status: NEEDS WORK**
-
-Disallows title attributes that duplicate link text.
-
-**Issues:**
-1. **Single generic message:** Original has distinct messages for "both attribute and argument" vs "redundant title". PR uses one message for all cases — confusing for `<LinkTo @title="foo" title="bar">`.
-2. **Missing strict-mode guard:** Original skips `LinkTo` in strict mode (`this.isStrictMode`). PR always checks it. Could produce false positives in GJS `<template>` context.
-3. **Dual-title early return:** PR returns after reporting dual-title, never checking if the title also duplicates link text. Original can report both errors.
-4. **Missing tests:** Empty `title` on `<LinkTo>`, empty `@title`, title-as-substring-of-text.
-
----
-
-### PR #2472 — template-no-invalid-link-text
-**ember-template-lint rule:** `no-invalid-link-text` | **Status: NEEDS WORK**
-
-Disallows generic link text ("click here", "more info", "read more", "more").
-
-**Issues:**
-1. **Nested element traversal:** PR recursively traverses child elements (e.g., `<a><span>click here</span></a>`). Original only checks direct `TextNode` children. This is a behavioral difference — arguably better but should be tested and documented.
-2. **Doc self-reference bug:** References section links to itself instead of the ember-template-lint rule.
-3. **Missing tests:** `allowEmptyLinks` in GJS mode, standalone `"more"` test, `aria-label` with disallowed value on `<LinkTo>`, nested element content.
-
----
-
-### PR #2471 — template-no-invalid-aria-attributes
-**ember-template-lint rule:** `no-invalid-aria-attributes` | **Status: NEEDS WORK**
-
-Validates ARIA attribute names and value types using `aria-query`.
-
-**Issues:**
-1. **Generic error messages:** Original provides type-specific messages (e.g., "must be a boolean or the string 'mixed'", "must be an integer"). PR uses `"Invalid value for ARIA attribute aria-checked."` — less actionable.
-2. **Stricter numeric validation (improvement):** PR uses `Number()` instead of `parseInt` — `"50px"` correctly rejected. Also distinguishes `integer` vs `number` types. These are improvements over the original.
-3. **`string` type accepts booleans:** PR accepts `aria-label="true"`, original rejects it. Minor difference.
-4. **Reports all errors per element:** Original stops after first invalid attribute. PR reports all. Better for ESLint UX.
-5. **Missing tests:** `aria-hidden` without value, empty string values, decimal for `number` type, negative integers.
-
----
-
-### PR #2453 — template-no-action-on-submit-button
-**ember-template-lint rule:** `no-input-block` | **Status: PASS**
-
-Disallows block usage of `{{input}}` helper. Simple rule, correctly ported.
-
-- **Error message:** Minor cosmetic difference — PR omits `{{}}` around "input" in message text. Acceptable.
-- **Schema/fix/test coverage:** All match. No configuration, no fix, good coverage.
-
----
-
-### PR #2469 — template-no-implicit-this
-**ember-template-lint rule:** `no-implicit-this` | **Status: NEEDS WORK**
-
-Requires explicit `this.` or `@` prefix for property access in templates.
-
-**Issues:**
-1. **`isControlFlowParam` wrongly skips flagging:** `{{if ambiguousPath "yes"}}` — `ambiguousPath` should be flagged (ember-template-lint flags it) but the PR skips it via `isControlFlowParam`.
-2. **Missing default allow entries:** ember-template-lint always includes `welcome-page` and `rootURL` in the allow list.
-3. **No RegExp support in `allow`:** ember-template-lint supports both strings and RegExp values. PR only allows strings.
-4. **Incomplete block param detection:** `isBlockParamPath` only checks direct parent, not ancestor scopes.
-5. **Missing SubExpression/ElementModifierStatement callee handling:** Callees of sub-expressions like `(some-helper ...)` could be incorrectly flagged.
-6. **Shorter error message:** Omits guidance about angle bracket invocation and rule configuration.
-
----
-
-### PR #2468 — template-no-extra-mut-helper-argument
-**ember-template-lint rule:** `no-extra-mut-helper-argument` | **Status: PASS**
-
-Disallows passing more than one argument to `mut`. Simple rule, correctly ported.
-
-- Error message identical.
-- Minor style note: uses inline `message` instead of `messageId`. Functional but inconsistent with other PRs.
-
----
-
-### PR #2467 — template-no-dynamic-subexpression-invocations
-**ember-template-lint rule:** `no-dynamic-subexpression-invocations` | **Status: NEEDS WORK**
-
-Disallows dynamic helper/modifier invocations with dynamic callee paths.
-
-**Issues:**
-1. **Behavioral extension:** `{{this.formatter this.data}}` in body context is flagged by PR but NOT by ember-template-lint (original only flags attr-position mustaches with dynamic callees).
-2. **Generic error message:** Original gives context-specific messages (e.g., "You must use the `fn` helper" for attr-position). PR uses one generic message.
-3. **Local scope tracking diverges:** PR implements own `localScopes` stack vs ember-template-lint's `this.scope.isLocal()`.
-
----
-
-### PR #2465 — template-linebreak-style
-**ember-template-lint rule:** `linebreak-style` | **Status: PASS**
-
-Enforces consistent linebreaks (LF vs CRLF).
-
-- **Improvement:** PR adds a fixer (ember-template-lint has none).
-- **Minor gaps:** Missing `true` (auto-detect consistency) mode. Missing editorconfig `end_of_line` integration.
-
----
-
-### PR #2464 — template-eol-last
-**ember-template-lint rule:** `eol-last` | **Status: PASS**
-
-Requires or disallows newline at end of template.
-
-- Error messages identical.
-- Fix behavior equivalent.
-- Correctly skips gjs/gts embedded templates.
-- **Minor gap:** Missing `editorconfig` option (`insert_final_newline`).
-
----
-
-### PR #2462 — template-block-indentation
-**ember-template-lint rule:** `block-indentation` | **Status: NEEDS WORK**
-
-Enforces consistent block indentation.
-
-**Issues:**
-1. **Missing `validateBlockStart`:** Top-level block/element indentation relative to template root is not validated.
-2. **No `columnOffset` handling:** ember-template-lint adjusts for embedded template offsets. PR does not, which could cause issues in certain nesting scenarios.
-3. **Missing `sourceDoesNotMatchNode` guard:** Safety check against false positives is absent.
-
-Configuration, error messages, and test coverage are otherwise equivalent.
-
----
-
-### PR #2461 — template-attribute-indentation
-**ember-template-lint rule:** `attribute-indentation` | **Status: NEEDS WORK**
-
-Enforces proper attribute indentation on multi-line invocations.
-
-**Issues:**
-1. **Missing `validateClosingTag`:** Misaligned `</div>` closing tags won't be caught.
-2. **`processElements` defaults to `true`:** ember-template-lint requires explicit opt-in. PR default could cause unexpected errors during migration.
-3. **Missing close-brace validation for paramless blocks with block params.**
-
-
-
-### PR #2459 — template-no-class-bindings
-**ember-template-lint rule:** `no-class-bindings` | **Status: PASS**
-
-Disallows `classBinding`/`classNameBindings` arguments.
-
-- Slightly stricter than original (also catches non-`@` prefixed attrs on angle bracket components). Reasonable enhancement.
-- Good test coverage of all 4 forbidden patterns.
-
----
-
-### PR #2457 — template-no-bare-yield
-**ember-template-lint rule:** `no-yield-only` | **Status: NEEDS WORK** (critical semantic mismatch)
-
-See [Critical Issues](#critical-issues-must-fix-before-merge) above. The PR flags any `{{yield}}` without params. The original only flags templates whose **entire content** is just `{{yield}}`.
-
----
-
-### PR #2456 — template-no-at-ember-render-modifiers
-**ember-template-lint rule:** `no-at-ember-render-modifiers` | **Status: NEEDS WORK** (minor)
-
-Disallows `did-insert`, `did-update`, `will-destroy` modifiers.
-
-**Issue:** Error message says *"Use (did-insert), (did-update), or (will-destroy) from ember-render-helpers instead"* — recommending the very modifiers being banned. ember-template-lint correctly says to use a custom modifier via `ember-modifier`.
-
-
-
-### PR #2450 — template-deprecated-inline-view-helper
-**ember-template-lint rule:** `deprecated-inline-view-helper` | **Status: NEEDS WORK**
-
-Disallows deprecated inline `{{view}}` helper and `view.*` paths.
-
-**Issues:**
-1. **No `isLocal` check:** `{{#each items as |view|}}{{view.name}}{{/each}}` would be falsely flagged.
-2. **Missing dedup guard:** No `_processedByInlineViewHelper` equivalent — same node could be reported twice (once from element visitor, once from mustache visitor).
-3. **No autofix:** Original provides fix suggestions (removing `view.` prefix).
-
----
-
-### PR #2441 — template-no-inline-linkto
-**ember-template-lint rule:** `inline-link-to` | **Status: NEEDS WORK**
-
-**Issues:**
-1. **Scope expanded beyond original:** PR also checks angle bracket `<LinkTo />` with empty children — ember-template-lint only checks curly `{{link-to ...}}`.
-2. **Missing autofix:** Original converts inline to block form.
-3. **Message differs.**
-
----
-
-### PR #2439 — template-no-index-component-invocation
-**ember-template-lint rule:** `no-index-component-invocation` | **Status: PASS**
-
-Disallows explicit `/index` or `::Index` suffix on component invocations.
-
-- All 4 node types covered (element, mustache, block, sub-expression).
-- Dynamic error messages match.
-- Good test coverage including nested namespaces and false positives.
-- **Minor:** Verify `.value` vs `.original` property for string literal params.
-
----
-
-### PR #2438 — template-no-html-comments
-**ember-template-lint rule:** `no-html-comments` | **Status: NEEDS WORK**
-
-**Issues:**
-1. **`originallyFrom.name` bug:** Set to `'eslint-plugin-ember'` instead of `'ember-template-lint'`.
-2. **Different detection mechanism:** Uses `sourceCode.getAllComments()` + raw source check instead of `CommentStatement` visitor. Reasonable for ESLint but different approach.
-3. **Missing config comment filtering:** ember-template-lint skips `<!-- template-lint-disable -->` comments. Less relevant for ESLint context but worth noting.
-4. **Message differs:** `"HTML comment detected"` → `"HTML comments should not be used in templates..."`.
-
----
-
-### PR #2436 — template-no-form-action
-**No ember-template-lint counterpart** | **Status: NEEDS WORK**
-
-See [Critical Issues](#critical-issues-must-fix-before-merge). No corresponding rule exists. The HTML `action` attribute on `<form>` is standard HTML. This is a new rule, not an extraction.
-
----
-
-### PR #2432 — template-no-extra-mut-helpers (Draft)
-**ember-template-lint rule:** `no-extra-mut-helper-argument` | **Status: CRITICAL**
-
-See [Critical Issues](#critical-issues-must-fix-before-merge). Logic is inverted — flags correct single-argument usage instead of incorrect multi-argument usage.
-
----
-
-### PR #2427 — template-no-duplicate-landmark-elements
-**ember-template-lint rule:** `no-duplicate-landmark-elements` | **Status: NEEDS WORK**
-
-**Issues:**
-1. **Missing dialog/popover scoping:** Landmarks inside `<dialog>` or `[popover]` elements should be treated as separate scope.
-2. **Incomplete block scoping:** Only `if`/`unless` blocks get scope treatment; `each`, `let`, component blocks do not.
-3. **Added `section` element** (not in original — `<section>` → `region` role).
-4. **Different reporting approach:** Batch analysis at `Program:exit` vs eager per-element reporting.
-
----
-
-### PR #2426 — template-no-duplicate-id
-**ember-template-lint rule:** `no-duplicate-id` | **Status: NEEDS WORK**
-
-**Issues:**
-1. **Missing ancestor yield block resolution:** ember-template-lint's `getMustacheValue` differentiates `{{inputProperties.id}}` across different `<MyComponent>` invocations. PR uses `sourceCode.getText()` which would produce identical strings → false positives.
-2. **Extra `conditionalReportedDuplicates` logic** not in original — may suppress valid reports.
-
----
-
-
-### PR #2418 — template-no-down-event-binding (Draft)
-**ember-template-lint rule:** `no-pointer-down-event-binding` | **Status: CRITICAL**
-
-See [Critical Issues](#critical-issues-must-fix-before-merge). Wrong event set, missing modifier handling, wrong error message.
-
----
-
-### PR #2413 — template-no-builtin-form-components (Draft)
-**ember-template-lint rule:** `no-builtin-form-components` | **Status: PASS**
-
-Simple rule checking `<Input>` and `<Textarea>`. Error messages identical. No configuration. Both gjs and hbs tested.
-
----
-
-### PR #2419 — template-no-duplicate-attributes
-**ember-template-lint rule:** `no-duplicate-attributes` | **Status: PASS**
-
-Clean port. All 4 node types covered (element, mustache, block, sub-expression). More efficient `Map`-based algorithm vs original's O(n^2). Fix uses ESLint's `fixer.removeRange` correctly. Good test coverage.
-
----
-
-### PR #2409 — template-no-bare-strings
-**ember-template-lint rule:** `no-bare-strings` | **Status: NEEDS WORK**
-
-Complex rule (371 lines in ember-template-lint). Areas requiring manual verification:
-- `isStrictMode` handling for `Input`/`Textarea` built-in component attributes
-- `page-title`/`if`/`unless`/`concat` helper string checking
-- Default allowlist completeness (67 entries in original)
-- `elementAttributes` and `globalAttributes` handling
-- Object and array config forms
-
----
-
-## Statistics
-
-| Status | Count |
-|--------|-------|
-| **PASS** | 9 |
-| **NEEDS WORK** | 23 |
-| **CRITICAL** | 4 |
-| **Total** | 36 |
-
-## Loose-Mode-Only Rules (irrelevant for .gts/.gjs)
-
-Some of the 36 PRs extract rules that only apply to loose-mode (classic `.hbs`) templates and are **irrelevant for strict-mode `.gjs`/`.gts` files**. In strict mode, the Glimmer compiler enforces these constraints at compile time, or the patterns simply cannot occur.
-
-### Explicitly `templateMode: 'loose'` in the PR branches
-
-Verified by checking the actual PR branch code:
-
-| PR | Rule | Why loose-only |
-|----|------|---------------|
-| #2469 | template-no-implicit-this | Strict mode enforces explicit `this`/`@` at parser level. ember-template-lint also returns early when `isStrictMode`. |
-| #2460 | template-no-curly-component-invocation | Curly component invocation is impossible in strict mode |
-| #2459 | template-no-class-bindings | `classBinding`/`classNameBindings` are a loose-mode-only API |
-| #2470 | template-no-input-block | Block `{{input}}` is a loose-mode-only built-in |
-| #2468 | template-no-extra-mut-helper-argument | `mut` helper is a loose-mode-only helper |
-
-### Explicitly `templateMode: 'both'` in the PR branches
-
-These are confirmed as applying to both loose and strict mode:
-
-`template-no-at-ember-render-modifiers`, `template-no-bare-yield`, `template-no-dynamic-subexpression-invocations`, `template-eol-last`, `template-linebreak-style`, `template-block-indentation`, `template-attribute-indentation`
-
-### Conceptually loose-only (not explicitly marked but pattern can't occur in strict mode)
-
-| PR | Rule | Why irrelevant in strict mode |
-|----|------|-------------------------------|
-| #2451 | template-deprecated-render-helper | `{{render}}` helper does not exist in strict mode |
-| #2450 | template-deprecated-inline-view-helper | `{{view}}` helper does not exist in strict mode |
-| #2441 | template-no-inline-linkto | Curly `{{link-to}}` syntax — in GJS you use `<LinkTo>` |
-| #2477 | template-no-link-to-positional-params | Positional params only possible with curly `{{link-to}}` |
-| #2454 | template-no-ambiguous-glimmer-paths | Docs say HBS only; strict mode enforces explicit paths |
-| #2432 | template-no-extra-mut-helpers (Draft) | Docs say HBS Only; `mut` doesn't exist in strict mode |
-
-### Rules with `isStrictMode` guards in ember-template-lint
-
-These ember-template-lint rules have `isStrictMode` guards that affect behavior in GJS/GTS context. The extract PRs should replicate these guards:
-
-| Rule | Guard behavior |
-|------|---------------|
-| `no-implicit-this` | Returns early (disables itself) when `isStrictMode` |
-| `no-invalid-link-title` | Skips `LinkTo` checks when `isStrictMode` |
-| `no-invalid-link-text` | Skips `LinkTo` checks when `isStrictMode` |
-| `no-bare-strings` | Excludes `Input`/`Textarea` built-in component attributes when `isStrictMode` |
-| `block-indentation` | Adjusts standalone validation when `isStrictMode && isEmbedded` |
-| `eol-last` | Disables when `isEmbedded` (GJS templates are embedded) |
-
-**Impact on the extract PRs:** PRs #2473 and #2472 are missing these `isStrictMode` guards, which could cause false positives in GJS/GTS files. PR #2464 (eol-last) correctly handles this by skipping templates starting with `<template>`. PR #2409 (no-bare-strings) needs verification that strict-mode attribute handling is correct.
-
-### Summary
-
-Of the 36 extract rule PRs, **~11 are loose-mode only** (5 explicitly + 6 conceptually) and do not contribute to the goal of replacing ember-template-lint for GJS/GTS projects. The remaining **~25 rules are relevant for GJS/GTS** and are the ones that matter for eliminating the ember-template-lint dependency.
-
----
-
-## Common Patterns Across PRs
-
-1. **Missing `isLocal` / block param checks** — PRs #2469, #2455, #2450 all fail to detect when a path refers to a block parameter, causing false positives.
-2. **Missing autofix** — Several ember-template-lint rules provide autofixes that are not ported: #2441, #2450, #2451, #2452 (partial).
-3. **Error message divergence** — Many PRs use shorter/generic messages losing actionable guidance from the originals (#2469, #2467, #2471, #2473, #2477).
-4. **Configuration gaps** — Several PRs don't support all config forms (RegExp in allow lists, object configs, editorconfig integration).
-5. **Scope expansion** — Some PRs flag cases the original doesn't (#2441, #2457, #2467, #2477), which would cause unexpected errors for projects migrating from ember-template-lint.
-6. **Case sensitivity bugs** — #2475 missing `.toLowerCase()` on role values; #2418 missing case-insensitive event matching.
-7. **Strict-mode guards missing** — #2473, #2472 don't implement `isStrictMode` checks that the originals use to skip `LinkTo` in strict (GJS) context.