Skip to content

fix: align the Index Exchange connector with the real /v3/deals API - #137

Open
josh-butcher-index wants to merge 1 commit into
IABTechLab:mainfrom
josh-butcher-index:buyer-agent-index-exchange-fix
Open

josh-butcher-index wants to merge 1 commit into
IABTechLab:mainfrom
josh-butcher-index:buyer-agent-index-exchange-fix

Conversation

@josh-butcher-index

Copy link
Copy Markdown

Summary

  • The Index Exchange SSP connector (src/ad_buyer/tools/deal_library/connectors/index_exchange.py) was implemented without access to IX API docs, so field names, enum values, auth headers, and URL paths were invented rather than matching the real deals-web-api /v3/deals contract.
  • Rewrites the connector against the actual API: correct auth, endpoint paths, pagination params, status values, deal-type/class-ID mapping, and price/field names; replaces the fabricated test fixture with the real response shape.
  • Also fixes seller_deal_id surfacing in list/search, portfolio summary seller scoping, import_deals_ssp filters, and an IX_API_URL override for the stdio entrypoint.
  • No new functionality or MCP tool-surface changes — this only makes the existing IX integration correct.

Test plan

  • tests/unit/test_ssp_index_exchange.py and tests/unit/test_mcp_ssp_tools.py pass against the corrected fixture
  • Manual smoke test against IX staging /v3/deals

Rewrites the Index Exchange SSP connector against the deals-web-api
/v3/deals contract.

Surfaces seller_deal_id in list/search, scopes the portfolio summary to
the seller, adds filters to import_deals_ssp, supports an IX_API_URL
override, and adds a stdio entrypoint.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@josh-butcher-index
josh-butcher-index force-pushed the buyer-agent-index-exchange-fix branch from c250b39 to 714c0fb Compare September 29, 2026 15:34
@josh-butcher-index josh-butcher-index changed the title Fix Index Exchange connector to match the real /v3/deals API fix: align the Index Exchange connector with the real /v3/deals API Sep 29, 2026

This branch has not been deployed

No deployments
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