Added LLM-based website docs validation section to CONTRIBUTING.md#713
Added LLM-based website docs validation section to CONTRIBUTING.md#713
Conversation
WalkthroughAdds a new "Validate website doc snippets" section to CONTRIBUTING.md with step-by-step JavaScript and React snippet verification prompts and report formats; expands the Release Process to include running doc-snippet validation and detailed doc/version/update steps; minor punctuation and organizational edits. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Coverage Report
File CoverageNo changed files found. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CONTRIBUTING.md`:
- Around line 16-17: Replace the specific model examples in the "[!IMPORTANT]"
block that currently reads "These prompts should be run with the most powerful
LLM available to you (e.g. Claude Opus, GPT-4, etc.)" with a non-timebound
recommendation: either "the most capable model available to you" or, if you
prefer examples, update them to current flagship models (e.g., "GPT-5.2" and
"Anthropic Opus 4.6") and add a short parenthetical like "(update examples as
newer models are released)"; locate and edit the exact "[!IMPORTANT]" paragraph
in CONTRIBUTING.md to make this change.
- Line 57: The CONTRIBUTING prompt creates temporary report files
chat_javascript_api_verification_report.md and
chat_react_api_verification_report.md but they aren’t ignored; update the repo
so these generated reports aren’t accidentally committed by either adding those
two filenames to .gitignore or changing the prompt to write reports to an
explicit output directory outside the repository (e.g., a tmp/ or /tmp path).
Ensure you update the prompt text in CONTRIBUTING.md (the lines that reference
chat_javascript_api_verification_report.md and
chat_react_api_verification_report.md) to reflect the chosen fix so the
filenames or output path stay consistent.
6842457 to
5636429
Compare
… js/react code snippets in the web documentation against the SDK source code.
5636429 to
6acf521
Compare
| 5. Push the branch (`git push origin my-new-feature`) | ||
| 6. Create a new Pull Request | ||
|
|
||
| ## Validate website doc snippets |
There was a problem hiding this comment.
Question - should this be a skill/plugin we add to the internal MCP tooling? Perhaps it's re-usable across repositories rather than having to be in every single repos files?
There was a problem hiding this comment.
This should definitely be a skill (I think I mentioned this on the last PR adding to CONTRIBUTING.md. I think this sort of thing is useful for external users, but maybe this can also be an CI step that asks claude to validate website snippets and apply a review?
There was a problem hiding this comment.
I looked into Claude Skills, and we can consider adding this as a Claude Skill in the near future.
However, that approach would be limited to Claude. We should also maintain a generic prompt that works with any LLM—especially in cases where Claude is unavailable or offline—so we can manually run it and verify the website documentation as a part of the release process.
Additionally, based on my experience, this task typically requires manual intervention and multiple refinement cycles to achieve accurate results.
There was a problem hiding this comment.
Seems claude was down yesterday itself https://www.reddit.com/r/ClaudeAI/comments/1rir4n5/claude_is_down/, it's not always reliable : (
There was a problem hiding this comment.
Can we add a ticket around having a Claude skill (so we don't forget to revisit this) please? :)
There was a problem hiding this comment.
Already have it here -> https://ably.atlassian.net/browse/DX-837
7045656 to
7408b22
Compare
splindsay-92
left a comment
There was a problem hiding this comment.
Approving with the understanding we will track and add claude skills for devs around this soon.
Summary by CodeRabbit
Note
Low Risk
Documentation-only changes that adjust contributor/release guidance without affecting runtime code or published artifacts.
Overview
Adds a new “Validate website doc snippets” section to
CONTRIBUTING.mdwith copy/paste LLM prompts to auditjavascriptandreactcode blocks inably/docschat.mdxpages against this SDK, including required checks and report file outputs.Updates the Release Process to reference this validation step when public APIs change, and expands the docs-release checklist to include fixing issues found and merging related
ably/docsPRs; also makes minor wording/punctuation tweaks.Written by Cursor Bugbot for commit 7408b22. This will update automatically on new commits. Configure here.