Skip to content

docs(skill): 让 resolve-pr 定位 PR 分支并处理冲突 - #342

Merged
Terry-Mao merged 2 commits into
mainfrom
docs/resolve-pr-conflicts
Nov 4, 2026
Merged

Terry-Mao merged 2 commits into
mainfrom
docs/resolve-pr-conflicts

Conversation

@Terry-Mao

Copy link
Copy Markdown
Owner

Summary

  • resolve-pr 在编辑前核对 PR head、当前分支和现有 worktree;已在目标分支时保持当前工作区,否则复用或创建目标分支的独立 worktree。
  • 遇到未合并路径时调用 resolve-merge-conflicts,完成冲突处理和验证后继续 PR 修复。
  • 报告中明确写出使用的分支与 worktree。

Testing

  • git diff --check 通过。
  • 手动核对 resolve-pr 与 git-worktree、resolve-merge-conflicts 的边界;本次仅修改技能说明,无自动化测试。

Risk

后续执行 resolve-pr 时会更早检查分支和工作区状态;遇到分叉、路径占用或 fork 无写入权限会在编辑前停止。

How to revert

合并后执行 git revert 067f327,恢复此前的技能流程。

@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.

该 PR 仅修改 resolve-pr 技能文档,安全审查未发现新增风险。新增的 worktree 定位流程与 git-worktree 的实际创建行为不一致,按文档执行可能在本地分支已存在或新建分支未设置 upstream 时中止,需在合并前修正。

Comment thread .agents/skills/resolve-pr/SKILL.md Resolved

@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.

未发现需要阻止合并的新问题。此前关于现有分支与新建分支命令不一致的未解决线程已得到处理:现有分支改为直接挂载,缺失分支使用 --track -b 从 PR head 的远端引用创建,并补充了 upstream 校验与停止条件。

@Terry-Mao
Terry-Mao merged commit b6a024d into main Nov 4, 2026
9 of 10 checks passed
@Terry-Mao
Terry-Mao deleted the docs/resolve-pr-conflicts branch November 4, 2026 10:46
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.

1 participant