linting

Author: jsendo

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

@@ -1,6 +1,9 @@
 import { FormContext } from "../hooks/form-context";
 import { FormFieldContext } from "../hooks/form-field-context";
-import { ConfigurableProp, ConfigurableProps } from "@pipedream/sdk";
+import {
+  ConfigurableProp,
+  ConfigurableProps,
+} from "@pipedream/sdk";
 // import { ControlAny } from "./ControlAny"
 import { ControlApp } from "./ControlApp";
 import { ControlBoolean } from "./ControlBoolean";

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

@@ -3,7 +3,10 @@ import type {
   CSSProperties, FormEventHandler,
 } from "react";
 import { useCustomize } from "../hooks/customization-context";
-import { useFormContext, skippablePropTypes } from "../hooks/form-context";
+import {
+  useFormContext,
+  skippablePropTypes,
+} from "../hooks/form-context";
 import { InternalField } from "./InternalField";
 import { Alert } from "./Alert";
 import { ErrorBoundary } from "./ErrorBoundary";

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

@@ -34,7 +34,11 @@ export type FormContext<T extends ConfigurableProps> = {
   userId: string;
 };
 
-export const skippablePropTypes = ["$.service.db", "$.interface.http", "$.interface.apphook"]
+export const skippablePropTypes = [
+  "$.service.db",
+  "$.interface.http",
+  "$.interface.apphook",
+]
 
 export const FormContext = createContext<FormContext<any /* XXX fix */> | undefined>(undefined); // eslint-disable-line @typescript-eslint/no-explicit-any
 
@@ -171,8 +175,7 @@ export const FormContextProvider = <T extends ConfigurableProps>({
   const propErrors = <T extends ConfigurableProps>(prop: ConfigurableProp, value: unknown): string[] => {
     const errs: string[] = [];
     if (value === undefined) {
-      if (prop.optional || prop.hidden || skippablePropTypes.includes(prop.type)) {
-      } else {
+      if (!prop.optional && !prop.hidden && !skippablePropTypes.includes(prop.type)) {
         errs.push("required");
       }
     } else if (prop.type === "integer") { // XXX type should be "number"? we don't support floats otherwise...