From 816ada0f542c90772e88349a0dfa6e5cf4f73dae Mon Sep 17 00:00:00 2001
From: "[._.]/ Adam Eivy" <atomantic@gmail.com>
Date: Sat, 15 Aug 2026 19:01:07 +0000
Subject: [PATCH 1/5] fix: name the unlabeled form controls on the a11y sweep's
 high-value pages (#4156)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Adds accessible names to the previously-unlabeled inline inputs on
CharacterSheet, GitHub, Sharing, Browser, MoodBoardDetail and
VideoTimelineEditor: aria-label where the control lives in a compact
toolbar/inline row, and a FormField (visible label + htmlFor/id) for the
GitHub add-secret form, which is a stacked column with room for one.

The repo-wide guard in a11yConventions.test.js could not see through a
page-local field wrapper that renders its children inside a <label>
(PipelineSeries.jsx's <Field label="…">), so three already-accessible
controls sat in PREEXISTING_INPUT_NAME_ALLOWLIST purely as a
static-analysis gap. Teach the guard to recognise same-file
label-wrapping components and drop those entries too — 20 allowlist
entries removed in total, so these controls are now enforced.
---
 client/src/a11yConventions.test.js       | 74 +++++++++++++++++-------
 client/src/pages/Browser.jsx             |  1 +
 client/src/pages/CharacterSheet.jsx      |  8 +++
 client/src/pages/GitHub.jsx              | 34 ++++++-----
 client/src/pages/MoodBoardDetail.jsx     |  1 +
 client/src/pages/Sharing.jsx             |  2 +
 client/src/pages/VideoTimelineEditor.jsx |  2 +
 7 files changed, 88 insertions(+), 34 deletions(-)

diff --git a/client/src/a11yConventions.test.js b/client/src/a11yConventions.test.js
index 6395687473..6d73c915ff 100644
--- a/client/src/a11yConventions.test.js
+++ b/client/src/a11yConventions.test.js
@@ -576,6 +576,59 @@ function isNestedInLabeledFormField(src, index) {
   return false;
 }
 
+// Page-local field wrappers (PipelineSeries.jsx's `<Field label="…">`) render
+// their children inside a real <label>, so the control they wrap is implicitly
+// named — the <label> just lives in the component definition instead of at the
+// call site. Recognise those wrappers from their own source so a correctly
+// labeled control isn't forced to carry a redundant aria-label that would
+// shadow the visible text. Only same-file definitions count; a wrapper imported
+// from elsewhere stays unknown (FormField has its own dedicated check).
+function localLabelWrapperNames(src) {
+  const names = new Set();
+  const re = /function\s+([A-Z][\w]*)\s*\(/g;
+  let match;
+  while ((match = re.exec(src))) {
+    let depth = 1;
+    let cursor = match.index + match[0].length;
+    for (; cursor < src.length && depth > 0; cursor++) {
+      if (src[cursor] === '(') depth++;
+      else if (src[cursor] === ')') depth--;
+    }
+    const bodyStart = src.indexOf('{', cursor);
+    if (bodyStart === -1) continue;
+    const bodyEnd = matchingBraceEnd(src, bodyStart);
+    if (bodyEnd === -1) continue;
+    const body = src.slice(bodyStart, bodyEnd);
+    // The wrapper must render {children} between a <label> and its </label>.
+    if (/<label\b[\s\S]*?\{\s*children\s*\}[\s\S]*?<\/label>/.test(body)) names.add(match[1]);
+  }
+  return names;
+}
+
+function isNestedInLabelWrappingComponent(src, index) {
+  for (const name of localLabelWrapperNames(src)) {
+    const re = new RegExp(`</?${name}\\b`, 'g');
+    let depth = 0;
+    let outerLabel = null;
+    let match;
+    while ((match = re.exec(src)) && match.index < index) {
+      if (match[0].startsWith('</')) {
+        depth = Math.max(0, depth - 1);
+        continue;
+      }
+      const tag = openingTagAt(src, match.index, name.length + 1);
+      if (!tag) continue;
+      re.lastIndex = match.index + tag.length;
+      if (/\/\s*>$/.test(tag)) continue;
+      if (depth === 0) outerLabel = normalizedAttributeValue(attributeValue(tag, 'label'));
+      depth++;
+    }
+    if (depth === 0 || outerLabel === null || outerLabel === '') continue;
+    if (!/^(?:undefined|null|false)$/i.test(outerLabel)) return true;
+  }
+  return false;
+}
+
 function hasUsableAriaLabelledByReference(src, tag) {
   const raw = attributeValue(tag, 'aria-labelledby');
   const value = normalizedAttributeValue(raw);
@@ -599,6 +652,7 @@ function hasAccessibleInputName(src, tag, index) {
   if (hasUsableAccessibleNameAttribute(tag, 'aria-labelledby') && hasUsableAriaLabelledByReference(src, tag)) return true;
   if (hasUsableNativeInputName(tag)) return true;
   if (isNestedInLabel(src, index) || isNestedInLabeledFormField(src, index)) return true;
+  if (isNestedInLabelWrappingComponent(src, index)) return true;
 
   const id = normalizedAttributeValue(attributeValue(tag, 'id'));
   return id !== null && id !== '' && hasMatchingExplicitLabel(src, id);
@@ -836,25 +890,7 @@ const PREEXISTING_INPUT_NAME_ALLOWLIST = new Set([
   "src/pages/AIProviders.jsx|type=newEnvSecret ? 'password' : 'text'|placeholder=value|value=newEnvValue",
   "src/pages/Authors.jsx|placeholder=Jane Doe|value=form.name",
   "src/pages/Authors.jsx|placeholder=/images/… or https://…|value=form.headshotImageUrl",
-  "src/pages/Browser.jsx|type=text|placeholder=https://example.com|value=navUrl",
-  "src/pages/CharacterSheet.jsx|value=classVal",
-  "src/pages/CharacterSheet.jsx|placeholder=1d8|value=dmgDice",
-  "src/pages/CharacterSheet.jsx|placeholder=Description (optional)|value=dmgDesc",
-  "src/pages/CharacterSheet.jsx|type=number|placeholder=XP amount|value=xpAmount",
-  "src/pages/CharacterSheet.jsx|placeholder=Description (optional)|value=xpDesc",
-  "src/pages/CharacterSheet.jsx|placeholder=What happened?|value=evtDesc",
-  "src/pages/CharacterSheet.jsx|type=number|placeholder=XP (optional)|value=evtXp",
-  "src/pages/CharacterSheet.jsx|placeholder=Dice (e.g. 2d6)|value=evtDice",
-  "src/pages/GitHub.jsx|type=text|placeholder=Search repos...|value=search",
-  "src/pages/GitHub.jsx|type=text|placeholder=SECRET_NAME|value=newSecretName",
-  "src/pages/GitHub.jsx|type=password|placeholder=Secret value|value=newSecretValue",
   "src/pages/MediaCollectionDetail.jsx|type=text|value=nameDraft",
-  "src/pages/MoodBoardDetail.jsx|type=text|placeholder=Add a caption…",
-  "src/pages/PipelineSeries.jsx|value=series.name || ''",
-  "src/pages/PipelineSeries.jsx|placeholder=One-sentence pitch|value=series.logline || ''",
-  "src/pages/PipelineSeries.jsx|type=number|value=series.issueCountTarget || 0|min=0|max=999",
-  "src/pages/Sharing.jsx|type=text|placeholder=Display name (e.g. atomantic)|value=sharingDisplayName",
-  "src/pages/Sharing.jsx|type=text|placeholder=Optional bio / contact note (visible to recipients)|value=sharingBio",
   "src/components/meatspace/post/PostDrillConfig.jsx|type=number|value=drillConfig[field.key] ?? ''|min=field.min|max=field.max",
   "src/pages/AIProviders.jsx|type=text|placeholder=claude-sonnet-4-20250514|value=formData.defaultModel",
   "src/pages/AIProviders.jsx|type=text|placeholder=haiku|value=formData.lightModel",
@@ -875,8 +911,6 @@ const PREEXISTING_INPUT_NAME_ALLOWLIST = new Set([
   "src/pages/StackerNews.jsx|id=`${prefix}-api-key`|type=password|value=form.apiKey",
   "src/pages/StackerNews.jsx|id=`${prefix}-slug`|value=form.slug",
   "src/pages/VideoTimeline.jsx|type=text|placeholder=New project name…|value=name",
-  "src/pages/VideoTimelineEditor.jsx|type=text|value=nameDraft",
-  "src/pages/VideoTimelineEditor.jsx|type=range|value=Math.min(t, total)|min=0|max=Math.max(0.01, total)|step=0.01",
   "src/pages/DataDog.jsx|name=site|type=text|placeholder=e.g., api.custom-datadog.com|value=formData.site",
 ]);
 
diff --git a/client/src/pages/Browser.jsx b/client/src/pages/Browser.jsx
index d4b69416ed..e19e15e347 100644
--- a/client/src/pages/Browser.jsx
+++ b/client/src/pages/Browser.jsx
@@ -445,6 +445,7 @@ export default function BrowserPage() {
               >
                 <input
                   type="text"
+                  aria-label="URL to open"
                   value={navUrl}
                   onChange={e => setNavUrl(e.target.value)}
                   placeholder="https://example.com"
diff --git a/client/src/pages/CharacterSheet.jsx b/client/src/pages/CharacterSheet.jsx
index 755c9ce6a0..2a67b0df64 100644
--- a/client/src/pages/CharacterSheet.jsx
+++ b/client/src/pages/CharacterSheet.jsx
@@ -521,6 +521,7 @@ export default function CharacterSheet() {
                 {editingClass ? (
                   <input
                     autoFocus
+                    aria-label="Character title"
                     value={classVal}
                     onChange={e => setClassVal(e.target.value)}
                     onBlur={handleClassSave}
@@ -736,6 +737,7 @@ export default function CharacterSheet() {
                 <div className="flex items-center gap-2">
                   <Dices className="w-4 h-4 text-gray-400" />
                   <input
+                    aria-label="Damage dice"
                     value={dmgDice}
                     onChange={e => setDmgDice(e.target.value)}
                     placeholder="1d8"
@@ -743,6 +745,7 @@ export default function CharacterSheet() {
                   />
                 </div>
                 <input
+                  aria-label="Damage description"
                   value={dmgDesc}
                   onChange={e => setDmgDesc(e.target.value)}
                   placeholder="Description (optional)"
@@ -769,12 +772,14 @@ export default function CharacterSheet() {
               <div className="flex flex-col sm:flex-row gap-2">
                 <input
                   type="number"
+                  aria-label="XP amount"
                   value={xpAmount}
                   onChange={e => setXpAmount(e.target.value)}
                   placeholder="XP amount"
                   className="bg-port-card border border-port-border rounded px-2 py-1.5 text-sm text-white w-28"
                 />
                 <input
+                  aria-label="XP description"
                   value={xpDesc}
                   onChange={e => setXpDesc(e.target.value)}
                   placeholder="Description (optional)"
@@ -800,6 +805,7 @@ export default function CharacterSheet() {
               </div>
               <div className="flex flex-col gap-2">
                 <input
+                  aria-label="Event description"
                   value={evtDesc}
                   onChange={e => setEvtDesc(e.target.value)}
                   placeholder="What happened?"
@@ -808,6 +814,7 @@ export default function CharacterSheet() {
                 <div className="flex flex-col sm:flex-row gap-2">
                   <input
                     type="number"
+                    aria-label="Event XP"
                     value={evtXp}
                     onChange={e => setEvtXp(e.target.value)}
                     placeholder="XP (optional)"
@@ -816,6 +823,7 @@ export default function CharacterSheet() {
                   <div className="flex items-center gap-2">
                     <Dices className="w-4 h-4 text-gray-400" />
                     <input
+                      aria-label="Event dice"
                       value={evtDice}
                       onChange={e => setEvtDice(e.target.value)}
                       placeholder="Dice (e.g. 2d6)"
diff --git a/client/src/pages/GitHub.jsx b/client/src/pages/GitHub.jsx
index dcf73671cc..b0ca15d357 100644
--- a/client/src/pages/GitHub.jsx
+++ b/client/src/pages/GitHub.jsx
@@ -13,6 +13,7 @@ import {
 import { timeAgo } from '../utils/formatters';
 import PageSkeleton from '../components/ui/PageSkeleton';
 import Modal from '../components/ui/Modal';
+import { FormField } from '../components/ui/FormField';
 
 const FILTERS = ['all', 'npm', 'secrets', 'archived'];
 
@@ -228,6 +229,7 @@ export default function GitHub() {
           <div className="flex flex-col sm:flex-row gap-2 mb-4">
             <input
               type="text"
+              aria-label="Search repositories"
               value={search}
               onChange={(e) => setSearch(e.target.value)}
               placeholder="Search repos..."
@@ -385,20 +387,24 @@ export default function GitHub() {
             )}
 
             <div className="flex flex-col gap-2">
-              <input
-                type="text"
-                value={newSecretName}
-                onChange={(e) => setNewSecretName(e.target.value.toUpperCase().replace(/[^A-Z0-9_]/g, ''))}
-                placeholder="SECRET_NAME"
-                className="px-3 py-2 bg-port-bg border border-port-border rounded text-white font-mono text-sm"
-              />
-              <input
-                type="password"
-                value={newSecretValue}
-                onChange={(e) => setNewSecretValue(e.target.value)}
-                placeholder="Secret value"
-                className="px-3 py-2 bg-port-bg border border-port-border rounded text-white text-sm"
-              />
+              <FormField label="Secret name">
+                <input
+                  type="text"
+                  value={newSecretName}
+                  onChange={(e) => setNewSecretName(e.target.value.toUpperCase().replace(/[^A-Z0-9_]/g, ''))}
+                  placeholder="SECRET_NAME"
+                  className="w-full px-3 py-2 bg-port-bg border border-port-border rounded text-white font-mono text-sm"
+                />
+              </FormField>
+              <FormField label="Secret value">
+                <input
+                  type="password"
+                  value={newSecretValue}
+                  onChange={(e) => setNewSecretValue(e.target.value)}
+                  placeholder="Secret value"
+                  className="w-full px-3 py-2 bg-port-bg border border-port-border rounded text-white text-sm"
+                />
+              </FormField>
               <button
                 onClick={handleSaveSecret}
                 disabled={savingSecret || !newSecretName.trim() || !newSecretValue}
diff --git a/client/src/pages/MoodBoardDetail.jsx b/client/src/pages/MoodBoardDetail.jsx
index 881069241b..c4037171a2 100644
--- a/client/src/pages/MoodBoardDetail.jsx
+++ b/client/src/pages/MoodBoardDetail.jsx
@@ -605,6 +605,7 @@ export default function MoodBoardDetail() {
                 <div className="p-2 flex flex-col gap-1">
                   <input
                     type="text"
+                    aria-label="Item caption"
                     defaultValue={item.caption || ''}
                     placeholder="Add a caption…"
                     maxLength={2000}
diff --git a/client/src/pages/Sharing.jsx b/client/src/pages/Sharing.jsx
index 4e6acd6f5f..17f93e65ac 100644
--- a/client/src/pages/Sharing.jsx
+++ b/client/src/pages/Sharing.jsx
@@ -316,6 +316,7 @@ function SharingBuckets({ selectedId }) {
         <div className="grid grid-cols-1 md:grid-cols-[1fr_2fr_auto] gap-3 items-start">
           <input
             type="text"
+            aria-label="Your display name"
             value={sharingDisplayName}
             onChange={(e) => setSharingDisplayName(e.target.value)}
             placeholder="Display name (e.g. atomantic)"
@@ -324,6 +325,7 @@ function SharingBuckets({ selectedId }) {
           />
           <input
             type="text"
+            aria-label="Your bio / contact note"
             value={sharingBio}
             onChange={(e) => setSharingBio(e.target.value)}
             placeholder="Optional bio / contact note (visible to recipients)"
diff --git a/client/src/pages/VideoTimelineEditor.jsx b/client/src/pages/VideoTimelineEditor.jsx
index 0338ca2378..3f4c149733 100644
--- a/client/src/pages/VideoTimelineEditor.jsx
+++ b/client/src/pages/VideoTimelineEditor.jsx
@@ -529,6 +529,7 @@ export default function VideoTimelineEditor() {
           </button>
           <input
             type="text"
+            aria-label="Project name"
             value={nameDraft}
             onChange={(e) => setNameDraft(e.target.value)}
             onBlur={async (e) => {
@@ -626,6 +627,7 @@ export default function VideoTimelineEditor() {
             </button>
             <input
               type="range"
+              aria-label="Playhead position"
               min={0}
               max={Math.max(0.01, total)}
               step={0.01}

From 7ae289b640dd8052397b05171d7cbb60649f7135 Mon Sep 17 00:00:00 2001
From: "[._.]/ Adam Eivy" <atomantic@gmail.com>
Date: Sat, 15 Aug 2026 19:01:14 +0000
Subject: [PATCH 2/5] docs: add changelog fragment for the form-control a11y
 sweep (#4156)

---
 .changelog/next/fixed-issue-4156.md | 1 +
 1 file changed, 1 insertion(+)
 create mode 100644 .changelog/next/fixed-issue-4156.md

diff --git a/.changelog/next/fixed-issue-4156.md b/.changelog/next/fixed-issue-4156.md
new file mode 100644
index 0000000000..4a6372101d
--- /dev/null
+++ b/.changelog/next/fixed-issue-4156.md
@@ -0,0 +1 @@
+- Screen readers now announce the inline form controls on the Character Sheet, GitHub, Sharing, Browser, Mood Board and Video Timeline Editor pages, which previously had no accessible name

From 7ea919157eab5e18d909669b0705034955cdbec2 Mon Sep 17 00:00:00 2001
From: "[._.]/ Adam Eivy" <atomantic@gmail.com>
Date: Sat, 15 Aug 2026 19:02:14 +0000
Subject: [PATCH 3/5] perf: memoize the label-wrapper scan per source file in
 the a11y guard (#4156)

---
 client/src/a11yConventions.test.js | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/client/src/a11yConventions.test.js b/client/src/a11yConventions.test.js
index 6d73c915ff..0ad396aaff 100644
--- a/client/src/a11yConventions.test.js
+++ b/client/src/a11yConventions.test.js
@@ -582,8 +582,14 @@ function isNestedInLabeledFormField(src, index) {
 // call site. Recognise those wrappers from their own source so a correctly
 // labeled control isn't forced to carry a redundant aria-label that would
 // shadow the visible text. Only same-file definitions count; a wrapper imported
-// from elsewhere stays unknown (FormField has its own dedicated check).
+// from elsewhere stays unknown (FormField has its own dedicated check), and
+// only `function` declarations are matched — a missed wrapper is a false
+// negative that simply leaves its controls on the allowlist.
+const labelWrapperNamesBySource = new Map();
+
 function localLabelWrapperNames(src) {
+  const cached = labelWrapperNamesBySource.get(src);
+  if (cached) return cached;
   const names = new Set();
   const re = /function\s+([A-Z][\w]*)\s*\(/g;
   let match;
@@ -602,6 +608,7 @@ function localLabelWrapperNames(src) {
     // The wrapper must render {children} between a <label> and its </label>.
     if (/<label\b[\s\S]*?\{\s*children\s*\}[\s\S]*?<\/label>/.test(body)) names.add(match[1]);
   }
+  labelWrapperNamesBySource.set(src, names);
   return names;
 }
 

From 60250dd34dcc4767effea32f4f639693ef6cb8ba Mon Sep 17 00:00:00 2001
From: "[._.]/ Adam Eivy" <atomantic@gmail.com>
Date: Sat, 15 Aug 2026 19:05:20 +0000
Subject: [PATCH 4/5] address review (antigravity): harden the a11y guard's
 local label-wrapper detection (#4156)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Skip the wrapper's parameter list with the string-aware balancedCallAt so a
  default value containing ')' cannot point the body scan at the destructuring.
- Require {children} to sit inside a <label> with no intervening </label>, so a
  component rendering sibling labels around its children can't register as a
  wrapper and exempt controls it never labels.
- Track every still-open wrapper instance rather than only the outermost, so an
  inner <Field label="…"> nested in an unlabeled outer one still counts.
---
 client/src/a11yConventions.test.js | 34 ++++++++++++++++--------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/client/src/a11yConventions.test.js b/client/src/a11yConventions.test.js
index 0ad396aaff..0cac0b2868 100644
--- a/client/src/a11yConventions.test.js
+++ b/client/src/a11yConventions.test.js
@@ -594,19 +594,21 @@ function localLabelWrapperNames(src) {
   const re = /function\s+([A-Z][\w]*)\s*\(/g;
   let match;
   while ((match = re.exec(src))) {
-    let depth = 1;
-    let cursor = match.index + match[0].length;
-    for (; cursor < src.length && depth > 0; cursor++) {
-      if (src[cursor] === '(') depth++;
-      else if (src[cursor] === ')') depth--;
-    }
-    const bodyStart = src.indexOf('{', cursor);
+    // Skip the parameter list with the string-aware scanner — a default value
+    // like `{ label = ')' }` would close the parens early on a naive count and
+    // point `bodyStart` at the destructuring instead of the body.
+    const parenIndex = match.index + match[0].length - 1;
+    const params = balancedCallAt(src, parenIndex);
+    if (!params) continue;
+    const bodyStart = src.indexOf('{', parenIndex + params.length);
     if (bodyStart === -1) continue;
     const bodyEnd = matchingBraceEnd(src, bodyStart);
     if (bodyEnd === -1) continue;
     const body = src.slice(bodyStart, bodyEnd);
-    // The wrapper must render {children} between a <label> and its </label>.
-    if (/<label\b[\s\S]*?\{\s*children\s*\}[\s\S]*?<\/label>/.test(body)) names.add(match[1]);
+    // {children} must sit inside a <label> — no </label> may intervene, or a
+    // component rendering `<label>Header</label>{children}<label>Footer</label>`
+    // would register as a wrapper and exempt controls it never labels.
+    if (/<label\b[^>]*>(?:(?!<\/label>)[\s\S])*?\{\s*children\s*\}/.test(body)) names.add(match[1]);
   }
   labelWrapperNamesBySource.set(src, names);
   return names;
@@ -615,23 +617,23 @@ function localLabelWrapperNames(src) {
 function isNestedInLabelWrappingComponent(src, index) {
   for (const name of localLabelWrapperNames(src)) {
     const re = new RegExp(`</?${name}\\b`, 'g');
-    let depth = 0;
-    let outerLabel = null;
+    // Keep the label of every wrapper instance still open at `index`, not just
+    // the outermost: an inner `<Field label="…">` nested in an unlabeled outer
+    // one still names the control.
+    const open = [];
     let match;
     while ((match = re.exec(src)) && match.index < index) {
       if (match[0].startsWith('</')) {
-        depth = Math.max(0, depth - 1);
+        open.pop();
         continue;
       }
       const tag = openingTagAt(src, match.index, name.length + 1);
       if (!tag) continue;
       re.lastIndex = match.index + tag.length;
       if (/\/\s*>$/.test(tag)) continue;
-      if (depth === 0) outerLabel = normalizedAttributeValue(attributeValue(tag, 'label'));
-      depth++;
+      open.push(normalizedAttributeValue(attributeValue(tag, 'label')));
     }
-    if (depth === 0 || outerLabel === null || outerLabel === '') continue;
-    if (!/^(?:undefined|null|false)$/i.test(outerLabel)) return true;
+    if (open.some((label) => label && !/^(?:undefined|null|false)$/i.test(label))) return true;
   }
   return false;
 }

From 171bcab12863d917055e3cac65ab1ddfa05a572a Mon Sep 17 00:00:00 2001
From: "[._.]/ Adam Eivy" <atomantic@gmail.com>
Date: Sat, 15 Aug 2026 19:09:10 +0000
Subject: [PATCH 5/5] address review (antigravity): reject a self-closing
 <label> as a wrapper tell (#4156)

---
 client/src/a11yConventions.test.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/client/src/a11yConventions.test.js b/client/src/a11yConventions.test.js
index 0cac0b2868..f85afb3122 100644
--- a/client/src/a11yConventions.test.js
+++ b/client/src/a11yConventions.test.js
@@ -607,8 +607,10 @@ function localLabelWrapperNames(src) {
     const body = src.slice(bodyStart, bodyEnd);
     // {children} must sit inside a <label> — no </label> may intervene, or a
     // component rendering `<label>Header</label>{children}<label>Footer</label>`
-    // would register as a wrapper and exempt controls it never labels.
-    if (/<label\b[^>]*>(?:(?!<\/label>)[\s\S])*?\{\s*children\s*\}/.test(body)) names.add(match[1]);
+    // would register as a wrapper and exempt controls it never labels. The
+    // opening tag must also not be self-closing (`<label … />` wraps nothing,
+    // so anything after it is outside the label).
+    if (/<label\b(?:[^>]*[^/>])?>(?:(?!<\/label>)[\s\S])*?\{\s*children\s*\}/.test(body)) names.add(match[1]);
   }
   labelWrapperNamesBySource.set(src, names);
   return names;