Use reservation ids in instant out quote

[sputn1ck]

This PR changes the way we quote for instantout by also providing the reservation ids.

Ai generated summary

This pull request includes changes to the InstantOutQuote functionality to replace the use of numReservations with reservationIds. The main goal is to improve the precision and flexibility of the reservation system.

Key changes include:

Updates to request and response handling:

• looprpc/client.proto: Deprecated the num_reservations field and added a new reservation_ids field in the InstantOutQuoteRequest message.
• swapserverrpc/instantout.pb.go: Updated the GetInstantOutQuoteRequest struct to include the new reservation_ids field and added a corresponding getter method. [1] [2]

Modifications in the quote retrieval logic:

• instantout/manager.go: Changed the GetInstantOutQuote method to accept reservationIDs instead of numReservations and updated the logic to use the length of reservationIDs for fee calculations. [1] [2] [3]

Adjustments in the client-server communication:

• loopd/swapclient_server.go: Updated the InstantOutQuote method to pass reservationIds to the GetInstantOutQuote method.
• cmd/loop/instantout.go: Modified the InstantOutQuote call to use ReservationIds instead of NumReservations.

These changes ensure that the system now uses specific reservation IDs, providing more accurate and detailed control over the reservations used in the swap process.

[Copilot]

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

instantout/manager.go:223

• [nitpick] The condition "len(reservationIDs) <= 0" can be replaced with "len(reservationIDs) == 0" for clarity, since a slice length will never be negative.

if len(reservationIDs) <= 0 {

[lightninglabs-deploy]

@bhandras: review reminder
@starius: review reminder
@sputn1ck, remember to re-request review from reviewers when ready

[bhandras]

LGTM 🎉

[hieblmi]

needs rebase, otherwise LGTM✅