d59bfb899f
mara: 'those comments seem very redundant'. true — the 16 pure-GET verbs all got the same 'transport error + stdout I/O' boilerplate, which just restates the Result<()> contract that's trivially derivable from the type. dropped # Errors from: assign, branches, close, comment_show, comments, diff, issue, labels, lint, list, milestone, pr, pr_reviews, subscription, timeline, tree_sha, view (17 files). kept on the 7 verbs that have a non-Forgejo failure surface worth documenting: - comment, comment_edit, issue_create, issue_edit — body input I/O via --body-file / stdin - pr_create — body input + --push shellout to git - attach::run_issue, attach::run_comment — explicit bail! on missing file net: 23 verbs touched in the original PR → 17 trimmed back to no-doc, 6 kept (with the 7th call being attach::run_comment in the same file). 38 tests still pass.
115 lines
3.3 KiB
Rust
115 lines
3.3 KiB
Rust
//! `labels <number> [list|add|remove] [labels...]` — manage labels on
|
|
//! an issue or PR. Default action: list.
|
|
|
|
use anyhow::{Result, bail};
|
|
use clap::{Args as ClapArgs, Subcommand};
|
|
use serde_json::{Value, json};
|
|
|
|
use crate::client::Client;
|
|
use crate::verbs::print_json;
|
|
|
|
#[derive(ClapArgs)]
|
|
pub struct Args {
|
|
/// Issue or PR number.
|
|
number: u64,
|
|
#[command(subcommand)]
|
|
action: Option<Action>,
|
|
}
|
|
|
|
#[derive(Subcommand)]
|
|
enum Action {
|
|
/// List labels (default when no action is given).
|
|
List,
|
|
/// Add labels by name.
|
|
Add {
|
|
/// Label names to add.
|
|
labels: Vec<String>,
|
|
},
|
|
/// Remove labels by name.
|
|
Remove {
|
|
/// Label names to remove.
|
|
labels: Vec<String>,
|
|
},
|
|
}
|
|
|
|
pub fn run(client: &Client, args: Args) -> Result<()> {
|
|
let repo = client.repo();
|
|
match args.action.unwrap_or(Action::List) {
|
|
Action::List => {
|
|
let labels = client.get_json(&format!("/repos/{repo}/issues/{}/labels", args.number))?;
|
|
print_label_names(&labels);
|
|
}
|
|
Action::Add { labels } => {
|
|
if labels.is_empty() {
|
|
bail!("hive-forge labels add: pass at least one label name");
|
|
}
|
|
let all = client.get_json(&format!("/repos/{repo}/labels?limit=100"))?;
|
|
let ids = resolve_ids(&all, &labels);
|
|
let resp = client.post_json(
|
|
&format!("/repos/{repo}/issues/{}/labels", args.number),
|
|
&json!({ "labels": ids }),
|
|
)?;
|
|
print_label_names(&resp);
|
|
}
|
|
Action::Remove { labels } => {
|
|
if labels.is_empty() {
|
|
bail!("hive-forge labels remove: pass at least one label name");
|
|
}
|
|
let all = client.get_json(&format!("/repos/{repo}/labels?limit=100"))?;
|
|
for name in &labels {
|
|
if let Some(id) = lookup_id(&all, name) {
|
|
let _ = client.delete(
|
|
&format!("/repos/{repo}/issues/{}/labels/{id}", args.number),
|
|
None,
|
|
);
|
|
}
|
|
}
|
|
let labels = client.get_json(&format!("/repos/{repo}/issues/{}/labels", args.number))?;
|
|
print_label_names(&labels);
|
|
}
|
|
}
|
|
Ok(())
|
|
}
|
|
|
|
fn resolve_ids(all: &Value, names: &[String]) -> Vec<u64> {
|
|
let Some(arr) = all.as_array() else {
|
|
return Vec::new();
|
|
};
|
|
names
|
|
.iter()
|
|
.filter_map(|n| {
|
|
arr.iter().find_map(|l| {
|
|
let lname = l.get("name").and_then(Value::as_str)?;
|
|
if lname == n {
|
|
l.get("id").and_then(Value::as_u64)
|
|
} else {
|
|
None
|
|
}
|
|
})
|
|
})
|
|
.collect()
|
|
}
|
|
|
|
fn lookup_id(all: &Value, name: &str) -> Option<u64> {
|
|
let arr = all.as_array()?;
|
|
arr.iter().find_map(|l| {
|
|
let lname = l.get("name").and_then(Value::as_str)?;
|
|
if lname == name {
|
|
l.get("id").and_then(Value::as_u64)
|
|
} else {
|
|
None
|
|
}
|
|
})
|
|
}
|
|
|
|
fn print_label_names(v: &Value) {
|
|
let names: Vec<&str> = v
|
|
.as_array()
|
|
.map(|a| {
|
|
a.iter()
|
|
.filter_map(|l| l.get("name").and_then(Value::as_str))
|
|
.collect()
|
|
})
|
|
.unwrap_or_default();
|
|
let _ = print_json(&json!(names));
|
|
}
|