From 46d4a1e5c8ed7b4871e22a9ae45024565fcb32b8 Mon Sep 17 00:00:00 2001 From: Scott Werner Date: Sun, 16 Aug 2026 11:04:05 -0400 Subject: [PATCH] Inline the blob_hash_from_response alias It was a one-line passthrough to parse_blob_ref with a single caller, leaving two names for the same operation; every other consumer calls parse_blob_ref directly. Co-Authored-By: Claude Fable 5 --- lib/apps/fabro-cli/src/commands/run/output.rs | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/apps/fabro-cli/src/commands/run/output.rs b/lib/apps/fabro-cli/src/commands/run/output.rs index 14d0e7c6f..9de4a3158 100644 --- a/lib/apps/fabro-cli/src/commands/run/output.rs +++ b/lib/apps/fabro-cli/src/commands/run/output.rs @@ -5,7 +5,7 @@ use anyhow::{Context as _, Result}; use cli_table::format::{Border, Justify, Separator}; use cli_table::{Cell, CellStruct, Style, Table}; use fabro_api::types; -use fabro_types::{BlobHash, PullRequestLink, RunId, StageId, parse_blob_ref}; +use fabro_types::{PullRequestLink, RunId, StageId, parse_blob_ref}; use fabro_util::check_report::{CheckDetail, CheckReport, CheckResult, CheckSection, CheckStatus}; use fabro_util::error::render_with_causes; use fabro_util::printer::Printer; @@ -325,7 +325,7 @@ async fn resolve_response_string( run_id: &RunId, response: &str, ) -> Result> { - let Some(blob_hash) = blob_hash_from_response(response) else { + let Some(blob_hash) = parse_blob_ref(response) else { return Ok(Some(response.to_string())); }; @@ -341,10 +341,6 @@ async fn resolve_response_string( })) } -fn blob_hash_from_response(response: &str) -> Option { - parse_blob_ref(response) -} - async fn list_artifact_display_entries_with_client( client: &server_client::Client, run_id: &RunId,