Skip to content

Conversation

@mdrxy
Copy link
Collaborator

@mdrxy mdrxy commented Oct 31, 2025

No description provided.

@mdrxy mdrxy requested a review from lnhsingh as a code owner October 31, 2025 17:47
Copilot AI review requested due to automatic review settings October 31, 2025 17:47
@github-actions github-actions bot added langchain For docs changes to LangChain oss labels Oct 31, 2025
@github-actions
Copy link
Contributor

Mintlify preview ID generated: preview-mdrxym-1761932888-352d89c

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 adds a Mermaid sequence diagram to the Tool Calling section of the models documentation to visually illustrate the basic tool calling flow. The diagram demonstrates how a model handles parallel tool calls and processes their results.

Key Changes

  • Adds a sequence diagram showing the interaction between User, Model, and Tools
  • Illustrates parallel tool execution using Mermaid's par syntax
  • Uses a weather query example consistent with the code examples later in the document

Comment on lines +529 to +548
sequenceDiagram
participant U as User
participant M as Model
participant T as Tools
U->>M: "What's the weather in SF and NYC?"
M->>M: Analyze request & decide tools needed
par Parallel Tool Calls
M->>T: get_weather("San Francisco")
M->>T: get_weather("New York")
end
par Tool Execution
T-->>M: SF weather data
T-->>M: NYC weather data
end
M->>M: Process results & generate response
M->>U: "SF: 72°F sunny, NYC: 68°F cloudy"
Copy link

Copilot AI Oct 31, 2025

Choose a reason for hiding this comment

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

[nitpick] The diagram lacks consistent styling found in other Mermaid diagrams throughout the documentation. Consider adding an init block with fontFamily, curve settings, and themeVariables as seen in agents.mdx and short-term-memory.mdx (e.g., %%{ init: { "fontFamily": "monospace", "themeVariables": {"edgeLabelBackground": "transparent"} } }%%).

Copilot uses AI. Check for mistakes.
@lnhsingh lnhsingh merged commit 113e371 into main Nov 6, 2025
17 checks passed
@lnhsingh lnhsingh deleted the mdrxy/model-tool-calling-flow branch November 6, 2025 16:49
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants