chore: add validation for quotes with incorrect sell amount

[woodenfurniture]

Description

Prevents a quote being marked as valid (and therefore executable) when the sell amount isn't exactly the same as the sell amount the user inputted.

Pull Request Type

• 🐛 Bug fix (Non-breaking Change: Fixes an issue)
• 🛠️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
• 💅 New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #5723 specifically point create sanity check that we never ask a user to sign a transaction that is larger than what a user input

Risk

High Risk PRs Require 2 approvals

Low risk as this prevents invalid trades. This check failing or being incorrectly implemented could have 2 possible outcomes

• Prevent valid trades going through. Simple to check and validate by testing trades
• Allow invalid quotes with bad sell amounts though. Very unlikely these quotes ever appear at all after resolving the other issues detailed in Swapper attempts to swap more than in wallet and fees beyond the input amount #5723. Testable with a monkey patch by eng team

What protocols, transaction types or contract interactions might be affected by this PR?

LiFi trades specifically, though theoretically any protocol used in trades

Testing

Check trades are working as normal.

Engineering

Worth monkey patching to ensure a quote with mismatching sell amount results in a generic quote error.

Operations

Screenshots (if applicable)

[woodenfurniture]

Current dependencies on/for this PR:

• develop
  • PR chore: add validation for quotes with incorrect sell amount #6241 👈

This stack of pull requests is managed by Graphite.

[gomesalexandre]

Tested locally:

• Li.Fi quotes are still happy ✅

• Li.Fi quotes are sad when monkey-patching input sell asset amount to be different than the quote one ✅