[model-gateway] wasm: some minor improvements (sgl-project#12471)

Signed-off-by: Tony Lu <tonylu@linux.alibaba.com>

Author: tonyluj

sgl-router/src/wasm/module_manager.rs

@@ -85,7 +85,7 @@ impl WasmModuleManager {
         Ok(hasher.finalize().into())
     }
 
-    fn validate_module_descriptor(&self, descriptor: WasmModuleDescriptor) -> Result<()> {
+    fn validate_module_descriptor(&self, descriptor: &WasmModuleDescriptor) -> Result<()> {
         if descriptor.name.is_empty() {
             return Err(WasmModuleError::InvalidDescriptor(
                 "Module name cannot be empty".to_string(),
@@ -109,7 +109,7 @@ impl WasmModuleManager {
 
     pub fn add_module(&self, descriptor: WasmModuleDescriptor) -> Result<Uuid> {
         // validate the module descriptor
-        self.validate_module_descriptor(descriptor.clone())?;
+        self.validate_module_descriptor(&descriptor)?;
 
         // calculate the sha256 hash of the module file
         let sha256_hash = self.calculate_sha256_hash(&descriptor.file_path)?;

sgl-router/src/wasm/types.rs

@@ -86,36 +86,6 @@ impl WasmComponentOutput {
     }
 }
 
-/// Helper trait for converting from WIT types to component input
-pub trait ToComponentInput {
-    fn to_component_input(self) -> WasmComponentInput;
-}
-
-/// Helper trait for converting from component output to WIT types
-pub trait FromComponentOutput {
-    fn from_component_output(output: &WasmComponentOutput) -> Option<&Self>;
-}
-
-impl ToComponentInput for middleware_types::Request {
-    fn to_component_input(self) -> WasmComponentInput {
-        WasmComponentInput::MiddlewareRequest(self)
-    }
-}
-
-impl ToComponentInput for middleware_types::Response {
-    fn to_component_input(self) -> WasmComponentInput {
-        WasmComponentInput::MiddlewareResponse(self)
-    }
-}
-
-impl FromComponentOutput for middleware_types::Action {
-    fn from_component_output(output: &WasmComponentOutput) -> Option<&Self> {
-        match output {
-            WasmComponentOutput::MiddlewareAction(action) => Some(action),
-        }
-    }
-}
-
 pub struct WasiState {
     pub ctx: WasiCtx,
     pub table: ResourceTable,