refactor(package-discovery): extend BaseCommand as supertrait

PHP's PackageDiscoveryTrait is mixed into classes extending BaseCommand
and calls its methods directly. Model that with a supertrait bound so
get_io, try_composer, require_composer, get_platform_requirement_filter
and normalize_requirements come from BaseCommand instead of being
redeclared and delegated per command. Propagate the Result returns at
the call sites and drop the now-redundant per-command impls.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

1 files changed, 0 insertions, 29 deletions

diff --git a/crates/shirabe/src/command/require_command.rs b/crates/shirabe/src/command/require_command.rs
index 3d8331b..a29f60d 100644
--- a/crates/shirabe/src/command/require_command.rs
+++ b/crates/shirabe/src/command/require_command.rs
@@ -107,35 +107,6 @@ impl PackageDiscoveryTrait for RequireCommand {
     ) -> &mut IndexMap<String, std::rc::Rc<std::cell::RefCell<RepositorySet>>> {
         &mut self.repository_sets
     }
-
-    fn get_io(&self) -> std::rc::Rc<std::cell::RefCell<dyn IOInterface>> {
-        todo!()
-    }
-
-    fn try_composer(&self) -> Option<PartialComposerHandle> {
-        todo!()
-    }
-
-    fn require_composer(
-        &self,
-        disable_plugins: Option<bool>,
-        disable_scripts: Option<bool>,
-    ) -> PartialComposerHandle {
-        todo!()
-    }
-
-    fn get_platform_requirement_filter(
-        &self,
-        input: std::rc::Rc<std::cell::RefCell<dyn InputInterface>>,
-    ) -> std::rc::Rc<
-        dyn crate::filter::platform_requirement_filter::PlatformRequirementFilterInterface,
-    > {
-        todo!()
-    }
-
-    fn normalize_requirements(&self, requires: Vec<String>) -> Vec<IndexMap<String, String>> {
-        todo!()
-    }
 }
 
 impl Command for RequireCommand {