| name | agentic-self-review |
| description | PR、マージ、handoff の前に swbt-daemon の変更を判定項目ごとにセルフRvする。Codex が要件の充足範囲、根拠監査の状態、CMake/CTest 結果、sanitizer/cross-build 結果、実機実行または未実行理由、BTstack/license impact、non-goals、remaining risks、work unit の完了を裏付ける根拠を整理するときに使う。 |
セルフRv
swbt work unit の完了宣言、PR 作成、大きな変更の handoff の前に、この skill を使う。
レビュー手順
- 対象 work unit、ユーザの意図、non-goals を特定する。
- 変更ファイルを work unit record、関連 spec、
AGENTS.md に照らして確認する。
- protocol、BTstack、backend、実機事実の根拠監査状態を記録する。
- 検証コマンドと正確な結果を記録する。
- 実機状態は自動テストと分けて記録する。
- BTstack license / notice impact を記録する。
- 弱い根拠、間接的な根拠、欠けている根拠は not proven として扱う。
判定表
次の判定項目を使う。
| 判定項目 | 結果 | 根拠 |
|---|
| Requirements | pass/fail/not proven | work unit record、関連 spec、ユーザ目的の充足範囲 |
| Non-goals | pass/fail | scope が広がっていないこと |
| Source Audit | pass/not applicable/not run | protocol/BTstack facts の根拠 |
| Tests | pass/fail/not run | CTest または targeted command |
| Static / Build | pass/fail/not run | CMake configure/build、sanitizer、cross build |
| Hardware | pass/fail/not run/not applicable | 承認、アダプター、結果、または理由 |
| BTstack / License | pass/not applicable/not proven | submodule untouched、notices checked |
| Integration Review | pass/fail/not proven | 境界に照らした diff review |
指摘の列挙
問題がある場合は、summary より前に指摘を列挙する。
### 指摘
| 重要度 | 指摘 | 根拠 | 推奨対応 |
|---|---|---|---|
問題がない場合でも、残るリスクと未実行の判定項目を列挙する。
報告テンプレート
## swbt セルフRv
### work unit
- 対象:
- 意図:
- 対象外:
### 指摘
| 重要度 | 指摘 | 根拠 | 推奨対応 |
|---|---|---|---|
### 判定
| 判定項目 | 結果 | 根拠 |
|---|---|---|
### 検証
- コマンド:
- not run:
### 根拠 / 実機
- 根拠監査:
- 実機:
- BTstack/license:
### 次
- 追加確認:
- open risk:
ルール
- テストが通っただけで work unit を完了としない。根拠を要件と対応させる。
- 単体テストの成功で実機未実行状態を隠さない。
- 明示的な根拠監査と license review なしに、
vendor/btstack の変更を安全扱いしない。
- configure、build、test は別の検証ステップとして記録する。
- 標準検証は
just recipe または CI 結果を正本にする。Codex は自分の判断で SWBT_ALLOW_HOST_BUILD=1 を付けた host 検証を primary verification として扱わない。
