feat(cli,memory): type agents fleet --status flag and add tracing span (#4365)

Closes #4357, #4358

- Add FleetStatus wrapper enum with clap::ValueEnum in src/cli.rs to
  keep clap out of zeph-memory; convert to SessionStatus via From impl
- Replace Option<String> with Option<FleetStatus> in AgentsCommand::Fleet;
  remove manual .parse::<SessionStatus>() from handle_fleet
- Add #[tracing::instrument(name = "memory.fleet.update_status", ...)]
  to SqliteStore::update_agent_session_status, matching sibling methods

Author: bug-ops

src/cli.rs

@@ -3,7 +3,8 @@
 
 use std::path::PathBuf;
 
-use clap::{Parser, Subcommand};
+use clap::{Parser, Subcommand, ValueEnum};
+use zeph_memory::store::agent_sessions::SessionStatus;
 
 #[derive(Parser)]
 #[command(
@@ -488,6 +489,28 @@ pub(crate) enum DbCommand {
     Migrate,
 }
 
+/// Typed session status filter for the `agents fleet` sub-command.
+#[derive(Clone, Copy, Debug, ValueEnum)]
+pub(crate) enum FleetStatus {
+    Active,
+    Completed,
+    Failed,
+    Cancelled,
+    Unknown,
+}
+
+impl From<FleetStatus> for SessionStatus {
+    fn from(s: FleetStatus) -> Self {
+        match s {
+            FleetStatus::Active => SessionStatus::Active,
+            FleetStatus::Completed => SessionStatus::Completed,
+            FleetStatus::Failed => SessionStatus::Failed,
+            FleetStatus::Cancelled => SessionStatus::Cancelled,
+            FleetStatus::Unknown => SessionStatus::Unknown,
+        }
+    }
+}
+
 #[derive(Subcommand)]
 pub(crate) enum AgentsCommand {
     /// List all available sub-agent definitions
@@ -526,9 +549,9 @@ pub(crate) enum AgentsCommand {
     },
     /// List agent sessions recorded in the fleet database
     Fleet {
-        /// Filter by session status (active, completed, failed, cancelled, unknown)
+        /// Filter by session status
         #[arg(long, short)]
-        status: Option<String>,
+        status: Option<FleetStatus>,
         /// Maximum number of sessions to show
         #[arg(long, default_value = "20")]
         limit: u32,

src/commands/agents.rs

@@ -12,7 +12,7 @@ use zeph_memory::store::agent_sessions::{AgentSessionRow, SessionStatus};
 use zeph_subagent::error::SubAgentError;
 use zeph_subagent::{SubAgentDef, ToolPolicy, is_valid_agent_name, resolve_agent_paths};
 
-use crate::cli::AgentsCommand;
+use crate::cli::{AgentsCommand, FleetStatus};
 
 pub(crate) async fn handle_agents_command(
     cmd: AgentsCommand,
@@ -29,9 +29,7 @@ pub(crate) async fn handle_agents_command(
         } => handle_create(&name, &description, dir.as_path(), model.as_deref()),
         AgentsCommand::Edit { name } => handle_edit(&name, config_path),
         AgentsCommand::Delete { name, yes } => handle_delete(&name, yes, config_path),
-        AgentsCommand::Fleet { status, limit } => {
-            handle_fleet(status.as_deref(), limit, config_path).await
-        }
+        AgentsCommand::Fleet { status, limit } => handle_fleet(status, limit, config_path).await,
     }
 }
 
@@ -230,7 +228,7 @@ fn handle_delete(name: &str, yes: bool, config_path: Option<&Path>) -> anyhow::R
 }
 
 async fn handle_fleet(
-    status_filter: Option<&str>,
+    status_filter: Option<FleetStatus>,
     limit: u32,
     config_path: Option<&Path>,
 ) -> anyhow::Result<()> {
@@ -241,9 +239,7 @@ async fn handle_fleet(
         .await
         .context("failed to open database")?;
 
-    let sf: Option<SessionStatus> = status_filter
-        .map(|s| s.parse().map_err(|e: String| anyhow::anyhow!("{e}")))
-        .transpose()?;
+    let sf: Option<SessionStatus> = status_filter.map(SessionStatus::from);
 
     let sessions = store
         .list_agent_sessions(limit, sf)