Skip to content

fix(docs): scope Python SQL agent review section#2998

Merged
Naomi Pentrel (npentrel) merged 1 commit intolangchain-ai:mainfrom
chen893:fix/sql-agent-python-tab-scope
Mar 9, 2026
Merged

fix(docs): scope Python SQL agent review section#2998
Naomi Pentrel (npentrel) merged 1 commit intolangchain-ai:mainfrom
chen893:fix/sql-agent-python-tab-scope

Conversation

@chen893
Copy link
Copy Markdown
Contributor

@chen893 chen (chen893) commented Mar 9, 2026

Summary

  • scope the human-in-the-loop SQL agent section to the Python tab
  • prevent Python examples from leaking into the JavaScript SQL agent page

Root cause

The Build a SQL agent doc was missing a :::python wrapper around the human-in-the-loop review section in src/oss/langchain/sql-agent.mdx, so that Python content appeared on the JavaScript page as well.

Testing

  • verified the MDX section is now wrapped with :::python and closed before the :::js section
image

@github-actions github-actions bot added langchain For docs changes to LangChain oss external User is not a member of langchain-ai labels Mar 9, 2026
Copy link
Copy Markdown
Contributor

@npentrel Naomi Pentrel (npentrel) left a comment

Choose a reason for hiding this comment

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

Thank you! Good catch!

@npentrel Naomi Pentrel (npentrel) merged commit 2eefdb2 into langchain-ai:main Mar 9, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external User is not a member of langchain-ai langchain For docs changes to LangChain oss

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants