@ductape/sdk 0.2.6 → 0.2.8

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

@@ -317,6 +317,7 @@ export default class ProcessorService implements IProcessorService {
         data?: undefined;
     }>;
     processPricingCost(additional_logs: Partial<ILogData>): Promise<IPricingResult>;
+    private processPricingCostEventually;
     addToSuccessOutput(event: IStepEvent, output: any, additional_logs: Partial<ILogData>): Promise<void>;
     checkIsSuccessful(): boolean;
     addToFailureOutput(e: any, event: IStepEvent, payload: IRetryMeta | any, additional_logs: Partial<ILogData>, policy?: Partial<IAppRetryPolicy>): Promise<IProcessingFailure>;
@@ -384,9 +385,9 @@ export default class ProcessorService implements IProcessorService {
     runBrokerPublish(data: IStepEvent, additional_logs?: Partial<ILogData>): Promise<{
         published: boolean;
     }>;
-    processStorageRequest(data: IStepEvent, input: IStorageRequest, storageEnv: IProductStorageEnvs, additional_logs: Partial<ILogData>, decryptionKey?: string): Promise<{
+    processStorageRequest(data: IStepEvent, input: IStorageRequest, storageEnv: IProductStorageEnvs, additional_logs: Partial<ILogData>, decryptionKey?: string): Promise<IProcessingFailure | {
         url: string;
-    } | IProcessingFailure>;
+    }>;
     writeResult(status: LogEventStatus, retryable?: boolean): Promise<void>;
     /**
      * Separate credentials into prefixed (e.g., 'headers:Authorization') and non-prefixed (e.g., 'api_key').

package/dist/processor/services/processor.service.js

@@ -2956,13 +2956,12 @@ class ProcessorService {
             const end = Date.now();
             this.requestTime += end - start;
             this.totalRequests += 1;
-            const pricing = await this.processPricingCost(Object.assign(Object.assign({}, additional_logs), { app_id, workspace_id: this.workspace_id }));
+            // Pricing/request accounting can touch Redis (including lock acquisition and
+            // a 5s command timeout). It is telemetry/accounting and must not hold a
+            // completed provider response on the business request path.
+            void this.processPricingCostEventually(Object.assign(Object.assign({}, additional_logs), { app_id, workspace_id: this.workspace_id }), event.app).then(() => this.logService.publish()).catch(() => { });
             const log = Object.assign(Object.assign(Object.assign({}, this.baseLogs), additional_logs), { message: 'Process http request - success', successful_execution: true, data: { response: results }, status: types_1.LogEventStatus.SUCCESS, app_id, action: event.event, start,
                 end });
-            if (pricing) {
-                const { pricing_tag, pricing_cost, is_overage, currency } = pricing;
-                Object.assign(log, { pricing_tag, pricing_cost, is_overage, currency });
-            }
             this.logService.add(log);
             await this.addToSuccessOutput(event, results, additional_logs);
             if (returnValue) {
@@ -3069,6 +3068,16 @@ class ProcessorService {
             return null;
         }
     }
+    async processPricingCostEventually(additional_logs, accessTag) {
+        try {
+            this.pricingService.fetchPricing();
+            this.pricingService.fetchOveragePricing();
+        }
+        catch (_a) {
+            await this.initializePricing(additional_logs, accessTag);
+        }
+        return this.processPricingCost(additional_logs);
+    }
     async addToSuccessOutput(event, output, additional_logs) {
         // Remove event from failed, skipped, and waiting arrays
         this.processingOutput.failure = this.processingOutput.failure.filter((data) => !(data.event.sequence_tag === event.sequence_tag && data.event.event === event.event));
@@ -3082,7 +3091,8 @@ class ProcessorService {
             this.logService.add(Object.assign(Object.assign(Object.assign({}, this.baseLogs), additional_logs), { successful_execution: true, message: 'Process event - success', data: {}, status: types_1.LogEventStatus.SUCCESS }));
             void this.logService.publish().catch(() => { });
             this.end = Date.now();
-            await this.writeResult(types_1.LogEventStatus.SUCCESS);
+            // Invocation persistence is telemetry and must not extend provider latency.
+            void this.writeResult(types_1.LogEventStatus.SUCCESS).catch(() => { });
         }
     }
     checkIsSuccessful() {
@@ -5031,7 +5041,8 @@ class ProcessorService {
             }
             this.end = Date.now();
             this.logService.add(Object.assign(Object.assign(Object.assign({}, this.baseLogs), additional_logs), { message: 'Execute action - success', data: { input, result }, status: types_1.LogEventStatus.SUCCESS }));
-            await this.writeResult(types_1.LogEventStatus.SUCCESS);
+            // Invocation persistence is telemetry and must not extend provider latency.
+            void this.writeResult(types_1.LogEventStatus.SUCCESS).catch(() => { });
             // The action result is authoritative; telemetry delivery must not add the
             // Logs API timeout to caller latency.
             void this.logService.publish().catch(() => { });
@@ -5042,7 +5053,8 @@ class ProcessorService {
                 this.logService.add(Object.assign(Object.assign(Object.assign({}, this.baseLogs), additional_logs), { message: 'Execute action - failed', data: { e: e.toString() }, status: types_1.LogEventStatus.FAIL }));
                 this.end = Date.now();
                 if (!(e === null || e === void 0 ? void 0 : e.processorFailureRecorded)) {
-                    await this.writeResult(types_1.LogEventStatus.FAIL);
+                    // Preserve the caller-visible provider/input failure immediately; analytics are eventual.
+                    void this.writeResult(types_1.LogEventStatus.FAIL).catch(() => { });
                 }
                 // Preserve and return the provider error immediately. LogsService keeps
                 // unsent entries queued when delivery fails.