Skip to content

Conversation

@matthewwalsh0
Copy link
Member

@matthewwalsh0 matthewwalsh0 commented Dec 18, 2025

Explanation

Update the relay submission logic to use a static status URL for polling the relay API, instead of dynamically extracting the endpoint from the quote response.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Switch relay polling to a static status endpoint using requestId, update status handling, and adjust types/tests accordingly.

  • Relay Strategy:
    • Use static status endpoint RELAY_STATUS_URL (/intents/status/v3) for polling with GET and requestId query (waitForRelayCompletion).
    • Stop deriving URL/method from quote check; build ${RELAY_STATUS_URL}?requestId=... instead.
    • Treat failure, refund, and refunded as terminal errors; return last txHashes item on success or fallback hash.
  • Types:
    • Add requestId to RelayQuote.steps[0].
    • Expand RelayStatus to include delayed and refunded; introduce RelayStatusResponse.
  • Constants:
    • Add RELAY_STATUS_URL in strategy/relay/constants.ts.
  • Tests:
    • Update tests to assert polling against static URL with requestId; include refunded in error cases.
  • Changelog:
    • Note change: "Poll relay status using static URL".

Written by Cursor Bugbot for commit e647969. This will update automatically on new commits. Configure here.

@matthewwalsh0 matthewwalsh0 requested a review from a team as a code owner December 18, 2025 00:39
@matthewwalsh0 matthewwalsh0 changed the title feat(transaction-pay-controller): Use static relay status URL feat(transaction-pay-controller): static relay status url Dec 18, 2025
cursor[bot]

This comment was marked as resolved.

@matthewwalsh0 matthewwalsh0 changed the title feat(transaction-pay-controller): static relay status url feat: static relay status url Dec 18, 2025
@matthewwalsh0 matthewwalsh0 requested a review from a team as a code owner December 18, 2025 00:45
cursor[bot]

This comment was marked as resolved.

@matthewwalsh0 matthewwalsh0 added this pull request to the merge queue Dec 18, 2025
Merged via the queue into main with commit 902f9fc Dec 18, 2025
287 checks passed
@matthewwalsh0 matthewwalsh0 deleted the feat/relay-static-status-url branch December 18, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants