Skip to content

feat(client): add combo market catalog#87

Merged
cesarenaldi merged 1 commit into
mainfrom
feat/rfq-combo-market-catalog
Jun 10, 2026
Merged

feat(client): add combo market catalog#87
cesarenaldi merged 1 commit into
mainfrom
feat/rfq-combo-market-catalog

Conversation

@cesarenaldi

@cesarenaldi cesarenaldi commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add RFQ REST routing and environment config for Combo market catalog requests
  • expose list_combo_markets across sync/async public and secure clients
  • add RFQ-owned Combo market models and pagination/request coverage

Verification

  • uv run ruff check .
  • uv run ruff format --check .
  • uv run pyright
  • uv run pytest -m "not integration"
  • uv run pytest tests/integration/test_rfq_paginated.py -q

Note

Low Risk
Additive public API and transport wiring; keyset pagination changes are backward-compatible defaults for existing gamma endpoints.

Overview
Adds RFQ REST support so clients can browse the Combo market catalog via a new keyset-paginated list_combo_markets API (optional exclude filter), wired through sync/async public and secure clients.

Introduces an rfq HTTP transport (environment.rfq_url), extends dispatch/context with service "rfq", and exports ComboMarket models that normalize binary yes/no outcomes from either structured or parallel outcomes / position_ids / outcome_prices wire shapes.

Keyset pagination is generalized with per-spec cursor_param (RFQ uses cursor instead of after_cursor) and optional max_page_size enforcement (100 for combo markets). Public exports and unit/integration tests cover models, specs, and live pagination.

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

@cesarenaldi cesarenaldi merged commit 10bdf99 into main Jun 10, 2026
7 checks passed
@cesarenaldi cesarenaldi deleted the feat/rfq-combo-market-catalog branch June 10, 2026 08:39
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