refactor(php-shim): drop Box wrapping from PhpMixed List/Array

The List and Array variants of PhpMixed boxed their elements
unnecessarily. Store PhpMixed values directly and update all callers
accordingly.

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

1 files changed, 5 insertions, 11 deletions

diff --git a/crates/shirabe/src/repository/artifact_repository.rs b/crates/shirabe/src/repository/artifact_repository.rs
index 3715c27..b913edb 100644
--- a/crates/shirabe/src/repository/artifact_repository.rs
+++ b/crates/shirabe/src/repository/artifact_repository.rs
@@ -190,23 +190,17 @@ impl ArtifactRepository {
         let shasum = hash_file("sha1", &real_path).unwrap_or_default();
 
         let mut dist = IndexMap::new();
-        dist.insert(
-            "type".to_string(),
-            Box::new(PhpMixed::String(file_type.to_string())),
-        );
-        dist.insert(
-            "url".to_string(),
-            Box::new(PhpMixed::String(url_normalized)),
-        );
-        dist.insert("shasum".to_string(), Box::new(PhpMixed::String(shasum)));
+        dist.insert("type".to_string(), PhpMixed::String(file_type.to_string()));
+        dist.insert("url".to_string(), PhpMixed::String(url_normalized));
+        dist.insert("shasum".to_string(), PhpMixed::String(shasum));
         if let Some(arr) = package.as_array_mut() {
-            arr.insert("dist".to_string(), Box::new(PhpMixed::Array(dist)));
+            arr.insert("dist".to_string(), PhpMixed::Array(dist));
         }
 
         let cfg: IndexMap<String, PhpMixed> = package
             .as_array()
             .cloned()
-            .map(|m| m.into_iter().map(|(k, v)| (k, *v)).collect())
+            .map(|m| m.into_iter().collect())
             .unwrap_or_default();
         match self.loader.load(cfg, None) {
             Ok(package) => Ok(Some(package)),