Skip to content

refactor: qntx.openai 1.0 (EVM/SVM/TVM, spend_controls, httpx2) - #45

Merged
gitctrlx merged 7 commits into
mainfrom
execute-plan/bec9e473-pr-4-docs-10-readme-and-examples-for-qntxopenai
Sep 2, 2026
Merged

refactor: qntx.openai 1.0 (EVM/SVM/TVM, spend_controls, httpx2)#45
gitctrlx merged 7 commits into
mainfrom
execute-plan/bec9e473-pr-4-docs-10-readme-and-examples-for-qntxopenai

Conversation

@gitctrlx

@gitctrlx gitctrlx commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

Rewrite x402-openai 0.3 as qntx-openai 1.0, product-parity with shipped @qntx/openai.

  • PyPI qntx-openai, import qntx.openai. No x402_openai shim.
  • Per-chain keys: evm= / svm= / tvm=. Wallets, BIP-39, max_amount, caller http_client= removed.
  • EVM: exact + upto on eip155:*. SVM: exact only on solana:*. TVM: exact on tvm:-239 / tvm:-3 (not client tvm:*).
  • Official spend_controls (omit = $1). Local TS-semantics prefer_network / prefer_scheme.
  • Owned httpx2 402 adapter: copy of official x402AsyncTransport retry loop, fail-closed.
  • Default base_url https://llm.qntx.org/v1.

Stacked commits (bottom → tip):

  1. Foundation: rename, httpx2 adapter, EVM exact+upto
  2. SVM exact
  3. TVM exact + close() dispose
  4. README / examples / URLs

Do not tag or publish from this PR. Tag v1.0.0 after merge, then yank x402-openai 0.2.0 / 0.2.1 / 0.3.0.

Test plan

  • ruff / mypy / pytest on the merged tree (127 passed)
  • CI on this PR

… httpx2)

Rename package to qntx-openai. Delete wallets/mnemonic. Own httpx2 402 adapter. Register EVM exact+upto. Official spend_controls. TS-semantics policies.
Bind svm=. KeypairSigner.from_base58 + ExactSvmScheme only. No upto.
Bind tvm=. ExactTvmScheme on concrete CAIP-2. Dispose calls scheme.close().
README matches TS 1.0 structure for EVM/SVM/TVM. URLs to qntx/openai-python. Production/Stable.
Copilot AI lite review requested due to automatic review settings September 2, 2026 09:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@gitctrlx

gitctrlx commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Audit of origin/main...a6276e3f77ee8479347af5e403b5a79aaf14902b (five independent reviewers: AGENTS.md, obvious bugs, git history, prior PR comments, in-file comment guidance). Two candidate findings scored below the 80 threshold and were discarded (_REMOVED TypeError is a hard delete of obsolete kwargs, not a compatibility shim; empty _NOT_YET is leftover no-op). Guidance used: https://github.com/qntx/openai/blob/main/Agents.md (this repo has no CLAUDE.md).

@gitctrlx
gitctrlx merged commit 7ed5c93 into main Sep 2, 2026
2 checks passed
@gitctrlx
gitctrlx deleted the execute-plan/bec9e473-pr-4-docs-10-readme-and-examples-for-qntxopenai branch September 2, 2026 09:57
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