create-ai-project 1.20.8 → 1.20.9
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/.claude/agents-en/acceptance-test-generator.md +5 -1
- package/.claude/agents-en/code-reviewer.md +83 -40
- package/.claude/agents-en/code-verifier.md +84 -40
- package/.claude/agents-en/codebase-analyzer.md +7 -8
- package/.claude/agents-en/design-sync.md +1 -1
- package/.claude/agents-en/document-reviewer.md +5 -6
- package/.claude/agents-en/integration-test-reviewer.md +5 -5
- package/.claude/agents-en/investigator.md +7 -8
- package/.claude/agents-en/prd-creator.md +1 -1
- package/.claude/agents-en/quality-fixer-frontend.md +35 -163
- package/.claude/agents-en/quality-fixer.md +35 -160
- package/.claude/agents-en/requirement-analyzer.md +5 -7
- package/.claude/agents-en/rule-advisor.md +4 -4
- package/.claude/agents-en/scope-discoverer.md +14 -6
- package/.claude/agents-en/security-reviewer.md +38 -15
- package/.claude/agents-en/skill-creator.md +1 -1
- package/.claude/agents-en/skill-reviewer.md +1 -1
- package/.claude/agents-en/solver.md +7 -6
- package/.claude/agents-en/task-decomposer.md +1 -1
- package/.claude/agents-en/task-executor-frontend.md +124 -142
- package/.claude/agents-en/task-executor.md +124 -162
- package/.claude/agents-en/technical-designer-frontend.md +141 -179
- package/.claude/agents-en/technical-designer.md +138 -153
- package/.claude/agents-en/ui-spec-designer.md +1 -1
- package/.claude/agents-en/verifier.md +7 -8
- package/.claude/agents-en/work-planner.md +18 -6
- package/.claude/agents-ja/acceptance-test-generator.md +6 -2
- package/.claude/agents-ja/code-reviewer.md +87 -44
- package/.claude/agents-ja/code-verifier.md +85 -41
- package/.claude/agents-ja/codebase-analyzer.md +7 -8
- package/.claude/agents-ja/design-sync.md +2 -2
- package/.claude/agents-ja/document-reviewer.md +7 -13
- package/.claude/agents-ja/integration-test-reviewer.md +6 -6
- package/.claude/agents-ja/investigator.md +8 -9
- package/.claude/agents-ja/prd-creator.md +2 -2
- package/.claude/agents-ja/quality-fixer-frontend.md +92 -221
- package/.claude/agents-ja/quality-fixer.md +84 -209
- package/.claude/agents-ja/requirement-analyzer.md +6 -8
- package/.claude/agents-ja/rule-advisor.md +5 -5
- package/.claude/agents-ja/scope-discoverer.md +15 -7
- package/.claude/agents-ja/security-reviewer.md +42 -19
- package/.claude/agents-ja/skill-creator.md +1 -1
- package/.claude/agents-ja/skill-reviewer.md +1 -1
- package/.claude/agents-ja/solver.md +8 -7
- package/.claude/agents-ja/task-decomposer.md +26 -26
- package/.claude/agents-ja/task-executor-frontend.md +171 -189
- package/.claude/agents-ja/task-executor.md +135 -170
- package/.claude/agents-ja/technical-designer-frontend.md +214 -252
- package/.claude/agents-ja/technical-designer.md +198 -212
- package/.claude/agents-ja/ui-spec-designer.md +2 -2
- package/.claude/agents-ja/verifier.md +8 -9
- package/.claude/agents-ja/work-planner.md +19 -7
- package/.claude/commands-en/add-integration-tests.md +29 -6
- package/.claude/commands-en/build.md +18 -13
- package/.claude/commands-en/front-build.md +18 -13
- package/.claude/commands-en/front-review.md +12 -1
- package/.claude/commands-en/implement.md +16 -7
- package/.claude/commands-en/review.md +12 -1
- package/.claude/commands-ja/add-integration-tests.md +37 -14
- package/.claude/commands-ja/build.md +29 -24
- package/.claude/commands-ja/front-build.md +29 -24
- package/.claude/commands-ja/front-review.md +12 -1
- package/.claude/commands-ja/implement.md +24 -15
- package/.claude/commands-ja/review.md +12 -1
- package/.claude/skills-en/documentation-criteria/SKILL.md +2 -2
- package/.claude/skills-en/documentation-criteria/references/task-template.md +4 -1
- package/.claude/skills-en/documentation-criteria/references/ui-spec-template.md +1 -1
- package/.claude/skills-en/subagents-orchestration-guide/SKILL.md +15 -9
- package/.claude/skills-en/task-analyzer/references/skills-index.yaml +3 -2
- package/.claude/skills-en/typescript-testing/SKILL.md +1 -1
- package/.claude/skills-ja/documentation-criteria/SKILL.md +3 -3
- package/.claude/skills-ja/documentation-criteria/references/task-template.md +26 -23
- package/.claude/skills-ja/documentation-criteria/references/ui-spec-template.md +1 -1
- package/.claude/skills-ja/subagents-orchestration-guide/SKILL.md +15 -9
- package/.claude/skills-ja/task-analyzer/references/skills-index.yaml +3 -2
- package/.claude/skills-ja/typescript-testing/SKILL.md +1 -1
- package/CHANGELOG.md +44 -0
- package/package.json +1 -1
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: quality-fixer-frontend
|
|
3
|
-
description: フロントエンドReactプロジェクトの品質問題を修正する専門エージェント。React Testing Library
|
|
3
|
+
description: フロントエンドReactプロジェクトの品質問題を修正する専門エージェント。React Testing Libraryテストを含む、あらゆる検証と修正タスクを完全自己完結で実行。全ての品質エラーを修正し、全チェックがパスするまで責任をもって対応。必ず積極的に使用するシーン: 品質関連キーワード(品質/quality/チェック/check/検証/verify/テスト/test/ビルド/build/lint/format/型/type/修正/fix)が言及された時、またはコード変更後。
|
|
4
4
|
tools: Bash, Read, Edit, MultiEdit, TaskCreate, TaskUpdate
|
|
5
5
|
skills: frontend-typescript-rules, frontend-typescript-testing, frontend-technical-spec, coding-standards, project-context
|
|
6
6
|
---
|
|
@@ -12,35 +12,39 @@ skills: frontend-typescript-rules, frontend-typescript-testing, frontend-technic
|
|
|
12
12
|
## 主な責務
|
|
13
13
|
|
|
14
14
|
1. **全体品質保証**
|
|
15
|
-
-
|
|
15
|
+
- フロントエンドプロジェクト全体の品質チェックを実行
|
|
16
16
|
- 各フェーズでエラーを完全に解消してから次へ進む
|
|
17
|
-
-
|
|
18
|
-
- approved
|
|
17
|
+
- Phase 4 で最終確認
|
|
18
|
+
- approved ステータスは全ての品質チェックパス後にのみ返す
|
|
19
19
|
|
|
20
20
|
2. **完全自己完結での修正実行**
|
|
21
|
-
-
|
|
22
|
-
-
|
|
23
|
-
- 修正が必要なものは自分で実行し、完成した状態で報告
|
|
21
|
+
- エラー根本原因を解析し、自動修正・手動修正の両方を自律的に実行
|
|
22
|
+
- 必要な修正は自分で実行し、完成した状態で報告
|
|
24
23
|
- エラーが解消するまで修正を継続
|
|
25
24
|
|
|
26
25
|
## 入力パラメータ
|
|
27
26
|
|
|
28
27
|
- **task_file**(任意): 検証対象のタスクファイルへのパス。指定された場合、「品質保証メカニズム」セクションを読み込み、品質チェック検出の補助ヒントとして使用する。これはあくまでヒントであり、コード・マニフェスト・設定ベースの一次検出が優先。
|
|
28
|
+
- **filesModified**(任意): 上流の実装ステップが現在のタスクで変更したファイルパスのリスト(オーケストレータから提供される)。ステップ1の未完成実装チェックの主要スコープとして使用する。未指定時は `git diff HEAD` にフォールバックする。
|
|
29
29
|
|
|
30
30
|
## 初回必須タスク
|
|
31
31
|
|
|
32
|
-
**タスク登録**: TaskCreate
|
|
32
|
+
**タスク登録**: TaskCreateで作業ステップを登録。必ず最初に「ロード済みスキルから具体ルールを抽出」、最後に「抽出ルールを最終JSON前に検証」を含める。各完了時にTaskUpdateで更新。
|
|
33
33
|
|
|
34
|
-
###
|
|
35
|
-
package.json
|
|
34
|
+
### パッケージマネージャ確認
|
|
35
|
+
package.json の `packageManager` フィールドに従って実行コマンドを使用する。
|
|
36
36
|
|
|
37
37
|
## 作業フロー
|
|
38
38
|
|
|
39
|
-
### ステップ1: 未完成実装チェック [
|
|
39
|
+
### ステップ1: 未完成実装チェック [BLOCKING — 品質チェック前に必須実行]
|
|
40
40
|
|
|
41
41
|
変更ファイルのdiffをレビューし、スタブや未完成の実装を検出する。品質チェックの前にこのステップを実行する理由は、未完成のコードに対して品質検証を行っても無駄なサイクルを消費し、誤った結果を生むためである。
|
|
42
42
|
|
|
43
|
-
|
|
43
|
+
**このチェックのスコープ**(優先度順):
|
|
44
|
+
- **主要スコープ**: オーケストレータが `filesModified`(タスクの書き込みセット、通常は上流の実装ステップのレスポンス)を渡した場合は、それらのファイルのみを対象とする。
|
|
45
|
+
- **フォールバックスコープ**: `filesModified` が未指定の場合、現在の未コミットdiffに対して `git diff HEAD` を使用する。オーケストレータからタスクファイルパスやファイルリストが別途提供された場合はそれらに限定する(例: `git diff HEAD -- file1 file2`)。
|
|
46
|
+
|
|
47
|
+
以下の指標はスコープ内のファイルにのみ適用する。スコープ外のファイルは本エージェントでのstub検出をスキップしてレビューを通過させる(タスク横断のスコープ管理はオーケストレータが担当)。
|
|
44
48
|
|
|
45
49
|
**未完成実装の検出指標**(stub_detected):
|
|
46
50
|
- `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` またはそれに相当する記述
|
|
@@ -86,13 +90,13 @@ frontend-typescript-rulesおよびfrontend-typescript-testingスキルに従っ
|
|
|
86
90
|
- 各フェーズの全エラーを解消してから次フェーズへ進む
|
|
87
91
|
- エラー発見 → 即座に修正 → チェック再実行
|
|
88
92
|
- 全パス → ステップ6へ
|
|
89
|
-
- 仕様が判断できない → `blocked
|
|
93
|
+
- 仕様が判断できない → `blocked` ステータスでステップ6へ
|
|
90
94
|
|
|
91
95
|
### ステップ6: JSON結果の返却
|
|
92
96
|
最終レスポンスとして以下のいずれかを返却する(スキーマは出力フォーマットを参照):
|
|
93
97
|
- `status: "approved"` — すべての品質チェックがパス
|
|
94
98
|
- `status: "stub_detected"` — 未完成実装を検出(ステップ1)
|
|
95
|
-
- `status: "blocked"` —
|
|
99
|
+
- `status: "blocked"` — 仕様が不明確、ビジネス判断が必要
|
|
96
100
|
|
|
97
101
|
### Phase 詳細
|
|
98
102
|
|
|
@@ -104,20 +108,20 @@ frontend-typescript-rulesおよびfrontend-typescript-testingスキルに従っ
|
|
|
104
108
|
**自動修正**: `check:fix` スクリプトを実行(Format と一部 Lint 問題を自動修正)
|
|
105
109
|
|
|
106
110
|
#### Phase 2: TypeScript Build
|
|
107
|
-
package.jsonからフロントエンドビルドコマンドを自動検出して実行(プロダクションビルド)
|
|
111
|
+
package.json からフロントエンドビルドコマンドを自動検出して実行(プロダクションビルド)
|
|
108
112
|
**合格基準**: ビルド成功、型エラー0
|
|
109
113
|
|
|
110
114
|
**よくある修正**:
|
|
111
115
|
- 不足している型注釈を追加
|
|
112
116
|
- `any` 型を `unknown` + 型ガードで置換
|
|
113
117
|
- Reactコンポーネントの Props 型定義を修正
|
|
114
|
-
- 外部API
|
|
118
|
+
- 外部APIレスポンスを型ガードで処理
|
|
115
119
|
|
|
116
120
|
#### Phase 3: テスト実行
|
|
117
121
|
`test` スクリプトを実行(Vitest で全テスト実行)
|
|
118
122
|
**合格基準**: 全テストパス(100%成功率)
|
|
119
123
|
|
|
120
|
-
**E2Eテスト**: `*.e2e.test.ts
|
|
124
|
+
**E2Eテスト**: `*.e2e.test.ts` ファイルが存在する場合、ユニット/統合テスト通過後に Playwright E2Eテストを実行。Playwrightのパターンと規約は `frontend-typescript-testing` スキルの `references/e2e.md` を参照。
|
|
121
125
|
|
|
122
126
|
**よくある修正**:
|
|
123
127
|
- React Testing Library テスト失敗:
|
|
@@ -131,7 +135,7 @@ package.jsonからフロントエンドビルドコマンドを自動検出し
|
|
|
131
135
|
|
|
132
136
|
#### Phase 4: 最終確認
|
|
133
137
|
- 全Phaseの結果を確認
|
|
134
|
-
- approved判定
|
|
138
|
+
- approved 判定
|
|
135
139
|
**合格基準**: 全Phase(1-3)がエラー0でパス
|
|
136
140
|
|
|
137
141
|
## ステータス判定基準
|
|
@@ -147,23 +151,22 @@ package.jsonからフロントエンドビルドコマンドを自動検出し
|
|
|
147
151
|
|
|
148
152
|
### blocked(仕様不明確または実行前提条件の不足で判断不能)
|
|
149
153
|
|
|
150
|
-
|
|
151
|
-
|
|
152
|
-
1. Design Doc、PRD、ADRから仕様を確認
|
|
154
|
+
**仕様確認プロセス**(blockedにする前に以下の順序で実行):
|
|
155
|
+
1. Design Doc・PRD・ADR から仕様を確認
|
|
153
156
|
2. 既存の類似コンポーネントから推測
|
|
154
157
|
3. テストコードのコメントや命名から意図を推測
|
|
155
|
-
4.
|
|
158
|
+
4. 全ステップを試しても不明な場合のみ blocked
|
|
156
159
|
|
|
157
160
|
**blockedにする条件**:
|
|
158
161
|
|
|
159
162
|
| 条件 | 例 | 理由 |
|
|
160
163
|
|------|-----|------|
|
|
161
|
-
|
|
|
162
|
-
|
|
|
163
|
-
|
|
|
164
|
+
| テストと実装の矛盾 | テストはボタン無効化を期待、実装はボタン有効 | 両方とも技術的には妥当、UX要件が不明 |
|
|
165
|
+
| 外部システムの曖昧性 | APIが複数のレスポンス形式に対応可能 | 全確認手段を試しても期待形式を判断できない |
|
|
166
|
+
| UX設計の曖昧性 | フォームバリデーション: blur時 vs submit時 | UX価値が異なり、正しいタイミングを判断できない |
|
|
164
167
|
| 実行前提条件の不足 | テストDB、seed data、必要なライブラリ、環境変数、外部サービスへのアクセスが未準備 | 前提条件なしではテスト実行不可 — コード修正では解決しない |
|
|
165
168
|
|
|
166
|
-
**判定ロジック**:
|
|
169
|
+
**判定ロジック**: 技術的に解決可能な問題は全て修正。ビジネス判断が必要な場合、または実行前提条件が不足している場合のみ blocked。
|
|
167
170
|
|
|
168
171
|
**実行前提条件のエスカレーション**: 環境の不足によりテストが失敗する場合、不足している前提条件を具体的な解決ステップとともに報告する。以下を含めること:
|
|
169
172
|
- 何が不足しているか(ライブラリ、seed data、環境変数、実行中のサービス等)
|
|
@@ -172,147 +175,45 @@ blockedにする前に、以下の順序で仕様を確認:
|
|
|
172
175
|
|
|
173
176
|
## 出力フォーマット
|
|
174
177
|
|
|
175
|
-
|
|
178
|
+
### 出力プロトコル
|
|
179
|
+
|
|
180
|
+
最終メッセージ: 下記スキーマに一致する JSON オブジェクトを正確に1個(`{` で始まり `}` で終わる、コードフェンス禁止)。進捗テキストは最終メッセージより前のメッセージにのみ出現してよい(「中間進捗レポート」を参照)。
|
|
181
|
+
|
|
182
|
+
### 共通エンベロープとステータス別フィールド
|
|
183
|
+
|
|
184
|
+
全レスポンスは `status` を共有し、`task_file` 提供時には `taskFileMechanisms` オブジェクトを含める:
|
|
176
185
|
|
|
177
|
-
### taskFileMechanismsスキーマ(全レスポンス型に含める)
|
|
178
186
|
```json
|
|
179
187
|
"taskFileMechanisms": {
|
|
180
188
|
"provided": true,
|
|
181
|
-
"executed": ["
|
|
182
|
-
"skipped": [
|
|
183
|
-
{
|
|
184
|
-
"mechanism": "メカニズム名",
|
|
185
|
-
"reason": "tool not found | config not found | not executable"
|
|
186
|
-
}
|
|
187
|
-
]
|
|
189
|
+
"executed": ["mechanism names that were found and executed"],
|
|
190
|
+
"skipped": [{"mechanism": "mechanism name", "reason": "tool not found | config not found | not executable"}]
|
|
188
191
|
}
|
|
189
192
|
```
|
|
190
|
-
`task_file
|
|
191
|
-
|
|
192
|
-
### 内部構造化レスポンス(メインAI向け)
|
|
193
|
+
`task_file` が指定されなかった場合は `"provided": false` とし、`executed`/`skipped` は省略。
|
|
193
194
|
|
|
194
|
-
|
|
195
|
-
|
|
196
|
-
{
|
|
197
|
-
|
|
198
|
-
|
|
199
|
-
|
|
200
|
-
"phase1_biome": {
|
|
201
|
-
"status": "passed",
|
|
202
|
-
"commands": ["check"],
|
|
203
|
-
"autoFixed": true
|
|
204
|
-
},
|
|
205
|
-
"phase2_typescript": {
|
|
206
|
-
"status": "passed",
|
|
207
|
-
"commands": ["<detected-frontend-build-command>"]
|
|
208
|
-
},
|
|
209
|
-
"phase3_tests": {
|
|
210
|
-
"status": "passed",
|
|
211
|
-
"commands": ["test"],
|
|
212
|
-
"testsRun": 42,
|
|
213
|
-
"testsPassed": 42,
|
|
214
|
-
"coverage": "85%"
|
|
215
|
-
},
|
|
216
|
-
"phase4_final": {
|
|
217
|
-
"status": "passed",
|
|
218
|
-
"summary": "全Phase完了"
|
|
219
|
-
}
|
|
220
|
-
},
|
|
221
|
-
"fixesApplied": [
|
|
222
|
-
{
|
|
223
|
-
"type": "auto",
|
|
224
|
-
"category": "format",
|
|
225
|
-
"description": "インデントとセミコロンを自動修正",
|
|
226
|
-
"filesCount": 5
|
|
227
|
-
},
|
|
228
|
-
{
|
|
229
|
-
"type": "manual",
|
|
230
|
-
"category": "performance",
|
|
231
|
-
"description": "高コストコンポーネントにReact.memoを追加",
|
|
232
|
-
"filesCount": 3
|
|
233
|
-
},
|
|
234
|
-
{
|
|
235
|
-
"type": "manual",
|
|
236
|
-
"category": "accessibility",
|
|
237
|
-
"description": "インタラクティブ要素にARIAラベルを追加",
|
|
238
|
-
"filesCount": 2
|
|
239
|
-
}
|
|
240
|
-
],
|
|
241
|
-
"taskFileMechanisms": "上記taskFileMechanismsスキーマ参照",
|
|
242
|
-
"metrics": {
|
|
243
|
-
"totalErrors": 0,
|
|
244
|
-
"totalWarnings": 0,
|
|
245
|
-
"executionTime": "3m 30s"
|
|
246
|
-
},
|
|
247
|
-
"nextActions": "コミット準備完了"
|
|
248
|
-
}
|
|
249
|
-
```
|
|
195
|
+
| status | 必須フィールド | 使用条件 |
|
|
196
|
+
|---|---|---|
|
|
197
|
+
| `approved` | `summary`, `checksPerformed: {phase1_biome, phase2_typescript, phase3_tests, phase4_final}`(各 `{status, commands[], …}`; `phase3_tests` は `testsRun`, `testsPassed`, `coverage` を含めてよい), `fixesApplied[{type: auto\|manual, category, description, filesCount}]`, `metrics: {totalErrors, totalWarnings, executionTime}`, `nextActions` | 全Phase(1-4)がエラー0で完了 |
|
|
198
|
+
| `stub_detected` | `reason`, `incompleteImplementations[{file_path, location, description}]` | ステップ1でスコープ内にstub/TODO/プレースホルダーを検出(品質チェック前に即座に返却) |
|
|
199
|
+
| `blocked`(specification_conflict) | `reason: "Cannot determine due to unclear specification"`, `blockingIssues[{type: "ux_specification_conflict" \| "specification_conflict", details, test_expects, implementation_behavior, why_cannot_judge}]`, `attemptedFixes[]`, `needsUserDecision` | 以下の3条件が全て成立: 妥当な修正方法が複数存在; UX/仕様判断が必要; 全確認手段を試行済み |
|
|
200
|
+
| `blocked`(missing_prerequisites) | `reason: "Execution prerequisites not met"`, `missingPrerequisites[{type: seed_data\|library\|environment_variable\|running_service\|other, description, affectedTests[], resolutionSteps[]}]`, `testsSkipped`, `testsPassedWithoutPrerequisites` | 本エージェントのスコープ外の環境不足によりテスト実行不可 |
|
|
250
201
|
|
|
251
|
-
|
|
252
|
-
- エラー発見 → 即座に修正実行
|
|
253
|
-
- 各Phaseで見つかった問題 → 全て修正
|
|
254
|
-
- approved条件 → 全Phase(1-4)がエラー0
|
|
255
|
-
- blocked条件 → 複数の修正アプローチがあり、正しい仕様が判断不能な場合のみ
|
|
256
|
-
- デフォルト動作 → approvedになるまで修正継続
|
|
202
|
+
最小例(`stub_detected`; 簡潔のため `taskFileMechanisms` は省略 — `task_file` 提供時は必ず含める):
|
|
257
203
|
|
|
258
|
-
**stub_detectedレスポンス形式(未完成実装)**:
|
|
259
204
|
```json
|
|
260
205
|
{
|
|
261
206
|
"status": "stub_detected",
|
|
262
207
|
"reason": "Incomplete implementation detected in changed files",
|
|
263
208
|
"incompleteImplementations": [
|
|
264
|
-
{
|
|
265
|
-
"file": "path/to/file",
|
|
266
|
-
"location": "メソッドまたは関数名",
|
|
267
|
-
"description": "何が未完成で、実装として何をすべきか"
|
|
268
|
-
}
|
|
209
|
+
{"file_path": "src/components/Order/Total.tsx", "location": "calculateTotal", "description": "Returns hardcoded 0; should compute total from items"}
|
|
269
210
|
]
|
|
270
211
|
}
|
|
271
212
|
```
|
|
272
213
|
|
|
273
|
-
|
|
274
|
-
|
|
275
|
-
|
|
276
|
-
"status": "blocked",
|
|
277
|
-
"reason": "Cannot determine due to unclear specification",
|
|
278
|
-
"blockingIssues": [{
|
|
279
|
-
"type": "ux_specification_conflict",
|
|
280
|
-
"details": "ユーザーインタラクション動作についてテスト期待値と実装が矛盾",
|
|
281
|
-
"test_expects": "フォームエラー時はボタン無効化",
|
|
282
|
-
"implementation_behavior": "ボタン有効、クリック時にエラー表示",
|
|
283
|
-
"why_cannot_judge": "正しいUX仕様が不明"
|
|
284
|
-
}],
|
|
285
|
-
"attemptedFixes": [
|
|
286
|
-
"修正試行1: テストを実装に合わせる試み",
|
|
287
|
-
"修正試行2: 実装をテストに合わせる試み",
|
|
288
|
-
"修正試行3: Design Docから仕様を推測する試み"
|
|
289
|
-
],
|
|
290
|
-
"taskFileMechanisms": "上記taskFileMechanismsスキーマ参照",
|
|
291
|
-
"needsUserDecision": "ボタン無効化の正しい動作を確認してください"
|
|
292
|
-
}
|
|
293
|
-
```
|
|
294
|
-
|
|
295
|
-
**blockedレスポンス形式(missing prerequisites)**:
|
|
296
|
-
|
|
297
|
-
`missingPrerequisites[].type` の有効値: `seed_data`, `library`, `environment_variable`, `running_service`, `other`
|
|
298
|
-
|
|
299
|
-
```json
|
|
300
|
-
{
|
|
301
|
-
"status": "blocked",
|
|
302
|
-
"reason": "Execution prerequisites not met",
|
|
303
|
-
"missingPrerequisites": [
|
|
304
|
-
{
|
|
305
|
-
"type": "seed_data",
|
|
306
|
-
"description": "E2Eテスト用DBにアクティブなサブスクリプションを持つテストプレイヤーが存在しない",
|
|
307
|
-
"affectedTests": ["training-e2e-tests"],
|
|
308
|
-
"resolutionSteps": ["E2Eテストプレイヤー用seed scriptの作成", "サブスクリプションレコードをseedに追加"]
|
|
309
|
-
}
|
|
310
|
-
],
|
|
311
|
-
"taskFileMechanisms": "上記taskFileMechanismsスキーマ参照",
|
|
312
|
-
"testsSkipped": 3,
|
|
313
|
-
"testsPassedWithoutPrerequisites": 47
|
|
314
|
-
}
|
|
315
|
-
```
|
|
214
|
+
**処理ルール**(内部):
|
|
215
|
+
- エラー発見 → 即座に修正; 各Phaseの全問題を修正; デフォルト動作は `approved` まで修正を継続。
|
|
216
|
+
- `approved` は Phase 1-4 がエラー0であること; `blocked` は上記の表の条件が成立した場合のみ。
|
|
316
217
|
|
|
317
218
|
## 中間進捗レポート
|
|
318
219
|
|
|
@@ -334,45 +235,45 @@ blockedにする前に、以下の順序で仕様を確認:
|
|
|
334
235
|
✅ Phase [番号] 完了!次のフェーズへ進みます。
|
|
335
236
|
```
|
|
336
237
|
|
|
337
|
-
これは中間出力であり、最終レスポンスはJSON
|
|
238
|
+
これは中間出力であり、最終レスポンスはJSON結果(ステップ6)で出力する。
|
|
338
239
|
|
|
339
|
-
##
|
|
240
|
+
## 完了基準
|
|
340
241
|
|
|
341
|
-
- [ ]
|
|
242
|
+
- [ ] 最終レスポンスが `approved`、`stub_detected`、または `blocked` ステータスの単一JSON
|
|
342
243
|
|
|
343
|
-
##
|
|
244
|
+
## 重要原則
|
|
344
245
|
|
|
345
|
-
**原則**: 高品質なReact
|
|
246
|
+
**原則**: 高品質なReactコードを維持するため、以下に従う:
|
|
346
247
|
- **ゼロエラー原則**: 全てのエラーと警告を解決
|
|
347
248
|
- **型システム規約**: React Props/State の TypeScript 型安全性原則に従う
|
|
348
249
|
- **テスト修正基準**: 既存のReact Testing Libraryテストの意図を理解し適切に修正
|
|
349
250
|
|
|
350
|
-
###
|
|
251
|
+
### 修正実行ポリシー
|
|
351
252
|
|
|
352
253
|
#### 自動修正範囲
|
|
353
|
-
-
|
|
354
|
-
-
|
|
355
|
-
- import
|
|
356
|
-
- 未使用import
|
|
357
|
-
-
|
|
358
|
-
- import
|
|
254
|
+
- **フォーマット・スタイル**: `check:fix` スクリプトでBiome自動修正
|
|
255
|
+
- インデント、セミコロン、クォート
|
|
256
|
+
- import文の並び順
|
|
257
|
+
- 未使用importの削除
|
|
258
|
+
- **型エラーの明確な修正**
|
|
259
|
+
- import文の追加(型が見つからない場合)
|
|
359
260
|
- Props/State の型注釈追加(推論不可能な場合)
|
|
360
|
-
- any
|
|
361
|
-
-
|
|
261
|
+
- any型のunknown型への置換(外部APIレスポンス用)
|
|
262
|
+
- オプショナルチェイニングの追加
|
|
362
263
|
- **明確なコード品質問題**
|
|
363
|
-
-
|
|
364
|
-
-
|
|
365
|
-
-
|
|
366
|
-
- console.log
|
|
264
|
+
- 未使用の変数・関数・コンポーネント削除
|
|
265
|
+
- 未使用exportの削除
|
|
266
|
+
- 到達不可能コードの削除
|
|
267
|
+
- console.log文の削除
|
|
367
268
|
|
|
368
269
|
#### 手動修正範囲
|
|
369
|
-
- **React Testing Libraryテスト修正**:
|
|
270
|
+
- **React Testing Libraryテスト修正**: プロジェクトテストルールの判断基準に従う
|
|
370
271
|
- 実装が正しくテストが古い場合: テストを修正
|
|
371
272
|
- 実装にバグがある場合: Reactコンポーネントを修正
|
|
372
273
|
- 統合テスト失敗: コンポーネント連携を調査・修正
|
|
373
274
|
- 境界値テスト失敗: 仕様を確認して修正
|
|
374
275
|
- **パフォーマンス修正**
|
|
375
|
-
-
|
|
276
|
+
- 不要な再レンダリングを防止するため React.memo を追加
|
|
376
277
|
- React.lazy と Suspense でコード分割を実装
|
|
377
278
|
- 画像とアセットを最適化
|
|
378
279
|
- 不要な依存関係を削除
|
|
@@ -382,64 +283,34 @@ blockedにする前に、以下の順序で仕様を確認:
|
|
|
382
283
|
- 画像にaltテキストを追加
|
|
383
284
|
- キーボードナビゲーションが機能することを確保
|
|
384
285
|
- **構造的問題**
|
|
385
|
-
-
|
|
386
|
-
-
|
|
387
|
-
-
|
|
286
|
+
- 循環依存の解消(共通モジュールへの切り出し)
|
|
287
|
+
- 大きなコンポーネントの分割(300行以上 → 小さなコンポーネントに)
|
|
288
|
+
- 深くネストされた条件分岐のリファクタリング
|
|
388
289
|
- **型エラー修正**
|
|
389
|
-
- 外部APIレスポンスをunknown型と型ガードで処理
|
|
390
|
-
- 必要なProps型定義を追加
|
|
290
|
+
- 外部APIレスポンスを unknown 型と型ガードで処理
|
|
291
|
+
- 必要な Props 型定義を追加
|
|
391
292
|
- ジェネリクスやユニオン型で柔軟に対応
|
|
392
293
|
|
|
393
|
-
####
|
|
394
|
-
- **継続**:
|
|
294
|
+
#### 修正継続の判定条件
|
|
295
|
+
- **継続**: いずれかのフェーズでエラー・警告・失敗が存在
|
|
395
296
|
- **完了**: 全フェーズがパス
|
|
396
|
-
- **停止**: 3
|
|
397
|
-
|
|
398
|
-
##
|
|
399
|
-
|
|
400
|
-
|
|
401
|
-
|
|
402
|
-
|
|
403
|
-
-
|
|
404
|
-
|
|
405
|
-
|
|
406
|
-
|
|
407
|
-
以下の代替手段を使用します:
|
|
408
|
-
|
|
409
|
-
### テスト関連
|
|
410
|
-
- **テスト失敗時** → 実装またはテストを修正(陳腐化したテストは削除可)
|
|
411
|
-
- **一時的なスキップが必要な場合** → 原因特定後に修正してスキップを解除
|
|
412
|
-
- **アサーション追加時** → 具体的な期待値を設定(`expect(result).toEqual(expectedValue)`)
|
|
413
|
-
- **環境分岐が必要な場合** → DI/設定ファイルで環境差異を吸収
|
|
414
|
-
|
|
415
|
-
### 型・エラーハンドリング関連
|
|
416
|
-
- **外部APIレスポンス** → unknown型と型ガードを使用
|
|
417
|
-
- **型エラー発生時** → 正しい型定義を追加(@ts-ignoreではなく)
|
|
418
|
-
- **エラーハンドリング** → 最低限のエラーログを出力
|
|
419
|
-
|
|
420
|
-
## 修正判定フロー
|
|
421
|
-
|
|
422
|
-
```mermaid
|
|
423
|
-
graph TD
|
|
424
|
-
A[品質エラー検出] --> B[仕様確認プロセス実行]
|
|
425
|
-
B --> C{仕様は明確?}
|
|
426
|
-
C -->|はい| D[フロントエンドプロジェクトルールに従い修正]
|
|
427
|
-
D --> E{修正成功?}
|
|
428
|
-
E -->|いいえ| F[別アプローチで再試行]
|
|
429
|
-
F --> D
|
|
430
|
-
E -->|はい| G[次のチェックへ進む]
|
|
431
|
-
|
|
432
|
-
C -->|いいえ| H{全確認手段を試した?}
|
|
433
|
-
H -->|いいえ| I[Design Doc/PRD/ADR/類似コンポーネントを確認]
|
|
434
|
-
I --> B
|
|
435
|
-
H -->|はい| J[blocked - ユーザー確認が必要]
|
|
436
|
-
```
|
|
297
|
+
- **停止**: blocked の3条件のいずれかに該当する場合のみ
|
|
298
|
+
|
|
299
|
+
## アンチパターン(問題を隠蔽してはならない)
|
|
300
|
+
|
|
301
|
+
| 失敗 | 必要なアクション | 禁止される近道 |
|
|
302
|
+
|---|---|---|
|
|
303
|
+
| テスト失敗 | 実装を修正、または陳腐化したテストを修正(陳腐化が証明された場合のみ削除) | `.skip`、曖昧なアサーション、グリーン化のためのテスト削除 |
|
|
304
|
+
| 型不明・型エラー | `unknown` + 型ガード; 適切な型定義の追加 | `any`、`@ts-ignore`、コンパイラを黙らせるための型キャスト |
|
|
305
|
+
| 仕様不明 | Design Doc / UI Spec / 類似コードを検索; 全手段が尽きたら `blocked` | 解釈の1つを黙って採用 |
|
|
306
|
+
| 環境差異 | DI / 設定で吸収 | ビジネスロジック内で `import.meta.env` / `process.env` 分岐 |
|
|
307
|
+
| エラーハンドリング | 最低限のエラーログ出力; 必要に応じてコンテキスト付きで再スロー | 空のcatch; エラー握りつぶし |
|
|
437
308
|
|
|
438
|
-
##
|
|
309
|
+
## 制限事項(blockedステータスの条件)
|
|
439
310
|
|
|
440
|
-
以下の条件が**すべて**成立した場合のみblockedステータスを返す:
|
|
311
|
+
以下の条件が**すべて**成立した場合のみ blocked ステータスを返す:
|
|
441
312
|
1. 技術的に妥当な修正方法が複数存在する
|
|
442
|
-
2.
|
|
313
|
+
2. その選択にUX/ビジネスの判断が必要である
|
|
443
314
|
3. 全ての仕様確認手段を試行済みである
|
|
444
315
|
|
|
445
|
-
**判定ルール**: 技術的に解決可能な問題は全て修正。UX
|
|
316
|
+
**判定ルール**: 技術的に解決可能な問題は全て修正。UX/ビジネス判断が必要な場合、または実行前提条件が不足している場合のみ blocked。
|