@checkstack/healthcheck-script-backend 0.3.6 → 0.3.8

package/CHANGELOG.md

@@ -1,5 +1,62 @@
 # @checkstack/healthcheck-script-backend
 
+## 0.3.8
+
+### 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/backend-api@0.11.1
+  - @checkstack/healthcheck-common@0.10.1
+
+## 0.3.7
+
+### Patch Changes
+
+- 54a5f80: ### Health Check Editor Redesign — IDE-Style Experience
+
+  Replaces the modal-based health check editor with a full-page, IDE-style experience:
+
+  - **Strategy Picker Page**: New `/config/create` page with categorized strategy discovery, search filtering, and grouped card grid layout
+  - **IDE Editor Page**: New `/config/:configId/edit` page with a split-view layout — explorer tree on the left, editor panel on the right
+  - **Strategy Categories**: Introduces `StrategyCategory` enum with 16 categories (Networking, Database, Infrastructure, etc.) — all 13 strategy plugins now declare their category
+  - **New RPC Endpoint**: Added `getConfiguration` (singular by ID) for efficient single-resource fetching on the edit page
+  - **Explorer Tree**: Left-hand navigation with General, Check Items (collectors), and Access Control sections, with real-time validation indicators
+  - **Validation Status Bar**: Bottom bar showing aggregated validation issues with clickable navigation
+  - **Unsaved Changes Guard**: Browser `beforeunload` protection when the form is dirty
+  - **Responsive Design**: Split-view on desktop, stacked layout on mobile
+  - **Deleted**: Legacy `HealthCheckEditor.tsx` modal component
+
+- Updated dependencies [54a5f80]
+  - @checkstack/healthcheck-common@0.10.0
+  - @checkstack/backend-api@0.11.0
+
 ## 0.3.6
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/healthcheck-script-backend",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "type": "module",
   "main": "src/index.ts",
   "checkstack": {
@@ -12,14 +12,14 @@
     "lint:code": "eslint . --max-warnings 0"
   },
   "dependencies": {
-    "@checkstack/backend-api": "0.10.0",
+    "@checkstack/backend-api": "0.11.0",
     "@checkstack/common": "0.6.4",
-    "@checkstack/healthcheck-common": "0.8.4"
+    "@checkstack/healthcheck-common": "0.10.0"
   },
   "devDependencies": {
     "@types/bun": "^1.0.0",
     "typescript": "^5.0.0",
-    "@checkstack/tsconfig": "0.0.4",
+    "@checkstack/tsconfig": "0.0.5",
     "@checkstack/scripts": "0.1.2"
   }
 }

package/src/inline-script-collector.ts

@@ -21,6 +21,7 @@ import {
 } from "@checkstack/healthcheck-common";
 import { pluginMetadata } from "./plugin-metadata";
 import type { ScriptTransportClient } from "./transport-client";
+import { extractErrorMessage } from "@checkstack/common";
 
 // ============================================================================
 // SCRIPT EXECUTION UTILITIES (shared with integration-script-backend pattern)
@@ -164,7 +165,7 @@ async function executeInlineScript({
       timedOut: false,
     };
   } catch (error) {
-    const message = error instanceof Error ? error.message : String(error);
+    const message = extractErrorMessage(error);
     if (message === "__TIMEOUT__") {
       return {
         result: undefined,

package/src/strategy.ts

@@ -20,12 +20,14 @@ import {
   healthResultNumber,
   healthResultString,
   healthResultSchema,
+  StrategyCategory,
 } from "@checkstack/healthcheck-common";
 import type {
   ScriptTransportClient,
   ScriptRequest,
   ScriptResult as ScriptResultType,
 } from "./transport-client";
+import { extractErrorMessage } from "@checkstack/common";
 
 // ============================================================================
 // SCHEMAS
@@ -214,6 +216,7 @@ export class ScriptHealthCheckStrategy implements HealthCheckStrategy<
   id = "script";
   displayName = "Script Health Check";
   description = "Execute local scripts or commands for health checking";
+  category = StrategyCategory.INFRASTRUCTURE;
 
   private executor: ScriptExecutor;
 
@@ -303,7 +306,7 @@ export class ScriptHealthCheckStrategy implements HealthCheckStrategy<
             stdout: "",
             stderr: "",
             timedOut: false,
-            error: error instanceof Error ? error.message : String(error),
+            error: extractErrorMessage(error),
           };
         }
       },