Skip to content

chore(ci): Add MCP inline comment tool to Claude Code Review workflow#884

Merged
yamadashy merged 1 commit intomainfrom
chore/claude-code-review
Oct 10, 2025
Merged

chore(ci): Add MCP inline comment tool to Claude Code Review workflow#884
yamadashy merged 1 commit intomainfrom
chore/claude-code-review

Conversation

@yamadashy
Copy link
Owner

Add mcp__github_inline_comment__create_inline_comment to the allowed tools in the Claude Code Review workflow. This enables the review agent to create inline comments on specific lines of code in pull requests.

Checklist

  • Run npm run test
  • Run npm run lint

Copilot AI review requested due to automatic review settings October 9, 2025 15:08
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 adds the MCP inline comment tool to the Claude Code Review workflow to enable creating inline comments on specific lines of code in pull requests.

  • Added mcp__github_inline_comment__create_inline_comment to the allowed tools list in the CI workflow

@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

The GitHub Actions workflow .github/workflows/claude-code-review.yml was updated to add mcp__github_inline_comment__create_inline_comment to the claude_args allowed tools list, preceding the existing Bash-based GH tools. No other files or public APIs were changed.

Changes

Cohort / File(s) Summary
Workflow: Claude code review tools list
\.github/workflows/claude-code-review.yml
Extended claude_args.allowed-tools to include mcp__github_inline_comment__create_inline_comment at the start of the list; retained all existing Bash-based GH tool entries and order otherwise shifts accordingly.

Sequence Diagram(s)

sequenceDiagram
  participant Dev as Developer
  participant GH as GitHub Actions (claude-code-review)
  participant Claude as Claude Reviewer
  participant Tools as Allowed Tools
  participant PR as Pull Request

  Dev->>GH: Open/Update PR
  GH->>Claude: Invoke review with claude_args (allowed-tools list)
  Claude->>Tools: Select tools as needed
  alt Uses inline comment tool
    Claude->>PR: mcp__github_inline_comment__create_inline_comment (post inline comments)
  else Uses existing Bash-based GH tools
    Claude->>Tools: Invoke existing GH Bash tools
    Tools->>PR: Report results/status
  end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Add Claude Code GitHub Workflow #876 — Also modifies .github/workflows/claude-code-review.yml, changing the claude_args allowed-tools list; closely related to adding the inline comment tool.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly summarizes the primary change by stating that the MCP inline comment tool is being added to the Claude Code Review workflow, and it uses concise, descriptive phrasing consistent with conventional commit style.
Description Check ✅ Passed The description follows the repository template by providing a clear summary of the change and including the required checklist section with the test and lint commands, both of which are marked as completed.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.82%. Comparing base (0fd4bb5) to head (da781bf).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #884   +/-   ##
=======================================
  Coverage   88.82%   88.82%           
=======================================
  Files         109      109           
  Lines        7612     7612           
  Branches     1438     1438           
=======================================
  Hits         6761     6761           
  Misses        851      851           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
.github/workflows/claude-code-review.yml (1)

56-56: Align the reviewer prompt with the new inline-comment tool

We now allow mcp__github_inline_comment__create_inline_comment, but the prompt still instructs reviewers to “Use gh pr comment with your Bash tool.” That guidance conflicts with the newly enabled workflow and may cause confusion. Please update the prompt (or add a note) so reviewers know they can and should use the inline comment tool when appropriate.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 07a94c5 and a10cd42.

📒 Files selected for processing (1)
  • .github/workflows/claude-code-review.yml (1 hunks)

@yamadashy yamadashy force-pushed the chore/claude-code-review branch from a10cd42 to 10bb41a Compare October 10, 2025 15:23
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 10, 2025

Deploying repomix with  Cloudflare Pages  Cloudflare Pages

Latest commit: da781bf
Status: ✅  Deploy successful!
Preview URL: https://b4cf1514.repomix.pages.dev
Branch Preview URL: https://chore-claude-code-review.repomix.pages.dev

View logs

Add formatting instructions to the Claude Code Review workflow to improve
readability of PR comments. The review will now show a brief summary upfront
with detailed feedback collapsed in a <details> section.
@yamadashy yamadashy force-pushed the chore/claude-code-review branch from 10bb41a to da781bf Compare October 10, 2025 15:27
@yamadashy yamadashy merged commit 72f8ab5 into main Oct 10, 2025
47 of 48 checks passed
@yamadashy yamadashy deleted the chore/claude-code-review branch October 10, 2025 15:34
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.

2 participants