@adaptic/utils 0.0.946 → 0.0.948

package/dist/index.cjs

@@ -5837,75 +5837,90 @@ async function closePosition$1(auth, symbolOrAssetId, params) {
         // Crypto positions cannot use limit orders with SIP quotes or time_in_force="day".
         // Use direct DELETE (market order) for crypto regardless of useLimitOrder flag.
         if (useLimitOrder && !isCryptoSymbol(symbolOrAssetId)) {
-            const { position } = await fetchPosition(auth, symbolOrAssetId);
-            if (!position) {
-                throw new Error(`Position not found for ${symbolOrAssetId}`);
-            }
-            // Use the passed auth for quote fetching so multi-account setups
-            // use the correct credentials per account
-            const quotesResponse = await getLatestQuotes$1(auth, {
-                symbols: [symbolOrAssetId],
-            });
-            const quote = quotesResponse.quotes[symbolOrAssetId];
-            if (!quote) {
-                throw new Error(`No quote available for ${symbolOrAssetId}`);
-            }
-            let qty = Math.abs(parseFloat(position.qty));
-            if (params?.qty !== undefined) {
-                qty = params.qty;
-            }
-            else if (params?.percentage !== undefined) {
-                qty = Math.abs(parseFloat(position.qty)) * (params.percentage / 100);
+            // Attempt limit order closure; if quotes are unavailable (after-hours, IEX gaps),
+            // fall back to market order (DELETE) so the position still gets closed.
+            try {
+                const { position } = await fetchPosition(auth, symbolOrAssetId);
+                if (!position) {
+                    throw new Error(`Position not found for ${symbolOrAssetId}`);
+                }
+                // Use the passed auth for quote fetching so multi-account setups
+                // use the correct credentials per account
+                const quotesResponse = await getLatestQuotes$1(auth, {
+                    symbols: [symbolOrAssetId],
+                });
+                const quote = quotesResponse.quotes[symbolOrAssetId];
+                if (!quote) {
+                    throw new Error(`No quote available for ${symbolOrAssetId}`);
+                }
+                let qty = Math.abs(parseFloat(position.qty));
+                if (params?.qty !== undefined) {
+                    qty = params.qty;
+                }
+                else if (params?.percentage !== undefined) {
+                    qty = Math.abs(parseFloat(position.qty)) * (params.percentage / 100);
+                }
+                const side = position.side === "long" ? "sell" : "buy";
+                const positionIntent = side === "sell" ? "sell_to_close" : "buy_to_close";
+                const currentPrice = side === "sell" ? quote.bp : quote.ap;
+                if (!currentPrice) {
+                    throw new Error(`No valid price available for ${symbolOrAssetId}`);
+                }
+                const limitSlippage = slippagePercent1 / 100;
+                const limitPrice = side === "sell"
+                    ? roundPriceForAlpaca$5(currentPrice * (1 - limitSlippage))
+                    : roundPriceForAlpaca$5(currentPrice * (1 + limitSlippage));
+                getLogger().info(`Creating limit order to close ${symbolOrAssetId} position: ${side} ${qty} shares at ${limitPrice.toFixed(2)}`, {
+                    account: auth.adapticAccountId || "direct",
+                    symbol: symbolOrAssetId,
+                });
+                return await createLimitOrder(auth, {
+                    symbol: symbolOrAssetId,
+                    qty,
+                    side,
+                    limitPrice,
+                    position_intent: positionIntent,
+                    extended_hours: extendedHours,
+                });
             }
-            const side = position.side === "long" ? "sell" : "buy";
-            const positionIntent = side === "sell" ? "sell_to_close" : "buy_to_close";
-            const currentPrice = side === "sell" ? quote.bp : quote.ap;
-            if (!currentPrice) {
-                throw new Error(`No valid price available for ${symbolOrAssetId}`);
+            catch (limitOrderError) {
+                // Quote unavailable or invalid price — fall back to market order (DELETE)
+                // so the position still gets closed rather than leaving it open
+                const errMsg = limitOrderError instanceof Error ? limitOrderError.message : String(limitOrderError);
+                getLogger().warn(`Limit order closure failed for ${symbolOrAssetId} (${errMsg}), falling back to market order`, {
+                    account: auth.adapticAccountId || "direct",
+                    symbol: symbolOrAssetId,
+                    type: "warn",
+                });
+                // Fall through to the DELETE (market order) path below
             }
-            const limitSlippage = slippagePercent1 / 100;
-            const limitPrice = side === "sell"
-                ? roundPriceForAlpaca$5(currentPrice * (1 - limitSlippage))
-                : roundPriceForAlpaca$5(currentPrice * (1 + limitSlippage));
-            getLogger().info(`Creating limit order to close ${symbolOrAssetId} position: ${side} ${qty} shares at ${limitPrice.toFixed(2)}`, {
-                account: auth.adapticAccountId || "direct",
-                symbol: symbolOrAssetId,
-            });
-            return await createLimitOrder(auth, {
-                symbol: symbolOrAssetId,
-                qty,
-                side,
-                limitPrice,
-                position_intent: positionIntent,
-                extended_hours: extendedHours,
-            });
         }
-        else {
-            if (isCryptoSymbol(symbolOrAssetId)) {
-                getLogger().info(`Closing crypto position ${symbolOrAssetId} via market order (DELETE endpoint)`, { account: auth.adapticAccountId || "direct", symbol: symbolOrAssetId });
-            }
-            const queryParams = new URLSearchParams();
-            if (params?.qty !== undefined) {
-                queryParams.append("qty", params.qty.toString());
-            }
-            if (params?.percentage !== undefined) {
-                queryParams.append("percentage", params.percentage.toString());
-            }
-            const queryString = queryParams.toString();
-            const url = `${apiBaseUrl}/positions/${encodeURIComponent(symbolOrAssetId)}${queryString ? `?${queryString}` : ""}`;
-            const response = await fetch(url, {
-                method: "DELETE",
-                headers: {
-                    "APCA-API-KEY-ID": APIKey,
-                    "APCA-API-SECRET-KEY": APISecret,
-                },
-            });
-            if (!response.ok) {
-                const errorText = await response.text();
-                throw new Error(`Failed to close position: ${response.status} ${response.statusText} ${errorText}`);
-            }
-            return (await response.json());
+        // Market order (DELETE) path — used when limit orders are not requested,
+        // for crypto symbols, or as a fallback when limit order quotes are unavailable
+        if (isCryptoSymbol(symbolOrAssetId)) {
+            getLogger().info(`Closing crypto position ${symbolOrAssetId} via market order (DELETE endpoint)`, { account: auth.adapticAccountId || "direct", symbol: symbolOrAssetId });
+        }
+        const queryParams = new URLSearchParams();
+        if (params?.qty !== undefined) {
+            queryParams.append("qty", params.qty.toString());
         }
+        if (params?.percentage !== undefined) {
+            queryParams.append("percentage", params.percentage.toString());
+        }
+        const queryString = queryParams.toString();
+        const url = `${apiBaseUrl}/positions/${encodeURIComponent(symbolOrAssetId)}${queryString ? `?${queryString}` : ""}`;
+        const response = await fetch(url, {
+            method: "DELETE",
+            headers: {
+                "APCA-API-KEY-ID": APIKey,
+                "APCA-API-SECRET-KEY": APISecret,
+            },
+        });
+        if (!response.ok) {
+            const errorText = await response.text();
+            throw new Error(`Failed to close position: ${response.status} ${response.statusText} ${errorText}`);
+        }
+        return (await response.json());
     }
     catch (error) {
         getLogger().error("Error in closePosition:", error);