無料GitHub で公開日本語紹介

test-audit

新規・既存テストが守る動作と不具合を確かめ、重複や実装への依存を見直すスキル。削除の根拠を整理し、必要な検証を残してテスト用コードも整理します。

原文原文の説明を見る

Writing/changing/reviewing/sweeping tests: reject duplicate or implementation-coupled proof; remove test-only production seams.

こんなときに便利

ソフトウェアのテストが実際に何を守るかを点検し、新規作成時の判断と既存テストの整理を支援します。確認するのは、外から観測できる動作、検出できる不具合、既存テストとの重複、テストだけのために公開した関数などです。削除前に検出能力や残る検証、経緯、リスクを記録し、必要なテストを残します。

動作を変えないコード整理でもテストが壊れる場合や、似たテストが増えている場合に向いています。不具合の再発防止テストが修正前に意図した理由で失敗するかも確認します。遅いテストやソースを調べるテストも、それだけを理由に削除せず、独立した検証としての価値を判断します。

資料の検証手順はOpenClawの環境を前提とし、Vitest、リポジトリ内の検証スクリプト、関連スキルを参照します。サブシステム全体の整理ではCAMPAIGN.mdの確認が必要です。調査は読み取りから始め、コミットやPR作成などは許可された場合に行います。

この紹介文は、公開されている SKILL.md をもとに AI（Claude Haiku）が作成しました。正確な仕様は下の原文を確認してください。

インストールする前に、エージェントに与えられる指示の中身を確認できます。

Authoring gates every new/changed test.
Audits: remove low-value proof in coherent owner batches; never chase deletion counts.
Whole-subsystem campaign: read CAMPAIGN.md first.

Before adding test, answer:

Protected observable behavior, invariant, or independent contract?
Credible regression that makes it fail?
Why existing proof misses it? Each contract: one primary owner at strongest boundary. Another layer needs distinct transport/lifecycle risk. Extend tables/shared fixtures; consolidate repeated setup.
Export, flag, wrapper, injection hook with no production caller needed? Move proof to real boundary instead.

Use boundary and cost guide.
Regressions must fail pre-fix for intended reason, pass after repair; report unavailable baseline proof.
One owner regression covers bug; never replay at every crossed layer.
Behavior-preserving refactors should not break test.

Reject self-comparisons, copied inventories, source greps, implementation/call-shape assertions, duplicate contracts, test-only seams, subject-generated expected values, mocks supplying asserted behavior.
Fixtures must not fabricate owner receipts/order or claim persistence path never writes.
Negative controls must fail at intended guard; names must match exercised behavior.
Auditing candidates: use full checklist.

Keep independent API, SDK, protocol, config, migration, storage, security, platform, default, prompt-byte, generated-language, package, release, architecture, observable-ordering contracts. Static/slow ≠ disposable.
Source inspection must survive identifier-only refactors and guard independent user-facing contract.
Baseline failures may be bugs to repair, not tests to delete.

Before audit edit: read candidate evidence and workflow; inspect full owners/callers/history; record remaining proof before deletion.
Delete obsolete production seams with tests; never optimize for deletion counts.
Follow proof policy; preserve active Vitest checkouts; run owner/sibling proof + changed checks; then autoreview.
Report causes, retained contracts, proof/cost, production versus test LOC, follow-ups. Commit/push/land only with task authority.

まだレビューはありません。使ってみた感想をお寄せください。

概要と使いどころ