Skip to content

Comments

remove same slot matching restriction#2104

Merged
crispheaney merged 3 commits intomasterfrom
ahzam/remove-same-slot-matching-restriction
Feb 9, 2026
Merged

remove same slot matching restriction#2104
crispheaney merged 3 commits intomasterfrom
ahzam/remove-same-slot-matching-restriction

Conversation

@0xahzam
Copy link
Contributor

@0xahzam 0xahzam commented Feb 7, 2026

Issue

Matching engine blocks maker/taker fills when both placed in same slot, forcing market makers to wait and retry.

Changes

  • Removed slot equality check in is_maker_for_taker
  • Self match protection already upstream via maker_key != taker_key check
  • Added tests for same slot scenarios

@0xahzam 0xahzam requested a review from moosecat2 February 7, 2026 10:06
@0xahzam 0xahzam changed the title remove same slot matching restriction for swift limit fills remove same slot matching restriction Feb 7, 2026
@0xahzam 0xahzam removed the request for review from moosecat2 February 9, 2026 17:37
@crispheaney crispheaney merged commit 4d4e5a4 into master Feb 9, 2026
8 of 9 checks passed
lowkeynicc added a commit that referenced this pull request Feb 9, 2026
* program: add auto cancel reduce only tpsl

* cargo test

* cargo fmt --

* tweaks

* program: rm should_expire_order_before_fill (#2103)

* fix conditional

* program: remove same slot matching restriction (#2104)

* remove same slot restriction for swift limit fills

* update comment

* add changelog

---------

Co-authored-by: Chris Heaney <chrisheaney30@gmail.com>
Co-authored-by: 0xahzam <104062587+0xahzam@users.noreply.github.com>
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.

2 participants