fix(gh): pass through gh pr merge instead of canned response (rtk-ai#938)

pszymkowiak · pszymkowiak · commit e1f2845df06a · 2026-04-02T17:25:25.000+02:00
pr_merge() was using run_filtered with a filter_fn that ignored stdout
and always returned "ok merged". This made AI agents believe PRs were
merged when they weren't.

Switch to run_passthrough so the real gh output is shown. gh pr merge
is a destructive action — the user must see exactly what happened.

Signed-off-by: Patrick szymkowiak &lt;patrick.szymkowiak@innovtech.eu&gt;
diff --git a/src/cmds/git/gh_cmd.rs b/src/cmds/git/gh_cmd.rs
@@ -785,31 +785,13 @@ fn pr_create(args: &[String], _verbose: u8) -> Result<i32> {
 }
 
 fn pr_merge(args: &[String], _verbose: u8) -> Result<i32> {
-    let pr_num = args
-        .iter()
-        .find(|a| !a.starts_with('-'))
-        .map(|s| s.as_str())
-        .unwrap_or("")
-        .to_string();
-    let mut cmd = resolved_command("gh");
-    cmd.args(["pr", "merge"]);
-    for arg in args {
-        cmd.arg(arg);
-    }
-    runner::run_filtered(
-        cmd,
-        "gh",
-        "pr merge",
-        move |_stdout| {
-            let detail = if !pr_num.is_empty() {
-                format!("#{}", pr_num)
-            } else {
-                String::new()
-            };
-            ok_confirmation("merged", &detail)
-        },
-        RunOptions::stdout_only().early_exit_on_failure(),
-    )
+    // gh pr merge is a destructive action — pass through the real output
+    // so the user (or AI agent) sees exactly what happened.
+    run_passthrough("gh", "pr", &{
+        let mut a = vec!["merge".to_string()];
+        a.extend_from_slice(args);
+        a
+    })
 }
 
 /// Flags that change `gh pr diff` output from unified diff to a different format.
