Files
momentum-system/runtime/flatten.py
T
momentum devandClaude Opus 4.8 9ab12bb3d6 Fix: open catch-up flatten wrongly flattened partially-filled fresh positions
Root cause of SSPC getting flattened on a mid-session restart (2026-06-24):
flatten_overnight_carries built its today's-buy set from orders with
status=="filled" only. But a marketable buy that PARTIALLY fills (SSPC: 84 of
88) cancels its unfilled remainder, so the order reads status=="canceled" even
though it opened 84 real shares. The carry-detector saw "no filled buy today"
for SSPC, tagged it an overnight carry, and market-closed a position opened ~6
min earlier. (Harmless this time — closed +$20 — but it could cut a runner or
realize a loss.) Marketable orders partial-fill far more than resting limits, so
flipping to marketable exposed this latent bug.

Fix: extract buy_fills_from_orders() (runtime/flatten.py) that counts a buy as a
real today-entry by filled_qty>0 + filled_at, NOT status=="filled" (scans legs
too). flatten_overnight_carries uses it. Pure + unit-tested: partial-fill counts,
full fill counts, zero-fill backout does not, and the partial-fill position is
no longer flagged a carry.

Note: only manifests on a mid-session restart (a clean session's EOD flatten
leaves the book flat, so the open finds no carries). Deploys on next restart.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-24 11:05:05 -04:00

141 lines
6.2 KiB
Python

"""Safely flatten a position that has resting protective orders.
THE PROBLEM this solves (learned the hard way, twice, in live trading):
a bracket's stop-loss / take-profit leg RESERVES the position's full quantity at
the broker (``held_for_orders``). So a plain ``close_position`` is rejected with
HTTP 403 ``"insufficient qty available for order (available: 0)"`` — the close
silently fails and the position is left sitting there, often unprotected once its
DAY stop expires. This bit us in two places independently:
1. the live exit manager's trail / first-red exits (positions never exited), and
2. the end-of-day flatten (the whole book was left naked overnight).
THE FIX, in one place: cancel the symbol's resting sell legs FIRST (which frees
the reserved shares), THEN market-close — with a few retries so a cancel that
hasn't settled at the broker yet doesn't abort the close. Both the exit manager
and the execution service (EOD flatten + circuit breaker) call this, so the fix
can never again drift out of sync between them.
"""
from __future__ import annotations
import logging
import time
from datetime import date, datetime, timezone
from zoneinfo import ZoneInfo
logger = logging.getLogger(__name__)
_ET = ZoneInfo("America/New_York")
def _fill_et_date(iso) -> date | None:
"""ET calendar date of a broker fill timestamp (stored UTC), or None."""
try:
dt = datetime.fromisoformat(str(iso).replace("Z", "+00:00"))
if dt.tzinfo is None:
dt = dt.replace(tzinfo=timezone.utc)
return dt.astimezone(_ET).date()
except (TypeError, ValueError):
return None
def find_overnight_carries(open_symbols, buy_fills, today_et: date) -> list[str]:
"""Open positions whose lot was NOT entered today (ET) — i.e. carried across
a prior session. A day-trading book should be flat by the open, so anything
still held that wasn't bought today survived overnight (the EOD flatten was
missed/failed) and should be caught up at the open before it bleeds for days
(the ATPC-over-Juneteenth case).
Args:
open_symbols: iterable of currently-open position symbols.
buy_fills: iterable of {"symbol", "filled_at"} for FILLED BUY orders.
today_et: today's ET date.
Returns sorted symbols open today with no buy filled today.
"""
bought_today: set[str] = set()
for f in buy_fills or []:
if _fill_et_date(f.get("filled_at")) == today_et:
sym = f.get("symbol")
if sym:
bought_today.add(sym)
return sorted(s for s in (open_symbols or ()) if s and s not in bought_today)
def buy_fills_from_orders(orders) -> list[dict]:
"""Extract {symbol, filled_at} for every BUY that actually FILLED shares —
keyed off ``filled_qty>0`` with a ``filled_at``, NOT ``status=='filled'``.
A marketable buy that PARTIALLY fills then cancels its unfilled remainder
reads ``status=='canceled'`` yet opened a real position; keying off status
would miss it, and the open catch-up flatten would then wrongly close that
fresh position as an overnight carry (the SSPC 84/88 case, 2026-06-24).
Scans bracket legs too.
"""
fills: list[dict] = []
for o in orders or []:
for x in [o, *(o.get("legs") or [])]:
try:
filled_qty = float(x.get("filled_qty") or 0)
except (TypeError, ValueError):
filled_qty = 0.0
if str(x.get("side")) == "buy" and x.get("filled_at") and filled_qty > 0:
fills.append({"symbol": x.get("symbol") or o.get("symbol"),
"filled_at": x.get("filled_at")})
return fills
# Order states in which a sell order still RESERVES position quantity. While an
# order is in one of these states, that many shares show as ``held_for_orders``
# and are unavailable to a separate close. (A ``filled``/``canceled`` leg no
# longer reserves anything, so we must not try to cancel it.)
RESERVING_STATES = frozenset({
"held", "new", "accepted", "pending_new",
"accepted_for_bidding", "partially_filled",
})
def cancel_protective_and_close(client, symbol: str, orders=None,
retries: int = 4) -> None:
"""Cancel ``symbol``'s resting protective sell orders, then market-close it.
Args:
client: broker client exposing ``cancel_order``, ``close_position`` and
(optionally) ``get_orders(status, nested, symbols)``.
symbol: the position to flatten.
orders: a pre-fetched *nested* order list to scan for legs to cancel. Pass
this when the caller already has a fresh snapshot (the exit manager
fetches one per pass) to avoid a redundant round-trip. If ``None`` and
the client supports it, this fetches the symbol's open orders itself.
retries: close attempts, spaced out so an in-flight cancel can settle.
Raises:
the last close exception if every attempt fails (callers collect, not
crash — one bad symbol must not block flattening the rest of the book).
"""
# 1) free the reserved shares: cancel the symbol's working sell legs
if orders is None and hasattr(client, "get_orders"):
try:
orders = client.get_orders(status="open", nested=True, symbols=[symbol])
except Exception: # noqa: BLE001
orders = None
for parent in (orders or []):
for o in [parent, *(parent.get("legs") or [])]:
sym = o.get("symbol") or parent.get("symbol")
if (sym == symbol and o.get("side") == "sell"
and o.get("status") in RESERVING_STATES and o.get("id")):
try:
client.cancel_order(o["id"])
except Exception as exc: # noqa: BLE001
logger.warning("cancel protective %s failed: %s", symbol, exc)
# 2) liquidate — retry so a not-yet-settled cancel doesn't leave it hanging
last_exc: Exception | None = None
for attempt in range(retries):
try:
client.close_position(symbol)
return
except Exception as exc: # noqa: BLE001
last_exc = exc
time.sleep(0.3 * (attempt + 1))
raise last_exc if last_exc else RuntimeError(f"flatten {symbol} failed")