Skip to content

test(mcp-adapters): verify elicitation recovery and examples - #11645

Open
Baihao Wang (byhow) wants to merge 4 commits into
codex/mcp-graph-elicitationfrom
codex/mcp-elicitation-recovery
Open

Baihao Wang (byhow) wants to merge 4 commits into
codex/mcp-graph-elicitationfrom
codex/mcp-elicitation-recovery

Conversation

@byhow

@byhow Baihao Wang (byhow) commented Sep 15, 2026

Copy link
Copy Markdown

What changes?

Add a runnable modern elicitation example that answers two forms and a URL request, rebuilding the adapter between resumes. Local stdio tests verify reconstruction of both the adapter and server process, and exercise accept/decline/cancel with modern and mixed legacy/modern servers.

The reference and migration guide explain checkpoint ownership, answer shapes, direct-call limits, connection-level headers and Python API differences. The example uses scripted answers and MemorySaver; applications supply their own input flow and persistent checkpoint storage for process recovery.

How was it tested?

  • 404 tests passed, two existing skips; package/example and modified-test type checks, lint and formatting passed.
  • Ran the HTTP example against the local modern server: two forms and a URL completed, and separate decline/cancel runs returned their expected results.
  • The complete replacement stack builds for ESM and CommonJS; attw and publint passed.

Scope

Fourth replacement slice of #11615, stacked on #11644. 514 additions and 25 deletions. Tests, examples, docs and a changeset only; no library runtime, OAuth, dependency or CI changes.

The stdio test reconstructs the server process while retaining an in-process checkpointer. It does not verify a separate-process checkpoint database. The migration guide retains the public SDK validator prerequisite for the major release.

@changeset-bot

changeset-bot Bot commented Sep 15, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 83e2482

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@langchain/mcp-adapters Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@byhow
Baihao Wang (byhow) marked this pull request as ready for review September 15, 2026 01:08

@open-swe open-swe Bot 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.

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in WebView Open SWE trace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant