refact: drop unused traits cleanup (#16)

* fix: update SSE URL to use localhost and adjust server readiness poll timeout

* refactor: implement exponential backoff for server readiness polling

* refactor: remove DynRunnerProcessManager and clean up unused traits and simplify process manager interfaces

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* refactor: clean up unused traits in process management

* [autofix.ci] apply automated fixes

---------

Co-authored-by: sapan <sapanshah2468@gmail.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: 3 people

crates/gpmcp-layer-core/src/process.rs

@@ -67,32 +67,6 @@ pub enum ProcessError {
     Other(String),
 }
 
-/// Core trait for process lifecycle management with associated types for better performance
-#[async_trait]
-pub trait ProcessLifecycle: Send + Sync {
-    /// The type of process handle this lifecycle manager produces
-    type Handle: ProcessHandle;
-
-    /// Spawn a new process with the given command and arguments
-    async fn spawn_process(
-        &self,
-        command: &str,
-        args: &[String],
-        working_dir: Option<&str>,
-        env: &HashMap<String, String>,
-    ) -> Result<Self::Handle>;
-}
-
-/// Trait for comprehensive process termination including process trees
-#[async_trait]
-pub trait ProcessTermination: Send + Sync {
-    /// Find all child processes of a given process
-    async fn find_child_processes(&self, pid: ProcessId) -> Result<Vec<ProcessId>>;
-
-    /// Terminate an entire process tree (parent and all descendants)
-    async fn terminate_process_tree(&self, root_pid: ProcessId) -> TerminationResult;
-}
-
 /// Trait representing a handle to a running process
 #[async_trait]
 pub trait ProcessHandle: Send + Sync {
@@ -108,14 +82,30 @@ pub trait ProcessHandle: Send + Sync {
 
 /// High-level process manager trait that combines lifecycle and termination
 #[async_trait]
-pub trait ProcessManager: ProcessLifecycle + ProcessTermination {
+pub trait ProcessManager {
+    /// The type of process handle this lifecycle manager produces
+    type Handle: ProcessHandle;
     /// Create a new process manager instance
     fn new() -> Self
     where
         Self: Sized;
 
     /// Cleanup any resources held by the process manager
     async fn cleanup(&self) -> Result<()>;
+    /// Find all child processes of a given process
+    async fn find_child_processes(&self, pid: ProcessId) -> Result<Vec<ProcessId>>;
+
+    /// Terminate an entire process tree (parent and all descendants)
+    async fn terminate_process_tree(&self, root_pid: ProcessId) -> TerminationResult;
+
+    /// Spawn a new process with the given command and arguments
+    async fn spawn_process(
+        &self,
+        command: &str,
+        args: &[String],
+        working_dir: Option<&str>,
+        env: &HashMap<String, String>,
+    ) -> Result<Self::Handle>;
 }
 
 /// Factory trait for creating platform-specific process managers

crates/gpmcp-layer-unix/src/runner_process_manager.rs

@@ -3,10 +3,7 @@ use crate::unix_process_manager::UnixProcessManager;
 use anyhow::{Context, Result};
 use async_trait::async_trait;
 use gpmcp_layer_core::config::RunnerConfig;
-use gpmcp_layer_core::process::{
-    ProcessHandle, ProcessId, ProcessLifecycle, ProcessManager, ProcessTermination,
-    TerminationResult,
-};
+use gpmcp_layer_core::process::{ProcessHandle, ProcessId, ProcessManager, TerminationResult};
 use gpmcp_layer_core::process_manager_trait::{RunnerProcessManager, RunnerProcessManagerFactory};
 use std::collections::HashMap;
 use std::sync::{Arc, Mutex};
@@ -148,10 +145,3 @@ impl RunnerProcessManagerFactory for UnixRunnerProcessManagerFactory {
         UnixRunnerProcessManager::new(config)
     }
 }
-
-#[cfg(test)]
-mod tests {
-
-    // Note: Tests will be added in a future task as requested by the user
-    // This module is here to show the structure for when tests are implemented
-}