@metamask-previews/perps-controller 8.0.0-preview-51b287c55 → 8.1.0-preview-3af52b79c

package/dist/services/TradingService.cjs

@@ -74,6 +74,8 @@ class TradingService {
         const traceId = (0, uuid_1.v4)();
         const startTime = __classPrivateFieldGet(this, _TradingService_deps, "f").performance.now();
         let traceData;
+        let orderSubmissionThresholdTimeoutId;
+        let didExceedOrderSubmissionThreshold = false;
         const paymentToken = params.trackingData?.tradeWithToken === true
             ? (params.trackingData.mmPayTokenSelected ?? 'unknown_token')
             : 'perps_balance';
@@ -122,16 +124,47 @@ class TradingService {
                 hasTP: Boolean(params.takeProfitPrice),
                 hasSL: Boolean(params.stopLossPrice),
             });
-            // Execute order with fee discount management
+            // Observational threshold: when the provider round-trip exceeds
+            // PlaceOrderTimeoutMs we tag the trace and emit a breadcrumb, but we
+            // intentionally do NOT cancel the in-flight order. Cancelling client-side
+            // (e.g. Promise.race rejection) does not stop the provider request, so a
+            // race-based timeout would let the UI mark an order as failed while
+            // HyperLiquid could still accept it. Instead, we always await
+            // provider.placeOrder(params) to terminal completion and surface
+            // late completions via trace `reason: 'late_success' | 'late_error'`.
+            orderSubmissionThresholdTimeoutId = setTimeout(() => {
+                didExceedOrderSubmissionThreshold = true;
+                __classPrivateFieldGet(this, _TradingService_deps, "f").tracer.addBreadcrumb({
+                    category: 'perps',
+                    message: 'Order submission exceeded threshold (still pending)',
+                    level: 'warning',
+                    data: {
+                        thresholdMs: perpsConfig_1.PERPS_CONSTANTS.PlaceOrderTimeoutMs,
+                        payment_token: paymentToken,
+                        market: params.symbol,
+                        orderType: params.orderType,
+                    },
+                });
+                __classPrivateFieldGet(this, _TradingService_deps, "f").debugLogger.log('TradingService: Order submission exceeded threshold (still pending)', {
+                    thresholdMs: perpsConfig_1.PERPS_CONSTANTS.PlaceOrderTimeoutMs,
+                    symbol: params.symbol,
+                    orderType: params.orderType,
+                });
+            }, perpsConfig_1.PERPS_CONSTANTS.PlaceOrderTimeoutMs);
             const result = await __classPrivateFieldGet(this, _TradingService_instances, "m", _TradingService_withFeeDiscount).call(this, {
                 provider,
                 feeDiscountBips,
                 operation: () => provider.placeOrder(params),
             });
+            if (orderSubmissionThresholdTimeoutId !== undefined) {
+                clearTimeout(orderSubmissionThresholdTimeoutId);
+                orderSubmissionThresholdTimeoutId = undefined;
+            }
             __classPrivateFieldGet(this, _TradingService_deps, "f").debugLogger.log('TradingService: Provider response received', {
                 success: result.success,
                 orderId: result.orderId,
                 error: result.error,
+                didExceedOrderSubmissionThreshold,
             });
             // Update state and handle success/failure
             const completionDuration = __classPrivateFieldGet(this, _TradingService_deps, "f").performance.now() - startTime;
@@ -142,13 +175,23 @@ class TradingService {
                     context,
                     reportOrderToDataLake,
                 });
-                traceData = { success: true, orderId: result.orderId ?? '' };
+                traceData = {
+                    success: true,
+                    orderId: result.orderId ?? '',
+                    ...(didExceedOrderSubmissionThreshold
+                        ? { reason: 'late_success' }
+                        : {}),
+                };
                 // Invalidate standalone caches so external hooks (e.g., usePerpsPositionForAsset) refresh
                 __classPrivateFieldGet(this, _TradingService_deps, "f").cacheInvalidator.invalidate({ cacheType: 'positions' });
                 __classPrivateFieldGet(this, _TradingService_deps, "f").cacheInvalidator.invalidate({ cacheType: 'accountState' });
             }
             else {
-                traceData = { success: false, error: result.error ?? 'Unknown error' };
+                traceData = {
+                    success: false,
+                    reason: didExceedOrderSubmissionThreshold ? 'late_error' : 'error',
+                    error: result.error ?? 'Unknown error',
+                };
             }
             // Track analytics (success or failure)
             __classPrivateFieldGet(this, _TradingService_instances, "m", _TradingService_trackOrderResult).call(this, {
@@ -187,11 +230,15 @@ class TradingService {
             });
             traceData = {
                 success: false,
+                reason: didExceedOrderSubmissionThreshold ? 'late_error' : 'error',
                 error: error instanceof Error ? error.message : 'Unknown error',
             };
             throw error;
         }
         finally {
+            if (orderSubmissionThresholdTimeoutId !== undefined) {
+                clearTimeout(orderSubmissionThresholdTimeoutId);
+            }
             // Always end trace on exit (success or failure)
             __classPrivateFieldGet(this, _TradingService_deps, "f").tracer.endTrace({
                 name: types_1.PerpsTraceNames.PlaceOrder,