@checkstack/notification-backstage-backend 0.1.21 → 0.1.22

package/CHANGELOG.md

@@ -1,5 +1,39 @@
 # @checkstack/notification-backstage-backend
 
+## 0.1.22
+
+### 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/notification-backend@0.1.21
+
 ## 0.1.21
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/notification-backstage-backend",
-  "version": "0.1.21",
+  "version": "0.1.22",
   "type": "module",
   "main": "src/index.ts",
   "checkstack": {
@@ -13,13 +13,13 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@checkstack/backend-api": "0.10.0",
-    "@checkstack/notification-backend": "0.1.18",
+    "@checkstack/backend-api": "0.11.0",
+    "@checkstack/notification-backend": "0.1.20",
     "@checkstack/common": "0.6.4",
     "zod": "^4.2.1"
   },
   "devDependencies": {
-    "@checkstack/tsconfig": "0.0.4",
+    "@checkstack/tsconfig": "0.0.5",
     "typescript": "^5.9.3"
   }
 }

package/src/index.ts

@@ -8,6 +8,7 @@ import {
 import { notificationStrategyExtensionPoint } from "@checkstack/notification-backend";
 import { z } from "zod";
 import { pluginMetadata } from "./plugin-metadata";
+import { extractErrorMessage } from "@checkstack/common";
 
 // ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 // Admin Configuration Schema
@@ -259,11 +260,11 @@ const backstageStrategy: NotificationStrategy<
       };
     } catch (error) {
       logger?.error?.("Failed to send notification to Backstage", {
-        error: error instanceof Error ? error.message : String(error),
+        error: extractErrorMessage(error),
       });
       return {
         success: false,
-        error: error instanceof Error ? error.message : String(error),
+        error: extractErrorMessage(error),
       };
     }
   },