Skip to content

fix(deps): upgrade the langchain stack together - #12

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/langchain-stack
Nov 6, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
fix/langchain-stack

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Upgrades the LangChain stack in one change, replacing #6, #7 and #8. Those PRs can't install on their own because the packages pin each other.

Package From To
langchain-core 0.3.72 1.7.6
langchain-openai 0.3.28 1.7.4
langgraph 0.5.4 1.2.20
langchain-mcp-adapters 0.1.9 0.3.2
mcp 1.19.0 1.30.0

mcp has to move as well: with 1.19.0 the install succeeds, but app.main fails on import because the adapters need streamable_http_client. verify wouldn't catch a runtime break like this, as the repo has no tests and only installs dependencies.

Verification

I ran run_turn_stream on Python 3.10 on the old and new pins, against a stub of the OpenAI Responses API. The turn covered routing, a calculator tool call and a streamed answer, and both runs produced the same event stream. All app modules also import.

I connected to a local MCP server over streamable HTTP as well. On the new pins, MCP tools return content blocks instead of a string; _preview and content_to_text turn those into the same text.

Not tested

No real model or Appwrite MCP server was used, so this should be checked on a real turn after deploy, particularly MCP write tools going through write_guard. create_react_agent still works here, so I left it in place.

Related: appwrite-labs/cloud#7351

langchain-core, langchain-openai, langgraph and the MCP adapters pin each other, so a security bump to any one of them cannot install alone. mcp moves too because adapters 0.3 import an API that 1.19 lacks, which only fails at runtime.
@appwrite

appwrite Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The dependency pins now upgrade LangChain, LangGraph, and the MCP client stack together. This keeps the related packages on compatible major-version lines.

Verdict New comments Fixed Still open
✅ Approved 0 1 0
📂 Walkthrough · 1
File Change
requirements.txt Updates the LangChain, LangGraph, MCP, and adapter pins.
✅ Fixed since the last review · 1
  • Pinned package releases are not yet published · requirements.txt:5

Reviewed 6eeb01b · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.

appwrite[bot]
appwrite Bot previously requested changes Nov 6, 2026

@appwrite appwrite Bot 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.

🟡 Tier B · 1 blocking finding to address. Summary

Comment thread requirements.txt Resolved
@ChiragAgg5k

Copy link
Copy Markdown
Member Author

@appwrite review

@appwrite
appwrite Bot dismissed their stale review November 6, 2026 10:48

The findings that requested changes are resolved. See the summary comment for what is still open.

@appwrite appwrite Bot 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.

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 29b49d5 into main Nov 6, 2026
4 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the fix/langchain-stack branch November 6, 2026 10:54
@ChiragAgg5k ChiragAgg5k self-assigned this Nov 9, 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.

1 participant