Linting fixes

Author: dannyroosevelt

packages/connect-react/CHANGELOG.md

@@ -4,8 +4,8 @@
 
 # [1.2.1] - 2025-06-07
 
-- Fix SelectApp component to properly handle controlled values when not found in search results
-- Fix infinite re-render issue in ComponentFormContainer by memoizing configurableProps
+- Fixing the SelectApp component to properly handle controlled values when not found in search results
+- Fixing infinite re-render issue in ComponentFormContainer by memoizing configurableProps
 
 # [1.2.0] - 2025-06-05
 

packages/connect-react/src/components/SelectApp.tsx

@@ -21,18 +21,20 @@ export function SelectApp({
     q,
     setQ,
   ] = useState(""); // Debounced query value
-  
+
   const instanceId = useId();
-  
+
   // Debounce the search query
   useEffect(() => {
     const timer = setTimeout(() => {
       setQ(inputValue);
     }, 300); // 300ms delay
-    
+
     return () => clearTimeout(timer);
-  }, [inputValue]);
-  
+  }, [
+    inputValue,
+  ]);
+
   const {
     isLoading,
     // TODO error
@@ -45,8 +47,10 @@ export function SelectApp({
     SingleValue,
   } = components;
   // If we have a value prop but it's not in the search results, use the value prop directly
-  const selectedValue = apps?.find((o) => o.name_slug === value?.name_slug) || 
-    (value?.name_slug ? value as AppResponse : null);
+  const selectedValue = apps?.find((o) => o.name_slug === value?.name_slug)
+    || (value?.name_slug
+      ? value as AppResponse
+      : null);
   return (
     <Select
       instanceId={instanceId}

packages/connect-react/src/hooks/form-context.tsx

@@ -101,7 +101,7 @@ export const FormContextProvider = <T extends ConfigurableProps>({
   const [
     sdkErrors,
     setSdkErrors,
-  ] = useState<SdkError[]>([])
+  ] = useState<SdkError[]>([]);
 
   const [
     enabledOptionalProps,
@@ -200,10 +200,17 @@ export const FormContextProvider = <T extends ConfigurableProps>({
       props = _configurableProps as unknown as T; // XXX
     }
     if (reloadPropIdx != null) {
-      props = props.slice(0, reloadPropIdx + 1) as unknown as T; // XXX
+      props = Array.isArray(props)
+        ? props.slice(0, reloadPropIdx + 1) as unknown as T // eslint-disable-line react/prop-types
+        : props; // XXX
     }
     return props;
-  }, [dynamicProps?.configurableProps, formProps.component.configurable_props, propNames, reloadPropIdx]);
+  }, [
+    dynamicProps?.configurableProps,
+    formProps.component.configurable_props,
+    propNames,
+    reloadPropIdx,
+  ]);
 
   // these validations are necessary because they might override PropInput for number case for instance
   // so can't rely on that base control form validation