fix(package): dispatch get_links_for_type on Link::TYPE_* values

PHP resolves a link kind through `$package->{'get'.ucfirst($linkType)}()`,
so the argument is a `Link::$TYPES` value ("requires", "devRequires", ...).
The helper matched the composer.json key names ("require", "require-dev",
...) instead, so every caller passing a `Link::TYPE_*` constant got an empty
map back: `show <package>` printed none of its requires/provides/conflicts/
replaces sections, and `--format=json` carried none of those keys.

The two callers that were passing composer.json keys now pass the matching
`method`, as PHP does.

2 files changed, 9 insertions, 8 deletions

diff --git a/crates/shirabe/src/package/dumper/array_dumper.rs b/crates/shirabe/src/package/dumper/array_dumper.rs
index 303a5871..336c96b0 100644
--- a/crates/shirabe/src/package/dumper/array_dumper.rs
+++ b/crates/shirabe/src/package/dumper/array_dumper.rs
@@ -92,8 +92,8 @@ impl ArrayDumper {
             data.insert("dist".to_string(), PhpMixed::Array(dist));
         }
 
-        for type_name in SUPPORTED_LINK_TYPES.keys() {
-            let links = package.get_links_for_type(type_name);
+        for (type_name, opts) in SUPPORTED_LINK_TYPES.iter() {
+            let links = package.get_links_for_type(opts.method);
             if links.is_empty() {
                 continue;
             }
diff --git a/crates/shirabe/src/package/package_interface.rs b/crates/shirabe/src/package/package_interface.rs
index d138fd83..c3cb1c56 100644
--- a/crates/shirabe/src/package/package_interface.rs
+++ b/crates/shirabe/src/package/package_interface.rs
@@ -191,17 +191,18 @@ pub trait PackageInterface: std::fmt::Display + std::fmt::Debug {
     /// @return array An array of package suggestions with descriptions
     fn get_suggests(&self) -> IndexMap<String, String>;
 
-    /// PHP helper that switches on the link kind (require/require-dev/conflict/etc.).
+    /// Stands in for PHP's `$package->{'get'.ucfirst($linkType)}()`, so `link_type` is one of the
+    /// `Link::TYPE_*` values.
     fn get_links_for_type(
         &self,
         link_type: &str,
     ) -> std::rc::Rc<IndexMap<String, crate::package::Link>> {
         match link_type {
-            "require" => self.get_requires(),
-            "require-dev" => self.get_dev_requires(),
-            "conflict" => self.get_conflicts(),
-            "provide" => self.get_provides(),
-            "replace" => self.get_replaces(),
+            crate::package::Link::TYPE_REQUIRE => self.get_requires(),
+            crate::package::Link::TYPE_DEV_REQUIRE => self.get_dev_requires(),
+            crate::package::Link::TYPE_CONFLICT => self.get_conflicts(),
+            crate::package::Link::TYPE_PROVIDE => self.get_provides(),
+            crate::package::Link::TYPE_REPLACE => self.get_replaces(),
             _ => std::rc::Rc::new(IndexMap::new()),
         }
     }