Skip to content
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: Report and store all executed orders during bidding #3004

Closed
fhenneke opened this issue Sep 18, 2024 · 3 comments · Fixed by #3048
Closed

feat: Report and store all executed orders during bidding #3004

fhenneke opened this issue Sep 18, 2024 · 3 comments · Fixed by #3048
Assignees
Labels
E:6.2 Time to Happy Moo See https://github.com/cowprotocol/pm/issues/77 for details

Comments

@fhenneke
Copy link

Problem

At the moment, a solver can include orders in a settlement at will if the order is settled at limit price. Those orders need not be reported during bidding.

This is generally a problem for protecting users but will also be problematic with most forms of combinatorial auction or payments with multiple winners. For example, a solver might execute an additional order on a token pair having no tokens in common with the reported trades. A second winner might now execute on that same token pair.

Suggested solution

A first step towards creating more rigorous rules around settling orders out of competition would be to force solvers to report all orders which are executed to the autopilot.

Additional context

There is a general problem with executing orders which can capture surplus at limit price.

The feature of executing jit orders at limit price without reporting the order is only used in obscure cases. For example for CoW AMM JIT orders with missing native prices. The impact of removing the feature should be small.

@sunce86 sunce86 added the E:6.2 Time to Happy Moo See https://github.com/cowprotocol/pm/issues/77 for details label Sep 18, 2024
@sunce86
Copy link
Contributor

sunce86 commented Sep 18, 2024

I assume that for these orders we don't have to require existence of native prices for tokens they swap?

@fleupold
Copy link
Contributor

The reporting is already covered by #3001 isn't it? Now we also need to store those (even if they don't up landed)

@sunce86
Copy link
Contributor

sunce86 commented Sep 24, 2024

The reporting is already covered by #3001 isn't it? Now we also need to store those (even if they don't up landed)

No, #3001 only expanded the response for orders that were already reported (regular user orders and surplus capturing JIT orders). Driver still do not report non surplus capturing JIT orders. I also think this needs to be communicated to solvers in advance.

@fleupold can you move the issue status to To Do if you agree we should work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E:6.2 Time to Happy Moo See https://github.com/cowprotocol/pm/issues/77 for details
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants