@checkstack/dependency-frontend 0.2.2 → 0.2.4

package/CHANGELOG.md

@@ -1,5 +1,55 @@
 # @checkstack/dependency-frontend
 
+## 0.2.4
+
+### Patch Changes
+
+- Updated dependencies [26d8bae]
+- Updated dependencies [26d8bae]
+  - @checkstack/ui@1.3.0
+  - @checkstack/healthcheck-common@0.11.0
+  - @checkstack/dashboard-frontend@0.3.27
+
+## 0.2.3
+
+### 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]
+- Updated dependencies [3c34b07]
+  - @checkstack/common@0.6.5
+  - @checkstack/ui@1.2.1
+  - @checkstack/dashboard-frontend@0.3.26
+  - @checkstack/frontend-api@0.3.9
+  - @checkstack/catalog-common@1.3.1
+  - @checkstack/healthcheck-common@0.10.1
+  - @checkstack/dependency-common@0.2.1
+  - @checkstack/signal-frontend@0.0.15
+
 ## 0.2.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/dependency-frontend",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "type": "module",
   "main": "src/index.tsx",
   "checkstack": {
@@ -12,14 +12,14 @@
     "lint:code": "eslint . --max-warnings 0"
   },
   "dependencies": {
-    "@checkstack/catalog-common": "1.2.11",
-    "@checkstack/common": "0.6.4",
-    "@checkstack/dashboard-frontend": "0.3.23",
-    "@checkstack/dependency-common": "0.1.0",
-    "@checkstack/frontend-api": "0.3.8",
-    "@checkstack/healthcheck-common": "0.8.4",
-    "@checkstack/signal-frontend": "0.0.14",
-    "@checkstack/ui": "1.2.0",
+    "@checkstack/catalog-common": "1.3.1",
+    "@checkstack/common": "0.6.5",
+    "@checkstack/dashboard-frontend": "0.3.26",
+    "@checkstack/dependency-common": "0.2.1",
+    "@checkstack/frontend-api": "0.3.9",
+    "@checkstack/healthcheck-common": "0.10.1",
+    "@checkstack/signal-frontend": "0.0.15",
+    "@checkstack/ui": "1.2.1",
     "@xyflow/react": "^12.10.2",
     "lucide-react": "^0.344.0",
     "react": "^18.2.0",
@@ -28,7 +28,7 @@
   "devDependencies": {
     "typescript": "^5.0.0",
     "@types/react": "^18.2.0",
-    "@checkstack/tsconfig": "0.0.4",
+    "@checkstack/tsconfig": "0.0.5",
     "@checkstack/scripts": "0.1.2"
   }
 }

package/src/components/DependencyMapPage.tsx

@@ -46,6 +46,7 @@ import {
   type DependencyEdge,
   type DependencyEdgeData,
 } from "./canvas/DependencyEdge";
+import { extractErrorMessage } from "@checkstack/common";
 
 const nodeTypes = { system: SystemNodeComponent };
 const edgeTypes = { dependency: DependencyEdgeComponent };
@@ -171,7 +172,7 @@ function DependencyMapContent() {
     },
     onError: (error) => {
       const message =
-        error instanceof Error ? error.message : "Failed to create dependency";
+        extractErrorMessage(error, "Failed to create dependency");
 
       // Check for cycle error and resolve names
       if (message.includes("circular chain")) {
@@ -193,7 +194,7 @@ function DependencyMapContent() {
       void refetchWarnings();
     },
     onError: (error) => {
-      toast.error(error instanceof Error ? error.message : "Failed to update");
+      toast.error(extractErrorMessage(error, "Failed to update"));
     },
   });
 
@@ -206,7 +207,7 @@ function DependencyMapContent() {
       void refetchWarnings();
     },
     onError: (error) => {
-      toast.error(error instanceof Error ? error.message : "Failed to delete");
+      toast.error(extractErrorMessage(error, "Failed to delete"));
     },
   });
 
@@ -350,6 +351,7 @@ function DependencyMapContent() {
         }, 2000);
       }
     },
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- handleSave reads from nodesRef (always current); including it would cause infinite re-render loops via setHasUnsaved
     [onNodesChange],
   );