|
| 1 | +//! Handles `Commands::Prs` — list submitted PRs from local memory. |
| 2 | +//! |
| 3 | +//! Reads the `submitted_prs` table via `Memory::get_prs`. The `stats` |
| 4 | +//! command shows only the most recent 5; this command exposes the full |
| 5 | +//! list with status filtering and a JSON mode for scripting. |
| 6 | +
|
| 7 | +use anyhow::{Context, Result}; |
| 8 | +use colored::Colorize; |
| 9 | +use std::collections::HashMap; |
| 10 | + |
| 11 | +use crate::cli::{create_memory, load_config}; |
| 12 | + |
| 13 | +/// Run `contribai prs`. |
| 14 | +/// |
| 15 | +/// - `status`: "open" | "merged" | "closed" | "all" (case-insensitive). Default "all". |
| 16 | +/// - `limit`: max rows to fetch from memory (default 20) |
| 17 | +/// - `json`: emit JSON array instead of pretty output |
| 18 | +pub fn run_prs(config_path: Option<&str>, status: &str, limit: usize, json: bool) -> Result<()> { |
| 19 | + let config = load_config(config_path)?; |
| 20 | + let memory = create_memory(&config)?; |
| 21 | + |
| 22 | + let normalized = status.trim().to_lowercase(); |
| 23 | + let status_filter = match normalized.as_str() { |
| 24 | + "all" | "" => None, |
| 25 | + other => Some(other), |
| 26 | + }; |
| 27 | + |
| 28 | + let prs = memory |
| 29 | + .get_prs(status_filter, limit) |
| 30 | + .context("reading submitted_prs from memory")?; |
| 31 | + |
| 32 | + if json { |
| 33 | + let arr = serde_json::Value::Array(prs.iter().map(map_to_json).collect::<Vec<_>>()); |
| 34 | + println!("{}", serde_json::to_string_pretty(&arr)?); |
| 35 | + return Ok(()); |
| 36 | + } |
| 37 | + |
| 38 | + println!("{}", "📦 ContribAI Submitted PRs".cyan().bold()); |
| 39 | + println!("{}", "━".repeat(60).dimmed()); |
| 40 | + println!( |
| 41 | + " {:<20} {}", |
| 42 | + "Status filter:".dimmed(), |
| 43 | + if status_filter.is_some() { |
| 44 | + normalized.cyan() |
| 45 | + } else { |
| 46 | + "all".cyan() |
| 47 | + } |
| 48 | + ); |
| 49 | + println!( |
| 50 | + " {:<20} {}", |
| 51 | + "Rows:".dimmed(), |
| 52 | + prs.len().to_string().cyan() |
| 53 | + ); |
| 54 | + println!(); |
| 55 | + |
| 56 | + if prs.is_empty() { |
| 57 | + println!(" {}", "No PRs match the current filter.".dimmed()); |
| 58 | + return Ok(()); |
| 59 | + } |
| 60 | + |
| 61 | + for pr in &prs { |
| 62 | + let status_str = pr.get("status").map(|s| s.as_str()).unwrap_or("unknown"); |
| 63 | + let pr_number = pr.get("pr_number").cloned().unwrap_or_default(); |
| 64 | + let repo = pr.get("repo").cloned().unwrap_or_default(); |
| 65 | + let title = pr.get("title").cloned().unwrap_or_default(); |
| 66 | + let url = pr.get("pr_url").cloned().unwrap_or_default(); |
| 67 | + let created = pr.get("created_at").cloned().unwrap_or_default(); |
| 68 | + |
| 69 | + let status_colored = match status_str { |
| 70 | + "merged" => status_str.green().bold(), |
| 71 | + "open" => status_str.cyan().bold(), |
| 72 | + "closed" => status_str.red().bold(), |
| 73 | + "failed" => status_str.yellow().bold(), |
| 74 | + _ => status_str.dimmed().bold(), |
| 75 | + }; |
| 76 | + |
| 77 | + println!( |
| 78 | + " {} #{} {} [{}]", |
| 79 | + short_date(&created).dimmed(), |
| 80 | + pr_number.cyan(), |
| 81 | + repo.dimmed(), |
| 82 | + status_colored |
| 83 | + ); |
| 84 | + println!(" {}", title); |
| 85 | + if !url.is_empty() { |
| 86 | + println!(" {}", url.blue().underline()); |
| 87 | + } |
| 88 | + println!(); |
| 89 | + } |
| 90 | + |
| 91 | + Ok(()) |
| 92 | +} |
| 93 | + |
| 94 | +/// Convert a memory row (HashMap<String,String>) into a JSON object. |
| 95 | +/// PR number is parsed to a number when possible so JSON consumers can |
| 96 | +/// sort/compare numerically. |
| 97 | +fn map_to_json(row: &HashMap<String, String>) -> serde_json::Value { |
| 98 | + let mut obj = serde_json::Map::with_capacity(row.len()); |
| 99 | + for (k, v) in row { |
| 100 | + if k == "pr_number" { |
| 101 | + if let Ok(n) = v.parse::<i64>() { |
| 102 | + obj.insert(k.clone(), serde_json::Value::from(n)); |
| 103 | + continue; |
| 104 | + } |
| 105 | + } |
| 106 | + obj.insert(k.clone(), serde_json::Value::String(v.clone())); |
| 107 | + } |
| 108 | + serde_json::Value::Object(obj) |
| 109 | +} |
| 110 | + |
| 111 | +/// Trim a YYYY-MM-DDTHH:MM:SS timestamp down to YYYY-MM-DD HH:MM for terminal display. |
| 112 | +/// Returns the input unchanged if it's shorter than expected. |
| 113 | +fn short_date(s: &str) -> String { |
| 114 | + // Tolerate either "T" or " " between date and time. |
| 115 | + let s = s.replace('T', " "); |
| 116 | + if s.len() >= 16 { |
| 117 | + s[..16].to_string() |
| 118 | + } else { |
| 119 | + s |
| 120 | + } |
| 121 | +} |
| 122 | + |
| 123 | +#[cfg(test)] |
| 124 | +mod tests { |
| 125 | + use super::*; |
| 126 | + |
| 127 | + #[test] |
| 128 | + fn map_to_json_parses_pr_number_as_int() { |
| 129 | + let mut row = HashMap::new(); |
| 130 | + row.insert("pr_number".to_string(), "42".to_string()); |
| 131 | + row.insert("title".to_string(), "fix bug".to_string()); |
| 132 | + let v = map_to_json(&row); |
| 133 | + assert_eq!(v["pr_number"], serde_json::json!(42)); |
| 134 | + assert_eq!(v["title"], serde_json::json!("fix bug")); |
| 135 | + } |
| 136 | + |
| 137 | + #[test] |
| 138 | + fn map_to_json_keeps_pr_number_as_string_when_unparseable() { |
| 139 | + let mut row = HashMap::new(); |
| 140 | + row.insert("pr_number".to_string(), "n/a".to_string()); |
| 141 | + let v = map_to_json(&row); |
| 142 | + assert_eq!(v["pr_number"], serde_json::json!("n/a")); |
| 143 | + } |
| 144 | + |
| 145 | + #[test] |
| 146 | + fn short_date_trims_iso_timestamp() { |
| 147 | + assert_eq!(short_date("2026-04-27T10:23:58Z"), "2026-04-27 10:23"); |
| 148 | + } |
| 149 | + |
| 150 | + #[test] |
| 151 | + fn short_date_passthrough_when_shorter_than_expected() { |
| 152 | + assert_eq!(short_date("2026-04-27"), "2026-04-27"); |
| 153 | + } |
| 154 | +} |
0 commit comments