Skip to content

feat: Claude開発用プラグインを追加 - #184

Merged
keito4 merged 1 commit into
mainfrom
feat/add-claude-plugins
Dec 29, 2025
Merged

feat: Claude開発用プラグインを追加#184
keito4 merged 1 commit into
mainfrom
feat/add-claude-plugins

Conversation

@keito4

@keito4 keito4 commented Dec 29, 2025

Copy link
Copy Markdown
Owner

概要

Claude Code用の開発支援プラグインを.claude/plugins/plugins.txtに追加しました。

変更内容

以下のプラグインを追加:

  • code-review-ai: AIによるコードレビュー支援
  • kubernetes-operations: Kubernetes操作サポート
  • javascript-typescript: JavaScript/TypeScript開発サポート
  • wshobson/agents: エージェント機能
  • playwright-skill: Playwright自動化支援

テスト

  • pre-commitフック通過
  • Lintチェック通過
  • テスト通過

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Five new plugins are now available: code review AI, Kubernetes operations, JavaScript/TypeScript tools, agents, and Playwright testing support.
    • Playwright testing support is additionally registered as a new marketplace entry for easy installation.
  • Other

    • No removals or breaking changes detected; existing workflows remain compatible.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Dec 29, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Added five plugin entries to .claude/plugins/plugins.txt and a new marketplace entry playwright-skill to .claude/plugins/known_marketplaces.json (new top-level key with source, installLocation, lastUpdated). No deletions or other structural edits.

Changes

Cohort / File(s) Summary
Plugin list
\.claude/plugins/plugins.txt
Added 5 new plugin lines: code-review-ai@claude-plugins-official, kubernetes-operations@claude-plugins-official, javascript-typescript@claude-plugins-official, agents@claude-code-workflows, playwright-skill@playwright-skill
Marketplace metadata
\.claude/plugins/known_marketplaces.json
Added new top-level marketplace playwright-skill with fields: source: { source: "github", repo: "lackeyjb/playwright-skill" }, installLocation: "/home/vscode/.claude/plugins/marketplaces/playwright-skill", lastUpdated: "2025-12-29T02:35:00.000Z"

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

released

Poem

🐇 I nibbled lines and planted a key,

Playwright doors and agents joined the tree.
JS, reviews, clusters in a row,
Little plugins sprout and softly glow,
A rabbit hops — new tools for thee. 🎩✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding Claude development plugins to the plugins configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 34558fc and cdb1355.

📒 Files selected for processing (2)
  • .claude/plugins/known_marketplaces.json
  • .claude/plugins/plugins.txt

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.

@claude

claude Bot commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

PR レビュー結果

このPRについてコード品質、セキュリティ、ベストプラクティスの観点からレビューを実施しました。

✅ 良い点

  1. 明確なコミットメッセージ: Conventional Commitsに準拠したfeat:タイプを使用しており、リポジトリのリリース要件(CLAUDE.md)に適合しています
  2. 適切なドキュメント: PR本文に変更内容とテスト結果が明記されています
  3. 基本的な品質ゲート通過: pre-commit、lint、テストが通過しています

⚠️ 潜在的な問題

1. マーケットプレイス参照の不整合(重大度: 中)

以下のプラグイン参照に問題があります:

# 問題のある行
+wshobson/agents@marketplace
+playwright-skill@playwright-skill

問題点:

  • known_marketplaces.json(.claude/plugins/known_marketplaces.json:28-33)にはclaude-code-workflowsとしてwshobson/agentsが登録されているため、正しくは:
    wshobson/agents@claude-code-workflows
    
  • playwright-skill@playwright-skillはマーケットプレイス名が不明確です。known_marketplaces.jsonに該当するマーケットプレイスが存在しません

影響:

  • インストールスクリプト(script/install-claude-plugins.sh:71)のclaude plugin installコマンドが失敗する可能性があります
  • DevContainer起動時にプラグインが正しくインストールされません

推奨修正:

-wshobson/agents@marketplace
+# Agents and Workflow Automation
+# (already covered by claude-code-workflows marketplace)

-playwright-skill@playwright-skill
+playwright-skill@claude-plugins-official
# または該当マーケットプレイスが存在する場合、そのマーケットプレイス名を使用

2. 重複の可能性(重大度: 低)

wshobson/agentsは既にマーケットプレイス全体として登録されているため、個別プラグインとして追加する必要性を確認してください。

3. コメントの言語一貫性(重大度: 低)

-# === Additional Plugins ===
-# AI and Code Review
+# === 追加プラグイン ===
+# AIとコードレビュー

既存のセクション見出しが日本語ではないため、一貫性のため英語コメントも許容範囲ですが、.claude/CLAUDE.mdでは日本語を推奨しています。

🔒 セキュリティ

  • ✅ 資格情報の漏洩なし
  • ✅ 外部リソースの参照は既知のマーケットプレイスのみ

📊 テストカバレッジ

このPRはプラグインリストファイルの変更のため、直接的なテストカバレッジは該当しませんが、以下の検証を推奨します:

# DevContainer内で実際にプラグインがインストールできるか確認
make claude-plugins

🎯 推奨アクション

  1. 必須: マーケットプレイス参照を修正

    • wshobson/agents@marketplacewshobson/agents@claude-code-workflowsまたは削除
    • playwright-skill@playwright-skill → 正しいマーケットプレイス名に修正、または該当マーケットプレイスをknown_marketplaces.jsonに追加
  2. 推奨: playwright-skillが必要な場合、known_marketplaces.jsonへの追加手順をドキュメント化

  3. 推奨: CI/CDパイプラインにプラグインインストールの検証ステップを追加

    - name: Verify plugin installation
      run: script/install-claude-plugins.sh .claude/plugins/plugins.txt

📝 総評

プラグイン追加という目的は明確ですが、マーケットプレイス参照の不整合により実際のインストールが失敗する可能性が高いです。上記の修正を適用してから再度レビューすることを推奨します。


🤖 このレビューはClaude Code AIによって生成されました

@keito4
keito4 force-pushed the feat/add-claude-plugins branch from 1f663f5 to 34558fc Compare December 29, 2025 02:35

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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)
.claude/plugins/plugins.txt (1)

25-35: Consider reorganizing the new plugins into categorized subsections.

The new "Additional Plugins" section breaks from the existing organizational structure (Official Plugins, Template Plugins, Workflow Plugins). To maintain consistency and improve discoverability, consider:

  • Grouping these plugins under the appropriate existing category (e.g., JavaScript/TypeScript plugin under a development-tools subsection)
  • Or, creating more descriptive subsection names that align with the existing taxonomy (e.g., "Development & Code Review Plugins" or "DevOps Plugins")

This is a minor organizational improvement—the current structure is functional but could be clearer as the plugin list grows.

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6c69a71 and 1f663f5.

📒 Files selected for processing (1)
  • .claude/plugins/plugins.txt
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: keito4/config PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-01T03:45:17.253Z
Learning: Applies to .github/workflows/claude.yml : Trigger automatic AI assistance on claude mentions in issues, PRs, and comments using .github/workflows/claude.yml
📚 Learning: 2025-12-01T03:45:17.253Z
Learnt from: CR
Repo: keito4/config PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-01T03:45:17.253Z
Learning: Applies to .github/workflows/claude.yml : Trigger automatic AI assistance on claude mentions in issues, PRs, and comments using .github/workflows/claude.yml

Applied to files:

  • .claude/plugins/plugins.txt
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: claude-review

@claude

claude Bot commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

PR Review: Claude開発用プラグインを追加

📋 概要

このPRは5つのClaude Codeプラグインを.claude/plugins/plugins.txtに追加するものです。コードレビュー、Kubernetes操作、JavaScript/TypeScript開発、エージェント機能、Playwright自動化の各プラグインが含まれています。


✅ 良い点

  1. 適切なコミット規約

    • Conventional Commits形式(feat:)を正しく使用
    • Release Notes記載も適切
    • Co-Authored-By でClaude連携を明示
  2. プロジェクトとの整合性

    • Kubernetes: 既存のVS Code拡張(ms-kubernetes-tools.vscode-kubernetes-tools)やCLIツール(kubernetes-cli)と整合
    • JavaScript/TypeScript: プロジェクトの主要技術スタック(Node.js、npm、ESLint、Prettier)に合致
    • Playwright: 既存のE2Eテストフレームワークの候補として適切
  3. シンプルな変更範囲

    • 12行の追加のみで影響範囲が限定的
    • 既存設定への影響なし
  4. 適切なコメント構造

    • セクション分けが明確(# === Additional Plugins ===
    • 各プラグインに説明コメント付き

⚠️ 懸念事項・改善提案

1. プラグインのマーケットプレイス指定の不整合

問題: 追加された5つのプラグインすべてが以下のように指定されていますが、マーケットプレイス指定に矛盾があります:

```txt

現在の指定

code-review-ai@claude-plugins-official
kubernetes-operations@claude-plugins-official # ❌
javascript-typescript@claude-plugins-official # ❌
agents@claude-code-workflows # ❌
playwright-skill@claude-plugins-official
```

根拠: .claude/settings.jsonenabledPlugins セクションを確認すると:

```json
"enabledPlugins": {
"kubernetes-operations@claude-code-workflows": true, // workflows指定
"javascript-typescript@claude-code-workflows": true, // workflows指定
...
}
```

推奨修正:
```txt

=== Additional Plugins ===

AI and Code Review

code-review-ai@claude-plugins-official

Kubernetes

kubernetes-operations@claude-code-workflows

JavaScript/TypeScript

javascript-typescript@claude-code-workflows

Agents

agents@claude-code-workflows

Playwright

playwright-skill@claude-plugins-official
```

影響: マーケットプレイス指定が間違っていると、プラグインのインストールに失敗する可能性があります。


2. agents プラグインの正確性確認が必要

問題: PR説明では wshobson/agents と記載されていますが、実際のファイルでは agents@claude-code-workflows となっています。

確認事項:

  • wshobson/agents は本当に claude-code-workflows マーケットプレイスで提供されているか?
  • それとも別のカスタムリポジトリからのインストールが必要か?

推奨: プラグインのソースを明確にし、必要に応じて .claude/plugins/known_marketplaces.json の更新も検討してください。


3. テストカバレッジの欠如

問題: CLAUDE.mdで定義されている品質基準との不一致:

```

CLAUDE.md より

TDD原則: Red → Green → Refactor
基準: 全リポジトリ 70%+ 行カバレッジ
```

現状:

  • プラグイン設定ファイルの変更に対するテストが存在しない
  • test/ディレクトリには credential-filtering.test.jsconfig-validation.test.js のみ

推奨:

  • プラグイン設定の妥当性を検証するテストを追加(例: マーケットプレイス指定の形式チェック、重複チェックなど)
  • または、設定ファイル変更のみの場合はテスト不要と判断した理由をPR説明に明記

:
```javascript
// test/plugin-validation.test.js
describe('Plugin configuration', () => {
test('should have valid marketplace references', () => {
// plugins.txtのマーケットプレイス指定が正しいか検証
});

test('should not have duplicate plugin entries', () => {
// 重複プラグインがないか検証
});
});
```


4. セキュリティ観点

低リスクではありますが確認推奨:

  • 追加されたプラグインは公式マーケットプレイスから提供されているため、基本的には安全
  • ただし、code-review-ai はコード解析を行うため、機密情報の取り扱いに注意
  • プラグインのアクセス権限を .claude/settings.jsonpermissions セクションで適切に制限することを推奨

5. ドキュメント更新の推奨

推奨事項:

  • README.md の「Available Plugins」セクションに新しいプラグインの説明を追加
  • または .claude/plugins/README.md に利用用途・目的を記載

:
```markdown

新規追加プラグイン

  • code-review-ai: AIによるコードレビュー支援。PRレビュー時の品質チェックを自動化
  • kubernetes-operations: K8sクラスタの操作をClaude経由で実行
  • javascript-typescript: JS/TS開発時のコード生成・リファクタリング支援
  • agents: エージェントベースのタスク自動化
  • playwright-skill: E2Eテスト自動生成・メンテナンス支援
    ```

🔍 パフォーマンス考慮事項

影響: プラグイン数の増加による起動時間への影響は軽微と予想されますが、以下を監視することを推奨:

  • DevContainer起動時のプラグインインストール時間
  • Claude Code起動時の初期化時間
  • 各プラグインのメモリ使用量

📝 推奨アクションアイテム

必須(マージ前)

  1. マーケットプレイス指定を修正(kubernetes-operations、javascript-typescriptをclaude-code-workflowsに変更)
  2. agentsプラグインのソース確認wshobson/agentsとの関係を明確化)

推奨(マージ後でも可)

  1. 🔄 プラグイン設定の妥当性テスト追加
  2. 📖 READMEまたはプラグインREADMEにドキュメント追加
  3. 🔐 セキュリティ観点でのpermissions設定レビュー

🎯 総合評価

判定: ⚠️ 条件付き承認(修正推奨)

プラグイン追加自体は妥当ですが、マーケットプレイス指定の不整合があるため、マージ前の修正を強く推奨します。また、TDD原則との整合性を保つため、可能であればプラグイン設定の妥当性テストの追加を検討してください。


参考情報

  • 既存のKubernetes関連設定: `vscode/extensions.txt:36` (ms-kubernetes-tools.vscode-kubernetes-tools)
  • settings.jsonの該当箇所: `.claude/settings.json:112-113`
  • CLAUDE.md開発基準: `.claude/CLAUDE.md:12-14` (TDD要件)

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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)
.claude/plugins/plugins.txt (1)

25-35: Consider reorganizing plugins into existing logical sections.

The new "Additional Plugins" section contains plugins that could logically belong in existing categories. For example:

  • code-review-ai, kubernetes-operations, and javascript-typescript appear to be general-purpose plugins that could be grouped under "Official Plugins"
  • agents already belongs in the "Workflow Plugins" section and could be moved there

Reorganizing would improve discoverability and maintainability. Alternatively, consider renaming "Additional Plugins" to better describe the intended purpose of these plugins (e.g., "Specialized Development Tools").

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1f663f5 and 34558fc.

📒 Files selected for processing (1)
  • .claude/plugins/plugins.txt
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: keito4/config PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-01T03:45:17.253Z
Learning: Applies to .github/workflows/claude.yml : Trigger automatic AI assistance on claude mentions in issues, PRs, and comments using .github/workflows/claude.yml
📚 Learning: 2025-12-01T03:45:17.253Z
Learnt from: CR
Repo: keito4/config PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-01T03:45:17.253Z
Learning: Applies to .github/workflows/claude.yml : Trigger automatic AI assistance on claude mentions in issues, PRs, and comments using .github/workflows/claude.yml

Applied to files:

  • .claude/plugins/plugins.txt
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: claude-review
🔇 Additional comments (2)
.claude/plugins/plugins.txt (2)

34-35: Clarify the playwright-skill plugin reference format.

The comment on line 34 references lackeyjb/playwright-skill in GitHub-style format, but line 35 specifies playwright-skill@claude-plugins-official. Verify that this is the correct and authoritative plugin reference from the marketplace, and update or remove the comment if it's misleading.


27-35: Verify plugin existence and compatibility with Claude Code.

Ensure that all newly added plugins:

  1. Exist and are available in their respective marketplaces (claude-plugins-official, claude-code-workflows)
  2. Are compatible with your Claude Code environment
  3. Don't introduce any conflicts with existing plugins

Since the PR summary indicates that pre-commit, lint, and test checks passed, basic validation appears sound. However, marketplace availability and functional compatibility should be confirmed.

以下のプラグインを.claude/plugins/plugins.txtに追加:
- code-review-ai: AIによるコードレビュー支援
- kubernetes-operations: Kubernetes操作サポート
- javascript-typescript: JS/TS開発サポート
- agents (wshobson/agents): エージェント機能
- playwright-skill (lackeyjb/playwright-skill): Playwright自動化支援

また、playwright-skillマーケットプレイスをknown_marketplaces.jsonに追加

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@keito4
keito4 force-pushed the feat/add-claude-plugins branch from 34558fc to cdb1355 Compare December 29, 2025 02:39
@keito4
keito4 merged commit 6c61336 into main Dec 29, 2025
3 of 4 checks passed
@claude

claude Bot commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

プルリクエストレビュー

このPRは、Claude Code用の開発支援プラグインを追加するもので、全体として良好な品質です。以下、詳細なレビュー結果を報告します。


✅ 良い点

1. コミット規約の遵守

  • feat: タイプを使用しており、Conventional Commits に準拠
  • コミットメッセージが明確で、変更内容を適切に表現

2. ドキュメント整合性

  • PR本文に変更内容が明確に記載されている
  • 追加されたプラグインの目的が明記されている

3. 既存の自動化との統合

  • .claude/plugins/plugins.txt はDockerfileの COPY 命令で参照されており(Dockerfile:78)、devcontainerビルド時に自動的に反映される
  • script/install-claude-plugins.sh がプラグインリストを読み込んで自動インストールを実行

4. マーケットプレイス設定

  • known_marketplaces.jsonplaywright-skill マーケットプレイスが適切に追加されている

🔍 懸念点と改善提案

1. リリースタイプの妥当性(重要度: 中)

現在の変更は .claude/plugins/plugins.txt.claude/plugins/known_marketplaces.json のみですが、commitlintの設定(commitlint.config.js)を確認すると、以下のパターンはリリース型コミット(feat|fix|perf|revert|docs)を要求します:

const releaseSensitivePatterns = [
  /^\.codex\//,
  /^\.devcontainer\/codex-config\.json$/,
  // ...
];

今回のファイルパスは releaseSensitivePatterns に含まれていないため、技術的には chore: でも問題ありませんでした。しかし、feat: を選択したことで semantic-release による自動リリースが発生します。

推奨事項:

  • プラグイン追加がユーザー向けの新機能と見なせる場合は feat: が適切
  • 内部的な開発環境の改善に留まる場合は、今後は chore: も検討可能
  • 今回は feat: で問題なし

2. プラグインの重複可能性(重要度: 低)

javascript-typescript@claude-plugins-official が追加されていますが、既存のプラグインリストには以下が存在します:

typescript-lsp@claude-plugins-official  # 17行目

確認事項:

  • 両者の役割が異なるか確認(LSP vs 開発ワークフロー)
  • 機能が重複する場合は統合を検討

3. テストカバレッジ(重要度: 低)

現在、プラグイン設定ファイルに対する自動テストが存在しません。

推奨事項:

  • plugins.txt のフォーマット検証テストを追加
    • 各行が plugin-name@marketplace-name 形式か
    • 参照されているマーケットプレイスが known_marketplaces.json に存在するか
  • CI パイプラインに組み込むことで品質保証を強化

参考実装案:

// test/plugins-validation.test.js
const fs = require('fs');
const pluginsList = fs.readFileSync('.claude/plugins/plugins.txt', 'utf8');
const knownMarketplaces = JSON.parse(
  fs.readFileSync('.claude/plugins/known_marketplaces.json', 'utf8')
);

pluginsList.split('\\n')
  .filter(line => line && \!line.startsWith('#'))
  .forEach(line => {
    const [, marketplace] = line.split('@');
    assert(
      marketplace in knownMarketplaces,
      `Unknown marketplace: ${marketplace}`
    );
  });

4. セキュリティ考慮(重要度: 低)

外部プラグイン(特に lackeyjb/playwright-skill)を追加する際は、以下を確認することを推奨します:

  • リポジトリのメンテナンス状態(最終更新日、Issue対応状況)
  • 依存関係の脆弱性スキャン
  • プラグインのコードレビュー(特に認証情報へのアクセス権限)

📊 コード品質評価

項目 評価
Git規約準拠 ✅ 優秀
ドキュメント ✅ 良好
自動化統合 ✅ 良好
テストカバレッジ ⚠️ なし(改善余地あり)
セキュリティ ⚠️ 外部プラグイン確認推奨

🎯 結論

承認推奨

このPRは以下の理由で承認可能です:

  1. 既存の品質基準(Conventional Commits、ドキュメント)を満たしている
  2. 既存の自動化システムと適切に統合されている
  3. 指摘した懸念点は致命的ではなく、将来の改善として対応可能

次回PR作成時の改善提案:

  • プラグイン検証テストの追加を検討
  • 外部プラグインのセキュリティ確認プロセスを文書化

📚 参考情報

  • Dockerfile:78 - プラグインリストのコピー先
  • script/install-claude-plugins.sh - プラグイン自動インストールスクリプト
  • commitlint.config.js:8-14 - リリース感知パターンの定義

🤖 Review generated by Claude Code

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.10.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released リリース済み

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant