@ntbjs/react-components 3.0.0-rc.1 → 3.0.0-rc.2

package/types/components/inputs/MultiSelect/MultiSelect.d.ts

@@ -1,11 +1,5 @@
 import React from 'react';
 import * as S from './MultiSelect.styled';
-/**
- * MultiSelect's OWN props — the 19 members that were the entirety of this component's
- * declared surface until NMB-3998, unchanged. They stay authoritative: `MultiSelectProps`
- * below `Omit`s these keys out of the base before intersecting, so nothing here is
- * narrowed by a colliding member of react-select's props.
- */
 export type MultiSelectOwnProps = {
     /**
      *
@@ -116,57 +110,6 @@ export type MultiSelectOwnProps = {
      */
     warning?: boolean | string;
 };
-/**
- * THE PUBLISHED PROPS TYPE. It has a base, and that is playbook §C.2, not a convenience.
- *
- * This component destructures `...rest` (below) and spreads it into react-select —
- * `sharedSelectProps = { ...rest, … }`, handed to `<SelectComponent {...sharedSelectProps} />`.
- * A closed 19-member type therefore REJECTED props that this component forwards and that
- * work at runtime: measured against mb-main-client, `className`, `name`, `onBlur` and
- * `required` are all real react-select props being passed today and all four were a hard
- * TS2322 the moment the library started shipping `.d.ts`. That is the library lying about
- * its own contract, which is the exact class §C.2 exists to prevent — and it is NOT the
- * same finding as the other 22 rejected consumer attributes, which are props no component
- * here spreads anywhere and which stay rejected on purpose.
- *
- * THE BASE IS READ OFF WHAT `rest` IS ACTUALLY SPREAD ONTO, never assumed. That target is
- * one of the four `styled(<react-select …>)` wrappers, chosen at runtime by `loadOptions`
- * and `creatable`. `S.MultiSelect` — `styled(Select)` — is the deliberate choice among the
- * four: `CreatableSelect`/`AsyncPaginate` add props on top of the plain select's surface,
- * so plain `Select` is the set that is valid in ALL FOUR configurations. Basing this on the
- * async-creatable wrapper instead would admit `debounceTimeout` and friends in the default
- * configuration, where they do nothing — the same mistake in the other direction.
- *
- * `Omit<Base, keyof Own> & Own`, NOT `Base & Own`. An intersection intersects a colliding
- * MEMBER rather than replacing it; measured on `TextArea`, `rows?: number | string` met
- * `styled.textarea`'s `rows?: number` and `rows="4"` became an error — half a published
- * prop silently deleted by the pass meant to widen it. Disjoint pairs collapse to `never`.
- *
- * `ComponentPropsWithoutRef`, NOT `…WithRef`. `React.forwardRef` below supplies `ref`
- * itself; `WithRef` was measured to let `<MultiSelect ref={…} />` compile with the ref
- * silently dropped.
- *
- * BOTH ALIASES ARE `export`ed AND THE `const` BELOW IS ANNOTATED, for declaration emit
- * only — same mechanism and same audit as `CompactAutocompleteSelect` (577fc8d, §G.4.1).
- * tsc prints an exported alias from its source text and EXPANDS an unexported one, and the
- * expansion here reaches react-select's `PublicBaseSelectProps` /
- * `StateManagerAdditionalProps`, which `react-select`'s `exports` map does not serve:
- * TS2883, and with `noEmitOnError: false` that means the `.d.ts` is silently not written.
- * The annotation on the `const` is needed for the same reason — `forwardRef` infers
- * `ForwardRefExoticComponent<PropsWithoutRef<P> & RefAttributes<any>>` and tsc computes
- * `PropsWithoutRef<P>` eagerly, which re-expands what the exported alias just avoided.
- *
- * THE SWALLOW THIS PARAGRAPH USED TO RECORD AS OPEN IS FIXED. `sharedSelectProps` still
- * lists 14 keys after `...rest` and they still win — deliberately, and the reasoning is
- * written out at the literal itself — but `onFocus`, `onBlur` and `onChange` are no longer
- * among them: they now run the component's own logic and then call the caller's handler.
- * So `onBlur`, the one a consumer passes today, is both admitted by this type and actually
- * forwarded. Do not narrow this type to "reflect" the remaining 14: they are the runtime
- * form of props this component already declares (`selectedOptions`, `availableOptions`,
- * `disabled`/`readOnly`, `loadOptions`) plus its own rendering and CSS contract, and
- * `Omit`ting them would only stop react-select's own names from being accepted while
- * changing nothing about which value wins.
- */
 export type MultiSelectProps = Omit<React.ComponentPropsWithoutRef<typeof S.MultiSelect>, keyof MultiSelectOwnProps> & MultiSelectOwnProps;
 declare const MultiSelect: React.ForwardRefExoticComponent<MultiSelectProps & React.RefAttributes<any>>;
 export default MultiSelect;

package/types/components/inputs/MultiSelect/MultiSelect.styled.d.ts

@@ -3,66 +3,9 @@ import Select from 'react-select';
 import CreatableSelect from 'react-select/creatable';
 import { AsyncPaginate } from 'react-select-async-paginate';
 import type { PartialNtbTheme } from '../../../utils/defaultTheme';
-/**
- * The `theme` prop of the four react-select wrappers below, restated as OUR theme.
- *
- * WHY THIS EXISTS. `theme` is the one prop name that both styled-components and
- * react-select claim. react-select declares it `ThemeConfig = Theme | ((t: Theme) => Theme)`
- * and its `Theme` requires `borderRadius`, which `NtbTheme` does not have — so
- * `.attrs(applyDefaultTheme)`, whose return type is `{ theme: NtbTheme }`, was
- * unassignable to `Attrs<SelectProps>` and each wrapper reported:
- *
- *   TS2345 … Types of property 'theme' are incompatible.
- *     Type 'NtbTheme' is not assignable to type 'ThemeConfig'.
- *       Property 'borderRadius' is missing in type 'NtbTheme' but required in type 'Theme'.
- *
- * `.attrs<{ theme?: PartialNtbTheme }>` fixes it because `attrs`' generic is applied as
- * `Substitute<OuterProps, P> = FastOmit<OuterProps, keyof P> & P` — i.e. declaring
- * `theme` REPLACES react-select's declaration of it rather than intersecting with it.
- *
- * AND THAT IS THE TRUTH ABOUT THIS BOUNDARY, not a workaround. At runtime `.attrs()`
- * hands react-select the merged `NtbTheme`, and react-select's `getTheme()` does
- * `{ ...its defaultTheme, ...this.props.theme }`: our `colors` and `spacing` win — which
- * is why `ThemeColors`/`ThemeSpacing` in `utils/defaultTheme.ts` carry react-select's
- * member names and not design-token names — and `borderRadius` falls through to
- * react-select's own `4`. `NtbTheme` is deliberately a PARTIAL override of react-select's
- * theme, so `ThemeConfig` was never the right type for this prop here.
- *
- * `PartialNtbTheme` and not `NtbTheme` because this is the INPUT side: a caller may pass
- * any subset and `applyDefaultTheme` fills the rest in, exactly as `ThemedProps` states
- * for every other file. Do NOT "fix" this by adding `borderRadius` to `NtbTheme`: that
- * would hand react-select a border radius it does not get today and is a render change.
- */
 interface SelectThemeProp {
-    /** A partial override merged over `defaultTheme` by `applyDefaultTheme`. */
     theme?: PartialNtbTheme;
 }
-/**
- * The WRITTEN type of the four `styled(<react-select component>)` exports below, and the
- * reason they are annotated at all when nothing else in this repo is.
- *
- * `styled(Select).attrs<SelectThemeProp>(…)` INFERS a correct type. It is not a
- * WRITEABLE one: `tsc --declaration` has to print it into `MultiSelect.styled.d.ts`, the
- * printed form expands to react-select's `PublicBaseSelectProps` and
- * `StateManagerAdditionalProps`, and neither is re-exported from `react-select`'s public
- * entry — react-select has an `exports` map with no wildcard, so
- * `react-select/dist/declarations/src/Select` does not resolve for a consumer at all.
- * The result was 11 x TS2883 in this file, and the consequence was worse than the
- * message: `noEmitOnError` is false, so the build "succeeded" while this file's `.d.ts`
- * was SILENTLY NOT WRITTEN.
- *
- * A generic alias fixes it because tsc prints an alias APPLICATION by reference —
- * `SelectWrapper<typeof Select>` — instead of expanding it, and `typeof Select` is
- * `typeof import('react-select').default`, which is nameable. Nothing here is widened or
- * narrowed for the sake of the declaration: `Omit<ComponentProps<C>, keyof P> & P` is
- * exactly styled-components' own `Substitute<OuterProps, P>`, which is the rule
- * `SelectThemeProp` above relies on.
- *
- * What IS given up is the styled-components-specific surface of these four bindings —
- * `.withConfig()`, `.attrs()` and re-wrapping with `styled(S.MultiSelect)`. Audited:
- * nothing in this repo does any of the three to them. Adding one means annotating with
- * `IStyledComponent` instead, which puts the same non-portable expansion back.
- */
 type SelectWrapper<C extends React.ComponentType<any>> = React.ComponentType<Omit<React.ComponentProps<C>, keyof SelectThemeProp> & SelectThemeProp>;
 declare const AsyncCreatableSelectPaginate: import("react-select-async-paginate").WithAsyncPaginateType;
 export declare const MultiSelectWrapper: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>, React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
@@ -83,16 +26,8 @@ export declare const DropdownOptionDeleteIcon: import("styled-components/dist/ty
 export declare const ShowMoreWrapper: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>, React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
 export declare const ShowMoreOverlay: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>, React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
 export declare const ShowMoreText: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>, React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
-/**
- * `MultiSelect.tsx` renders `<S.ErrorMessage $error={!!error} $warning={!!warning}>`
- * (line 461), so both are `boolean` — the `!!` is in the JSX, not here. Optional per
- * §J.5; the template's own ternary chain already ends in `'inherit'` for the
- * neither-set case.
- */
 interface ErrorMessageStyleProps {
-    /** `!!error` — red. Wins over `$warning` when both are set. */
     $error?: boolean;
-    /** `!!warning` — orange. */
     $warning?: boolean;
 }
 export declare const ErrorMessage: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>, React.DetailedHTMLProps<React.HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, ErrorMessageStyleProps>, never>> & string;

package/types/components/inputs/Radio/Radio.styled.d.ts

@@ -1,10 +1,10 @@
 export declare const RadioLabel: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
 interface RadioStyleProps {
-    /** `checked` — `PropTypes.bool`, passed through as the checked-state affordance. */
+    /** `checked` — passed through as the checked-state affordance. */
     $isChecked?: boolean;
-    /** `disabled` — `PropTypes.bool`. Leaks to the DOM; see above. */
+    /** `disabled` — leaks to DOM. */
     disabled?: boolean;
-    /** `readOnly` — `PropTypes.bool`. Leaks to the DOM; see above. */
+    /** `readOnly` — leaks to DOM. */
     readOnly?: boolean;
 }
 export declare const Radio: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").LabelHTMLAttributes<HTMLLabelElement>, HTMLLabelElement>, import("react").DetailedHTMLProps<import("react").LabelHTMLAttributes<HTMLLabelElement>, HTMLLabelElement>>, RadioStyleProps>, never>> & string;

package/types/components/inputs/Switch/Switch.styled.d.ts

@@ -1,7 +1,7 @@
 interface SwitchStyleProps {
-    /** `disabled` — dims and blocks pointer events. Leaks to the DOM; see above. */
+    /** `disabled` — dims and blocks pointer events. Leaks to DOM. */
     disabled?: boolean;
-    /** `readOnly` — dims and blocks pointer events. Leaks to the DOM; see above. */
+    /** `readOnly` — dims and blocks pointer events. Leaks to DOM. */
     readOnly?: boolean;
 }
 export declare const Switch: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, SwitchStyleProps>, never>> & string;
@@ -10,11 +10,7 @@ export declare const SwitchIndicatorButton: import("styled-components/dist/types
 export declare const SwitchIndicatorLabelText: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, never>, never>> & string;
 export declare const SwitchLabel: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").LabelHTMLAttributes<HTMLLabelElement>, HTMLLabelElement>, import("react").DetailedHTMLProps<import("react").LabelHTMLAttributes<HTMLLabelElement>, HTMLLabelElement>>, never>, never>> & string;
 export declare const Description: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, {
-    /**
-     * `spaciousDescription` — `PropTypes.bool`. Lower-case `$spaciousdescription` by
-     * existing convention at both ends; renaming it would be a two-file change for no
-     * type benefit.
-     */
+    /** `spaciousDescription` — lower-case by convention; renaming would require two-file change. */
     $spaciousdescription?: boolean;
 }>, never>> & string;
 export {};

package/types/components/inputs/TextArea/TextArea.d.ts

@@ -73,10 +73,6 @@ type TextAreaOwnProps = {
     style?: any;
     /**
      * Add padding (small-medium-large)
-     *
-     * `''` is not from this component's propTypes; `Instructions` forwards its own
-     * `'' | 'small' | 'medium' | 'large'` straight in at `Instructions.tsx:354`. Kept in
-     * sync with `TextAreaPadding` in `TextArea.styled.ts`, which has the full rationale.
      */
     padding?: '' | 'small' | 'medium' | 'large';
     /**

package/types/components/inputs/TextArea/TextArea.styled.d.ts

@@ -4,10 +4,9 @@
  * carried as `$type` by `TextInputField`, `TextInputFieldIconAlert`, `TextInput` and
  * `Description`.
  *
- * Closed on purpose (§J.6): the templates below branch on this value 26 times, and with a
- * closed union a mistyped state name is TS2367 instead of a rule that silently never
- * fires — which is exactly the failure this file already contains three of. `''` is a
- * member because the component defaults `type = ''` (tsx line 40).
+ * Closed on purpose: the templates branch on this value 26 times, so a mistyped state name
+ * becomes a type error instead of a rule that silently never fires. `''` is a member because
+ * the component defaults `type = ''`.
  */
 type TextAreaType = '' | 'error' | 'error-border' | 'warning-border' | 'warning' | 'loading' | 'success' | 'instructions-warning';
 /**
@@ -33,9 +32,8 @@ export declare const InputIconContainer: import("styled-components/dist/types").
  * Props of `TextInputField`, the `<textarea>` itself. Six transient props plus the two
  * halves of a defect.
  *
- * `readOnly` and `disabled` are read by the template (7 times between them) and are
- * deliberately ABSENT from this interface: both are real `TextareaHTMLAttributes` members,
- * so those reads already typecheck and declaring them would add nothing. §J.9's first case.
+ * `readOnly` and `disabled` are deliberately absent: both are real `TextareaHTMLAttributes`
+ * members, so the template's reads already typecheck.
  */
 export interface TextInputFieldStyleProps {
     /**

package/types/components/inputs/TextInput/TextInput.styled.d.ts

@@ -4,7 +4,7 @@
  * `boolean` for both, and NOT the `bool | string` that `error`'s and `warning`'s own
  * propTypes allow: `TextInput.tsx` derives `hasError` / `hasWarning`
  * (`isBoolean(x) ? x : !isEmpty(x)`) and passes those, keeping the message text for
- * `Description`'s children. §J.6, rule 2 — the JSX site says which value arrives.
+ * `Description`'s children. The JSX site says which value arrives.
  */
 export interface ValidationStyleProps {
     /** `hasError` — red border / red message. */

package/types/components/layout/SectionSeparator/SectionSeparator.styled.d.ts

@@ -1,14 +1,7 @@
-/**
- * Props of the styled root, i.e. `SectionSeparator.tsx`'s `vertical` / `margin`
- * carried across the `shouldForwardProp` boundary.
- */
 export interface SectionSeparatorStyleProps {
-    /** `vertical` — swaps which axis gets the 1px hairline and which gets 100%. */
+    /** `vertical` — swaps which axis gets the 1px hairline. */
     $vertical?: boolean;
-    /**
-     * `margin` — px, interpolated as `${props.$margin}px`, so `number` and not
-     * `number | string`: a string here would already render `margin-top: 8pxpx`.
-     */
+    /** `margin` — px, interpolated so type is `number`. */
     $margin?: number;
 }
 export declare const SectionSeparator: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, SectionSeparatorStyleProps>, never>> & string;

package/types/components/widgets/AssetAction/AssetAction.styled.d.ts

@@ -1,14 +1,9 @@
 /**
- * `AssetActionButton`'s props, i.e. what `AssetAction.tsx` passes to it:
- * `$actionWidthHeight={action.actionWidthHeight}` and nothing else.
+ * Props passed by `AssetAction.tsx` to `<S.AssetActionButton>`.
  */
 interface AssetActionButtonStyleProps {
     /**
-     * `action.actionWidthHeight` — `PropTypes.arrayOf(PropTypes.number)`, read as
-     * `[width, height]`. `number[]` and not `[number, number]`: the propTypes say an
-     * array of numbers of any length, and the template guards on the array's presence
-     * rather than its length. The values are px and are also used arithmetically
-     * (`- 14`), so `number` and not `number | string`.
+     * `[width, height]` in pixels, used arithmetically.
      */
     $actionWidthHeight?: number[];
 }

package/types/components/widgets/AssetGallery/AssetGalleryBase/AssetGalleryBase.d.ts

@@ -32,18 +32,6 @@ type AssetGalleryBaseOwnProps = {
     customSelectedBorder?: string[];
     /**
      * An array containing the key of all selected assets
-     *
-     * NMB-3998: `(string | number)[]`, not the `string[] | number[]` the propTypes codemod
-     * produced from `oneOfType([arrayOf(string), arrayOf(number)])`. Calling `.includes()` on
-     * a union OF ARRAY TYPES intersects the candidate signatures' parameters, giving
-     * `string & number` = `never`, so all three `selectedAssetKeys.includes(...)` reads in
-     * this file failed with "not assignable to parameter of type 'never'".
-     *
-     * The union was also the wrong contract, not merely an awkward one. `Asset['key']` is
-     * `string | number`, and this array is compared against `asset.key` at lines 585, 659 and
-     * 676 — so a gallery whose assets have mixed key types needs a mixed array, which
-     * `arrayOf(string) | arrayOf(number)` cannot express. This records what the component
-     * does; it is a pure widening, so no existing consumer call site can break.
      */
     selectedAssetKeys: (string | number)[];
     /**

package/types/components/widgets/AssetGallery/AssetGalleryBase/AssetGalleryCompactCard/AssetGalleryCompactCard.styled.d.ts

@@ -1,68 +1,20 @@
 import type { AssetNoteType } from '../../asset.types';
-/**
- * `asset.disabled` — `PropTypes.bool` on the asset shape.
- *
- * NO `$`, AND THAT IS A DEFECT — AND THIS IS THE ONE OF THE FOUR THAT IS ALREADY IN
- * THE BASELINE. `shouldForwardProp` forwards every non-`$` prop; React does recognise
- * `disabled` and renders it as a bare boolean attribute even on a `<div>`, so
- * `disabled=""` sits on both the wrapper and the card root in
- * `scripts/r19-harness/baseline.json` under `widgets/AssetGalleryCompactCard#disabled`.
- * Grep it: `disabled=` appears 35 times across that file.
- *
- * The fix is to rename it `$disabled` at both ends, which REMOVES that attribute and
- * therefore MOVES the render baseline — C10 work, §E.4.
- *
- * Hosts: `AssetGalleryWrapper` and `AssetGalleryCompactCard`.
- */
+/** NO `$` is a defect but already in baseline (`disabled=""` on wrapper and card). Deferred to C10. */
 interface AssetGalleryDisabledStyleProps {
     disabled?: boolean;
 }
-/**
- * The selection-state props, declared once because six components take overlapping
- * subsets of them; each one below `Pick`s exactly what its JSX site passes.
- * `AssetGalleryCompactCard.tsx` is the only caller.
- */
+/** Selection-state props used across multiple components. AssetGalleryCompactCard.tsx is the only caller. */
 interface AssetGallerySelectionStyleProps {
-    /**
-     * `selected` — `PropTypes.bool`.
-     *
-     * NO `$`, AND THAT IS A DEFECT — BUT NOT A DOM LEAK, and the difference is worth
-     * stating because the obvious guess is wrong. `shouldForwardProp` does forward it to
-     * React, but React DROPS a boolean `selected` on anything that is not an `<option>`,
-     * silently and with no warning; measured by rendering `<div selected>` through
-     * react-dom@19.2.4, and confirmed by `selected=` appearing ZERO times in
-     * `scripts/r19-harness/baseline.json`.
-     *
-     * So the defect is that the prop is unfiltered and the render only stays clean because
-     * React cleans up after it. Prefixing it is still the right fix and still a behaviour
-     * change to review on its own (C10) — it just would not move the baseline.
-     */
+    /** NO `$` is a defect but not a DOM leak: React drops boolean `selected` on non-`<option>` elements. Deferred to C10. */
     selected?: boolean;
-    /** `softSelected` — `PropTypes.bool.isRequired` on the COMPONENT, but optional here: a
-     *  required declaration produces TS2769 at `<X />` and at `<X {...props} />`. §J.5. */
+    /** Optional here to avoid TS2769 at call sites, though required on the component itself. */
     $softSelected?: boolean;
-    /** `customSelectedBorder` — `PropTypes.arrayOf(PropTypes.string)`, read as
-     *  `[darkModeColor, lightModeColor]`. `string[]` and not `[string, string]`, because
-     *  that is what the propTypes say. */
+    /** Read as `[darkModeColor, lightModeColor]`; `string[]` not `[string, string]` to match propTypes. */
     $customSelectedBorder?: string[];
 }
-/**
- * The card root's props. `as={ComputedRootComponent}` is passed alongside these and that
- * wrapper re-renders this same styled component with the leftover props, so both passes
- * see the same shape.
- */
+/** Card root props (both passes see the same shape via ComputedRootComponent). */
 interface AssetGalleryCompactCardStyleProps extends AssetGalleryDisabledStyleProps, Pick<AssetGallerySelectionStyleProps, 'selected'> {
-    /**
-     * `extendedSelectMode` — `PropTypes.bool`. Makes the whole card a selectable area.
-     *
-     * NO `$`, AND THAT IS A DEFECT — and it reaches the DOM even less than `selected`
-     * does, for two independent reasons. `ComputedRootComponent` explicitly `omit`s this
-     * key before re-rendering the card, so the inner pass never sees it; and React drops
-     * an unrecognised camelCase prop whose value is `true` anyway, warning "React does
-     * not recognize the `extendedSelectMode` prop on a DOM element". Both measured.
-     * `extendedselectmode` appears ZERO times in the recorded baseline, with the base
-     * fixture passing `extendedSelectMode: true`. Declared here, renamed in C10.
-     */
+    /** NO `$` is a defect but not a DOM leak: ComputedRootComponent omits it and React drops camelCase props. Deferred to C10. */
     extendedSelectMode?: boolean;
 }
 export declare const AssetGalleryWrapper: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, AssetGalleryDisabledStyleProps>, never>> & string;

package/types/components/widgets/AssetGallery/AssetGalleryBase/AssetGalleryGridCard/AssetGalleryGridCard.styled.d.ts

@@ -1,56 +1,20 @@
 export declare const AssetGalleryWrapper: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, Pick<AssetGalleryGridCardStyleProps, "disabled">>, never>> & string;
-/**
- * What `ringState` reads. Also the reason the card root needs a generic at all: three of
- * these five are only ever read HERE, inside the `.attrs()` callback, not in the template.
- */
+/** Props that ringState reads. Reason card root needs a generic at all. */
 interface AssetGalleryGridCardRingProps {
-    /**
-     * `selected` — `PropTypes.bool`.
-     *
-     * NO `$`, AND THAT IS A DEFECT — but NOT a DOM leak. `ComputedRootComponent` spreads it
-     * onto a plain `<div>` and React DROPS a boolean `selected` on any element that is not
-     * an `<option>`, silently; `selected=` occurs zero times in
-     * `scripts/r19-harness/baseline.json`. Prefixing it is still a behaviour change to
-     * review on its own (C10, §E.4) — it just would not move the baseline.
-     */
+    /** NO `$` is a defect but not a DOM leak: React drops boolean `selected` on non-`<option>` elements. Deferred to C10. */
     selected?: boolean;
-    /**
-     * `asset.completed` — `PropTypes.bool`.
-     *
-     * NO `$`, AND THAT IS A DEFECT — and again not a DOM leak, for a different reason:
-     * `ComputedRootComponent` explicitly `omit`s `completed`, and React would drop it
-     * anyway ("Received `true` for a non-boolean attribute `completed`"). Measured both ways.
-     */
+    /** NO `$` is a defect but not a DOM leak: ComputedRootComponent explicitly omits it. Deferred to C10. */
     completed?: boolean;
-    /** `softSelected` — `PropTypes.bool.isRequired` on the COMPONENT, optional here per §J.5. */
     $softSelected?: boolean;
-    /** `asset.hasError` — `PropTypes.bool`. */
     $hasError?: boolean;
-    /** `customSelectedBorder` — `PropTypes.arrayOf(PropTypes.string)`, read as
-     *  `[darkModeColor, lightModeColor]` and written into the `--ring-dark` / `--ring-light`
-     *  custom properties by the `.attrs()` callback. */
+    /** Read as `[darkModeColor, lightModeColor]` and written to `--ring-dark` / `--ring-light` custom properties. */
     $customSelectedBorder?: string[];
 }
-/**
- * The card root's props: everything `ringState` reads, plus the two the TEMPLATE reads.
- */
+/** Card root props: everything ringState reads, plus what the template reads. */
 interface AssetGalleryGridCardStyleProps extends AssetGalleryGridCardRingProps {
-    /**
-     * `asset.disabled` — `PropTypes.bool`.
-     *
-     * NO `$`, AND THIS IS THE ONE THAT REALLY REACHES THE DOM. React recognises
-     * `disabled` and renders it as a bare boolean attribute even on a `<div>`, so
-     * `disabled=""` is on both the wrapper and the card root in
-     * `widgets/AssetGalleryGridCard#disabled` in the recorded baseline. Renaming it
-     * `$disabled` REMOVES that attribute and therefore MOVES the baseline — C10, §E.4.
-     */
+    /** NO `$` reaches the DOM; defect deferred to C10. Baseline shows `disabled=""` on wrapper and card root. */
     disabled?: boolean;
-    /**
-     * `extendedSelectMode` — `PropTypes.bool`. Makes the whole card a selectable area.
-     *
-     * NO `$`, AND THAT IS A DEFECT — not a DOM leak twice over: `ComputedRootComponent`
-     * `omit`s it, and React drops an unrecognised camelCase prop whose value is `true`.
-     */
+    /** NO `$` is a defect but not a DOM leak: ComputedRootComponent omits it and React drops unrecognised camelCase props. */
     extendedSelectMode?: boolean;
 }
 export declare const AssetGalleryGridCard: import("styled-components/dist/types").IStyledComponentBase<"web", import("styled-components").FastOmit<import("styled-components/dist/types").Substitute<import("styled-components/dist/types").Substitute<import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>, import("react").DetailedHTMLProps<import("react").HTMLAttributes<HTMLDivElement>, HTMLDivElement>>, AssetGalleryGridCardStyleProps>, never>> & string;