Skip to content

feat(skill): 增加 PR 审查反馈处理流程 - #333

Merged
Terry-Mao merged 1 commit into
mainfrom
feat/resolve-pr-conversation-327
Nov 3, 2026
Merged

Terry-Mao merged 1 commit into
mainfrom
feat/resolve-pr-conversation-327

Conversation

@Terry-Mao

Copy link
Copy Markdown
Owner

Summary

  • 新增 resolve-pr skill,处理开放的 review thread、review 正文和 PR 顶层评论,并跟进修复后的 CI。
  • 添加按需读取的 gh 命令参考,覆盖反馈分页、线程回复与 Resolve。

Fixes #327

Testing

  • uv run --no-project --with PyYAML python3 /Users/maojian/.codex/skills/.system/skill-creator/scripts/quick_validate.py .agents/skills/resolve-pr:通过。
  • git diff --check origin/main...HEAD:通过。
  • 在现有 PR 上只读验证 GraphQL 线程查询:通过。

Risk

  • 线程回复和 Resolve 命令未对真实 PR 执行写入验证;使用时仍需具备相应 GitHub 权限。

How to revert

  • 执行 git revert 00f680c 移除新增 skill 及命令参考。

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本次变更新增 resolve-pr 本地 skill 及其 gh 命令参考,覆盖 review thread、review 正文、PR 顶层评论的分页读取,以及回复、Resolve 和 CI 跟进边界。命令与现有 GitHub 辅助脚本的分页及写入模式一致,并明确将反馈和 CI 输出视为不可信数据,未发现需要阻塞合并的问题。PR 已说明写入路径尚未在真实 PR 上验证;该测试缺口属于已知风险,不影响当前 skill 文档的正确性。

@Terry-Mao
Terry-Mao merged commit 3a720aa into main Nov 3, 2026
4 checks passed
@Terry-Mao
Terry-Mao deleted the feat/resolve-pr-conversation-327 branch November 3, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

增加一个 resolve-pr-conversation

1 participant