search

cowprotocol / cow-amm

An automated market maker running on top of CoW Protocol
17 stars 5 forks source link

Abridged cow amm #84

Closed fedgiac closed 4 months ago

fedgiac commented 4 months ago

(Fixed version of #79, keeping history so that the review from @mfw78 isn't wasted, see here for a diff since the last review.)

Necessary changes to the standalone CoW AMM for back-end testing purposes:

  1. it removes any need for tradingParams (and at the same time gets rid of the default order) and
  2. makes the contracts conform to the testing interface that will be supported by the Balancer AMM as well.

There are no guarantees whatsoever that this is secure. It will not be merged to main ever and is not supposed to be used in production.