Remove implicit clone from ExtraName and document requirement in PackageName (#262)

per discussion in #137


https://discord.com/channels/1039017663004942429/1148719284013510676/1169000261746962473

crates/puffin-package/src/extra_name.rs

@@ -7,12 +7,6 @@ use regex::Regex;
 #[derive(Debug, Clone, PartialEq, Eq, Hash, PartialOrd, Ord)]
 pub struct ExtraName(String);
 
-impl From<&ExtraName> for ExtraName {
-    fn from(extra_name: &ExtraName) -> Self {
-        extra_name.clone()
-    }
-}
-
 impl Display for ExtraName {
     fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
         self.0.fmt(f)

crates/puffin-package/src/package_name.rs

@@ -10,6 +10,7 @@ use crate::dist_info_name::DistInfoName;
 pub struct PackageName(String);
 
 impl From<&PackageName> for PackageName {
+    /// Required for `WaitMap::wait`
     fn from(package_name: &PackageName) -> Self {
         package_name.clone()
     }