Skip to content

fix: allow pinned COOF market-entry limit recross - #199

Merged
luisleo526 merged 1 commit into
mainfrom
round15/joat-limit-recross
Sep 6, 2026
Merged

fix: allow pinned COOF market-entry limit recross#199
luisleo526 merged 1 commit into
mainfrom
round15/joat-limit-recross

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

Allow a full long limit exit armed by a calc-on-order-fills market entry at the first high to fill when price recrosses its level later in the bar. Whole-entry-bar suppression previously delayed these exits until the next bar. The exception is tied to the actual opening fill and its first callback; existing path, stop and competing-order guards remain in force.

Validation on engine 86049406d8f675d7ad237f519d1a023ae21f6096 with codegen 3fd97fe28abd7b191cb376d9f5db4e056fcfae4b:

  • Cloud Run completed 72/72 cases and measured all 4,190 probes. Ford and EURUSD JOAT aureate move from strong to excellent: count differences fall from 5 and 6 to zero, with 100% match and zero canonical P90 errors. All other 4,188 probes retain their canonical metrics. Zero regressions, engine errors, coverage gaps or new moderate results.
  • Gate pineforge-pr-gate-8bcpn: PASS against baseline dd98f02ed492f64ae8b48358117c83530f88ae74, target score +2, zero regressions or tolerated exceptions across 704 hard probes. Result: 4,156 excellent / 33 strong / 1 moderate.
  • Fresh all-target build, 209/209 C++ unit tests, C ABI and freshness checks passed. The focused fixture passes 227 checks; 51 scope checks also pass on the unchanged parent runtime. Independent Grok 4.6 review of this exact candidate is GREEN, P0/P1/P2 = 0.

The rule is limited to the pinned historical, single-lot long market-entry case. Grading metrics, thresholds, tolerances, verifier, harness, tapes and population are unchanged.

Candidate snapshot: 1544c156154fccd4789afa50ffc7db900c49e34f63d36fdbc6dc213cb31661bd (788,854 bytes).

@luisleo526
luisleo526 merged commit 8604940 into main Sep 6, 2026
8 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