Skip to content

Remove unused internal SDK HTTP helpers - #10

Merged
okJiang merged 1 commit into
mainfrom
codex/sdk-remove-unused-http
Oct 28, 2026
Merged

okJiang merged 1 commit into
mainfrom
codex/sdk-remove-unused-http

Conversation

@okJiang

@okJiang okJiang commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove two unused internal HTTP helpers, put and delete. Current Client Tool updates, uploads and deletions already use the response-aware paths. Supported SDK methods and behavior stay the same.

Cleanup 1 of 4, completed before starting the next cleanup.

Public behavior verified

  • Existing public-interface and installed-consumer tests pass; no new test is needed for unused-code removal.
  • Typechecking, all 123 SDK tests, and packed-package checks pass.
  • Public exports, documentation and service compatibility are unchanged.
  • No credentials or issued plaintext API Keys are included.

Evidence

Before: two internal helpers had no source, test, example or dynamic references. After: callers continue using the existing response-aware helpers and the SDK checks pass.

Merge Danger

Door: two-way. Blast Radius: internal SDK code. No API or deployment change.

@okJiang
okJiang merged commit 8c86235 into main Oct 28, 2026
1 check passed
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