Autopilot to use score based on domain::Settlement instead of trusting the driver.

[sunce86]

Instead of using driver provided score to compare the solutions, autopilot should build domain::Settlement from each solution and call score function and use that to compare solutions.

Dependent on #2719 and #2718

[github-actions]

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

[github-actions]

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

[github-actions]

This issue has been marked as stale because it has been inactive a while. Please update this issue or it will be automatically closed.

[sunce86]

I worked on this but eventually got blocked by actually using new Auction table and removing the duplicated struct in https://github.com/cowprotocol/services/blob/main/crates/autopilot/src/domain/settlement/auction.rs

Actually using new Auction object is still in progress here https://github.com/cowprotocol/services/tree/use-new-auction-table so will first create a PR for that.