@checkstack/healthcheck-backend 0.16.2 → 0.16.3

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @checkstack/healthcheck-backend
 
+## 0.16.3
+
+### Patch Changes
+
+- b53a40e: Fix GitOps entity update failures due to pending error records
+
+  - Ensured the `existingEntityId` parameter in the Reconciler engine is set to `undefined` instead of a `"pending-UUID"` when handling entities that failed to sync initially.
+  - Hardened the `Healthcheck` GitOps kind logic to explicitly ignore `"pending-"` IDs, preventing SQL update errors on synthetic provenance IDs.
+  - Fixed a bug where resolving YAML syntax errors would cause the subsequent sync to fail with `failed query: update [...]` because it attempted to update the nonexistent `"pending-"` entity instead of creating a new one.
+
+- Updated dependencies [b53a40e]
+  - @checkstack/gitops-backend@0.2.2
+  - @checkstack/catalog-backend@0.5.3
+  - @checkstack/satellite-backend@0.2.11
+
 ## 0.16.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/healthcheck-backend",
-  "version": "0.16.2",
+  "version": "0.16.3",
   "type": "module",
   "main": "src/index.ts",
   "checkstack": {
@@ -14,8 +14,8 @@
   },
   "dependencies": {
     "@checkstack/backend-api": "0.12.0",
-    "@checkstack/catalog-backend": "0.5.0",
-    "@checkstack/catalog-common": "1.4.0",
+    "@checkstack/catalog-backend": "0.5.1",
+    "@checkstack/catalog-common": "1.4.1",
     "@checkstack/command-backend": "0.1.19",
     "@checkstack/common": "0.6.5",
     "@checkstack/gitops-backend": "0.2.0",
@@ -25,7 +25,7 @@
     "@checkstack/integration-backend": "0.1.19",
     "@checkstack/maintenance-common": "0.4.9",
     "@checkstack/queue-api": "0.2.13",
-    "@checkstack/satellite-backend": "0.2.8",
+    "@checkstack/satellite-backend": "0.2.9",
     "@checkstack/signal-common": "0.1.9",
     "@hono/zod-validator": "^0.7.6",
     "drizzle-orm": "^0.45.0",

package/src/healthcheck-gitops-kinds.test.ts

@@ -259,6 +259,36 @@ describe("Healthcheck GitOps Kind: Healthcheck", () => {
     expect(mockService.configs[0].strategyId).toBe("postgres");
   });
 
+  it("creates a new configuration when existingEntityId is a pending error record", async () => {
+    const kind = buildKind();
+
+    const result = await kind.reconcile({
+      entity: {
+        apiVersion: CHECKSTACK_API_VERSION,
+        kind: "Healthcheck",
+        metadata: { name: "payment-db-check" },
+        spec: {
+          strategy: "postgres",
+          intervalSeconds: 30,
+          config: {
+            host: "db.internal",
+            port: 5432,
+            database: "payments",
+            user: "monitor",
+            password: "secret",
+          },
+        },
+      },
+      existingEntityId: "pending-12345",
+      context: mockContext,
+    });
+
+    expect(result.entityId).toBe("hc-1");
+    expect(mockService.createConfiguration).toHaveBeenCalledTimes(1);
+    expect(mockService.updateConfiguration).not.toHaveBeenCalled();
+    expect(mockService.configs).toHaveLength(1);
+  });
+
   it("updates an existing configuration using existingEntityId", async () => {
     const kind = buildKind();
 

package/src/healthcheck-gitops-kinds.ts

@@ -196,7 +196,7 @@ export function buildHealthcheckKind(
       // Create or update configuration
       const displayName = entity.metadata.title ?? entity.metadata.name;
 
-      if (existingEntityId) {
+      if (existingEntityId && !existingEntityId.startsWith("pending-")) {
         await service.updateConfiguration(existingEntityId, {
           name: displayName,
           strategyId: spec.strategy,