feat: add vercel-labs/agent-skills for AI development - #386
Conversation
Install vercel-labs/agent-skills with 4 skills: - vercel-composition-patterns: React composition patterns - vercel-react-best-practices: React performance best practices - vercel-react-native-skills: React Native development skills - web-design-guidelines: Web design guidelines Update Dockerfile to include skills in DevContainer image. Add .prettierignore to exclude external skill files from formatting. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughThis pull request establishes an automated skill installation system for the devcontainer environment. It introduces a skills list file, a bash installation script, Dockerfile integration to execute the installation during container build, and gitignore entries to exclude agent skill directories from version control. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~13 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
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 |
PR Review: vercel-labs/agent-skills の追加概要このPRは、Vercel Labsが提供する4つのAIエージェントスキルを統合し、React/React Nativeの開発における品質向上を図るものです。 ✅ 良い点1. 適切なスキル選定
これらは実務で頻繁に必要となる領域をカバーしており、選定は適切です。 2. インフラ統合が正確
3. コミットメッセージの品質Conventional Commitsに準拠し、 4. テスト計画の完全性Test planの4項目すべてがチェック済みで、検証が適切に実施されています。
|
| 項目 | ステータス | 備考 |
|---|---|---|
| Quality Gates Pass | ✅ | pre-commitフック通過確認済み |
| ドキュメント更新 | README/CLAUDE.mdへのスキル利用方法追記を推奨 | |
| モニタリング閾値内で安定 | N/A | 設定変更のためモニタリング対象外 |
| リリースノート記載 | ✅ | PR本文が十分なリリースノートとして機能 |
🎯 総合評価
判定: ✅ Approve with Minor Suggestions
このPRは技術的に健全であり、マージ可能な状態です。ただし、以下の点を考慮することで、さらに品質を向上できます:
- Must Fix(なし): ブロッカーとなる問題はありません
- Should Fix(推奨):
- スキルの実動作検証を実施
- ドキュメントに利用方法を追記
- Nice to Have(任意):
- Dockerイメージサイズへの影響確認
- 将来的な外部依存の管理戦略検討
マージ推奨: このPRはマージして問題ありません。上記の改善提案は、フォローアップIssueとして別途対応することも可能です。
📚 参考資料
🤖 レビュー実施: Claude Sonnet 4.5
Add skills.txt and install-skills.sh similar to the plugin system: - .claude/skills/skills.txt: List of skills to install - script/install-skills.sh: Script to install skills from the list - Dockerfile: Copy skills.txt and install-skills.sh, create symlinks Skills installation mechanism: - skills.txt lists GitHub repositories to install - install-skills.sh can be run manually or during startup - Pre-installed skills are copied from .agents/skills/ - Symlinks are created from .claude/skills/ to .agents/skills/ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
| Size | Lines | Files |
|---|---|---|
| XS | ≤50 | ≤3 |
| S | ≤200 | ≤10 |
| M | ≤500 | ≤15 |
| L | ≤1000 | ≤30 |
| XL | >1000 | >30 |
Change from pre-installed skills to dynamic installation: - Remove pre-installed skill files from repository - Update Dockerfile to run install-skills.sh during build - Add .agents/skills/ and related paths to .gitignore Skills are now installed during Docker build using: npx skills add <repo> -y -g This approach: - Keeps repository clean (no external files committed) - Always gets latest skill versions during build - Same pattern as plugin installation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
PR Review: Skills インストール機構の追加概要vercel-labs/agent-skills を動的にインストールする仕組みを追加。プラグインと同じパターンで実装されており、アーキテクチャの一貫性が保たれています。 良い点
改善提案1. テストカバレッジ不足(重要度: 高)install-skills.sh に対応する BATS テストが存在しません。CLAUDE.md で要求される 70%+ カバレッジ基準を満たすため、test/integration/install_skills.bats を追加してください。 参考: test/integration/install_claude_plugins.bats 2. Docker ビルド失敗時の影響(重要度: 中)Dockerfile:118-121 でスキルインストール失敗を警告のみで継続していますが、ユーザーが気づかない可能性があります。README にトラブルシューティングセクションの追加を推奨します。 3. ドキュメント不足(重要度: 低).claude/skills/skills.txt の使用方法が README に記載されていません。スキル追加手順の追加を推奨します。 コード品質チェック
マージ前の推奨アクション必須(ブロッカー): test/integration/install_skills.bats を追加(TDD 基準準拠) 推奨(非ブロッカー): Dockerfile 失敗ハンドリングの改善、README へのスキル追加手順記載 総合評価推奨判定: ✅ 条件付き承認 BATS テスト追加後にマージ可能。その他の改善提案は後続 PR で対応可。 🤖 Reviewed by Claude Sonnet 4.5 |
|
🎉 This PR is included in version 1.57.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
vercel-labs/agent-skillsをデフォルトでインストールSkills インストール機構
プラグイン(
plugins.txt+install-claude-plugins.sh)と同じパターンで、スキルの動的インストール仕組みを提供:.claude/skills/skills.txtscript/install-skills.shnpx skills addを実行してスキルをインストール.gitignore.agents/skills/を除外(ビルド時に生成)Dockerfile での動作
スキルの追加方法
.claude/skills/skills.txtにリポジトリを追加DevContainerを再ビルドするとスキルがインストールされる
手動インストール(ローカル)
インストールされるスキル
vercel-labs/agent-skillsに含まれる4つのスキル:Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes
New Features
Chores
✏️ Tip: You can customize this high-level summary in your review settings.