From f5ad53c5b32316e5ae53d2799dc1f8938cae6c4c Mon Sep 17 00:00:00 2001 From: John Wiegley Date: Mon, 11 May 2026 20:26:37 +0000 Subject: [PATCH] chore(authorship): remove dead VirtualAttributions::add_pathspec MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit add_pathspec was a thin wrapper around add_pathspecs_concurrent with zero callers — all call sites use the concurrent variant directly. Also removes the now-unnecessary #[allow(dead_code)] suppression from get_attributions, which has active callers in integration tests. Co-Authored-By: Claude Sonnet 4.6 --- src/authorship/virtual_attribution.rs | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/authorship/virtual_attribution.rs b/src/authorship/virtual_attribution.rs index aa1527e9e9..cc894fc69b 100644 --- a/src/authorship/virtual_attribution.rs +++ b/src/authorship/virtual_attribution.rs @@ -243,12 +243,6 @@ impl VirtualAttributions { ))) } - /// Add a single pathspec to the virtual attributions - #[allow(dead_code)] - pub async fn add_pathspec(&mut self, pathspec: &str) -> Result<(), GitAiError> { - self.add_pathspecs_concurrent(&[pathspec.to_string()]).await - } - /// Add multiple pathspecs concurrently async fn add_pathspecs_concurrent(&mut self, pathspecs: &[String]) -> Result<(), GitAiError> { const MAX_CONCURRENT: usize = 30; @@ -306,7 +300,6 @@ impl VirtualAttributions { } /// Get both character and line attributions for a file - #[allow(dead_code)] pub fn get_attributions( &self, file_path: &str,