@checkstack/ui 1.2.0 → 1.2.1

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @checkstack/ui
 
+## 1.2.1
+
+### Patch Changes
+
+- d1a2796: Enforce stricter code quality standards and eliminate AI slop anti-patterns.
+
+  **New utility**
+
+  - `extractErrorMessage(error, fallback?)` in `@checkstack/common` for consistent error extraction
+
+  **ESLint rules**
+
+  - `react-hooks/rules-of-hooks` and `exhaustive-deps` for hook correctness
+  - `no-console` in frontend packages — forces `toast` over silent `console.error`
+  - `no-restricted-syntax` banning `instanceof Error` — forces `extractErrorMessage`
+  - Custom `no-eslint-disable-any` rule preventing `@typescript-eslint/no-explicit-any` circumvention
+
+  **Refactoring**
+
+  - Replace 141 `instanceof Error` boilerplate patterns across the codebase
+  - Replace swallowed `console.error` with user-visible `toast.error()` feedback
+  - Remove 15 redundant `as` type casts in IntegrationsPage and ProviderConnectionsPage
+  - Consolidate 3 identical callback handlers into `handleDialogClose`
+  - Fix conditional React hook call in `FormField.tsx`
+  - Fix unstable useMemo deps in `Dashboard.tsx`
+  - Replace `useEffect`→`setState` with derived `useMemo` in `RegisterPage.tsx`
+  - Rewrite `keystore.test.ts` with typed `DrizzleMockChain` (eliminating 7 `any` suppressions)
+  - Delete obvious comments in `encryption.ts` and Teams `provider.ts`
+
+- Updated dependencies [d1a2796]
+  - @checkstack/common@0.6.5
+  - @checkstack/frontend-api@0.3.9
+
 ## 1.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/ui",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "type": "module",
   "main": "src/index.ts",
   "dependencies": {
@@ -31,7 +31,7 @@
     "@types/react": "^18.2.0",
     "@testing-library/react": "^16.0.0",
     "@checkstack/test-utils-frontend": "0.0.4",
-    "@checkstack/tsconfig": "0.0.4",
+    "@checkstack/tsconfig": "0.0.5",
     "@checkstack/scripts": "0.1.2"
   },
   "scripts": {

package/src/components/DynamicForm/DynamicForm.tsx

@@ -33,7 +33,8 @@ export const DynamicForm: React.FC<DynamicFormProps> = ({
     if (JSON.stringify(merged) !== JSON.stringify(value)) {
       onChange(merged);
     }
-  }, [schema]); // Only run when schema changes
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- Intentional: runs only on schema change. Including onChange would re-fire on parent re-renders; including value would cause an infinite loop since this effect calls onChange(merged)
+  }, [schema]);
 
   // Compute validity and report changes
   React.useEffect(() => {

package/src/components/DynamicForm/DynamicOptionsField.tsx

@@ -13,6 +13,7 @@ import {
 
 import type { DynamicOptionsFieldProps, ResolverOption } from "./types";
 import { getCleanDescription, NONE_SENTINEL } from "./utils";
+import { extractErrorMessage } from "@checkstack/common";
 
 /**
  * Field component for dynamically resolved options.
@@ -72,7 +73,7 @@ export const DynamicOptionsField: React.FC<DynamicOptionsFieldProps> = ({
       .catch((error_) => {
         if (!cancelled) {
           setError(
-            error_ instanceof Error ? error_.message : "Failed to load options",
+            extractErrorMessage(error_, "Failed to load options"),
           );
           setLoading(false);
         }

package/src/components/DynamicForm/FormField.tsx

@@ -1,4 +1,4 @@
-import React from "react";
+import { useEffect, useState } from "react";
 import { Plus, Trash2 } from "lucide-react";
 
 import {
@@ -37,6 +37,14 @@ export const FormField: React.FC<FormFieldProps> = ({
 }) => {
   const description = propSchema.description || "";
 
+  // Const field handling - must be before any early returns (rules-of-hooks)
+  const isConstField = propSchema.const !== undefined;
+  useEffect(() => {
+    if (isConstField && value !== propSchema.const) {
+      onChange(propSchema.const);
+    }
+  }, [isConstField, value, propSchema.const, onChange]);
+
   // Dynamic options via resolver
   const resolverName = propSchema["x-options-resolver"];
   if (resolverName && optionsResolvers) {
@@ -57,14 +65,7 @@ export const FormField: React.FC<FormFieldProps> = ({
     );
   }
 
-  // Const field handling - auto-set value and hide (value is fixed)
-  if (propSchema.const !== undefined) {
-    // Silently ensure the value is set, no UI needed
-    React.useEffect(() => {
-      if (value !== propSchema.const) {
-        onChange(propSchema.const);
-      }
-    }, [value, propSchema.const, onChange]);
+  if (isConstField) {
     return <></>;
   }
 
@@ -653,7 +654,7 @@ const SecretField: React.FC<{
   isRequired?: boolean;
   onChange: (val: unknown) => void;
 }> = ({ id, label, description, value, isRequired, onChange }) => {
-  const [showPassword, setShowPassword] = React.useState(false);
+  const [showPassword, setShowPassword] = useState(false);
   const currentValue = value || "";
   const hasExistingValue = currentValue.length > 0;
 
@@ -702,7 +703,7 @@ const SecretTextareaField: React.FC<{
   isRequired?: boolean;
   onChange: (val: unknown) => void;
 }> = ({ id, label, description, value, isRequired, onChange }) => {
-  const [showContent, setShowContent] = React.useState(false);
+  const [showContent, setShowContent] = useState(false);
   const currentValue = value || "";
   const hasExistingValue = currentValue.length > 0;
 
@@ -723,7 +724,9 @@ const SecretTextareaField: React.FC<{
             value={currentValue}
             onChange={(e) => onChange(e.target.value)}
             placeholder={
-              hasExistingValue ? "Leave empty to keep existing value" : "Paste content here"
+              hasExistingValue
+                ? "Leave empty to keep existing value"
+                : "Paste content here"
             }
             rows={5}
             className="pr-10 font-mono text-xs"
@@ -750,7 +753,9 @@ const SecretTextareaField: React.FC<{
               }
             }}
             placeholder={
-              hasExistingValue ? "Leave empty to keep existing value" : "Paste content here"
+              hasExistingValue
+                ? "Leave empty to keep existing value"
+                : "Paste content here"
             }
             rows={3}
             className="pr-10"