fix: add .claude and script paths to docker build trigger

[keito4]

Summary

• Add .claude/** to CI trigger paths (skills.txt, plugins.txt, etc.)
• Add script/** to CI trigger paths (install-skills.sh, install-claude-plugins.sh)

This ensures that changes to agent skills and plugins will automatically trigger DevContainer image builds.

Test plan

• Merge this PR
• Verify docker-image.yml workflow is triggered
• Verify new image is published with skills

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Updated continuous integration workflow configuration to optimize build triggers.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request adds two path filters (.claude/** and script/**) to the Docker image GitHub Actions workflow's push trigger, ensuring the workflow is triggered when files in these directories are modified.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow Configuration .github/workflows/docker-image.yml	Added .claude/** and script/** to the push trigger path filters, enabling the Docker image workflow to respond to changes in these directories.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• feat: add agent skills for Supabase and Vercel tools #394: Updates CI push path filters to include .claude/** and script/**, directly relating to files added under .claude/ (agent skills).
• feat: add user-level Claude commands sync and automation scripts #238: Introduces files under .claude/ and script/ directories that this workflow now monitors through the new path filters.
• Add Claude Code GitHub Workflow #167: Adds Claude workflow files that are now targeted by the .claude/** path filter in the Docker image workflow.

Suggested labels

size/XS

Poem

🐰 Paths of wonder, triggers set,
.claude and script, no need to fret,
When code updates in folders deep,
Docker builds while workflows sleep!

🚥 Pre-merge checks | ✅ 3

✅ 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 accurately describes the main change: adding .claude and script paths to the docker build trigger in the workflow file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/docker-build-trigger-paths

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[claude]

プルリクエストレビュー

このPRは .claude と script パスをDocker イメージビルドのトリガーパスに追加するものです。

良い点

1. 論理的な変更: .claude/skills/skills.txt や .claude/plugins/plugins.txt の変更時に DevContainer イメージを自動再ビルドする必要があるため、この変更は理にかなっています
2. 明確な目的: PR説明が明確で、変更の理由と期待される動作が明記されています
3. 最小限の変更: 必要最小限の変更のみが含まれており、過度な変更がありません

コード品質

• コード品質: 問題なし（2行のみの追加）
• セキュリティ: 問題なし
• パフォーマンス: 影響なし

詳細な確認

Dockerfile との整合性確認

.devcontainer/Dockerfile を確認したところ、以下の箇所で該当ファイルが使用されています:

• Line 103-104: skills.txt と plugins.txt がコンテナにコピーされています
• Line 106-107: インストールスクリプトがコピーされています
• Line 118-121: skills.txt を使用してスキルがインストールされます
• Line 130-135: plugins.txt を使用してプラグインがインストールされます

これらのファイルはビルド時に重要な役割を果たすため、変更時に自動ビルドをトリガーすることは正しい設計です。

軽微な懸念点

1. トリガー範囲が広すぎる可能性

.claude には commands、agents、hooks、settings.json も含まれ、これらは Dockerfile:100-102 でコピーされていますが、変更頻度が高い可能性があります。

script には多数のシェルスクリプト（50ファイル以上）が含まれ、これらすべての変更でビルドがトリガーされます。

2. CI コスト増加の可能性

.claude と script のファイルは開発中に頻繁に変更される可能性があり、不要なイメージビルドが増える可能性があります。

推奨事項

より限定的なパス指定を検討することで、CIコストを最適化できる可能性があります。特に Dockerfile で COPY されているファイルに限定することで、無駄なビルドを避けられます。

ただし、もし .claude と script 配下のすべてのファイルがイメージに影響する場合は、現在の変更で問題ありません。

結論

承認可能: この変更は技術的に正しく、必要な機能を実装しています。

---

品質スコア:

• コード品質: 合格
• セキュリティ: 合格
• パフォーマンス: CI頻度増加の可能性（許容範囲内）
• テストカバレッジ: N/A（ワークフロー変更のため）

レビュー by Claude Sonnet 4.5