ln-61-skill-reviewer

Skill 审查

目标： 审查技能质量和已配置的分发渠道，不修改仓库或外部状态。区分静态契约保障与实际观察到的执行行为。

执行契约： 按顺序排列的复选框即为完成定义。内部将每项标记为 PENDING、附有具体证据的 PROVEN、附有条件不存在证据的 CLEARED，或存在缺口的 UNPROVEN；阅读、委派或工具故障均不构成证明。每完成一节后都要核对各项。返回前，处理所有 PENDING 项，仅统计 PROVEN 和 CLEARED 项；针对每个缺口，遵循技能的判定和批准规则。 保留用户意图、范围和现有授权。继续已获授权的工作；仅针对尚未解决且影响重大的选择或必需的外部批准进行询问。根据实际风险调整审查深度，不得悄然跳过检查。保留依赖关系和安全顺序；其他情况下，选择适合各项要求的验证方法。

工具路由

需求	首选能力	回退方案
仓库范围和证据	原生文件搜索、重点读取和 Git 差异	等效的只读 Shell 命令
Frontmatter 和技能结构	仓库定义的或宿主原生的技能验证器	手动检查 YAML、路径、命名和仓库契约
插件集成	仓库定义的插件验证器及清单解析	手动检查清单，并与适用的目录进行比对
宿主发现	针对每个已配置分发渠道使用宿主原生验证器	手动检查目录、清单和源路径
当前宿主规则	宿主官方文档	无法获取时，将该声明标记为 UNVERIFIED
行为独立性	全新子 Agent 或干净上下文	通过降低置信度并披露这一点，分别进行多轮自我审查

仅在需要确认当前宿主行为或不断变化的标准时，才进行外部研究。不得使用网络来源推翻仓库中的实际文件、已安装版本或本地验证输出。

缺少工具本身并不代表技能存在缺陷。采用文档所述的回退方案；只有当缺失的能力妨碍可靠判定时，才使用 BLOCKED。

检查清单

确定范围并收集证据

• 确认目标路径、Git 差异、暂存文件和未跟踪文件；确定审查模式：静态内容/分发审查，或在用户要求、确有必要且获准时进行行为验证。遵守明确的禁止测试指示；仅限静态审查时，不要运行场景或委派评估。
• 在判断仓库特定约定之前，阅读仓库说明、技能契约以及所有已配置的宿主目录；不要要求仓库未分发的目录。
• 区分主要技能与受同一变更影响的清单、目录和文档。
• 确认仓库定义的规范技能布局；将未经授权、过时或与规范不一致的宿主专属副本视为缺陷，同时允许仓库策略明确要求的适配器或生成副本。
• 确认前置元数据和文件夹命名符合仓库契约及各目标宿主的要求；如果本地约定声明只需 name 和 description，则只要求这两项，不要将其普遍强加于所有项目。
• 检查每项描述是否说明了能力、正向触发条件和重要的近似负向边界。
• 检查描述是否符合宿主限制，且没有作出超出工作流支持范围的主张。
• 确认每项技能均可独立运行：不依赖必需的技能、MCP 服务、跟踪器、协调器、工作器或共享运行时。
• 遵循仓库声明的完成约定；如果有序清单就是完成定义（Definition of Done），则标记重复的完成章节。
• 保留领域算法、工具路由、安全门槛、证据、结论、输出和剩余风险。对于已删除的检查项，指出保留该义务的责任方，或说明该检查为何不支持目标；仅凭条目数量既不能证明内容丢失，也不能证明内容得到保留。
• 标记不能提供决策价值的通用解释；即使此前没有记录过相关失败，也要保留不显而易见的领域知识和具体保障措施。
• 检查内容层级和单一事实来源的归属：每条规则应放在范围最窄的规范章节中，并标记正文、辅助文件、清单和仓库说明之间重复或相互矛盾的指导。
• 标记填充内容、生成式摘要文案、复制的实现或业务逻辑，以及可用稳定契约、权威来源、能力描述或明确更新触发条件替代的易变版本、路径、数量、默认值或宿主行为。
• 确认每项必需能力都有可用工具路径、可信的备用方案，或明确的 BLOCKED 结果。
• 检查每项技能的变更边界是否符合其声明的结果；只读工作流不得隐含获得写入权限。
• 对优化或实验类技能，如果会修改状态，则要求基于证据作出保留、放弃或回滚的决定。
• 对测试构建或其他有界写入技能，确认除非其声明的契约明确授权，否则它们不能修复产品代码或触及未经批准的外部状态。
• 跟踪文本在正常完成、工具缺失、上下文不足、Git 状态不干净、命令失败和证据冲突等情况下的表现。检查前置条件是否先于依赖操作、停止条件是否允许安全推进，以及结论是否涵盖失败状态且不存在矛盾。
• 检查输出契约是否区分事实、推断、缺失证据、结论和剩余风险。
• 将相邻技能的触发条件与每项技能的目标进行比较；要求相关说明能够支持某项结果、证据需求或范围/安全约束。标记无关工作、遗漏义务和含糊分支，同时避免让技能彼此依赖。

仓库验证

• 查明并运行仓库要求的所有技能验证器，覆盖已更改的技能目录；不得在缺乏仓库证据的情况下臆测验证器名称或位置。
• 如果无法使用必需的验证器，请依据仓库和宿主契约，手动验证 YAML 解析、命名、描述约束和必需的文件布局。
• 运行受影响分发单元所需的所有仓库插件或软件包验证器。
• 如果存在相应目录或清单，则运行每个宿主原生严格验证器；记录其实际覆盖范围。除非能证明市场验证会遍历这些技能，否则不得将其视为技能 frontmatter 验证。
• 解析所有已配置的目录；仅当仓库政策要求跨宿主保持一致时，才比较插件名称和顺序。
• 确认所有声明的目录来源、清单路径和技能路径均存在。
• 在仓库要求保持一致的所有位置，确认重复元数据（例如插件描述）彼此一致。
• 对于市场仓库，确认稳定标识符未被意外更改；展示品牌不能替代标识符。
• 搜索过时名称、已删除路径、草稿标记、MCP 耦合、编排相关术语和生成的副本。
• 仅当仓库政策规定行数目标时才执行；并将其视为维护约束，而非行为质量的证据。

范围内的行为审查

• 仅当行为执行属于范围且已获授权时，才执行此阶段；否则，将仅适用于执行的项目标记为 CLEARED，注明范围原因，并说明未测试行为。静态检查不能证明技能已激活或执行质量良好。
• 根据指定的触发条件推导具有代表性的正向提示词，而非仅依据技能标题。
• 根据相邻能力和用户可能使用的模糊措辞，推导相近的负向提示词。
• 验证技能会因正向提示词而激活，并且对相近的负向提示词保持未激活状态。
• 至少演练一个正常场景、一个缺少证据的场景，以及一个安全边界场景。
• 当技能会编写代码或其他产物时，评估 Agent 实际生成的结果，并与独立的任务契约进行差异比较；不得以文字审查、答案简短程度或静态检查替代对执行行为的评估。
• 分别评定任务完整性、正确性、安全性、范围控制和清理情况；只有在所需结果达标后，才将代码或令牌减少作为辅助证据。
• 对复杂或高风险更改，在条件允许时使用全新的独立上下文，且只提供技能和原始任务产物。
• 不得向独立评估者透露预期发现、计划修复内容或先前结论。
• 如果前向测试只有在泄露上下文的情况下才能成功，则视为技能缺陷。
• 检查独立上下文是否受到全局安装的技能、钩子、插件、用户指令、环境设置、缓存或产物的干扰；证明目标技能已激活，并使行为无法可靠归因的测试组无效。
• 除非具体且反复出现的故障证明确有必要，否则不得创建或保留评估测试框架。

证据规则

• 将每项发现引用到确切文件及最小的有效行或章节。
• 区分验证器失败、人工关注事项和推测性改进。
• 只有在有确定性的验证器证据、经授权的复现，或完整的静态矛盾／失败路径时，才判定为阻断级别；不得执行审查模式排除的场景。
• 记录已执行的命令、退出状态和重要输出，不得泄露机密。
• 如果无法运行必需的宿主环境或干净上下文测试，请报告覆盖缺口。任何允许使用的诊断产物都必须是临时文件，存放在规范的插件目录之外，并予以披露；审查期间不得暂存、发布或创建外部状态。

结论

• PASS — 声明模式所需的所有检查均已通过，且不存在实质性关注事项；仅静态审查通过并不代表执行行为已获认证。
• PASS WITH CONCERNS — 在已审查范围内没有确认的阻断缺陷，但仍存在有限的非阻断性不确定因素；不得据此推断未经测试的发布或行为已做好准备。
• FAIL — 已确认的缺陷可能导致触发错误、行为不安全、安装失败或契约无效。
• BLOCKED — 所需证据或工具不可用，且不存在可信的替代方案。

自检

• 返回前核对。 检查各项证据、需求覆盖情况、矛盾、范围、结论及适用的清理事项。修正报告或经授权的产物。复用有效证据；不要自动重新扫描仓库或重新运行已成功的命令。仅针对相关变更、失败或未解决的证据重复检查。披露剩余缺口。

输出契约

使用用户的语言，按以下顺序报告；保留全部五个字段，每项事实只陈述一次。结果较少时，每个字段可占一行；省略空表格，不要复制链接中的产物：

1. 结果： 技能专属结论及有证据支持的结果。
2. 范围： 已审查／变更的范围、排除项、基线和重要假设。
3. 证据： 下方所列的技能专属字段；区分事实、推断和未经验证的声明。提供产物链接；适用时使用表格。
4. 验证： 检查结果、不可用的证据，以及适用的清理工作／外部状态。
5. 完成情况： Checklist: X/Y complete；Incomplete: None，或列出每个 UNPROVEN 项目的原因、对结果的影响及确切的后续操作；剩余风险和所需决策。

技能专属证据： 目标技能和分发渠道；结构／宿主一致性、触发边界、静态与行为覆盖情况，以及验证器结果。发现项必须包含严重程度、确切位置、影响、证据和最小修正方案。区分已确认的缺陷、人工关注事项和推测性改进；报告不可用的干净上下文或运行时证据，以及已披露的临时产物。