Skip to content

fix(torghut): align breakout exit trace - #12427

Closed
gregkonush wants to merge 1 commit into
mainfrom
codex/fix-torghut-breakout-exit-review-20260714
Closed

fix(torghut): align breakout exit trace#12427
gregkonush wants to merge 1 commit into
mainfrom
codex/fix-torghut-breakout-exit-review-20260714

Conversation

@gregkonush

Copy link
Copy Markdown
Member

Summary

  • Uses the shared OR-semantics exit-trigger gate for breakout continuation exits.
  • Records breakout failure, loss of opening-range structure, and session-strength reversal as the actual independent sell triggers.
  • Adds trace coverage proving a confirmed breakout failure produces a passing exit gate.

Related Issues

Addresses historical Codex review: #4815 (comment)

Testing

  • Python 3.12 pytest -q tests/strategy_runtime/test_runtime_breakout_b.py — 13 passed.
  • Pyright pyrightconfig.json — 0 errors, 0 warnings.
  • Pyright pyrightconfig.alpha.json — 0 errors, 0 warnings.
  • Pyright pyrightconfig.scripts.json — 0 errors, 0 warnings.
  • Ruff format and lint on changed files — passed.
  • uv sync --frozen --extra dev remains blocked in agents-shell by the missing standard /lib64 loader; validation used the repository’s existing synced Python 3.12 environment through the pinned Nix loader.

Breaking Changes

None.

Checklist

  • Testing section documents exact validation and the local toolchain limitation.
  • Screenshots are not applicable; Breaking Changes is filled in.
  • No documentation or release-note update is required for this trace-semantics correction.

@gregkonush gregkonush closed this Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant