Skip to content

fix(python-sdk): export feed client support types#1172

Open
realfishsam wants to merge 1 commit into
mainfrom
fix/issue-1170-python-feed-types
Open

fix(python-sdk): export feed client support types#1172
realfishsam wants to merge 1 commit into
mainfrom
fix/issue-1170-python-feed-types

Conversation

@realfishsam

Copy link
Copy Markdown
Contributor

Summary

  • Export Python FeedClient support types at the package root: Ticker, Tickers, OHLCV, Market, FeedMarket, and OracleRound.
  • Add a public-export regression test for the feed support types.

Fixes #1170

Test Plan

  • python3 -m py_compile sdks/python/pmxt/feed_client.py sdks/python/pmxt/__init__.py sdks/python/tests/test_public_exports.py
  • AST regression check that the feed support types are imported from feed_client and present in __all__
  • git diff --check

Notes

  • python3 -m pytest was not available in this cron image (No module named pip / no pytest command), so I ran the equivalent targeted AST assertions directly.
  • npm run generate:docs --workspace=pmxt-core produced broad unrelated API reference/config drift, so those generated files were reverted to keep this PR scoped.

@realfishsam

Copy link
Copy Markdown
Contributor Author

CI is currently red on generated-sync checks that are unrelated to this focused export fix:

  • Verify COMPLIANCE.md is up-to-date
  • Verify client.py methods are up-to-date
  • Verify client.ts methods are up-to-date
  • Verify API_REFERENCE.md files are up-to-date

I reproduced the generator path on a fresh origin/main worktree after #1150 and it wants broad generated drift: remove core/COMPLIANCE.md Hunch rows and strip large hosted client overrides in both SDK clients, plus API reference timestamp/source drift. I am intentionally keeping this PR scoped to #1170 instead of folding that unrelated generated drift into it.

Focused validation passed:

  • python3 -m py_compile sdks/python/pmxt/feed_client.py sdks/python/pmxt/__init__.py sdks/python/tests/test_public_exports.py
  • direct AST regression check for FeedClient, Ticker, Tickers, OHLCV, Market, FeedMarket, and OracleRound imports + __all__
  • git diff --check

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: FeedClient supporting types (Ticker, Tickers, OHLCV, Market, OracleRound) not publicly exported from the Python package

1 participant