@checkstack/healthcheck-http-backend 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @checkstack/healthcheck-http-backend
 
+## 0.1.1
+
+### Patch Changes
+
+- 97c5a6b: Add UUID-based collector identification for better multiple collector support
+
+  **Breaking Change**: Existing health check configurations with collectors need to be recreated.
+
+  - Each collector instance now has a unique UUID assigned on creation
+  - Collector results are stored under the UUID key with `_collectorId` and `_assertionFailed` metadata
+  - Auto-charts correctly display separate charts for each collector instance
+  - Charts are now grouped by collector instance with clear headings
+  - Assertion status card shows pass/fail for each collector
+  - Renamed "Success" to "HTTP Success" to clarify it's about HTTP request success
+  - Fixed deletion of collectors not persisting to database
+  - Fixed duplicate React key warnings in auto-chart grid
+
+- Updated dependencies [97c5a6b]
+- Updated dependencies [8e43507]
+- Updated dependencies [97c5a6b]
+  - @checkstack/backend-api@0.2.0
+  - @checkstack/common@0.1.0
+  - @checkstack/healthcheck-common@0.2.0
+
 ## 0.1.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/healthcheck-http-backend",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "type": "module",
   "main": "src/index.ts",
   "scripts": {

package/src/request-collector.ts

@@ -64,7 +64,7 @@ const requestResultSchema = z.object({
   }),
   success: healthResultBoolean({
     "x-chart-type": "boolean",
-    "x-chart-label": "Success",
+    "x-chart-label": "HTTP Success",
   }),
 });
 

package/src/strategy.test.ts

@@ -181,16 +181,16 @@ describe("HttpHealthCheckStrategy", () => {
   });
 
   describe("aggregateResult", () => {
-    it("should calculate status code counts correctly", () => {
+    it("should count errors correctly", () => {
       const runs = [
         {
           id: "1",
-          status: "healthy" as const,
+          status: "unhealthy" as const,
           latencyMs: 100,
           checkId: "c1",
           timestamp: new Date(),
           metadata: {
-            statusCode: 200,
+            error: "Connection refused",
           },
         },
         {
@@ -199,40 +199,34 @@ describe("HttpHealthCheckStrategy", () => {
           latencyMs: 150,
           checkId: "c1",
           timestamp: new Date(),
-          metadata: {
-            statusCode: 200,
-          },
+          metadata: {},
         },
         {
           id: "3",
-          status: "healthy" as const,
+          status: "unhealthy" as const,
           latencyMs: 120,
           checkId: "c1",
           timestamp: new Date(),
           metadata: {
-            statusCode: 404,
+            error: "Timeout",
           },
         },
       ];
 
       const aggregated = strategy.aggregateResult(runs);
 
-      expect(aggregated.statusCodeCounts["200"]).toBe(2);
-      expect(aggregated.statusCodeCounts["404"]).toBe(1);
-      expect(aggregated.successRate).toBe(100);
+      expect(aggregated.errorCount).toBe(2);
     });
 
-    it("should count errors", () => {
+    it("should return zero errors when all runs succeed", () => {
       const runs = [
         {
           id: "1",
-          status: "unhealthy" as const,
+          status: "healthy" as const,
           latencyMs: 100,
           checkId: "c1",
           timestamp: new Date(),
-          metadata: {
-            error: "Connection refused",
-          },
+          metadata: {},
         },
         {
           id: "2",
@@ -240,17 +234,13 @@ describe("HttpHealthCheckStrategy", () => {
           latencyMs: 150,
           checkId: "c1",
           timestamp: new Date(),
-          metadata: {
-            statusCode: 200,
-            responseTime: 100,
-          },
+          metadata: {},
         },
       ];
 
       const aggregated = strategy.aggregateResult(runs);
 
-      expect(aggregated.errorCount).toBe(1);
-      expect(aggregated.successRate).toBe(50);
+      expect(aggregated.errorCount).toBe(0);
     });
   });
 });

package/src/strategy.ts

@@ -48,14 +48,6 @@ interface HttpConfigV2 extends HttpConfigV1 {
 
 /** Per-run result metadata */
 const httpResultMetadataSchema = z.object({
-  statusCode: healthResultNumber({
-    "x-chart-type": "pie",
-    "x-chart-label": "Status Code",
-  }).optional(),
-  contentType: healthResultString({
-    "x-chart-type": "counter",
-    "x-chart-label": "Content Type",
-  }).optional(),
   error: healthResultString({
     "x-chart-type": "status",
     "x-chart-label": "Error",
@@ -66,14 +58,6 @@ type HttpResultMetadata = z.infer<typeof httpResultMetadataSchema>;
 
 /** Aggregated metadata for buckets */
 const httpAggregatedMetadataSchema = z.object({
-  statusCodeCounts: z.record(z.string(), z.number()).meta({
-    "x-chart-type": "pie",
-    "x-chart-label": "Status Code Distribution",
-  }),
-  successRate: healthResultNumber({
-    "x-chart-type": "gauge",
-    "x-chart-label": "Success Rate (%)",
-  }),
   errorCount: healthResultNumber({
     "x-chart-type": "counter",
     "x-chart-label": "Errors",
@@ -137,26 +121,15 @@ export class HttpHealthCheckStrategy
   aggregateResult(
     runs: HealthCheckRunForAggregation<HttpResultMetadata>[]
   ): HttpAggregatedMetadata {
-    const statusCodeCounts: Record<string, number> = {};
     let errorCount = 0;
-    let successCount = 0;
 
     for (const run of runs) {
       if (run.metadata?.error) {
         errorCount++;
-      } else {
-        successCount++;
-      }
-
-      if (run.metadata?.statusCode !== undefined) {
-        const key = String(run.metadata.statusCode);
-        statusCodeCounts[key] = (statusCodeCounts[key] || 0) + 1;
       }
     }
-    const successRate =
-      runs.length > 0 ? Math.round((successCount / runs.length) * 100) : 0;
 
-    return { statusCodeCounts, successRate, errorCount };
+    return { errorCount };
   }
 
   /**