Skill v1.0.1

currentAutomated scan100/100

timzaak/web-dev-skills/t-simplify

+2 new

──Details

PublishedOctober 25, 2026 at 08:09 PM

Content Hashsha256:b1d9eb21e12d28b7...

Git SHAff59146a7d10

Bump Typepatch

Compare with v1.0.0

──Files

Files (1 file, 5.4 KB)

SKILL.md5.4 KBactive

SKILL.md · 106 lines · 5.4 KB

version: "1.0.1" name: t-simplify description: Review the changed code for reuse, simplification, efficiency, and altitude issues with 4 parallel read-only reviewers, then apply the fixes directly. The default scope is the upstream range plus uncommitted changes; a PR / branch / file target replaces it. Quality cleanup only, not bug hunting. argument-hint: "[<target>（可选：PR / 分支 / 路径 / revision range）]" allowed-tools:

Read
Glob
Grep
Bash
Edit
Write
Agent

变更代码清理

t-simplify → 4 个清理审查 agent 并行 → 直接应用修复

运行时边界：${CLAUDE_PLUGIN_ROOT}/protocols/runtime-boundaries.md（判断脚本入口或项目事实与插件默认冲突时读）

目标

提升本次变更代码的质量，不是找 bug：从复用、简化、效率、抽象层级四个角度审查并直接修复。
不找正确性缺陷——那是 /t-tools:t-review 与各阶段 accept 的职责。
修复保持行为不变：改变预期行为的修法一律跳过。

使用方式

bash

/t-tools:t-simplify [<target>]

参数	说明
`[<target>]`	可选审查目标（PR / 分支 / 路径 / Git revision range）；按共享范围契约解析并替换默认范围

推荐位置：/t-run（或 /t-super-run）完成实现与测试后、/t-push 提交前。

共享契约

审查角度定义、finding 结构、去重/跳过/修复边界和报告结构统一参考：${CLAUDE_PLUGIN_ROOT}/protocols/simplify-cleanup-contract.md

变更收集

先读取 ${CLAUDE_PLUGIN_ROOT}/protocols/review-scope-contract.md，按默认范围或显式目标收集 diff、文件版本和项目约束；范围不可确定时停止，成功解析且为空时返回 NO_CHANGES。

执行流程

Phase 1 — 审查（4 个清理 agent 并行）

按 ${CLAUDE_PLUGIN_ROOT}/protocols/subagent-dispatch.md，先 Read ${CLAUDE_PLUGIN_ROOT}/agents/simplify-reviewer.md 全文（同批次复用，见该协议的 token 优化规则）。
通过 Agent tool 在同一条消息里启动 4 个 simplify-reviewer sub agent，使其并行执行。每个 prompt 包含：

agent 角色规范全文（作为首段角色指令段）
一个角度的完整指引（从 ${CLAUDE_PLUGIN_ROOT}/protocols/simplify-cleanup-contract.md 摘取：Reuse / Simplification / Efficiency / Altitude 各一个，不重复、不遗漏）
变更文件清单和获取完整 diff 的命令

每个 reviewer 返回 findings：file、line、单行 summary、具体 cost（什么被重复、浪费或更难维护）。

Phase 2 — 应用修复

等待全部 4 个 agent 完成，按契约去重：指向同一行或同一机制的发现合并。
按共享范围契约核对本地文件与受审版本，再直接修复剩余发现；每条修复前按 ${CLAUDE_PLUGIN_ROOT}/protocols/simplify-cleanup-contract.md 的 Skip Rules 判断是否跳过。
修复后运行能覆盖所改文件的最小定向验证（编译/类型检查/相关测试）；验证失败时回退该条修复并记为跳过。
按契约把报告写入 .ai/quality/simplify-[YYYYMMDD-HHMMSS].md，并以简要总结收尾：修了什么、跳了什么（含原因），或确认代码已干净。

Inline 降级

Agent tool 在当前上下文不可用时，不终止：主会话在当前上下文按四个角度单遍依次审查全部变更，不得因无法并行而跳过任何角度；Phase 2 与报告要求不变，且报告和总结必须声明这是单主会话单遍审查、不是 4-agent 并行审查。

错误处理

错误码	触发条件	用户可见提示	恢复动作
`NO_CHANGES`	变更收集结果为空	没有可审查的变更	先实现变更后再运行
`SCOPE_UNAVAILABLE`	目标无法解析或没有范围依据	无法确定审查范围（附原因）	补齐 PR 工具/引用或明确指定目标后重跑
`AGENT_FAILED`	reviewer 失败或角度结果缺失	清理审查未完成，尚未应用修改	按契约重试失败项，不能视作无发现
`AGENT_UNAVAILABLE`	Agent tool 不可用	已切换为单遍 inline 审查	无需恢复；报告会如实声明
`DIFF_TOO_LARGE`	变更规模超出单次可审查范围	变更过大，建议缩小范围	按文件分批运行，或先提交部分变更

Forbidden

找正确性 bug、提出会改变预期行为的修复。
修复需要明显超出本次审查 diff 的改动（共享 helper 新建除外，见契约 Skip Rules）。
reviewer sub agent 修改代码或文档。
跳过任何审查角度，或串行分批启动本应并行的 4 个 reviewer。
把 inline 降级审查冒充 4-agent 并行审查写入报告。

示例

bash

/t-tools:t-simplify
/t-tools:t-simplify feat/user-management
/t-tools:t-simplify src/api/orders.rs

输出：

text

已修复 3 项，跳过 1 项（超出本次 diff 范围）
- src/api/users.rs:42 | Reuse | 重复实现既有 validate_tenant | 改用 shared/tenancy::validate
- src/api/users.rs:88 | Simplification | 单调用方的间接包装层 | 内联展开
- web/src/routes/posts.tsx:31 | Efficiency | 列表 render 内逐项新建排序数组 | 提出循环并 memo
- 跳过: src/domain/order.rs:120 | Altitude | 共享函数新增特例分支 | 修复需重构底层机制，超出范围
报告: .ai/quality/simplify-20260815-103000.md

← v1.0.0 All versions