Skip to content

packaging: require typing-extensions 4.x - #1257

Merged
Carter Tinney (cartertinney) merged 2 commits into
mainfrom
agents/typing-extensions-minimum
Sep 2, 2026
Merged

packaging: require typing-extensions 4.x#1257
Carter Tinney (cartertinney) merged 2 commits into
mainfrom
agents/typing-extensions-minimum

Conversation

@cartertinney

Copy link
Copy Markdown
Member

Summary

  • require typing-extensions>=4.0.0,<5.0.0 because the SDK imports Self
  • update the lockfile metadata for the declared runtime constraint

Testing

  • built the wheel and verified its Requires-Dist metadata
  • installed the wheel with the minimum typing-extensions==4.0.0 and imported every SDK module that uses Self, ParamSpec, or TypedDict
  • uv lock --check

Declare the minimum version that provides Self and retain a major-version upper bound for dependency compatibility.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@cartertinney
Carter Tinney (cartertinney) merged commit 8c0d51a into main Sep 2, 2026
42 checks passed
@cartertinney
Carter Tinney (cartertinney) deleted the agents/typing-extensions-minimum branch September 2, 2026 14:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The dependency constraint correctly covers Self availability and the lockfile metadata is consistent.

Pull request overview

Updates runtime dependency metadata to support SDK imports from typing_extensions.

Changes:

  • Requires typing-extensions>=4.0.0,<5.0.0.
  • Synchronizes lockfile metadata.
File summaries
File Description
pyproject.toml Declares the supported dependency range.
uv.lock Records the updated runtime constraint.
Review details
  • Files reviewed: 1/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants