Make PR, issue, release, and CI mutation failures actionable #636

New issue

Closed

opened 2026-05-06 19:55:42 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-05-06 19:55:42 +00:00

Owner

Parent: #634

Problem

Mutation actions route through lua/forge/action/ops.lua and often fall back to generic messages such as merge failed, close failed, delete failed, cancel failed, or rerun failed. In the merge case, this can hide the actual CLI reason while the PR remains open.

Scope

Improve run_forge_cmd / run_forge_task failure handling.
Include useful backend CLI output for PR merge, PR close/reopen/approve/draft, issue close/reopen, release delete, and CI cancel/rerun.
Thread failure details to callbacks where picker refresh/revert behavior depends on the failure.
Add tests for a failed merge that includes stderr and a failed command with blank streams.

Non-goals

Do not change optimistic success behavior.
Do not implement new backend actions.

Parent: #634 ## Problem Mutation actions route through `lua/forge/action/ops.lua` and often fall back to generic messages such as `merge failed`, `close failed`, `delete failed`, `cancel failed`, or `rerun failed`. In the merge case, this can hide the actual CLI reason while the PR remains open. ## Scope - Improve `run_forge_cmd` / `run_forge_task` failure handling. - Include useful backend CLI output for PR merge, PR close/reopen/approve/draft, issue close/reopen, release delete, and CI cancel/rerun. - Thread failure details to callbacks where picker refresh/revert behavior depends on the failure. - Add tests for a failed merge that includes stderr and a failed command with blank streams. ## Non-goals - Do not change optimistic success behavior. - Do not implement new backend actions.

barrettruth added the

bug

v0.2.0

labels

2026-05-06 19:55:42 +00:00

barrettruth referenced this issue

2026-05-06 19:56:06 +00:00

Tracker: error handling and edge cases #634

barrettruth referenced this issue from a pull request that will close it,

2026-05-06 21:10:14 +00:00

Make mutation failures actionable #648

barrettruth closed this issue