Introduce CodeGraph integration#932
Conversation
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
📝 WalkthroughWalkthroughCodeGraphツールの統合を実施しました。 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9f3478df-0788-4081-95b9-36fb9a044ed9
⛔ Files ignored due to path filters (1)
not_config/memo/codegraph.mdis excluded by!not_config/**
📒 Files selected for processing (4)
dot_claude/settings.jsondot_config/rulesync/exact_dot_rulesync/mcp.json.tmpldot_config/rulesync/exact_dot_rulesync/rules/CODEGRAPH.mdmise.toml
| "Bash(shfmt:*)", | ||
| "WebFetch(domain:docs.anthropic.com)" | ||
| "WebFetch(domain:docs.anthropic.com)", | ||
| "mcp__codegraph__*" |
There was a problem hiding this comment.
CodeGraph 権限がワイルドカードで広すぎます。
Line 16 の mcp__codegraph__* は将来追加される CodeGraph 側の全ツールまで許可対象になり、最小権限の原則から外れます。少なくとも現時点で運用対象のツール名へ明示的に絞ることを推奨します。
権限を明示列挙へ寄せる例
- "mcp__codegraph__*"
+ "mcp__codegraph__codegraph_search",
+ "mcp__codegraph__codegraph_callers",
+ "mcp__codegraph__codegraph_callees",
+ "mcp__codegraph__codegraph_impact",
+ "mcp__codegraph__codegraph_node"
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces CodeGraph to the repository, enabling advanced semantic code analysis. The changes include installing the necessary dependencies, configuring the tool for use with MCP, and establishing clear guidelines for AI agents to interact with the CodeGraph toolset efficiently without overloading the context window. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| "codegraph": { | ||
| "type": "stdio", | ||
| "command": "codegraph", | ||
| "args": ["serve", "--mcp"] | ||
| }, |
There was a problem hiding this comment.
このリポジトリ内の他の npm パッケージを利用した MCP サーバー設定（context7 や chrome-devtools-mcp など）のパターンに従い、npx を使用して実行することを推奨します。これにより、実行環境の PATH 設定に依存せず、mise.toml で指定されたバージョンと一致するバイナリを確実に実行できるようになり、再現性と保守性が向上します。
"codegraph": {
"type": "stdio",
"command": "npx",
"args": ["-y", "@colbymchenry/codegraph@0.7.3", "serve", "--mcp"]
},
References
- 保守性、安全性、再現性に直結する点に絞って指摘を行う。 (link)
| > **ルール:** | ||
| > | ||
| > 1. `codegraph_explore` のツール説明にある探索コール予算に従ってください。 | ||
| > 2. `codegraph_explore` が既にソースコードを返したファイルを再読回しないでください。 |
There was a problem hiding this comment.
「再読回」は「再読込」の誤字と思われます。ドキュメントの正確性のために修正を推奨します。
| > 2. `codegraph_explore` が既にソースコードを返したファイルを再読回しないでください。 | |
| > 2. `codegraph_explore` が既にソースコードを返したファイルを再読込しないでください。 |
1 participant
Integrated CodeGraph, a semantic code intelligence tool, into the repository. Updated mise.toml, MCP configuration, Claude Code settings, and added usage documentation and AI rules.
PR created automatically by Jules for task 12053809015488874666 started by @ryo246912
変更内容概要
CodeGraph（semantic code intelligence ツール）をリポジトリに統合するための以下の変更を実施しました：
dot_claude/settings.jsonにmcp__codegraph__*MCP 権限を追加mcp.json.tmplに codegraph MCP サーバー（stdio 型、codegraph serve --mcpコマンド）を追加CODEGRAPH.mdを新規作成し、主要セッション・Explore エージェント・初期化時の使用方法を定義mise.tomlに npm パッケージ@colbymchenry/codegraph@0.7.3を追加変更理由
CodeGraph を semantic code intelligence ツールとしてリポジトリに統合し、コード分析・探索機能を拡張するため。特に Explore エージェントとの連携により、コード理解と変更検討を効率化します。
確認した項目