@checkstack/common 0.6.3 → 0.6.5

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @checkstack/common
 
+## 0.6.5
+
+### 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`
+
+## 0.6.4
+
+### Patch Changes
+
+- 67158e2: Standardize package metadata, unify AJV versions to 8.18.0, and enforce monorepo architecture rules via updated ESLint configuration. This ensures consistent package discovery and runtime dependency safety across the platform.
+
 ## 0.6.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/common",
-  "version": "0.6.3",
+  "version": "0.6.5",
   "type": "module",
   "main": "./src/index.ts",
   "types": "./src/index.ts",
@@ -12,18 +12,21 @@
     }
   },
   "dependencies": {
-    "@orpc/contract": "^1.5.0",
+    "@orpc/contract": "^1.13.14",
     "lucide-react": "0.562.0",
     "zod": "^4.0.0"
   },
   "devDependencies": {
     "typescript": "^5.7.2",
-    "@checkstack/tsconfig": "0.0.3",
-    "@checkstack/scripts": "0.1.1"
+    "@checkstack/tsconfig": "0.0.5",
+    "@checkstack/scripts": "0.1.2"
   },
   "scripts": {
     "typecheck": "tsc --noEmit",
     "lint": "bun run lint:code",
     "lint:code": "eslint . --max-warnings 0"
+  },
+  "checkstack": {
+    "type": "common"
   }
 }

package/src/error-utils.test.ts

@@ -0,0 +1,30 @@
+import { describe, it, expect } from "bun:test";
+import { extractErrorMessage } from "./error-utils";
+
+describe("extractErrorMessage", () => {
+  it("should extract message from Error instances", () => {
+    const error = new Error("something went wrong");
+    expect(extractErrorMessage(error)).toBe("something went wrong");
+  });
+
+  it("should extract message from Error subclasses", () => {
+    const error = new TypeError("invalid type");
+    expect(extractErrorMessage(error)).toBe("invalid type");
+  });
+
+  it("should return string errors directly", () => {
+    expect(extractErrorMessage("network failure")).toBe("network failure");
+  });
+
+  it("should return the default fallback for non-Error, non-string values", () => {
+    expect(extractErrorMessage(42)).toBe("An error occurred");
+    expect(extractErrorMessage(undefined)).toBe("An error occurred");
+    // eslint-disable-next-line unicorn/no-null
+    expect(extractErrorMessage(null)).toBe("An error occurred");
+    expect(extractErrorMessage({ code: 500 })).toBe("An error occurred");
+  });
+
+  it("should return custom fallback when provided", () => {
+    expect(extractErrorMessage(42, "Custom fallback")).toBe("Custom fallback");
+  });
+});

package/src/error-utils.ts

@@ -0,0 +1,16 @@
+/**
+ * Extracts a human-readable message from an unknown error value.
+ *
+ * Handles Error instances, plain strings, and arbitrary values.
+ * Designed as a single source of truth — use this instead of
+ * inline `error instanceof Error ? error.message : fallback` patterns.
+ */
+export function extractErrorMessage(
+  error: unknown,
+  fallback = "An error occurred",
+): string {
+  // eslint-disable-next-line no-restricted-syntax -- This IS the canonical implementation
+  if (error instanceof Error) return error.message;
+  if (typeof error === "string") return error;
+  return fallback;
+}

package/src/index.ts

@@ -9,3 +9,4 @@ export * from "./transport-client";
 export * from "./json-schema";
 export * from "./chart-types";
 export * from "./procedure-builder";
+export * from "./error-utils";