@ductape/sdk 0.3.5 → 0.3.7

package/dist/index.mjs

@@ -0,0 +1,3 @@
+import sdk from './index.js';
+export default sdk.default;
+export * from './index.js';

package/dist/products/services/products.service.d.ts

@@ -586,6 +586,7 @@ export default class ProductsBuilderService implements IProductsBuilderService {
     fetchJobs(): Promise<Array<IProductJobs>>;
     fetchJob(tag: string): Promise<IProductJobs | null>;
     deleteJob(tag: string): Promise<void>;
+    private featureMatchesWrite;
     createFeature(data: Partial<IProductFeature>, throwErrorIfExists?: boolean): Promise<IProductFeature>;
     updateFeature(tag: string, data: Partial<IProductFeature>): Promise<void>;
     fetchFeatures(): Promise<Array<IProductFeature>>;

package/dist/products/services/products.service.js

@@ -2267,7 +2267,11 @@ class ProductsBuilderService {
                 component: enums_1.ProductComponents.VECTOR,
                 action: enums_1.RequestAction.UPDATE,
             }, this.getUserAccess());
-            return newAction;
+            const persisted = await this.fetchVectorAction(vectorTag, fullTag);
+            if (!persisted || !this.featureMatchesWrite(persisted, { template: newAction.template, parameters: newAction.parameters, type: newAction.type })) {
+                throw new Error(`Vector action ${fullTag} persistence differs from the submitted definition; do not execute it`);
+            }
+            return persisted;
         }
         catch (e) {
             throw e;
@@ -2304,7 +2308,11 @@ class ProductsBuilderService {
                 component: enums_1.ProductComponents.VECTOR,
                 action: enums_1.RequestAction.UPDATE,
             }, this.getUserAccess());
-            return updatedAction;
+            const persisted = await this.fetchVectorAction(vectorTag, updatedAction.tag);
+            if (!persisted || !this.featureMatchesWrite(persisted, { template: updatedAction.template, parameters: updatedAction.parameters, type: updatedAction.type })) {
+                throw new Error(`Vector action ${updatedAction.tag} persistence differs from the submitted definition; do not execute it`);
+            }
+            return persisted;
         }
         catch (e) {
             throw e;
@@ -5276,6 +5284,17 @@ class ProductsBuilderService {
         }
     }
     // ==================== WORKFLOW CRUD METHODS ====================
+    featureMatchesWrite(actual, expected) {
+        if (expected === undefined)
+            return true;
+        if (expected === null || typeof expected !== 'object')
+            return actual === expected;
+        if (!actual || typeof actual !== 'object')
+            return false;
+        if (Array.isArray(expected))
+            return Array.isArray(actual) && actual.length === expected.length && expected.every((value, index) => this.featureMatchesWrite(actual[index], value));
+        return Object.entries(expected).every(([key, value]) => this.featureMatchesWrite(actual[key], value));
+    }
     async createFeature(data, throwErrorIfExists = false) {
         var _a, _b, _c;
         try {
@@ -5307,8 +5326,16 @@ class ProductsBuilderService {
                     // the feature was there all along and this is that false negative resolving
                     // itself -- fall through to the read-back below instead of failing the caller.
                     const isDuplicate = createErr instanceof Error && /already exists|duplicate|in use/i.test(createErr.message);
-                    if (!isDuplicate)
-                        throw createErr;
+                    if (!isDuplicate) {
+                        let persisted;
+                        try {
+                            persisted = await this.fetchFeature(data.tag);
+                        }
+                        catch ( /* Preserve an unverified mutation outcome. */_d) { /* Preserve an unverified mutation outcome. */ }
+                        if (persisted && this.featureMatchesWrite(persisted, data))
+                            return persisted;
+                        throw Object.assign(new Error(`Feature ${data.tag} create could not be verified; inspect its catalogue entry before retrying`), { mutationState: 'unknown', featureTag: data.tag });
+                    }
                 }
                 // A successful mutation response is insufficient: execution and Workbench
                 // use the catalogue read API. Confirm that exact path can observe the write.
@@ -5368,7 +5395,23 @@ class ProductsBuilderService {
                     }
                 }
             }
-            await this.productApi.updateProduct(this.product_id, Object.assign(Object.assign({ _id }, data), { tag, component: enums_1.ProductComponents.FEATURE, action: enums_1.RequestAction.UPDATE }), this.getUserAccess());
+            let writeError;
+            try {
+                await this.productApi.updateProduct(this.product_id, Object.assign(Object.assign({ _id }, data), { tag, component: enums_1.ProductComponents.FEATURE, action: enums_1.RequestAction.UPDATE }), this.getUserAccess());
+            }
+            catch (error) {
+                writeError = error;
+            }
+            let persisted;
+            try {
+                persisted = await this.fetchFeature(tag);
+            }
+            catch ( /* Readback failed; do not replay the mutation. */_d) { /* Readback failed; do not replay the mutation. */ }
+            if (!persisted || !this.featureMatchesWrite(persisted, Object.assign(Object.assign({}, data), { tag }))) {
+                throw Object.assign(new Error(`Feature ${tag} update could not be verified; inspect its catalogue entry before retrying`), {
+                    mutationState: 'unknown', featureTag: tag, writeAcknowledged: !writeError,
+                });
+            }
         }
         catch (e) {
             throw e;