diff --git a/skills/gitlink-shared/SKILL.md b/skills/gitlink-shared/SKILL.md index d5b6d87..ddc227c 100644 --- a/skills/gitlink-shared/SKILL.md +++ b/skills/gitlink-shared/SKILL.md @@ -10,7 +10,7 @@ metadata: # gitlink-cli 共享规则 -维护者类 Skill 的报告协议见 [`references/maintenance-report-contract.md`](references/maintenance-report-contract.md),安全检查见 [`references/security-review-matrix.md`](references/security-review-matrix.md),五个维护 Skill 的边界和交接见 [`references/skill-scope-and-handoff.md`](references/skill-scope-and-handoff.md)。生成报告时先给执行摘要,再提供可追溯的证据附录;JSON 不得混入展示层样式。 +维护者类 Skill 的报告协议见 [`references/maintenance-report-contract.md`](references/maintenance-report-contract.md),安全检查见 [`references/security-review-matrix.md`](references/security-review-matrix.md),五个维护 Skill 的核心职责、允许重叠范围和交接见 [`references/skill-scope-and-handoff.md`](references/skill-scope-and-handoff.md)。生成报告时先给执行摘要,再提供可追溯的证据附录;JSON 不得混入展示层样式。 本技能指导你如何通过 gitlink-cli 操作 GitLink 平台资源。 diff --git a/skills/gitlink-shared/references/skill-scope-and-handoff.md b/skills/gitlink-shared/references/skill-scope-and-handoff.md index b375d5e..5b7599e 100644 --- a/skills/gitlink-shared/references/skill-scope-and-handoff.md +++ b/skills/gitlink-shared/references/skill-scope-and-handoff.md @@ -4,7 +4,7 @@ ## 职责矩阵 -| Skill | 独占职责 | 明确不负责 | 主要输入 | 主要输出 | +| Skill | 核心职责 | 明确不负责 | 主要输入 | 主要输出 | |------|----------|------------|----------|----------| | `gitlink-code-review` | 代码正确性、可维护性、测试充分性、代码级安全漏洞 | 不判断队列 SLA、PR 间关系或最终合并顺序 | 单个 PR 的 diff、文件、提交、review、测试证据 | `CR-xxx` 发现、修复建议和代码审查结论 | | `gitlink-cli-contract-guard` | flags、help、JSON、错误、退出码、文档和 CLI 边界安全 | 不评价业务设计价值、一般代码风格或 reviewer 负载 | CLI 改动、旧用法、golden 输出、错误路径测试 | `CG-xxx` 契约门禁和兼容性结论 | @@ -12,6 +12,25 @@ | `gitlink-pr-integrator` | 合并态、rebase、构建、测试、契约、安全门禁和发布影响 | 不重新进行完整代码审查或维护者值班排序 | 单 PR 证据、其他 Skill 结论、主线和 CI 状态 | `IN-xxx` 集成门禁、决策和合并后动作 | | `gitlink-maintainer-radar` | 首响 SLA、reviewer 负载、责任停滞、等待方和队列变化 | 不判断代码漏洞、CLI 兼容性或 PR 功能优劣 | 队列快照、review 状态、评论时间、分配关系和安全优先级 | `MR-xxx` 维护动作、责任调整和催办建议 | +## 允许的功能重叠 + +重叠本身不是问题,关键是不能让一个 Skill 的完整功能覆盖另一个 Skill。以下能力可以被多个 Skill 使用: + +- **证据采集**:多个 Skill 可以读取同一个 PR 上下文、Diff、Review、CI 和评论,但不应各自产生互不一致的事实。 +- **安全信号**:多个 Skill 可以发现安全相关信号,但必须按照不同层次输出;代码漏洞、CLI 边界、安全热点、合并门禁和维护优先级不能混为一谈。 +- **测试状态**:代码审查关注测试是否覆盖行为,契约守卫关注兼容性回归测试,集成器关注主线合并后的构建测试是否通过。 +- **报告格式**:所有 Skill 都可以使用统一的执行摘要、严重性、颜色和 JSON 字段,但发现编号和最终决策必须保持各自前缀与职责。 +- **排序信息**:拓扑提供依赖顺序,集成器提供合并顺序,维护雷达提供值班顺序;三者可能引用同一 PR,但排序依据不同。 + +以下情况视为错误设计: + +- 代码审查已经替代 CLI 契约守卫的 flags/help/JSON 兼容检查。 +- 集成器直接替代代码审查,凭“构建通过”推断代码质量和安全通过。 +- 维护雷达直接决定 PR 是否可合并,或拓扑直接判定代码实现优劣。 +- 一个综合 Skill 包含其他四个 Skill 的全部输入、规则、输出和决策,导致其他 Skill 只剩转发作用。 + +每个 Skill 至少保留一个不可替代的决策对象:代码问题、CLI 契约、PR 关系、集成门禁或维护动作。组合运行只是汇总这些判断,不把它们压扁成一个万能 Skill。 + ## 安全职责分层 安全可以在多个 Skill 中出现,但检查对象不同,不能重复输出同一条泛化结论: