fix(typescript): replace any with proper types for Exchange router methods#1419
fix(typescript): replace any with proper types for Exchange router methods#1419AbhilashG12 wants to merge 2 commits into
Conversation
…thods - fetchMarketMatches: any → typed params + MatchResult[] - fetchEventMatches: any → typed params + EventMatchResult[] - compareMarketPrices: any → typed params + PriceComparison[] - fetchMatchedPrices: any → typed params + PriceComparison[] - fetchArbitrage: any → typed params + ArbitrageOpportunity[] Fixes pmxt-dev#1403
- Sync with BaseExchange.ts - Remove fetchEventsPaginated (not in BaseExchange) Fixes pmxt-dev#1403
PR Review: FAILWhat This DoesThis PR is intended to tighten TypeScript SDK typings for router/matching methods. In the final head commit, however, the regenerated Blast RadiusTypeScript SDK only ( Consumer VerificationBefore (base branch): After (PR branch): Test Results
Findings
PMXT Pipeline Check
Semver Impactpatch if fixed -- intended SDK type-only bug fix, but current head contains a breaking hosted SDK regression. RiskI could not run the full TypeScript build because generated SDK artifacts are absent in the review checkout, but the blocking issues above are visible in the checked-in source and do not depend on build output. |
What this PR does
Adds proper TypeScript types to Exchange router methods.
The Problem
Five Exchange methods used
params?: anyinstead of typed params:fetchMarketMatchesfetchEventMatchescompareMarketPricesfetchMatchedPricesfetchArbitrageThe Fix
router.tsWhy it matters
TypeScript users now get proper type checking when calling these methods.
Fixes #1403