Skip to content

chore: regen WS generated files (drop stale param_note caveats)#172

Merged
martinkersner merged 1 commit into
mainfrom
chore/regen-ws-endpoints
Jul 5, 2026
Merged

chore: regen WS generated files (drop stale param_note caveats)#172
martinkersner merged 1 commit into
mainfrom
chore/regen-ws-endpoints

Conversation

@martinkersner

Copy link
Copy Markdown
Member

Summary

Regenerates datamaxi/_ws_endpoints.py + datamaxi/_ws_models.py from datamaxi-codegen now that it reads the backend's AsyncAPI spec (codegen #53) instead of the route-table regex + param overrides.

Sole change: the two obsolete param_note caveats are dropped —

  • forex: "fx pair e.g. USD-KRW, no exchange"
  • funding-rate: the "code/doc disagree" note (the backend doc-comment ambiguity was fixed in datamaxi-backend #7929).

_ws_models.py is byte-unchanged. Nothing in the SDK reads param_note, so this is purely keeping the committed generated files in sync with the generator.

Verification

  • git diff = 2 deletions in _ws_endpoints.py, nothing else.
  • pytest tests/test_ws.py tests/test_async.py: 19 passed.

Context

This is the trivial downstream sync noted in codegen #53. A WS drift-guard (check-ws-python, codegen #47) would automate catching this going forward.

Known failures

None.

Regenerated datamaxi/_ws_endpoints.py + _ws_models.py from datamaxi-codegen now
that it reads the backend AsyncAPI spec (Bisonai/datamaxi-codegen#53). Only
change: the two obsolete param_note caveats are gone (forex; funding-rate — the
backend doc-comment ambiguity was fixed in Bisonai/datamaxi-backend#7929).
_ws_models.py is unchanged. Nothing in the SDK reads param_note.
@martinkersner martinkersner merged commit 99f75c9 into main Jul 5, 2026
5 checks passed
@martinkersner martinkersner deleted the chore/regen-ws-endpoints branch July 5, 2026 05:02
@martinkersner martinkersner self-assigned this Jul 5, 2026
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.

1 participant