facet: implement/fixにビルド（型チェック）必須ゲートを追加
テスト中心の品質ゲートではtscでしか検出できない型エラーを 取りこぼしていたため、ビルド確認を必須項目として追加
This commit is contained in:
nrslib
parent
05865eb04e
commit
f6d3ef3ec1
@ -2,12 +2,18 @@ Address the reviewer's feedback.
|
||||
Use reports in the Report Directory shown in the Piece Context and fix the issues raised by the reviewer.
|
||||
Use files in the Report Directory as primary evidence. If additional context is needed, you may consult Previous Response and conversation history as secondary sources (Previous Response may be unavailable). If information conflicts, prioritize reports in the Report Directory and actual file contents.
|
||||
|
||||
**Important**: After fixing, run both build and tests.
|
||||
- Build verification is mandatory. Run the build (type check) and verify there are no type errors
|
||||
- Running tests is mandatory. After build succeeds, always run tests and verify results
|
||||
|
||||
**Required output (include headings)**
|
||||
## Work results
|
||||
- {Summary of actions taken}
|
||||
## Changes made
|
||||
- {Summary of changes}
|
||||
## Build results
|
||||
- {Build execution results}
|
||||
## Test results
|
||||
- {Command executed and results}
|
||||
- {Test command executed and results}
|
||||
## Evidence
|
||||
- {List key points from files checked/searches/diffs/logs}
|
||||
|
||||
@ -6,7 +6,8 @@ Use reports in the Report Directory as the primary source of truth. If additiona
|
||||
- Add unit tests for newly created classes and functions
|
||||
- Update relevant tests when modifying existing code
|
||||
- Test file placement: follow the project's conventions
|
||||
- Running tests is mandatory. After completing implementation, always run tests and verify results
|
||||
- Build verification is mandatory. After completing implementation, run the build (type check) and verify there are no type errors
|
||||
- Running tests is mandatory. After build succeeds, always run tests and verify results
|
||||
- When introducing new contract strings (file names, config key names, etc.), define them as constants in one place
|
||||
|
||||
**Scope output contract (create at the start of implementation):**
|
||||
@ -44,5 +45,7 @@ Small / Medium / Large
|
||||
- {Summary of actions taken}
|
||||
## Changes made
|
||||
- {Summary of changes}
|
||||
## Build results
|
||||
- {Build execution results}
|
||||
## Test results
|
||||
- {Command executed and results}
|
||||
- {Test command executed and results}
|
||||
|
||||
@ -10,6 +10,7 @@ Every behavior change requires a corresponding test, and every bug fix requires
|
||||
| One test, one concept | Do not mix multiple concerns in a single test |
|
||||
| Test behavior | Test behavior, not implementation details |
|
||||
| Independence | Do not depend on other tests or execution order |
|
||||
| Type safety | Code must pass the build (type check) |
|
||||
| Reproducibility | Do not depend on time or randomness. Same result every run |
|
||||
|
||||
## Coverage Criteria
|
||||
@ -19,6 +20,7 @@ Every behavior change requires a corresponding test, and every bug fix requires
|
||||
| New behavior | Test required. REJECT if missing |
|
||||
| Bug fix | Regression test required. REJECT if missing |
|
||||
| Behavior change | Test update required. REJECT if missing |
|
||||
| Build (type check) | Build must succeed. REJECT if it fails |
|
||||
| Edge cases / boundary values | Test recommended (Warning) |
|
||||
|
||||
## Test Priority
|
||||
|
||||
@ -2,12 +2,18 @@
|
||||
Piece Contextに示されたReport Directory内のレポートを確認し、レビュアーの指摘事項を修正してください。
|
||||
必要な根拠はReport Directory内のファイルを一次情報として取得してください。不足情報の補完が必要な場合に限り、Previous Responseや会話履歴を補助的に参照して構いません（Previous Responseは提供されない場合があります）。情報が競合する場合は、Report Directory内のレポートと実際のファイル内容を優先してください。
|
||||
|
||||
**重要**: 修正後、ビルドとテストの両方を実行してください。
|
||||
- ビルド確認は必須。ビルド（型チェック）を実行し、型エラーがないことを確認
|
||||
- テスト実行は必須。ビルド成功後、必ずテストを実行して結果を確認
|
||||
|
||||
**必須出力（見出しを含める）**
|
||||
## 作業結果
|
||||
- {実施内容の要約}
|
||||
## 変更内容
|
||||
- {変更内容の要約}
|
||||
## ビルド結果
|
||||
- {ビルド実行結果}
|
||||
## テスト結果
|
||||
- {実行コマンドと結果}
|
||||
- {テスト実行コマンドと結果}
|
||||
## 証拠
|
||||
- {確認したファイル/検索/差分/ログの要点を列挙}
|
||||
|
||||
@ -6,7 +6,8 @@ Report Directory内のレポートを一次情報として参照してくださ
|
||||
- 新規作成したクラス・関数には単体テストを追加
|
||||
- 既存コードを変更した場合は該当するテストを更新
|
||||
- テストファイルの配置: プロジェクトの規約に従う
|
||||
- テスト実行は必須。実装完了後、必ずテストを実行して結果を確認
|
||||
- ビルド確認は必須。実装完了後、ビルド（型チェック）を実行し、型エラーがないことを確認
|
||||
- テスト実行は必須。ビルド成功後、必ずテストを実行して結果を確認
|
||||
- ファイル名・設定キー名などの契約文字列を新規導入する場合は、定数として1箇所で定義すること
|
||||
|
||||
**Scope出力契約（実装開始時に作成）:**
|
||||
@ -44,5 +45,7 @@ Small / Medium / Large
|
||||
- {実施内容の要約}
|
||||
## 変更内容
|
||||
- {変更内容の要約}
|
||||
## ビルド結果
|
||||
- {ビルド実行結果}
|
||||
## テスト結果
|
||||
- {実行コマンドと結果}
|
||||
- {テスト実行コマンドと結果}
|
||||
|
||||
@ -19,6 +19,7 @@
|
||||
| 新しい振る舞い | テスト必須。テストがなければ REJECT |
|
||||
| バグ修正 | リグレッションテスト必須。テストがなければ REJECT |
|
||||
| 振る舞いの変更 | テストの更新必須。更新がなければ REJECT |
|
||||
| ビルド（型チェック） | ビルド成功必須。失敗は REJECT |
|
||||
| エッジケース・境界値 | テスト推奨（Warning） |
|
||||
|
||||
## テスト優先度
|
||||
@ -49,6 +50,7 @@ test('ユーザーが存在しない場合、NotFoundエラーを返す', async
|
||||
| 観点 | 良い | 悪い |
|
||||
|------|------|------|
|
||||
| 独立性 | 他のテストに依存しない | 実行順序に依存 |
|
||||
| 型安全 | コードはビルド（型チェック）が通ること |
|
||||
| 再現性 | 毎回同じ結果 | 時間やランダム性に依存 |
|
||||
| 明確性 | 失敗時に原因が分かる | 失敗しても原因不明 |
|
||||
| 焦点 | 1テスト1概念 | 複数の関心事が混在 |
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user