Skip to content

docs(pr-fix): add path routing for internal branch vs fork PRs#1467

Merged
IceyLiu merged 1 commit intomainfrom
zynx/feat/pr-fix-path-routing
Mar 19, 2026
Merged

docs(pr-fix): add path routing for internal branch vs fork PRs#1467
IceyLiu merged 1 commit intomainfrom
zynx/feat/pr-fix-path-routing

Conversation

@piorpua
Copy link
Copy Markdown
Contributor

@piorpua piorpua commented Mar 19, 2026

Summary

  • Add pre-flight detection of PR state and isCrossRepository in Step 2
  • Route to Path A (create follow-up PR) when the original PR is merged
  • Route to Path B (push to existing head branch) when PR is open and internal
  • Abort with a clear message when PR is open but from an external fork

Changes

.claude/skills/pr-fix/SKILL.md

  • Step 2: add two new parallel checks (state and isCrossRepository) with a routing table
  • Step 3: split into Path A (checkout new fix branch) and Path B (checkout existing head branch)
  • Step 7: split into Path A (gh pr create) and Path B (git push, no new PR)
  • Step 8: both paths post verification report as PR comment — Path A to the new PR, Path B to the original PR
  • Quick Reference: updated to reflect the new three-way routing

Related Issue

Closes #1466

Test Plan

  • Merged PR scenario → enters Path A, creates follow-up PR, posts comment to new PR
  • Open internal-branch PR scenario → enters Path B, pushes to head branch, posts comment to original PR
  • Open fork PR scenario → aborts with prompt to wait for merge

- Detect PR state (merged/open) and isCrossRepository in pre-flight
- Path A: create follow-up PR when original PR is merged
- Path B: push directly to head branch when PR is open and internal
- Abort with clear message when PR is open and from external fork
- Both paths post verification report as PR comment
@IceyLiu IceyLiu merged commit a282ae5 into main Mar 19, 2026
4 checks passed
@piorpua piorpua deleted the zynx/feat/pr-fix-path-routing branch March 19, 2026 10:51
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.

docs(pr-fix): add path routing for internal branch vs fork PRs

2 participants