refactor(semver): change ConstraintInterface to a closed enum

Replace the dyn ConstraintInterface trait objects with an AnyConstraint
enum closing over its four implementors (Simple, Multi, MatchAll,
MatchNone), mirroring the earlier Rule enum conversion. Rename
constraint.rs to simple_constraint.rs to match the renamed Constraint
type.

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

1 files changed, 14 insertions, 41 deletions

diff --git a/crates/shirabe-semver/src/constraint/match_all_constraint.rs b/crates/shirabe-semver/src/constraint/match_all_constraint.rs
index b695ce2..8d4e870 100644
--- a/crates/shirabe-semver/src/constraint/match_all_constraint.rs
+++ b/crates/shirabe-semver/src/constraint/match_all_constraint.rs
@@ -1,68 +1,41 @@
 //! ref: composer/vendor/composer/semver/src/Constraint/MatchAllConstraint.php
 
 use crate::constraint::Bound;
-use crate::constraint::ConstraintInterface;
 
-#[derive(Debug)]
+#[derive(Debug, Clone, Default)]
 pub struct MatchAllConstraint {
     pub(crate) pretty_string: Option<String>,
 }
 
 impl MatchAllConstraint {
-    pub fn new() -> Self {
-        Self {
-            pretty_string: None,
-        }
+    pub fn new(pretty_string: Option<String>) -> Self {
+        Self { pretty_string }
     }
-}
-
-impl Default for MatchAllConstraint {
-    fn default() -> Self {
-        Self::new()
-    }
-}
 
-impl ConstraintInterface for MatchAllConstraint {
-    fn matches(&self, _provider: &dyn ConstraintInterface) -> bool {
-        true
-    }
-
-    fn compile(&self, _other_operator: i64) -> String {
+    pub fn compile(&self, _other_operator: i64) -> String {
         "true".to_string()
     }
 
-    fn set_pretty_string(&mut self, pretty_string: Option<String>) {
-        self.pretty_string = pretty_string;
-    }
-
-    fn get_pretty_string(&self) -> String {
+    pub fn get_pretty_string(&self) -> String {
         if let Some(ref s) = self.pretty_string
             && !s.is_empty()
         {
             return s.clone();
         }
-        self.__to_string()
+        self.to_string()
     }
 
-    fn __to_string(&self) -> String {
-        "*".to_string()
-    }
-
-    fn clone_box(&self) -> Box<dyn ConstraintInterface> {
-        Box::new(MatchAllConstraint {
-            pretty_string: self.pretty_string.clone(),
-        })
-    }
-
-    fn as_any(&self) -> &dyn std::any::Any {
-        self
-    }
-
-    fn get_upper_bound(&self) -> Bound {
+    pub fn get_upper_bound(&self) -> Bound {
         Bound::positive_infinity()
     }
 
-    fn get_lower_bound(&self) -> Bound {
+    pub fn get_lower_bound(&self) -> Bound {
         Bound::zero()
     }
 }
+
+impl std::fmt::Display for MatchAllConstraint {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(f, "*")
+    }
+}