From fd9c410470dda739d6554cca85f2bf3b273bd6ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Mon, 13 Apr 2026 14:43:19 +0200
Subject: [PATCH 1/2] Fix template-require-media-caption: skip caption check
 when muted is dynamic

Port had no GlimmerConcatStatement handling for the muted attribute,
so <video muted="{{isMuted}}"> false-positively required captions.
Treat ConcatStatement (and other non-text values) as muted/exempt
to match upstream.
---
 lib/rules/template-require-media-caption.js       | 11 +++++++++++
 tests/lib/rules/template-require-media-caption.js |  4 ++++
 2 files changed, 15 insertions(+)

diff --git a/lib/rules/template-require-media-caption.js b/lib/rules/template-require-media-caption.js
index 512b29324c..4033211d9b 100644
--- a/lib/rules/template-require-media-caption.js
+++ b/lib/rules/template-require-media-caption.js
@@ -53,6 +53,17 @@ module.exports = {
               return;
             }
           }
+
+          // Any other dynamic value (e.g. muted="{{isMuted}}" → ConcatStatement,
+          // or muted={{#if ...}}...{{/if}} → BlockStatement) → treat as exempt.
+          // Matches upstream ember-template-lint behavior where MustacheStatement,
+          // BlockStatement, or any non-text/non-"false" value is considered muted.
+          if (
+            value.type !== 'GlimmerTextNode' &&
+            value.type !== 'GlimmerMustacheStatement'
+          ) {
+            return;
+          }
         }
 
         // Check if there's a track element with kind="captions" as a child
diff --git a/tests/lib/rules/template-require-media-caption.js b/tests/lib/rules/template-require-media-caption.js
index e943665965..c7d9f25b8e 100644
--- a/tests/lib/rules/template-require-media-caption.js
+++ b/tests/lib/rules/template-require-media-caption.js
@@ -34,6 +34,8 @@ ruleTester.run('template-require-media-caption', rule, {
     '<template><audio muted="true"></audio></template>',
     '<template><video muted></video></template>',
     '<template><audio muted={{this.muted}}></audio></template>',
+    '<template><video muted="{{isMuted}}"><source src="movie.mp4" /></video></template>',
+    '<template><audio muted="{{this.isMuted}}"></audio></template>',
     '<template><video><track kind="captions" /><track kind="descriptions" /></video></template>',
   ],
 
@@ -148,6 +150,8 @@ hbsRuleTester.run('template-require-media-caption', rule, {
     '<audio muted="true"></audio>',
     '<video muted></video>',
     '<audio muted={{this.muted}}></audio>',
+    '<video muted="{{isMuted}}"><source src="movie.mp4" /></video>',
+    '<audio muted="{{this.isMuted}}"></audio>',
     '<video><track kind="captions" /><track kind="descriptions" /></video>',
   ],
   invalid: [

From e3a8d8b43e2832387b93485cc5134f5621601fad Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johan=20R=C3=B8ed?= <johanroed@gmail.com>
Date: Mon, 13 Apr 2026 19:07:24 +0200
Subject: [PATCH 2/2] Fix lint: apply Prettier formatting

---
 lib/rules/template-require-media-caption.js | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/lib/rules/template-require-media-caption.js b/lib/rules/template-require-media-caption.js
index 4033211d9b..84d2e07bfd 100644
--- a/lib/rules/template-require-media-caption.js
+++ b/lib/rules/template-require-media-caption.js
@@ -58,10 +58,7 @@ module.exports = {
           // or muted={{#if ...}}...{{/if}} → BlockStatement) → treat as exempt.
           // Matches upstream ember-template-lint behavior where MustacheStatement,
           // BlockStatement, or any non-text/non-"false" value is considered muted.
-          if (
-            value.type !== 'GlimmerTextNode' &&
-            value.type !== 'GlimmerMustacheStatement'
-          ) {
+          if (value.type !== 'GlimmerTextNode' && value.type !== 'GlimmerMustacheStatement') {
             return;
           }
         }