@bpmsoftwaresolutions/ai-engine-client 1.1.15 → 1.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bpmsoftwaresolutions/ai-engine-client",
-  "version": "1.1.15",
+  "version": "1.1.16",
   "description": "Thin npm client for the AI Engine operator and retrieval APIs",
   "type": "module",
   "main": "./src/index.js",

package/src/index.js

@@ -1164,9 +1164,40 @@ export class AIEngineClient {
   }
 
   async updateImplementationItemStatus(implementationItemId, body) {
-    return this._request(`/api/governed-implementation/items/${implementationItemId}/status`, {
+    const result = await this._request(`/api/governed-implementation/items/${implementationItemId}/status`, {
       method: 'PATCH', body,
     });
+    const expectedStatus = cleanText(body?.status);
+    const authoritativeItem = isPlainObject(result?.implementation_item) ? result.implementation_item : null;
+    const actualStatus = cleanText(authoritativeItem?.status);
+    if (expectedStatus && !authoritativeItem) {
+      throw buildVerificationError(
+        `updateImplementationItemStatus did not return authoritative item state for ${implementationItemId}.`,
+        {
+          mutation_name: 'updateImplementationItemStatus',
+          mutation_attempted: true,
+          authoritative_read_performed: false,
+          post_condition_verified: false,
+          expected_state: { status: expectedStatus },
+          mutation_result: result ?? null,
+        },
+      );
+    }
+    if (expectedStatus && actualStatus !== expectedStatus) {
+      throw buildVerificationError(
+        `updateImplementationItemStatus returned ${actualStatus ?? 'unknown'} instead of ${expectedStatus}.`,
+        {
+          mutation_name: 'updateImplementationItemStatus',
+          mutation_attempted: true,
+          authoritative_read_performed: true,
+          post_condition_verified: false,
+          expected_state: { status: expectedStatus },
+          verified_current_state: authoritativeItem ?? result ?? null,
+          mutation_result: result ?? null,
+        },
+      );
+    }
+    return result;
   }
 
   async addImplementationItemEvidence(implementationItemId, body) {
@@ -1316,15 +1347,17 @@ export class AIEngineClient {
   }
 
   async updateImplementationItemStatusVerified(implementationItemId, status, body = {}) {
-    const workflowId = cleanText(body.workflowId) || cleanText(body.workflow_id);
-    if (!workflowId) {
-      throw new Error('workflowId is required to verify implementation item status.');
-    }
     return this.executeVerifiedMutation({
       mutationName: 'updateImplementationItemStatus',
       evidenceLabel: `implementation-item-status:${implementationItemId}`,
       mutationFn: () => this.updateImplementationItemStatus(implementationItemId, { ...body, status }),
-      verificationFn: async () => {
+      verificationFn: async (mutationResult) => {
+        const workflowId = cleanText(body.workflowId)
+          || cleanText(body.workflow_id)
+          || cleanText(mutationResult?.workflow_id);
+        if (!workflowId) {
+          throw new Error('workflowId is required to verify implementation item status.');
+        }
         const roadmap = await this.getWorkflowImplementationRoadmap(workflowId);
         const items = Array.isArray(roadmap?.items) ? roadmap.items : [];
         const matchedItem = items.find((item) => item?.implementation_item_id === implementationItemId) || null;