@checkstack/integration-backend 0.1.17 → 0.1.19

package/CHANGELOG.md

@@ -1,5 +1,51 @@
 # @checkstack/integration-backend
 
+## 0.1.19
+
+### Patch Changes
+
+- Updated dependencies [26d8bae]
+  - @checkstack/backend-api@0.12.0
+  - @checkstack/command-backend@0.1.19
+  - @checkstack/queue-api@0.2.13
+
+## 0.1.18
+
+### 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/backend-api@0.11.1
+  - @checkstack/command-backend@0.1.18
+  - @checkstack/integration-common@0.2.8
+  - @checkstack/signal-common@0.1.9
+  - @checkstack/queue-api@0.2.12
+
 ## 0.1.17
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/integration-backend",
-  "version": "0.1.17",
+  "version": "0.1.19",
   "type": "module",
   "main": "src/index.ts",
   "checkstack": {
@@ -14,12 +14,12 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@checkstack/integration-common": "0.2.7",
-    "@checkstack/backend-api": "0.10.0",
-    "@checkstack/signal-common": "0.1.8",
-    "@checkstack/queue-api": "0.2.9",
-    "@checkstack/common": "0.6.4",
-    "@checkstack/command-backend": "0.1.15",
+    "@checkstack/integration-common": "0.2.8",
+    "@checkstack/backend-api": "0.11.1",
+    "@checkstack/signal-common": "0.1.9",
+    "@checkstack/queue-api": "0.2.12",
+    "@checkstack/common": "0.6.5",
+    "@checkstack/command-backend": "0.1.18",
     "drizzle-orm": "^0.45.0",
     "zod": "^4.2.1",
     "@orpc/server": "^1.13.2"
@@ -27,8 +27,8 @@
   "devDependencies": {
     "@checkstack/drizzle-helper": "0.0.4",
     "@checkstack/scripts": "0.1.2",
-    "@checkstack/tsconfig": "0.0.4",
-    "@checkstack/test-utils-backend": "0.1.15",
+    "@checkstack/tsconfig": "0.0.5",
+    "@checkstack/test-utils-backend": "0.1.18",
     "@types/node": "^20.0.0",
     "typescript": "^5.0.0"
   }

package/src/delivery-coordinator.ts

@@ -8,6 +8,7 @@ import type { IntegrationProviderRegistry } from "./provider-registry";
 import type { ConnectionStore } from "./connection-store";
 import * as schema from "./schema";
 import { INTEGRATION_DELIVERY_COMPLETED } from "@checkstack/integration-common";
+import { extractErrorMessage } from "@checkstack/common";
 
 /**
  * Event payload for delivery routing
@@ -201,7 +202,7 @@ export function createDeliveryCoordinator(
       }
     } catch (error) {
       const errorMessage =
-        error instanceof Error ? error.message : String(error);
+        extractErrorMessage(error);
 
       // Get current attempt count
       const [log] = await db

package/src/hook-subscriber.ts

@@ -5,6 +5,7 @@ import type {
 import type { IntegrationEventRegistry } from "./event-registry";
 import type { DeliveryCoordinator } from "./delivery-coordinator";
 import type { RegisteredIntegrationEvent } from "./provider-types";
+import { extractErrorMessage } from "@checkstack/common";
 
 /**
  * Hook subscription function type (matches env.onHook signature)
@@ -84,7 +85,7 @@ function subscribeToEvent(
       } catch (error) {
         logger.error(
           `Failed to route integration event ${event.eventId}:`,
-          error instanceof Error ? error.message : String(error)
+          extractErrorMessage(error)
         );
         // Don't re-throw - we don't want to fail the entire hook chain
         // The event will be logged but not retried at the hook level

package/src/router.ts

@@ -17,6 +17,7 @@ import {
   integrationContract,
   INTEGRATION_SUBSCRIPTION_CHANGED,
 } from "@checkstack/integration-common";
+import { extractErrorMessage } from "@checkstack/common";
 
 /**
  * Recursively extracts flattened property paths from a JSON Schema.
@@ -430,7 +431,7 @@ export function createIntegrationRouter(deps: RouterDeps) {
         } catch (error) {
           return {
             success: false,
-            message: error instanceof Error ? error.message : String(error),
+            message: extractErrorMessage(error),
           };
         }
       }
@@ -544,7 +545,7 @@ export function createIntegrationRouter(deps: RouterDeps) {
       } catch (error) {
         throw new ORPCError("NOT_FOUND", {
           message:
-            error instanceof Error ? error.message : "Connection not found",
+            extractErrorMessage(error, "Connection not found"),
         });
       }
     }),
@@ -590,7 +591,7 @@ export function createIntegrationRouter(deps: RouterDeps) {
       } catch (error) {
         return {
           success: false,
-          message: error instanceof Error ? error.message : String(error),
+          message: extractErrorMessage(error),
         };
       }
     }),
@@ -632,7 +633,7 @@ export function createIntegrationRouter(deps: RouterDeps) {
         logger.error(`Failed to get connection options: ${error}`);
         throw new ORPCError("INTERNAL_SERVER_ERROR", {
           message:
-            error instanceof Error ? error.message : "Failed to fetch options",
+            extractErrorMessage(error, "Failed to fetch options"),
         });
       }
     }),