fix: enable all Claude Code plugins and resolve claude.yml conflicts - #650
Conversation
…licts プロジェクトの enabledPlugins に全プラグインを追加し、他リポジトリでもデフォルトで有効化されるようにする。 また claude.yml に残っていたマージコンフリクトマーカーを解消(sender.type による Bot 判定と詳細な concurrency group を採用)。 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughConfiguration updates to enable additional Claude plugins (swift-lsp, code-templates, code-workflows, and code-plugins) in Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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 correct 補足: CIやテストの実行確認は行っていません。 🤖 Generated by post_pr_ai_review.py hook |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.claude/settings.json (1)
280-298: Consider grouping plugins by source for maintainability.The plugins from
@claude-code-templatesare currently split across the list (lines 283-284 and 296-297). Grouping plugins by their source would make it easier to audit and update configurations in the future.♻️ Optional: Reorganize plugins by source
"enabledPlugins": { "typescript-lsp@claude-plugins-official": true, "swift-lsp@claude-plugins-official": true, "devops-automation@claude-code-templates": true, "documentation-generator@claude-code-templates": true, + "nextjs-vercel-pro@claude-code-templates": true, + "supabase-toolkit@claude-code-templates": true, "backend-development@claude-code-workflows": true, "code-refactoring@claude-code-workflows": true, "database-design@claude-code-workflows": true, "database-migrations@claude-code-workflows": true, "full-stack-orchestration@claude-code-workflows": true, "javascript-typescript@claude-code-workflows": true, "kubernetes-operations@claude-code-workflows": true, "feature-dev@claude-code-plugins": true, "frontend-design@claude-code-plugins": true, "hookify@claude-code-plugins": true, - "security-guidance@claude-code-plugins": true, - "nextjs-vercel-pro@claude-code-templates": true, - "supabase-toolkit@claude-code-templates": true + "security-guidance@claude-code-plugins": true }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/settings.json around lines 280 - 298, The enabledPlugins list mixes entries from the same source (e.g., devops-automation@claude-code-templates, documentation-generator@claude-code-templates, nextjs-vercel-pro@claude-code-templates, supabase-toolkit@claude-code-templates) across the object; reorder the entries so plugins are grouped by their package source (group all `@claude-code-templates` together, all `@claude-code-workflows` together, all `@claude-plugins-official` together, etc.), keeping each existing plugin key (for example typescript-lsp@claude-plugins-official, devops-automation@claude-code-templates, documentation-generator@claude-code-templates, backend-development@claude-code-workflows, feature-dev@claude-code-plugins) unchanged to preserve behavior and avoid duplicates, and run a quick lint/JSON validation after reordering.
🤖 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/settings.json:
- Around line 280-298: The enabledPlugins list mixes entries from the same
source (e.g., devops-automation@claude-code-templates,
documentation-generator@claude-code-templates,
nextjs-vercel-pro@claude-code-templates, supabase-toolkit@claude-code-templates)
across the object; reorder the entries so plugins are grouped by their package
source (group all `@claude-code-templates` together, all `@claude-code-workflows`
together, all `@claude-plugins-official` together, etc.), keeping each existing
plugin key (for example typescript-lsp@claude-plugins-official,
devops-automation@claude-code-templates,
documentation-generator@claude-code-templates,
backend-development@claude-code-workflows, feature-dev@claude-code-plugins)
unchanged to preserve behavior and avoid duplicates, and run a quick lint/JSON
validation after reordering.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 05a652e7-35a8-42e5-a5ba-b079ae271cfe
📒 Files selected for processing (2)
.claude/settings.json.github/workflows/claude.yml
💤 Files with no reviewable changes (1)
- .github/workflows/claude.yml
|
🎉 This PR is included in version 1.107.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
enabledPluginsに全プラグイン(16個)を追加し、他リポジトリでもデフォルトで有効化claude.ymlに残っていたマージコンフリクトマーカーを解消sender.type != 'Bot'を採用(ハードコード不要)Test plan
claude plugins listが全て enabled であることclaude.ymlワークフローが正常に動作すること🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes