feat(skills): 丰富五个维护 Skill 的协同审查能力 #428

Closed
Mengz wants to merge 4 commits from Mengz/gitlink-cli:mengz/skill-maintenance-efficiency into master
Contributor

变更说明

这是对已经合并的五个维护类 Skill 的增量优化 PR,重点不是新增第六个总控 Skill,而是让现有五个 Skill 在单独运行时保持职责独立,在组合运行时共享统一证据并减少重复请求。本 PR 依赖前置基础能力 PR #426 和 PR #427:#426 提供单 PR 证据包,#427 提供 PR 队列快照差异。

对五个 Skill 的完善

  • gitlink-code-review:复用变更文件、Review、提交记录和 CI 证据,补充安全与测试结论的证据完整性判断;不替代集成门禁。
  • gitlink-pr-integrator:把提交记录和 CI 结果纳入合并态验证,同时继续要求独立 worktree、构建、测试和契约检查;不替代代码审查。
  • gitlink-pr-topology:使用队列差异作为关系图谱的增量输入,优先分析新出现或仍在变化的依赖、重叠和冲突;不把队列变化误判为代码缺陷。
  • gitlink-maintainer-radar:优先展示新增、优先级变化、风险变化和已解决事项,将未变化项压缩为数量;继续只负责 SLA、Reviewer 负载、责任停滞和今日待办。
  • gitlink-cli-contract-guard:把两个前置 PR 的新 flags、默认值和可选 JSON 字段纳入契约检查,验证旧调用兼容、帮助文本、JSON 无 ANSI/HTML/敏感值;不代替业务和集成判断。

协同与独立运行

新增共享文档明确了证据包、队列快照、sections、notes、changes 和各 Skill 的消费边界。每个 Skill 可以独立运行,只输出自己的 CR/IN/TP/MR/CG 前缀结论;组合运行时允许共享事实和安全信号,但不会让任何一个 Skill 覆盖另一个 Skill 的完整职责。

报告效率

五个 Skill 统一采用先结论、再最多五条行动、最后证据附录的报告结构,并区分 passed、failed、not_run、not_applicable,避免维护者阅读大量重复文本。Markdown 使用醒目的重点格式,JSON 保持稳定字段和纯机器可读内容。

验证

  • go build ./... 通过。
  • UTF-8 替换字符和 NUL 扫描通过。
  • git diff --check 通过。
  • 目标五个 Skill 的仓库 frontmatter 检查通过。
  • 仓库整体 go test ./internal/skillmeta -run TestRepoSkillsValid 仍被其他历史 Skill 元数据问题阻断,错误列表不包含本次修改的五个目标 Skill。
  • 前置 PR #426 的证据包测试和构建已通过;前置 PR #427 的队列差异、非法快照和 Markdown 输出测试已通过。

关联基础 PR

  • #426:为五个维护 Skill 提供 PR 证据包。
  • #427:增加 PR 审查队列快照差异。
## 变更说明 这是对已经合并的五个维护类 Skill 的增量优化 PR,重点不是新增第六个总控 Skill,而是让现有五个 Skill 在单独运行时保持职责独立,在组合运行时共享统一证据并减少重复请求。本 PR 依赖前置基础能力 PR #426 和 PR #427:#426 提供单 PR 证据包,#427 提供 PR 队列快照差异。 ## 对五个 Skill 的完善 - gitlink-code-review:复用变更文件、Review、提交记录和 CI 证据,补充安全与测试结论的证据完整性判断;不替代集成门禁。 - gitlink-pr-integrator:把提交记录和 CI 结果纳入合并态验证,同时继续要求独立 worktree、构建、测试和契约检查;不替代代码审查。 - gitlink-pr-topology:使用队列差异作为关系图谱的增量输入,优先分析新出现或仍在变化的依赖、重叠和冲突;不把队列变化误判为代码缺陷。 - gitlink-maintainer-radar:优先展示新增、优先级变化、风险变化和已解决事项,将未变化项压缩为数量;继续只负责 SLA、Reviewer 负载、责任停滞和今日待办。 - gitlink-cli-contract-guard:把两个前置 PR 的新 flags、默认值和可选 JSON 字段纳入契约检查,验证旧调用兼容、帮助文本、JSON 无 ANSI/HTML/敏感值;不代替业务和集成判断。 ## 协同与独立运行 新增共享文档明确了证据包、队列快照、sections、notes、changes 和各 Skill 的消费边界。每个 Skill 可以独立运行,只输出自己的 CR/IN/TP/MR/CG 前缀结论;组合运行时允许共享事实和安全信号,但不会让任何一个 Skill 覆盖另一个 Skill 的完整职责。 ## 报告效率 五个 Skill 统一采用先结论、再最多五条行动、最后证据附录的报告结构,并区分 passed、failed、not_run、not_applicable,避免维护者阅读大量重复文本。Markdown 使用醒目的重点格式,JSON 保持稳定字段和纯机器可读内容。 ## 验证 - go build ./... 通过。 - UTF-8 替换字符和 NUL 扫描通过。 - git diff --check 通过。 - 目标五个 Skill 的仓库 frontmatter 检查通过。 - 仓库整体 go test ./internal/skillmeta -run TestRepoSkillsValid 仍被其他历史 Skill 元数据问题阻断,错误列表不包含本次修改的五个目标 Skill。 - 前置 PR #426 的证据包测试和构建已通过;前置 PR #427 的队列差异、非法快照和 Markdown 输出测试已通过。 ## 关联基础 PR - #426:为五个维护 Skill 提供 PR 证据包。 - #427:增加 PR 审查队列快照差异。
Mengz added 4 commits 2026-07-20 16:27:18 +08:00
Mengz closed this pull request 2026-07-20 17:16:06 +08:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Gitlink/gitlink-cli#428
No description provided.