AgentSkillsCN

impl-check

对照需求文档中的验收标准与实际实现情况,检查是否存在遗漏或疏漏的技能。适用于实现过程中及实现完成后使用。

SKILL.md
--- frontmatter
name: impl-check
description: REQの受け入れ基準と実装を照合し、漏れがないかチェックするスキル。実装中・実装後に使用。

Implementation Check - 実装チェック

REQの受け入れ基準(AC)と実装を照合し、漏れがないかチェックするスキル。

いつ使うか

  • 実装中に「これで合っているか」確認したいとき
  • PR作成前の最終チェック
  • 「実装完了」と言う前の確認
  • レビューで指摘を受けた後の再確認

チェック項目

1. REQ受け入れ基準(AC)の照合

対象REQのACを1つずつ確認:

markdown
## AC照合結果

| # | 受け入れ基準 | 実装状況 | 確認方法 |
|---|-------------|----------|----------|
| 1 | `f`キーでフィルタ切替 | ✅ 実装済み | main.rs:286 |
| 2 | 期間フィルタが動作 | ✅ 実装済み | app.rs:2893 |
| 3 | テキスト検索 | ⚠️ 未実装 | - |

### 未完了項目
- [ ] AC#3: テキスト検索の実装が必要

2. キーバインド表示チェック

新しいキーバインドを追加した場合:

markdown
## キーバインド表示チェック

| キー | 機能 | フッター | 詳細パネル | 判定 |
|------|------|----------|------------|------|
| `f` | フィルタ | ✅ あり | - | OK |
| `p` | 期間 | ❌ なし | - | **要修正** |
| `F` | クリア | ✅ 条件付き | - | OK |

### 修正が必要
- [ ] `p` キーをフッターに追加(ui.rs draw_footer)

3. 翻訳キーチェック

新しい文字列を追加した場合:

markdown
## 翻訳キーチェック

| キー | English | Japanese | 判定 |
|------|---------|----------|------|
| footer.filter | ✅ Filter | ✅ フィルタ | OK |
| footer.period | ❌ なし | ❌ なし | **要追加** |

### 修正が必要
- [ ] `footer.period` を i18n.rs に追加(英語・日本語両方)

4. テストカバレッジチェック

markdown
## テストチェック

| 機能 | ユニットテスト | 統合テスト | 判定 |
|------|----------------|------------|------|
| WorklogFilter | ✅ あり | - | OK |
| filtered_worklogs | ✅ あり | - | OK |
| 検索機能 | ❌ なし | - | **要追加** |

### 修正が必要
- [ ] 検索機能のテストを追加

5. 仕様書更新チェック

markdown
## 仕様書チェック

| ファイル | 更新必要 | 更新済み | 判定 |
|----------|----------|----------|------|
| worklog.md | ✅ 要 | ❌ 未 | **要更新** |
| data-structures.md | ❌ 不要 | - | OK |

### 修正が必要
- [ ] docs/specification/features/worklog.md を更新

実行手順

Step 1: 対象REQを特定

bash
# 作業中のREQを確認
ls docs/requirements/active/

Step 2: ACを読み込む

REQファイルの「## 6. 受け入れ基準(AC)」セクションを確認。

Step 3: 各チェックを実行

  1. AC照合
  2. キーバインド表示(該当する場合)
  3. 翻訳キー(該当する場合)
  4. テスト
  5. 仕様書

Step 4: 結果を報告

出力テンプレート

markdown
# Implementation Check - REQ-XXXXXXX

## サマリー

| チェック項目 | 結果 | 修正必要 |
|-------------|------|----------|
| AC照合 | 5/6 完了 | 1件 |
| キーバインド | OK | 0件 |
| 翻訳キー | NG | 2件 |
| テスト | OK | 0件 |
| 仕様書 | NG | 1件 |

**総合判定**: ⚠️ 修正が必要(4件)

## 詳細

### AC照合
[詳細結果]

### キーバインド表示
[詳細結果]

### 翻訳キー
[詳細結果]

### テスト
[詳細結果]

### 仕様書
[詳細結果]

## 修正アクション

- [ ] AC#3: テキスト検索を実装
- [ ] footer.period を i18n.rs に追加
- [ ] footer.search を i18n.rs に追加
- [ ] worklog.md を更新

## 修正完了後

すべてのチェックがOKになったら:
1. `cargo test` 実行
2. `cargo build --release` 実行
3. PR作成

自動チェックコマンド

bash
# キーバインドがフッターに含まれているか確認
grep -n "footer\." src/ui.rs | grep -i "filter\|period\|search"

# 翻訳キーの存在確認
grep -n "footer.filter\|footer.period\|footer.search" src/i18n.rs

# テストの存在確認
grep -n "#\[test\]" src/app.rs | wc -l

Tips

  • PR作成前に必ず実行: 「完了した」と思ったら、このスキルでチェック
  • 漏れは恥ではない: チェックで見つかるのは良いこと
  • 1つずつ修正: 複数の修正がある場合、1つずつ確実に
  • 再チェック: 修正後は再度このスキルを実行