@adia-ai/web-components 0.5.15 → 0.5.17

package/core/template.test.js

@@ -95,3 +95,87 @@ describe('html template — §250 (v0.5.11) ?attr=${bool} silent-failure trap',
     expect(warnSpy).not.toHaveBeenCalled();
   });
 });
+
+describe('html template — FB-40 (v0.5.16) apostrophe in HTML comments does NOT mis-classify subsequent placeholders', () => {
+  // FEEDBACK-06 §1 P0 (long resolved) made `inTag()` comment-aware:
+  // when scan() hits `<!--` outside a string context, it jumps past
+  // `-->` wholesale. FB-40 (filed 2026-05-16 post-v0.5.16 cut) reported
+  // crashes attributed to apostrophes in comments. This suite verifies
+  // empirically that the comment-aware logic still holds — the FB-40
+  // diagnosis is mistaken about the mechanism (the existing fix covers
+  // the apostrophe-in-comment case).
+
+  let container;
+  let warnSpy;
+
+  beforeEach(() => {
+    container = document.createElement('div');
+    document.body.appendChild(container);
+    warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    container.remove();
+    warnSpy.mockRestore();
+  });
+
+  it('@event=${handler} placeholder AFTER apostrophe-bearing comment is classified as attribute-position', () => {
+    let received = null;
+    const handler = (e) => { received = e; };
+    const tpl = html`
+      <div>
+        <!-- editor-shell's named slots -->
+        <button data-test="probe" @click=${handler}>X</button>
+      </div>
+    `;
+    stamp(tpl, container);
+    const btn = container.querySelector('[data-test="probe"]');
+    expect(btn).not.toBeNull();
+    btn.click();
+    expect(received).not.toBeNull();
+    expect(received.type).toBe('click');
+  });
+
+  it('double-quote in HTML comment does not mis-classify subsequent attribute placeholder', () => {
+    const tpl = html`
+      <div>
+        <!-- the "Stop Override" button below -->
+        <button data-test="probe" data-id=${'42'}></button>
+      </div>
+    `;
+    stamp(tpl, container);
+    const btn = container.querySelector('[data-test="probe"]');
+    expect(btn).not.toBeNull();
+    expect(btn.getAttribute('data-id')).toBe('42');
+  });
+
+  it('multiple apostrophes across multiple comments still keep subsequent placeholders attribute-position', () => {
+    let received = null;
+    const tpl = html`
+      <div>
+        <!-- one's first comment -->
+        <span>x</span>
+        <!-- two's second comment -->
+        <button data-test="probe" @click=${(e) => { received = e; }}></button>
+      </div>
+    `;
+    stamp(tpl, container);
+    container.querySelector('[data-test="probe"]').click();
+    expect(received).not.toBeNull();
+  });
+
+  it('apostrophe + em-dash (U+2014) in a single comment does not break parsing', () => {
+    // dts-export-drawer.ts case from FB-40's cited crashes:
+    // <!-- 2026-05-14 — migrated number inputs ... -->
+    let received = null;
+    const tpl = html`
+      <div>
+        <!-- 2026-05-14 — author's notes about migrated number inputs -->
+        <button data-test="probe" @click=${(e) => { received = e; }}></button>
+      </div>
+    `;
+    stamp(tpl, container);
+    container.querySelector('[data-test="probe"]').click();
+    expect(received).not.toBeNull();
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adia-ai/web-components",
-  "version": "0.5.15",
+  "version": "0.5.17",
   "description": "AdiaUI web components — vanilla custom elements. A2UI runtime (renderer, registry, streams, wiring) lives in @adia-ai/a2ui-runtime.",
   "type": "module",
   "types": "./index.d.ts",