From 3d4a0cadc950d4440251efa11b55b0bfb0afdef7 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Thu, 6 Aug 2026 04:22:46 +0900
Subject: chore: drop @param/@return tags that only restate Rust types

The ported docblocks copied @param and @return straight from the PHP
source. When such a tag carries nothing but a type and an argument name,
the Rust signature already states it, so the line is noise. Tags whose
text adds prose beyond the type are kept.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 crates/shirabe/src/command/require_command.rs | 9 ---------
 1 file changed, 9 deletions(-)

(limited to 'crates/shirabe/src/command/require_command.rs')

diff --git a/crates/shirabe/src/command/require_command.rs b/crates/shirabe/src/command/require_command.rs
index 9d7d7553..d2b8360d 100644
--- a/crates/shirabe/src/command/require_command.rs
+++ b/crates/shirabe/src/command/require_command.rs
@@ -673,8 +673,6 @@ impl BaseCommand for RequireCommand {
 }
 
 impl RequireCommand {
-    /// @param array<string, string> $newRequirements
-    /// @return string[]
     fn get_inconsistent_require_keys(
         &self,
         new_requirements: &IndexMap<String, String>,
@@ -694,7 +692,6 @@ impl RequireCommand {
         inconsistent_requirements
     }
 
-    /// @return array<string, string>
     fn get_packages_by_require_key(&self) -> IndexMap<String, String> {
         let json = self.json.borrow().as_ref().unwrap().clone();
         let composer_definition = json.borrow_mut().read().unwrap_or_default();
@@ -748,9 +745,6 @@ impl RequireCommand {
         .unwrap_or_default()
     }
 
-    /// @param array<string, string> $requirements
-    /// @param 'require'|'require-dev' $requireKey
-    /// @param 'require'|'require-dev' $removeKey
     /// @throws \Exception
     fn do_update(
         &self,
@@ -1053,7 +1047,6 @@ impl RequireCommand {
         Ok(status)
     }
 
-    /// @param list<string> $requirementsToUpdate
     fn update_requirements_after_resolution(
         &self,
         requirements_to_update: &[String],
@@ -1192,7 +1185,6 @@ impl RequireCommand {
         Ok(0)
     }
 
-    /// @param array<string, string> $new
     fn update_file(
         &self,
         json: &std::rc::Rc<std::cell::RefCell<JsonFile>>,
@@ -1235,7 +1227,6 @@ impl RequireCommand {
         let _ = json.borrow().write(PhpMixed::Array(composer_definition));
     }
 
-    /// @param array<string, string> $new
     fn update_file_cleanly(
         &self,
         json: &std::rc::Rc<std::cell::RefCell<JsonFile>>,
-- 
cgit v1.3.1