@checkstack/healthcheck-http-backend 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @checkstack/healthcheck-http-backend
 
+## 0.3.1
+
+### Patch Changes
+
+- 869b4ab: ## Health Check Execution Improvements
+
+  ### Breaking Changes (backend-api)
+
+  - `HealthCheckStrategy.createClient()` now accepts `unknown` instead of `TConfig` due to TypeScript contravariance constraints. Implementations should use `this.config.validate(config)` to narrow the type.
+
+  ### Features
+
+  - **Platform-level hard timeout**: The executor now wraps the entire health check execution (connection + all collectors) in a single timeout, ensuring checks never hang indefinitely.
+  - **Parallel collector execution**: Collectors now run in parallel using `Promise.allSettled()`, improving performance while ensuring all collectors complete regardless of individual failures.
+  - **Base strategy config schema**: All strategy configs now extend `baseStrategyConfigSchema` which provides a standardized `timeout` field with sensible defaults (30s, min 100ms).
+
+  ### Fixes
+
+  - Fixed HTTP and Jenkins strategies clearing timeouts before reading the full response body.
+  - Simplified registry type signatures by using default type parameters.
+
+- Updated dependencies [869b4ab]
+  - @checkstack/backend-api@0.8.0
+
 ## 0.3.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/healthcheck-http-backend",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "type": "module",
   "main": "src/index.ts",
   "scripts": {
@@ -9,10 +9,10 @@
     "lint:code": "eslint . --max-warnings 0"
   },
   "dependencies": {
-    "@checkstack/backend-api": "0.5.2",
-    "@checkstack/healthcheck-common": "0.8.1",
+    "@checkstack/backend-api": "0.7.0",
+    "@checkstack/healthcheck-common": "0.8.2",
     "jsonpath-plus": "^10.3.0",
-    "@checkstack/common": "0.6.1"
+    "@checkstack/common": "0.6.2"
   },
   "devDependencies": {
     "@types/bun": "^1.0.0",

package/src/strategy.ts

@@ -8,6 +8,7 @@ import {
   z,
   type InferAggregatedResult,
   type ConnectedClient,
+  baseStrategyConfigSchema,
 } from "@checkstack/backend-api";
 import {
   healthResultString,
@@ -27,14 +28,7 @@ import type {
  * HTTP health check configuration schema.
  * Global defaults only - action params moved to RequestCollector.
  */
-export const httpHealthCheckConfigSchema = z.object({
-  timeout: z
-    .number()
-    .int()
-    .min(100)
-    .default(30_000)
-    .describe("Default request timeout in milliseconds"),
-});
+export const httpHealthCheckConfigSchema = baseStrategyConfigSchema.extend({});
 
 export type HttpHealthCheckConfig = z.infer<typeof httpHealthCheckConfigSchema>;
 
@@ -154,9 +148,10 @@ export class HttpHealthCheckStrategy implements HealthCheckStrategy<
             signal: controller.signal,
           });
 
-          clearTimeout(timeoutId);
-
+          // Read body BEFORE clearing timeout - body streaming can also hang
           const body = await response.text();
+
+          clearTimeout(timeoutId);
           const headers: Record<string, string> = {};
 
           // eslint-disable-next-line unicorn/no-array-for-each