hive-forge/src/verbs/pr_merge.rs

@@ -127,22 +127,8 @@ pub fn run(client: &Client, args: Args) -> Result<()> {
 fn check_ready(client: &Client, repo: &str, number: u64, pull: &PullRequest) -> Result<()> {
     match pull.mergeable {
         Some(true) => {}
-        // Forgejo's `mergeable` is one boolean covering several distinct
-        // causes (draft, a real conflict, a blocked required check, ...) —
-        // asserting "conflicts" and telling the caller to rebase is only
-        // correct for one of those causes and actively wrong advice for the
-        // others (rebasing a draft does nothing). Only name the
-        // cause when the API actually confirms it (draft); otherwise report
-        // the bare observation and let `pr-status` be consulted for detail.
-        // Report the confirmed cause without prescribing the fix — a draft
-        // can be a deliberate signal (from the author, or the operator)
-        // rather than an oversight, so don't tell the caller to just
-        // undraft it if it isn't theirs to undraft.
-        Some(false) if pull.draft.unwrap_or(false) => bail!(
-            "pr-merge: PR #{number} is a draft, not mergeable. Pass --force, or check with whoever set draft before un-drafting it."
-        ),
         Some(false) => bail!(
-            "pr-merge: PR #{number} is not mergeable. Check `pr-status --pr {number}` for detail, or pass --force."
+            "pr-merge: PR #{number} is not mergeable (conflicts). Rebase it, or pass --force."
         ),
         None => bail!(
             "pr-merge: PR #{number} mergeability is still being computed. Retry shortly, or pass --force."

hive-forge/src/verbs/pr_status.rs

@@ -107,7 +107,6 @@ fn pr_status(client: &Client, repo: &str, pr: u64) -> Result<()> {
     let merged = pull.merged.unwrap_or(false);
     // `mergeable` is `null` while the forge is still computing it.
     let mergeable = pull.mergeable;
-    let draft = pull.draft.unwrap_or(false);
     let sha = pull
         .head
         .as_ref()
@@ -133,7 +132,6 @@ fn pr_status(client: &Client, repo: &str, pr: u64) -> Result<()> {
             "state": state,
             "merged": merged,
             "mergeable": mergeable,
-            "draft": draft,
             "head_sha": sha,
             "ci_state": ci_state,
             "ci_statuses": ci_statuses,
@@ -156,7 +154,6 @@ fn pr_status(client: &Client, repo: &str, pr: u64) -> Result<()> {
             state,
             merged,
             mergeable,
-            draft,
             &sha,
             &ci_state,
             &ci_statuses,
@@ -307,7 +304,6 @@ fn print_pr(
     state: &str,
     merged: bool,
     mergeable: Option<bool>,
-    draft: bool,
     sha: &str,
     ci_state: &str,
     ci_statuses: &[Value],
@@ -319,25 +315,14 @@ fn print_pr(
     let state_line = if merged {
         "merged".to_owned()
     } else {
-        // Forgejo's `mergeable` is a single boolean folding together several
-        // distinct causes (draft, real conflicts, blocked required checks,
-        // ...) — printing a specific cause we didn't actually observe is a
-        // wrong diagnosis in a health view, worse than no diagnosis at all.
-        // Only name a cause we can actually confirm from the API response;
-        // otherwise report the bare observation.
         let m = match mergeable {
             Some(true) => "mergeable: yes",
-            Some(false) => "mergeable: NO",
+            Some(false) => "mergeable: NO (conflicts)",
             None => "mergeable: computing…",
         };
         format!("{state} ({m})")
     };
     println!("  state:        {state_line}");
-    // Surfaced as its own line rather than folded into the mergeable text
-    // above — it's a real, confirmed cause (unlike the ones we can't name),
-    // and it's currently invisible unless you happen to notice a `WIP:`
-    // title prefix.
-    println!("  draft:        {}", if draft { "yes" } else { "no" });
 
     print!("  CI:           ");
     print_ci(sha, ci_state, ci_statuses);