fix: name hardcoded chain IDs#878
Conversation
PR Review: PASS (NOT VERIFIED)What This DoesReplaces hard-coded EVM chain-id literals with named constants across exchange setup. Runtime response shapes are unchanged; the win is safer configuration/readability. Blast RadiusLimitless, Polymarket, and Probable auth/client/websocket setup; no SDK model or OpenAPI shape change. Consumer VerificationBefore (base branch): After (PR branch): Test Results
FindingsNo blocking findings. PMXT Pipeline Check
Semver Impactpatch -- bug fix/internal compatibility change unless SDK consumers rely on newly added APIs RiskThe branch was built, core-tested, and authenticated-smoke-tested through the sidecar. Full live-exchange before/after behavior remains unverified for this daily batch; credentialed venue calls and venue API drift should be treated as residual risk unless covered by tests in this PR. |
Summary
Tests
Fixes #251