Skip to content

Conversation

@mdrxy
Copy link
Collaborator

@mdrxy mdrxy commented Oct 10, 2025

Initial pass assisted by Claude

@Copilot Copilot AI review requested due to automatic review settings October 10, 2025 00:51
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 transfers the LangSmith Python SDK documentation into the unified documentation repository by adding comprehensive reference documentation for the LangSmith Python SDK alongside existing LangChain and LangGraph documentation.

  • Adds LangSmith as a dependency in the Python reference docs build system
  • Creates complete navigation structure with core and additional API sections
  • Adds 17 new documentation pages covering all major LangSmith SDK modules

Reviewed Changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated no comments.

File Description
reference/python/pyproject.toml Adds langsmith dependency and git source configuration
reference/python/mkdocs.yml Adds langsmith to preloaded modules and creates comprehensive navigation structure
reference/python/docs/platform/python_sdk.md Removes redundant handler specifications from existing platform SDK docs
reference/python/docs/langsmith/*.md Creates 17 new reference documentation pages for all LangSmith SDK modules

@vercel
Copy link

vercel bot commented Oct 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
reference-docs Ready Ready Preview Comment Oct 10, 2025 0:57am

@github-actions
Copy link
Contributor

Preview ID generated: preview-mdrxyr-1760057523-a6e038d

@mdrxy mdrxy self-assigned this Oct 13, 2025
@github-actions
Copy link
Contributor

Preview ID generated: preview-mdrxyr-1760382138-9f40922

@Copilot Copilot AI review requested due to automatic review settings October 14, 2025 19:26
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

Copilot reviewed 17 out of 18 changed files in this pull request and generated 1 comment.

- [Wrappers](wrappers.md): Tracing wrappers for popular LLM providers.
- [Anonymizer](anonymizer.md): Tools for anonymizing sensitive data.
- [Testing](testing.md): Testing utilities and pytest integration.
- [Expect API](expect.md): Assertions and expectations for testing.
Copy link

Copilot AI Oct 14, 2025

Choose a reason for hiding this comment

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

The navigation entry lists 'Expect API' but references a private module langsmith._expect (with underscore prefix) in expect.md. Consider documenting whether this is intended to be a public API or if the module reference should point to a public interface.

Suggested change
- [Expect API](expect.md): Assertions and expectations for testing.
- [Expect API](expect.md): Assertions and expectations for testing. **(Internal/private API)**

Copilot uses AI. Check for mistakes.
@github-actions
Copy link
Contributor

Preview ID generated: preview-mdrxyr-1760470024-70244da

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