@jobber/components 6.103.4-uncontroll-d4ef425.5 → 6.103.4-uncontroll-951d07b.6

package/dist/Autocomplete/index.cjs

@@ -258,6 +258,9 @@ function useAutocomplete(props) {
         if (prevValueRef.current !== value) {
             prevValueRef.current = value;
             const currentValue = value;
+            // Mark this as a programmatic change, not user input
+            // This prevents onChange from being called when we sync from prop changes
+            lastInputWasUser.current = false;
             setInternalInputValue(currentValue ? getOptionLabel(currentValue) : "");
         }
     }, [value, inputValueProp, multiple, getOptionLabel]);
@@ -413,8 +416,11 @@ function useAutocomplete(props) {
         // In multiple mode, clearing the input should NOT clear the selection
         if (multiple)
             return;
-        // For single-select, treat clearing input as clearing the selection
-        if (hasSelection) {
+        // Only clear the selection if the user actually cleared the input
+        // (not from internal state sync when parent changes the value prop)
+        // This prevents calling onChange when we're syncing state from a controlled value prop,
+        // but still allows onChange to fire when the user deletes the input text
+        if (lastInputWasUser.current && hasSelection) {
             onChange === null || onChange === void 0 ? void 0 : onChange(undefined);
         }
     }, [inputValue, multiple, hasSelection, setActiveIndex, onChange, open]);

package/dist/Autocomplete/index.mjs

@@ -256,6 +256,9 @@ function useAutocomplete(props) {
         if (prevValueRef.current !== value) {
             prevValueRef.current = value;
             const currentValue = value;
+            // Mark this as a programmatic change, not user input
+            // This prevents onChange from being called when we sync from prop changes
+            lastInputWasUser.current = false;
             setInternalInputValue(currentValue ? getOptionLabel(currentValue) : "");
         }
     }, [value, inputValueProp, multiple, getOptionLabel]);
@@ -411,8 +414,11 @@ function useAutocomplete(props) {
         // In multiple mode, clearing the input should NOT clear the selection
         if (multiple)
             return;
-        // For single-select, treat clearing input as clearing the selection
-        if (hasSelection) {
+        // Only clear the selection if the user actually cleared the input
+        // (not from internal state sync when parent changes the value prop)
+        // This prevents calling onChange when we're syncing state from a controlled value prop,
+        // but still allows onChange to fire when the user deletes the input text
+        if (lastInputWasUser.current && hasSelection) {
             onChange === null || onChange === void 0 ? void 0 : onChange(undefined);
         }
     }, [inputValue, multiple, hasSelection, setActiveIndex, onChange, open]);

package/dist/Autocomplete/tests/Autocomplete.setup.d.ts

@@ -36,3 +36,20 @@ export declare function FreeFormWrapper({ initialValue, initialInputValue, onCha
     readonly inputEqualsOption?: (input: string, option: OptionLike) => boolean;
     readonly debounce?: number;
 }): React.JSX.Element;
+export declare function UncontrolledWrapper<T extends OptionLike>({ defaultValue, menu, placeholder, }: {
+    readonly defaultValue?: T;
+    readonly menu: MenuItem<T>[];
+    readonly placeholder?: string;
+}): React.JSX.Element;
+export declare function SemiControlledWrapper<T extends OptionLike>({ initialValue, onChange, menu, placeholder, }: {
+    readonly initialValue?: T;
+    readonly onChange?: (v: T | undefined) => void;
+    readonly menu: MenuItem<T>[];
+    readonly placeholder?: string;
+}): React.JSX.Element;
+export declare function ControlledValueWrapper<T extends OptionLike>({ value, onChange, menu, placeholder, }: {
+    readonly value: T | undefined;
+    readonly onChange: (v: T | undefined) => void;
+    readonly menu: MenuItem<T>[];
+    readonly placeholder?: string;
+}): React.JSX.Element;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jobber/components",
-  "version": "6.103.4-uncontroll-d4ef425.5+d4ef4259d",
+  "version": "6.103.4-uncontroll-951d07b.6+951d07b93",
   "license": "MIT",
   "type": "module",
   "main": "dist/index.cjs",
@@ -538,5 +538,5 @@
     "> 1%",
     "IE 10"
   ],
-  "gitHead": "d4ef4259debb09593eabc29a42b18faa208329b8"
+  "gitHead": "951d07b936bd3e0cc6dc48c9a4cb98f2a93a838e"
 }