test-review

name	test-review
description	Use when auditing an existing retikz test suite, module test directory, or docs tests for stale, duplicate, implementation-coupled, migration-only, or temporary cases; classify which tests remain durable contract or regression coverage and produce evidence-backed cleanup recommendations before refactoring tests.

Test Review：测试资产评审

审查测试本身是否仍在保护当前契约。本 skill 默认只读：输出保留、合并、转正或删除建议，不直接改测试、不提交。

边界

场景	用哪个 skill
审计已有测试是否过期、重复或临时	test-review
横向审实现、架构与文档一致性	develop-review
为疑似缺陷构造边界 case	cross-test
Alpha 实现完成后的对抗性验证	develop-test
已接受测试清理建议并实施	develop-refactor

不要用测试数量、行数或覆盖率百分比直接决定保留与删除。唯一标准是：该 case 是否以最小必要粒度保护仍然存在的行为契约。

启动前

1. 声明范围：一个 workspace、子模块或测试目录；全仓先拆包，不做无边界扫描。
2. 记录 git rev-parse HEAD 和 git status --short，保留并区分并发工作区改动。
3. 读取根与就近 AGENTS.md；审 docs 时再读 apps/docs/AGENTS.md 及相关 docs skill。
4. 列出范围内测试、对应实现入口与公开契约。测试名、注释和 Git 历史只能说明当初意图，当前实现与公开契约才是判断真源。

评审方法

1. 建立行为映射

对每个测试文件或可独立删除的 describe 块，记录：

• 触发的生产入口与用户可见行为
• 精确断言了什么，以及删掉哪种真实回归会失败
• 同一行为的其它测试位置
• 需要时用 git log -- <file>、git blame 或关联 issue / commit 追溯它是否只是迁移或一次性验证

优先从实现入口反查测试，不要只按测试文件名归类。

2. 分类

分类	判定	默认动作
契约测试	保护公开 API、schema、编译/渲染语义或跨包边界	保留
回归测试	对应已确认真实缺陷，且仍能解释风险	保留
行为集成测试	覆盖真实用户路径，且无法由更低层契约替代	保留或收敛
重复测试	与其它 case 保护同一规则，额外 case 不增加失效模式	合并到最接近真实行为的一层
历史护栏	仅保护已完成迁移、旧命名、旧目录或已删除实现	删除
实现耦合测试	断言私有源码、固定文案、内部调用顺序或无意义快照	删除或改成行为断言
临时探索测试	只为当次定位、验证或 canary 存在	验证后删除；确认缺陷才转为回归测试

Snapshot 不是天然无效：只有当它不能表达用户可见输出差异、没有明确失效价值，或被更小的行为断言覆盖时，才建议删除。

3. 给出可安全执行的动作

每条“合并”或“删除”建议必须写清：

• 当前受保护的契约或历史目的
• 过期、重复或耦合的证据
• 替代它的测试文件与具体 case；若没有替代，明确说明删除后风险由何处承担
• 需要保留的最小断言，避免清理时削弱唯一回归守卫

docs 测试额外检查：

• 删除一次性 smoke、静态源码字符串和固定展示文案断言前，确认通用 preview 契约或实际渲染/交互测试仍覆盖该行为。
• zh/en 可共享结构契约，但保留语言内容、路由或可访问性确有不同的测试。
• 同一控件家族优先一个遍历所有定义的通用契约测试，而不是每页复制相同断言。

临时测试生命周期

• 临时 case 只放在受影响 workspace 的 tests/_scratch/，以复用真实 Vitest 配置且默认不会进入 Git。
• 用 pnpm temp:test -- --workspace <workspace-directory> --file <tests/_scratch/*.test.ts> 运行；脚本会在成功或失败后删除文件与空的 _scratch 目录。只有人工连续调试时才传 --keep。
• 只有确认了真实 bug、稳定公开契约或高风险边界时，才重写为具名正式测试并随修复提交。
• 不要 git add -f tests/_scratch/；提交前检查暂存 diff，确认没有 canary、scratch、debug、temporary 或仅服务本轮排查的测试。

报告

报告写入 ignored 路径：

notes/reports/test-review-YYYY-MM-DD-<scope>.md

# Test Review Report: <scope>

日期：
范围：
基准快照：
测试与实现读取范围：

## 结论概览

## 保留

| 测试 | 保护的当前契约 | 保留证据 |

## 合并

| 测试 | 重复证据 | 保留目标 | 最小保留断言 |

## 删除

| 测试 | 分类 | 过期或耦合证据 | 替代覆盖 / 风险承担 |

## 临时测试

| 测试 | 验证结论 | 删除或转正动作 |

## 建议实施顺序

没有某类时写“无”。报告不得 stage 或 commit。

完成标志

• 每个删除或合并建议都关联当前实现与测试证据。
• 未把历史意图误当成现行契约。
• 临时 case 已明确为删除或转正，没有模糊保留项。
• 工作区除 ignored 报告外未被修改。