@ductape/sdk 0.1.149 → 0.1.151

package/dist/features/feature-executor.js

@@ -13,7 +13,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 var _a, _b, _c, _d;
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.FeatureExecutor = exports.UnresolvedInputReferenceError = void 0;
+exports.FeatureExecutor = exports.assertSuccessfulStepOutput = exports.UnresolvedInputReferenceError = void 0;
 const crypto_1 = require("crypto");
 const processor_service_1 = __importDefault(require("../processor/services/processor.service"));
 const products_service_1 = __importDefault(require("../products/services/products.service"));
@@ -110,6 +110,22 @@ const stringifyProcessorValue = (value) => {
         return nested;
     });
 };
+/** Convert explicit SDK failure envelopes into thrown step failures. */
+const assertSuccessfulStepOutput = (output, stepTag) => {
+    var _a, _b;
+    if (!output || typeof output !== 'object' || Array.isArray(output))
+        return;
+    const result = output;
+    const status = typeof result.status === 'string' ? result.status.toLowerCase() : '';
+    const failedStatus = ['fail', 'failed', 'failure', 'rolled_back', 'cancelled', 'canceled'].includes(status);
+    const explicitFailure = result.failed_execution === true || failedStatus;
+    const unsuccessfulWithError = (result.success === false || result.ok === false) && result.error != null;
+    if (!explicitFailure && !unsuccessfulWithError)
+        return;
+    const detail = (_b = (_a = result.error) !== null && _a !== void 0 ? _a : result.message) !== null && _b !== void 0 ? _b : `returned status '${status || 'failed'}'`;
+    throw new Error(`Step '${stepTag}' failed: ${String(detail)}`);
+};
+exports.assertSuccessfulStepOutput = assertSuccessfulStepOutput;
 /** Normalize error to a string and extract HTTP details when present (e.g. Axios) */
 function formatErrorDetails(error) {
     var _a, _b, _c, _d;
@@ -927,6 +943,7 @@ class FeatureExecutor {
                 return value;
             };
             let output = asset ? await (0, context_1.withAssetSpan)(asset, runOperation) : await runOperation();
+            (0, exports.assertSuccessfulStepOutput)(output, step.tag);
             // For action steps with no app: if step.output is a template with $ refs (e.g. $Sequence{main}{step-a}{value}-b),
             // resolve it so downstream steps get the correct value (checkpoint / closure-style steps).
             if (step.type === types_1.FeatureStepType.ACTION &&
@@ -1072,7 +1089,7 @@ class FeatureExecutor {
                 const action = ((_b = step.event) === null || _b === void 0 ? void 0 : _b.startsWith(`${database}:`))
                     ? step.event.slice(database.length + 1)
                     : step.event;
-                return db.execute({
+                const result = await db.execute({
                     product,
                     env,
                     database,
@@ -1080,6 +1097,17 @@ class FeatureExecutor {
                     input: input,
                     session: this.inheritedSession,
                 });
+                if (result &&
+                    typeof result === 'object' &&
+                    'action' in result &&
+                    'resolvedTemplate' in result &&
+                    ('message' in result
+                        ? /actual execution|processorservice|direct adapter/i.test(String(result.message))
+                        : false)) {
+                    throw new Error(`Database action '${database}:${action}' returned an execution preview instead of a database result. ` +
+                        'Upgrade @ductape/sdk and restart the application process; previews cannot be accepted as successful Feature steps.');
+                }
+                return result;
         }
     }
     /**
@@ -1245,7 +1273,7 @@ class FeatureExecutor {
      * Execute a vector step (query, upsert, deleteVectors, or custom action via execute).
      */
     async executeVectorStep(step, input) {
-        var _a, _b, _c, _d, _e;
+        var _a, _b, _c;
         const vectorTag = step.vector;
         if (!vectorTag) {
             throw new Error(`Vector step "${step.tag}" has no vector tag`);
@@ -1279,8 +1307,7 @@ class FeatureExecutor {
                 return result;
             }
             default: {
-                // Custom vector action: resolve template then execute (resolve returns options for query/upsert/delete)
-                const resolved = await svc.actions.resolve({
+                return svc.executeAction({
                     product,
                     env,
                     vector: vectorTag,
@@ -1288,18 +1315,6 @@ class FeatureExecutor {
                     input: input,
                     session: this.inheritedSession,
                 });
-                const op = (_d = resolved === null || resolved === void 0 ? void 0 : resolved._operation) !== null && _d !== void 0 ? _d : resolved === null || resolved === void 0 ? void 0 : resolved.operation;
-                const opts = Object.assign(Object.assign({ product, env, tag: vectorTag }, resolved), { session: (_e = resolved.session) !== null && _e !== void 0 ? _e : this.inheritedSession });
-                if (op === 'query' || !op) {
-                    return svc.query(opts);
-                }
-                if (op === 'upsert') {
-                    return svc.upsert(opts);
-                }
-                if (op === 'delete' || op === 'deleteVectors') {
-                    return svc.deleteVectors(opts);
-                }
-                return svc.query(opts);
             }
         }
     }
@@ -1706,7 +1721,7 @@ class FeatureExecutor {
      * Execute a rollback handler
      */
     async executeRollbackHandler(rollback, stepOutput) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k;
         if (!rollback)
             return;
         debugLog('[FeatureExecutor] executeRollbackHandler', { feature_id: this.state.feature_id, rollback_type: rollback.type });
@@ -1798,7 +1813,7 @@ class FeatureExecutor {
                     await svc.deleteVectors(Object.assign(Object.assign({ product, env, tag: vectorTag }, inp), { session: (_k = inp.session) !== null && _k !== void 0 ? _k : this.inheritedSession }));
                 }
                 else {
-                    const resolved = await svc.actions.resolve({
+                    await svc.executeAction({
                         product,
                         env,
                         vector: vectorTag,
@@ -1806,14 +1821,6 @@ class FeatureExecutor {
                         input: inp,
                         session: this.inheritedSession,
                     });
-                    const op = (_l = resolved === null || resolved === void 0 ? void 0 : resolved._operation) !== null && _l !== void 0 ? _l : resolved === null || resolved === void 0 ? void 0 : resolved.operation;
-                    const opts = Object.assign(Object.assign({ product, env, tag: vectorTag }, resolved), { session: (_m = resolved.session) !== null && _m !== void 0 ? _m : this.inheritedSession });
-                    if (op === 'upsert')
-                        await svc.upsert(opts);
-                    else if (op === 'delete' || op === 'deleteVectors')
-                        await svc.deleteVectors(opts);
-                    else
-                        await svc.query(opts);
                 }
                 break;
             }