@charcuterie/ui 2.15.0 → 2.16.0

package/dist/Overlay/useAnchoredOverlay.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useAnchoredOverlay.d.ts","sourceRoot":"","sources":["../../src/Overlay/useAnchoredOverlay.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAA;AAYnD,OAAO,KAAK,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,OAAO,CAAA;AAGpD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4CG;AACH,MAAM,MAAM,mBAAmB,GAC3B,QAAQ,GACR,SAAS,GACT,MAAM,CAAA;AAEV,MAAM,MAAM,yBAAyB,GAAG;IACtC;;;;;;;OAOG;IACH,SAAS,CAAC,EAAE,SAAS,CAAC,WAAW,GAAG,IAAI,CAAC,CAAA;IACzC;;;;;;OAMG;IACH,mBAAmB,CAAC,EAAE,OAAO,CAAA;IAC7B,yBAAyB,CAAC,EAAE,OAAO,CAAA;IACnC,+DAA+D;IAC/D,qBAAqB,CAAC,EAAE,OAAO,CAAA;IAC/B,SAAS,EAAE,OAAO,CAAA;IAClB,iEAAiE;IACjE,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB;;;;;;OAMG;IACH,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,0CAA0C;IAC1C,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,SAAS,EAAE,MAAM,IAAI,CAAA;IACrB,SAAS,CAAC,EAAE,SAAS,CAAA;IACrB,IAAI,EAAE,mBAAmB,CAAA;IACzB;;;OAGG;IACH,OAAO,CAAC,EAAE,YAAY,CAAA;CACvB,CAAA;AAQD,eAAO,MAAM,kBAAkB,GAAI,6KAahC,yBAAyB;;;;;;;;;;;;;;;;;;;;;;;;CAoI3B,CAAA"}
+{"version":3,"file":"useAnchoredOverlay.d.ts","sourceRoot":"","sources":["../../src/Overlay/useAnchoredOverlay.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAA;AAYnD,OAAO,KAAK,EAAE,YAAY,EAAE,SAAS,EAAE,MAAM,OAAO,CAAA;AAGpD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4CG;AACH,MAAM,MAAM,mBAAmB,GAC3B,QAAQ,GACR,SAAS,GACT,MAAM,CAAA;AAEV,MAAM,MAAM,yBAAyB,GAAG;IACtC;;;;;;;OAOG;IACH,SAAS,CAAC,EAAE,SAAS,CAAC,WAAW,GAAG,IAAI,CAAC,CAAA;IACzC;;;;;;OAMG;IACH,mBAAmB,CAAC,EAAE,OAAO,CAAA;IAC7B,yBAAyB,CAAC,EAAE,OAAO,CAAA;IACnC,+DAA+D;IAC/D,qBAAqB,CAAC,EAAE,OAAO,CAAA;IAC/B,SAAS,EAAE,OAAO,CAAA;IAClB,iEAAiE;IACjE,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB;;;;;;OAMG;IACH,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,0CAA0C;IAC1C,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,SAAS,EAAE,MAAM,IAAI,CAAA;IACrB,SAAS,CAAC,EAAE,SAAS,CAAA;IACrB,IAAI,EAAE,mBAAmB,CAAA;IACzB;;;OAGG;IACH,OAAO,CAAC,EAAE,YAAY,CAAA;CACvB,CAAA;AAQD,eAAO,MAAM,kBAAkB,GAAI,6KAahC,yBAAyB;;;;;;;;;;;;;;;;;;;;;;;;CAoK3B,CAAA"}

package/dist/Overlay/useAnchoredOverlay.js

@@ -75,7 +75,36 @@ export const useAnchoredOverlay = ({ anchorRef, isEscapeDismissable = true, isOu
     // a `listbox` panel needs, since a bare `role="listbox"` is an ARIA
     // input field and must be named.
     const generatedTriggerId = useUniqueId();
-    const triggerId = referenceProps.id ??
+    /**
+     * The trigger's OWN id wins over both.
+     *
+     * This hook used to mint an id and clone it over whatever the
+     * trigger already had, on the reasoning that the panel needs a
+     * stable target for `aria-labelledby`. It does — but any id serves
+     * that purpose, including the caller's, and overwriting had a
+     * consequence nobody traced: `Field` clones a `controlId` onto its
+     * child and renders `<label htmlFor={controlId}>`, so for every
+     * `Picker`/`Listbox`/`Combobox`/`Menu` inside a `Field` the label
+     * pointed at an element that did not exist. Measured in a real
+     * browser: zero nodes carried the id the `<label>` named.
+     *
+     * That is the exact defect `Field`'s docstring calls "precisely the
+     * defect this component was built to make impossible" — arriving by
+     * a different route, and silent, because a dangling `htmlFor`
+     * throws nothing and shows nothing.
+     *
+     * Preferring the child's id fixes the label without weakening the
+     * panel: `aria-labelledby` now points at the caller's id instead of
+     * a generated one, which is the same element either way.
+     *
+     * Two rediscoveries of the overwrite are already in the fleet —
+     * queuepilot and mux-magic both moved their e2e handles to
+     * `data-testid` because `id` "did not survive". `data-testid` is
+     * still the more robust handle, but `id` is no longer a trap.
+     */
+    const triggerOwnId = trigger?.props?.id;
+    const triggerId = triggerOwnId ??
+        referenceProps.id ??
         generatedTriggerId;
     // In anchor mode floating-ui's reference is the consumer's element, set
     // imperatively — there is no cloned trigger to carry `refs.setReference`.

package/dist/Overlay/useAnchoredOverlay.js.map

@@ -1 +1 @@
-{"version":3,"file":"useAnchoredOverlay.js","sourceRoot":"","sources":["../../src/Overlay/useAnchoredOverlay.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,WAAW,GACZ,MAAM,oBAAoB,CAAA;AAE3B,OAAO,EACL,UAAU,EACV,IAAI,EACJ,MAAM,EACN,KAAK,EACL,IAAI,EACJ,UAAU,EACV,WAAW,EACX,eAAe,EACf,OAAO,GACR,MAAM,oBAAoB,CAAA;AAE3B,OAAO,EAAE,aAAa,EAAE,SAAS,EAAE,MAAM,OAAO,CAAA;AAgGhD,uEAAuE;AACvE,0EAA0E;AAC1E,oEAAoE;AACpE,8DAA8D;AAC9D,MAAM,mBAAmB,GAAG,aAAa,CAAC,MAAM,CAAC,CAAA;AAEjD,MAAM,CAAC,MAAM,kBAAkB,GAAG,CAAC,EACjC,SAAS,EACT,mBAAmB,GAAG,IAAI,EAC1B,yBAAyB,GAAG,IAAI,EAChC,qBAAqB,GAAG,KAAK,EAC7B,SAAS,EACT,WAAW,EACX,UAAU,EACV,WAAW,GAAG,CAAC,EACf,SAAS,EACT,SAAS,GAAG,cAAc,EAC1B,IAAI,EACJ,OAAO,GACmB,EAAE,EAAE;IAC9B,MAAM,UAAU,GAAG;QACjB,MAAM,CAAC,WAAW,CAAC;QACnB,IAAI,EAAE;QACN,KAAK,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC;KACtB,CAAA;IAED,IACE,qBAAqB;QACrB,WAAW,KAAK,SAAS;QACzB,UAAU,KAAK,SAAS,EACxB,CAAC;QACD,UAAU,CAAC,IAAI,CACb,IAAI,CAAC;YACH,OAAO,EAAE,CAAC;YACV,KAAK,CAAC,EACJ,eAAe,EACf,cAAc,EACd,QAAQ,EACR,KAAK,GACN;gBACC,MAAM,SAAS,GACb,WAAW,KAAK,SAAS;oBACvB,CAAC,CAAC,eAAe;oBACjB,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,WAAW,EAAE,eAAe,CAAC,CAAA;gBAE5C,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,SAAS,GAAG,GAAG,SAAS,IAAI,CAAA;gBAEpD,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;oBAC7B,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,CACvB,UAAU,EACV,cAAc,CACf,CAAA;oBAED,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,GAAG,GAAG,QAAQ,IAAI,CAAA;gBACpD,CAAC;gBAED,IAAI,qBAAqB,EAAE,CAAC;oBAC1B,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,KAAK,GAAG,GAAG,KAAK,CAAC,SAAS,CAAC,KAAK,IAAI,CAAA;gBAC9D,CAAC;YACH,CAAC;SACF,CAAC,CACH,CAAA;IACH,CAAC;IAED,MAAM,UAAU,GAAG,SAAS,KAAK,SAAS,CAAA;IAE1C,MAAM,EAAE,OAAO,EAAE,cAAc,EAAE,IAAI,EAAE,GAAG,WAAW,CAAC;QACpD,gEAAgE;QAChE,0CAA0C;QAC1C,IAAI,EAAE,SAAS;QACf,YAAY,EAAE,CAAC,aAAa,EAAE,EAAE;YAC9B,IAAI,CAAC,aAAa,EAAE,CAAC;gBACnB,SAAS,EAAE,CAAA;YACb,CAAC;QACH,CAAC;QACD,SAAS;QACT,UAAU;QACV,QAAQ,EAAE,OAAO;QACjB,oBAAoB,EAAE,UAAU;KACjC,CAAC,CAAA;IAEF,mEAAmE;IACnE,kEAAkE;IAClE,mEAAmE;IACnE,6DAA6D;IAC7D,oCAAoC;IACpC,MAAM,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,GAC3C,eAAe,CAAC;QACd,UAAU,CAAC,OAAO,EAAE;YAClB,SAAS,EAAE,mBAAmB;YAC9B,YAAY,EAAE,yBAAyB;SACxC,CAAC;QACF,OAAO,CAAC,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC;KAC3B,CAAC,CAAA;IAEJ,mEAAmE;IACnE,oEAAoE;IACpE,mEAAmE;IACnE,+DAA+D;IAC/D,yDAAyD;IACzD,uEAAuE;IACvE,qEAAqE;IACrE,kDAAkD;IAClD,MAAM,EAAE,IAAI,EAAE,qBAAqB,EAAE,GAAG,cAAc,EAAE,GACtD,iBAAiB,EAAE,CAAA;IAErB,qEAAqE;IACrE,qEAAqE;IACrE,kEAAkE;IAClE,sEAAsE;IACtE,oEAAoE;IACpE,iCAAiC;IACjC,MAAM,kBAAkB,GAAG,WAAW,EAAE,CAAA;IAExC,MAAM,SAAS,GACZ,cAAc,CAAC,EAAyB;QACzC,kBAAkB,CAAA;IAEpB,wEAAwE;IACxE,0EAA0E;IAC1E,yEAAyE;IACzE,yDAAyD;IACzD,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAM;QACR,CAAC;QAED,IAAI,CAAC,YAAY,CAAC,SAAS,CAAC,OAAO,CAAC,CAAA;IACtC,CAAC,EAAE,CAAC,UAAU,EAAE,SAAS,EAAE,IAAI,CAAC,CAAC,CAAA;IAEjC,MAAM,WAAW,GAAG,cAAc,CAChC,OAAO,IAAI,mBAAmB,EAC9B;QACE,GAAG,cAAc;QACjB,EAAE,EAAE,SAAS;QACb,4DAA4D;QAC5D,gEAAgE;QAChE,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,IAAI,CAAC,YAAY,EAAE,CAAC;KAClD,CACF,CAAA;IAED,MAAM,aAAa,GAAG,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,WAAW,CAAA;IAErD,OAAO;QACL,aAAa;QACb,OAAO;QACP,cAAc;QACd,gBAAgB;QAChB,WAAW,EAAE,IAAI,CAAC,WAAW;QAC7B,SAAS;KACV,CAAA;AACH,CAAC,CAAA"}
+{"version":3,"file":"useAnchoredOverlay.js","sourceRoot":"","sources":["../../src/Overlay/useAnchoredOverlay.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,WAAW,GACZ,MAAM,oBAAoB,CAAA;AAE3B,OAAO,EACL,UAAU,EACV,IAAI,EACJ,MAAM,EACN,KAAK,EACL,IAAI,EACJ,UAAU,EACV,WAAW,EACX,eAAe,EACf,OAAO,GACR,MAAM,oBAAoB,CAAA;AAE3B,OAAO,EAAE,aAAa,EAAE,SAAS,EAAE,MAAM,OAAO,CAAA;AAgGhD,uEAAuE;AACvE,0EAA0E;AAC1E,oEAAoE;AACpE,8DAA8D;AAC9D,MAAM,mBAAmB,GAAG,aAAa,CAAC,MAAM,CAAC,CAAA;AAEjD,MAAM,CAAC,MAAM,kBAAkB,GAAG,CAAC,EACjC,SAAS,EACT,mBAAmB,GAAG,IAAI,EAC1B,yBAAyB,GAAG,IAAI,EAChC,qBAAqB,GAAG,KAAK,EAC7B,SAAS,EACT,WAAW,EACX,UAAU,EACV,WAAW,GAAG,CAAC,EACf,SAAS,EACT,SAAS,GAAG,cAAc,EAC1B,IAAI,EACJ,OAAO,GACmB,EAAE,EAAE;IAC9B,MAAM,UAAU,GAAG;QACjB,MAAM,CAAC,WAAW,CAAC;QACnB,IAAI,EAAE;QACN,KAAK,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC;KACtB,CAAA;IAED,IACE,qBAAqB;QACrB,WAAW,KAAK,SAAS;QACzB,UAAU,KAAK,SAAS,EACxB,CAAC;QACD,UAAU,CAAC,IAAI,CACb,IAAI,CAAC;YACH,OAAO,EAAE,CAAC;YACV,KAAK,CAAC,EACJ,eAAe,EACf,cAAc,EACd,QAAQ,EACR,KAAK,GACN;gBACC,MAAM,SAAS,GACb,WAAW,KAAK,SAAS;oBACvB,CAAC,CAAC,eAAe;oBACjB,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,WAAW,EAAE,eAAe,CAAC,CAAA;gBAE5C,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,SAAS,GAAG,GAAG,SAAS,IAAI,CAAA;gBAEpD,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;oBAC7B,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,CACvB,UAAU,EACV,cAAc,CACf,CAAA;oBAED,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,GAAG,GAAG,QAAQ,IAAI,CAAA;gBACpD,CAAC;gBAED,IAAI,qBAAqB,EAAE,CAAC;oBAC1B,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC,KAAK,GAAG,GAAG,KAAK,CAAC,SAAS,CAAC,KAAK,IAAI,CAAA;gBAC9D,CAAC;YACH,CAAC;SACF,CAAC,CACH,CAAA;IACH,CAAC;IAED,MAAM,UAAU,GAAG,SAAS,KAAK,SAAS,CAAA;IAE1C,MAAM,EAAE,OAAO,EAAE,cAAc,EAAE,IAAI,EAAE,GAAG,WAAW,CAAC;QACpD,gEAAgE;QAChE,0CAA0C;QAC1C,IAAI,EAAE,SAAS;QACf,YAAY,EAAE,CAAC,aAAa,EAAE,EAAE;YAC9B,IAAI,CAAC,aAAa,EAAE,CAAC;gBACnB,SAAS,EAAE,CAAA;YACb,CAAC;QACH,CAAC;QACD,SAAS;QACT,UAAU;QACV,QAAQ,EAAE,OAAO;QACjB,oBAAoB,EAAE,UAAU;KACjC,CAAC,CAAA;IAEF,mEAAmE;IACnE,kEAAkE;IAClE,mEAAmE;IACnE,6DAA6D;IAC7D,oCAAoC;IACpC,MAAM,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,GAC3C,eAAe,CAAC;QACd,UAAU,CAAC,OAAO,EAAE;YAClB,SAAS,EAAE,mBAAmB;YAC9B,YAAY,EAAE,yBAAyB;SACxC,CAAC;QACF,OAAO,CAAC,OAAO,EAAE,EAAE,IAAI,EAAE,CAAC;KAC3B,CAAC,CAAA;IAEJ,mEAAmE;IACnE,oEAAoE;IACpE,mEAAmE;IACnE,+DAA+D;IAC/D,yDAAyD;IACzD,uEAAuE;IACvE,qEAAqE;IACrE,kDAAkD;IAClD,MAAM,EAAE,IAAI,EAAE,qBAAqB,EAAE,GAAG,cAAc,EAAE,GACtD,iBAAiB,EAAE,CAAA;IAErB,qEAAqE;IACrE,qEAAqE;IACrE,kEAAkE;IAClE,sEAAsE;IACtE,oEAAoE;IACpE,iCAAiC;IACjC,MAAM,kBAAkB,GAAG,WAAW,EAAE,CAAA;IAExC;;;;;;;;;;;;;;;;;;;;;;;;;;OA0BG;IACH,MAAM,YAAY,GAChB,OAAO,EAAE,KACV,EAAE,EAAE,CAAA;IAEL,MAAM,SAAS,GACb,YAAY;QACX,cAAc,CAAC,EAAyB;QACzC,kBAAkB,CAAA;IAEpB,wEAAwE;IACxE,0EAA0E;IAC1E,yEAAyE;IACzE,yDAAyD;IACzD,SAAS,CAAC,GAAG,EAAE;QACb,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,OAAM;QACR,CAAC;QAED,IAAI,CAAC,YAAY,CAAC,SAAS,CAAC,OAAO,CAAC,CAAA;IACtC,CAAC,EAAE,CAAC,UAAU,EAAE,SAAS,EAAE,IAAI,CAAC,CAAC,CAAA;IAEjC,MAAM,WAAW,GAAG,cAAc,CAChC,OAAO,IAAI,mBAAmB,EAC9B;QACE,GAAG,cAAc;QACjB,EAAE,EAAE,SAAS;QACb,4DAA4D;QAC5D,gEAAgE;QAChE,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,IAAI,CAAC,YAAY,EAAE,CAAC;KAClD,CACF,CAAA;IAED,MAAM,aAAa,GAAG,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,WAAW,CAAA;IAErD,OAAO;QACL,aAAa;QACb,OAAO;QACP,cAAc;QACd,gBAAgB;QAChB,WAAW,EAAE,IAAI,CAAC,WAAW;QAC7B,SAAS;KACV,CAAA;AACH,CAAC,CAAA"}

package/dist/Picker/Picker.d.ts

@@ -70,14 +70,20 @@ export type PickerProps = Omit<ComponentPropsWithRef<"button">, "disabled" | "on
  *
  * Query it as `getByRole("button", { name: /^Language: / })`.
  *
- * ### `id` does not survive; `data-testid` does
+ * ### `id` survives — it did not always
  *
- * `useAnchoredOverlay` overwrites the trigger's `id` with a generated
- * one, so the portalled listbox can point `aria-labelledby` across at
- * it. An `id` passed here is therefore silently replaced — queuepilot
- * found this the hard way and switched its e2e handles to
- * `data-testid`, which nothing injects. Everything else on the button
- * (`data-*`, `onClick`, `form`, …) passes straight through.
+ * `useAnchoredOverlay` used to overwrite the trigger's `id` with a
+ * generated one so the portalled listbox could point `aria-labelledby`
+ * across at it. That silently broke `Field`, which renders
+ * `<label htmlFor>` against an id it clones onto its child: the label
+ * pointed at nothing for every overlay control in a `Field`. The hook
+ * now prefers the trigger's own id and mints one only when absent, so
+ * both the label and the panel name the same element.
+ *
+ * `data-testid` remains the sturdier e2e handle — queuepilot and
+ * mux-magic both moved to it while `id` was still being replaced — but
+ * `id` is no longer a trap. Everything else on the button (`data-*`,
+ * `onClick`, `form`, …) passes straight through.
  */
 export declare const Picker: ({ appearance, className, iconEnd, intent, isDisabled, label, onChange, options, placement, placeholder, size, value, ...buttonProps }: PickerProps) => ReactNode;
 //# sourceMappingURL=Picker.d.ts.map

package/dist/Picker/Picker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Picker.d.ts","sourceRoot":"","sources":["../../src/Picker/Picker.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,WAAW,EACX,UAAU,EACX,MAAM,qBAAqB,CAAA;AAC5B,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAA;AACnD,OAAO,KAAK,EACV,qBAAqB,EACrB,SAAS,EACV,MAAM,OAAO,CAAA;AAGd,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AAC1D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AAGzD,MAAM,MAAM,YAAY,GAAG,WAAW,CAAA;AAwBtC,MAAM,MAAM,WAAW,GAAG,IAAI,CAC5B,qBAAqB,CAAC,QAAQ,CAAC,EAC/B,UAAU,GAAG,UAAU,GAAG,OAAO,CAClC,GAAG;IACF,UAAU,CAAC,EAAE,gBAAgB,CAAA;IAC7B,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,UAAU,CAAA;IACnB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB;;;OAGG;IACH,KAAK,EAAE,MAAM,CAAA;IACb;;;;;;OAMG;IACH,OAAO,CAAC,EAAE,SAAS,CAAA;IACnB,QAAQ,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAA;IACjC,OAAO,EAAE,SAAS,YAAY,EAAE,CAAA;IAChC,SAAS,CAAC,EAAE,SAAS,CAAA;IACrB,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,IAAI,CAAC,EAAE,WAAW,CAAA;IAClB,KAAK,CAAC,EAAE,MAAM,CAAA;CACf,CAAA;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgDG;AACH,eAAO,MAAM,MAAM,GAAI,uIAcpB,WAAW,KAAG,SAoDhB,CAAA"}
+{"version":3,"file":"Picker.d.ts","sourceRoot":"","sources":["../../src/Picker/Picker.tsx"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,WAAW,EACX,UAAU,EACX,MAAM,qBAAqB,CAAA;AAC5B,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAA;AACnD,OAAO,KAAK,EACV,qBAAqB,EACrB,SAAS,EACV,MAAM,OAAO,CAAA;AAGd,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AAC1D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAA;AAGzD,MAAM,MAAM,YAAY,GAAG,WAAW,CAAA;AAwBtC,MAAM,MAAM,WAAW,GAAG,IAAI,CAC5B,qBAAqB,CAAC,QAAQ,CAAC,EAC/B,UAAU,GAAG,UAAU,GAAG,OAAO,CAClC,GAAG;IACF,UAAU,CAAC,EAAE,gBAAgB,CAAA;IAC7B,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,MAAM,CAAC,EAAE,UAAU,CAAA;IACnB,UAAU,CAAC,EAAE,OAAO,CAAA;IACpB;;;OAGG;IACH,KAAK,EAAE,MAAM,CAAA;IACb;;;;;;OAMG;IACH,OAAO,CAAC,EAAE,SAAS,CAAA;IACnB,QAAQ,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAA;IACjC,OAAO,EAAE,SAAS,YAAY,EAAE,CAAA;IAChC,SAAS,CAAC,EAAE,SAAS,CAAA;IACrB,0DAA0D;IAC1D,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,IAAI,CAAC,EAAE,WAAW,CAAA;IAClB,KAAK,CAAC,EAAE,MAAM,CAAA;CACf,CAAA;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAsDG;AACH,eAAO,MAAM,MAAM,GAAI,uIAcpB,WAAW,KAAG,SAoDhB,CAAA"}

package/dist/Picker/Picker.js

@@ -48,14 +48,20 @@ const CHEVRON_DOWN = (_jsx("svg", { "aria-hidden": "true", className: "size-4",
  *
  * Query it as `getByRole("button", { name: /^Language: / })`.
  *
- * ### `id` does not survive; `data-testid` does
+ * ### `id` survives — it did not always
  *
- * `useAnchoredOverlay` overwrites the trigger's `id` with a generated
- * one, so the portalled listbox can point `aria-labelledby` across at
- * it. An `id` passed here is therefore silently replaced — queuepilot
- * found this the hard way and switched its e2e handles to
- * `data-testid`, which nothing injects. Everything else on the button
- * (`data-*`, `onClick`, `form`, …) passes straight through.
+ * `useAnchoredOverlay` used to overwrite the trigger's `id` with a
+ * generated one so the portalled listbox could point `aria-labelledby`
+ * across at it. That silently broke `Field`, which renders
+ * `<label htmlFor>` against an id it clones onto its child: the label
+ * pointed at nothing for every overlay control in a `Field`. The hook
+ * now prefers the trigger's own id and mints one only when absent, so
+ * both the label and the panel name the same element.
+ *
+ * `data-testid` remains the sturdier e2e handle — queuepilot and
+ * mux-magic both moved to it while `id` was still being replaced — but
+ * `id` is no longer a trap. Everything else on the button (`data-*`,
+ * `onClick`, `form`, …) passes straight through.
  */
 export const Picker = ({ appearance = "outline", className, iconEnd = CHEVRON_DOWN, intent = "neutral", isDisabled = false, label, onChange, options, placement, placeholder, size, value, ...buttonProps }) => {
     const { hide, isVisible, toggle } = useVisibility();

package/dist/Picker/Picker.js.map

@@ -1 +1 @@
-{"version":3,"file":"Picker.js","sourceRoot":"","sources":["../../src/Picker/Picker.tsx"],"names":[],"mappings":";AAAA,OAAO,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAA;AAWlD,OAAO,EAAE,MAAM,EAAE,MAAM,qBAAsB,CAAA;AAG7C,OAAO,EAAE,OAAO,EAAE,MAAM,uBAAwB,CAAA;AAIhD;;;;GAIG;AACH,MAAM,YAAY,GAAG,CACnB,6BACc,MAAM,EAClB,SAAS,EAAC,QAAQ,EAClB,IAAI,EAAC,MAAM,EACX,SAAS,EAAE,KAAK,EAChB,MAAM,EAAC,cAAc,EACrB,aAAa,EAAC,OAAO,EACrB,cAAc,EAAC,OAAO,EACtB,WAAW,EAAE,IAAI,EACjB,OAAO,EAAC,WAAW,EACnB,KAAK,EAAC,4BAA4B,YAElC,eAAM,CAAC,EAAC,cAAc,GAAG,GACrB,CACP,CAAA;AAgCD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgDG;AACH,MAAM,CAAC,MAAM,MAAM,GAAG,CAAC,EACrB,UAAU,GAAG,SAAS,EACtB,SAAS,EACT,OAAO,GAAG,YAAY,EACtB,MAAM,GAAG,SAAS,EAClB,UAAU,GAAG,KAAK,EAClB,KAAK,EACL,QAAQ,EACR,OAAO,EACP,SAAS,EACT,WAAW,EACX,IAAI,EACJ,KAAK,EACL,GAAG,WAAW,EACF,EAAa,EAAE;IAC3B,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,GAAG,aAAa,EAAE,CAAA;IAEnD,MAAM,aAAa,GAAG,OAAO,CAAC,IAAI,CAChC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,KAAK,KAAK,KAAK,CACnC,CAAA;IAED,MAAM,WAAW,GACf,aAAa,KAAK,SAAS;QACzB,CAAC,CAAC,WAAW;QACb,CAAC,CAAC,CAAC,aAAa,CAAC,SAAS;YACxB,CAAC,OAAO,aAAa,CAAC,KAAK,KAAK,QAAQ;gBACtC,CAAC,CAAC,aAAa,CAAC,KAAK;gBACrB,CAAC,CAAC,SAAS,CAAC,CAAC,CAAA;IAErB,MAAM,WAAW,GAAG,WAAW,IAAI,WAAW,IAAI,EAAE,CAAA;IAEpD,OAAO,CACL,KAAC,OAAO;IACN,8DAA8D;IAC9D,kEAAkE;IAClE,2DAA2D;;QAF3D,8DAA8D;QAC9D,kEAAkE;QAClE,2DAA2D;QAC3D,SAAS,EAAE,SAAS,IAAI,CAAC,UAAU,EACnC,SAAS,EAAE,IAAI,EACf,QAAQ,EAAE,QAAQ,EAClB,OAAO,EAAE,OAAO,EAChB,SAAS,EAAE,SAAS,EACpB,aAAa,EAAE,KAAK,EACpB,OAAO,EACL,KAAC,MAAM,OACD,WAAW,EACf,UAAU,EAAE,UAAU,gBACV,GAAG,KAAK,KAAK,WAAW,EAAE,EACtC,SAAS,EAAE,SAAS,EACpB,OAAO,EAAE,OAAO,EAChB,MAAM,EAAE,MAAM,EACd,UAAU,EAAE,UAAU,EACtB,OAAO,EAAE,CAAC,UAAU,EAAE,EAAE;gBACtB,WAAW,CAAC,OAAO,EAAE,CAAC,UAAU,CAAC,CAAA;gBAEjC,MAAM,EAAE,CAAA;YACV,CAAC,EACD,IAAI,EAAE,IAAI,EACV,IAAI,EAAC,QAAQ,YAEZ,aAAa,KAAK,SAAS;gBAC1B,CAAC,CAAC,WAAW;gBACb,CAAC,CAAC,aAAa,CAAC,KAAK,GAChB,GAEX,CACH,CAAA;AACH,CAAC,CAAA"}
+{"version":3,"file":"Picker.js","sourceRoot":"","sources":["../../src/Picker/Picker.tsx"],"names":[],"mappings":";AAAA,OAAO,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAA;AAWlD,OAAO,EAAE,MAAM,EAAE,MAAM,qBAAsB,CAAA;AAG7C,OAAO,EAAE,OAAO,EAAE,MAAM,uBAAwB,CAAA;AAIhD;;;;GAIG;AACH,MAAM,YAAY,GAAG,CACnB,6BACc,MAAM,EAClB,SAAS,EAAC,QAAQ,EAClB,IAAI,EAAC,MAAM,EACX,SAAS,EAAE,KAAK,EAChB,MAAM,EAAC,cAAc,EACrB,aAAa,EAAC,OAAO,EACrB,cAAc,EAAC,OAAO,EACtB,WAAW,EAAE,IAAI,EACjB,OAAO,EAAC,WAAW,EACnB,KAAK,EAAC,4BAA4B,YAElC,eAAM,CAAC,EAAC,cAAc,GAAG,GACrB,CACP,CAAA;AAgCD;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAsDG;AACH,MAAM,CAAC,MAAM,MAAM,GAAG,CAAC,EACrB,UAAU,GAAG,SAAS,EACtB,SAAS,EACT,OAAO,GAAG,YAAY,EACtB,MAAM,GAAG,SAAS,EAClB,UAAU,GAAG,KAAK,EAClB,KAAK,EACL,QAAQ,EACR,OAAO,EACP,SAAS,EACT,WAAW,EACX,IAAI,EACJ,KAAK,EACL,GAAG,WAAW,EACF,EAAa,EAAE;IAC3B,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,EAAE,GAAG,aAAa,EAAE,CAAA;IAEnD,MAAM,aAAa,GAAG,OAAO,CAAC,IAAI,CAChC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,CAAC,KAAK,KAAK,KAAK,CACnC,CAAA;IAED,MAAM,WAAW,GACf,aAAa,KAAK,SAAS;QACzB,CAAC,CAAC,WAAW;QACb,CAAC,CAAC,CAAC,aAAa,CAAC,SAAS;YACxB,CAAC,OAAO,aAAa,CAAC,KAAK,KAAK,QAAQ;gBACtC,CAAC,CAAC,aAAa,CAAC,KAAK;gBACrB,CAAC,CAAC,SAAS,CAAC,CAAC,CAAA;IAErB,MAAM,WAAW,GAAG,WAAW,IAAI,WAAW,IAAI,EAAE,CAAA;IAEpD,OAAO,CACL,KAAC,OAAO;IACN,8DAA8D;IAC9D,kEAAkE;IAClE,2DAA2D;;QAF3D,8DAA8D;QAC9D,kEAAkE;QAClE,2DAA2D;QAC3D,SAAS,EAAE,SAAS,IAAI,CAAC,UAAU,EACnC,SAAS,EAAE,IAAI,EACf,QAAQ,EAAE,QAAQ,EAClB,OAAO,EAAE,OAAO,EAChB,SAAS,EAAE,SAAS,EACpB,aAAa,EAAE,KAAK,EACpB,OAAO,EACL,KAAC,MAAM,OACD,WAAW,EACf,UAAU,EAAE,UAAU,gBACV,GAAG,KAAK,KAAK,WAAW,EAAE,EACtC,SAAS,EAAE,SAAS,EACpB,OAAO,EAAE,OAAO,EAChB,MAAM,EAAE,MAAM,EACd,UAAU,EAAE,UAAU,EACtB,OAAO,EAAE,CAAC,UAAU,EAAE,EAAE;gBACtB,WAAW,CAAC,OAAO,EAAE,CAAC,UAAU,CAAC,CAAA;gBAEjC,MAAM,EAAE,CAAA;YACV,CAAC,EACD,IAAI,EAAE,IAAI,EACV,IAAI,EAAC,QAAQ,YAEZ,aAAa,KAAK,SAAS;gBAC1B,CAAC,CAAC,WAAW;gBACb,CAAC,CAAC,aAAa,CAAC,KAAK,GAChB,GAEX,CACH,CAAA;AACH,CAAC,CAAA"}

package/dist/toClassName.d.ts

@@ -1,12 +1,52 @@
 /**
- * Filter-and-join. Deliberately not `clsx`.
- *
- * Every call site in this package passes a flat list of strings and
- * conditionals, which is four lines of work — and a published
- * component library's dependency list is a thing consumers inherit,
- * so it earns its entries. `clsx` also accepts objects and nested
- * arrays, both of which make a className harder for the Tailwind
- * scanner and for `tailwindCandidates.test.ts` to read statically.
+ * Join a class list, and let a later class beat an earlier one that
+ * targets the same CSS property.
+ *
+ * ### Why this is not a plain join any more
+ *
+ * It was, and that was a bug. `getControlClassName` assembles
+ * `toClassName(CONTROL_BASE_CLASS, …, className)`, and
+ * `CONTROL_BASE_CLASS` opens with `inline-flex`. A caller writing
+ *
+ * ```tsx
+ * <Button className="hidden lg:inline-flex" />
+ * ```
+ *
+ * emitted `class="inline-flex … hidden lg:inline-flex"`. Both are
+ * display utilities at equal specificity, so the winner is decided by
+ * **source order in the generated stylesheet**, not by order in the
+ * class attribute — and the caller has no way to win. `className` is a
+ * documented escape hatch, and it did not work for the most common
+ * thing an escape hatch is used for.
+ *
+ * It failed *silently and invisibly*. mail-sifter's header used exactly
+ * that class to hide a duplicate button below `lg`; it never hid, and
+ * nobody noticed for weeks because at 390px the header happened to fit
+ * anyway. It only surfaced when an unrelated type-ramp change pushed the
+ * width to 427px and scrolled the page sideways. Two more repos hit the
+ * same shape independently.
+ *
+ * ### Why `tailwind-merge` and not `clsx`
+ *
+ * The old docblock argued against taking a dependency, on the grounds
+ * that a published component library's dependency list is inherited by
+ * consumers and that `clsx`'s object and nested-array forms make a
+ * className harder for the Tailwind scanner and for
+ * `tailwindCandidates.test.ts` to read statically.
+ *
+ * The first half still stands and this entry earns its place; the
+ * second half does not transfer. `tailwind-merge` does not change the
+ * *input* shape at all — this still takes a flat list of strings and
+ * conditionals, and every call site is still statically scannable. It
+ * changes only which of two conflicting classes survives.
+ *
+ * ### It is a no-op on everything this package emits
+ *
+ * Merging could in principle drop one of our own classes. It does not:
+ * the test file checks all 358 class-string literals in `src` and all
+ * 288 strings `getControlClassName` composes, and the merge changes
+ * none of them. So this only ever acts on a genuine caller conflict —
+ * which is the entire point.
  */
 export declare const toClassName: (...parts: (false | string | null | undefined)[]) => string;
 //# sourceMappingURL=toClassName.d.ts.map

package/dist/toClassName.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"toClassName.d.ts","sourceRoot":"","sources":["../src/toClassName.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AACH,eAAO,MAAM,WAAW,GACtB,GAAG,OAAO,CAAC,KAAK,GAAG,MAAM,GAAG,IAAI,GAAG,SAAS,CAAC,EAAE,WACb,CAAA"}
+{"version":3,"file":"toClassName.d.ts","sourceRoot":"","sources":["../src/toClassName.ts"],"names":[],"mappings":"AAqBA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiDG;AACH,eAAO,MAAM,WAAW,GACtB,GAAG,OAAO,CAAC,KAAK,GAAG,MAAM,GAAG,IAAI,GAAG,SAAS,CAAC,EAAE,WACI,CAAA"}

package/dist/toClassName.js

@@ -1,12 +1,71 @@
+import { extendTailwindMerge } from "tailwind-merge";
 /**
- * Filter-and-join. Deliberately not `clsx`.
- *
- * Every call site in this package passes a flat list of strings and
- * conditionals, which is four lines of work — and a published
- * component library's dependency list is a thing consumers inherit,
- * so it earns its entries. `clsx` also accepts objects and nested
- * arrays, both of which make a className harder for the Tailwind
- * scanner and for `tailwindCandidates.test.ts` to read statically.
+ * `ease-standard` is ours, from the motion tokens — Tailwind ships
+ * `ease-linear`/`ease-in`/`ease-out`/`ease-in-out` and knows nothing
+ * about it. Without this it lands in no class group, so a caller
+ * passing `ease-linear` would get BOTH and the winner would be decided
+ * by stylesheet source order — exactly the bug this file now exists to
+ * prevent, one utility further along.
+ *
+ * Nothing else needs registering. Verified empirically rather than
+ * assumed: `toClassName.test.ts` asserts the merge is a no-op across
+ * every class string in this package and every string
+ * `getControlClassName` composes.
+ */
+const mergeClassNames = extendTailwindMerge({
+    extend: {
+        classGroups: { ease: [{ ease: ["standard"] }] },
+    },
+});
+/**
+ * Join a class list, and let a later class beat an earlier one that
+ * targets the same CSS property.
+ *
+ * ### Why this is not a plain join any more
+ *
+ * It was, and that was a bug. `getControlClassName` assembles
+ * `toClassName(CONTROL_BASE_CLASS, …, className)`, and
+ * `CONTROL_BASE_CLASS` opens with `inline-flex`. A caller writing
+ *
+ * ```tsx
+ * <Button className="hidden lg:inline-flex" />
+ * ```
+ *
+ * emitted `class="inline-flex … hidden lg:inline-flex"`. Both are
+ * display utilities at equal specificity, so the winner is decided by
+ * **source order in the generated stylesheet**, not by order in the
+ * class attribute — and the caller has no way to win. `className` is a
+ * documented escape hatch, and it did not work for the most common
+ * thing an escape hatch is used for.
+ *
+ * It failed *silently and invisibly*. mail-sifter's header used exactly
+ * that class to hide a duplicate button below `lg`; it never hid, and
+ * nobody noticed for weeks because at 390px the header happened to fit
+ * anyway. It only surfaced when an unrelated type-ramp change pushed the
+ * width to 427px and scrolled the page sideways. Two more repos hit the
+ * same shape independently.
+ *
+ * ### Why `tailwind-merge` and not `clsx`
+ *
+ * The old docblock argued against taking a dependency, on the grounds
+ * that a published component library's dependency list is inherited by
+ * consumers and that `clsx`'s object and nested-array forms make a
+ * className harder for the Tailwind scanner and for
+ * `tailwindCandidates.test.ts` to read statically.
+ *
+ * The first half still stands and this entry earns its place; the
+ * second half does not transfer. `tailwind-merge` does not change the
+ * *input* shape at all — this still takes a flat list of strings and
+ * conditionals, and every call site is still statically scannable. It
+ * changes only which of two conflicting classes survives.
+ *
+ * ### It is a no-op on everything this package emits
+ *
+ * Merging could in principle drop one of our own classes. It does not:
+ * the test file checks all 358 class-string literals in `src` and all
+ * 288 strings `getControlClassName` composes, and the merge changes
+ * none of them. So this only ever acts on a genuine caller conflict —
+ * which is the entire point.
  */
-export const toClassName = (...parts) => parts.filter(Boolean).join(" ");
+export const toClassName = (...parts) => mergeClassNames(parts.filter(Boolean).join(" "));
 //# sourceMappingURL=toClassName.js.map

package/dist/toClassName.js.map

@@ -1 +1 @@
-{"version":3,"file":"toClassName.js","sourceRoot":"","sources":["../src/toClassName.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,WAAW,GAAG,CACzB,GAAG,KAA4C,EAC/C,EAAE,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA"}
+{"version":3,"file":"toClassName.js","sourceRoot":"","sources":["../src/toClassName.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAA;AAEpD;;;;;;;;;;;;GAYG;AACH,MAAM,eAAe,GAAG,mBAAmB,CAAC;IAC1C,MAAM,EAAE;QACN,WAAW,EAAE,EAAE,IAAI,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,UAAU,CAAC,EAAE,CAAC,EAAE;KAChD;CACF,CAAC,CAAA;AAEF;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiDG;AACH,MAAM,CAAC,MAAM,WAAW,GAAG,CACzB,GAAG,KAA4C,EAC/C,EAAE,CAAC,eAAe,CAAC,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@charcuterie/ui",
-  "version": "2.15.0",
+  "version": "2.16.0",
   "description": "The Charcuterie components. Tokens for look, @charcuterie/logic for state, ARIA that agents can actually drive.",
   "license": "MIT",
   "type": "module",
@@ -48,10 +48,11 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@charcuterie/logic": "1.5.0",
+    "@charcuterie/logic": "2.0.0",
     "@charcuterie/tokens": "1.5.0",
     "@floating-ui/react": "^0.27.20",
-    "@tanstack/react-virtual": "^3.13.12"
+    "@tanstack/react-virtual": "^3.13.12",
+    "tailwind-merge": "^3.6.0"
   },
   "peerDependencies": {
     "react": ">=19",

package/src/Field/Field.stories.tsx

@@ -4,6 +4,7 @@ import {
   StoryCell,
   StoryGrid,
 } from "../board.storyHelpers.tsx"
+import { Picker } from "../Picker/Picker.tsx"
 import { Select } from "../Select/Select.tsx"
 import { Tooltip } from "../Tooltip/Tooltip.tsx"
 import { Field } from "./Field.tsx"
@@ -55,6 +56,33 @@ export const AdoptsChildId: Story = {
   },
 }
 
+/**
+ * An overlay-triggered control — `Picker`, and by the same route
+ * `Listbox`, `Combobox` and `Menu` — is the case that used to break.
+ *
+ * `useAnchoredOverlay` minted its own trigger id and cloned it over
+ * whatever was there, so the `<label htmlFor>` above pointed at an
+ * element that did not exist. It failed silently: a dangling `htmlFor`
+ * throws nothing, renders nothing, and only shows up if you go looking
+ * for the id in the DOM. The hook now prefers the trigger's own id.
+ */
+export const AdoptsOverlayTriggerId: Story = {
+  args: {
+    children: (
+      <Picker
+        label="Language"
+        onChange={() => {}}
+        options={[
+          { label: "English", value: "en" },
+          { label: "Japanese", value: "ja" },
+        ]}
+        value="en"
+      />
+    ),
+    label: "Language",
+  },
+}
+
 /**
  * The control is a slot, not a hardcoded `<input>` — `Select` goes
  * in the same hole, and so does an app's own control. The `id`,

package/src/Field/Field.test.tsx

@@ -9,6 +9,7 @@ import * as stories from "./Field.stories.tsx"
 
 const {
   AdoptsChildId,
+  AdoptsOverlayTriggerId,
   AllStates,
   AllVariants,
   Default,
@@ -38,6 +39,43 @@ test("a control's own id is adopted, not overwritten", async () => {
   await expectNoAxeViolations(canvasElement)
 })
 
+test("an overlay trigger keeps the id the label points at", async () => {
+  const { canvas, canvasElement } = await mountStory(
+    AdoptsOverlayTriggerId,
+  )
+
+  // The regression this exists for: `useAnchoredOverlay` minted its own
+  // trigger id and cloned it over the Field's, so the `<label htmlFor>`
+  // named an element that was not in the document. Nothing threw and
+  // nothing looked wrong — the only way to see it was to go looking for
+  // the id.
+  const label = canvasElement.querySelector("label")
+
+  await expect(label).not.toBeNull()
+
+  const htmlFor = label?.getAttribute("for")
+
+  await expect(htmlFor).toBeTruthy()
+
+  // The assertion that fails on the old behaviour: the id the label
+  // names must actually exist, and must BE the trigger.
+  const target = canvasElement.ownerDocument.getElementById(
+    htmlFor as string,
+  )
+
+  await expect(target).not.toBeNull()
+  await expect(target?.tagName).toBe("BUTTON")
+
+  // And the round trip callers actually rely on: findable by the
+  // label's own text.
+  expectAgentDrivable(canvas, {
+    name: /^Language: /,
+    role: "button",
+  })
+
+  await expectNoAxeViolations(canvasElement)
+})
+
 test("the label names the control it points at", async () => {
   const { canvas, canvasElement } =
     await mountStory(Default)

package/src/Overlay/useAnchoredOverlay.ts

@@ -225,7 +225,39 @@ export const useAnchoredOverlay = ({
   // input field and must be named.
   const generatedTriggerId = useUniqueId()
 
+  /**
+   * The trigger's OWN id wins over both.
+   *
+   * This hook used to mint an id and clone it over whatever the
+   * trigger already had, on the reasoning that the panel needs a
+   * stable target for `aria-labelledby`. It does — but any id serves
+   * that purpose, including the caller's, and overwriting had a
+   * consequence nobody traced: `Field` clones a `controlId` onto its
+   * child and renders `<label htmlFor={controlId}>`, so for every
+   * `Picker`/`Listbox`/`Combobox`/`Menu` inside a `Field` the label
+   * pointed at an element that did not exist. Measured in a real
+   * browser: zero nodes carried the id the `<label>` named.
+   *
+   * That is the exact defect `Field`'s docstring calls "precisely the
+   * defect this component was built to make impossible" — arriving by
+   * a different route, and silent, because a dangling `htmlFor`
+   * throws nothing and shows nothing.
+   *
+   * Preferring the child's id fixes the label without weakening the
+   * panel: `aria-labelledby` now points at the caller's id instead of
+   * a generated one, which is the same element either way.
+   *
+   * Two rediscoveries of the overwrite are already in the fleet —
+   * queuepilot and mux-magic both moved their e2e handles to
+   * `data-testid` because `id` "did not survive". `data-testid` is
+   * still the more robust handle, but `id` is no longer a trap.
+   */
+  const triggerOwnId = (
+    trigger?.props as { id?: string } | undefined
+  )?.id
+
   const triggerId =
+    triggerOwnId ??
     (referenceProps.id as string | undefined) ??
     generatedTriggerId
 

package/src/Picker/Picker.tsx

@@ -108,14 +108,20 @@ export type PickerProps = Omit<
  *
  * Query it as `getByRole("button", { name: /^Language: / })`.
  *
- * ### `id` does not survive; `data-testid` does
+ * ### `id` survives — it did not always
  *
- * `useAnchoredOverlay` overwrites the trigger's `id` with a generated
- * one, so the portalled listbox can point `aria-labelledby` across at
- * it. An `id` passed here is therefore silently replaced — queuepilot
- * found this the hard way and switched its e2e handles to
- * `data-testid`, which nothing injects. Everything else on the button
- * (`data-*`, `onClick`, `form`, …) passes straight through.
+ * `useAnchoredOverlay` used to overwrite the trigger's `id` with a
+ * generated one so the portalled listbox could point `aria-labelledby`
+ * across at it. That silently broke `Field`, which renders
+ * `<label htmlFor>` against an id it clones onto its child: the label
+ * pointed at nothing for every overlay control in a `Field`. The hook
+ * now prefers the trigger's own id and mints one only when absent, so
+ * both the label and the panel name the same element.
+ *
+ * `data-testid` remains the sturdier e2e handle — queuepilot and
+ * mux-magic both moved to it while `id` was still being replaced — but
+ * `id` is no longer a trap. Everything else on the button (`data-*`,
+ * `onClick`, `form`, …) passes straight through.
  */
 export const Picker = ({
   appearance = "outline",

package/src/sourceRules.test.ts

@@ -463,17 +463,22 @@ const ENTRY_POINT_RUNTIMES: Record<
     "./core": [],
     "./jotai": ["jotai"],
     "./preact": ["preact", "preact/hooks"],
-    // The request/response data layer. It reaches react-query, the
-    // two openapi-* primitives it re-exports, and `react` (the
-    // provider's `useState`) — all optional peers, so an app that
-    // never imports `./query` resolves none of them. `react/jsx-runtime`
-    // is added only at emit, so it is not a source specifier here.
-    "./query": [
-      "@tanstack/react-query",
-      "openapi-fetch",
-      "openapi-react-query",
-      "react",
-    ],
+    // The typed OpenAPI seam, split out of `./query` so that only an
+    // app whose backend publishes a spec resolves these two. This
+    // table is the thing that makes that split legible: what the row
+    // costs is the whole argument for it being its own entry point.
+    // See the [split
+    // decision](../../../docs/decisions/2026-08-13-the-openapi-seam-is-its-own-subpath-not-part-of-query.md).
+    "./openapi": ["openapi-fetch", "openapi-react-query"],
+    // The request/response data layer: react-query and `react` (the
+    // provider's `useState`), both optional peers, so an app that
+    // never imports `./query` resolves neither. It deliberately does
+    // NOT reach the openapi-* packages — when it did (via the
+    // re-export this row used to list), the type-only edge above was
+    // enough to force every non-OpenAPI consumer to install both.
+    // `react/jsx-runtime` is added only at emit, so it is not a
+    // source specifier here.
+    "./query": ["@tanstack/react-query", "react"],
     "./signals": ["@preact/signals-core"],
   },
   tokens: {
@@ -494,6 +499,16 @@ const ENTRY_POINT_RUNTIMES: Record<
       // tree-shakeable, ~4 KB gz.
       "@tanstack/react-virtual",
       "react",
+      // `toClassName`'s conflict resolution — the third runtime
+      // dependency this package has ever taken. MIT, German origin
+      // (Dany Castillo), ~7 KB gz, no transitive deps.
+      //
+      // It earns the entry: without it `className` is not an escape
+      // hatch. A caller's `hidden` and the base `inline-flex` are
+      // display utilities at equal specificity, so the winner came
+      // from stylesheet source order and the caller could not win —
+      // silently, which is how it shipped broken in three repos.
+      "tailwind-merge",
     ],
     // The router seam's one shipped adapter, and the reason it is a
     // subpath rather than part of the barrel: `react-router` appears

package/src/toClassName.test.ts

@@ -0,0 +1,229 @@
+import { readdirSync, readFileSync } from "node:fs"
+import { join } from "node:path"
+import { describe, expect, test } from "vitest"
+
+import {
+  type BadgeSize,
+  getControlClassName,
+} from "./controlStyles.ts"
+import { INTENT_APPEARANCE_CLASS } from "./intentStyles.ts"
+import { toClassName } from "./toClassName.ts"
+
+describe("a caller's class beats the base class", () => {
+  // The bug this file exists for. `CONTROL_BASE_CLASS` opens with
+  // `inline-flex`; both it and `hidden` are display utilities at equal
+  // specificity, so before the merge the winner was decided by
+  // stylesheet source order and the caller could not win.
+  test("display: `hidden` removes `inline-flex`", () => {
+    expect(
+      toClassName("inline-flex items-center", "hidden"),
+    ).toBe("items-center hidden")
+  })
+
+  test("the responsive form that shipped broken still works", () => {
+    // mail-sifter's header, verbatim: hide below `lg`, show above.
+    // The unprefixed `hidden` must beat the base `inline-flex` while
+    // the `lg:` variant is left alone — they are different breakpoints
+    // and must not cancel each other.
+    const result = toClassName(
+      "inline-flex cursor-pointer items-center",
+      "hidden lg:inline-flex",
+    )
+
+    expect(result).toContain("hidden")
+    expect(result).toContain("lg:inline-flex")
+    expect(result.split(" ")).not.toContain("inline-flex")
+  })
+
+  test.each([
+    ["rounded-md", "rounded-full"],
+    ["border-border-default", "border-transparent"],
+    ["bg-surface-base", "bg-surface-raised"],
+    ["whitespace-nowrap", "whitespace-normal"],
+    ["font-medium", "font-bold"],
+    ["h-(--control-height-md)", "h-(--control-height-lg)"],
+    ["px-(--control-padding-inline-md)", "px-0"],
+    ["text-sm", "text-lg"],
+    // Ours, not Tailwind's — registered explicitly in the merge
+    // config, and it would silently fail to merge without that.
+    ["ease-standard", "ease-linear"],
+  ])("%s is overridden by %s", (base, override) => {
+    expect(toClassName(base, override)).toBe(override)
+  })
+})
+
+describe("it does not over-merge", () => {
+  // The dangerous direction, and the reason this was measured rather
+  // than assumed: `text-md` is OUR font-size (theme.css bridges
+  // `--text-*` onto `--font-size-*`), and `text-content-primary` is a
+  // colour. A merge that classified both as `text-*` would silently
+  // drop the colour from every control in the package.
+  test("a custom font size does not eat a custom text colour", () => {
+    const result = toClassName(
+      "text-md",
+      "text-content-primary",
+    )
+
+    expect(result).toContain("text-md")
+    expect(result).toContain("text-content-primary")
+  })
+
+  test("a custom easing does not eat a duration", () => {
+    const result = toClassName(
+      "ease-standard duration-(--duration-fast)",
+    )
+
+    expect(result).toContain("ease-standard")
+    expect(result).toContain("duration-(--duration-fast)")
+  })
+
+  test("falsy parts are still dropped", () => {
+    expect(
+      toClassName(
+        "a-class",
+        false,
+        null,
+        undefined,
+        "b-class",
+      ),
+    ).toBe("a-class b-class")
+  })
+})
+
+/**
+ * Adopting a merge could in principle drop one of the package's OWN
+ * classes — a conflict we shipped on purpose, or one it misreads.
+ * These two tests are the evidence that it does not, and they are the
+ * reason the change is safe to make globally rather than only where a
+ * consumer `className` is folded in.
+ *
+ * They also fail loudly if a future class introduces a real internal
+ * conflict, which is worth knowing about on its own.
+ */
+describe("merging is a no-op on what this package emits", () => {
+  const walk = (dir: string): string[] =>
+    readdirSync(dir, { withFileTypes: true }).flatMap(
+      (entry) => {
+        const path = join(dir, entry.name)
+
+        if (entry.isDirectory()) {
+          return entry.name === "node_modules"
+            ? []
+            : walk(path)
+        }
+
+        return /\.tsx?$/.test(path) &&
+          !/\.test\./.test(path)
+          ? [path]
+          : []
+      },
+    )
+
+  const looksLikeClassList = (value: string) =>
+    value.includes(" ") &&
+    value
+      .split(" ")
+      .every((token) =>
+        /^[a-z0-9:[\]()\-_/.&>*+#!,%'"$@=~^|?]+$/i.test(
+          token,
+        ),
+      ) &&
+    /(flex|grid|text-|bg-|border|rounded|p[xytblr]?-|m[xytblr]?-|h-|w-|size-|gap-|items-|justify-|absolute|relative|hidden|block|inline|font-|shadow|outline|transition|duration|ease-|select-|cursor-|overflow|z-|opacity|ring|space-|min-|max-|whitespace|truncate|leading|tracking)/.test(
+      value,
+    )
+
+  test("every class-string literal in src survives untouched", () => {
+    const damaged: {
+      after: string
+      before: string
+      file: string
+    }[] = []
+
+    let checked = 0
+
+    for (const file of walk(
+      join(import.meta.dirname ?? "src"),
+    )) {
+      const source = readFileSync(file, "utf8")
+
+      for (const match of source.matchAll(
+        /"([a-z0-9][^"\n]*?)"/g,
+      )) {
+        const value = match[1]
+
+        if (!looksLikeClassList(value)) {
+          continue
+        }
+
+        checked++
+
+        const merged = toClassName(value)
+
+        if (merged !== value) {
+          damaged.push({
+            after: merged,
+            before: value,
+            file,
+          })
+        }
+      }
+    }
+
+    // A guard on the guard: if the heuristic stops matching, this
+    // test would pass by checking nothing.
+    expect(checked).toBeGreaterThan(300)
+    expect(damaged).toEqual([])
+  })
+
+  test("every composed control class string survives untouched", () => {
+    const intents = Object.keys(
+      INTENT_APPEARANCE_CLASS,
+    ) as (keyof typeof INTENT_APPEARANCE_CLASS)[]
+
+    const appearances = Object.keys(
+      INTENT_APPEARANCE_CLASS[intents[0]],
+    ) as (keyof (typeof INTENT_APPEARANCE_CLASS)[(typeof intents)[0]])[]
+
+    const damaged: string[] = []
+
+    let composed = 0
+
+    for (const intent of intents) {
+      for (const appearance of appearances) {
+        for (const size of [
+          "sm",
+          "md",
+          "lg",
+        ] as BadgeSize[] & string[]) {
+          for (const sizing of [
+            "control",
+            "icon",
+          ] as const) {
+            for (const isFullWidth of [false, true]) {
+              const value = getControlClassName({
+                appearance,
+                className: undefined,
+                disabledClass: "disabled:opacity-50",
+                intent,
+                isFullWidth,
+                size,
+                sizing,
+              })
+
+              composed++
+
+              // `getControlClassName` already routes through
+              // `toClassName`, so re-merging must be idempotent.
+              if (toClassName(value) !== value) {
+                damaged.push(value)
+              }
+            }
+          }
+        }
+      }
+    }
+
+    expect(composed).toBeGreaterThan(200)
+    expect(damaged).toEqual([])
+  })
+})

package/src/toClassName.ts

@@ -1,13 +1,74 @@
+import { extendTailwindMerge } from "tailwind-merge"
+
 /**
- * Filter-and-join. Deliberately not `clsx`.
- *
- * Every call site in this package passes a flat list of strings and
- * conditionals, which is four lines of work — and a published
- * component library's dependency list is a thing consumers inherit,
- * so it earns its entries. `clsx` also accepts objects and nested
- * arrays, both of which make a className harder for the Tailwind
- * scanner and for `tailwindCandidates.test.ts` to read statically.
+ * `ease-standard` is ours, from the motion tokens — Tailwind ships
+ * `ease-linear`/`ease-in`/`ease-out`/`ease-in-out` and knows nothing
+ * about it. Without this it lands in no class group, so a caller
+ * passing `ease-linear` would get BOTH and the winner would be decided
+ * by stylesheet source order — exactly the bug this file now exists to
+ * prevent, one utility further along.
+ *
+ * Nothing else needs registering. Verified empirically rather than
+ * assumed: `toClassName.test.ts` asserts the merge is a no-op across
+ * every class string in this package and every string
+ * `getControlClassName` composes.
+ */
+const mergeClassNames = extendTailwindMerge({
+  extend: {
+    classGroups: { ease: [{ ease: ["standard"] }] },
+  },
+})
+
+/**
+ * Join a class list, and let a later class beat an earlier one that
+ * targets the same CSS property.
+ *
+ * ### Why this is not a plain join any more
+ *
+ * It was, and that was a bug. `getControlClassName` assembles
+ * `toClassName(CONTROL_BASE_CLASS, …, className)`, and
+ * `CONTROL_BASE_CLASS` opens with `inline-flex`. A caller writing
+ *
+ * ```tsx
+ * <Button className="hidden lg:inline-flex" />
+ * ```
+ *
+ * emitted `class="inline-flex … hidden lg:inline-flex"`. Both are
+ * display utilities at equal specificity, so the winner is decided by
+ * **source order in the generated stylesheet**, not by order in the
+ * class attribute — and the caller has no way to win. `className` is a
+ * documented escape hatch, and it did not work for the most common
+ * thing an escape hatch is used for.
+ *
+ * It failed *silently and invisibly*. mail-sifter's header used exactly
+ * that class to hide a duplicate button below `lg`; it never hid, and
+ * nobody noticed for weeks because at 390px the header happened to fit
+ * anyway. It only surfaced when an unrelated type-ramp change pushed the
+ * width to 427px and scrolled the page sideways. Two more repos hit the
+ * same shape independently.
+ *
+ * ### Why `tailwind-merge` and not `clsx`
+ *
+ * The old docblock argued against taking a dependency, on the grounds
+ * that a published component library's dependency list is inherited by
+ * consumers and that `clsx`'s object and nested-array forms make a
+ * className harder for the Tailwind scanner and for
+ * `tailwindCandidates.test.ts` to read statically.
+ *
+ * The first half still stands and this entry earns its place; the
+ * second half does not transfer. `tailwind-merge` does not change the
+ * *input* shape at all — this still takes a flat list of strings and
+ * conditionals, and every call site is still statically scannable. It
+ * changes only which of two conflicting classes survives.
+ *
+ * ### It is a no-op on everything this package emits
+ *
+ * Merging could in principle drop one of our own classes. It does not:
+ * the test file checks all 358 class-string literals in `src` and all
+ * 288 strings `getControlClassName` composes, and the merge changes
+ * none of them. So this only ever acts on a genuine caller conflict —
+ * which is the entire point.
  */
 export const toClassName = (
   ...parts: (false | string | null | undefined)[]
-) => parts.filter(Boolean).join(" ")
+) => mergeClassNames(parts.filter(Boolean).join(" "))