Skip to content

fix: migrate to mcp 2.x and drop the archived langchain-mcp-adapters, update other dependencies, remove warnings - #147

Merged
araujof merged 3 commits into
mainfrom
hl/dependancy
Nov 4, 2026
Merged

araujof merged 3 commits into
mainfrom
hl/dependancy

Conversation

@dhl123

@dhl123 dhl123 commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

…ependency, update other dependencies, remove warnings

Summary

Mcp 2.x was blocked by langchain-mcp-adapters, which every example agent used and which pins mcp<2.0.0. That package is now archived upstream, so it will never support mcp 2.x and waiting wasn't an option.

LangChain has since moved MCP support in-tree as the langchain.mcp namespace, which requires mcp 2.x. Adopting it turns the blocker into a reason to upgrade.

Closes

Closes: #146
This PR also covers other PRs: #140 #141 #142 #143 #144 #145

Changes

Verification

  • Tool extraction output is identical to before for all four examples
  • Unit tests and integration tests pass, lint clean
  • Full /smith pipeline (steps 1–4) on two examples:
    • call-for-papers-mcp — 101 cases, scorecard 71/77 correct, 91.4% coverage
    • employee — 123 cases, scorecard 111/117 correct, zero false allows, 89.2% coverage

Checks

  • make ci passes (lint, Rego lint, license headers, build smoke)
  • make test passes (policy scorecard — needed if policy behavior changed)
  • CHANGELOG.md updated under ## [Unreleased] (if user-facing)
  • PR title uses a conventional prefix (feat:, fix:, test:, docs:, chore:, or refactor:)
  • Commits are signed off for the DCO (git commit -s)

@dhl123 dhl123 linked an issue Nov 3, 2026 that may be closed by this pull request
…ependency, update other dependencies, remove warnings

Signed-off-by: Hailun Ding <hailun.ding@ibm.com>
@dhl123 dhl123 added this to the 0.3.0 milestone Nov 3, 2026
@dhl123 dhl123 changed the title fix: migrate to mcp 2.x, update dependencies, remove warnings fix: migrate to mcp 2.x and drop the archived langchain-mcp-adapters, update other dependencies, remove warnings Nov 3, 2026
Signed-off-by: Hailun Ding <hailun.ding@ibm.com>
@dhl123
dhl123 marked this pull request as ready for review November 3, 2026 19:06
@dhl123
dhl123 requested a review from araujof as a code owner November 3, 2026 19:06
@dhl123
dhl123 marked this pull request as draft November 4, 2026 18:33
@dhl123
dhl123 marked this pull request as ready for review November 4, 2026 18:45
Signed-off-by: Hailun Ding <hailun.ding@ibm.com>

@araujof araujof left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@araujof
araujof merged commit 7247185 into main Nov 4, 2026
14 checks 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.

chore: check dependency conflicts and update mcp version to 2.x.

2 participants