fix(sdk): expose venue filters in fetch params#1325
Conversation
PR Review: PASS (NOT VERIFIED)What This DoesAdds venue-filter aliases to SDK fetch parameter types: Blast RadiusType-only SDK surface in Consumer VerificationBefore (base branch): // before: no sourceExchange/exchange properties in MarketFilterParams/EventFetchParams
await client.fetchMarkets({ sourceExchange: 'polymarket' }); // runtime-forwardable, not typedAfter (PR branch): // sdks/typescript/pmxt/models.ts:453-457, 506-510
sourceExchange?: string;
exchange?: string;# sdks/python/pmxt/models.py:749-750, 769-770
source_exchange: str
exchange: strThis is not directly observable through the HTTP consumer path because the PR only changes static type definitions; the existing SDK transport already serializes supplied params. Test Results
FindingsNo blocking findings. PMXT Pipeline Check
Semver Impactpatch -- type-surface fix for existing/forwarded fetch params, with no runtime API break. RiskGenerated SDK artifacts are unavailable in this checkout, so I could not run the full TypeScript SDK compile. Since the PR only widens parameter types, the remaining risk is limited to publication/type-generation parity rather than runtime behavior. |
Summary
sourceExchange/exchangevenue filter aliases to TypeScript market/event fetch parameter interfaces.source_exchange/exchangeTypedDict keys to Python market/event fetch parameter types.Fixes #1317
Fixes #1318
Test Plan
python3 -m py_compile sdks/python/pmxt/models.pygit diff --checkNote: TypeScript compiler validation was attempted, but this cron host could not install workspace dependencies because
npm installfailed withENOSPC: no space left on device. The change is type-only and mirrors the already-present coreBaseExchangefetch parameter fields.