Feat Request for ProtoRev: Add input/output coins to CL Swap listener

[NotJeremyLiu]

Background

• For v16, ProtoRev will be using a hook-based approach to backrunning transactions, instead of the current message parsing based approach.

• The information ProtoRev needs is the token(s) in and token(s) out from a given swap.

• This information is currently provided in the AfterCFMMSwap hook for the gamm module

  osmosis/x/gamm/types/hooks.go

  Line 47 in ed08552

  func (h MultiGammHooks) AfterCFMMSwap(ctx sdk.Context, sender sdk.AccAddress, poolId uint64, input sdk.Coins, output sdk.Coins) {

• But is not provided in the AfterConcentratedPoolSwap listener in the concentrated-liquidity module

  osmosis/x/concentrated-liquidity/types/listeners.go

  Line 40 in ed08552

  func (l ConcentratedLiquidityListeners) AfterConcentratedPoolSwap(ctx sdk.Context, sender sdk.AccAddress, poolId uint64) {

• This issue is opened to request that input/output coins for a swap are provided in the AfterConcentratedPoolSwap listener to allow protorev to fully move to a hook-based approach for extracting swaps.

Suggested Design

• Provide input/output coins for the swap in the AfterConcentratedPoolSwap listener, similar to the gamm module swap hook.