API docs advertise POST /api/v2/repos/:owner/:repo/pulls/:number/merge but no route implements it#194
Closedccantynz opened this issue 25d ago
Closed by pull request #3525.
Linked pull requests1
feat(api): implement the documented v2 PR merge endpoint (closes #194)#3525mergedc comment · e edit title · x close/reopen · ? shortcuts
src/routes/docs.tsx:1287documentsPOST /api/v2/repos/:owner/:repo/pulls/:number/merge, but no such route exists insrc/routes/api-v2.ts(or anywhere else). The only real merge path is the web routePOST /:owner/:repo/pulls/:number/merge(src/routes/pulls.tsx:6143), which accepts a Bearer PAT via softAuth but 302-redirects to the PR page on BOTH success and failure, so API callers cannot tell whether the merge happened without a follow-up GET.Another instance of the two-sources-of-truth pattern. Fix either way:
src/lib/pr-merge.tsalready factors the logic), orOption 1 is strongly preferred: with the GitHub-outage urgency, external tooling and AI agents need a real JSON merge API, and the MCP
gluecron_merge_prtool is unavailable whenever the connector token expires (as happened today).?? Filed by Claude Code while shipping PR #3524