Skip to content

Conversation

@moeodeh3
Copy link

@moeodeh3 moeodeh3 commented Jan 10, 2026

Overview

this pr includes:

  • api-key-stamper
  • sdk-types
  • http

the main difference between sdk-server and turnkey_http is that the Python client is built on top of sdk-types, which provides cleaner, easier-to-read generated types that the client uses directly. On the other hand, sdk-server relies on the OpenAPI-generated types from mono, which are a lot harder to read and work with. So I'd say the Python http client feels closer to the core client than it does to sdk-server

It also includes a client.send_signed_request helper that makes sending stamped requests really easy as a bonus

Why use Pydantic in our generated types?

we use it to handle field name conflicts with Python keywords. For example, some Turnkey API requests include a from field (like v1EthSendTransactionIntent), which is a reserved keyword in Python

pydantic's Field aliasing allows us to:

  • use from_ in Python code (avoiding syntax errors)
  • automatically serialize to from when making API requests to Turnkey

Tests

I’ve added 10 tests covering:

  • activities
  • queries
  • stamping functions
  • orgId overrides

@socket-security
Copy link

socket-security bot commented Jan 10, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedsetuptools@​80.9.07410010010070
Addedmypy@​1.19.182100100100100
Addedpytest@​9.0.287100100100100
Addedwheel@​0.45.198100100100100
Addedrequests@​2.32.599100100100100
Addeddatamodel-code-generator@​0.52.2100100100100100
Addedcryptography@​46.0.3100100100100100
Addedpydantic@​2.12.5100100100100100
Addedruff@​0.14.11100100100100100
Addedpytest-asyncio@​1.3.0100100100100100
Addedtypes-requests@​2.32.4.20260107100100100100100

View full report

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.

2 participants