fix: reopen a closed matching PR instead of 422 #9

Merged

stephen merged 1 commit from fix/reopen-closed-pr into main

2026-06-30 21:02:34 +00:00

stephen commented

2026-06-30 20:59:54 +00:00

Owner

When a head/base match existed but was CLOSED, the lookup queried only state=open and fell through to POST /pulls, hard-422ing. Now the lookup includes closed PRs; on a closed match it reopens (PATCH state=open) and updates title/body instead of creating. Test covers the closed-match path.

Backlog #15.

stephen added 1 commit

2026-06-30 20:59:54 +00:00

fix: reopen a closed PR for the same head/base instead of 422

test / unit (pull_request) Successful in 11s

Details

79d70459e8

The existing-PR lookup only queried state=open, so when a matching
head/base PR was closed it was missed and the action fell through to
POST /pulls, which Forgejo rejects with a 422 for the duplicate
head/base.

Include closed PRs in the lookup (state=all). On a closed match, reopen
it (PATCH state=open) and refresh title/body instead of creating a new
PR. An open match still updates, and a no-match still creates, as before.
Adds a reopened value to pull-request-operation and covers the closed
path in the offline suite.

stephen merged commit 5e6c70256d into main

2026-06-30 21:02:34 +00:00

stephen referenced this pull request from a commit

2026-06-30 21:02:35 +00:00

Merge pull request 'fix: reopen a closed matching PR instead of 422' (#9) from fix/reopen-closed-pr into main