From 81b9fc9d92bb74aa8428ae4db39bd84e8c16095c Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 20 Jun 2026 18:34:54 +0900 Subject: 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) --- crates/shirabe/src/repository/artifact_repository.rs | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) (limited to 'crates/shirabe/src/repository/artifact_repository.rs') 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 = 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)), -- cgit v1.3.1