Skip to content

Rework docs strategy: parallel maintenance, remove Vale, add audit/sync skills#1

Merged
shixish merged 4 commits into
mainfrom
docs/readme-single-source-of-truth
Jun 10, 2026
Merged

Rework docs strategy: parallel maintenance, remove Vale, add audit/sync skills#1
shixish merged 4 commits into
mainfrom
docs/readme-single-source-of-truth

Conversation

@shixish

@shixish shixish commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Removes Vale prose linting from this repo (.vale.ini, .github/vale/styles/, lint_prose Makefile target, lint-readme hook) — prose validation now runs in the docs repo, with findings propagated back here via the audit skill
  • Updates CLAUDE.md to describe four equally-weighted docs surfaces (README, providers hub, tools page, retrievers page) with no single source of truth
  • Rewrites sync-langchain-docs skill: reads all four docs files before acting, updates whichever need it, enforces link-don't-duplicate within langchain-docs
  • Adds audit-langchain-docs skill: runs docs-repo Vale, checks API coverage and import freshness, propagates fixes back to README and Python source
  • Switches user-facing imports from langchain_core.messages/langchain_core.tools to langchain.messages/langchain.tools
  • Promotes the web app example to first in examples/README.md

Test plan

  • make lint passes
  • make typing passes
  • make test passes

🤖 Generated with Claude Code

shixish and others added 4 commits June 9, 2026 13:31
README is now the canonical, complete reference (all 12 components plus a
components-reference table). The LangChain provider page is generated from it by
the sync-langchain-docs skill (README -> MDX), replacing the deleted
docs/provider.mdx as canonical; sync.py is trimmed to docs-repo path resolution
(--path/--repo).

Add Vale prose linting in lockstep with the docs-repo house style: .vale.ini
and .github/vale/styles/ copied from langchain-ai/docs, a 'make lint_prose'
target, a gitignored PostToolUse hook (.claude/) that lints README.md after
edits, and CLAUDE.md documenting the 'lint the README after any change'
convention. Docs-parity tests are retargeted from provider.mdx to README
(test_readme_parity.py, test_readme_examples.py unit+integration, tests/readme.py
helpers); tests/docs.py and test_docs_parity.py are removed.

Also includes pending working-tree changes to the package, tests, and examples
(client-only base refinements, tool kwargs, the dql_explorer web app, and the
quickstart notebook).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…throughout user-facing code

The langchain-ai/docs CI flags langchain_core imports that have langchain
re-export equivalents (messages, tools). Fixes the docs PR check by sourcing
from langchain.messages / langchain.tools in README, examples, and tests.
langchain added to the test dep group so the ChatDiffbot README block stays
executable in the unit suite.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove Vale config (.vale.ini, .github/vale/styles/) and the
  lint-readme PostToolUse hook — prose linting now runs in the docs
  repo only, with findings propagated back here manually
- Remove lint_prose Makefile target
- Update CLAUDE.md Documentation section to describe four equally-weighted
  docs surfaces (README, providers hub, tools page, retrievers page) with
  no single source of truth
- Rewrite sync-langchain-docs skill: reads all four files before acting,
  updates whichever need it, links instead of duplicating within langchain-docs
- Add audit-langchain-docs skill: runs docs-repo Vale, checks API coverage
  and import freshness, propagates fixes back to README.md and Python source
- README: drop prose-linting section; update Authentication section wording

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@shixish shixish merged commit 50b3242 into main Jun 10, 2026
@shixish shixish deleted the docs/readme-single-source-of-truth branch June 10, 2026 21:20
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