@multiplatform.one/forms 7.90.6 → 8.0.0

package/src/FloatingPanel/useFloatingPanel.spec.tsx

@@ -30,20 +30,17 @@ import {
   panelAnchorOffset,
   panelCoverOffset,
   panelViewportPadding,
-  resolvePanelWidthMode,
   useFloatingPanel,
 } from "./useFloatingPanel";
 
 function Harness({
   open,
-  fitContent,
   widthMode,
   panelWidth,
   echo,
   floor,
 }: {
   open: boolean;
-  fitContent?: boolean;
   widthMode?: PanelWidthMode;
   panelWidth?: PanelWidth;
   echo?: boolean;
@@ -53,7 +50,6 @@ function Harness({
     open,
     onOpenChange: () => {},
     useAutoUpdate: false,
-    fitContent,
     widthMode,
     panelWidth,
     echo,
@@ -89,7 +85,6 @@ function mockRect(el: HTMLElement, rect: { x: number; y: number; width: number;
  *  floating element once floating-ui has positioned it. */
 async function openAgainstTrigger(
   props: {
-    fitContent?: boolean;
     widthMode?: PanelWidthMode;
     panelWidth?: PanelWidth;
     echo?: boolean;
@@ -276,15 +271,6 @@ describe("FloatingPanel panelWidth setting (LC-112, FZ-8438 r3)", () => {
 });
 
 describe("FloatingPanel width modes (MPO-38)", () => {
-  it("resolves the deprecated boolean onto the three modes", () => {
-    expect(resolvePanelWidthMode(undefined, undefined)).toBe("at-least-trigger");
-    expect(resolvePanelWidthMode(undefined, false)).toBe("at-least-trigger");
-    expect(resolvePanelWidthMode(undefined, true)).toBe("fit-content");
-    // An explicit mode always wins over the boolean, in both directions.
-    expect(resolvePanelWidthMode("match-trigger", true)).toBe("match-trigger");
-    expect(resolvePanelWidthMode("fit-content", false)).toBe("fit-content");
-  });
-
   it("at-least-trigger writes min-width only, so content may still grow it", async () => {
     const result = await openAgainstTrigger({ widthMode: "at-least-trigger" }, trigger);
     await waitFor(() => {
@@ -302,13 +288,6 @@ describe("FloatingPanel width modes (MPO-38)", () => {
     });
   });
 
-  it("at-least-trigger is what an explicit fitContent={false} means", async () => {
-    const result = await openAgainstTrigger({ fitContent: false }, trigger);
-    await waitFor(() => {
-      expect(result.getByTestId("anchor-floating").style.minWidth).toBe("240px");
-    });
-  });
-
   it("match-trigger pins the width exactly — narrow content cannot shrink it", async () => {
     const result = await openAgainstTrigger({ widthMode: "match-trigger" }, trigger);
     await waitFor(() => {
@@ -336,11 +315,6 @@ describe("FloatingPanel width modes (MPO-38)", () => {
     expect(el.style.width).toBe("");
     expect(el.style.maxWidth).toBe("");
   });
-
-  it("fitContent still reaches fit-content, so existing consumers do not move", async () => {
-    const result = await openAgainstTrigger({ fitContent: true }, trigger);
-    expect(result.getByTestId("anchor-floating").style.minWidth).toBe("");
-  });
 });
 
 /**

package/src/FloatingPanel/useFloatingPanel.ts

@@ -133,13 +133,6 @@ export interface UseFloatingPanelOptions {
    * Default `"at-least-trigger"`.
    */
   widthMode?: PanelWidthMode;
-  /**
-   * @deprecated Pass `widthMode` instead. `fitContent` is the two-valued
-   * spelling of the same idea and cannot express `"match-trigger"`.
-   * `true` → `"fit-content"`, `false`/undefined → `"at-least-trigger"`.
-   * An explicit `widthMode` always wins.
-   */
-  fitContent?: boolean;
   /**
    * The panel opens at a point, not from the trigger's box (ContextMenu). It
    * shares no corner with the trigger, so its radius stays the CONTAINER-CAP
@@ -163,16 +156,6 @@ export interface UseFloatingPanelOptions {
   floor?: number;
 }
 
-/** One place that resolves the deprecated boolean onto the three modes, so
- *  the hook and the component cannot disagree about what a consumer meant. */
-export function resolvePanelWidthMode(
-  widthMode: PanelWidthMode | undefined,
-  fitContent: boolean | undefined,
-): PanelWidthMode {
-  if (widthMode) return widthMode;
-  return fitContent ? "fit-content" : "at-least-trigger";
-}
-
 // ── Return type ───────────────────────────────────────────────
 
 export interface UseFloatingPanelReturn {
@@ -287,14 +270,13 @@ export function useFloatingPanel({
   useAutoUpdate: useAutoUpdateProp = true,
   wheelGrowRef,
   widthMode: widthModeProp,
-  fitContent,
   freeOverlay = false,
   cover = true,
   panelWidth,
   echo = cover && !freeOverlay,
   floor = 0,
 }: UseFloatingPanelOptions): UseFloatingPanelReturn {
-  const widthMode = resolvePanelWidthMode(widthModeProp, fitContent);
+  const widthMode = widthModeProp ?? "at-least-trigger";
   // An overlay floats over the page: it never inherits a painted ancestor.
   const { knobProps } = useResolvedKnobs({ painted: false });
   const hasBorder = knobProps.borderRadius.borderWidth > 0;

package/src/FloatingPanel/viewportHydration.spec.tsx

@@ -23,14 +23,14 @@ function Fixture({ kind }: { kind: string }) {
           aria-label="Choice"
           options={options}
           value={value}
-          onValueChange={(next) => setValue(String(next))}
+          onChange={(next) => setValue(String(next))}
         />
       ) : kind === "Combobox" ? (
         <Combobox
           aria-label="Choice"
           options={options}
           value={value}
-          onValueChange={(next) => setValue(String(next))}
+          onChange={(next) => setValue(String(next))}
         />
       ) : (
         <FloatingPanel

package/src/fields/Checkbox/Checkbox.spec.tsx

@@ -62,14 +62,14 @@ describe("Checkbox", () => {
       expect(result.container.querySelector("[role='checkbox']")).toBeNull();
     });
 
-    it("calls onCheckedChange when clicked", () => {
-      const onCheckedChange = vi.fn();
+    it("calls onChange when clicked", () => {
+      const onChange = vi.fn();
       const result = renderWithProviders(
-        <Checkbox name="cb" defaultValue={false} onCheckedChange={onCheckedChange} />,
+        <Checkbox name="cb" defaultValue={false} onChange={onChange} />,
       );
       const checkbox = result.getCheckbox();
       if (checkbox) fireEvent.click(checkbox);
-      expect(onCheckedChange).toHaveBeenCalledWith(true);
+      expect(onChange).toHaveBeenCalledWith(true);
     });
 
     it("clicking the painted box still toggles", () => {
@@ -86,23 +86,15 @@ describe("Checkbox", () => {
       expect(onChange).toHaveBeenCalledWith(true);
     });
 
-    it("fires canonical onChange and the deprecated onCheckedChange alias once each", () => {
+    it("fires canonical onChange once", () => {
       const onChange = vi.fn();
-      const onCheckedChange = vi.fn();
       const result = renderWithProviders(
-        <Checkbox
-          name="cb"
-          defaultValue={false}
-          onChange={onChange}
-          onCheckedChange={onCheckedChange}
-        />,
+        <Checkbox name="cb" defaultValue={false} onChange={onChange} />,
       );
       const checkbox = result.getCheckbox();
       if (checkbox) fireEvent.click(checkbox);
       expect(onChange).toHaveBeenCalledTimes(1);
       expect(onChange).toHaveBeenCalledWith(true);
-      expect(onCheckedChange).toHaveBeenCalledTimes(1);
-      expect(onCheckedChange).toHaveBeenCalledWith(true);
     });
   });
 
@@ -236,25 +228,16 @@ describe("CheckboxGroup", () => {
     expect(result.findTextElement("Option C")).toBeDefined();
   });
 
-  it("fires canonical onChange and the deprecated onValueChange alias once each", () => {
+  it("fires canonical onChange once", () => {
     const onChange = vi.fn();
-    const onValueChange = vi.fn();
     const result = renderWithProviders(
-      <CheckboxGroup
-        label="Pick options"
-        name="opts"
-        options={options}
-        onChange={onChange}
-        onValueChange={onValueChange}
-      />,
+      <CheckboxGroup label="Pick options" name="opts" options={options} onChange={onChange} />,
     );
     const checkbox = result.container.querySelector("[role='checkbox']");
     expect(checkbox).toBeTruthy();
     if (checkbox) fireEvent.click(checkbox);
     expect(onChange).toHaveBeenCalledTimes(1);
     expect(onChange).toHaveBeenCalledWith(["a"]);
-    expect(onValueChange).toHaveBeenCalledTimes(1);
-    expect(onValueChange).toHaveBeenCalledWith(["a"]);
   });
 
   it("renders required label with asterisk", () => {

package/src/fields/Checkbox/Checkbox.stories.tsx

@@ -182,9 +182,9 @@ export const Comparison: StoryObj = {
             <Checkbox
               label="Accept terms"
               defaultValue={customChecked}
-              onCheckedChange={(val) => {
+              onChange={(val) => {
                 setCustomChecked(!!val);
-                action("custom.onCheckedChange")(val);
+                action("custom.onChange")(val);
               }}
             />
           </YStack>
@@ -332,9 +332,9 @@ export const CheckboxGroupInteractive: StoryObj = {
               { value: "opt3", label: "Option 3" },
             ]}
             value={selected}
-            onValueChange={(values) => {
+            onChange={(values) => {
               setSelected(values);
-              action("onValueChange")(values);
+              action("onChange")(values);
             }}
           />
           <Paragraph size="$2">Selected: {selected.join(", ") || "None"}</Paragraph>

package/src/fields/Checkbox/Checkbox.tsx

@@ -64,8 +64,6 @@ export interface CheckboxProps {
    * (Input/Select/Switch all use `onChange`). Prefer this.
    */
   onChange?: (checked: CheckedState) => void;
-  /** @deprecated Use `onChange` instead. Kept as an alias for the tamagui/Radix name. */
-  onCheckedChange?: (checked: CheckedState) => void;
   onBlur?: (...args: any[]) => void;
   checkboxProps?: Omit<TamaguiCheckboxProps, "children" | "checked" | "id" | "onCheckedChange">;
   children?: ReactNode;
@@ -102,18 +100,16 @@ export function Checkbox({
   checked,
   defaultValue,
   onChange,
-  onCheckedChange,
   onBlur,
   checkboxProps,
   children,
   pointy,
   skeleton,
 }: CheckboxProps) {
-  // Canonical `onChange` and the alias `onCheckedChange` both fire from a
+  // Canonical `onChange` fires from a
   // single emit site (same reconciliation as Switch).
   const handleValueChange = (val: CheckedState) => {
     onChange?.(val);
-    onCheckedChange?.(val);
   };
   const { resolvedForm, knobProps, text, elevation, disabledState, id } = useFormField({
     form: formProp,

package/src/fields/Checkbox/CheckboxGroup.stories.tsx

@@ -40,7 +40,7 @@ type Story = StoryObj<typeof CheckboxGroup>;
 export const Basic: Story = {
   render: (args) => (
     <YStack maxWidth={400}>
-      <CheckboxGroup {...args} onValueChange={action("onValueChange")} />
+      <CheckboxGroup {...args} onChange={action("onChange")} />
     </YStack>
   ),
 };
@@ -226,7 +226,7 @@ export const ElevationLarge: Story = {
   render: (args) => (
     <Preset overrides={{ elevation: "large" }}>
       <YStack maxWidth={400}>
-        <CheckboxGroup {...args} onValueChange={action("onValueChange")} />
+        <CheckboxGroup {...args} onChange={action("onChange")} />
       </YStack>
     </Preset>
   ),

package/src/fields/Checkbox/CheckboxGroup.tsx

@@ -54,11 +54,8 @@ export interface CheckboxGroupProps {
   defaultValue?: string[];
   /**
    * Canonical change handler, consistent with the rest of the field family.
-   * Prefer this over `onValueChange`.
    */
   onChange?: (value: string[]) => void;
-  /** @deprecated Use `onChange` instead. */
-  onValueChange?: (value: string[]) => void;
   onBlur?: (...args: any[]) => void;
   options?: CheckboxGroupOption[];
   orientation?: "horizontal" | "vertical";
@@ -89,7 +86,6 @@ export function CheckboxGroup({
   value,
   defaultValue,
   onChange,
-  onValueChange,
   onBlur,
   options,
   orientation = "vertical",
@@ -98,11 +94,10 @@ export function CheckboxGroup({
   skeleton,
   card,
 }: CheckboxGroupProps) {
-  // Canonical `onChange` and the alias `onValueChange` both fire from the
+  // Canonical `onChange` fires from the
   // single emit site below (same reconciliation as Switch).
   const handleValueChange = (newValues: string[]) => {
     onChange?.(newValues);
-    onValueChange?.(newValues);
   };
   const { resolvedForm, knobProps, disabledState, id } = useFormField({
     form: formProp,
@@ -212,7 +207,7 @@ export function CheckboxGroup({
                 <Checkbox
                   id={itemId}
                   checked={isChecked}
-                  onCheckedChange={(checked) => {
+                  onChange={(checked) => {
                     if (isDisabled || readOnly) return;
                     emit(toggleValue(currentValues, option.value, checked));
                   }}
@@ -260,7 +255,7 @@ export function CheckboxGroup({
               label={option.label}
               id={itemId}
               checked={isChecked}
-              onCheckedChange={(checked) => {
+              onChange={(checked) => {
                 if (isDisabled || readOnly) return;
                 const newValues = toggleValue(currentValues, option.value, checked);
                 handleChange?.(newValues);

package/src/fields/ColorPicker/ColorPicker.spec.tsx

@@ -118,18 +118,11 @@ describe("ColorPicker", () => {
     });
   });
 
-  it("fires canonical onChange and the deprecated onValueChange alias once when cleared", async () => {
+  it("fires canonical onChange once when cleared", async () => {
     const ColorPicker = await loadColorPicker();
     const onChange = vi.fn();
-    const onValueChange = vi.fn();
     const result = renderWithProviders(
-      <ColorPicker
-        label="Color"
-        name="color"
-        defaultValue="#112233"
-        onChange={onChange}
-        onValueChange={onValueChange}
-      />,
+      <ColorPicker label="Color" name="color" defaultValue="#112233" onChange={onChange} />,
     );
 
     const triggerButton = result.container.querySelector("button");
@@ -142,8 +135,6 @@ describe("ColorPicker", () => {
       expect(onChange).toHaveBeenCalledWith("");
     });
     expect(onChange).toHaveBeenCalledTimes(1);
-    expect(onValueChange).toHaveBeenCalledTimes(1);
-    expect(onValueChange).toHaveBeenCalledWith("");
   });
 
   it("shows format validation error for unrecognized rgb value", async () => {

package/src/fields/ColorPicker/index.tsx

@@ -77,11 +77,9 @@ export type ColorPickerProps<
     value?: string;
     /**
      * Canonical change handler, consistent with the rest of the field family.
-     * Fires in both standalone and form mode. Prefer this over `onValueChange`.
+     * Fires in both standalone and form mode.
      */
     onChange?: (value: string) => void;
-    /** @deprecated Use `onChange` instead. */
-    onValueChange?: (value: string) => void;
     /** When true, renders a skeleton placeholder instead of the color picker */
     skeleton?: boolean;
   };
@@ -743,7 +741,6 @@ export function ColorPicker<
     onBlur,
     value: controlledValue,
     onChange,
-    onValueChange,
     skeleton,
     id: idProp,
   } = useProps(props);
@@ -751,14 +748,13 @@ export function ColorPicker<
   const themePresets = useThemePresetColors();
   const presetColors = presetColorsProp ?? themePresets;
 
-  // Canonical `onChange` and the alias `onValueChange` both fire from a
+  // Canonical `onChange` fires from a
   // single emit per change (same reconciliation as Switch).
   const emitValueChange = useCallback(
     (newValue: string) => {
       onChange?.(newValue);
-      onValueChange?.(newValue);
     },
-    [onChange, onValueChange],
+    [onChange],
   );
 
   const { resolvedForm, knobProps, id } = useFormField({ form, id: idProp });

package/src/fields/Combobox/Combobox.spec.tsx

@@ -110,9 +110,8 @@ describe("Combobox", () => {
       expect(findInPage("Disabled")).toBeDefined();
     });
 
-    it("fires canonical onChange and the deprecated onValueChange alias once when dismissing a chip", async () => {
+    it("fires canonical onChange once when dismissing a chip", async () => {
       const onChange = vi.fn();
-      const onValueChange = vi.fn();
       const result = renderWithProviders(
         <Combobox
           label="Fruits"
@@ -122,7 +121,6 @@ describe("Combobox", () => {
           dismissible
           value={["apple", "banana"]}
           onChange={onChange}
-          onValueChange={onValueChange}
         />,
       );
       // Labels are differentiated per chip now — "Remove {label}".
@@ -134,8 +132,6 @@ describe("Combobox", () => {
       // Exactly once each — guards against the standalone double-fire bug.
       expect(onChange).toHaveBeenCalledTimes(1);
       expect(onChange).toHaveBeenCalledWith(["banana"]);
-      expect(onValueChange).toHaveBeenCalledTimes(1);
-      expect(onValueChange).toHaveBeenCalledWith(["banana"]);
     });
 
     it("stable keeps Cherry in third position when selected and open", async () => {

package/src/fields/Combobox/Combobox.stories.tsx

@@ -182,9 +182,9 @@ export const Basic: Story = {
         <Combobox
           {...args}
           value={value}
-          onValueChange={(val) => {
+          onChange={(val) => {
             setValue(Array.isArray(val) ? val[0] : val);
-            action("onValueChange")(val);
+            action("onChange")(val);
           }}
         />
         {value && <Text color="$colorHover">Selected: {value}</Text>}
@@ -203,7 +203,7 @@ export const WithLabel: Story = {
           helperText="Select an author from the list"
           options={authorOptions}
           value={value}
-          onValueChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
+          onChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
           placeholder="Select an author..."
           searchPlaceholder="Type to search..."
         />
@@ -251,7 +251,7 @@ export const Interactive: Story = {
             label="Pick an author"
             options={authorOptions}
             value={value}
-            onValueChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
+            onChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
             placeholder="Select an author..."
           />
           <Text>Selected: {value || "None"}</Text>
@@ -275,7 +275,7 @@ export const CustomTrigger: StoryObj<typeof Combobox> = {
             <Combobox
               options={authorOptions}
               value={value}
-              onValueChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
+              onChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
               searchPlaceholder="Type to search..."
               popoverWidth={260}
               trigger={(selected) => (
@@ -321,7 +321,7 @@ export const ManyOptions: StoryObj<typeof Combobox> = {
           label="Pick an item"
           options={options}
           value={value}
-          onValueChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
+          onChange={(v) => setValue(Array.isArray(v) ? v[0] : v)}
           placeholder="Select from 100 items..."
           searchPlaceholder="Type to filter..."
           emptyMessage="No matching items."

package/src/fields/Combobox/index.tsx

@@ -163,11 +163,8 @@ export interface ComboboxProps {
   defaultValue?: string | string[];
   /**
    * Canonical change handler, consistent with the rest of the field family.
-   * Prefer this over `onValueChange`.
    */
   onChange?: (value: string | string[]) => void;
-  /** @deprecated Use `onChange` instead. */
-  onValueChange?: (value: string | string[]) => void;
   placeholder?: string;
   searchPlaceholder?: string;
   emptyMessage?: string;
@@ -2564,7 +2561,6 @@ export function Combobox({
   value: valueProp,
   defaultValue,
   onChange,
-  onValueChange,
   placeholder: placeholderProp,
   searchPlaceholder: searchPlaceholderProp,
   emptyMessage: emptyMessageProp,
@@ -2704,10 +2700,9 @@ export function Combobox({
         onValueChange={(value) => {
           if (disabled) return;
           handleValueChange?.(value);
-          // Canonical `onChange` and the alias `onValueChange` both fire from
+          // Canonical `onChange` fires from
           // this single emit site (same reconciliation as Switch).
           onChange?.(value);
-          onValueChange?.(value);
         }}
         onBlur={handleBlur}
         placeholder={placeholder}

package/src/fields/DatePicker/DatePicker.spec.tsx

@@ -82,12 +82,11 @@ describe("DatePicker", () => {
       expect(document.querySelector('[data-testid="floating-panel-viewport"]')).toBeNull();
     });
 
-    it("fires canonical onChange and the deprecated onValueChange alias once when a day is picked", async () => {
+    it("fires canonical onChange once when a day is picked", async () => {
       const DatePicker = await loadDatePicker();
       const onChange = vi.fn();
-      const onValueChange = vi.fn();
       const result = renderWithProviders(
-        <DatePicker label="Date" name="date" onChange={onChange} onValueChange={onValueChange} />,
+        <DatePicker label="Date" name="date" onChange={onChange} />,
       );
 
       const trigger = result.container.querySelector("input,button,[role='button']");
@@ -106,9 +105,7 @@ describe("DatePicker", () => {
       });
 
       expect(onChange).toHaveBeenCalledTimes(1);
-      expect(onValueChange).toHaveBeenCalledTimes(1);
       expect(onChange.mock.calls[0][0]).toBeInstanceOf(Date);
-      expect(onChange.mock.calls[0][0]).toEqual(onValueChange.mock.calls[0][0]);
     });
   });
 
@@ -262,32 +259,18 @@ describe("DatePicker", () => {
     it("standalone controlled: a value arriving after mount displays, without onChange", async () => {
       const DatePicker = await loadDatePicker();
       const onChange = vi.fn();
-      const onValueChange = vi.fn();
       const result = renderWithProviders(
-        <DatePicker
-          label="Due Date"
-          value={null}
-          onChange={onChange}
-          onValueChange={onValueChange}
-        />,
+        <DatePicker label="Due Date" value={null} onChange={onChange} />,
       );
       expect(getDisplayInput(result.container)?.value ?? "").toBe("");
 
       // Late seed: the upstream doc value (frappe wire string) arrives after
       // first paint.
-      result.rerender(
-        <DatePicker
-          label="Due Date"
-          value="2026-03-05"
-          onChange={onChange}
-          onValueChange={onValueChange}
-        />,
-      );
+      result.rerender(<DatePicker label="Due Date" value="2026-03-05" onChange={onChange} />);
       await waitFor(() => {
         expect(getDisplayInput(result.container)?.value).toBe("Mar 5, 2026");
       });
       expect(onChange).not.toHaveBeenCalled();
-      expect(onValueChange).not.toHaveBeenCalled();
     });
 
     it("form-integrated: a field value set after mount displays, without onChange", async () => {

package/src/fields/DatePicker/DatetimePicker.catalog.tsx

@@ -75,11 +75,8 @@ export type DatetimePickerProps<
     value?: Date | string | null;
     /**
      * Canonical change handler, consistent with the rest of the field family.
-     * Prefer this over `onValueChange`.
      */
     onChange?: (value: Date | null) => void;
-    /** @deprecated Use `onChange` instead. */
-    onValueChange?: (value: Date | null) => void;
     /** Native only. `false` keeps the catalog overlay; omit for the OS picker. */
     native?: boolean;
     /**
@@ -359,7 +356,6 @@ export function DatetimePicker<
     defaultValue,
     value: controlledValue,
     onChange,
-    onValueChange,
     form,
     inputProps: _inputProps,
     placeholder = t("Select date and time"),
@@ -419,11 +415,10 @@ export function DatetimePicker<
       setSelectedDatetime(date);
       setDisplayDate(date);
       fieldChange?.(date);
-      // Canonical `onChange` and the alias `onValueChange` both fire (single emit).
+      // Canonical `onChange` fires (single emit).
       onChange?.(date);
-      onValueChange?.(date);
     },
-    [minDate, maxDate, setDisplayDate, onChange, onValueChange],
+    [minDate, maxDate, setDisplayDate, onChange],
   );
 
   const handleClearDatetime = useCallback(
@@ -432,9 +427,8 @@ export function DatetimePicker<
       setDisplayDate(new Date());
       fieldChange?.(null);
       onChange?.(null);
-      onValueChange?.(null);
     },
-    [setDisplayDate, onChange, onValueChange],
+    [setDisplayDate, onChange],
   );
 
   const displayValue = useMemo(

package/src/fields/DatePicker/DatetimePicker.native.spec.tsx

@@ -66,10 +66,7 @@ describe("native DatetimePicker ownership", () => {
 
   it("opens from its full-width native target and commits exactly once", () => {
     const onChange = vi.fn();
-    const onValueChange = vi.fn();
-    const result = renderWithProviders(
-      <DatetimePicker onChange={onChange} onValueChange={onValueChange} />,
-    );
+    const result = renderWithProviders(<DatetimePicker onChange={onChange} />);
     const trigger = result.getByRole("button", { name: "Date and time" });
     expect(getComputedStyle(trigger).width).toBe("100%");
     fireEvent.click(trigger);
@@ -77,7 +74,6 @@ describe("native DatetimePicker ownership", () => {
     fireEvent.click(result.getByRole("button", { name: "Confirm OS selection" }));
     expect(result.queryByRole("dialog")).toBeNull();
     expect(onChange).toHaveBeenCalledExactlyOnceWith(picked);
-    expect(onValueChange).toHaveBeenCalledExactlyOnceWith(picked);
     expect(result.container.querySelector("input")?.value).toContain("Mar 5, 2026");
   });
 
@@ -107,19 +103,13 @@ describe("native DatetimePicker ownership", () => {
 
   it("adopts controlled values after mount without emitting changes or opening", async () => {
     const onChange = vi.fn();
-    const onValueChange = vi.fn();
-    const result = renderWithProviders(
-      <DatetimePicker value={null} onChange={onChange} onValueChange={onValueChange} />,
-    );
+    const result = renderWithProviders(<DatetimePicker value={null} onChange={onChange} />);
     expect(result.container.querySelector("input")?.value).toBe("");
-    result.rerender(
-      <DatetimePicker value={picked} onChange={onChange} onValueChange={onValueChange} />,
-    );
+    result.rerender(<DatetimePicker value={picked} onChange={onChange} />);
     await waitFor(() =>
       expect(result.container.querySelector("input")?.value).toContain("Mar 5, 2026"),
     );
     expect(onChange).not.toHaveBeenCalled();
-    expect(onValueChange).not.toHaveBeenCalled();
     expect(result.queryByRole("dialog")).toBeNull();
   });