fix: use /pulls instead of /pulls/pinned #4

lyn · 2026-04-01T15:57:32Z

lyn commented

2026-04-01 15:57:32 +00:00

No description provided.

lyn added 10 commits 2026-04-01 17:08:55 +00:00

Resolve conflicts: keep pr command (renamed from create-pr) 2d1d77f57a

fix: extract actual token from secret-mgr output 3930adf362

secret-mgr get-secret outputs 'Access token was successfully created: <token>'
so we need to extract just the token using awk '{print $NF}'

Resolve conflicts: keep pr command with show subcommand bd41addc7d

fix(pr): use /pulls instead of /pulls/pinned for list command 77fa2f77bc

fix(pr): proper HTTP status code handling for merge command d764f30589

Handle different HTTP status codes returned by merge API:
- 200: success (check merged field)
- 409: conflict (output error message)
- 404: not found
- 423: repo archived
- 405: empty response / try again later
- etc: generic error handling

fix(pr): for 405 errors, output API message directly a7700a5f15

feat(pr): add show subcommand and refine merge output messages 5725ac79bb

- Add 'pr show' subcommand: GET /repos/{owner}/{repo}/pulls/{index}
- Change 200 success: 'merge success'
- Change 405 error: 'merge failed check the pr status'

fix(pr show): slim down output, remove nested repo/user objects 1feef0c5b6

fix(pr show): remove user and assignee fields f782bce0a3

fix(pr merge): on 405, fetch and print PR merge status b85d40f587

lyn force-pushed fix/pr-list-endpoint from 6d9144830b to b85d40f587

2026-04-01 17:08:55 +00:00

Compare

lyn scheduled this pull request to auto merge when all checks succeed 2026-04-01 17:09:03 +00:00

lyn merged commit 4d98d83fbb into main

2026-04-01 17:09:04 +00:00

lyn deleted branch fix/pr-list-endpoint

2026-04-01 17:09:04 +00:00

lyn referenced this issue from a commit