docs(ai): add CodeRabbit CLI setup guide and update AI reviews overview#36
docs(ai): add CodeRabbit CLI setup guide and update AI reviews overview#36
Conversation
|
Important Review skippedThis PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughUpdated AI Reviews doc to state CodeRabbit AI is enabled on some (not all) LFX orgs/repos, added steps to request repo enablement (Slack channels and ticket portal), updated ticket URL to CloudOps, clarified admin console access and GitHub authentication, and linked a new CLI setup doc. Added Changes
Sequence Diagram(s)mermaid Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes SLIGHT Manifest File AnalyzerNONE 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
mcp/jira.md (1)
250-265:⚠️ Potential issue | 🟡 MinorRefresh the validation transcript to match the renamed MCP server.
The guide now standardizes on
atlassian, but this sample still showsmcp-atlassian:getJiraIssue. That leaves the validation section looking like it was captured before the rename and makes it harder for readers to confirm they configured the right server.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@mcp/jira.md` around lines 250 - 265, Update the validation transcript so the renamed MCP server is reflected by replacing the old service identifier mcp-atlassian:getJiraIssue with the new atlassian:getJiraIssue wherever it appears in the sample output/transcript (the block that shows the assistant calling the integration and the line starting with "mcp-atlassian:getJiraIssue (MCP)..."); ensure the surrounding explanatory text remains consistent with the new name so readers can confirm they configured the correct server.ai/ai-reviews.md (1)
14-19:⚠️ Potential issue | 🟠 MajorFix the rollout statement and support path in this paragraph.
This now says CodeRabbit is enabled for all LFX organizations/repositories, but
ai/coderabbit-cli.mdsays it is enabled on only some repositories and teams may need platform help to turn it on. Also, the “file a ticket with the CloudOps team” link points to the CodeRabbit dashboard, not a ticket form. As written, readers get both the wrong availability status and the wrong escalation path.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@ai/ai-reviews.md` around lines 14 - 19, The paragraph in ai/ai-reviews.md incorrectly states global enablement and has a wrong escalation link; update the rollout statement to match ai/coderabbit-cli.md by stating CodeRabbit AI is enabled only on select LFX repositories/organizations and that teams may need to request enablement from Platform/CloudOps, and replace the “file a ticket with the CloudOps team” dashboard URL with the correct CloudOps ticket/escalation link (or internal support form) and clarify that administrators access the CodeRabbit console via the dashboard using GitHub login; reference the paragraph in ai/ai-reviews.md and the guidance in ai/coderabbit-cli.md when making the edits.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ai/coderabbit-cli.md`:
- Around line 25-39: Update the Installation section to mark the Homebrew method
as macOS-only by appending "(macOS only)" or similar to the Homebrew
heading/label for the command `brew install --cask coderabbit`, and add a short
clarifying sentence near the "Manual Install Script" text indicating that the
curl/sh installer is the cross-platform (macOS and Linux) alternative; ensure
the existing phrasing "Homebrew (Recommended)" is changed to "Homebrew
(Recommended, macOS only)" and the "Manual Install Script" paragraph explicitly
states it supports Linux and macOS.
In `@mcp/jira.md`:
- Around line 73-75: Replace the hardcoded cloudId examples ("linuxfoundation" /
"linuxfoundation.atlassian.net") with guidance that cloudId is a UUID and must
be discovered programmatically; update the explanatory text around the previous
examples to instruct users to call the getAccessibleAtlassianResources helper
(per Atlassian docs) to retrieve their cloudId (showing the UUID format like
"c497ae62-1596-4157-81fb-96205956683e" as the canonical example) and remove any
instructions that imply using site name or domain as the cloudId.
---
Outside diff comments:
In `@ai/ai-reviews.md`:
- Around line 14-19: The paragraph in ai/ai-reviews.md incorrectly states global
enablement and has a wrong escalation link; update the rollout statement to
match ai/coderabbit-cli.md by stating CodeRabbit AI is enabled only on select
LFX repositories/organizations and that teams may need to request enablement
from Platform/CloudOps, and replace the “file a ticket with the CloudOps team”
dashboard URL with the correct CloudOps ticket/escalation link (or internal
support form) and clarify that administrators access the CodeRabbit console via
the dashboard using GitHub login; reference the paragraph in ai/ai-reviews.md
and the guidance in ai/coderabbit-cli.md when making the edits.
In `@mcp/jira.md`:
- Around line 250-265: Update the validation transcript so the renamed MCP
server is reflected by replacing the old service identifier
mcp-atlassian:getJiraIssue with the new atlassian:getJiraIssue wherever it
appears in the sample output/transcript (the block that shows the assistant
calling the integration and the line starting with "mcp-atlassian:getJiraIssue
(MCP)..."); ensure the surrounding explanatory text remains consistent with the
new name so readers can confirm they configured the correct server.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 76993683-549c-4f8e-b7db-a2310b3fca4c
📒 Files selected for processing (3)
ai/ai-reviews.mdai/coderabbit-cli.mdmcp/jira.md
There was a problem hiding this comment.
Pull request overview
This PR updates internal developer documentation by adding a dedicated CodeRabbit CLI setup guide and refreshing related docs to reflect current tool usage and configuration guidance.
Changes:
- Added a new CodeRabbit CLI setup guide for LFX developers (install, auth, usage, IDE integrations, workflows).
- Updated the Jira MCP documentation to use Atlassian’s newer MCP endpoint and added an API-token authentication alternative.
- Added a cross-reference from the AI reviews overview to the new CodeRabbit CLI guide.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
mcp/jira.md |
Updates Jira MCP configuration examples to the new endpoint and expands auth/usage guidance. |
ai/coderabbit-cli.md |
New comprehensive CodeRabbit CLI installation and usage guide for LFX developers. |
ai/ai-reviews.md |
Adds a link to the new CodeRabbit CLI setup guide. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
418adf9 to
9cd73df
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
ai/coderabbit-cli.md (1)
25-39:⚠️ Potential issue | 🟡 MinorLabel the Homebrew installation method as macOS-only.
The Installation section presents
brew install --cask coderabbitas "(Recommended)" without platform qualification. The Homebrew cask requires macOS, and Linux users following this guidance will encounter an error. Explicitly scope the Homebrew option to macOS and clarify that the Manual Install Script is the cross-platform alternative.📝 Suggested fix for platform clarity
-### Homebrew (Recommended) +### Homebrew (Recommended, macOS only)And add a clarifying note to the Manual Install Script section:
### Manual Install Script + +This method works on both macOS and Linux:🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@ai/coderabbit-cli.md` around lines 25 - 39, Update the Installation section to indicate platform scope: mark the "Homebrew (Recommended)" heading or the line with "brew install --cask coderabbit" as macOS-only (e.g., "Homebrew (macOS, Recommended)") and add a brief note that Homebrew casks require macOS, and Linux users should use the "Manual Install Script" instead; also update the "Manual Install Script" paragraph to state it is a cross-platform alternative (works on macOS and Linux) and recommend it for non-macOS users so readers can immediately choose the correct install method.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ai/coderabbit-cli.md`:
- Around line 97-99: In the numbered list items within ai/coderabbit-cli.md
update the two entries that lack a space after the colon so the quoted text is
separated: change `results:"Evaluate the fixes and considerations. Fix major
issues only."` to `results: "Evaluate the fixes and considerations. Fix major
issues only."` and change `confirm:"Run cr one more time to make sure we
addressed all critical issues."` to `confirm: "Run cr one more time to make sure
we addressed all critical issues."` to restore consistent punctuation and
spacing.
---
Duplicate comments:
In `@ai/coderabbit-cli.md`:
- Around line 25-39: Update the Installation section to indicate platform scope:
mark the "Homebrew (Recommended)" heading or the line with "brew install --cask
coderabbit" as macOS-only (e.g., "Homebrew (macOS, Recommended)") and add a
brief note that Homebrew casks require macOS, and Linux users should use the
"Manual Install Script" instead; also update the "Manual Install Script"
paragraph to state it is a cross-platform alternative (works on macOS and Linux)
and recommend it for non-macOS users so readers can immediately choose the
correct install method.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f494b00d-d716-49b4-a9a4-821df2074546
📒 Files selected for processing (2)
ai/ai-reviews.mdai/coderabbit-cli.md
🚧 Files skipped from review as they are similar to previous changes (1)
- ai/ai-reviews.md
c323bdf to
8c1369a
Compare
Create ai/coderabbit-cli.md with comprehensive guide covering installation (Homebrew and install script), authentication, CLI usage, multi-repo analysis, IDE integration (VSCode and Cursor), Anthropic CoWork and Claude Code workflows, and LFX-specific platform engineering contact information. Update ai/ai-reviews.md with cross-reference link to the new CLI documentation and align CodeRabbit enablement messaging between documents. Generated with [Claude Code](https://claude.ai/code) Signed-off-by: David Deal <ddeal@linuxfoundation.org>
8c1369a to
b867a83
Compare
Summary
Create comprehensive CodeRabbit CLI documentation for LFX developers covering installation, setup, and usage.
Changes
Key highlights
Validation
Generated with Claude Code