@checkstack/gitops-backend 0.2.0 → 0.2.2

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # @checkstack/gitops-backend
 
+## 0.2.2
+
+### 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.
+
+## 0.2.1
+
+### Patch Changes
+
+- 57d54de: Fix GitOps Healthcheck reconciliation engine and Kind Registry UI
+
+  - Mandated fully qualified IDs for all healthcheck strategies and collector definitions.
+  - Refactored the Kind Registry UI to display schema documentation in beautifully formatted, interactive YAML examples.
+  - Entity Envelope Fields and Base Spec Schema are now displayed in collapsed accordions.
+  - Fixed condition logic that broke the collector documentation display.
+  - Enhanced UX by dynamically injecting fully-qualified strategy variants directly into the YAML examples.
+
 ## 0.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/gitops-backend",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "type": "module",
   "main": "src/index.ts",
   "checkstack": {
@@ -14,7 +14,7 @@
   },
   "dependencies": {
     "@checkstack/backend-api": "0.12.0",
-    "@checkstack/gitops-common": "0.1.1",
+    "@checkstack/gitops-common": "0.2.0",
     "@checkstack/common": "0.6.5",
     "@checkstack/command-backend": "0.1.19",
     "@checkstack/queue-api": "0.2.13",

package/src/sync/reconciler.ts

@@ -308,7 +308,7 @@ async function reconcileEntity(params: {
   // Plugins use context.resolveSecretsBySchema() to resolve specific fields.
   const reconcileResult = await kindDef.reconcile({
     entity: entity as typeof entity & { spec: Record<string, unknown> },
-    existingEntityId: existing?.entityId ?? undefined,
+    existingEntityId: existing && !existing.entityId.startsWith("pending-") ? existing.entityId : undefined,
     context,
   });
 
@@ -381,7 +381,8 @@ async function detectOrphans(params: {
           kind: prov.kind,
         });
 
-        if (kindDef?.delete) {
+        // Do not call delete reconciler if it's a pending error record
+        if (kindDef?.delete && !prov.entityId.startsWith("pending-")) {
           try {
             await kindDef.delete({
               entityName: prov.entityName,
@@ -484,18 +485,17 @@ async function upsertProvenance(params: {
     return;
   }
 
-  // First-time error: no entityId available yet, skip provenance creation.
-  // The entity will be retried on the next sync cycle.
-  if (!entityId) {
-    return;
-  }
+  // First-time error: no entityId available yet because the entity failed to create.
+  // We MUST create a provenance record anyway so the error shows in the UI.
+  // We use a "pending-" prefix so the orphan detector knows not to call the delete reconciler.
+  const resolvedEntityId = entityId ?? `pending-${uuidv4()}`;
 
   await db.insert(schema.provenance).values({
     id: uuidv4(),
     apiVersion: entity.apiVersion,
     kind: entity.kind,
     entityName: entity.metadata.name,
-    entityId,
+    entityId: resolvedEntityId,
     providerId,
     repository: file.repository,
     filePath: file.filePath,