Skip to content

revert(rfq): remove confirmed trade broadcasts#109

Merged
cesarenaldi merged 1 commit into
mainfrom
revert/rfq-trade-broadcast
Jun 17, 2026
Merged

revert(rfq): remove confirmed trade broadcasts#109
cesarenaldi merged 1 commit into
mainfrom
revert/rfq-trade-broadcast

Conversation

@cesarenaldi

@cesarenaldi cesarenaldi commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

Verification

  • uv run ruff format --check && uv run ruff check
  • uv run pyright
  • uv run pytest tests/integration/test_rfq.py started but did not complete because live API key derivation was rate limited with HTTP 429 after 10 tests passed

Note

Low Risk
Narrow API removal aligned with undeployed backend behavior; no changes to quote, confirm, or execution-update flows.

Overview
Reverts RFQ confirmed trade websocket support until the backend RFQ_TRADE broadcast is deployed.

The public RfqTradeEvent type and its export from polymarket / polymarket.rfq are removed, and RfqEvent again only covers quote requests, confirmation requests, and execution updates. The quoter session no longer handles RFQ_TRADE messages or _parse_trade.

Integration tests drop the dedicated trade-broadcast case and the happy-path flow stops after RFQ_EXECUTION_UPDATE instead of also expecting a trade event.

Reviewed by Cursor Bugbot for commit a3eab9f. Bugbot is set up for automated code reviews on this repo. Configure here.

…ast"

This reverts commit f55b613, reversing
changes made to 24d3cf2.
@cesarenaldi cesarenaldi merged commit b5315cd into main Jun 17, 2026
7 checks passed
@cesarenaldi cesarenaldi deleted the revert/rfq-trade-broadcast branch June 17, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant