Skip to content

Conversation

@Nischal2015
Copy link
Contributor

Overview

Fixes a small typo in the LangChain documentation where Middleware was used instead of AgentMiddleware in an example for ExpertiseBasedToolMiddleware.

Type of change

Type: Fix typo/bug/link/formatting

Related issues/PRs

  • None

Checklist

  • I have read the contributing guidelines
  • I have verified the class name in the current LangChain API
  • All code examples have been tested and work correctly
  • Root relative paths verified
  • N/A — no navigation updates needed
  • I have gotten approval from the relevant reviewers

Additional notes

This change aligns the example with the current LangChain API (AgentMiddleware is the correct base class).

Copilot AI review requested due to automatic review settings October 23, 2025 09:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a typo in the LangChain v1 release documentation where an example incorrectly used Middleware as a base class instead of the correct AgentMiddleware class.

Key Changes

  • Corrected the base class name in the ExpertiseBasedToolMiddleware example from Middleware to AgentMiddleware

@github-actions github-actions bot added langchain For docs changes to LangChain python For content related to the Python version of LangChain projects oss labels Oct 23, 2025
@eyurtsev eyurtsev enabled auto-merge (squash) October 24, 2025 15:48
eyurtsev pushed a commit that referenced this pull request Oct 24, 2025
## Overview
Refactor ExpertiseBasedToolMiddleware to use ChatOpenAI and Callable for
model requests.

## Type of change

**Type:** [bug/ Remove outdated content]

## Related issues/PRs
<!-- 

To automatically close an issue when this PR is merged, use closing
keywords:
- "closes #123" or "fixes #123" or "resolves #123"

For regular references without auto-closing, just use:
- "#123" or "See issue #123"

Examples:
- closes #456 (will auto-close issue #456 when PR is merged)
- See #789 for context (will reference but not auto-close issue #789)
-->
- GitHub issue: #1113
- Feature PR: #1104
- closes #1113
- please merge first #1104

<!-- For LangChain employees, if applicable: -->
- Linear issue:
- Slack thread:

## Checklist
<!-- Put an 'x' in all boxes that apply -->
- [x] I have read the [contributing guidelines](README.md)
- [x] I have tested my changes locally using `docs dev`
- [x] All code examples have been tested and work correctly
- [x] I have used **root relative** paths for internal links
- [x] I have updated navigation in `src/docs.json` if needed
- I have gotten approval from the relevant reviewers

## Additional notes
<!-- Any other information that would be helpful for reviewers -->
@eyurtsev eyurtsev disabled auto-merge October 24, 2025 15:48
@eyurtsev eyurtsev merged commit 6a78f3f into langchain-ai:main Oct 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

langchain For docs changes to LangChain oss python For content related to the Python version of LangChain projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants