chore(spdx-licenses): drop members Composer never calls

getLicenses(), getExceptionByIdentifier(), getIdentifierByName(),
isOsiApprovedByIdentifier(), isDeprecatedByIdentifier() and the
LICENSES_FILE / EXCEPTIONS_FILE constants had no callers.

Unused members are normally kept so the port stays a faithful mirror of
the PHP class, but SpdxLicenses is pure logic over the bundled SPDX data
that holds no Composer state, and plugins reach the real PHP
implementation rather than this port, so nothing can observe the
difference. The struct now records what is left out and why.

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

1 files changed, 8 insertions, 67 deletions

diff --git a/crates/shirabe-spdx-licenses/src/spdx_licenses.rs b/crates/shirabe-spdx-licenses/src/spdx_licenses.rs
index d16b2d2f..36d87a1e 100644
--- a/crates/shirabe-spdx-licenses/src/spdx_licenses.rs
+++ b/crates/shirabe-spdx-licenses/src/spdx_licenses.rs
@@ -3,9 +3,6 @@
 use indexmap::IndexMap;
 use shirabe_php_shim::PhpMixed;
 
-pub const LICENSES_FILE: &str = "spdx-licenses.json";
-pub const EXCEPTIONS_FILE: &str = "spdx-exceptions.json";
-
 // PHP reads the resource files from `dirname(__DIR__) . '/res'` at runtime via
 // file_get_contents. Composer's res directory ships with the vendored package; embed it at compile
 // time so the data is available without filesystem access.
@@ -14,6 +11,14 @@ const LICENSES_JSON: &str =
 const EXCEPTIONS_JSON: &str =
     include_str!("../../../composer/vendor/composer/spdx-licenses/res/spdx-exceptions.json");
 
+// Only the members Composer itself calls are ported: SpdxLicenses::getLicenses(),
+// ::getExceptionByIdentifier(), ::getIdentifierByName(), ::isOsiApprovedByIdentifier(),
+// ::isDeprecatedByIdentifier(), ::getResourcesDir() and the LICENSES_FILE / EXCEPTIONS_FILE
+// constants are left out.
+//
+// Dropping unused members is only acceptable because of what this class is: pure logic over the
+// bundled SPDX data, holding no Composer state, and plugins reach the real PHP implementation
+// rather than this port.
 #[derive(Debug)]
 pub struct SpdxLicenses {
     // [ lowercased license identifier => (identifier, full name, osi certified, deprecated) ]
@@ -68,70 +73,6 @@ impl SpdxLicenses {
         ]))
     }
 
-    /// Returns all licenses information, keyed by the lowercased license identifier.
-    ///
-    /// Each item is [ 0 => identifier, 1 => full name, 2 => osi certified, 3 => deprecated ].
-    pub fn get_licenses(&self) -> PhpMixed {
-        let mut out: IndexMap<String, PhpMixed> = IndexMap::new();
-        for (key, (identifier, name, is_osi_approved, is_deprecated_license_id)) in &self.licenses {
-            out.insert(
-                key.clone(),
-                PhpMixed::List(vec![
-                    PhpMixed::String(identifier.clone()),
-                    PhpMixed::String(name.clone()),
-                    PhpMixed::Bool(*is_osi_approved),
-                    PhpMixed::Bool(*is_deprecated_license_id),
-                ]),
-            );
-        }
-        PhpMixed::Array(out)
-    }
-
-    /// Returns license exception metadata by license exception identifier.
-    ///
-    /// The returned list is in the form of:
-    ///   [ 0 => full name, 1 => link to license text ]
-    pub fn get_exception_by_identifier(&self, identifier: &str) -> Option<PhpMixed> {
-        let key = identifier.to_lowercase();
-
-        let (identifier, name) = self.exceptions.get(&key)?;
-
-        Some(PhpMixed::List(vec![
-            PhpMixed::String(name.clone()),
-            PhpMixed::String(format!(
-                "https://spdx.org/licenses/{}.html#licenseExceptionText",
-                identifier
-            )),
-        ]))
-    }
-
-    /// Returns the short identifier of a license (or license exception) by full name.
-    pub fn get_identifier_by_name(&self, name: &str) -> Option<String> {
-        for (identifier, full_name, _, _) in self.licenses.values() {
-            if full_name == name {
-                return Some(identifier.clone());
-            }
-        }
-
-        for (identifier, full_name) in self.exceptions.values() {
-            if full_name == name {
-                return Some(identifier.clone());
-            }
-        }
-
-        None
-    }
-
-    /// Returns the OSI Approved status for a license by identifier.
-    pub fn is_osi_approved_by_identifier(&self, identifier: &str) -> bool {
-        self.licenses[&identifier.to_lowercase()].2
-    }
-
-    /// Returns the deprecation status for a license by identifier.
-    pub fn is_deprecated_by_identifier(&self, identifier: &str) -> bool {
-        self.licenses[&identifier.to_lowercase()].3
-    }
-
     pub fn validate(&self, license: &str) -> bool {
         // PHP also accepts string[] and joins it with ' OR '. The Rust signature here takes a single
         // &str, matching the string-input path of the PHP code; the array path lives in callers.