Skip to content

fix(ts-sdk): use shared transport for market comparison - #2027

Closed
Purin1410 wants to merge 1 commit into
pmxt-dev:mainfrom
Purin1410:codex/fix-1975-router-transport
Closed

fix(ts-sdk): use shared transport for market comparison#2027
Purin1410 wants to merge 1 commit into
pmxt-dev:mainfrom
Purin1410:codex/fix-1975-router-transport

Conversation

@Purin1410

Copy link
Copy Markdown

Summary

  • Route Router.compareMarketPrices through the shared sidecar read transport.
  • Add dynamic URL resolution, connection retries, and the existing compatibility fallback without changing response mapping.

Test plan

  • npm test --workspace=pmxtjs -- --runInBand tests/router-compareMarketPrices.test.ts
  • npm test --workspace=pmxtjs -- --runInBand
  • npm run build --workspace=pmxtjs
  • git diff --check origin/main...HEAD

Fixes #1485

@Purin1410
Purin1410 force-pushed the codex/fix-1975-router-transport branch from 2958523 to 8f8f6de Compare August 12, 2026 07:07
@Purin1410

Copy link
Copy Markdown
Author

Replaced by #2030 after renaming the source branch.

@Purin1410 Purin1410 closed this Aug 12, 2026
@Purin1410
Purin1410 deleted the codex/fix-1975-router-transport branch August 12, 2026 07:52
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.

SDK drift: compareMarketPrices bypasses the shared retrying transport in TypeScript, unlike Python and unlike its own TS sibling Router methods

1 participant