Skip to content

Fix deferred close-all and frozen reversal lifecycle#102

Merged
luisleo526 merged 2 commits into
mainfrom
codex/pyr1-two-bug-unmasking
Jul 15, 2026
Merged

Fix deferred close-all and frozen reversal lifecycle#102
luisleo526 merged 2 commits into
mainfrom
codex/pyr1-two-bug-unmasking

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

Summary

  • preserve an admitted prior-bar same-ID, same-side pure STOP across the exact deferred close_all cleanup by binding it to a fresh pending-order incarnation and the actual flattening order
  • bind the priced explicit-FIXED frozen reversal transaction exception to a nonzero position cycle so replacement and fills cannot cross position instances
  • add load-bearing ordinary, COOF, magnifier, replacement, RAW/ANY, and double-flip controls

Root cause

The corpus fixture's scalar count gap hid two independent bug sets: six TradingView-only STOP carries were deleted by close_all cleanup, while seven engine-only priced reversals crossed position cycles. On the accepted cumulative stack, A0B0 is 6 missing + 7 extra, A1B0 is 0 + 7, A0B1 is 6 + 0, and A1B1 is exact 2356/2356.

Upstream main intentionally still reports one unrelated engine-only trade because accepted predecessor 7638461 has not landed upstream; this PR does not fold that separate change into scope.

Validation

  • build success
  • CTest 99/99
  • focused A 45/45; B 35/35; PR fix: enforce paired market affordability #101 overlap 178/178; focused CTest 10/10
  • codegen 1537 passed / 2 skipped
  • corpus 252/252 (240 Excellent / 11 Strong / 1 Anomaly on the current upstream stack)
  • final fresh-context R7 PASS; diff-check clean

@luisleo526 luisleo526 merged commit 37caa8f into main Jul 15, 2026
5 checks passed
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