Cleanup code

Author: owjs3901

crates/vespera_core/src/openapi.rs

@@ -192,18 +192,6 @@ impl OpenApi {
             }
         }
     }
-
-    /// Merge from a JSON string. Returns error if parsing fails.
-    ///
-    /// # Errors
-    ///
-    /// Returns a `serde_json::Error` when `json_str` is not valid JSON or does not
-    /// deserialize into an `OpenApi` document.
-    pub fn merge_from_str(&mut self, json_str: &str) -> Result<(), serde_json::Error> {
-        let other: Self = serde_json::from_str(json_str)?;
-        self.merge(other);
-        Ok(())
-    }
 }
 
 #[cfg(test)]
@@ -460,35 +448,6 @@ mod tests {
         assert_eq!(base.tags.as_ref().unwrap().len(), 1);
     }
 
-    #[test]
-    fn test_merge_from_str() {
-        let mut base = create_base_openapi();
-        base.paths
-            .insert("/users".to_string(), create_path_item("Get users"));
-
-        let other_json = r#"{
-            "openapi": "3.1.0",
-            "info": { "title": "Other API", "version": "2.0.0" },
-            "paths": {
-                "/posts": { "get": { "summary": "Get posts", "responses": {} } }
-            }
-        }"#;
-
-        let result = base.merge_from_str(other_json);
-        assert!(result.is_ok());
-        assert!(base.paths.contains_key("/users"));
-        assert!(base.paths.contains_key("/posts"));
-    }
-
-    #[test]
-    fn test_merge_from_str_invalid_json() {
-        let mut base = create_base_openapi();
-        let invalid_json = "{ invalid json }";
-
-        let result = base.merge_from_str(invalid_json);
-        assert!(result.is_err());
-    }
-
     #[test]
     fn test_merge_empty_other() {
         let mut base = create_base_openapi();

crates/vespera_core/src/route.rs

@@ -236,32 +236,6 @@ impl PathItem {
             HttpMethod::Trace => self.trace = Some(operation),
         }
     }
-
-    /// Get an operation for a specific HTTP method
-    #[must_use]
-    pub const fn get_operation(&self, method: &HttpMethod) -> Option<&Operation> {
-        match method {
-            HttpMethod::Get => self.get.as_ref(),
-            HttpMethod::Post => self.post.as_ref(),
-            HttpMethod::Put => self.put.as_ref(),
-            HttpMethod::Patch => self.patch.as_ref(),
-            HttpMethod::Delete => self.delete.as_ref(),
-            HttpMethod::Head => self.head.as_ref(),
-            HttpMethod::Options => self.options.as_ref(),
-            HttpMethod::Trace => self.trace.as_ref(),
-        }
-    }
-}
-
-/// Route information (for internal use)
-#[derive(Debug, Clone)]
-pub struct RouteInfo {
-    /// HTTP method
-    pub method: HttpMethod,
-    /// Path
-    pub path: String,
-    /// Operation information
-    pub operation: Operation,
 }
 
 #[cfg(test)]
@@ -404,65 +378,6 @@ mod tests {
         assert!(path_item.trace.is_some());
     }
 
-    #[test]
-    fn test_path_item_get_operation() {
-        let mut path_item = PathItem::default();
-
-        let operation = Operation {
-            operation_id: Some("test_operation".to_string()),
-            tags: None,
-            summary: None,
-            description: None,
-            parameters: None,
-            request_body: None,
-            responses: BTreeMap::new(),
-            security: None,
-        };
-
-        // Initially, all operations should be None
-        assert!(path_item.get_operation(&HttpMethod::Get).is_none());
-        assert!(path_item.get_operation(&HttpMethod::Post).is_none());
-
-        // Set GET operation
-        path_item.set_operation(HttpMethod::Get, operation.clone());
-        let retrieved = path_item.get_operation(&HttpMethod::Get);
-        assert!(retrieved.is_some());
-        assert_eq!(
-            retrieved.unwrap().operation_id,
-            Some("test_operation".to_string())
-        );
-
-        // Set POST operation
-        let mut operation_post = operation.clone();
-        operation_post.operation_id = Some("post_operation".to_string());
-        path_item.set_operation(HttpMethod::Post, operation_post);
-        let retrieved = path_item.get_operation(&HttpMethod::Post);
-        assert!(retrieved.is_some());
-        assert_eq!(
-            retrieved.unwrap().operation_id,
-            Some("post_operation".to_string())
-        );
-
-        // Test all methods
-        path_item.set_operation(HttpMethod::Put, operation.clone());
-        assert!(path_item.get_operation(&HttpMethod::Put).is_some());
-
-        path_item.set_operation(HttpMethod::Patch, operation.clone());
-        assert!(path_item.get_operation(&HttpMethod::Patch).is_some());
-
-        path_item.set_operation(HttpMethod::Delete, operation.clone());
-        assert!(path_item.get_operation(&HttpMethod::Delete).is_some());
-
-        path_item.set_operation(HttpMethod::Head, operation.clone());
-        assert!(path_item.get_operation(&HttpMethod::Head).is_some());
-
-        path_item.set_operation(HttpMethod::Options, operation.clone());
-        assert!(path_item.get_operation(&HttpMethod::Options).is_some());
-
-        path_item.set_operation(HttpMethod::Trace, operation);
-        assert!(path_item.get_operation(&HttpMethod::Trace).is_some());
-    }
-
     #[test]
     fn test_path_item_set_operation_overwrites() {
         let mut path_item = PathItem::default();

crates/vespera_core/src/schema.rs

@@ -48,33 +48,6 @@ pub enum SchemaType {
     Null,
 }
 
-/// Number format
-#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
-#[serde(rename_all = "lowercase")]
-pub enum NumberFormat {
-    Float,
-    Double,
-    Int32,
-    Int64,
-}
-
-/// String format
-#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
-#[serde(rename_all = "lowercase")]
-pub enum StringFormat {
-    Date,
-    DateTime,
-    Password,
-    Byte,
-    Binary,
-    Email,
-    Uuid,
-    Uri,
-    Hostname,
-    IpV4,
-    IpV6,
-}
-
 /// Serialize `Option<f64>` as integer when the value has no fractional part.
 ///
 /// Ensures OpenAPI JSON uses `0` instead of `0.0` for integer constraints like