From 05002acda4ade4f7e0a5d52bd3b5421ed9a378d0 Mon Sep 17 00:00:00 2001 From: damocles Date: Mon, 8 Jun 2026 20:55:14 +0200 Subject: [PATCH] chore(#1474): document the too_many_arguments keeps in hive-forge + hive-ag3nt --- hive-ag3nt/src/bin/hive.rs | 7 ++++++- hive-ag3nt/src/forge_notify.rs | 7 ++++++- hive-forge/src/verbs/pr_status.rs | 7 ++++++- 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/hive-ag3nt/src/bin/hive.rs b/hive-ag3nt/src/bin/hive.rs index a565b6ce..45e76995 100644 --- a/hive-ag3nt/src/bin/hive.rs +++ b/hive-ag3nt/src/bin/hive.rs @@ -421,7 +421,12 @@ async fn serve_main(socket: &Path, poll_ms: u64) -> Result<()> { /// The long-running message loop. Long-polls the broker via /// `S::recv_next`, drives a turn per message, parks on auth-failed, /// otherwise retries. -#[allow(clippy::too_many_arguments)] +#[allow( + clippy::too_many_arguments, + reason = "the harness's long-lived deps threaded into one serve loop, \ + wired once from main; bundling into a struct would just move the \ + same fields one level out (cf. Coordinator::open)" +)] async fn serve_loop( socket: &Path, interval: Duration, diff --git a/hive-ag3nt/src/forge_notify.rs b/hive-ag3nt/src/forge_notify.rs index e16598bc..607b79a3 100644 --- a/hive-ag3nt/src/forge_notify.rs +++ b/hive-ag3nt/src/forge_notify.rs @@ -625,7 +625,12 @@ fn format_state_change_notification( Some(out) } -#[allow(clippy::too_many_arguments)] +#[allow( + clippy::too_many_arguments, + reason = "the notification poll's config + mutable subscription state, \ + wired once from the poll loop; a struct would just move the \ + same fields one level out" +)] #[allow( clippy::too_many_lines, reason = "single-pass notification poll loop — split would obscure the \ diff --git a/hive-forge/src/verbs/pr_status.rs b/hive-forge/src/verbs/pr_status.rs index 89b51d16..4ed9da40 100644 --- a/hive-forge/src/verbs/pr_status.rs +++ b/hive-forge/src/verbs/pr_status.rs @@ -254,7 +254,12 @@ fn print_ci(sha: &str, state: &str, statuses: &[Value]) { } /// Render the full PR health block. -#[allow(clippy::too_many_arguments)] +#[allow( + clippy::too_many_arguments, + reason = "pure display helper — the args are the already-fetched PR fields \ + it prints; a struct would just mirror the API response for a \ + single call site" +)] fn print_pr( pr: u64, title: &str,