-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Enable jit orders for CoW AMMs #2607
Comments
The list for those can probably be hardcoded initially (and should be similar to the list of addresses that are exempt from protocol fees). Also, those orders require to execute a pre interaction (to commit the AMM to a unique order id and prevent replay attacks). JIT orders and high level solutions currently don't have a way of specifying pre/post interactions. This should be added as well (not entirely sure if on the JIT order level or solution struct, my hunch would be the latter for more flexibility). |
While not strictly required, it would probably be good to complete #2215 before implementing this issue as otherwise we have to add the logic to encode clearing prices for JIT orders that accrue surplus in the legacy encoder (which may be a bit hairy) |
This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed. |
Last bit for submitting cow amm JIT orders was implemented in #2764 |
In order to realize the potential of CoW AMMs, solvers need to be able to post jit orders to the CoW AMM. A special property of these orders is that they are eligible for surplus, that should be accounted for when computing the score of a solution. This requires that the autopilot and the default driver are aware that these orders can accumulate surplus.
Copy-pasting an important observation from the first comment below.
The text was updated successfully, but these errors were encountered: