360 Secure SkillHub
360 安全技能中心

Minimalist Review

该 Skill 用单一视角评审给定代码、diff 或 PR:哪些内容可以在保持正确与安全的前提下删除。逐条输出 file:line — cut/keep — 原因,并给出预计代码行数下限与前三项删减建议。检查顺序涵盖未被需求的过度设计、重复实现、可被标准库替代的新依赖、单调用方抽象、治标补丁与顺手改动;同时把缺失的校验、鉴权、错误处理等列为更高优先级问题。

divyeshjayswal
divyeshjayswal
热度 48.06
最近更新 2026-07-04 00:27:08
分类:开发依赖:纯提示词应用场景:代码审查

安装技能

复制下面的话术发给你的 Agent,它会通过 secure-skills 检索并安装该技能。

执行命令安装此 skill: npx secure-skills-cli add divyeshjayswal/minimalist-review
Metadata
name
minimalist-review
description
Review code, a diff, or a PR strictly for bloat: unrequested abstractions, dead scope, dependency creep, symptom-patching, and drive-by changes. Use when the user says "minimalist review", asks "is this over-engineered?", or wants a leanness review of a change.
license
MIT

Minimalist Review

Review the given code or diff through one lens: what can be removed while staying fully correct and safe?

For each finding output one line: file:line — cut/keep — why (≤15 words). Then a verdict: the estimated LOC floor for this change and the top 3 cuts.

Check, in order:

  1. Scope that no requirement asked for (YAGNI violations).
  2. New code duplicating an existing helper/util/pattern in this repo.
  3. New dependencies replaceable by stdlib/platform/installed deps.
  4. Abstractions with a single caller/implementation.
  5. Symptom patches where a root-cause fix is smaller.
  6. Drive-by refactors, renames, reformatting outside the ticket.

Never flag as bloat: validation, auth, error handling on fallible ops, resource cleanup, tests. If the diff lacks those, flag that instead — missing guards outrank extra lines.