Skip to content

fix: expose Polymarket auth initialization in SDKs#601

Open
iccccccccccccc wants to merge 2 commits into
pmxt-dev:mainfrom
iccccccccccccc:fix-polymarket-init-auth-sdk
Open

fix: expose Polymarket auth initialization in SDKs#601
iccccccccccccc wants to merge 2 commits into
pmxt-dev:mainfrom
iccccccccccccc:fix-polymarket-init-auth-sdk

Conversation

@iccccccccccccc
Copy link
Copy Markdown
Contributor

Fixes #505

Changes

  • add Polymarket.initAuth() to the TypeScript SDK
  • generate Polymarket.init_auth() for the Python SDK so it survives the exchange wrapper generator
  • add a small regression test that checks the Python wrapper keeps forwarding to the core initAuth method

Verification

  • node core/scripts/generate-python-exchanges.js
  • python -m pytest sdks/python/tests/test_public_exports.py
  • TypeScript syntax check via typescript.transpileModule on sdks/typescript/pmxt/client.ts
  • git diff --check

I couldn't run the full TypeScript package build in this Windows checkout: the clean script uses rm -rf, and generating the missing OpenAPI client locally needs Java, which isn't installed here.

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.

Polymarket.initAuth() is in core and documented in both SDK API references but missing from both SDK source files

1 participant