@jobber/components 6.103.4-uncontroll-d4ef425.5 → 6.103.4-uncontroll-3a6b89f.8

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]);
@@ -356,9 +359,8 @@ function useAutocomplete(props) {
     // This is only options
     const optionCount = renderable.reduce((count, item) => count + (item.kind === "option" ? 1 : 0), 0);
     const hasSelection = React.useMemo(() => {
-        var _a;
         if (multiple) {
-            const current = (_a = value) !== null && _a !== void 0 ? _a : [];
+            const current = value !== null && value !== void 0 ? value : [];
             return Array.isArray(current) && current.length > 0;
         }
         return value != null;
@@ -413,11 +415,15 @@ 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
+        const currentlyHasSelection = value != null;
+        if (lastInputWasUser.current && currentlyHasSelection) {
             onChange === null || onChange === void 0 ? void 0 : onChange(undefined);
         }
-    }, [inputValue, multiple, hasSelection, setActiveIndex, onChange, open]);
+    }, [inputValue, multiple, setActiveIndex, onChange, open, value]);
     function selectOption(option) {
         var _a;
         if (multiple) {

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]);
@@ -354,9 +357,8 @@ function useAutocomplete(props) {
     // This is only options
     const optionCount = renderable.reduce((count, item) => count + (item.kind === "option" ? 1 : 0), 0);
     const hasSelection = useMemo(() => {
-        var _a;
         if (multiple) {
-            const current = (_a = value) !== null && _a !== void 0 ? _a : [];
+            const current = value !== null && value !== void 0 ? value : [];
             return Array.isArray(current) && current.length > 0;
         }
         return value != null;
@@ -411,11 +413,15 @@ 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
+        const currentlyHasSelection = value != null;
+        if (lastInputWasUser.current && currentlyHasSelection) {
             onChange === null || onChange === void 0 ? void 0 : onChange(undefined);
         }
-    }, [inputValue, multiple, hasSelection, setActiveIndex, onChange, open]);
+    }, [inputValue, multiple, setActiveIndex, onChange, open, value]);
     function selectOption(option) {
         var _a;
         if (multiple) {

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-3a6b89f.8+3a6b89f05",
   "license": "MIT",
   "type": "module",
   "main": "dist/index.cjs",
@@ -538,5 +538,5 @@
     "> 1%",
     "IE 10"
   ],
-  "gitHead": "d4ef4259debb09593eabc29a42b18faa208329b8"
+  "gitHead": "3a6b89f051c7bf248c95263b514cd02722010dd7"
 }