关于 ai-slop-cleaner
这是用于限定范围内清理和重构冗余、重复或 AI 生成代码的 OMX 任务卡。它强调用针对性测试保持行为不变、分类回退逻辑并记录验证证据。
主要功能
- 01通过回归测试锁定行为
- 02分类类似回退的代码
- 03删除死代码和重复逻辑
- 04检查边界与抽象
- 05记录验证和清理证据
适用场景
- 01清理限定范围内的重复或未使用代码
- 02检查先前实现遗留的问题
- 03重构时保留兼容与安全回退路径
- 04记录清理所做的测试和质量检查
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ yeachan-heo/ oh-my-codex --skill ai-slop-cleaner安装后在 Claude Code 中输入/ai-slop-cleaner确认已加载。
详细介绍
AI Slop 清理任务卡
将此有界辅助工具用于清理 / 重构 / 去 slop 工作,而不要将其用作与之竞争的上层工作流。共享操作不变量位于 templates/AGENTS.md 中;本卡片定义范围、坏味道分类、处理轮次和证据。
何时使用及输入
当可运行的代码出现臃肿、嘈杂、重复、过度抽象或由 AI 生成时;当用户请求清理 / 重构 / 去 slop 时;或后续工作留下了重复 / 死代码、薄弱边界、缺失测试、类回退路径或包装器时使用。输入是所请求的功能 / 文件以及要保留的行为。文件列表范围是有效的;将处理轮次限制在该范围内。除非请求了更广泛的清理,否则将处理轮次限制在调用任务所更改的文件内。
编辑之前
- 先用回归测试锁定行为:识别要保留的行为,运行 / 添加最窄范围的目标测试,并覆盖主要路径以及要保留的兼容性 / 故障安全回退路径。
- 在编写代码前制定清理计划:列出范围和坏味道,包含回退发现 / 分类 / 升级,并将最安全 / 信号最强的修复排在最前面。
- 清点范围内类回退代码:快速 hack、临时变通方案、临时回退、直接绕过、直接跳过、失败时回退、被吞掉的错误、静默默认值、宽泛的兼容性垫片,以及重复的备用执行路径。
- 对每个回退进行分类:掩盖型回退 slop 隐藏证据、绕过契约、抑制校验、吞掉失败、静默使用默认值,或添加未测试的路径;有依据的兼容性 / 故障安全回退 在外部 / 版本 / 故障安全边界处范围狭窄,记录理由,保留失败证据,并测试主要路径和回退路径。
- 优先选择根因修复、删除、边界修复或显式失败行为。对于宽泛 / 模糊 / 跨层 / 架构性的发现,调用
$ralplan进行共识解决;当已经处于 ralplan、ultragoal、team 或其他 OMX 工作流中时,不要生成嵌套的$ralplan——将发现附加到活动的交接中。
坏味道分类与处理轮次
在更改前先分类:
- 类回退代码:掩盖型回退、变通分支、绕过、被吞掉的错误、静默默认值、宽泛垫片、备用路径。
- 重复:重复逻辑、复制粘贴分支、冗余辅助函数。
- 死代码:未使用 / 不可达代码、过期标志、调试残留。
- 不必要的抽象:直通包装器、投机性间接层、一次性层。
- 边界违规:隐藏耦合、职责泄漏、错误层级的导入 / 副作用。
- 缺失测试:行为未锁定或边界情况未覆盖。
- UI / 设计 slop:上下文相关信号,而非绝对禁令;保留有意的品牌、设计系统、无障碍或产品上下文例外。质疑 11-12px 的韩文正文文本(通常应为 14px 或更大);无谓的盒阴影;重复的眉题 + 标题 + 描述 + 段落堆叠和通用 emoji 徽章;默认 AI 蓝 / 紫,例如
#3B82F6;反射式的 3 列或 4 列网格;以及除非上下文证明合理,否则应避免的极端渐变。
先解决类回退代码处理门槛,然后一次处理一个坏味道: 第 1 轮:死代码删除;第 2 轮:重复移除;第 3 轮:命名 / 错误处理清理;第 4 轮:测试强化。每轮后重新运行目标验证,并避免无关重构。优先删除 / 使用现有工具;除非明确要求,否则不要引入新的抽象或依赖。
证据 / 输出契约
报告:
AI SLOP CLEANUP REPORT
Scope: [files/feature]
Behavior Lock: [targeted tests added/run]
Cleanup Plan: [bounded smells/order]
Fallback Findings: [finding -> masking fallback slop | grounded compatibility/fail-safe fallback -> escalation]
UI/Design Findings: [none/N/A or signal -> action/defer -> intentional rationale]
Passes Completed: [resolution gate; Passes 1-4]
Quality Gates: Regression tests, Lint, Typecheck, Tests, Static/security scan (PASS/FAIL/N/A)
Changed Files: [path -> simplification]
Remaining Risks: [none or deferred item]
包括已更改文件、简化、回退分类 / 升级状态、已运行的测试 / 诊断 / 构建检查、相关时的 UI 发现,以及延期风险。对清理计划和批准保持编写者 / 审查者分离。
退出条件
当所请求的范围具备行为锁定证据、每个选定的坏味道检查都已通过或已明确延期并说明理由、验证结果已报告,且没有无关文件或临时产物残留时,即可停止。切勿将未经核实的清理呈现为已完成;遇到真正的架构阻塞时应上报,而不是加以掩盖。
评论
支持 Markdown