@k2works/claude-code-booster 2.2.0 → 2.3.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/bin/claude-code-booster +11 -0
- package/lib/assets/.claude/README.md +15 -0
- package/lib/assets/.claude/skills/analyzing-review/SKILL.md +174 -0
- package/lib/assets/.claude/skills/developing-review/SKILL.md +175 -0
- package/lib/assets/.claude/skills/operating-review/SKILL.md +171 -0
- package/lib/assets/CLAUDE.md +8 -0
- package/package.json +1 -1
package/bin/claude-code-booster
CHANGED
|
@@ -52,6 +52,17 @@ async function setupProject() {
|
|
|
52
52
|
console.warn(`[claude-code-booster] Skip: source not found "${src}"`);
|
|
53
53
|
}
|
|
54
54
|
}
|
|
55
|
+
|
|
56
|
+
// ルート直下の設定ファイルも上書きコピー
|
|
57
|
+
const rootFiles = ['AGENTS.md', 'CLAUDE.md'];
|
|
58
|
+
for (const file of rootFiles) {
|
|
59
|
+
const src = path.join(templateDir, file);
|
|
60
|
+
const dest = path.join(projectDir, file);
|
|
61
|
+
if (await fs.pathExists(src)) {
|
|
62
|
+
await fs.copy(src, dest, { overwrite: true, errorOnExist: false });
|
|
63
|
+
console.log(`[claude-code-booster] Updated "${file}"`);
|
|
64
|
+
}
|
|
65
|
+
}
|
|
55
66
|
} else {
|
|
56
67
|
// それ以外は従来通り assets 全体をコピー
|
|
57
68
|
await fs.copy(templateDir, projectDir, { overwrite: true, errorOnExist: false });
|
|
@@ -54,6 +54,14 @@ Claude Code をより効率的に使うための基本設定テンプレート
|
|
|
54
54
|
| `developing-frontend` | フロントエンド開発の TDD ワークフロー。アウトサイドインアプローチ。 |
|
|
55
55
|
| `developing-release` | リリースワークフロー。品質ゲート、バージョンバンプ、CHANGELOG 生成、git commit + tag。 |
|
|
56
56
|
|
|
57
|
+
#### レビュー系
|
|
58
|
+
|
|
59
|
+
| スキル | 説明 |
|
|
60
|
+
| :--- | :--- |
|
|
61
|
+
| `analyzing-review` | 分析成果物のマルチパースペクティブレビュー。XP エージェント 5 名(PM、アーキテクト、ID、テスター、ユーザー代表)を並列起動。 |
|
|
62
|
+
| `developing-review` | 開発成果物のマルチパースペクティブレビュー。XP エージェント 5 名(プログラマー、テスター、アーキテクト、TW、ユーザー代表)を並列起動。 |
|
|
63
|
+
| `operating-review` | 運用成果物のマルチパースペクティブレビュー。XP エージェント 3 名(アーキテクト、テスター、PM)を並列起動。 |
|
|
64
|
+
|
|
57
65
|
#### 計画・進捗系
|
|
58
66
|
|
|
59
67
|
| スキル | 説明 |
|
|
@@ -72,6 +80,8 @@ Claude Code をより効率的に使うための基本設定テンプレート
|
|
|
72
80
|
| `operating-deploy` | 各環境へのデプロイを実行。ローリング、Blue/Green、ロールバック。 |
|
|
73
81
|
| `operating-backup` | データベースのバックアップ・リストアを実行。 |
|
|
74
82
|
| `operating-docs` | 設計ドキュメントの一覧表示、進捗確認、インデックス更新、Markdown Lint。 |
|
|
83
|
+
| `operating-qt` | SonarQube によるコード品質管理。スキャン実行、Quality Gate 確認、イシュー分析。 |
|
|
84
|
+
| `operating-script` | 運用スクリプト(Gulp タスク)の作成・更新。 |
|
|
75
85
|
| `killing-processes` | 開発サーバーや Node.js プロセスを強制終了。ポート競合の解決。 |
|
|
76
86
|
|
|
77
87
|
#### ドキュメント・Git 系
|
|
@@ -148,10 +158,15 @@ Claude Code をより効率的に使うための基本設定テンプレート
|
|
|
148
158
|
│ ├── analyzing-test-strategy/SKILL.md
|
|
149
159
|
│ ├── analyzing-non-functional/SKILL.md
|
|
150
160
|
│ ├── analyzing-operation/SKILL.md
|
|
161
|
+
│ ├── analyzing-review/SKILL.md
|
|
151
162
|
│ ├── developing-backend/SKILL.md
|
|
152
163
|
│ ├── developing-frontend/SKILL.md
|
|
153
164
|
│ ├── developing-release/SKILL.md
|
|
165
|
+
│ ├── developing-review/SKILL.md
|
|
154
166
|
│ ├── operating-setup/SKILL.md
|
|
167
|
+
│ ├── operating-qt/SKILL.md
|
|
168
|
+
│ ├── operating-script/SKILL.md
|
|
169
|
+
│ ├── operating-review/SKILL.md
|
|
155
170
|
│ ├── operating-provision/SKILL.md
|
|
156
171
|
│ ├── operating-cicd/SKILL.md
|
|
157
172
|
│ ├── operating-deploy/SKILL.md
|
|
@@ -0,0 +1,174 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: analyzing-review
|
|
3
|
+
description: 分析成果物のマルチパースペクティブレビューを実施。XP エージェント(プロダクトマネージャー、アーキテクト、インタラクションデザイナー、テスター、ユーザー代表)を並列起動し、専門的視点からフィードバックを収集・統合する。「分析ドキュメントをレビューしたい」「要件定義をレビューして」「アーキテクチャ設計のレビューをお願い」「設計ドキュメントにフィードバックがほしい」「分析結果を検証したい」「ユースケースをチェックして」といった場面で発動する。分析フェーズの成果物に対するレビュー依頼があれば積極的に使用すること。
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# 分析成果物レビュー
|
|
7
|
+
|
|
8
|
+
分析フェーズの成果物(要件定義、アーキテクチャ設計、データモデル、ドメインモデル、UI 設計、テスト戦略、非機能要件、運用要件など)を複数の XP エージェントで並列レビューし、多角的なフィードバックを統合する。
|
|
9
|
+
|
|
10
|
+
## レビューの価値
|
|
11
|
+
|
|
12
|
+
分析成果物のレビューは、開発フェーズに入る前に問題を発見する最も効果的な手段。1 人の視点では見落としがちな矛盾や不足を、異なる専門性を持つエージェントが同時に検証することで、手戻りコストを大幅に削減できる。
|
|
13
|
+
|
|
14
|
+
## レビューエージェント
|
|
15
|
+
|
|
16
|
+
各エージェントは独自の専門的視点でレビューを行う。全エージェントを並列起動し、独立したフィードバックを収集する。
|
|
17
|
+
|
|
18
|
+
| エージェント | 視点 | 着眼点 |
|
|
19
|
+
|:---|:---|:---|
|
|
20
|
+
| `xp-product-manager` | ビジネス価値 | 要件の完全性、ビジネス価値との整合性、ストーリーの INVEST 基準、優先順位の妥当性 |
|
|
21
|
+
| `xp-architect` | 技術的実現性 | アーキテクチャの妥当性、非機能要件の実現可能性、技術的リスク、変更容易性 |
|
|
22
|
+
| `xp-interaction-designer` | ユーザー体験 | UI/UX の一貫性、ユーザーの心理モデルとの整合、アクセシビリティ、画面遷移の自然さ |
|
|
23
|
+
| `xp-tester` | テスト可能性 | 要件のテスト可能性、受入基準の明確さ、エッジケースの網羅性、テスト戦略との整合 |
|
|
24
|
+
| `xp-user-representative` | 利用者視点 | 実際の業務での使いやすさ、業務フローとの整合性、ユーザーが本当に必要としている機能か |
|
|
25
|
+
|
|
26
|
+
## レビュー対象と重点エージェント
|
|
27
|
+
|
|
28
|
+
成果物の種類に応じて、特に重要なエージェントが異なる。ただし全エージェントを起動して多角的な視点を確保する。
|
|
29
|
+
|
|
30
|
+
| 成果物 | 重点エージェント |
|
|
31
|
+
|:---|:---|
|
|
32
|
+
| 要件定義・ユースケース | xp-product-manager, xp-user-representative, xp-tester |
|
|
33
|
+
| アーキテクチャ設計 | xp-architect, xp-tester |
|
|
34
|
+
| データモデル・ドメインモデル | xp-architect, xp-product-manager |
|
|
35
|
+
| UI 設計 | xp-interaction-designer, xp-user-representative |
|
|
36
|
+
| テスト戦略 | xp-tester, xp-architect |
|
|
37
|
+
| 非機能要件・運用要件 | xp-architect, xp-tester |
|
|
38
|
+
|
|
39
|
+
## レビューワークフロー
|
|
40
|
+
|
|
41
|
+
### 1. レビュー対象の特定
|
|
42
|
+
|
|
43
|
+
レビュー対象のドキュメントを特定する。ユーザーが明示的に指定しない場合は `docs/design/` 配下を確認し、対象を提案する。
|
|
44
|
+
|
|
45
|
+
### 2. エージェントの並列起動
|
|
46
|
+
|
|
47
|
+
Agent ツールを使い、5 つのエージェントを **同一メッセージで並列起動** する。各エージェントには以下を指示する。
|
|
48
|
+
|
|
49
|
+
```
|
|
50
|
+
あなたは {エージェント名} です。
|
|
51
|
+
.claude/agents/{エージェント名}.md の定義に従ってレビューしてください。
|
|
52
|
+
|
|
53
|
+
## レビュー対象
|
|
54
|
+
{ドキュメントパスとその内容の要約}
|
|
55
|
+
|
|
56
|
+
## レビュー対象の種類
|
|
57
|
+
{成果物カテゴリ: 要件定義 / アーキテクチャ設計 / データモデル / UI 設計 等}
|
|
58
|
+
|
|
59
|
+
## レビュー観点
|
|
60
|
+
{エージェント固有の着眼点}
|
|
61
|
+
|
|
62
|
+
## 出力形式
|
|
63
|
+
以下の形式でフィードバックを返してください:
|
|
64
|
+
|
|
65
|
+
### 評価サマリー
|
|
66
|
+
(1-2 文で全体評価)
|
|
67
|
+
|
|
68
|
+
### 良い点
|
|
69
|
+
- (具体的に)
|
|
70
|
+
|
|
71
|
+
### 改善提案
|
|
72
|
+
- 【重要度: 高/中/低】(具体的な改善提案と理由)
|
|
73
|
+
|
|
74
|
+
### 懸念事項
|
|
75
|
+
- (リスクや見落とされている観点)
|
|
76
|
+
|
|
77
|
+
### スコープ外の発見
|
|
78
|
+
- (レビュー対象外だが報告すべき問題)
|
|
79
|
+
```
|
|
80
|
+
|
|
81
|
+
#### Agent ツールが利用できない場合のフォールバック
|
|
82
|
+
|
|
83
|
+
Agent ツールが利用できない環境では、5 つの視点を逐次的にシミュレートする。各エージェントの定義ファイル(`.claude/agents/{エージェント名}.md`)を読み、その視点でレビューを順次実施する。出力形式は同一のテンプレートに従う。
|
|
84
|
+
|
|
85
|
+
### 3. フィードバックの統合
|
|
86
|
+
|
|
87
|
+
全エージェントのフィードバックを受け取った後、以下の形式で統合レポートを作成する。
|
|
88
|
+
|
|
89
|
+
```markdown
|
|
90
|
+
## レビュー結果
|
|
91
|
+
|
|
92
|
+
### レビュー対象
|
|
93
|
+
- {ドキュメント名}
|
|
94
|
+
|
|
95
|
+
### 総合評価
|
|
96
|
+
(全エージェントの評価を統合した 2-3 文の評価)
|
|
97
|
+
|
|
98
|
+
### 改善提案(重要度順)
|
|
99
|
+
|
|
100
|
+
#### 高
|
|
101
|
+
| # | 提案 | 指摘元 | 理由 |
|
|
102
|
+
|---|------|--------|------|
|
|
103
|
+
|
|
104
|
+
#### 中
|
|
105
|
+
| # | 提案 | 指摘元 | 理由 |
|
|
106
|
+
|---|------|--------|------|
|
|
107
|
+
|
|
108
|
+
#### 低
|
|
109
|
+
| # | 提案 | 指摘元 | 理由 |
|
|
110
|
+
|---|------|--------|------|
|
|
111
|
+
|
|
112
|
+
### 矛盾事項
|
|
113
|
+
|
|
114
|
+
複数エージェントの指摘が相反する場合、ここに記載する。
|
|
115
|
+
|
|
116
|
+
| # | 視点 A | 視点 B | 論点 | 推奨判断 |
|
|
117
|
+
|---|--------|--------|------|----------|
|
|
118
|
+
|
|
119
|
+
### エージェント別フィードバック詳細
|
|
120
|
+
|
|
121
|
+
<details>
|
|
122
|
+
<summary>xp-product-manager(高: N / 中: N / 低: N)</summary>
|
|
123
|
+
(フィードバック全文)
|
|
124
|
+
</details>
|
|
125
|
+
|
|
126
|
+
<details>
|
|
127
|
+
<summary>xp-architect(高: N / 中: N / 低: N)</summary>
|
|
128
|
+
(フィードバック全文)
|
|
129
|
+
</details>
|
|
130
|
+
|
|
131
|
+
<details>
|
|
132
|
+
<summary>xp-interaction-designer(高: N / 中: N / 低: N)</summary>
|
|
133
|
+
(フィードバック全文)
|
|
134
|
+
</details>
|
|
135
|
+
|
|
136
|
+
<details>
|
|
137
|
+
<summary>xp-tester(高: N / 中: N / 低: N)</summary>
|
|
138
|
+
(フィードバック全文)
|
|
139
|
+
</details>
|
|
140
|
+
|
|
141
|
+
<details>
|
|
142
|
+
<summary>xp-user-representative(高: N / 中: N / 低: N)</summary>
|
|
143
|
+
(フィードバック全文)
|
|
144
|
+
</details>
|
|
145
|
+
```
|
|
146
|
+
|
|
147
|
+
### 4. 改善アクションの提案
|
|
148
|
+
|
|
149
|
+
統合レポートに基づき、具体的な改善アクションを提案する。改善は重要度「高」から着手することを推奨する。
|
|
150
|
+
|
|
151
|
+
### 5. レビュー結果の保存
|
|
152
|
+
|
|
153
|
+
統合レポートを `docs/review/{対象名}_review_{YYYYMMDD}.md` に保存する。過去のレビュー結果と比較できるようにし、改善のトレーサビリティを確保する。
|
|
154
|
+
|
|
155
|
+
## レビュー完了条件
|
|
156
|
+
|
|
157
|
+
以下をすべて満たした場合にレビュー完了とする。
|
|
158
|
+
|
|
159
|
+
- 全エージェントのフィードバックが収集済み
|
|
160
|
+
- 統合レポートが作成済み
|
|
161
|
+
- 重要度「高」の指摘について対応方針(修正する / 許容する / 保留する)が決定済み
|
|
162
|
+
- レビュー結果がドキュメントとして保存済み
|
|
163
|
+
|
|
164
|
+
## 注意事項
|
|
165
|
+
|
|
166
|
+
- レビューは成果物の品質向上が目的であり、批判が目的ではない
|
|
167
|
+
- レビュー対象が複数ドキュメントにまたがる場合は、関連性の高いものをまとめてレビューする
|
|
168
|
+
- TDD サイクルの遵守状況(テストが先に書かれているか)もレビュー観点に含める
|
|
169
|
+
|
|
170
|
+
## 関連スキル
|
|
171
|
+
|
|
172
|
+
- `orchestrating-analysis` — 分析フェーズの全体ワークフロー
|
|
173
|
+
- `analyzing-*` — 各分析工程のスキル
|
|
174
|
+
- `git-commit` — レビュー結果の保存後のコミット
|
|
@@ -0,0 +1,175 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: developing-review
|
|
3
|
+
description: 開発成果物のマルチパースペクティブレビューを実施。XP エージェント(プログラマー、テスター、アーキテクト、テクニカルライター、ユーザー代表)を並列起動し、コード品質・テスト品質・設計整合性・ドキュメント品質・利用者視点からフィードバックを収集・統合する。「コードレビューして」「PR をレビューして」「実装をチェックして」「テストコードをレビューして」「この変更をレビューしてほしい」「コードの品質を確認して」といった場面で発動する。開発成果物に対するレビュー依頼があれば積極的に使用すること。
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# 開発成果物レビュー
|
|
7
|
+
|
|
8
|
+
開発フェーズの成果物(プロダクションコード、テストコード、PR、リファクタリング結果など)を複数の XP エージェントで並列レビューし、多角的なフィードバックを統合する。
|
|
9
|
+
|
|
10
|
+
## レビューの価値
|
|
11
|
+
|
|
12
|
+
コードレビューは単なるバグ発見の手段ではない。設計の妥当性、テストの十分性、将来の変更容易性、ドキュメントとの整合性、そして実際のユーザーにとっての価値を同時に検証することで、「変更を楽に安全にできて役に立つソフトウェア」を維持する。
|
|
13
|
+
|
|
14
|
+
## レビューエージェント
|
|
15
|
+
|
|
16
|
+
| エージェント | 視点 | 着眼点 |
|
|
17
|
+
|:---|:---|:---|
|
|
18
|
+
| `xp-programmer` | コード品質 | SOLID 原則、DRY、命名、可読性、TDD サイクルの遵守、シンプルな設計 |
|
|
19
|
+
| `xp-tester` | テスト品質 | テストカバレッジ、境界値、エッジケース、テストの信頼性、テストピラミッドのバランス |
|
|
20
|
+
| `xp-architect` | 設計整合性 | アーキテクチャとの整合、依存関係の方向、変更容易性、技術的負債の蓄積 |
|
|
21
|
+
| `xp-technical-writer` | ドキュメント品質 | API ドキュメント、コメントの適切さ、README の更新必要性、機能の理解しやすさ |
|
|
22
|
+
| `xp-user-representative` | 利用者視点 | 実装された機能がユーザーの期待に合っているか、業務フローとの整合、使い勝手 |
|
|
23
|
+
|
|
24
|
+
## レビュー対象の判定
|
|
25
|
+
|
|
26
|
+
ユーザーのリクエストに応じてレビュー範囲を決定する。
|
|
27
|
+
|
|
28
|
+
| リクエスト | レビュー範囲 |
|
|
29
|
+
|:---|:---|
|
|
30
|
+
| 「PR をレビュー」 | `git diff` で変更差分を取得 |
|
|
31
|
+
| 「このファイルをレビュー」 | 指定ファイルの全体 |
|
|
32
|
+
| 「最近の変更をレビュー」 | `git log` + `git diff` で直近の変更を取得 |
|
|
33
|
+
| 「実装をレビュー」 | 指定された機能に関連するファイル群 |
|
|
34
|
+
|
|
35
|
+
## レビューワークフロー
|
|
36
|
+
|
|
37
|
+
### 1. レビュー対象の収集
|
|
38
|
+
|
|
39
|
+
レビュー対象のコードを特定し、変更内容を把握する。
|
|
40
|
+
|
|
41
|
+
- PR の場合: `git diff` でベースブランチとの差分を取得
|
|
42
|
+
- ファイル指定の場合: 指定ファイルと関連ファイルを読み取る
|
|
43
|
+
- 関連するテストファイル、設計ドキュメントも収集する
|
|
44
|
+
|
|
45
|
+
### 2. エージェントの並列起動
|
|
46
|
+
|
|
47
|
+
Agent ツールを使い、5 つのエージェントを **同一メッセージで並列起動** する。各エージェントには以下を指示する。
|
|
48
|
+
|
|
49
|
+
```
|
|
50
|
+
あなたは {エージェント名} です。
|
|
51
|
+
.claude/agents/{エージェント名}.md の定義に従ってレビューしてください。
|
|
52
|
+
|
|
53
|
+
## レビュー対象
|
|
54
|
+
{変更差分 / ファイル内容}
|
|
55
|
+
|
|
56
|
+
## 関連コンテキスト
|
|
57
|
+
{関連する設計ドキュメント、テストファイル、ユーザーストーリー}
|
|
58
|
+
|
|
59
|
+
## レビュー観点
|
|
60
|
+
{エージェント固有の着眼点}
|
|
61
|
+
|
|
62
|
+
## 出力形式
|
|
63
|
+
以下の形式でフィードバックを返してください:
|
|
64
|
+
|
|
65
|
+
### 評価サマリー
|
|
66
|
+
(1-2 文で全体評価)
|
|
67
|
+
|
|
68
|
+
### 良い点
|
|
69
|
+
- (具体的に)
|
|
70
|
+
|
|
71
|
+
### 改善提案
|
|
72
|
+
- 【重要度: 高/中/低】(ファイル:行番号)(具体的な改善提案と理由)
|
|
73
|
+
|
|
74
|
+
### 懸念事項
|
|
75
|
+
- (潜在的なバグ、セキュリティリスク、パフォーマンス問題など)
|
|
76
|
+
|
|
77
|
+
### スコープ外の発見
|
|
78
|
+
- (レビュー対象外だが報告すべき問題)
|
|
79
|
+
```
|
|
80
|
+
|
|
81
|
+
#### Agent ツールが利用できない場合のフォールバック
|
|
82
|
+
|
|
83
|
+
Agent ツールが利用できない環境では、5 つの視点を逐次的にシミュレートする。各エージェントの定義ファイル(`.claude/agents/{エージェント名}.md`)を読み、その視点でレビューを順次実施する。出力形式は同一のテンプレートに従う。
|
|
84
|
+
|
|
85
|
+
### 3. フィードバックの統合
|
|
86
|
+
|
|
87
|
+
全エージェントのフィードバックを受け取った後、以下の形式で統合レポートを作成する。
|
|
88
|
+
|
|
89
|
+
```markdown
|
|
90
|
+
## コードレビュー結果
|
|
91
|
+
|
|
92
|
+
### レビュー対象
|
|
93
|
+
- {ファイル名 / PR 番号}
|
|
94
|
+
|
|
95
|
+
### 総合評価
|
|
96
|
+
(全エージェントの評価を統合した 2-3 文の評価)
|
|
97
|
+
|
|
98
|
+
### 改善提案(重要度順)
|
|
99
|
+
|
|
100
|
+
#### 高(マージ前に対応すべき)
|
|
101
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
102
|
+
|---|------|------|--------|------|
|
|
103
|
+
|
|
104
|
+
#### 中(対応推奨)
|
|
105
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
106
|
+
|---|------|------|--------|------|
|
|
107
|
+
|
|
108
|
+
#### 低(改善の余地あり)
|
|
109
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
110
|
+
|---|------|------|--------|------|
|
|
111
|
+
|
|
112
|
+
### 矛盾事項
|
|
113
|
+
|
|
114
|
+
複数エージェントの指摘が相反する場合、ここに記載する。
|
|
115
|
+
|
|
116
|
+
| # | 視点 A | 視点 B | 論点 | 推奨判断 |
|
|
117
|
+
|---|--------|--------|------|----------|
|
|
118
|
+
|
|
119
|
+
### エージェント別フィードバック詳細
|
|
120
|
+
|
|
121
|
+
<details>
|
|
122
|
+
<summary>xp-programmer(高: N / 中: N / 低: N)</summary>
|
|
123
|
+
(フィードバック全文)
|
|
124
|
+
</details>
|
|
125
|
+
|
|
126
|
+
<details>
|
|
127
|
+
<summary>xp-tester(高: N / 中: N / 低: N)</summary>
|
|
128
|
+
(フィードバック全文)
|
|
129
|
+
</details>
|
|
130
|
+
|
|
131
|
+
<details>
|
|
132
|
+
<summary>xp-architect(高: N / 中: N / 低: N)</summary>
|
|
133
|
+
(フィードバック全文)
|
|
134
|
+
</details>
|
|
135
|
+
|
|
136
|
+
<details>
|
|
137
|
+
<summary>xp-technical-writer(高: N / 中: N / 低: N)</summary>
|
|
138
|
+
(フィードバック全文)
|
|
139
|
+
</details>
|
|
140
|
+
|
|
141
|
+
<details>
|
|
142
|
+
<summary>xp-user-representative(高: N / 中: N / 低: N)</summary>
|
|
143
|
+
(フィードバック全文)
|
|
144
|
+
</details>
|
|
145
|
+
```
|
|
146
|
+
|
|
147
|
+
### 4. 改善アクションの提案
|
|
148
|
+
|
|
149
|
+
重要度「高」の指摘は具体的な修正コードを提案する。修正を適用するかはユーザーの判断に委ねる。
|
|
150
|
+
|
|
151
|
+
### 5. レビュー結果の保存
|
|
152
|
+
|
|
153
|
+
統合レポートを `docs/review/{対象名}_review_{YYYYMMDD}.md` に保存する。過去のレビュー結果と比較できるようにし、改善のトレーサビリティを確保する。
|
|
154
|
+
|
|
155
|
+
## レビュー完了条件
|
|
156
|
+
|
|
157
|
+
以下をすべて満たした場合にレビュー完了とする。
|
|
158
|
+
|
|
159
|
+
- 全エージェントのフィードバックが収集済み
|
|
160
|
+
- 統合レポートが作成済み
|
|
161
|
+
- 重要度「高」の指摘について対応方針(修正する / 許容する / 保留する)が決定済み
|
|
162
|
+
- レビュー結果がドキュメントとして保存済み
|
|
163
|
+
|
|
164
|
+
## 注意事項
|
|
165
|
+
|
|
166
|
+
- レビューは「変更を楽に安全にできる」コードを維持するための協調的な活動
|
|
167
|
+
- TDD サイクルの遵守状況(テストが先に書かれているか)もレビュー観点に含める
|
|
168
|
+
- セキュリティに関する指摘は重要度「高」として扱う
|
|
169
|
+
|
|
170
|
+
## 関連スキル
|
|
171
|
+
|
|
172
|
+
- `orchestrating-development` — 開発フェーズの全体ワークフロー
|
|
173
|
+
- `developing-backend` — バックエンド TDD 開発
|
|
174
|
+
- `developing-frontend` — フロントエンド TDD 開発
|
|
175
|
+
- `git-commit` — レビュー結果の保存後のコミット
|
|
@@ -0,0 +1,171 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: operating-review
|
|
3
|
+
description: 運用成果物のマルチパースペクティブレビューを実施。XP エージェント(アーキテクト、テスター、プロジェクトマネージャー)を並列起動し、インフラ設計・セキュリティ・信頼性・運用性の観点からフィードバックを収集・統合する。「インフラ設定をレビューして」「CI/CD パイプラインをチェックして」「デプロイ設定をレビュー」「Terraform コードをレビュー」「運用スクリプトを確認して」「Dockerfile をレビュー」といった場面で発動する。運用成果物に対するレビュー依頼があれば積極的に使用すること。
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# 運用成果物レビュー
|
|
7
|
+
|
|
8
|
+
運用フェーズの成果物(インフラ定義、CI/CD パイプライン、デプロイ設定、運用スクリプト、Dockerfile、環境設定など)を複数の XP エージェントで並列レビューし、信頼性・セキュリティ・運用性の観点からフィードバックを統合する。
|
|
9
|
+
|
|
10
|
+
## レビューの価値
|
|
11
|
+
|
|
12
|
+
運用成果物の不備は本番障害に直結する。インフラ設定のミス、セキュリティの穴、デプロイ手順の不備は、発見が遅れるほど影響が大きくなる。複数の専門的視点で事前に検証することで、本番環境のリスクを最小化する。
|
|
13
|
+
|
|
14
|
+
## レビューエージェント
|
|
15
|
+
|
|
16
|
+
| エージェント | 視点 | 着眼点 |
|
|
17
|
+
|:---|:---|:---|
|
|
18
|
+
| `xp-architect` | インフラ設計 | アーキテクチャとの整合、スケーラビリティ、可用性、コスト効率、IaC のモジュール化 |
|
|
19
|
+
| `xp-tester` | 信頼性・セキュリティ | セキュリティ設定、権限の最小化、障害シナリオ、ロールバック手順、テスト可能性 |
|
|
20
|
+
| `xp-project-manager` | 運用性・持続可能性 | 運用手順の明確さ、ドキュメントの十分性、チームへの影響、持続可能なペースの維持 |
|
|
21
|
+
|
|
22
|
+
## レビュー対象と重点観点
|
|
23
|
+
|
|
24
|
+
| 成果物 | 重点観点 |
|
|
25
|
+
|:---|:---|
|
|
26
|
+
| Terraform / IaC コード | セキュリティグループ、IAM ポリシー、状態管理、モジュール設計 |
|
|
27
|
+
| CI/CD パイプライン | ビルドの再現性、テスト網羅性、シークレット管理、OIDC 認証 |
|
|
28
|
+
| Dockerfile / docker-compose | ベースイメージの安全性、レイヤーの最適化、マルチステージビルド |
|
|
29
|
+
| デプロイ設定 | ロールバック手順、ヘルスチェック、環境変数管理、段階的デプロイ |
|
|
30
|
+
| 運用スクリプト | エラーハンドリング、冪等性、ログ出力、ドキュメント整合性 |
|
|
31
|
+
| 環境設定 | 環境間の差異管理、シークレット管理、設定の外部化 |
|
|
32
|
+
|
|
33
|
+
## レビューワークフロー
|
|
34
|
+
|
|
35
|
+
### 1. レビュー対象の特定
|
|
36
|
+
|
|
37
|
+
レビュー対象のファイルを特定する。ユーザーが明示的に指定しない場合は、以下のディレクトリを確認する。
|
|
38
|
+
|
|
39
|
+
- `ops/` — 運用スクリプト、Gulp タスク
|
|
40
|
+
- `infra/` / `terraform/` — IaC 定義
|
|
41
|
+
- `.github/workflows/` — CI/CD パイプライン
|
|
42
|
+
- `docker-compose*.yml` / `Dockerfile*` — コンテナ定義
|
|
43
|
+
- `docs/operation/` — 運用手順書
|
|
44
|
+
|
|
45
|
+
### 2. エージェントの並列起動
|
|
46
|
+
|
|
47
|
+
Agent ツールを使い、3 つのエージェントを **同一メッセージで並列起動** する。各エージェントには以下を指示する。
|
|
48
|
+
|
|
49
|
+
```
|
|
50
|
+
あなたは {エージェント名} です。
|
|
51
|
+
.claude/agents/{エージェント名}.md の定義に従ってレビューしてください。
|
|
52
|
+
|
|
53
|
+
## レビュー対象
|
|
54
|
+
{ファイルパスとその内容}
|
|
55
|
+
|
|
56
|
+
## 関連コンテキスト
|
|
57
|
+
{関連する設計ドキュメント、運用要件、環境情報}
|
|
58
|
+
|
|
59
|
+
## レビュー観点
|
|
60
|
+
{エージェント固有の着眼点}
|
|
61
|
+
|
|
62
|
+
## 出力形式
|
|
63
|
+
以下の形式でフィードバックを返してください:
|
|
64
|
+
|
|
65
|
+
### 評価サマリー
|
|
66
|
+
(1-2 文で全体評価)
|
|
67
|
+
|
|
68
|
+
### 良い点
|
|
69
|
+
- (具体的に)
|
|
70
|
+
|
|
71
|
+
### 改善提案
|
|
72
|
+
- 【重要度: 高/中/低】(ファイル:行番号)(具体的な改善提案と理由)
|
|
73
|
+
|
|
74
|
+
### セキュリティ懸念
|
|
75
|
+
- (権限、シークレット、ネットワーク設定に関する懸念)
|
|
76
|
+
|
|
77
|
+
### 運用リスク
|
|
78
|
+
- (障害時の影響範囲、復旧手順の有無、監視の網羅性)
|
|
79
|
+
|
|
80
|
+
### スコープ外の発見
|
|
81
|
+
- (レビュー対象外だが報告すべき問題)
|
|
82
|
+
```
|
|
83
|
+
|
|
84
|
+
#### Agent ツールが利用できない場合のフォールバック
|
|
85
|
+
|
|
86
|
+
Agent ツールが利用できない環境では、3 つの視点を逐次的にシミュレートする。各エージェントの定義ファイル(`.claude/agents/{エージェント名}.md`)を読み、その視点でレビューを順次実施する。出力形式は同一のテンプレートに従う。
|
|
87
|
+
|
|
88
|
+
### 3. フィードバックの統合
|
|
89
|
+
|
|
90
|
+
全エージェントのフィードバックを受け取った後、以下の形式で統合レポートを作成する。
|
|
91
|
+
|
|
92
|
+
```markdown
|
|
93
|
+
## 運用レビュー結果
|
|
94
|
+
|
|
95
|
+
### レビュー対象
|
|
96
|
+
- {ファイル名 / 設定名}
|
|
97
|
+
|
|
98
|
+
### 総合評価
|
|
99
|
+
(全エージェントの評価を統合した 2-3 文の評価)
|
|
100
|
+
|
|
101
|
+
### 改善提案(重要度順)
|
|
102
|
+
|
|
103
|
+
#### 高(本番適用前に対応必須)
|
|
104
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
105
|
+
|---|------|------|--------|------|
|
|
106
|
+
|
|
107
|
+
#### 中(対応推奨)
|
|
108
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
109
|
+
|---|------|------|--------|------|
|
|
110
|
+
|
|
111
|
+
#### 低(改善の余地あり)
|
|
112
|
+
| # | 提案 | 箇所 | 指摘元 | 理由 |
|
|
113
|
+
|---|------|------|--------|------|
|
|
114
|
+
|
|
115
|
+
### 矛盾事項
|
|
116
|
+
|
|
117
|
+
複数エージェントの指摘が相反する場合、ここに記載する。
|
|
118
|
+
|
|
119
|
+
| # | 視点 A | 視点 B | 論点 | 推奨判断 |
|
|
120
|
+
|---|--------|--------|------|----------|
|
|
121
|
+
|
|
122
|
+
### エージェント別フィードバック詳細
|
|
123
|
+
|
|
124
|
+
<details>
|
|
125
|
+
<summary>xp-architect(高: N / 中: N / 低: N)</summary>
|
|
126
|
+
(フィードバック全文)
|
|
127
|
+
</details>
|
|
128
|
+
|
|
129
|
+
<details>
|
|
130
|
+
<summary>xp-tester(高: N / 中: N / 低: N)</summary>
|
|
131
|
+
(フィードバック全文)
|
|
132
|
+
</details>
|
|
133
|
+
|
|
134
|
+
<details>
|
|
135
|
+
<summary>xp-project-manager(高: N / 中: N / 低: N)</summary>
|
|
136
|
+
(フィードバック全文)
|
|
137
|
+
</details>
|
|
138
|
+
```
|
|
139
|
+
|
|
140
|
+
### 4. 改善アクションの提案
|
|
141
|
+
|
|
142
|
+
重要度「高」の指摘(特にセキュリティ関連)は具体的な修正内容を提案する。本番環境に関する変更は、適用前にユーザーの明示的な承認を得る。
|
|
143
|
+
|
|
144
|
+
### 5. レビュー結果の保存
|
|
145
|
+
|
|
146
|
+
統合レポートを `docs/review/{対象名}_review_{YYYYMMDD}.md` に保存する。過去のレビュー結果と比較できるようにし、改善のトレーサビリティを確保する。
|
|
147
|
+
|
|
148
|
+
## レビュー完了条件
|
|
149
|
+
|
|
150
|
+
以下をすべて満たした場合にレビュー完了とする。
|
|
151
|
+
|
|
152
|
+
- 全エージェントのフィードバックが収集済み
|
|
153
|
+
- 統合レポートが作成済み
|
|
154
|
+
- 重要度「高」の指摘について対応方針(修正する / 許容する / 保留する)が決定済み
|
|
155
|
+
- レビュー結果がドキュメントとして保存済み
|
|
156
|
+
|
|
157
|
+
## 注意事項
|
|
158
|
+
|
|
159
|
+
- セキュリティに関する指摘は常に重要度「高」として扱う
|
|
160
|
+
- 本番環境に影響する変更は慎重に扱い、段階的な適用を推奨する
|
|
161
|
+
- IaC の変更は `plan` 結果を確認してから `apply` する原則を維持する
|
|
162
|
+
- 運用手順書の更新漏れがないかも確認する
|
|
163
|
+
|
|
164
|
+
## 関連スキル
|
|
165
|
+
|
|
166
|
+
- `orchestrating-operation` — 運用フェーズの全体ワークフロー
|
|
167
|
+
- `operating-setup` — 環境構築
|
|
168
|
+
- `operating-provision` — IaC プロビジョニング
|
|
169
|
+
- `operating-cicd` — CI/CD パイプライン構築
|
|
170
|
+
- `operating-deploy` — デプロイ・ロールバック
|
|
171
|
+
- `git-commit` — レビュー結果の保存後のコミット
|
package/lib/assets/CLAUDE.md
CHANGED
|
@@ -49,6 +49,14 @@
|
|
|
49
49
|
| `developing-backend` | バックエンド TDD(インサイドアウト) |
|
|
50
50
|
| `developing-frontend` | フロントエンド TDD(アウトサイドイン) |
|
|
51
51
|
|
|
52
|
+
### レビュー
|
|
53
|
+
|
|
54
|
+
| スキル | 用途 |
|
|
55
|
+
| :--- | :--- |
|
|
56
|
+
| `analyzing-review` | 分析成果物のマルチパースペクティブレビュー |
|
|
57
|
+
| `developing-review` | 開発成果物のマルチパースペクティブレビュー |
|
|
58
|
+
| `operating-review` | 運用成果物のマルチパースペクティブレビュー |
|
|
59
|
+
|
|
52
60
|
### 計画・進捗
|
|
53
61
|
|
|
54
62
|
| スキル | 用途 |
|