@checkstack/healthcheck-jenkins-backend 0.3.0 → 0.3.2

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @checkstack/healthcheck-jenkins-backend
 
+## 0.3.2
+
+### Patch Changes
+
+- Updated dependencies [0ebbe56]
+  - @checkstack/backend-api@0.8.1
+  - @checkstack/common@0.6.3
+  - @checkstack/healthcheck-common@0.8.3
+
+## 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-jenkins-backend",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "type": "module",
   "main": "src/index.ts",
   "scripts": {
@@ -10,9 +10,9 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@checkstack/backend-api": "0.5.2",
-    "@checkstack/common": "0.6.1",
-    "@checkstack/healthcheck-common": "0.8.1"
+    "@checkstack/backend-api": "0.8.0",
+    "@checkstack/common": "0.6.2",
+    "@checkstack/healthcheck-common": "0.8.2"
   },
   "devDependencies": {
     "@types/bun": "^1.0.0",

package/src/strategy.ts

@@ -11,9 +11,9 @@ import {
   mergeCounter,
   z,
   configString,
-  configNumber,
   type ConnectedClient,
   type InferAggregatedResult,
+  baseStrategyConfigSchema,
 } from "@checkstack/backend-api";
 import {
   healthResultNumber,
@@ -35,7 +35,7 @@ import type {
  * Jenkins health check configuration schema.
  * Provides connectivity settings for the Jenkins API.
  */
-export const jenkinsConfigSchema = z.object({
+export const jenkinsConfigSchema = baseStrategyConfigSchema.extend({
   baseUrl: z
     .string()
     .url()
@@ -46,11 +46,6 @@ export const jenkinsConfigSchema = z.object({
   apiToken: configString({ "x-secret": true }).describe(
     "Jenkins API token (generate from User > Configure > API Token)",
   ),
-  timeout: configNumber({})
-    .int()
-    .min(1000)
-    .default(30_000)
-    .describe("Request timeout in milliseconds"),
 });
 
 export type JenkinsConfig = z.infer<typeof jenkinsConfigSchema>;
@@ -164,12 +159,11 @@ export class JenkinsHealthCheckStrategy implements HealthCheckStrategy<
             signal: controller.signal,
           });
 
-          clearTimeout(timeoutId);
-
           // Get Jenkins version from header
           const jenkinsVersion = response.headers.get("X-Jenkins") || undefined;
 
           if (!response.ok) {
+            clearTimeout(timeoutId);
             return {
               statusCode: response.status,
               data: undefined,
@@ -178,8 +172,11 @@ export class JenkinsHealthCheckStrategy implements HealthCheckStrategy<
             };
           }
 
+          // Read body BEFORE clearing timeout - body streaming can also hang
           const data = await response.json();
 
+          clearTimeout(timeoutId);
+
           return {
             statusCode: response.status,
             data,