refactor(package): introduce Rc<RefCell<_>> handles for packages

PHP packages have reference semantics, so introduce shared-ownership
handles over an AnyPackage enum (PackageInterfaceHandle and friends)
and replace Box<dyn PackageInterface> throughout.

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

1 files changed, 0 insertions, 8 deletions

diff --git a/crates/shirabe/src/package/package_interface.rs b/crates/shirabe/src/package/package_interface.rs
index 1c9cf2e..97aac26 100644
--- a/crates/shirabe/src/package/package_interface.rs
+++ b/crates/shirabe/src/package/package_interface.rs
@@ -290,14 +290,6 @@ pub trait PackageInterface: std::fmt::Display + std::fmt::Debug {
     /// Set dist and source references and update dist URL for ones that contain a reference
     fn set_source_dist_references(&mut self, reference: &str);
 
-    // clone_box was moved to BasePackage with a Box<dyn BasePackage> return type;
-    // exposing it here too caused trait-method ambiguity at every BasePackage call site.
-    // Callers holding `&dyn PackageInterface` (rather than `&dyn BasePackage`) can use
-    // `clone_package_box` instead.
-    fn clone_package_box(&self) -> Box<dyn PackageInterface> {
-        todo!()
-    }
-
     fn as_alias_package(&self) -> Option<&crate::package::AliasPackage> {
         None
     }