docs: o3/Linear の古い参照を完全削除し壊れたリンクを修正 - #494
Conversation
o3 MCP / Linear MCP は既に削除済みのため、テンプレートと credentials/README.md からも OPENAI_API_KEY / LINEAR_API_KEY の 参照を削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
setup-mcp.sh の OPENAI_API_KEY チェック・警告と、 lib/config.sh の MCP インポート時の o3/Linear 案内、 エクスポート時の sk- プレフィックス置換を削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Linear MCP サーバーは使用停止済みのため、グローバルパッケージ 定義から削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Linear MCP は削除済みのため、linear.app ドメインへの WebFetch 許可も不要になった。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- o3/Linear 関連の言及を削除（.codex 説明、Vault 構造、 手動セットアップ例、Glossary） - setup-recommended-ci の古いパス参照を /setup-ci に更新 - devcontainer-recommendations のパスを .codex → .claude に修正 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- MCP 設定例から o3 エントリと OPENAI_API_KEY 環境変数を削除 - o3 MCP トラブルシューティングセクションを削除 - 参考リンクから o3 Search MCP を削除 - チェックリストの環境変数確認から OPENAI_API_KEY を削除 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Linear MCP は使用停止済みのため、ツールカタログの MCP/Automation セクションからも削除する。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughThis PR removes references to o3 and Linear MCP server integrations, along with their associated API keys (OPENAI_API_KEY, LINEAR_API_KEY) from configuration files, templates, scripts, documentation, and npm dependencies. The changes consolidate the MCP setup around remaining integrations like n8n and Playwright. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested labels
Poem
🚥 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 |
🔍 AI Code Review (Local Hook)Models: Codex (default) / Gemini (default) 🤖 Codex Review指摘事項（重大度順）
判定: patch is incorrect — エクスポート処理でAPIキーのマスキングがなくなり、セキュリティ上の退行があるため。信頼度 0.31
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 96c2f08d0e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # APIキーをプレースホルダーに置換 | ||
| sed -E 's/"(sk-[a-zA-Z0-9]+)"/"${OPENAI_API_KEY}"/g' "$source_file" > "$target_file" | ||
| echo "✅ Exported .mcp.json (API keys replaced with placeholders)" | ||
| cp "$source_file" "$target_file" |
There was a problem hiding this comment.
config::export_mcp now copies ~/.mcp.json directly into the tracked repo file without any redaction, so running ./script/export.sh on a workstation that stores MCP tokens in env/headers will write real credentials into .mcp.json and makes accidental secret commits likely. This is a regression from the prior export behavior that replaced API-key-like values before writing the file.
Useful? React with 👍 / 👎.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/devcontainer-recommendations.md (1)
790-791: Nit: UUOC in example command.♻️ Suggested simplification
- cat .mcp.json | grep '\${' + grep '\${' .mcp.json🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/devcontainer-recommendations.md around lines 790 - 791, Replace the useless use of cat in the example command: instead of piping "cat .mcp.json | grep '\${'" call grep directly on the file (grep '\${' .mcp.json) so the example uses "grep '\${' .mcp.json" while preserving the existing comment "# ${VAR_NAME}が残っている場合は未展開".
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude/devcontainer-recommendations.md:
- Around line 790-791: Replace the useless use of cat in the example command:
instead of piping "cat .mcp.json | grep '\${'" call grep directly on the file
(grep '\${' .mcp.json) so the example uses "grep '\${' .mcp.json" while
preserving the existing comment "# ${VAR_NAME}が残っている場合は未展開".
.prettierignore から CLAUDE.md の除外設定が欠落していたため、 Prettier が symlink ファイルに末尾改行を追加し、CI 環境 （core.symlinks=false）で壊れたリンクとなる問題を修正。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Claude Code レビュー概要PR #492, #493 に続く整合性確保のクリーンアップとして、変更の目的は明確です。削除漏れの参照を一括処理しており、全体的に正しい方向性です。 セキュリティ懸念（要確認）
Codex レビューが指摘した点と同様ですが、掘り下げて確認します。 変更前は OpenAI の ただし、この関数の用途（エクスポート先は何か）によっては、 確認してほしい点：エクスポート先が git-tracked なファイルになりえますか？もしそうであれば展開済みキーの流出リスクがあります。git-ignored なファイル間のコピーのみであれば、実質的なリスク増加はないと判断できます。 軽微な指摘
CodeRabbit が指摘済みですが同意。 良い点
判定
|
|
🎉 This PR is included in version 1.80.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
1 participant
Summary
OPENAI_API_KEY/LINEAR_API_KEY参照を削除setup-mcp.sh/lib/config.shから o3/Linear 関連のチェック・案内を削除npm/global.jsonから@mseep/linear-mcpエントリを削除.claude/settings.json/.devcontainer/claude-settings.jsonからlinear.appWebFetch 許可を削除README.mdの壊れたリンク修正（setup-recommended-ci→/setup-ci、devcontainer-recommendationsパス修正）.claude/devcontainer-recommendations.mdから o3 設定例・トラブルシューティングを削除.claude/commands/devcontainer-checklist.mdからOPENAI_API_KEYチェック項目を削除docs/tool-catalog.mdから Linear MCP エントリを削除Context
PR #492, #493 で
.mcp.json/ CLAUDE.md / settings から o3/Linear を削除したが、credentials テンプレート・スクリプト・README 等に古い参照が多数残存していた。Test plan
npm run format:check通過npm run lint通過npm test通過（101 tests）grep -rでOPENAI_API_KEY/LINEAR_API_KEY/linear-mcpの残存ゼロ確認（git-ignored ファイルを除く）🤖 Generated with Claude Code
Summary by CodeRabbit
Bug Fixes
Documentation
Chores