Skip to content

Update dependencies - #313

Merged
AayushTyagi1 merged 1 commit into
mainfrom
update-openai-dependency
Oct 12, 2026
Merged

AayushTyagi1 merged 1 commit into
mainfrom
update-openai-dependency

Conversation

@AayushTyagi1

Copy link
Copy Markdown
Member

Related Issue

Resolves:

What does this change do?

Why is this change needed?

Evidence of Testing

Automated tests — commands run and results summary:

# e.g. uv run pytest tests/unit/  →  142 passed
# e.g. uv run pytest tests/integration/  →  38 passed, 2 skipped

Environments tested (both are required):

  • Couchbase Capella (version: ____)
  • Self-managed Couchbase Server (version: ____, e.g. via Docker)

Manual verification — MCP client used (Claude Desktop, Cursor, MCP Inspector, ...) and what was exercised. Screenshots or tool-call transcripts are very helpful:

Compatibility Considerations

Checklist

  • Linked to an issue (required for new tools). The issue can be on JIRA (preferred for internal contributors) or GitHub.
  • Uses the Couchbase SDK (REST fallback justified in the description, if any)
  • Works on both Capella and self-managed Couchbase Server
  • No changes to cb_mcp.core contracts / managed MCP interfaces (or discussed first)
  • Unit tests added/updated
  • Integration tests added/updated (for cluster-touching changes)
  • Read-only mode and tool annotations handled (for new/changed tools)
  • Evidence of testing included above
  • Docs updated (README, DOCKER.md) for user-facing changes
  • Lint and pre-commit pass

@AayushTyagi1
AayushTyagi1 merged commit 4f44ea8 into main Oct 12, 2026
20 checks passed

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.

🟡 Changes recommended

The OpenAI major-version upgrade lacks PR-gated compatibility coverage for the SDK call paths used by the server.

1 open finding
What changed in this PR

Updates the dependency set, primarily adopting OpenAI SDK v3.

Changes:

  • Raises OpenAI from v2 to v3.
  • Refreshes the lockfile, including Operational Insights and transitive dependencies.
File Description
pyproject.toml Updates the supported OpenAI SDK range.
uv.lock Records updated direct and transitive resolutions.

🧠 Review effort: Balanced


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

Comment thread pyproject.toml
# remove [tool.hatch.metadata] allow-direct-references below.
"couchbase-operational-insights @ git+https://github.com/couchbaselabs/operational-insights-python-client.git",
"openai>=2.0.0,<3.0.0",
"openai>=3.0.0,<4.0.0",
nithishr pushed a commit that referenced this pull request Oct 12, 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.

3 participants