From 96113f7df105b241fcdbdf0a8dc7e07afa130de2 Mon Sep 17 00:00:00 2001
From: dupontbertrand <dupontbertrand59@gmail.com>
Date: Fri, 3 Apr 2026 02:00:53 +0200
Subject: [PATCH 1/2] fix: improve error messages for invalid template tags in
 HTML start tags

Replace the generic "Reactive HTML attributes must either have a constant name..."
message with specific, actionable errors for each case:
- {{#if}}/{{#unless}} in start tag: suggests attribute="{{helper}}" pattern
- {{{triple-stache}}} in start tag: explains raw HTML issue
- {{> inclusion}} in start tag: mentions it's not allowed
- Custom block helpers: clear rejection message
---
 .../spacebars-compiler/spacebars_tests.js     | 25 +++++++++++++--
 packages/spacebars-compiler/templatetag.js    | 31 +++++++++++++++++--
 2 files changed, 51 insertions(+), 5 deletions(-)

diff --git a/packages/spacebars-compiler/spacebars_tests.js b/packages/spacebars-compiler/spacebars_tests.js
index 530a47bc3..390f1779d 100644
--- a/packages/spacebars-compiler/spacebars_tests.js
+++ b/packages/spacebars-compiler/spacebars_tests.js
@@ -246,12 +246,29 @@ Tinytest.add("spacebars-compiler - parse", function (test) {
   test.equal(BlazeTools.toJS(SpacebarsCompiler.parse('{{#foo}}x{{else bar}}{{#baz}}z{{/baz}}{{/foo}}')),
              'SpacebarsCompiler.TemplateTag({type: "BLOCKOPEN", path: ["foo"], content: "x", elseContent: SpacebarsCompiler.TemplateTag({type: "BLOCKOPEN", path: ["bar"], content: SpacebarsCompiler.TemplateTag({type: "BLOCKOPEN", path: ["baz"], content: "z"})})})');
 
+  // Triple-stache in start tag — should mention raw HTML
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{{x}}}></a>');
-  });
+  }, /Triple-stache.*raw HTML/);
+
+  // Block helper in start tag — should suggest attribute="{{helper}}" pattern
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{#if x}}{{/if}}></a>');
-  });
+  }, /not allowed in an HTML start tag/);
+
+  // Common real-world patterns that trigger this error
+  test.throws(function () {
+    SpacebarsCompiler.parse('<input {{#if isDisabled}}disabled{{/if}}>');
+  }, /Bad syntax.*Good syntax/);
+  test.throws(function () {
+    SpacebarsCompiler.parse('<option {{#unless isActive}}selected{{/unless}}>');
+  }, /not allowed in an HTML start tag/);
+
+  // Custom block helper in start tag
+  test.throws(function () {
+    SpacebarsCompiler.parse('<a {{#myHelper}}class="foo"{{/myHelper}}></a>');
+  }, /not allowed in an HTML start tag/);
+
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{k}}={[v}}></a>');
   });
@@ -261,9 +278,11 @@ Tinytest.add("spacebars-compiler - parse", function (test) {
   test.throws(function () {
     SpacebarsCompiler.parse('<a x{{y}}=z></a>');
   });
+
+  // Template inclusion in start tag — should mention it's not allowed
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{> x}}></a>');
-  });
+  }, /template inclusions are not allowed/);
 
   test.equal(BlazeTools.toJS(SpacebarsCompiler.parse('<a {{! x--}} b=c{{! x}} {{! x}}></a>')),
              'HTML.A({b: "c"})');
diff --git a/packages/spacebars-compiler/templatetag.js b/packages/spacebars-compiler/templatetag.js
index bb2554532..982226de9 100644
--- a/packages/spacebars-compiler/templatetag.js
+++ b/packages/spacebars-compiler/templatetag.js
@@ -517,8 +517,35 @@ const validateTag = function (ttag, scanner) {
       scanner.fatal(`${ttag.type} template tag is not allowed in an HTML attribute`);
     }
   } else if (position === TEMPLATE_TAG_POSITION.IN_START_TAG) {
-    if (! (ttag.type === 'DOUBLE')) {
-      scanner.fatal(`Reactive HTML attributes must either have a constant name or consist of a single {{helper}} providing a dictionary of names and values.  A template tag of type ${ttag.type} is not allowed here.`);
+    if (ttag.type === 'BLOCKOPEN') {
+      const path0 = ttag.path[0];
+      const isBuiltIn = ttag.path.length === 1 && (path0 === 'if' || path0 === 'unless' || path0 === 'with' || path0 === 'each');
+      if (isBuiltIn) {
+        scanner.fatal(
+          `{{#${path0}}} is not allowed in an HTML start tag. ` +
+          'To conditionally add a boolean attribute like "disabled", "checked", or "selected", ' +
+          'use the attribute="{{helper}}" syntax instead.\n' +
+          '  Bad syntax:  <input {{#if isDisabled}}disabled{{/if}}>\n' +
+          '  Good syntax: <input disabled="{{isDisabled}}">'
+        );
+      } else {
+        scanner.fatal(
+          `{{#${path0}}} is not allowed in an HTML start tag. ` +
+          'Only a single {{helper}} returning a dictionary of attribute name=value pairs is allowed here.'
+        );
+      }
+    } else if (ttag.type === 'TRIPLE') {
+      scanner.fatal(
+        'Triple-stache {{{...}}} is not allowed in an HTML start tag because it outputs raw HTML. ' +
+        'Use a double-stache {{helper}} that returns a dictionary of attribute name=value pairs instead.'
+      );
+    } else if (ttag.type === 'INCLUSION') {
+      scanner.fatal(
+        `{{> ...}} template inclusions are not allowed in an HTML start tag. ` +
+        'Only a single {{helper}} returning a dictionary of attribute name=value pairs is allowed here.'
+      );
+    } else if (ttag.type !== 'DOUBLE') {
+      scanner.fatal(`A template tag of type "${ttag.type}" is not allowed in an HTML start tag.`);
     }
     if (scanner.peek() === '=') {
       scanner.fatal("Template tags are not allowed in attribute names, only in attribute values or in the form of a single {{helper}} that evaluates to a dictionary of name=value pairs.");

From 919524f6a4aed9e4036d149f5899636fb6ff0a65 Mon Sep 17 00:00:00 2001
From: dupontbertrand <dupontbertrand59@gmail.com>
Date: Fri, 3 Apr 2026 13:19:16 +0200
Subject: [PATCH 2/2] fix: use string matching instead of regex in test.throws

Switch from regex to string substring matching for test.throws
assertions, which is more reliable across Tinytest versions.
---
 packages/spacebars-compiler/spacebars_tests.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/packages/spacebars-compiler/spacebars_tests.js b/packages/spacebars-compiler/spacebars_tests.js
index 390f1779d..01ea0f725 100644
--- a/packages/spacebars-compiler/spacebars_tests.js
+++ b/packages/spacebars-compiler/spacebars_tests.js
@@ -249,25 +249,25 @@ Tinytest.add("spacebars-compiler - parse", function (test) {
   // Triple-stache in start tag — should mention raw HTML
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{{x}}}></a>');
-  }, /Triple-stache.*raw HTML/);
+  }, 'not allowed in an HTML start tag');
 
   // Block helper in start tag — should suggest attribute="{{helper}}" pattern
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{#if x}}{{/if}}></a>');
-  }, /not allowed in an HTML start tag/);
+  }, 'not allowed in an HTML start tag');
 
   // Common real-world patterns that trigger this error
   test.throws(function () {
     SpacebarsCompiler.parse('<input {{#if isDisabled}}disabled{{/if}}>');
-  }, /Bad syntax.*Good syntax/);
+  }, 'Bad syntax');
   test.throws(function () {
     SpacebarsCompiler.parse('<option {{#unless isActive}}selected{{/unless}}>');
-  }, /not allowed in an HTML start tag/);
+  }, 'not allowed in an HTML start tag');
 
   // Custom block helper in start tag
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{#myHelper}}class="foo"{{/myHelper}}></a>');
-  }, /not allowed in an HTML start tag/);
+  }, 'not allowed in an HTML start tag');
 
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{k}}={[v}}></a>');
@@ -282,7 +282,7 @@ Tinytest.add("spacebars-compiler - parse", function (test) {
   // Template inclusion in start tag — should mention it's not allowed
   test.throws(function () {
     SpacebarsCompiler.parse('<a {{> x}}></a>');
-  }, /template inclusions are not allowed/);
+  }, 'template inclusions are not allowed');
 
   test.equal(BlazeTools.toJS(SpacebarsCompiler.parse('<a {{! x--}} b=c{{! x}} {{! x}}></a>')),
              'HTML.A({b: "c"})');