feat(maestro): self-host JetBrains Mono for web terminal

Sets it as the primary --font-mono/xterm font, system stack kept as
fallback.

CLAUDE.md

@@ -63,8 +63,8 @@ nix/                     # Home Manager module (`services.maestro`)
 See `docs/INSTALL.md` for end-user install instructions.
 
 `docs/WORKTREES.md` is the contract for session worktrees — placement,
-collisions, what blocks a kill, what `--force` retains, and what reclaim
-touches. Read it before changing anything that creates or removes one.
+collisions, what blocks a kill, and what `--force` leaves behind. Read it
+before changing anything that creates or removes one.
 
 `docs/SIDEBAR.md` is the contract for session cards in the web and TUI
 sidebars — line layout, activity colours, duration format, state precedence.

crates/maestro-cli/src/cli.rs

@@ -41,8 +41,6 @@ pub enum Command {
     Attach(AttachArgs),
     /// Kill a session by ID.
     Kill(KillArgs),
-    /// List retained worktrees, or reclaim one by ID (#86).
-    Worktrees(WorktreesArgs),
     /// Install or update the agent activity hook (#106).
     Hooks(HooksArgs),
 }
@@ -62,34 +60,6 @@ pub enum HooksCommand {
     Install,
 }
 
-/// `maestro worktrees <list|reclaim>`
-#[derive(Debug, clap::Args)]
-pub struct WorktreesArgs {
-    #[command(subcommand)]
-    pub command: WorktreesCommand,
-}
-
-#[derive(Debug, Subcommand)]
-pub enum WorktreesCommand {
-    /// List retained worktrees and what is in the way of reclaiming each.
-    List(ListWorktreesArgs),
-    /// Reclaim one retained worktree: removes the directory, never the branch.
-    Reclaim(ReclaimArgs),
-}
-
-#[derive(Debug, clap::Args)]
-pub struct ListWorktreesArgs {
-    /// Output as JSON instead of a table.
-    #[arg(long)]
-    pub json: bool,
-}
-
-#[derive(Debug, clap::Args)]
-pub struct ReclaimArgs {
-    /// Retained worktree ID to reclaim. Required — there is no reclaim-all.
-    pub id: Uuid,
-}
-
 #[derive(Debug, clap::Args)]
 pub struct ListArgs {
     /// Output as JSON instead of a table.
@@ -138,8 +108,7 @@ pub struct KillArgs {
     /// Session ID to kill.
     pub session_id: Uuid,
     /// End the session even when its worktree holds unsaved work (#81). The
-    /// worktree is retained — never removed — and enters the retained store;
-    /// the output says where the work went.
+    /// worktree is left on disk — never removed.
     #[arg(long)]
     pub force: bool,
 }
@@ -301,63 +270,11 @@ mod tests {
             }
             _ => panic!("wrong subcommand"),
         }
-        // No subcommand → help, like `worktrees`.
+        // No subcommand → help.
         let err = Cli::try_parse_from(["maestro", "hooks"]).unwrap_err();
         assert!(matches!(
             err.kind(),
             clap::error::ErrorKind::DisplayHelpOnMissingArgumentOrSubcommand
         ));
     }
-
-    #[test]
-    fn parses_worktrees_list() {
-        let cli = Cli::try_parse_from(["maestro", "worktrees", "list", "--json"]).unwrap();
-        match cli.command {
-            Command::Worktrees(w) => match w.command {
-                WorktreesCommand::List(a) => assert!(a.json),
-                _ => panic!("wrong worktrees subcommand"),
-            },
-            _ => panic!("wrong subcommand"),
-        }
-    }
-
-    #[test]
-    fn parses_worktrees_reclaim() {
-        let id = "01234567-89ab-cdef-0123-456789abcdef";
-        let cli = Cli::try_parse_from(["maestro", "worktrees", "reclaim", id]).unwrap();
-        match cli.command {
-            Command::Worktrees(w) => match w.command {
-                WorktreesCommand::Reclaim(a) => assert_eq!(a.id.to_string(), id),
-                _ => panic!("wrong worktrees subcommand"),
-            },
-            _ => panic!("wrong subcommand"),
-        }
-    }
-
-    /// Contract rule: reclaim refuses to run without an explicit target.
-    /// There is no reclaim-everything convenience.
-    #[test]
-    fn reclaim_requires_an_explicit_target() {
-        let err = Cli::try_parse_from(["maestro", "worktrees", "reclaim"]).unwrap_err();
-        assert!(
-            matches!(
-                err.kind(),
-                clap::error::ErrorKind::InvalidSubcommand
-                    | clap::error::ErrorKind::MissingRequiredArgument
-            ),
-            "{err}"
-        );
-    }
-
-    #[test]
-    fn worktrees_without_subcommand_shows_help() {
-        let err = Cli::try_parse_from(["maestro", "worktrees"]).unwrap_err();
-        assert!(
-            matches!(
-                err.kind(),
-                clap::error::ErrorKind::DisplayHelpOnMissingArgumentOrSubcommand
-            ),
-            "{err}"
-        );
-    }
 }

crates/maestro-cli/src/dispatch.rs

@@ -9,9 +9,8 @@ use std::path::{Path, PathBuf};
 use maestro_protocol::DaemonClient;
 
 use crate::attach::{parse_detach_key, run_attach_url};
-use crate::cli::{Command, WorktreesCommand};
+use crate::cli::Command;
 use crate::sessions::{run_kill, run_list, run_spawn};
-use crate::worktrees::{run_list as run_worktree_list, run_reclaim as run_worktree_reclaim};
 use maestro_config::{default_config_path, MaestroConfigFile};
 
 #[derive(Debug, thiserror::Error)]
@@ -80,37 +79,10 @@ pub async fn dispatch(
             if !args.force {
                 warn_if_blocked(&daemon, args.session_id).await;
             }
-            let outcome = run_kill(&daemon, &args).await?;
+            run_kill(&daemon, &args).await?;
             println!("killed {}", args.session_id);
-            if let Some(w) = outcome.retained {
-                println!(
-                    "worktree retained at {} (branch {}, force-killed)",
-                    w.path.display(),
-                    w.branch
-                );
-            }
         }
         Command::Hooks(_) => unreachable!("hooks handled before the daemon client exists"),
-        Command::Worktrees(args) => match args.command {
-            WorktreesCommand::List(list_args) => {
-                let mut stdout = std::io::stdout().lock();
-                run_worktree_list(&daemon, &list_args, &mut stdout).await?;
-            }
-            WorktreesCommand::Reclaim(reclaim_args) => {
-                // The daemon removes the directory and the store entry; the
-                // branch is never deleted — worth saying out loud (#86).
-                let entry = run_worktree_reclaim(&daemon, &reclaim_args).await?;
-                match entry {
-                    Some(wt) => println!(
-                        "reclaimed {} ({})\nbranch '{}' kept",
-                        wt.id,
-                        wt.path.display(),
-                        wt.branch
-                    ),
-                    None => println!("reclaimed {}\nbranch kept", reclaim_args.id),
-                }
-            }
-        },
     }
     Ok(())
 }
@@ -150,7 +122,7 @@ async fn warn_if_blocked(daemon: &DaemonClient, id: uuid::Uuid) {
     {
         if !b.is_clear() {
             eprintln!(
-                "warning: this kill will be refused — the session's worktree holds unsaved work ({b}); use --force to retain it"
+                "warning: this kill will be refused — the session's worktree holds unsaved work ({b}); use --force to end it anyway"
             );
         }
     }

crates/maestro-cli/src/lib.rs

@@ -6,6 +6,5 @@ pub mod attach;
 pub mod cli;
 pub mod dispatch;
 pub mod sessions;
-pub mod worktrees;
 
 pub use dispatch::{dispatch, CliError};

crates/maestro-cli/src/sessions.rs

@@ -62,10 +62,7 @@ pub async fn run_spawn(client: &DaemonClient, args: SpawnArgs) -> Result<Session
 /// `maestro kill`. A forced kill (#81) retains the guarded worktree instead
 /// of removing it; the outcome names what was kept so dispatch can say where
 /// the work went.
-pub async fn run_kill(
-    client: &DaemonClient,
-    args: &KillArgs,
-) -> Result<maestro_protocol::KillOutcome, ClientError> {
+pub async fn run_kill(client: &DaemonClient, args: &KillArgs) -> Result<(), ClientError> {
     client.kill_session(args.session_id, args.force).await
 }
 

crates/maestro-cli/src/worktrees.rs

@@ -1,204 +0,0 @@
-//! `maestro worktrees list / reclaim` — the retained-store surface (#86).
-//!
-//! `reclaim` removes one directory; it never deletes the branch, which is
-//! said out loud in the output because "reclaim" otherwise sounds like it
-//! takes the commits too.
-
-use std::io::Write;
-
-use maestro_protocol::display::abbreviate_dir;
-use maestro_protocol::{ClientError, DaemonClient, DaemonInfo, InTheWay, RetainedWorktree};
-
-use crate::cli::{ListWorktreesArgs, ReclaimArgs};
-
-/// `maestro worktrees list`
-pub async fn run_list(
-    client: &DaemonClient,
-    args: &ListWorktreesArgs,
-    out: &mut impl Write,
-) -> Result<(), ClientError> {
-    let worktrees = client.list_worktrees().await?;
-    let info = client.daemon_info().await.unwrap_or_default();
-
-    if args.json {
-        let body = serde_json::json!({ "worktrees": worktrees });
-        writeln!(out, "{}", serde_json::to_string_pretty(&body)?).ok();
-        return Ok(());
-    }
-
-    print_worktree_table(out, &worktrees, &info);
-    Ok(())
-}
-
-/// `maestro worktrees reclaim <id>`
-///
-/// Returns the entry as listed just before the reclaim, when it was found in
-/// the listing, so the confirmation can name what was removed.
-pub async fn run_reclaim(
-    client: &DaemonClient,
-    args: &ReclaimArgs,
-) -> Result<Option<RetainedWorktree>, ClientError> {
-    let listed = client
-        .list_worktrees()
-        .await?
-        .into_iter()
-        .find(|w| w.id == args.id);
-    client.reclaim_worktree(args.id).await?;
-    Ok(listed)
-}
-
-fn print_worktree_table(out: &mut impl Write, worktrees: &[RetainedWorktree], info: &DaemonInfo) {
-    if worktrees.is_empty() {
-        writeln!(out, "no retained worktrees").ok();
-        return;
-    }
-
-    let mut sorted: Vec<&RetainedWorktree> = worktrees.iter().collect();
-    sorted.sort_by_key(|w| w.retained_at);
-
-    if !info.hostname.is_empty() {
-        let warn = if maestro_protocol::is_compatible(&info.version) {
-            ""
-        } else {
-            "  (protocol mismatch)"
-        };
-        let version = if info.version.is_empty() {
-            String::new()
-        } else {
-            format!("  maestro {}{warn}", info.version)
-        };
-        writeln!(out, "\n{}{version}", info.hostname).ok();
-    }
-    writeln!(
-        out,
-        "  {:<36}  {:<24}  {:<28}  {:<24}  IN THE WAY",
-        "ID", "BRANCH", "PATH", "REASON"
-    )
-    .ok();
-    for w in sorted {
-        writeln!(
-            out,
-            "  {:<36}  {:<24}  {:<28}  {:<24}  {}",
-            w.id,
-            w.branch,
-            abbreviate_dir(&w.path, &info.home_dir),
-            reason_label(w.reason),
-            blocker_label(&w.in_the_way),
-        )
-        .ok();
-    }
-    // The branch survives every reclaim — see docs/WORKTREES.md.
-    writeln!(
-        out,
-        "\nreclaim removes the directory only; branches are never deleted"
-    )
-    .ok();
-}
-
-fn reason_label(reason: maestro_protocol::RetainedReason) -> &'static str {
-    match reason {
-        maestro_protocol::RetainedReason::ForceKilled => "force-killed",
-        maestro_protocol::RetainedReason::ExitedWithUnsavedWork => "exited-with-unsaved-work",
-    }
-}
-
-/// The same breakdown that blocks an unforced kill, as words: what is still
-/// in the way, or `clear` when the work has since been committed and pushed.
-fn blocker_label(in_the_way: &InTheWay) -> String {
-    if in_the_way.is_clear() {
-        return "clear".into();
-    }
-    let mut parts: Vec<String> = Vec::new();
-    if in_the_way.uncommitted_files > 0 {
-        parts.push(format!("{} uncommitted", in_the_way.uncommitted_files));
-    }
-    if in_the_way.untracked_files > 0 {
-        parts.push(format!("{} untracked", in_the_way.untracked_files));
-    }
-    if in_the_way.unpushed_commits > 0 {
-        parts.push(format!("{} unpushed", in_the_way.unpushed_commits));
-    }
-    parts.join(", ")
-}
-
-#[cfg(test)]
-mod tests {
-    use std::path::PathBuf;
-
-    use chrono::{TimeZone, Utc};
-    use maestro_protocol::{DaemonInfo, InTheWay, RetainedReason, RetainedWorktree};
-    use uuid::Uuid;
-
-    fn worktree(branch: &str, path: &str, id: u128) -> RetainedWorktree {
-        RetainedWorktree {
-            id: Uuid::from_u128(id),
-            path: PathBuf::from(path),
-            base_repo: PathBuf::from("/home/naps/maestro"),
-            branch: branch.into(),
-            reason: RetainedReason::ForceKilled,
-            retained_at: Utc.with_ymd_and_hms(2026, 8, 24, 10, 0, 0).unwrap(),
-            in_the_way: InTheWay::default(),
-        }
-    }
-
-    fn info(home: &str) -> DaemonInfo {
-        DaemonInfo {
-            hostname: "naps-laptop".into(),
-            home_dir: home.into(),
-            ..Default::default()
-        }
-    }
-
-    #[test]
-    fn formats_table() {
-        let mut w = worktree("feat/login", "/home/naps/maestro/worktrees/feat/login", 1);
-        w.in_the_way = InTheWay {
-            uncommitted_files: 2,
-            untracked_files: 1,
-            unpushed_commits: 3,
-        };
-        let mut buf = Vec::new();
-        super::print_worktree_table(&mut buf, &[w], &info("/home/naps"));
-        let s = String::from_utf8(buf).unwrap();
-        assert!(s.contains("naps-laptop"), "{s}");
-        assert!(s.contains("feat/login"), "{s}");
-        // Home dir collapses to ~ in the PATH column, like sessions' DIR.
-        assert!(s.contains("~/maestro/worktrees/feat/login"), "{s}");
-        assert!(s.contains("force-killed"), "{s}");
-        assert!(s.contains("2 uncommitted, 1 untracked, 3 unpushed"), "{s}");
-        assert!(s.contains("IN THE WAY"), "{s}");
-    }
-
-    /// Contract rule, visible to the operator: reclaiming a tree must not
-    /// throw away commits on its branch.
-    #[test]
-    fn says_the_branch_survives() {
-        let mut buf = Vec::new();
-        super::print_worktree_table(
-            &mut buf,
-            &[worktree("feat/x", "/repo/worktrees/feat/x", 1)],
-            &info(""),
-        );
-        let s = String::from_utf8(buf).unwrap();
-        assert!(s.contains("never deleted"), "{s}");
-    }
-
-    #[test]
-    fn clear_tree_says_clear() {
-        let mut buf = Vec::new();
-        super::print_worktree_table(
-            &mut buf,
-            &[worktree("main-ish", "/repo/worktrees/main-ish", 1)],
-            &info(""),
-        );
-        let s = String::from_utf8(buf).unwrap();
-        assert!(s.contains("clear"), "{s}");
-    }
-
-    #[test]
-    fn empty_store() {
-        let mut buf = Vec::new();
-        super::print_worktree_table(&mut buf, &[], &info(""));
-        assert_eq!(String::from_utf8(buf).unwrap(), "no retained worktrees\n");
-    }
-}

crates/maestro-daemon/src/cli.rs

@@ -70,9 +70,6 @@ async fn run(config_override: Option<PathBuf>) -> Result<(), Box<dyn Error>> {
         Some(p) => Tmux::new().with_tmpdir(p),
         None => Tmux::new(),
     };
-    let retained = std::sync::Arc::new(crate::retained_store::RetainedStore::new(
-        config.retained_worktree_dir(),
-    ));
     let manager = SessionManager::new_with_replay(
         std::sync::Arc::new(tmux),
         sidecar,
@@ -83,10 +80,7 @@ async fn run(config_override: Option<PathBuf>) -> Result<(), Box<dyn Error>> {
     .with_spawn_history(crate::spawn_history::SpawnHistory::new(
         config.spawn_history_path(),
         config.spawn_history_days,
-    ))
-    // One store instance shared with the API handlers: forced kills (#81)
-    // record here, `GET /api/worktrees` (#80) reads back from here.
-    .with_retained_store((*retained).clone());
+    ));
     // Spawned sessions learn where to report back (#103). Derived from the
     // configured bind address; set before any spawn can happen.
     manager.set_daemon_url(format!("http://{}", config.bind_addr));
@@ -108,7 +102,6 @@ async fn run(config_override: Option<PathBuf>) -> Result<(), Box<dyn Error>> {
         config.home_dir.clone(),
         config.spawn_presets.clone(),
         projects,
-        retained,
     ));
 
     let listener = TcpListener::bind(config.bind_addr).await?;

crates/maestro-daemon/src/config.rs

@@ -107,11 +107,6 @@ impl DaemonConfig {
     pub fn spawn_history_path(&self) -> PathBuf {
         self.state_dir.join("spawn-history.jsonl")
     }
-
-    /// Retained-worktree store directory (#78): `<state_dir>/retained-worktrees/`.
-    pub fn retained_worktree_dir(&self) -> PathBuf {
-        self.state_dir.join("retained-worktrees")
-    }
 }
 
 /// Env-var overrides applied by [`load_config`]. Tests pass explicit

crates/maestro-daemon/src/lib.rs

@@ -20,7 +20,6 @@ pub mod panes;
 pub mod proc;
 pub mod projects;
 pub mod pty;
-pub mod retained_store;
 pub mod session;
 pub mod sidecar;
 pub mod spawn_history;

crates/maestro-daemon/src/local_api.rs

@@ -38,9 +38,6 @@ pub struct LocalApiState {
     pub presets: Arc<Vec<SpawnPreset>>,
     /// Project discovery (#54) backing `GET /api/projects`.
     pub projects: Arc<crate::projects::ProjectScanner>,
-    /// Retained worktrees (#78) backing `GET /api/worktrees` and their
-    /// reclaim endpoint (#80).
-    pub retained: Arc<crate::retained_store::RetainedStore>,
 }
 
 pub fn router(state: LocalApiState) -> Router {
@@ -57,8 +54,6 @@ pub fn router(state: LocalApiState) -> Router {
         .route(routes::GITROOT, get(git_root))
         .route(routes::BRANCHES, get(crate::branches::handler))
         .route(routes::NAMES, get(names))
-        .route(routes::WORKTREES, get(list_worktrees))
-        .route(routes::patterns::WORKTREE, delete(reclaim_worktree))
         .route(routes::INFO, get(daemon_info))
         .route(routes::EVENTS, any(events_ws))
         .route(routes::patterns::ATTACH, any(attach_ws))
@@ -105,8 +100,6 @@ fn session_error_status(code: &str) -> StatusCode {
         "invalid_env" => StatusCode::BAD_REQUEST,
         // A guard refusal is the daemon doing its job, not a failure (#79).
         "kill_guarded" => StatusCode::CONFLICT,
-        // The retention record failed before anything was torn down (#81).
-        "retain_failed" => StatusCode::INTERNAL_SERVER_ERROR,
         _ => StatusCode::BAD_GATEWAY,
     }
 }
@@ -186,9 +179,8 @@ async fn create_session(
 /// code + message: a `kill_guarded` refusal (#79) carries the unsaved-work
 /// breakdown clients render as "3 uncommitted files, 2 unpushed commits".
 ///
-/// With `force=true` (#81) the refusal is overridden and the worktree
-/// retained; the answer is `200` with a [`KillOutcome`] naming what was kept.
-/// Without it the historical `204` stands.
+/// With `force=true` (#81) the refusal is overridden and the worktree is left
+/// on disk rather than removed. Either way the answer is `204`.
 #[derive(Debug, Deserialize)]
 struct KillQuery {
     #[serde(default)]
@@ -201,13 +193,7 @@ async fn kill_session(
     Query(q): Query<KillQuery>,
 ) -> Response {
     match s.manager.kill(id, q.force).await {
-        Ok(outcome) => {
-            if q.force {
-                (StatusCode::OK, Json(outcome)).into_response()
-            } else {
-                StatusCode::NO_CONTENT.into_response()
-            }
-        }
+        Ok(()) => StatusCode::NO_CONTENT.into_response(),
         Err(e) => {
             let info = e.to_error_info();
             let status = session_error_status(&info.code);
@@ -238,112 +224,6 @@ async fn rename_session(
     }
 }
 
-/// `GET /api/worktrees` (#80): every retained worktree, with the in-the-way
-/// breakdown recomputed at read time — never stored, so a tree whose work has
-/// since been committed or pushed says so. Entries whose directory somebody
-/// removed by hand are pruned on the way through.
-async fn list_worktrees(State(s): State<LocalApiState>) -> Response {
-    let retained = s.retained.clone();
-    let listed = tokio::task::spawn_blocking(move || {
-        retained.prune_missing()?;
-        retained
-            .load_all()?
-            .iter()
-            .map(|entry| {
-                // A tree git cannot read (permissions, corrupt checkout)
-                // still gets listed; it just reports nothing in the way.
-                let in_the_way = crate::worktree_probe::in_the_way(&entry.path).unwrap_or_default();
-                Ok(maestro_protocol::RetainedWorktree {
-                    id: entry.id,
-                    path: entry.path.clone(),
-                    base_repo: entry.base_repo.clone(),
-                    branch: entry.branch.clone(),
-                    reason: entry.reason,
-                    retained_at: entry.retained_at,
-                    in_the_way,
-                })
-            })
-            .collect::<Result<Vec<_>, crate::retained_store::RetainedStoreError>>()
-    })
-    .await;
-    match listed {
-        Ok(Ok(worktrees)) => Json(worktrees).into_response(),
-        Ok(Err(e)) => error_response(
-            StatusCode::INTERNAL_SERVER_ERROR,
-            "store_failed",
-            &e.to_string(),
-        ),
-        Err(e) => error_response(
-            StatusCode::INTERNAL_SERVER_ERROR,
-            "join_failed",
-            &e.to_string(),
-        ),
-    }
-}
-
-/// `DELETE /api/worktrees/:id` (#80): reclaim one retained worktree. Removes
-/// the directory and drops the store entry, without consulting the guard —
-/// reclaim is the deliberate destructive act, chosen while looking at what is
-/// in the way. The branch itself is never deleted; commits on it survive.
-async fn reclaim_worktree(State(s): State<LocalApiState>, Path(id): Path<Uuid>) -> Response {
-    let found = {
-        let retained = s.retained.clone();
-        tokio::task::spawn_blocking(move || -> Result<
-            Option<crate::retained_store::RetainedWorktreeRecord>,
-            crate::retained_store::RetainedStoreError,
-        > {
-            Ok(retained.load_all()?.into_iter().find(|e| e.id == id))
-        })
-        .await
-    };
-    let entry = match found {
-        Ok(Ok(Some(entry))) => entry,
-        Ok(Ok(None)) => {
-            return error_response(
-                StatusCode::NOT_FOUND,
-                "not_found",
-                "no such retained worktree",
-            )
-        }
-        Ok(Err(e)) => {
-            return error_response(
-                StatusCode::INTERNAL_SERVER_ERROR,
-                "store_failed",
-                &e.to_string(),
-            )
-        }
-        Err(e) => {
-            return error_response(
-                StatusCode::INTERNAL_SERVER_ERROR,
-                "join_failed",
-                &e.to_string(),
-            )
-        }
-    };
-
-    let outcome = {
-        let retained = s.retained.clone();
-        tokio::task::spawn_blocking(move || -> Result<(), String> {
-            crate::worktree_probe::reclaim(&entry.base_repo, &entry.path)
-                .map_err(|e| e.to_string())?;
-            retained.remove(id).map_err(|e| e.to_string())?;
-            Ok(())
-        })
-        .await
-    };
-    match outcome {
-        Ok(Ok(())) => StatusCode::NO_CONTENT.into_response(),
-        // The directory could not be removed: leave the entry so the
-        // operator still sees the tree and can retry.
-        Ok(Err(e)) => error_response(StatusCode::BAD_GATEWAY, "reclaim_failed", &e.to_string()),
-        Err(e) => error_response(
-            StatusCode::INTERNAL_SERVER_ERROR,
-            "join_failed",
-            &e.to_string(),
-        ),
-    }
-}
-
 /// `POST /api/sessions/:id/activity` (#105): an agent reports its own state.
 /// The report holds above the daemon's inference for a TTL, then inference
 /// resumes — an agent that dies mid-run cannot pin its session at `Error`
@@ -382,7 +262,6 @@ pub fn state_for(
     home_dir: impl Into<String>,
     presets: Vec<SpawnPreset>,
     projects: Arc<crate::projects::ProjectScanner>,
-    retained: Arc<crate::retained_store::RetainedStore>,
 ) -> LocalApiState {
     LocalApiState {
         manager,
@@ -390,7 +269,6 @@ pub fn state_for(
         home_dir: Arc::new(home_dir.into()),
         presets: Arc::new(presets),
         projects,
-        retained,
     }
 }
 

crates/maestro-daemon/src/panes.rs

@@ -39,6 +39,10 @@ pub trait Panes: std::fmt::Debug + Send + Sync + 'static {
     fn pipe_pane(&self, id: Uuid, sink_command: &str) -> Result<(), TmuxError>;
 
     fn resize_window(&self, id: Uuid, cols: u16, rows: u16) -> Result<(), TmuxError>;
+
+    /// Escape-sequence snapshot of the pane's current screen, or `None` when
+    /// there is no pane to capture. See [`Tmux::capture_pane`].
+    fn capture_pane(&self, id: Uuid) -> Result<Option<Vec<u8>>, TmuxError>;
 }
 
 impl Panes for Tmux {
@@ -76,6 +80,10 @@ impl Panes for Tmux {
     fn resize_window(&self, id: Uuid, cols: u16, rows: u16) -> Result<(), TmuxError> {
         Tmux::resize_window(self, id, cols, rows)
     }
+
+    fn capture_pane(&self, id: Uuid) -> Result<Option<Vec<u8>>, TmuxError> {
+        Tmux::capture_pane(self, id)
+    }
 }
 
 // ---------------------------------------------------------------------------
@@ -92,6 +100,7 @@ pub enum Op {
     SendKeys,
     PipePane,
     ResizeWindow,
+    CapturePane,
 }
 
 /// One recorded call, in the order the manager made it. Rollback ordering is a
@@ -116,6 +125,7 @@ pub enum Call {
         cols: u16,
         rows: u16,
     },
+    CapturePane(Uuid),
 }
 
 /// In-memory [`Panes`] for tests.
@@ -133,6 +143,7 @@ struct FakeState {
     live: HashSet<Uuid>,
     panes: HashMap<Uuid, PaneInfo>,
     calls: Vec<Call>,
+    captures: HashMap<Uuid, Vec<u8>>,
     fail: HashMap<Op, String>,
 }
 
@@ -162,6 +173,12 @@ impl FakePanes {
         self.lock().live.contains(&id)
     }
 
+    /// Screen snapshot `capture_pane` hands back for `id`. Unset means "no
+    /// pane", which is what a dead session looks like.
+    pub fn set_capture(&self, id: Uuid, bytes: impl Into<Vec<u8>>) {
+        self.lock().captures.insert(id, bytes.into());
+    }
+
     pub fn set_pane_info(&self, id: Uuid, info: PaneInfo) {
         self.lock().panes.insert(id, info);
     }
@@ -242,6 +259,12 @@ impl Panes for FakePanes {
     fn resize_window(&self, id: Uuid, cols: u16, rows: u16) -> Result<(), TmuxError> {
         self.record(Call::ResizeWindow { id, cols, rows }, Op::ResizeWindow)
     }
+
+    fn capture_pane(&self, id: Uuid) -> Result<Option<Vec<u8>>, TmuxError> {
+        self.record(Call::CapturePane(id), Op::CapturePane)?;
+        let st = self.lock();
+        Ok(st.captures.get(&id).cloned())
+    }
 }
 
 #[cfg(test)]

crates/maestro-daemon/src/pty.rs

@@ -141,8 +141,21 @@ impl PtyBridge {
     /// `subscribe()` keeps the reader task from publishing in between, so
     /// the seam has no gap and no duplicates.
     pub fn subscribe_with_replay(&self) -> (Vec<u8>, broadcast::Receiver<Vec<u8>>) {
+        self.subscribe_with_snapshot(|| None)
+    }
+
+    /// As [`Self::subscribe_with_replay`], but lets the caller supply the
+    /// replay payload instead of the raw byte buffer — used to hand a fresh
+    /// client tmux's own view of the screen, which the byte tail cannot
+    /// reconstruct for a full-screen program. `capture` runs while the reader
+    /// is held off, so the payload and the live stream still meet with no gap
+    /// and no duplicate. Returning `None` falls back to the byte buffer.
+    pub fn subscribe_with_snapshot<F>(&self, capture: F) -> (Vec<u8>, broadcast::Receiver<Vec<u8>>)
+    where
+        F: FnOnce() -> Option<Vec<u8>>,
+    {
         let guard = self.replay.lock_for_snapshot();
-        let snapshot = guard.snapshot();
+        let snapshot = capture().unwrap_or_else(|| guard.snapshot());
         let rx = match &self.sender {
             Some(tx) => tx.subscribe(),
             // Sealed: hand back the scrollback plus a receiver that is already

crates/maestro-daemon/src/retained_store.rs

@@ -1,310 +0,0 @@
-//! Store of retained worktrees (#78): worktrees the daemon created whose
-//! session is gone.
-//!
-//! One JSON file per entry under `<state_dir>/retained-worktrees/`, written
-//! the way `sidecar.rs` writes session records — atomic tmp+rename replace,
-//! missing or malformed files tolerated on read. The store survives a daemon
-//! restart; it is the recovery path for everything else in the worktree
-//! lifecycle.
-//!
-//! It is not an ownership record: it must never be consulted to decide whether
-//! a path may be adopted. Its only job is to remember what the daemon is
-//! holding so it can be listed and reclaimed.
-
-use std::path::{Path, PathBuf};
-
-use chrono::{DateTime, Utc};
-/// Why a worktree outlived its session — [`maestro_protocol::RetainedReason`],
-/// so the store and the wire payload cannot drift apart.
-pub use maestro_protocol::RetainedReason;
-use uuid::Uuid;
-
-#[derive(Debug, thiserror::Error)]
-pub enum RetainedStoreError {
-    #[error("io: {0}")]
-    Io(#[from] std::io::Error),
-    #[error("json: {0}")]
-    Json(#[from] serde_json::Error),
-}
-
-#[derive(Debug, Clone, PartialEq, serde::Serialize, serde::Deserialize)]
-pub struct RetainedWorktreeRecord {
-    pub id: Uuid,
-    pub path: PathBuf,
-    pub base_repo: PathBuf,
-    pub branch: String,
-    pub reason: RetainedReason,
-    pub retained_at: DateTime<Utc>,
-}
-
-#[derive(Debug, Clone)]
-pub struct RetainedStore {
-    dir: PathBuf,
-}
-
-impl RetainedStore {
-    pub fn new(dir: impl Into<PathBuf>) -> Self {
-        Self { dir: dir.into() }
-    }
-
-    pub fn dir(&self) -> &Path {
-        &self.dir
-    }
-
-    fn path_for(&self, id: Uuid) -> PathBuf {
-        self.dir.join(format!("{id}.json"))
-    }
-
-    /// Record a retained worktree and return the entry that was written.
-    pub fn retain(
-        &self,
-        path: &Path,
-        base_repo: &Path,
-        branch: &str,
-        reason: RetainedReason,
-    ) -> Result<RetainedWorktreeRecord, RetainedStoreError> {
-        let entry = RetainedWorktreeRecord {
-            id: Uuid::new_v4(),
-            path: path.to_path_buf(),
-            base_repo: base_repo.to_path_buf(),
-            branch: branch.to_string(),
-            reason,
-            retained_at: Utc::now(),
-        };
-        self.overwrite(&entry)?;
-        Ok(entry)
-    }
-
-    /// Atomically write `entry`. Creates the directory if it does not exist.
-    pub fn overwrite(&self, entry: &RetainedWorktreeRecord) -> Result<(), RetainedStoreError> {
-        std::fs::create_dir_all(&self.dir)?;
-        let final_path = self.path_for(entry.id);
-        let tmp = self.dir.join(format!("{}.json.tmp", entry.id));
-        let json = serde_json::to_vec_pretty(entry)?;
-        std::fs::write(&tmp, json)?;
-        std::fs::rename(&tmp, &final_path)?;
-        Ok(())
-    }
-
-    /// Drop one entry. No-op when the id is unknown — reclaiming an entry the
-    /// store already lost (e.g. pruned) must not fail.
-    pub fn remove(&self, id: Uuid) -> Result<(), RetainedStoreError> {
-        match std::fs::remove_file(self.path_for(id)) {
-            Ok(()) => Ok(()),
-            Err(e) if e.kind() == std::io::ErrorKind::NotFound => Ok(()),
-            Err(e) => Err(e.into()),
-        }
-    }
-
-    /// Read every entry. Files that are missing or malformed are skipped
-    /// silently: a corrupt record must not hide the others, and an empty or
-    /// absent directory simply means nothing is retained.
-    pub fn load_all(&self) -> Result<Vec<RetainedWorktreeRecord>, RetainedStoreError> {
-        let mut out = Vec::new();
-        let entries = match std::fs::read_dir(&self.dir) {
-            Ok(e) => e,
-            Err(e) if e.kind() == std::io::ErrorKind::NotFound => return Ok(out),
-            Err(e) => return Err(e.into()),
-        };
-        for entry in entries {
-            let entry = entry?;
-            let path = entry.path();
-            if path.extension().and_then(|s| s.to_str()) != Some("json") {
-                continue;
-            }
-            let Ok(bytes) = std::fs::read(&path) else {
-                continue;
-            };
-            if let Ok(worktree) = serde_json::from_slice::<RetainedWorktreeRecord>(&bytes) {
-                out.push(worktree);
-            }
-        }
-        out.sort_by_key(|w| w.retained_at);
-        Ok(out)
-    }
-
-    /// Drop entries whose worktree directory no longer exists — somebody
-    /// removed it by hand. Returns the ids that were pruned.
-    pub fn prune_missing(&self) -> Result<Vec<Uuid>, RetainedStoreError> {
-        let mut pruned = Vec::new();
-        for entry in self.load_all()? {
-            if !entry.path.exists() {
-                self.remove(entry.id)?;
-                pruned.push(entry.id);
-            }
-        }
-        Ok(pruned)
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use chrono::Duration;
-    use std::path::PathBuf;
-
-    fn tmp_dir() -> tempfile::TempDir {
-        tempfile::tempdir().expect("tempdir")
-    }
-
-    fn sample_path(root: &std::path::Path, name: &str) -> PathBuf {
-        let p = root.join(name);
-        std::fs::create_dir_all(&p).expect("mkdir");
-        p
-    }
-
-    #[test]
-    fn retain_then_load_round_trips_every_field() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let wt = sample_path(dir.path(), "wt");
-        let repo = sample_path(dir.path(), "repo");
-
-        let entry = store
-            .retain(&wt, &repo, "feature/login", RetainedReason::ForceKilled)
-            .expect("retain");
-
-        let all = store.load_all().expect("load_all");
-        assert_eq!(all, vec![entry.clone()]);
-        assert_eq!(entry.path, wt);
-        assert_eq!(entry.base_repo, repo);
-        assert_eq!(entry.branch, "feature/login");
-        assert_eq!(entry.reason, RetainedReason::ForceKilled);
-        let age = Utc::now() - entry.retained_at;
-        assert!(age >= Duration::zero() && age < Duration::minutes(1));
-    }
-
-    #[test]
-    fn reason_serializes_kebab_case() {
-        let json =
-            serde_json::to_string(&RetainedReason::ExitedWithUnsavedWork).expect("serialize");
-        assert_eq!(json, r#""exited-with-unsaved-work""#);
-    }
-
-    #[test]
-    fn entries_survive_a_restart() {
-        let dir = tmp_dir();
-        let wt = sample_path(dir.path(), "wt");
-
-        let first = RetainedStore::new(dir.path());
-        let entry = first
-            .retain(&wt, dir.path(), "main", RetainedReason::ForceKilled)
-            .expect("retain");
-
-        // A fresh store over the same state_dir — what a daemon restart sees.
-        let second = RetainedStore::new(dir.path());
-        assert_eq!(second.load_all().expect("load_all"), vec![entry]);
-    }
-
-    #[test]
-    fn missing_store_reads_as_empty() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path().join("does-not-exist"));
-        assert_eq!(store.load_all().expect("load_all"), Vec::new());
-    }
-
-    #[test]
-    fn malformed_entries_are_skipped_silently() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let wt = sample_path(dir.path(), "wt");
-        store
-            .retain(&wt, dir.path(), "kept", RetainedReason::ForceKilled)
-            .expect("retain");
-        std::fs::write(dir.path().join("not-a-uuid.json"), b"{oops").expect("write junk");
-
-        let all = store.load_all().expect("load_all");
-        assert_eq!(all.len(), 1);
-        assert_eq!(all[0].branch, "kept");
-    }
-
-    #[test]
-    fn ids_are_unique() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let a = store
-            .retain(dir.path(), dir.path(), "a", RetainedReason::ForceKilled)
-            .expect("retain");
-        let b = store
-            .retain(dir.path(), dir.path(), "b", RetainedReason::ForceKilled)
-            .expect("retain");
-        assert_ne!(a.id, b.id);
-    }
-
-    #[test]
-    fn remove_drops_the_entry() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let wt = sample_path(dir.path(), "wt");
-        let entry = store
-            .retain(&wt, dir.path(), "main", RetainedReason::ForceKilled)
-            .expect("retain");
-
-        store.remove(entry.id).expect("remove");
-        assert_eq!(store.load_all().expect("load_all"), Vec::new());
-
-        // Removing an unknown id is a no-op, not an error.
-        store.remove(entry.id).expect("remove again");
-    }
-
-    #[test]
-    fn load_all_is_sorted_by_time() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let mut first = store
-            .retain(dir.path(), dir.path(), "first", RetainedReason::ForceKilled)
-            .expect("retain");
-        first.retained_at -= Duration::hours(1);
-        store.overwrite(&first).expect("rewrite");
-
-        let second = store
-            .retain(
-                dir.path(),
-                dir.path(),
-                "second",
-                RetainedReason::ForceKilled,
-            )
-            .expect("retain");
-
-        let branches: Vec<_> = store
-            .load_all()
-            .expect("load_all")
-            .into_iter()
-            .map(|e| e.branch)
-            .collect();
-        assert_eq!(branches, vec!["first", "second"]);
-        let _ = second;
-    }
-
-    #[test]
-    fn prune_missing_drops_gone_directories_only() {
-        let dir = tmp_dir();
-        let store = RetainedStore::new(dir.path());
-        let alive = sample_path(dir.path(), "alive");
-        let gone = sample_path(dir.path(), "gone");
-        let keep = store
-            .retain(&alive, dir.path(), "keep", RetainedReason::ForceKilled)
-            .expect("retain");
-        let drop = store
-            .retain(
-                &gone,
-                dir.path(),
-                "drop",
-                RetainedReason::ExitedWithUnsavedWork,
-            )
-            .expect("retain");
-        std::fs::remove_dir_all(&gone).expect("rm");
-
-        let pruned = store.prune_missing().expect("prune_missing");
-        assert_eq!(pruned, vec![drop.id]);
-
-        let branches: Vec<_> = store
-            .load_all()
-            .expect("load_all")
-            .into_iter()
-            .map(|e| e.branch)
-            .collect();
-        assert_eq!(branches, vec!["keep"]);
-        let _ = keep;
-    }
-}

crates/maestro-daemon/src/session.rs

@@ -12,8 +12,8 @@ use std::time::Duration;
 
 use chrono::Utc;
 use maestro_protocol::{
-    ActivityState, CatalogEvent, CreateSessionRequest, ErrorInfo, KillOutcome, RetainedReason,
-    RetainedWorktree, Session, SessionStatus, WorktreeSpec,
+    ActivityState, CatalogEvent, CreateSessionRequest, ErrorInfo, Session, SessionStatus,
+    WorktreeSpec,
 };
 use tokio::sync::{broadcast, Mutex};
 use uuid::Uuid;
@@ -51,11 +51,6 @@ pub struct SessionRecord {
     /// `base_repo` from the original `WorktreeSpec`. Needed because `git
     /// worktree remove` is invoked from the base repo.
     pub worktree_base_repo: Option<PathBuf>,
-    /// Branch the worktree was created for, recorded at spawn time so a
-    /// forced kill (#81) can name it in the retained store without asking
-    /// git. Sessions adopted from a sidecar have no branch on record and
-    /// fall back to a probe.
-    pub worktree_branch: Option<String>,
     pub bridge: PtyBridge,
     /// Agent-reported activity (#105) still within its TTL. While in force it
     /// outranks the activity pass's own inference; when it lapses, inference
@@ -88,11 +83,6 @@ pub enum SessionError {
     /// so it stays the operator's handle on their own unfinished work.
     #[error("kill refused: the session's worktree holds unsaved work ({0})")]
     KillGuarded(maestro_protocol::InTheWay),
-    /// A forced kill (#81) records the retained worktree *before* tearing
-    /// anything down; if that write fails nothing was destroyed, and the
-    /// error says so rather than silently dropping unrecorded work.
-    #[error("could not record the retained worktree: {0}")]
-    RetainFailed(String),
 }
 
 impl SessionError {
@@ -105,7 +95,6 @@ impl SessionError {
             SessionError::NotFound(_) => "not_found",
             SessionError::InvalidEnv(_) => "invalid_env",
             SessionError::KillGuarded(_) => "kill_guarded",
-            SessionError::RetainFailed(_) => "retain_failed",
         };
         let unsaved_work = match self {
             SessionError::KillGuarded(b) => Some(*b),
@@ -138,9 +127,6 @@ struct Inner {
     /// `None` when not wired (tests, or before `with_spawn_history`); a present
     /// but disabled (`0`-day) history no-ops internally.
     spawn_history: Option<SpawnHistory>,
-    /// Retained-worktree store (#78). `None` when not wired (tests); a forced
-    /// kill (#81) without one fails rather than destroying unrecorded work.
-    retained: Option<crate::retained_store::RetainedStore>,
     /// The daemon's own HTTP base URL (#103), exported to spawned sessions as
     /// `MAESTRO_URL`. `None` when unknown — those sessions simply don't get
     /// the variable. A mutex because the runner sets it after construction,
@@ -187,7 +173,6 @@ impl SessionManager {
                 fifo_dir,
                 replay_bytes,
                 spawn_history: None,
-                retained: None,
                 daemon_url: std::sync::Mutex::new(None),
                 report_ttl: crate::activity::REPORT_TTL,
                 sessions: Mutex::new(HashMap::new()),
@@ -212,16 +197,6 @@ impl SessionManager {
         self
     }
 
-    /// Attach the retained-worktree store (#78). A forced kill (#81) needs it
-    /// to record what it retained; without one, forcing a guarded kill fails
-    /// instead of destroying unrecorded work.
-    pub fn with_retained_store(mut self, store: crate::retained_store::RetainedStore) -> Self {
-        let inner = Arc::get_mut(&mut self.inner)
-            .expect("with_retained_store must be called before the SessionManager is cloned");
-        inner.retained = Some(store);
-        self
-    }
-
     /// Override the agent-report TTL (#105). Tests pass a short one to watch
     /// inference resume without waiting out [`crate::activity::REPORT_TTL`].
     pub fn with_activity_report_ttl(mut self, ttl: Duration) -> Self {
@@ -285,8 +260,29 @@ impl SessionManager {
         &self,
         id: Uuid,
     ) -> Option<(Vec<u8>, broadcast::Receiver<Vec<u8>>)> {
+        let tmux = self.inner.tmux.clone();
         let sessions = self.inner.sessions.lock().await;
-        sessions.get(&id).map(|r| r.bridge.subscribe_with_replay())
+        let rec = sessions.get(&id)?;
+        // For a live session, tmux's own screen beats the byte tail: the tail
+        // starts mid-stream, so a full-screen program (an agent TUI) replayed
+        // from it renders as garbage until something forces a repaint. A dead
+        // session has no pane left to capture, so it keeps the byte tail.
+        let alive = rec.session.status == SessionStatus::Running;
+        // The capture shells out to tmux while the replay lock is held, which
+        // parks the reader for the length of one tmux round-trip. That only
+        // happens on attach, and the alternative is a seam in the stream.
+        Some(rec.bridge.subscribe_with_snapshot(|| {
+            if !alive {
+                return None;
+            }
+            match tmux.capture_pane(id) {
+                Ok(snapshot) => snapshot,
+                Err(e) => {
+                    tracing::warn!(error = ?e, %id, "capture-pane failed; replaying raw bytes");
+                    None
+                }
+            }
+        }))
     }
 
     pub async fn has(&self, id: Uuid) -> bool {
@@ -442,7 +438,6 @@ impl SessionManager {
                     fifo_path,
                     worktree_path,
                     worktree_base_repo,
-                    worktree_branch: req.worktree.as_ref().map(|w| w.branch.clone()),
                     bridge,
                     reported_activity: None,
                     settled: false,
@@ -494,19 +489,19 @@ impl SessionManager {
     /// is removed; the error names what blocked it and how much of it there
     /// is. The session stays as the operator's handle on their own unfinished
     /// work, so ending a session that still matters becomes a deliberate
-    /// `--force`, which retains rather than destroys (WORKTREES.md).
+    /// `--force`, which leaves the tree alone rather than destroying it
+    /// (WORKTREES.md).
     ///
     /// With `force` (#81) the refusal — and nothing else — is overridden:
-    /// the session ends and the guarded worktree is retained into the store
-    /// with reason `force-killed`, never removed. The outcome names what was
-    /// kept so a client can say where the work went. A tree whose guard is
-    /// clear never needed the override; force removes it as usual.
+    /// the session ends and the guarded worktree is left on disk, never
+    /// removed. A tree whose guard is clear never needed the override; force
+    /// removes it as usual.
     ///
     /// This is also how an already-dead session is dismissed. tmux is only
     /// asked to kill something it still has — calling `kill-session` on a
     /// session tmux already lost fails, and that failure is not interesting.
-    pub async fn kill(&self, id: Uuid, force: bool) -> Result<KillOutcome, SessionError> {
-        let (worktree, alive, known_branch) = {
+    pub async fn kill(&self, id: Uuid, force: bool) -> Result<(), SessionError> {
+        let (worktree, alive) = {
             let sessions = self.inner.sessions.lock().await;
             match sessions.get(&id) {
                 None => return Err(SessionError::NotFound(id)),
@@ -516,7 +511,6 @@ impl SessionManager {
                         .clone()
                         .zip(record.worktree_path.clone()),
                     record.session.status == SessionStatus::Running,
-                    record.worktree_branch.clone(),
                 ),
             }
         };
@@ -529,35 +523,13 @@ impl SessionManager {
             return Err(SessionError::KillGuarded(probe.expect("guarded ⇒ probed")));
         }
 
-        let mut retained = None;
-        if guarded {
-            let entry = record_retention(
-                &self.inner,
-                worktree.as_ref().expect("guarded ⇒ worktree"),
-                known_branch,
-                RetainedReason::ForceKilled,
-            )
-            .await?;
-            retained = Some((entry, probe.expect("guarded ⇒ probed")));
-        }
-
         let removed = {
             let mut sessions = self.inner.sessions.lock().await;
             sessions.remove(&id)
         };
         let record = removed.ok_or(SessionError::NotFound(id))?;
-        cleanup_record(&self.inner, id, record, alive, retained.is_some()).await;
-        Ok(KillOutcome {
-            retained: retained.map(|(entry, in_the_way)| RetainedWorktree {
-                id: entry.id,
-                path: entry.path,
-                base_repo: entry.base_repo,
-                branch: entry.branch,
-                reason: entry.reason,
-                retained_at: entry.retained_at,
-                in_the_way,
-            }),
-        })
+        cleanup_record(&self.inner, id, record, alive, guarded).await;
+        Ok(())
     }
 
     /// Spawn a background task that polls tmux every `interval` and reaps any
@@ -669,9 +641,6 @@ impl SessionManager {
                     // directory blocks that branch forever (#54 review).
                     worktree_path: session.worktree.as_ref().map(|w| w.path.clone()),
                     worktree_base_repo: session.worktree.as_ref().map(|w| w.base_repo.clone()),
-                    // The sidecar carries no branch; a forced kill of a
-                    // recovered session probes git for it at retain time.
-                    worktree_branch: None,
                     bridge,
                     reported_activity: None,
                     settled: false,
@@ -840,32 +809,6 @@ async fn probe_worktree(
         .map_err(|e| SessionError::Worktree(crate::worktree::WorktreeError::from(e)))
 }
 
-/// Record a guarded worktree into the retained store with `reason`, BEFORE
-/// the caller tears anything down: a failed write then leaves every artifact
-/// untouched instead of destroying work nothing records anymore. Shared by
-/// the forced kill (`ForceKilled`, #81) and the natural exit
-/// (`ExitedWithUnsavedWork`, #82) so the two paths cannot drift apart.
-async fn record_retention(
-    inner: &Arc<Inner>,
-    (base_repo, path): &(PathBuf, PathBuf),
-    known_branch: Option<String>,
-    reason: RetainedReason,
-) -> Result<crate::retained_store::RetainedWorktreeRecord, SessionError> {
-    let store = inner
-        .retained
-        .clone()
-        .ok_or_else(|| SessionError::RetainFailed("no retained store is wired".into()))?;
-    // Sessions adopted from a sidecar carry no branch; ask the tree itself.
-    let branch = known_branch
-        .or_else(|| crate::worktree_probe::current_branch(path))
-        .unwrap_or_default();
-    let base_repo = base_repo.clone();
-    let path = path.clone();
-    tmux_blocking(move || store.retain(&path, &base_repo, &branch, reason))
-        .await
-        .map_err(|e| SessionError::RetainFailed(e.to_string()))
-}
-
 /// One pass of the liveness watcher: diff the in-memory session set against
 /// the tmux-reported one, and reap anything we still track but tmux has lost.
 async fn run_liveness_pass(inner: &Arc<Inner>) {
@@ -1069,13 +1012,11 @@ async fn run_blocked_work_pass(inner: &Arc<Inner>) {
 /// The record, its [`PtyBridge`] and its sidecar all survive, so a client can
 /// still attach and read the replay buffer — the last thing a crashed agent
 /// printed. The worktree, though, is settled right away: a pristine tree is
-/// removed, anything else is retained into the store with reason
-/// `exited-with-unsaved-work` (#82) — nobody is waiting for an answer here,
-/// so there is nothing to refuse. Once settled, the session drops its
-/// worktree reference so a later dismissal neither re-probes nor retains
-/// twice. When settling is impossible (probe or store failure) nothing is
-/// destroyed: the tree stays attached to the dead record for a later forced
-/// dismissal (#81).
+/// removed (#82) — nobody is waiting for an answer here, so there is nothing
+/// to refuse. Once settled, the session drops its worktree reference so a
+/// later dismissal does not re-probe it. A tree holding unsaved work is left
+/// alone and stays attached to the dead record for a later forced dismissal
+/// (#81), which leaves it on disk too.
 ///
 /// Dead sessions do not survive a daemon restart: [`SessionManager::recover`]
 /// adopts only what tmux still holds.
@@ -1093,21 +1034,20 @@ async fn mark_session_dead(inner: &Arc<Inner>, id: Uuid) {
                     .worktree_base_repo
                     .clone()
                     .zip(record.worktree_path.clone());
-                let branch = record.worktree_branch.clone();
-                Some((session, worktree, branch))
+                Some((session, worktree))
             }
             _ => None,
         }
     };
-    let Some((mut session, worktree, branch)) = flipped else {
+    let Some((mut session, worktree)) = flipped else {
         return;
     };
     tracing::info!(%id, "session vanished from tmux — marking dead");
 
     if let Some(wt) = &worktree {
-        if settle_worktree_on_exit(inner, id, wt, branch).await {
-            // The tree is settled (removed or retained); a stale "in the
-            // way" would describe something the session no longer holds.
+        if settle_worktree_on_exit(inner, id, wt).await {
+            // The tree is settled (removed); a stale "in the way" would
+            // describe something the session no longer holds.
             session.blocked_work = None;
             let mut sessions = inner.sessions.lock().await;
             if let Some(record) = sessions.get_mut(&id) {
@@ -1115,7 +1055,6 @@ async fn mark_session_dead(inner: &Arc<Inner>, id: Uuid) {
                 record.session.blocked_work = None;
                 record.worktree_path = None;
                 record.worktree_base_repo = None;
-                record.worktree_branch = None;
             }
         }
     }
@@ -1134,10 +1073,8 @@ async fn mark_session_dead(inner: &Arc<Inner>, id: Uuid) {
 
 /// Apply the exit rule (#82) to a vanished session's worktree: probe what
 /// would be lost through the kill's own predicate, then remove a pristine
-/// tree or retain anything else under `exited-with-unsaved-work` — recorded
-/// before anything is torn down, exactly as the forced kill does (#81). A
-/// removal that fails falls back to retaining, since whatever blocked it is
-/// by definition something worth keeping.
+/// tree. Anything holding unsaved work is left exactly where it is — the
+/// same thing a forced kill does with a guarded tree (#81).
 ///
 /// Returns whether the worktree was fully accounted for. When it wasn't,
 /// nothing was destroyed and the caller keeps the reference on the record.
@@ -1145,7 +1082,6 @@ async fn settle_worktree_on_exit(
     inner: &Arc<Inner>,
     id: Uuid,
     (base_repo, path): &(PathBuf, PathBuf),
-    known_branch: Option<String>,
 ) -> bool {
     let probe = match probe_worktree(&(base_repo.clone(), path.clone())).await {
         Ok(probe) => probe,
@@ -1159,45 +1095,25 @@ async fn settle_worktree_on_exit(
             return false;
         }
     };
-    if probe.is_clear() {
-        match inner.worktree_mgr.remove(base_repo, path) {
-            Ok(()) => {
-                tracing::info!(%id, path = %path.display(), "natural exit: pristine worktree removed");
-                return true;
-            }
-            Err(e) => {
-                tracing::warn!(
-                    %id,
-                    path = %path.display(),
-                    error = ?e,
-                    "natural exit: removal failed; falling back to retention"
-                );
-            }
-        }
+    if !probe.is_clear() {
+        tracing::info!(
+            %id,
+            path = %path.display(),
+            "natural exit: worktree holds unsaved work; leaving it on disk"
+        );
+        return false;
     }
-    match record_retention(
-        inner,
-        &(base_repo.clone(), path.clone()),
-        known_branch,
-        RetainedReason::ExitedWithUnsavedWork,
-    )
-    .await
-    {
-        Ok(entry) => {
-            tracing::info!(
-                %id,
-                path = %path.display(),
-                entry = %entry.id,
-                "natural exit: worktree retained with unsaved work"
-            );
+    match inner.worktree_mgr.remove(base_repo, path) {
+        Ok(()) => {
+            tracing::info!(%id, path = %path.display(), "natural exit: pristine worktree removed");
             true
         }
         Err(e) => {
-            tracing::error!(
+            tracing::warn!(
                 %id,
                 path = %path.display(),
                 error = ?e,
-                "natural exit: could not record the retained worktree; leaving it untouched"
+                "natural exit: removal failed; leaving the worktree on disk"
             );
             false
         }
@@ -1214,15 +1130,15 @@ async fn settle_worktree_on_exit(
 /// resurrect it. `kill_tmux` is `true` for an explicit kill and `false` for the
 /// liveness reaper (tmux already lost the session).
 ///
-/// `retain_worktree` is `true` for a forced kill (#81) that retained the
-/// worktree instead of removing it: the store entry is already written, so
-/// only the removal is skipped.
+/// `keep_worktree` is `true` for a forced kill (#81) over a guarded tree: the
+/// session ends but the worktree is left on disk, so only the removal is
+/// skipped.
 async fn cleanup_record(
     inner: &Arc<Inner>,
     id: Uuid,
     record: SessionRecord,
     kill_tmux: bool,
-    retain_worktree: bool,
+    keep_worktree: bool,
 ) {
     if kill_tmux {
         let tmux = inner.tmux.clone();
@@ -1233,11 +1149,11 @@ async fn cleanup_record(
         tracing::warn!(error = ?e, %id, "cleanup: failed to delete sidecar");
     }
     if let (Some(base), Some(path)) = (&record.worktree_base_repo, &record.worktree_path) {
-        if retain_worktree {
+        if keep_worktree {
             tracing::info!(
                 %id,
                 path = %path.display(),
-                "forced kill: worktree retained in the store"
+                "forced kill: worktree left on disk"
             );
         } else if let Err(e) = inner.worktree_mgr.remove(base, path) {
             // The kill already passed the unsaved-work guard (#79), so this

crates/maestro-daemon/src/tmux.rs

@@ -246,6 +246,70 @@ impl Tmux {
         check(out)
     }
 
+    /// Escape-sequence snapshot of a pane's current screen (#…), for clients
+    /// that just attached.
+    ///
+    /// The raw byte replay buffer cannot reconstruct a full-screen program:
+    /// its tail starts mid-stream, long after the app set up the modes,
+    /// colours and cursor position that give those bytes meaning. A browser
+    /// refresh therefore painted garbage until something forced the app to
+    /// repaint. tmux already holds the authoritative screen, so ask it.
+    ///
+    /// Returns `Ok(None)` when the session or the tmux server is gone — the
+    /// caller falls back to the byte replay, which is all a dead session has.
+    pub fn capture_pane(&self, id: Uuid) -> Result<Option<Vec<u8>>, TmuxError> {
+        let name = session_name(id);
+        let start = format!("-{CAPTURE_HISTORY_LINES}");
+        // One invocation, so tmux runs both commands in a single pass of its
+        // command queue: the cursor position cannot belong to an older screen
+        // than the captured content. Two invocations raced — a program that
+        // switched to the alternate screen in between produced a snapshot with
+        // the new screen and the old cursor.
+        let out = self
+            .cmd()
+            .args([
+                "display-message",
+                "-p",
+                "-t",
+                &name,
+                "-F",
+                "#{cursor_x} #{cursor_y} #{pane_height} #{alternate_on} #{cursor_flag}",
+                ";",
+                "capture-pane",
+                "-p",
+                "-e",
+                // Keep trailing spaces, so a line's background colour reaches
+                // the edge instead of stopping at the last glyph and letting
+                // the client's own background show through.
+                "-N",
+                "-t",
+                &name,
+                "-S",
+                &start,
+            ])
+            .output()?;
+        let Some(stdout) = soft_output(out)? else {
+            return Ok(None);
+        };
+        let Some((head, content)) = stdout.split_once('\n') else {
+            return Ok(None);
+        };
+        let Some(geom) = parse_screen_geometry(head) else {
+            return Ok(None);
+        };
+        let mut lines: Vec<&str> = content
+            .strip_suffix('\n')
+            .unwrap_or(content)
+            .split('\n')
+            .collect();
+        // Scrollback belongs to the normal screen. Replaying it under an
+        // alternate-screen program would paste the shell's history behind it.
+        if geom.alternate {
+            lines.drain(..lines.len().saturating_sub(geom.pane_height));
+        }
+        Ok(Some(render_screen_snapshot(&lines, geom)))
+    }
+
     /// `tmux resize-window -t maestro-<id> -x <cols> -y <rows>`.
     pub fn resize_window(&self, id: Uuid, cols: u16, rows: u16) -> Result<(), TmuxError> {
         let name = session_name(id);
@@ -446,6 +510,124 @@ fn new_session_args(
     args
 }
 
+/// How much scrollback a fresh attach inherits from tmux's history, in lines.
+/// Enough to land in context after a browser refresh without making the first
+/// frame huge.
+const CAPTURE_HISTORY_LINES: usize = 300;
+
+/// Cursor position and screen geometry of a pane, from `display-message`.
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+struct ScreenGeometry {
+    cursor_x: usize,
+    cursor_y: usize,
+    pane_height: usize,
+    alternate: bool,
+    cursor_visible: bool,
+}
+
+fn parse_screen_geometry(s: &str) -> Option<ScreenGeometry> {
+    let line = s.lines().next()?;
+    let mut it = line.split_whitespace();
+    let cursor_x = it.next()?.parse().ok()?;
+    let cursor_y = it.next()?.parse().ok()?;
+    let pane_height: usize = it.next()?.parse().ok()?;
+    let alternate = it.next()? == "1";
+    // Older tmux without `cursor_flag` just means "assume visible".
+    let cursor_visible = it.next().map(|f| f == "1").unwrap_or(true);
+    if pane_height == 0 {
+        return None;
+    }
+    Some(ScreenGeometry {
+        cursor_x,
+        cursor_y,
+        pane_height,
+        alternate,
+        cursor_visible,
+    })
+}
+
+/// Turn captured pane lines into bytes a fresh terminal can replay: clear,
+/// print the lines, then put the cursor back where the pane has it.
+///
+/// The cursor is restored with *relative* moves so the snapshot renders the
+/// same whatever height the client's terminal has — it may not match the
+/// pane's yet, since the client's first `Resize` arrives after this.
+fn render_screen_snapshot(lines: &[&str], geom: ScreenGeometry) -> Vec<u8> {
+    // Screen rows are the last `pane_height` captured lines; anything before
+    // them is scrollback.
+    let screen_top = lines.len().saturating_sub(geom.pane_height);
+    let cursor_row = (screen_top + geom.cursor_y).min(lines.len().saturating_sub(1));
+    // Trailing blank rows would scroll the real content off a short client
+    // terminal, so drop them — but never the cursor's own row or anything
+    // above it, which is where a program that draws below its cursor lives.
+    let last = lines
+        .iter()
+        .rposition(|l| !strip_sgr(l).trim().is_empty())
+        .unwrap_or(0)
+        .max(cursor_row);
+
+    let mut out = Vec::new();
+    // Put the client on the same buffer the pane is using, so a program's
+    // repaints stay out of the client's scrollback while it runs.
+    if geom.alternate {
+        out.extend_from_slice(b"\x1b[?1049h");
+    }
+    out.extend_from_slice(b"\x1b[H\x1b[2J\x1b[m");
+    for (i, line) in lines[..=last].iter().enumerate() {
+        if i > 0 {
+            out.extend_from_slice(b"\r\n");
+        }
+        out.extend_from_slice(line.as_bytes());
+    }
+    out.extend_from_slice(b"\x1b[m\r");
+    if last > cursor_row {
+        out.extend_from_slice(format!("\x1b[{}A", last - cursor_row).as_bytes());
+    }
+    if geom.cursor_x > 0 {
+        out.extend_from_slice(format!("\x1b[{}C", geom.cursor_x).as_bytes());
+    }
+    if !geom.cursor_visible {
+        out.extend_from_slice(b"\x1b[?25l");
+    }
+    out
+}
+
+/// Text of a line with SGR escapes removed, for emptiness checks.
+fn strip_sgr(line: &str) -> String {
+    let mut out = String::with_capacity(line.len());
+    let mut chars = line.chars();
+    while let Some(c) = chars.next() {
+        if c != '\x1b' {
+            out.push(c);
+            continue;
+        }
+        // Skip CSI ... final-byte. Anything else escape-ish is not produced by
+        // `capture-pane -e`, so treating it the same way is harmless.
+        for c in chars.by_ref() {
+            if c.is_ascii_alphabetic() {
+                break;
+            }
+        }
+    }
+    out
+}
+
+/// `Ok(None)` when tmux failed because the server or the target is gone,
+/// `Ok(Some(stdout))` on success, `Err` for anything else.
+fn soft_output(out: Output) -> Result<Option<String>, TmuxError> {
+    if out.status.success() {
+        return Ok(Some(String::from_utf8_lossy(&out.stdout).into_owned()));
+    }
+    let stderr = String::from_utf8_lossy(&out.stderr);
+    if is_no_server_error(&stderr) || stderr.contains("can't find") {
+        return Ok(None);
+    }
+    Err(TmuxError::NonZero {
+        code: out.status.code().unwrap_or(-1),
+        stderr: stderr.into_owned(),
+    })
+}
+
 fn check(out: Output) -> Result<(), TmuxError> {
     if out.status.success() {
         Ok(())
@@ -461,6 +643,85 @@ fn check(out: Output) -> Result<(), TmuxError> {
 mod tests {
     use super::*;
 
+    #[test]
+    fn parse_screen_geometry_reads_cursor_and_alternate_flag() {
+        let g = parse_screen_geometry("12 3 40 1 0\n").unwrap();
+        assert_eq!(
+            g,
+            ScreenGeometry {
+                cursor_x: 12,
+                cursor_y: 3,
+                pane_height: 40,
+                alternate: true,
+                cursor_visible: false,
+            }
+        );
+        let g = parse_screen_geometry("0 0 24 0").unwrap();
+        assert!(!g.alternate);
+        assert!(g.cursor_visible, "missing cursor_flag means visible");
+        assert!(parse_screen_geometry("0 0 0 0").is_none());
+        assert!(parse_screen_geometry("garbage").is_none());
+    }
+
+    #[test]
+    fn snapshot_clears_prints_lines_and_parks_the_cursor() {
+        // Two scrollback lines, then a 3-row screen with the cursor on row 1.
+        let lines = ["old", "older", "prompt> hi", "", ""];
+        let out = render_screen_snapshot(
+            &lines,
+            ScreenGeometry {
+                cursor_x: 10,
+                cursor_y: 0,
+                pane_height: 3,
+                alternate: false,
+                cursor_visible: true,
+            },
+        );
+        let out = String::from_utf8(out).unwrap();
+        assert_eq!(
+            out,
+            "\x1b[H\x1b[2J\x1b[mold\r\nolder\r\nprompt> hi\x1b[m\r\x1b[10C"
+        );
+    }
+
+    #[test]
+    fn snapshot_keeps_rows_below_the_cursor_and_moves_back_up() {
+        let lines = ["prompt> hi", "hint line", ""];
+        let out = render_screen_snapshot(
+            &lines,
+            ScreenGeometry {
+                cursor_x: 0,
+                cursor_y: 0,
+                pane_height: 3,
+                alternate: true,
+                cursor_visible: false,
+            },
+        );
+        let out = String::from_utf8(out).unwrap();
+        assert!(out.starts_with("\x1b[?1049h\x1b[H\x1b[2J"), "{out:?}");
+        assert!(
+            out.ends_with("prompt> hi\r\nhint line\x1b[m\r\x1b[1A\x1b[?25l"),
+            "{out:?}"
+        );
+    }
+
+    #[test]
+    fn snapshot_ignores_colour_only_rows_when_trimming() {
+        let lines = ["text", "\x1b[38;5;4m\x1b[m", ""];
+        let out = String::from_utf8(render_screen_snapshot(
+            &lines,
+            ScreenGeometry {
+                cursor_x: 4,
+                cursor_y: 0,
+                pane_height: 3,
+                alternate: false,
+                cursor_visible: true,
+            },
+        ))
+        .unwrap();
+        assert_eq!(out, "\x1b[H\x1b[2J\x1b[mtext\x1b[m\r\x1b[4C");
+    }
+
     /// Every phrasing must be recognised: an unmatched one makes
     /// `list_sessions` fail, and a failed pass reaps nothing.
     #[test]

crates/maestro-daemon/src/worktree_probe.rs

@@ -1,14 +1,8 @@
-//! The one git probe for what a worktree holds that would be lost, plus
-//! guard-free reclaim (#79, #80).
+//! The one git probe for what a worktree holds that would be lost (#79).
 //!
-//! [`in_the_way`] is the single predicate behind all three paths: the kill
-//! refusal (#79), the natural-exit classifier (#82), and the retained list
-//! (#80). It lives here rather than in `worktree.rs` so that none of them can
-//! answer the question differently — #79 and #80 were built in parallel and
-//! each grew its own copy, which is exactly the drift this collapses.
-//!
-//! Reclaim never consults it: reclaim is the deliberate destructive act, and
-//! the operator chose it while looking at what was in the way.
+//! [`in_the_way`] is the single predicate behind both paths: the kill refusal
+//! (#79) and the natural-exit classifier (#82). It lives here rather than in
+//! `worktree.rs` so that neither can answer the question differently.
 
 use std::path::Path;
 
@@ -27,7 +21,7 @@ pub enum ProbeError {
 /// Ignored files never count (`--ignored=no`) — every Rust worktree has a
 /// `target/`, and counting those would put every tree permanently in the way.
 /// A directory that no longer exists reports nothing in the way: there is
-/// nothing left to lose, and the store's own pruning drops the entry.
+/// nothing left to lose.
 pub fn in_the_way(path: &Path) -> Result<InTheWay, ProbeError> {
     let mut breakdown = InTheWay::default();
     // `symlink_metadata` rather than `exists`: a broken symlink is gone for
@@ -51,6 +45,25 @@ pub fn in_the_way(path: &Path) -> Result<InTheWay, ProbeError> {
             breakdown.uncommitted_files += 1;
         }
     }
+    // The line delta behind the sidebar's `+N −M` (docs/SIDEBAR.md). Diffing
+    // the working tree against HEAD covers staged and unstaged changes in one
+    // pass, so a `git add` never makes the counter drop.
+    let numstat = run_git(path, ["diff", "--numstat", "HEAD"]);
+    if let Ok(numstat) = numstat {
+        for line in String::from_utf8_lossy(&numstat.stdout).lines() {
+            let mut cols = line.split('\t');
+            // A binary file reports `-` for both columns; it has no lines to
+            // add to either total.
+            let added: usize = cols.next().and_then(|c| c.parse().ok()).unwrap_or(0);
+            let removed: usize = cols.next().and_then(|c| c.parse().ok()).unwrap_or(0);
+            breakdown.insertions += added;
+            breakdown.deletions += removed;
+        }
+    }
+    // A repo with no commits yet has no HEAD to diff against, and the counts
+    // stay zero rather than failing the whole probe — the file counts above
+    // already reported the work.
+
     // Resolved inside the worktree so a detached HEAD still has a starting
     // point; asking the base repo for `refs/heads/<branch>` fails there.
     let head = run_git(path, ["rev-parse", "--abbrev-ref", "HEAD"])?;
@@ -67,37 +80,6 @@ pub fn in_the_way(path: &Path) -> Result<InTheWay, ProbeError> {
     Ok(breakdown)
 }
 
-/// Remove a retained worktree's directory without consulting any guard, per
-/// the reclaim contract: the operator chose this while looking at what was in
-/// the way. `git worktree remove` keeps `refs/heads/<branch>`, so commits on
-/// the branch survive even when they were never pushed.
-///
-/// A directory that is already gone only leaves a stale registration behind;
-/// `worktree prune` clears it so the branch can be spawned again.
-pub fn reclaim(base_repo: &Path, path: &Path) -> Result<(), ProbeError> {
-    if !path.exists() {
-        return run_git(base_repo, ["worktree", "prune"]).map(|_| ());
-    }
-    run_git(
-        base_repo,
-        [
-            "worktree",
-            "remove",
-            "--force",
-            path.to_str().expect("worktree path is valid utf-8"),
-        ],
-    )
-    .map(|_| ())
-}
-
-/// Branch checked out at `path`. Sessions adopted from a sidecar carry no
-/// branch on record, so a forced kill (#81) asks the tree itself when it
-/// needs one for the retained store. `None` when git cannot answer.
-pub fn current_branch(path: &Path) -> Option<String> {
-    let out = run_git(path, ["rev-parse", "--abbrev-ref", "HEAD"]).ok()?;
-    Some(String::from_utf8_lossy(&out.stdout).trim().to_string())
-}
-
 fn run_git<I, S>(dir: &Path, args: I) -> Result<std::process::Output, ProbeError>
 where
     I: IntoIterator<Item = S>,
@@ -247,6 +229,67 @@ mod tests {
         assert_eq!(dirty.untracked_files, 1);
     }
 
+    /// docs/SIDEBAR.md: the sidebar renders `+N −M`, so the probe carries the
+    /// line delta of the tracked changes — staged and unstaged together,
+    /// against HEAD.
+    #[test]
+    fn staged_and_unstaged_lines_both_count() {
+        let r = repo_with_worktree();
+        std::fs::write(r.wt().join("staged.txt"), b"a\nb\n").unwrap();
+        git(&r.wt(), &["add", "staged.txt"]);
+        // README loses its one line and gains three, unstaged.
+        std::fs::write(r.wt().join("README"), b"x\ny\nz\n").unwrap();
+
+        let breakdown = in_the_way(&r.wt()).expect("probe");
+        assert_eq!(breakdown.insertions, 5, "2 staged + 3 unstaged");
+        assert_eq!(breakdown.deletions, 1);
+    }
+
+    /// Untracked files are counted as files and never as lines — reading each
+    /// one off disk is the cost the sidebar contract declines to pay.
+    #[test]
+    fn untracked_files_contribute_no_lines() {
+        let r = repo_with_worktree();
+        std::fs::write(r.wt().join("scratch.txt"), b"a\nb\nc\n").unwrap();
+        let breakdown = in_the_way(&r.wt()).expect("probe");
+        assert_eq!(breakdown.untracked_files, 1);
+        assert_eq!(breakdown.insertions, 0);
+        assert_eq!(breakdown.deletions, 0);
+    }
+
+    /// A binary file's numstat is `-\t-`. It must not poison the totals, and
+    /// it must not abort the probe either.
+    #[test]
+    fn binary_changes_add_no_lines() {
+        let r = repo_with_worktree();
+        std::fs::write(r.wt().join("blob.bin"), [0u8, 159, 0, 12]).unwrap();
+        git(&r.wt(), &["add", "blob.bin"]);
+        git(&r.wt(), &["commit", "-qm", "blob"]);
+        std::fs::write(r.wt().join("blob.bin"), [0u8, 1, 2, 3, 4]).unwrap();
+
+        let breakdown = in_the_way(&r.wt()).expect("probe");
+        assert_eq!(breakdown.uncommitted_files, 1);
+        assert_eq!(breakdown.insertions, 0);
+        assert_eq!(breakdown.deletions, 0);
+    }
+
+    /// A line delta alone never decides `is_clear` — the file counts do — so a
+    /// mode change with no line delta still blocks a kill.
+    #[test]
+    fn a_line_less_change_still_blocks() {
+        let r = repo_with_worktree();
+        let script = r.wt().join("run.sh");
+        std::fs::write(&script, b"echo hi\n").unwrap();
+        git(&r.wt(), &["add", "run.sh"]);
+        git(&r.wt(), &["commit", "-qm", "script"]);
+        git(&r.wt(), &["update-index", "--chmod=+x", "run.sh"]);
+
+        let breakdown = in_the_way(&r.wt()).expect("probe");
+        assert_eq!(breakdown.insertions, 0);
+        assert_eq!(breakdown.deletions, 0);
+        assert!(!breakdown.is_clear());
+    }
+
     #[test]
     fn ignored_files_never_count() {
         let r = repo_with_worktree();
@@ -272,54 +315,4 @@ mod tests {
         let breakdown = in_the_way(&gone).expect("probe");
         assert!(breakdown.is_clear());
     }
-
-    #[test]
-    fn reclaim_removes_the_directory_and_prunes_a_gone_one() {
-        let r = repo_with_worktree();
-        r.commit_in_wt("precious");
-        reclaim(&r.repo, &r.wt()).expect("reclaim");
-        assert!(!r.wt().exists());
-
-        // A second reclaim over the now-missing path just prunes.
-        reclaim(&r.repo, &r.wt()).expect("reclaim again");
-    }
-
-    #[test]
-    fn reclaim_keeps_the_branch_and_its_commits() {
-        let r = repo_with_worktree();
-        r.commit_in_wt("precious");
-        let sha = {
-            let out = Command::new("git")
-                .arg("-C")
-                .arg(r.wt())
-                .args(["rev-parse", "HEAD"])
-                .output()
-                .unwrap();
-            String::from_utf8_lossy(&out.stdout).trim().to_string()
-        };
-
-        reclaim(&r.repo, &r.wt()).expect("reclaim");
-
-        let out = Command::new("git")
-            .arg("-C")
-            .arg(&r.repo)
-            .args(["rev-parse", "--verify", "refs/heads/feature"])
-            .output()
-            .unwrap();
-        assert!(
-            out.status.success(),
-            "branch must survive reclaim: {}",
-            String::from_utf8_lossy(&out.stderr)
-        );
-        assert_eq!(String::from_utf8_lossy(&out.stdout).trim(), sha);
-    }
-
-    #[test]
-    fn reclaim_is_guard_free_on_a_dirty_tree() {
-        let r = repo_with_worktree();
-        std::fs::write(r.wt().join("README"), b"unsaved\n").unwrap();
-        std::fs::write(r.wt().join("notes.txt"), b"unsaved\n").unwrap();
-        reclaim(&r.repo, &r.wt()).expect("reclaim ignores the guard");
-        assert!(!r.wt().exists());
-    }
 }

crates/maestro-daemon/tests/attach.rs

@@ -38,7 +38,6 @@ async fn spawn_daemon_ws() -> Harness {
         "/home/test",
         Vec::new(),
         std::sync::Arc::new(maestro_daemon::projects::ProjectScanner::disabled()),
-        std::sync::Arc::new(fx.retained_store()),
     ));
     let listener = TcpListener::bind(SocketAddr::from((Ipv4Addr::LOCALHOST, 0)))
         .await

crates/maestro-daemon/tests/branches.rs

@@ -27,7 +27,6 @@ async fn spawn() -> Harness {
         "/home/test",
         presets,
         Arc::new(maestro_daemon::projects::ProjectScanner::disabled()),
-        std::sync::Arc::new(fx.retained_store()),
     ));
     let listener = TcpListener::bind(SocketAddr::from((Ipv4Addr::LOCALHOST, 0)))
         .await

crates/maestro-daemon/tests/common/mod.rs

@@ -176,11 +176,6 @@ impl DaemonFixture {
             manager =
                 manager.with_spawn_history(SpawnHistory::new(Self::history_path(&state_dir), days));
         }
-        // Same wiring as production: the manager records retained worktrees
-        // into the same dir the API reads back from (#78/#81).
-        let retained =
-            maestro_daemon::retained_store::RetainedStore::new(Self::retained_path(&state_dir));
-        manager = manager.with_retained_store(retained.clone());
         Self {
             tmux,
             manager,
@@ -192,10 +187,6 @@ impl DaemonFixture {
         state_dir.path().join("spawn-history.jsonl")
     }
 
-    fn retained_path(state_dir: &TempDir) -> PathBuf {
-        state_dir.path().join("retained-worktrees")
-    }
-
     pub fn spawn_history_path(&self) -> PathBuf {
         Self::history_path(&self.state_dir)
     }
@@ -204,15 +195,6 @@ impl DaemonFixture {
         self.state_dir.path().join("sessions")
     }
 
-    /// Retained-worktree store rooted at the fixture's own state dir (#78).
-    pub fn retained_store(&self) -> maestro_daemon::retained_store::RetainedStore {
-        maestro_daemon::retained_store::RetainedStore::new(self.retained_dir())
-    }
-
-    pub fn retained_dir(&self) -> PathBuf {
-        self.state_dir.path().join("retained-worktrees")
-    }
-
     pub fn fifo_dir(&self) -> PathBuf {
         self.state_dir.path().join("fifos")
     }

crates/maestro-daemon/tests/local_api.rs

@@ -25,13 +25,6 @@ struct Harness {
     _fx: DaemonFixture,
 }
 
-impl Harness {
-    /// Store over the same dir the manager records into.
-    fn fx_retained(&self) -> maestro_daemon::retained_store::RetainedStore {
-        self._fx.retained_store()
-    }
-}
-
 async fn spawn(hostname: &str) -> Harness {
     spawn_with(
         hostname,
@@ -55,7 +48,6 @@ async fn spawn_with(
         "/home/test",
         presets,
         projects,
-        std::sync::Arc::new(fx.retained_store()),
     ));
     let listener = TcpListener::bind(SocketAddr::from((Ipv4Addr::LOCALHOST, 0)))
         .await
@@ -537,11 +529,10 @@ async fn delete_a_session_whose_worktree_holds_unsaved_work_returns_409_with_the
     assert!(h.manager.has(session.id).await, "the session keeps running");
 }
 
-/// `?force=true` on the wire (#81): the refusal is overridden, the session
-/// ends, and the answer is 200 with the retained worktree named — path,
-/// branch, reason — while the directory survives on disk.
+/// `?force=true` on the wire (#81): the refusal is overridden and the session
+/// ends, while the guarded worktree survives on disk.
 #[tokio::test]
-async fn delete_with_force_ends_the_session_and_answers_what_was_retained() {
+async fn delete_with_force_ends_the_session_and_keeps_the_worktree() {
     let h = spawn("naps-laptop").await;
     let repo = tempfile::TempDir::new().unwrap();
     common::init_repo(repo.path());
@@ -574,22 +565,11 @@ async fn delete_with_force_ends_the_session_and_answers_what_was_retained() {
         .send()
         .await
         .unwrap();
-    assert_eq!(resp.status(), reqwest::StatusCode::OK);
-    let body: maestro_protocol::KillOutcome = resp.json().await.unwrap();
-    let retained = body.retained.expect("force names what was kept");
-    assert_eq!(retained.path, wt_path);
-    assert_eq!(retained.branch, "feat/guarded");
-    assert_eq!(
-        retained.reason,
-        maestro_protocol::RetainedReason::ForceKilled
-    );
+    assert_eq!(resp.status(), reqwest::StatusCode::NO_CONTENT);
 
     // The tree would have blocked; force must not have removed it.
     assert!(wt_path.exists(), "a forced kill never removes the worktree");
-    // And the store entry exists, so reclaim can find it (#80).
-    let all = h.fx_retained().load_all().unwrap();
-    assert_eq!(all.len(), 1);
-    assert_eq!(all[0].id, retained.id);
+    assert!(!h.manager.has(session.id).await, "the session is gone");
 }
 
 #[tokio::test]
@@ -746,234 +726,3 @@ async fn next_event(ws: &mut WsClient) -> maestro_protocol::CatalogEvent {
         }
     }
 }
-
-// ---------------------------------------------------------------------------
-// GET /api/worktrees + DELETE /api/worktrees/:id (#80)
-// ---------------------------------------------------------------------------
-
-/// `git -C dir args`, asserting success — the worktree endpoints are tested
-/// against real git, like everything else in this crate.
-fn git(dir: &std::path::Path, args: &[&str]) {
-    let out = std::process::Command::new("git")
-        .arg("-C")
-        .arg(dir)
-        .args(args)
-        .output()
-        .expect("git spawn");
-    assert!(
-        out.status.success(),
-        "git {args:?} failed: {}",
-        String::from_utf8_lossy(&out.stderr)
-    );
-}
-
-struct RetainedFixture {
-    repo: tempfile::TempDir,
-    wt: std::path::PathBuf,
-}
-
-/// A real repo with a linked worktree for branch `feat/login` whose tree holds
-/// an uncommitted tracked edit and an unpushed commit.
-fn retained_fixture() -> RetainedFixture {
-    let repo = tempfile::tempdir().unwrap();
-    common::init_repo(repo.path());
-    // A pushed main, so "unpushed" means commits made on the feature branch,
-    // not the shared history.
-    let remote = repo.path().join("remote.git");
-    std::fs::create_dir_all(&remote).unwrap();
-    let remote_s = remote.to_string_lossy().into_owned();
-    git(&remote, &["init", "-q", "--bare"]);
-    git(repo.path(), &["remote", "add", "origin", &remote_s]);
-    git(repo.path(), &["push", "-q", "-u", "origin", "main"]);
-    git(
-        repo.path(),
-        &[
-            "worktree",
-            "add",
-            "-q",
-            "-b",
-            "feat/login",
-            "worktrees/feat/login",
-        ],
-    );
-    let wt = repo.path().join("worktrees").join("feat").join("login");
-    RetainedFixture { repo, wt }
-}
-
-impl RetainedFixture {
-    fn commit_in_wt(&self, msg: &str) {
-        std::fs::write(self.wt.join("README"), msg.as_bytes()).unwrap();
-        git(&self.wt, &["add", "-A"]);
-        git(&self.wt, &["commit", "-qm", msg]);
-    }
-
-    fn branch_sha(&self, branch: &str) -> String {
-        let out = std::process::Command::new("git")
-            .arg("-C")
-            .arg(self.repo.path())
-            .args(["rev-parse", "--verify", &format!("refs/heads/{branch}")])
-            .output()
-            .unwrap();
-        assert!(out.status.success(), "branch {branch} must exist");
-        String::from_utf8_lossy(&out.stdout).trim().to_string()
-    }
-}
-
-#[tokio::test]
-async fn worktrees_lists_retained_entries_with_recomputed_breakdown() {
-    let h = spawn("naps-laptop").await;
-    let fx = retained_fixture();
-    // Uncommitted tracked edit + an untracked file; the branch has no commits
-    // beyond main yet.
-    std::fs::write(fx.wt.join("README"), b"unsaved\n").unwrap();
-    std::fs::write(fx.wt.join("scratch.txt"), b"loose\n").unwrap();
-
-    let store = h._fx.retained_store();
-    let entry = store
-        .retain(
-            &fx.wt,
-            fx.repo.path(),
-            "feat/login",
-            maestro_daemon::retained_store::RetainedReason::ForceKilled,
-        )
-        .unwrap();
-
-    let listed: Vec<maestro_protocol::RetainedWorktree> =
-        reqwest::get(url(h.addr, "/api/worktrees"))
-            .await
-            .unwrap()
-            .json()
-            .await
-            .unwrap();
-    assert_eq!(listed.len(), 1);
-    let w = &listed[0];
-    assert_eq!(w.id, entry.id);
-    assert_eq!(w.path, fx.wt);
-    assert_eq!(w.base_repo, fx.repo.path());
-    assert_eq!(w.branch, "feat/login");
-    assert_eq!(w.reason, maestro_protocol::RetainedReason::ForceKilled);
-    assert_eq!(w.in_the_way.uncommitted_files, 1);
-    assert_eq!(w.in_the_way.untracked_files, 1);
-    assert_eq!(w.in_the_way.unpushed_commits, 0);
-
-    // The operator saves the work themselves. The stored entry is untouched —
-    // the next read must report the tree as it is *now*, not as it was when
-    // it was retained.
-    fx.commit_in_wt("saved");
-    let listed: Vec<maestro_protocol::RetainedWorktree> =
-        reqwest::get(url(h.addr, "/api/worktrees"))
-            .await
-            .unwrap()
-            .json()
-            .await
-            .unwrap();
-    assert_eq!(listed.len(), 1, "same entry, still retained");
-    assert_eq!(listed[0].id, entry.id);
-    assert_eq!(listed[0].in_the_way.uncommitted_files, 0);
-    assert_eq!(listed[0].in_the_way.unpushed_commits, 1);
-}
-
-#[tokio::test]
-async fn reclaim_removes_directory_and_entry_but_never_the_branch() {
-    let h = spawn("naps-laptop").await;
-    let fx = retained_fixture();
-    fx.commit_in_wt("precious");
-    let sha = fx.branch_sha("feat/login");
-    std::fs::write(fx.wt.join("README"), b"unsaved\n").unwrap(); // guard would refuse
-
-    let store = h._fx.retained_store();
-    let entry = store
-        .retain(
-            &fx.wt,
-            fx.repo.path(),
-            "feat/login",
-            maestro_daemon::retained_store::RetainedReason::ExitedWithUnsavedWork,
-        )
-        .unwrap();
-
-    let resp = reqwest::Client::new()
-        .delete(url(h.addr, &format!("/api/worktrees/{}", entry.id)))
-        .send()
-        .await
-        .unwrap();
-    assert_eq!(resp.status(), reqwest::StatusCode::NO_CONTENT);
-
-    assert!(!fx.wt.exists(), "reclaim removes the directory");
-    let listed: Vec<maestro_protocol::RetainedWorktree> =
-        reqwest::get(url(h.addr, "/api/worktrees"))
-            .await
-            .unwrap()
-            .json()
-            .await
-            .unwrap();
-    assert!(listed.is_empty(), "reclaim drops the store entry");
-    assert_eq!(
-        fx.branch_sha("feat/login"),
-        sha,
-        "the branch and its commits survive"
-    );
-}
-
-#[tokio::test]
-async fn reclaim_of_an_already_gone_directory_still_drops_the_entry() {
-    let h = spawn("naps-laptop").await;
-    let fx = retained_fixture();
-    let store = h._fx.retained_store();
-    let entry = store
-        .retain(
-            &fx.wt,
-            fx.repo.path(),
-            "feat/login",
-            maestro_daemon::retained_store::RetainedReason::ForceKilled,
-        )
-        .unwrap();
-    std::fs::remove_dir_all(&fx.wt).unwrap(); // operator rm -rf
-
-    let resp = reqwest::Client::new()
-        .delete(url(h.addr, &format!("/api/worktrees/{}", entry.id)))
-        .send()
-        .await
-        .unwrap();
-    assert_eq!(resp.status(), reqwest::StatusCode::NO_CONTENT);
-    assert!(store.load_all().unwrap().is_empty());
-}
-
-#[tokio::test]
-async fn reclaim_unknown_id_returns_404() {
-    let h = spawn("naps-laptop").await;
-    let resp = reqwest::Client::new()
-        .delete(url(h.addr, &format!("/api/worktrees/{}", Uuid::new_v4())))
-        .send()
-        .await
-        .unwrap();
-    assert_eq!(resp.status(), reqwest::StatusCode::NOT_FOUND);
-}
-
-#[tokio::test]
-async fn worktrees_prunes_entries_whose_directory_vanished_by_hand() {
-    let h = spawn("naps-laptop").await;
-    let fx = retained_fixture();
-    let store = h._fx.retained_store();
-    store
-        .retain(
-            &fx.wt,
-            fx.repo.path(),
-            "feat/login",
-            maestro_daemon::retained_store::RetainedReason::ForceKilled,
-        )
-        .unwrap();
-    std::fs::remove_dir_all(&fx.wt).unwrap();
-
-    let listed: Vec<maestro_protocol::RetainedWorktree> =
-        reqwest::get(url(h.addr, "/api/worktrees"))
-            .await
-            .unwrap()
-            .json()
-            .await
-            .unwrap();
-    assert!(listed.is_empty(), "a hand-removed directory is not listed");
-    assert!(
-        store.load_all().unwrap().is_empty(),
-        "and its entry is gone"
-    );
-}

crates/maestro-daemon/tests/session_manager.rs

@@ -8,9 +8,7 @@ use std::process::Command;
 use std::sync::Arc;
 use std::time::Duration;
 
-use common::{
-    init_bare_repo, init_repo, push_all_to_origin, wait_for_async, DaemonFixture, TmuxFixture,
-};
+use common::{init_bare_repo, init_repo, push_all_to_origin, wait_for_async, DaemonFixture};
 use maestro_daemon::SessionError;
 use maestro_protocol::CreateSessionRequest;
 use maestro_protocol::{CatalogEvent, SessionStatus, WorktreeSpec};
@@ -190,6 +188,64 @@ async fn create_captures_initial_output_in_replay_buffer() {
     fx.manager.kill(s.id, false).await.unwrap();
 }
 
+/// A fresh attach must land on the pane's *current* screen. Replaying the raw
+/// byte tail instead reproduced a browser refresh painting garbage over an
+/// agent TUI: the tail starts mid-stream, so the clears, modes and cursor
+/// moves that gave those bytes meaning are long gone.
+#[tokio::test]
+async fn attach_replay_is_the_current_screen_not_the_byte_tail() {
+    let fx = DaemonFixture::new();
+    let cwd = TempDir::new().unwrap();
+
+    let s = fx
+        .manager
+        .create(CreateSessionRequest {
+            cwd: cwd.path().to_path_buf(),
+            // Print a line, then switch to the alternate screen the way a
+            // full-screen program does on startup. Only the second marker is
+            // on the pane; the first belongs to the screen behind it.
+            command: "sh -c 'printf STALE_MARKER_LINE\\\\n; sleep 0.5; printf \"\\\\033[?1049h\\\\033[2J\\\\033[HLIVE_MARKER\"; exec sleep 30'"
+                .into(),
+            env: HashMap::new(),
+            metadata: HashMap::new(),
+            worktree: None,
+        })
+        .await
+        .expect("create");
+
+    let want = b"LIVE_MARKER";
+    let deadline = std::time::Instant::now() + Duration::from_secs(5);
+    let mut replay = Vec::new();
+    while std::time::Instant::now() < deadline {
+        let (bytes, _rx) = fx
+            .manager
+            .subscribe_pty_with_replay(s.id)
+            .await
+            .expect("subscribe");
+        replay = bytes;
+        if replay.windows(want.len()).any(|w| w == want) {
+            break;
+        }
+        tokio::time::sleep(Duration::from_millis(50)).await;
+    }
+
+    let text = String::from_utf8_lossy(&replay).into_owned();
+    assert!(
+        replay.starts_with(b"\x1b[?1049h\x1b[H\x1b[2J"),
+        "replay did not start with an alternate-screen snapshot: {text:?}"
+    );
+    assert!(
+        text.contains("LIVE_MARKER"),
+        "replay lost the live screen: {text:?}"
+    );
+    assert!(
+        !text.contains("STALE_MARKER_LINE"),
+        "replay carried bytes from behind the alternate screen: {text:?}"
+    );
+
+    fx.manager.kill(s.id, false).await.unwrap();
+}
+
 /// Regression for NAP-228: after `recover`, PTY output must reach new
 /// subscribers. Before the fix tmux's stale `pipe-pane` (with `-o`) silently
 /// kept feeding the old FIFO and the revived manager's subscribers saw
@@ -609,11 +665,9 @@ async fn killing_refuses_until_the_branch_is_pushed() {
 }
 
 /// `--force` overrides the refusal and nothing else (#81): the session ends,
-/// but the guarded worktree is RETAINED — never removed — and enters the
-/// retained store with reason `force-killed`. The outcome names what was
-/// kept, so a client can say where the work went.
+/// but the guarded worktree is left on disk — never removed.
 #[tokio::test]
-async fn force_killing_a_guarded_worktree_retains_it_in_the_store() {
+async fn force_killing_a_guarded_worktree_leaves_it_on_disk() {
     let fx = DaemonFixture::new();
     let repo = TempDir::new().unwrap();
     init_repo(repo.path());
@@ -639,8 +693,7 @@ async fn force_killing_a_guarded_worktree_retains_it_in_the_store() {
     // Without force the same kill is refused (#79).
     assert!(fx.manager.kill(s.id, false).await.is_err());
 
-    let outcome = fx
-        .manager
+    fx.manager
         .kill(s.id, true)
         .await
         .expect("force overrides the refusal");
@@ -653,30 +706,11 @@ async fn force_killing_a_guarded_worktree_retains_it_in_the_store() {
     // ...but the session itself still ends.
     assert!(!fx.manager.has(s.id).await);
     assert!(!fx.sidecar_dir().join(format!("{}.json", s.id)).exists());
-
-    let retained = outcome.retained.expect("the outcome names what was kept");
-    assert_eq!(retained.path, wt_path);
-    assert_eq!(retained.branch, "feat/forced");
-    assert_eq!(
-        retained.reason,
-        maestro_protocol::RetainedReason::ForceKilled
-    );
-    assert!(
-        retained.in_the_way.uncommitted_files >= 1,
-        "the outcome carries what blocked the unforced kill"
-    );
-
-    // The store holds the same entry, so reclaim can find it later (#80).
-    let all = fx.retained_store().load_all().expect("load_all");
-    assert_eq!(all.len(), 1);
-    assert_eq!(all[0].id, retained.id);
-    assert_eq!(all[0].path, wt_path);
-    assert_eq!(all[0].branch, "feat/forced");
 }
 
 /// Force overrides the *refusal*, not ordinary cleanup (#81): a tree whose
 /// guard is clear never needed the override, so a forced kill removes it as
-/// usual and retains nothing.
+/// usual.
 #[tokio::test]
 async fn force_on_a_clear_worktree_removes_it_as_usual() {
     let fx = DaemonFixture::new();
@@ -704,10 +738,8 @@ async fn force_on_a_clear_worktree_removes_it_as_usual() {
     init_bare_repo(origin.path());
     push_all_to_origin(repo.path(), origin.path());
 
-    let outcome = fx.manager.kill(s.id, true).await.expect("kill");
+    fx.manager.kill(s.id, true).await.expect("kill");
     assert!(!wt_path.exists(), "a clear tree needs no override");
-    assert_eq!(outcome.retained, None);
-    assert!(fx.retained_store().load_all().unwrap().is_empty());
 }
 
 /// Ignored files NEVER block (#79): build output like `target/` exists in
@@ -757,12 +789,11 @@ async fn killing_an_ignored_only_worktree_succeeds() {
 
 /// Natural exit (#82): a session that ends on its own never reaches the kill
 /// path, so the watcher applies the kill's rule itself. A worktree holding
-/// unsaved work is RETAINED into the store with reason
-/// `exited-with-unsaved-work`; the session stays in the catalog marked dead
-/// until dismissed — and dismissing it afterwards neither refuses nor
-/// retains a second time.
+/// unsaved work is LEFT ON DISK and stays attached to the dead record; the
+/// session stays in the catalog marked dead until dismissed, and dismissing
+/// it goes through `--force`, which leaves the tree alone too.
 #[tokio::test]
-async fn natural_exit_of_a_guarded_worktree_retains_it_in_the_store() {
+async fn natural_exit_of_a_guarded_worktree_leaves_it_on_disk() {
     let fx = DaemonFixture::new();
     let repo = TempDir::new().unwrap();
     init_repo(repo.path());
@@ -793,56 +824,35 @@ async fn natural_exit_of_a_guarded_worktree_retains_it_in_the_store() {
     })
     .await;
 
-    // The tree would have blocked a kill, so the exit kept it...
+    // The settle step has been applied or refused either way — here refused,
+    // since the tree holds unsaved work. Only from this point is "the tree
+    // survived" a settled fact rather than a race (#115).
+    wait_for_async(Duration::from_secs(3), || async {
+        fx.manager.is_settled(s.id).await == Some(true)
+    })
+    .await;
+
+    // The tree would have blocked a kill, so the exit kept it.
     assert!(
         wt_path.exists(),
-        "a dirty worktree must be retained on natural exit"
+        "a dirty worktree must be left alone on natural exit"
     );
-    // ...in the store, under the exit reason.
-    // (`Dead` is observable a moment before the settle step's store write lands.)
-    wait_for_async(Duration::from_secs(3), || async {
-        fx.retained_store()
-            .load_all()
-            .map(|all| all.len() == 1)
-            .unwrap_or(false)
-    })
-    .await;
-    let all = fx.retained_store().load_all().expect("load_all");
-    assert_eq!(all.len(), 1);
-    assert_eq!(all[0].path, wt_path);
-    assert_eq!(all[0].branch, "feat/exited");
-    assert_eq!(
-        all[0].reason,
-        maestro_protocol::RetainedReason::ExitedWithUnsavedWork
-    );
-
-    // The session itself stays until dismissed; its worktree reference is
-    // settled, so dismissal is neither refused nor retained twice.
-    // (`Dead` is observable a moment before the settle step finishes.)
-    wait_for_async(Duration::from_secs(3), || async {
-        let s = &fx.manager.snapshot().await[0];
-        s.worktree.is_none() && s.blocked_work.is_none()
-    })
-    .await;
     let snap = fx.manager.snapshot().await;
     assert_eq!(snap.len(), 1);
-    assert_eq!(snap[0].worktree, None);
-
-    fx.manager.kill(s.id, false).await.expect("dismiss");
-    assert!(!fx.manager.has(s.id).await);
     assert!(
-        wt_path.exists(),
-        "dismissal must not remove what was retained"
-    );
-    assert_eq!(
-        fx.retained_store().load_all().unwrap().len(),
-        1,
-        "dismissal must not record a second entry for the same tree"
+        snap[0].worktree.is_some(),
+        "the tree stays attached to the dead record"
     );
+
+    // Dismissing it needs --force, and force never removes what would block.
+    assert!(fx.manager.kill(s.id, false).await.is_err());
+    fx.manager.kill(s.id, true).await.expect("forced dismissal");
+    assert!(!fx.manager.has(s.id).await);
+    assert!(wt_path.exists(), "dismissal must not remove the work");
 }
 
 /// Natural exit (#82), pristine side of the same rule: nothing in the tree,
-/// nothing to keep — the worktree is removed and no store entry appears.
+/// nothing to keep — the worktree is removed.
 #[tokio::test]
 async fn natural_exit_of_a_pristine_worktree_removes_it() {
     let fx = DaemonFixture::new();
@@ -879,92 +889,18 @@ async fn natural_exit_of_a_pristine_worktree_removes_it() {
     .await;
 
     // (`Dead` is observable a moment before the settle step removes the tree.)
-    wait_for_async(Duration::from_secs(3), || async {
-        !wt_path.exists()
-            && fx
-                .retained_store()
-                .load_all()
-                .map(|all| all.is_empty())
-                .unwrap_or(false)
-    })
-    .await;
+    wait_for_async(Duration::from_secs(3), || async { !wt_path.exists() }).await;
 
     assert!(
         !wt_path.exists(),
         "a pristine worktree leaves nothing behind"
     );
-    assert!(
-        fx.retained_store().load_all().unwrap().is_empty(),
-        "nothing was retained"
-    );
     assert!(
         fx.manager.has(s.id).await,
         "the session stays until dismissed"
     );
 }
 
-/// When nothing can be recorded — here: no retained store wired — the exit
-/// destroys nothing. The tree stays on disk AND stays attached to the dead
-/// record, where dismissing it later still goes through #81's forced-kill
-/// retention. Losing the store must not lose the work.
-#[tokio::test]
-async fn natural_exit_without_a_retained_store_destroys_nothing() {
-    let tmux = TmuxFixture::new();
-    let state_dir = TempDir::new().unwrap();
-    let manager = maestro_daemon::SessionManager::new(
-        Arc::new(tmux.tmux.clone()),
-        maestro_daemon::sidecar::SidecarStore::new(state_dir.path().join("sessions")),
-        maestro_daemon::worktree::WorktreeManager::new(None),
-        state_dir.path().join("fifos"),
-    );
-    // Deliberately NO with_retained_store.
-
-    let repo = TempDir::new().unwrap();
-    init_repo(repo.path());
-    let wt_path = repo.path().join("worktrees").join("feat").join("nostore");
-
-    let s = manager
-        .create(CreateSessionRequest {
-            cwd: repo.path().to_path_buf(),
-            command: "sleep 30".into(),
-            env: HashMap::new(),
-            metadata: HashMap::new(),
-            worktree: Some(WorktreeSpec {
-                base_repo: repo.path().to_path_buf(),
-                branch: "feat/nostore".into(),
-                create_branch: true,
-            }),
-        })
-        .await
-        .expect("create");
-    std::fs::write(wt_path.join("README"), b"unsaved edits\n").unwrap();
-
-    tmux.tmux.kill_session(s.id).expect("kill tmux directly");
-    manager.start_liveness_watcher(Duration::from_millis(50));
-    wait_for_async(Duration::from_secs(3), || async {
-        manager.status(s.id).await == Some(SessionStatus::Dead)
-    })
-    .await;
-
-    // The settle step has been applied or refused either way — here refused,
-    // since there is no store to record the tree into. Only from this point
-    // is "the tree survived" a settled fact rather than a race (#115).
-    wait_for_async(Duration::from_secs(3), || async {
-        manager.is_settled(s.id).await == Some(true)
-    })
-    .await;
-
-    assert!(
-        wt_path.exists(),
-        "nothing may be destroyed when it cannot be recorded"
-    );
-    let snap = manager.snapshot().await;
-    assert!(
-        snap[0].worktree.is_some(),
-        "the tree must stay attached to the record so a later forced kill can retain it"
-    );
-}
-
 fn run_git_in(dir: &std::path::Path, args: &[&str]) {
     let out = Command::new("git")
         .arg("-C")

crates/maestro-e2e/src/client.rs

@@ -12,8 +12,7 @@ use std::time::Duration;
 
 use futures_util::{SinkExt, StreamExt};
 use maestro_protocol::{
-    BranchInfo, CatalogEvent, CreateSessionRequest, DaemonClient, KillOutcome, Project,
-    RetainedWorktree, Session,
+    BranchInfo, CatalogEvent, CreateSessionRequest, DaemonClient, Project, Session,
 };
 use tokio_tungstenite::tungstenite::client::IntoClientRequest;
 use tokio_tungstenite::tungstenite::Message as WsMessage;
@@ -65,20 +64,12 @@ impl TestClient {
         Ok(self.daemon.kill_session(id, false).await.map(|_| ())?)
     }
 
-    /// Kill with `force=true`: overrides a kill refusal and retains the
-    /// guarded worktree (#81). Answers with what was kept.
-    pub async fn kill_session_forced(&self, id: Uuid) -> Result<KillOutcome, ClientError> {
+    /// Kill with `force=true`: overrides a kill refusal and leaves the
+    /// guarded worktree on disk (#81).
+    pub async fn kill_session_forced(&self, id: Uuid) -> Result<(), ClientError> {
         Ok(self.daemon.kill_session(id, true).await?)
     }
 
-    pub async fn list_worktrees(&self) -> Result<Vec<RetainedWorktree>, ClientError> {
-        Ok(self.daemon.list_worktrees().await?)
-    }
-
-    pub async fn reclaim_worktree(&self, id: Uuid) -> Result<(), ClientError> {
-        Ok(self.daemon.reclaim_worktree(id).await?)
-    }
-
     pub async fn list_projects(&self) -> Result<Vec<Project>, ClientError> {
         Ok(self.daemon.list_projects().await?)
     }

crates/maestro-e2e/tests/e2e.rs

@@ -415,7 +415,7 @@ async fn spawn_from_linked_worktree_places_in_main_checkout() {
 }
 
 // ---------------------------------------------------------------------------
-// Worktree lifecycle: collisions, retention, reclaim (#88)
+// Worktree lifecycle: collisions, guards, settling (#88)
 // ---------------------------------------------------------------------------
 
 /// #77/#88: the target path `<repo>/worktrees/<branch>` is occupied by a real
@@ -621,10 +621,10 @@ async fn guarded_kill_then_commit_and_push_makes_the_same_kill_succeed() {
 }
 
 /// #81/#88: commits that no remote has refuse the kill; `--force` ends the
-/// session anyway but RETAINS the worktree — force never destroys blocked
-/// work, it parks it in the retained store with reason `force-killed`.
+/// session anyway but LEAVES the worktree on disk — force never destroys
+/// blocked work.
 #[tokio::test(flavor = "multi_thread")]
-async fn unpushed_commits_block_kill_and_force_retains_the_tree() {
+async fn unpushed_commits_block_kill_and_force_keeps_the_tree() {
     let (_daemon, client) = bring_up().await;
 
     let base = TempDir::new().unwrap();
@@ -655,17 +655,7 @@ async fn unpushed_commits_block_kill_and_force_retains_the_tree() {
     }
 
     // Force overrides the refusal and nothing else: session gone, tree kept.
-    let outcome = client.kill_session_forced(s.id).await.expect("forced kill");
-    let retained = outcome
-        .retained
-        .expect("force must retain the guarded tree");
-    assert_eq!(
-        retained.reason,
-        maestro_protocol::RetainedReason::ForceKilled
-    );
-    assert_eq!(retained.path, wt);
-    assert_eq!(retained.branch, "feature/local");
-    assert_eq!(retained.in_the_way.unpushed_commits, 1);
+    client.kill_session_forced(s.id).await.expect("forced kill");
 
     assert!(
         !client
@@ -676,18 +666,18 @@ async fn unpushed_commits_block_kill_and_force_retains_the_tree() {
             .any(|x| x.id == s.id),
         "the forced kill ends the session"
     );
-    assert!(wt.is_dir(), "a force-retained worktree must stay on disk");
+    assert!(wt.is_dir(), "a guarded worktree must stay on disk");
     assert!(
         git_worktree_list(base.path())
             .iter()
             .any(|l| l.contains("feature/local")),
-        "the retained tree stays registered in git"
+        "the kept tree stays registered in git"
     );
 }
 
 /// #82/#88: a session that exits on its own never reaches the kill path, but
-/// its worktree follows the same rule. Dirty → retained with reason
-/// `exited-with-unsaved-work`; pristine → removed, leaving nothing behind.
+/// its worktree follows the same rule. Dirty: left on disk. Pristine:
+/// removed, leaving nothing behind.
 #[tokio::test(flavor = "multi_thread")]
 async fn natural_exit_settles_dirty_and_pristine_worktrees_differently() {
     let (_daemon, client) = bring_up().await;
@@ -734,46 +724,18 @@ async fn natural_exit_settles_dirty_and_pristine_worktrees_differently() {
 
     // The watcher reaps dead tmux sessions every couple of ticks and settles
     // each worktree accordingly. Poll for both outcomes.
-    wait_for_async(
-        "dirty exit retains its worktree",
-        Duration::from_secs(20),
-        || {
-            let client = client.clone();
-            let path = dirty_wt.clone();
-            async move {
-                client
-                    .list_worktrees()
-                    .await
-                    .ok()
-                    .map(|wts| {
-                        wts.iter().any(|w| {
-                            w.path == path
-                                && w.reason
-                                    == maestro_protocol::RetainedReason::ExitedWithUnsavedWork
-                        })
-                    })
-                    .unwrap_or(false)
-            }
-        },
-    )
-    .await;
     wait_for_async(
         "pristine exit removes its worktree entirely",
         Duration::from_secs(20),
         || {
-            let client = client.clone();
             let repo = pristine_repo.path().to_path_buf();
             let path = pristine_wt.clone();
             async move {
                 if path.exists() {
                     return false;
                 }
-                let listed = client.list_worktrees().await.ok();
-                let listed_clear = listed
-                    .map(|wts| !wts.iter().any(|w| w.path == path))
-                    .unwrap_or(false);
                 let registered = git_worktree_list(&repo);
-                !registered.iter().any(|l| l.contains("feature/tidy")) && listed_clear
+                !registered.iter().any(|l| l.contains("feature/tidy"))
             }
         },
     )
@@ -797,94 +759,6 @@ fn create_req_cmd(cwd: &std::path::Path, command: &str, branch: &str) -> CreateS
     }
 }
 
-/// #80/#88: `GET /api/worktrees` lists what the daemon is holding, with the
-/// reason and a live in-the-way breakdown; reclaim removes the directory and
-/// the store entry but NEVER the branch — its commits outlive the reclaim.
-#[tokio::test(flavor = "multi_thread")]
-async fn worktrees_endpoint_lists_reasons_and_reclaim_keeps_the_branch() {
-    let (_daemon, client) = bring_up().await;
-
-    let base = TempDir::new().unwrap();
-    init_repo(base.path());
-    let _origin = throwaway_origin(base.path());
-
-    let wt = base.path().join("worktrees").join("feature").join("parked");
-    let s = client
-        .spawn_session(&worktree_req(base.path(), "feature/parked", true))
-        .await
-        .expect("spawn");
-    std::fs::write(wt.join("README"), b"worth keeping\n").unwrap();
-    commit_all(&wt, "worth keeping");
-    let sha = branch_sha(&wt, "feature/parked");
-
-    let outcome = client.kill_session_forced(s.id).await.expect("force kill");
-    assert!(outcome.retained.is_some());
-
-    let listed = client.list_worktrees().await.expect("list worktrees");
-    assert_eq!(listed.len(), 1, "exactly one retained entry: {listed:?}");
-    let entry = &listed[0];
-    assert_eq!(entry.path, wt);
-    assert_eq!(entry.branch, "feature/parked");
-    assert_eq!(entry.reason, maestro_protocol::RetainedReason::ForceKilled);
-    assert_eq!(entry.in_the_way.unpushed_commits, 1);
-
-    client.reclaim_worktree(entry.id).await.expect("reclaim");
-    assert!(!wt.exists(), "reclaim removes the directory");
-    assert!(
-        client.list_worktrees().await.unwrap().is_empty(),
-        "reclaim drops the store entry"
-    );
-
-    // The whole point of reclaim: the branch and its commits survive.
-    assert_eq!(
-        branch_sha(base.path(), "feature/parked"),
-        sha,
-        "the branch must keep pointing at the same commit after reclaim"
-    );
-}
-
-/// The retained store is state-dir backed (#80): a daemon restart must not
-/// orphan the trees it is holding — they are still on disk with entries
-/// pointing at them, and still reclaimable afterwards.
-#[tokio::test(flavor = "multi_thread")]
-async fn retained_store_survives_a_daemon_restart() {
-    let (mut daemon, client) = bring_up().await;
-
-    let base = TempDir::new().unwrap();
-    init_repo(base.path());
-    let _origin = throwaway_origin(base.path());
-
-    let wt = base
-        .path()
-        .join("worktrees")
-        .join("feature")
-        .join("durable");
-    let s = client
-        .spawn_session(&worktree_req(base.path(), "feature/durable", true))
-        .await
-        .expect("spawn");
-    std::fs::write(wt.join("README"), b"survives restarts\n").unwrap();
-    commit_all(&wt, "survives restarts");
-    client.kill_session_forced(s.id).await.expect("force kill");
-
-    daemon.restart().await;
-
-    let listed = client.list_worktrees().await.expect("list after restart");
-    assert_eq!(listed.len(), 1, "store must survive restart: {listed:?}");
-    assert_eq!(listed[0].path, wt);
-    assert_eq!(
-        listed[0].reason,
-        maestro_protocol::RetainedReason::ForceKilled
-    );
-    assert!(wt.is_dir(), "the tree itself must survive the restart too");
-
-    client
-        .reclaim_worktree(listed[0].id)
-        .await
-        .expect("reclaim");
-    assert!(!wt.exists());
-}
-
 // ---------------------------------------------------------------------------
 // GET /api/projects + /api/branches (#54/#55/#66)
 // ---------------------------------------------------------------------------
@@ -1280,21 +1154,6 @@ fn commit_all(dir: &std::path::Path, message: &str) {
     run_git(&["commit", "-q", "-m", message], dir);
 }
 
-fn branch_sha(repo: &std::path::Path, branch: &str) -> String {
-    let out = Command::new("git")
-        .arg("-C")
-        .arg(repo)
-        .args(["rev-parse", "--verify", &format!("refs/heads/{branch}")])
-        .output()
-        .expect("git rev-parse");
-    assert!(
-        out.status.success(),
-        "branch {branch} missing: {}",
-        String::from_utf8_lossy(&out.stderr)
-    );
-    String::from_utf8_lossy(&out.stdout).trim().to_string()
-}
-
 #[tokio::test(flavor = "multi_thread")]
 async fn daemon_events_ws_streams_snapshot_then_session_added() {
     // NAP-248: subscribe to the daemon's WS /api/events, expect an initial

crates/maestro-protocol/src/client.rs

@@ -16,8 +16,7 @@ use uuid::Uuid;
 use crate::routes::{self, UrlError};
 use crate::{
     ApiError, BranchInfo, CreateSessionRequest, DaemonInfo, DirListing, ErrorInfo, GitRoot,
-    KillOutcome, Project, RecentDirs, RenameSessionRequest, ReportActivityRequest,
-    RetainedWorktree, Session,
+    Project, RecentDirs, RenameSessionRequest, ReportActivityRequest, Session,
 };
 
 /// Time budget for establishing a TCP connection to the daemon.
@@ -116,9 +115,8 @@ impl DaemonClient {
     }
 
     /// Kill a session. With `force`, the daemon overrides the kill refusal
-    /// (#79) and retains the worktree instead of removing it (#81); the
-    /// outcome then names what was kept and where.
-    pub async fn kill_session(&self, id: Uuid, force: bool) -> Result<KillOutcome, ClientError> {
+    /// (#79) and leaves the worktree on disk instead of removing it (#81).
+    pub async fn kill_session(&self, id: Uuid, force: bool) -> Result<(), ClientError> {
         let query: &[(&str, &str)] = if force {
             &[(routes::FORCE_QUERY, "true")]
         } else {
@@ -127,12 +125,8 @@ impl DaemonClient {
         let resp = self
             .send(Method::DELETE, &routes::session(id), query, None::<&()>)
             .await?;
-        let resp = check(resp).await?;
-        // An unforced kill keeps its historical `204 No Content`.
-        if resp.status() == reqwest::StatusCode::NO_CONTENT {
-            return Ok(KillOutcome { retained: None });
-        }
-        json(resp).await
+        check(resp).await?;
+        Ok(())
     }
 
     /// `PATCH /api/sessions/:id` (#125): rename a live session. A blank name
@@ -187,23 +181,6 @@ impl DaemonClient {
             .await
     }
 
-    /// `GET /api/worktrees` (#80): every retained worktree with its in-the-way
-    /// breakdown as of this read.
-    pub async fn list_worktrees(&self) -> Result<Vec<RetainedWorktree>, ClientError> {
-        self.get(routes::WORKTREES, &[]).await
-    }
-
-    /// `DELETE /api/worktrees/:id` (#86): reclaim one retained worktree. The
-    /// daemon removes the directory and drops the store entry; the branch is
-    /// never deleted.
-    pub async fn reclaim_worktree(&self, id: Uuid) -> Result<(), ClientError> {
-        let resp = self
-            .send(Method::DELETE, &routes::worktree(id), &[], None::<&()>)
-            .await?;
-        check(resp).await?;
-        Ok(())
-    }
-
     // -- plumbing -----------------------------------------------------------
 
     async fn get<T: DeserializeOwned>(

crates/maestro-protocol/src/lib.rs

@@ -222,23 +222,9 @@ pub struct SessionWorktree {
     pub base_repo: PathBuf,
 }
 
-/// Why a retained worktree outlived its session (#80). Same values the
-/// daemon's retained store persists; new variants are additive but clients
-/// should render unknown reasons generically.
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize)]
-#[serde(rename_all = "kebab-case")]
-pub enum RetainedReason {
-    /// The session was killed with `--force` while its worktree held unsaved
-    /// work.
-    ForceKilled,
-    /// The session ended by itself and its worktree held unsaved work.
-    ExitedWithUnsavedWork,
-}
-
-/// What a kill refusal would report about a worktree right now (#80) — the
-/// same breakdown that blocks an unforced kill, recomputed on every read of
-/// [`RetainedWorktree`] so it reflects the tree now rather than when it was
-/// retained.
+/// What a kill refusal reports about a worktree (#80) — the breakdown that
+/// blocks an unforced kill, recomputed on every probe so it reflects the tree
+/// now.
 #[derive(Debug, Clone, Copy, Default, PartialEq, Eq, Serialize, Deserialize)]
 pub struct InTheWay {
     /// Tracked files with uncommitted modifications.
@@ -250,41 +236,72 @@ pub struct InTheWay {
     /// Commits on the branch reachable from no remote-tracking ref.
     #[serde(default)]
     pub unpushed_commits: usize,
+    /// Lines added across the tracked changes — staged and unstaged alike,
+    /// measured against `HEAD`. Untracked files contribute nothing: counting
+    /// their lines means reading every one off disk, and they already have
+    /// their own count above (docs/SIDEBAR.md).
+    #[serde(default)]
+    pub insertions: usize,
+    /// Lines removed across the same tracked changes.
+    #[serde(default)]
+    pub deletions: usize,
 }
 
 impl InTheWay {
     /// Nothing blocks any more — the work has since been committed and pushed.
+    ///
+    /// The line counts are not consulted: they describe the same tracked
+    /// changes `uncommitted_files` counts, and a change with no line delta —
+    /// a mode change, a rename — still blocks.
     pub fn is_clear(&self) -> bool {
-        *self == Self::default()
+        self.uncommitted_files == 0 && self.untracked_files == 0 && self.unpushed_commits == 0
     }
 }
 
-/// One entry of `GET /api/worktrees` (#80): a retained worktree with its
-/// in-the-way state as of this read. Reclaiming it via
-/// `DELETE /api/worktrees/:id` removes the directory but **never** the branch.
-#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
-pub struct RetainedWorktree {
-    pub id: Uuid,
-    pub path: PathBuf,
-    pub base_repo: PathBuf,
-    pub branch: String,
-    pub reason: RetainedReason,
-    pub retained_at: DateTime<Utc>,
-    #[serde(default)]
-    pub in_the_way: InTheWay,
+/// Which fragment of the compact blocked-work stat a value is, and so which
+/// colour the client paints it (docs/SIDEBAR.md).
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+pub enum StatKind {
+    /// Lines added.
+    Added,
+    /// Lines removed.
+    Removed,
+    /// Changed tracked files, when the change carries no line delta.
+    Files,
+    /// Untracked files.
+    Untracked,
+    /// Unpushed commits.
+    Unpushed,
 }
 
-/// Body of a successful forced `DELETE /api/sessions/:id?force=true` (#81).
-/// A forced kill ends the session and retains its worktree instead of
-/// removing it; this names what was kept, where, and why, so a client can say
-/// where the work went rather than leaving the operator to wonder.
-#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
-pub struct KillOutcome {
-    /// Set when the worktree was retained with reason `force-killed`; `None`
-    /// when there was nothing to retain — no worktree at all, or one whose
-    /// guard was clear, which a forced kill removes as usual.
-    #[serde(default)]
-    pub retained: Option<RetainedWorktree>,
+impl InTheWay {
+    /// The breakdown as diff notation: `+124`, `-18`, `?3`, `↑2`.
+    ///
+    /// The line counts cover tracked changes only, so a tree whose only
+    /// tracked change carries no line delta — a mode change, a rename —
+    /// reports the file count (`~2`) rather than claiming `+0 -0`. Both
+    /// sidebars render exactly these fragments; [`Display`](std::fmt::Display)
+    /// remains the spelled-out phrasing for the kill refusal and the tooltip.
+    pub fn stats(&self) -> Vec<(StatKind, String)> {
+        let mut out = Vec::new();
+        if self.insertions > 0 || self.deletions > 0 {
+            if self.insertions > 0 {
+                out.push((StatKind::Added, format!("+{}", self.insertions)));
+            }
+            if self.deletions > 0 {
+                out.push((StatKind::Removed, format!("-{}", self.deletions)));
+            }
+        } else if self.uncommitted_files > 0 {
+            out.push((StatKind::Files, format!("~{}", self.uncommitted_files)));
+        }
+        if self.untracked_files > 0 {
+            out.push((StatKind::Untracked, format!("?{}", self.untracked_files)));
+        }
+        if self.unpushed_commits > 0 {
+            out.push((StatKind::Unpushed, format!("↑{}", self.unpushed_commits)));
+        }
+        out
+    }
 }
 
 impl std::fmt::Display for InTheWay {
@@ -586,6 +603,50 @@ mod tests {
         assert!(is_compatible("unknown"));
     }
 
+    /// docs/SIDEBAR.md: the sidebars render the breakdown as diff notation.
+    #[test]
+    fn stats_read_as_a_diff() {
+        let w = InTheWay {
+            uncommitted_files: 3,
+            untracked_files: 2,
+            unpushed_commits: 1,
+            insertions: 124,
+            deletions: 18,
+        };
+        let text: Vec<String> = w.stats().into_iter().map(|(_, t)| t).collect();
+        assert_eq!(text, ["+124", "-18", "?2", "↑1"]);
+        let kinds: Vec<StatKind> = w.stats().into_iter().map(|(k, _)| k).collect();
+        assert_eq!(
+            kinds,
+            [
+                StatKind::Added,
+                StatKind::Removed,
+                StatKind::Untracked,
+                StatKind::Unpushed
+            ]
+        );
+    }
+
+    /// A change with no line delta — a mode change — still has to say
+    /// something, and `+0 -0` would be a lie about the tree.
+    #[test]
+    fn stats_fall_back_to_the_file_count_without_a_line_delta() {
+        let w = InTheWay {
+            uncommitted_files: 2,
+            ..InTheWay::default()
+        };
+        let text: Vec<String> = w.stats().into_iter().map(|(_, t)| t).collect();
+        assert_eq!(text, ["~2"]);
+        assert!(!w.is_clear());
+    }
+
+    /// A clear tree says nothing at all — the fragment is omitted, not empty.
+    #[test]
+    fn a_clear_tree_has_no_stats() {
+        assert!(InTheWay::default().stats().is_empty());
+        assert!(InTheWay::default().is_clear());
+    }
+
     fn round_trip<T>(value: &T) -> T
     where
         T: Serialize + for<'de> Deserialize<'de>,
@@ -873,76 +934,6 @@ mod tests {
         assert_eq!(spec, round_trip(&spec));
     }
 
-    fn sample_retained_worktree() -> RetainedWorktree {
-        RetainedWorktree {
-            id: Uuid::from_u128(0x0123_4567_89ab_cdef_0123_4567_89ab_cdef),
-            path: PathBuf::from("/work/maestro/worktrees/feat/login"),
-            base_repo: PathBuf::from("/work/maestro"),
-            branch: "feat/login".into(),
-            reason: RetainedReason::ForceKilled,
-            retained_at: DateTime::parse_from_rfc3339("2026-08-24T10:00:00Z")
-                .unwrap()
-                .with_timezone(&Utc),
-            in_the_way: InTheWay {
-                uncommitted_files: 2,
-                untracked_files: 1,
-                unpushed_commits: 3,
-            },
-        }
-    }
-
-    #[test]
-    fn retained_worktree_round_trips() {
-        let wt = sample_retained_worktree();
-        assert_eq!(wt, round_trip(&wt));
-    }
-
-    /// A payload from a daemon predating `in_the_way` still decodes; the
-    /// breakdown defaults to "nothing in the way".
-    #[test]
-    fn retained_worktree_defaults_in_the_way_on_legacy_payload() {
-        let legacy = serde_json::json!({
-            "id": Uuid::from_u128(1),
-            "path": "/w/t",
-            "base_repo": "/w",
-            "branch": "feat/login",
-            "reason": "force-killed",
-            "retained_at": "2026-08-24T10:00:00Z",
-        });
-        let wt: RetainedWorktree = serde_json::from_value(legacy).unwrap();
-        assert_eq!(wt.reason, RetainedReason::ForceKilled);
-        assert_eq!(wt.in_the_way, InTheWay::default());
-        assert!(wt.in_the_way.is_clear());
-    }
-
-    #[test]
-    fn retained_reason_serializes_kebab_case() {
-        assert_eq!(
-            serde_json::to_value(RetainedReason::ForceKilled).unwrap(),
-            serde_json::json!("force-killed")
-        );
-        assert_eq!(
-            serde_json::to_value(RetainedReason::ExitedWithUnsavedWork).unwrap(),
-            serde_json::json!("exited-with-unsaved-work")
-        );
-    }
-
-    #[test]
-    fn kill_outcome_round_trips_a_retained_worktree() {
-        let outcome = KillOutcome {
-            retained: Some(sample_retained_worktree()),
-        };
-        assert_eq!(outcome, round_trip(&outcome));
-    }
-
-    /// A kill with nothing to retain — no worktree, or a clean one removed as
-    /// usual — decodes from an absent field.
-    #[test]
-    fn kill_outcome_defaults_to_nothing_retained() {
-        let outcome: KillOutcome = serde_json::from_value(serde_json::json!({})).unwrap();
-        assert_eq!(outcome.retained, None);
-    }
-
     #[test]
     fn catalog_event_round_trips_and_tags() {
         let session = sample_session();

crates/maestro-protocol/src/routes.rs

@@ -26,9 +26,6 @@ pub const BRANCHES: &str = "/api/branches";
 /// `GET` a batch of generated session handles (#53). Takes a `count` query
 /// param (default 1, capped at [`maestro_names`]' batch cap).
 pub const NAMES: &str = "/api/names";
-/// `GET` retained worktrees (#80): what the store holds, with the in-the-way
-/// breakdown recomputed on read.
-pub const WORKTREES: &str = "/api/worktrees";
 /// `WS` catalog snapshot + session deltas.
 pub const EVENTS: &str = "/api/events";
 
@@ -39,7 +36,7 @@ pub const PATH_QUERY: &str = "path";
 pub const COUNT_QUERY: &str = "count";
 
 /// Name of the boolean query parameter `DELETE /api/sessions/:id` takes (#81).
-/// `force=true` overrides the kill refusal and retains the worktree.
+/// `force=true` overrides the kill refusal and leaves the worktree on disk.
 pub const FORCE_QUERY: &str = "force";
 
 /// `DELETE /api/sessions/:id`.
@@ -52,11 +49,6 @@ pub fn session_activity(id: Uuid) -> String {
     format!("{SESSIONS}/{id}/activity")
 }
 
-/// `DELETE /api/worktrees/:id` — reclaim one retained worktree (#80).
-pub fn worktree(id: Uuid) -> String {
-    format!("{WORKTREES}/{id}")
-}
-
 /// `WS /api/attach/:id` — raw PTY bytes.
 pub fn attach(id: Uuid) -> String {
     format!("{API_PREFIX}/attach/{id}")
@@ -68,7 +60,6 @@ pub mod patterns {
     pub const SESSION: &str = "/api/sessions/:id";
     pub const SESSION_ACTIVITY: &str = "/api/sessions/:id/activity";
     pub const ATTACH: &str = "/api/attach/:id";
-    pub const WORKTREE: &str = "/api/worktrees/:id";
 }
 
 /// A daemon base URL that could not be turned into a WebSocket base.
@@ -183,10 +174,6 @@ mod tests {
             attach(id()),
             "/api/attach/6f1a7c0e-0f5e-4a2b-9a3d-6b1c2d3e4f50"
         );
-        assert_eq!(
-            worktree(id()),
-            "/api/worktrees/6f1a7c0e-0f5e-4a2b-9a3d-6b1c2d3e4f50"
-        );
         assert_eq!(
             attach_url("https://laptop.lan:7070", id()).unwrap(),
             "wss://laptop.lan:7070/api/attach/6f1a7c0e-0f5e-4a2b-9a3d-6b1c2d3e4f50"
@@ -209,10 +196,6 @@ mod tests {
             patterns::ATTACH.trim_end_matches(":id"),
             attach(id()).trim_end_matches(&id().to_string())
         );
-        assert_eq!(
-            patterns::WORKTREE.trim_end_matches(":id"),
-            worktree(id()).trim_end_matches(&id().to_string())
-        );
     }
 
     #[test]
@@ -226,7 +209,6 @@ mod tests {
             GITROOT,
             BRANCHES,
             NAMES,
-            WORKTREES,
             EVENTS,
         ] {
             assert!(path.starts_with(API_PREFIX), "{path} escapes {API_PREFIX}");

crates/maestro-tui/src/action.rs

@@ -51,8 +51,6 @@ pub enum Action {
     /// Rename the highlighted session (#125): opens a prompt pre-filled with
     /// its current name.
     RenameHighlighted,
-    /// Open the retained-worktrees view (#85).
-    ShowWorktrees,
 
     // Chrome
     ToggleSidebar,
@@ -96,7 +94,6 @@ impl Action {
             Action::FastForwardSpawn => "Create session with defaults".into(),
             Action::KillHighlighted => "Kill selected session".into(),
             Action::RenameHighlighted => "Rename selected session".into(),
-            Action::ShowWorktrees => "Show retained worktrees".into(),
             Action::ToggleSidebar => "Show or hide sidebar".into(),
             Action::ShowHelp => "Show help".into(),
             Action::Quit => "Quit Maestro".into(),
@@ -123,7 +120,6 @@ impl Action {
             "FastForwardSpawn" => return Ok(Action::FastForwardSpawn),
             "KillHighlighted" => return Ok(Action::KillHighlighted),
             "RenameHighlighted" => return Ok(Action::RenameHighlighted),
-            "ShowWorktrees" => return Ok(Action::ShowWorktrees),
             "ToggleSidebar" => return Ok(Action::ToggleSidebar),
             "ShowHelp" => return Ok(Action::ShowHelp),
             "Quit" => return Ok(Action::Quit),

crates/maestro-tui/src/app.rs

@@ -18,9 +18,7 @@ use std::time::Duration;
 use chrono::Utc;
 use crossterm::event::{Event as CtEvent, EventStream};
 use futures_util::StreamExt;
-use maestro_protocol::{
-    routes, BranchInfo, CreateSessionRequest, DaemonClient, NameBatch, RetainedWorktree,
-};
+use maestro_protocol::{routes, BranchInfo, CreateSessionRequest, DaemonClient, NameBatch};
 use ratatui::backend::CrosstermBackend;
 use ratatui::Terminal;
 use uuid::Uuid;
@@ -41,7 +39,6 @@ use crate::session_task::{SessionInput, SessionTask};
 use crate::sidebar::Sidebar;
 use crate::state::{AppState, Focus, OverlayKind, PaneRef, SidebarItem};
 use crate::text_input::TextInput;
-use crate::worktrees::WorktreesView;
 use crate::TuiError;
 
 #[derive(Clone, Copy, Debug, PartialEq, Eq)]
@@ -61,8 +58,7 @@ pub struct App {
     help: HelpOverlay,
     chord: ChordState,
     daemon: DaemonClient,
-    /// Results of draft data loads (and the worktrees view fetch, #85),
-    /// delivered back to the event loop.
+    /// Results of draft data loads, delivered back to the event loop.
     ///
     /// These are network calls — `/api/projects` on a cold cache walks all of
     /// `$HOME` — and awaiting them inside `perform` froze the whole TUI: no
@@ -73,9 +69,6 @@ pub struct App {
     /// Direct HTTP for the one endpoint [`DaemonClient`] doesn't cover yet:
     /// `GET /api/names` (#64).
     http: reqwest::Client,
-    /// Entries of the retained-worktrees view (#85). Daemon data, kept here
-    /// rather than in `AppState` so the overlay kind stays payload-free.
-    worktrees: WorktreesView,
     /// Edit buffer behind [`OverlayKind::Rename`] (#125), kept here for the
     /// same reason the worktree entries are.
     rename_input: TextInput,
@@ -122,11 +115,6 @@ pub enum Effect {
     },
     /// Fetch the draft target's branches from `/api/branches` (#65).
     LoadBranches(usize),
-    /// Fetch `/api/worktrees` for the retained-worktrees view (#85).
-    LoadWorktrees,
-    /// Reclaim one retained worktree (#85) — removes its directory; the
-    /// branch is never deleted.
-    ReclaimWorktree(Uuid),
     /// Rename a session (#125). An empty name clears the custom one, so the
     /// card falls back to the command.
     Rename {
@@ -157,7 +145,6 @@ impl App {
             chord: ChordState::new(chord_timeout),
             daemon,
             http,
-            worktrees: WorktreesView::default(),
             rename_input: TextInput::default(),
             draft_tx,
             draft_rx,
@@ -293,8 +280,6 @@ impl App {
                     self.spawn_probe_git_root(tab, seq, path)
                 }
                 Effect::LoadBranches(tab) => self.spawn_load_branches(tab),
-                Effect::LoadWorktrees => self.spawn_load_worktrees(),
-                Effect::ReclaimWorktree(id) => self.submit_reclaim(id).await,
                 Effect::Rename { id, name } => self.submit_rename(id, &name).await,
                 Effect::Forward { id, bytes } => {
                     if let Some(handle) = self.emulators.get(id) {
@@ -333,44 +318,6 @@ impl App {
                     (OverlayDecision::Consumed, Vec::new())
                 }
             },
-            Some(OverlayKind::Worktrees) => match key {
-                InputKey::Char { c: 'j', .. } | InputKey::Down => {
-                    self.worktrees.move_highlight(1);
-                    (OverlayDecision::Consumed, Vec::new())
-                }
-                InputKey::Char { c: 'k', .. } | InputKey::Up => {
-                    self.worktrees.move_highlight(-1);
-                    (OverlayDecision::Consumed, Vec::new())
-                }
-                InputKey::Char { c: 'r', .. } => {
-                    if let Some(wt) = self.worktrees.selected() {
-                        self.state.show_overlay(OverlayKind::ReclaimConfirm {
-                            id: wt.id,
-                            branch: wt.branch.clone(),
-                            // Shared phrasing: InTheWay's Display impl is what
-                            // the kill refusal and the web client use too.
-                            blockers: wt.in_the_way.to_string(),
-                        });
-                    }
-                    (OverlayDecision::Consumed, Vec::new())
-                }
-                InputKey::Char { c: 'q', .. } | InputKey::Esc => {
-                    self.state.dismiss_overlay();
-                    (OverlayDecision::Consumed, Vec::new())
-                }
-                _ => (OverlayDecision::Consumed, Vec::new()),
-            },
-            Some(OverlayKind::ReclaimConfirm { id, .. }) => match key {
-                InputKey::Char { c: 'y', .. } => {
-                    self.state.dismiss_overlay();
-                    (OverlayDecision::Consumed, vec![Effect::ReclaimWorktree(id)])
-                }
-                InputKey::Char { c: 'n', .. } | InputKey::Esc => {
-                    self.state.show_overlay(OverlayKind::Worktrees);
-                    (OverlayDecision::Consumed, Vec::new())
-                }
-                _ => (OverlayDecision::Consumed, Vec::new()),
-            },
             Some(OverlayKind::Help) => {
                 self.state.dismiss_overlay();
                 (OverlayDecision::Consumed, Vec::new())
@@ -406,22 +353,8 @@ impl App {
             Action::ScrollPanePageDown => {
                 self.scroll_active_pane(-(self.pane_page_rows() as isize))
             }
-            Action::SidebarNext => {
-                let order = visible_sidebar_order(
-                    &self.catalog.snapshot(),
-                    &self.state.collapsed_sidebar_groups,
-                );
-                self.state.move_sidebar_highlight(&order, 1);
-                self.state.focus = Focus::Sidebar;
-            }
-            Action::SidebarPrev => {
-                let order = visible_sidebar_order(
-                    &self.catalog.snapshot(),
-                    &self.state.collapsed_sidebar_groups,
-                );
-                self.state.move_sidebar_highlight(&order, -1);
-                self.state.focus = Focus::Sidebar;
-            }
+            Action::SidebarNext => self.step_sidebar_highlight(1),
+            Action::SidebarPrev => self.step_sidebar_highlight(-1),
             Action::ToggleSidebarGroup => self.toggle_highlighted_sidebar_group(),
             Action::ToggleSidebar => self.state.toggle_sidebar(),
             Action::ShowHelp => {
@@ -474,10 +407,6 @@ impl App {
                     self.state.show_overlay(OverlayKind::Rename(sid));
                 }
             }
-            Action::ShowWorktrees => {
-                self.state.show_overlay(OverlayKind::Worktrees);
-                effects.push(Effect::LoadWorktrees);
-            }
         }
         effects
     }
@@ -516,6 +445,21 @@ impl App {
         snap.sessions.first().map(|s| s.id)
     }
 
+    /// `j`/`k`: step the sidebar highlight one row. Group headers are skipped
+    /// — a directory is not something to focus, and the cards under it are one
+    /// row away anyway. A *collapsed* group is the exception: its header is the
+    /// only row it has, so skipping it would make the group unreachable.
+    fn step_sidebar_highlight(&mut self, delta: i32) {
+        let collapsed = self.state.collapsed_sidebar_groups.clone();
+        let order = visible_sidebar_order(&self.catalog.snapshot(), &collapsed);
+        self.state
+            .move_sidebar_highlight(&order, delta, |item| match item {
+                SidebarItem::Group(key) => collapsed.contains(key),
+                SidebarItem::Session(_) => true,
+            });
+        self.state.focus = Focus::Sidebar;
+    }
+
     fn toggle_highlighted_sidebar_group(&mut self) {
         let key = match self.state.sidebar_highlight.clone() {
             Some(SidebarItem::Group(key)) => Some(key),
@@ -791,22 +735,13 @@ impl App {
             self.fast_forward_spawn(info, projects, names).await;
             return;
         }
-        if let DraftUpdate::WorktreesLoaded {
-            worktrees,
-            home_dir,
-        } = update
-        {
-            self.worktrees.apply(worktrees);
-            self.worktrees.home_dir = home_dir;
-            return;
-        }
         let want = match &update {
             DraftUpdate::Data { draft, .. }
             | DraftUpdate::Names { draft, .. }
             | DraftUpdate::GitRoot { draft, .. }
             | DraftUpdate::Branches { draft, .. } => *draft,
             // Handled above; it targets no particular draft.
-            DraftUpdate::FastForward { .. } | DraftUpdate::WorktreesLoaded { .. } => unreachable!(),
+            DraftUpdate::FastForward { .. } => unreachable!(),
         };
         // Find it by identity: closing a tab shifts every index to its right,
         // so an in-flight load addressed by index would be dropped or land on
@@ -861,7 +796,7 @@ impl App {
             DraftUpdate::Branches { cwd, branches, .. } => {
                 draft.set_branches(&cwd, branches);
             }
-            DraftUpdate::FastForward { .. } | DraftUpdate::WorktreesLoaded { .. } => unreachable!(),
+            DraftUpdate::FastForward { .. } => unreachable!(),
         }
         if follow_up_branches {
             self.spawn_load_branches(tab_idx);
@@ -961,13 +896,7 @@ impl App {
 
     async fn submit_kill(&mut self, sid: Uuid) {
         match self.daemon.kill_session(sid, false).await {
-            Ok(outcome) => {
-                if let Some(w) = outcome.retained {
-                    self.state
-                        .toast(format!("worktree retained: {}", w.path.display()));
-                }
-                self.release_session(sid)
-            }
+            Ok(()) => self.release_session(sid),
             Err(e) => self.state.toast(format!("kill failed: {e}")),
         }
     }
@@ -980,38 +909,6 @@ impl App {
         }
     }
 
-    /// Fetch the retained-worktree store without blocking the loop (#85) —
-    /// same off-loop pattern as the draft loaders.
-    fn spawn_load_worktrees(&self) {
-        let (daemon, _, _, tx) = self.loader_handles();
-        tokio::spawn(async move {
-            let (worktrees, info) = tokio::join!(daemon.list_worktrees(), daemon.daemon_info());
-            let _ = tx.send(DraftUpdate::WorktreesLoaded {
-                worktrees: worktrees.unwrap_or_default(),
-                home_dir: info.map(|i| i.home_dir).unwrap_or_default(),
-            });
-        });
-    }
-
-    /// Reclaim one retained worktree (#85): the directory goes, the branch
-    /// never does. The open view drops the entry so the list tells the truth
-    /// without waiting for a re-fetch.
-    async fn submit_reclaim(&mut self, id: Uuid) {
-        match self.daemon.reclaim_worktree(id).await {
-            Ok(()) => {
-                self.worktrees.remove(id);
-                self.state.toast("worktree reclaimed; branch kept");
-            }
-            Err(e) => {
-                let message = e
-                    .daemon_message()
-                    .map(String::from)
-                    .unwrap_or_else(|| e.to_string());
-                self.state.toast(format!("reclaim failed: {message}"));
-            }
-        }
-    }
-
     fn redraw(
         &mut self,
         terminal: &mut Terminal<CrosstermBackend<Stdout>>,
@@ -1059,7 +956,6 @@ impl App {
             state: &self.state,
             draft,
             help: &self.help,
-            worktrees: Some(&self.worktrees),
             rename_input: &self.rename_input,
         };
 
@@ -1248,11 +1144,6 @@ enum DraftUpdate {
         projects: Option<Vec<maestro_protocol::Project>>,
         names: Vec<String>,
     },
-    /// The retained-worktree store arrived for the worktrees view (#85).
-    WorktreesLoaded {
-        worktrees: Vec<RetainedWorktree>,
-        home_dir: String,
-    },
 }
 
 /// `GET /api/names?count=N`. Free function: the spawned loaders have no `self`.
@@ -1448,11 +1339,9 @@ mod tests {
         app.catalog.apply_event(CatalogEvent::Snapshot {
             sessions: vec![test_session(sid, "/repo")],
         });
-        app.dispatch(Action::SidebarNext);
-        assert_eq!(
-            app.state.sidebar_highlight,
-            Some(SidebarItem::Group(PathBuf::from("/repo")))
-        );
+        // `j` never lands on an expanded group, so reach it the way a click
+        // would.
+        app.state.sidebar_highlight = Some(SidebarItem::Group(PathBuf::from("/repo")));
         assert!(app.dispatch(Action::AttachHighlighted).is_empty());
         assert!(app
             .state
@@ -1467,6 +1356,58 @@ mod tests {
         assert!(app.state.overlay().is_none());
     }
 
+    #[test]
+    fn sidebar_navigation_skips_expanded_group_headers() {
+        use maestro_protocol::CatalogEvent;
+
+        let a = Uuid::from_u128(20);
+        let b = Uuid::from_u128(21);
+        let mut app = make_app();
+        app.catalog.apply_event(CatalogEvent::Snapshot {
+            sessions: vec![test_session(a, "/one"), test_session(b, "/two")],
+        });
+
+        // Two groups, one card each: `j` visits the cards and nothing else.
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(a)));
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(b)));
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(a)));
+        app.dispatch(Action::SidebarPrev);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(b)));
+    }
+
+    /// A collapsed group hides its cards, so its header is the only row left
+    /// to land on — skipping it would strand the group.
+    #[test]
+    fn sidebar_navigation_stops_on_a_collapsed_group() {
+        use maestro_protocol::CatalogEvent;
+
+        let a = Uuid::from_u128(22);
+        let b = Uuid::from_u128(23);
+        let mut app = make_app();
+        app.catalog.apply_event(CatalogEvent::Snapshot {
+            sessions: vec![test_session(a, "/one"), test_session(b, "/two")],
+        });
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(a)));
+        // `c` on a card collapses the group that holds it.
+        app.dispatch(Action::ToggleSidebarGroup);
+        assert_eq!(
+            app.state.sidebar_highlight,
+            Some(SidebarItem::Group(PathBuf::from("/one")))
+        );
+
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(app.state.sidebar_highlight, Some(SidebarItem::Session(b)));
+        app.dispatch(Action::SidebarNext);
+        assert_eq!(
+            app.state.sidebar_highlight,
+            Some(SidebarItem::Group(PathBuf::from("/one")))
+        );
+    }
+
     #[test]
     fn collapsing_the_group_containing_a_card_highlight_keeps_a_visible_group_highlight() {
         let sid = Uuid::from_u128(15);
@@ -2198,162 +2139,4 @@ mod tests {
             }]
         );
     }
-
-    // -- retained worktrees (#85) -------------------------------------------
-
-    fn sample_worktree(id: u128) -> maestro_protocol::RetainedWorktree {
-        use chrono::TimeZone;
-        maestro_protocol::RetainedWorktree {
-            id: Uuid::from_u128(id),
-            path: "/home/naps/tea/maestro/worktrees/feat/login".into(),
-            base_repo: "/home/naps/tea/maestro".into(),
-            branch: "feat/login".into(),
-            reason: maestro_protocol::RetainedReason::ForceKilled,
-            retained_at: chrono::Utc.with_ymd_and_hms(2026, 8, 24, 10, 0, 0).unwrap(),
-            in_the_way: maestro_protocol::InTheWay {
-                uncommitted_files: 2,
-                untracked_files: 1,
-                unpushed_commits: 3,
-            },
-        }
-    }
-
-    fn app_with_one_retained_worktree() -> App {
-        let mut app = make_app();
-        app.worktrees.home_dir = "/home/naps".into();
-        app.worktrees.apply(vec![sample_worktree(42)]);
-        app.state.show_overlay(OverlayKind::Worktrees);
-        app
-    }
-
-    #[test]
-    fn show_worktrees_opens_the_view_and_fetches() {
-        let mut app = make_app();
-        assert_eq!(
-            app.dispatch(Action::ShowWorktrees),
-            vec![Effect::LoadWorktrees]
-        );
-        assert_eq!(app.state.overlay(), Some(&OverlayKind::Worktrees));
-    }
-
-    #[test]
-    fn r_over_the_highlighted_worktree_asks_for_a_confirmation() {
-        let mut app = app_with_one_retained_worktree();
-
-        let (decision, effects) = app.handle_overlay_input(&InputKey::char('r'));
-        assert_eq!(decision, OverlayDecision::Consumed);
-        assert!(effects.is_empty());
-        match app.state.overlay() {
-            Some(OverlayKind::ReclaimConfirm { id, branch, .. }) => {
-                assert_eq!(*id, Uuid::from_u128(42));
-                assert_eq!(branch, "feat/login");
-            }
-            other => panic!("expected a reclaim confirm, got {other:?}"),
-        }
-    }
-
-    /// The confirmation names what is in the way using the shared phrasing,
-    /// so the confirm step cannot drift from the list or the CLI.
-    #[test]
-    fn the_reclaim_confirmation_carries_the_shared_in_the_way_phrasing() {
-        let mut app = app_with_one_retained_worktree();
-        app.handle_overlay_input(&InputKey::char('r'));
-        match app.state.overlay() {
-            Some(OverlayKind::ReclaimConfirm { blockers, .. }) => {
-                assert_eq!(
-                    blockers,
-                    "2 uncommitted files, 1 untracked file, 3 unpushed commits"
-                );
-            }
-            other => panic!("expected a reclaim confirm, got {other:?}"),
-        }
-    }
-
-    #[test]
-    fn confirming_a_reclaim_asks_for_the_reclaim() {
-        let mut app = app_with_one_retained_worktree();
-        app.handle_overlay_input(&InputKey::char('r'));
-
-        let (decision, effects) = app.handle_overlay_input(&InputKey::char('y'));
-        assert_eq!(decision, OverlayDecision::Consumed);
-        assert_eq!(effects, vec![Effect::ReclaimWorktree(Uuid::from_u128(42))]);
-        assert!(app.state.overlay().is_none());
-    }
-
-    #[test]
-    fn declining_a_reclaim_returns_to_the_list() {
-        let mut app = app_with_one_retained_worktree();
-        app.handle_overlay_input(&InputKey::char('r'));
-
-        let (decision, effects) = app.handle_overlay_input(&InputKey::char('n'));
-        assert_eq!(decision, OverlayDecision::Consumed);
-        assert!(effects.is_empty());
-        assert_eq!(app.state.overlay(), Some(&OverlayKind::Worktrees));
-    }
-
-    #[test]
-    fn j_and_k_move_the_highlight_inside_the_view() {
-        let mut app = make_app();
-        app.worktrees
-            .apply(vec![sample_worktree(1), sample_worktree(2)]);
-        // Distinct ids so movement is observable.
-        app.worktrees.items[1].branch = "other".into();
-        app.state.show_overlay(OverlayKind::Worktrees);
-
-        app.handle_overlay_input(&InputKey::char('j'));
-        assert_eq!(app.worktrees.selected().unwrap().branch, "other");
-        app.handle_overlay_input(&InputKey::char('k'));
-        assert_eq!(app.worktrees.selected().unwrap().branch, "feat/login");
-    }
-
-    #[tokio::test]
-    async fn loading_worktrees_fills_the_view_from_the_daemon() {
-        use axum::routing::{delete, get};
-        use axum::Json;
-        use serde_json::json;
-
-        async fn info() -> Json<serde_json::Value> {
-            Json(json!({"hostname": "box", "version": "0.1.0", "home_dir": "/home/naps"}))
-        }
-        async fn worktrees() -> Json<serde_json::Value> {
-            Json(json!([{
-                "id": Uuid::from_u128(42).to_string(),
-                "path": "/home/naps/tea/maestro/worktrees/feat/login",
-                "base_repo": "/home/naps/tea/maestro",
-                "branch": "feat/login",
-                "reason": "force-killed",
-                "retained_at": "2026-08-24T10:00:00Z",
-                "in_the_way": {"uncommitted_files": 2, "untracked_files": 1, "unpushed_commits": 3},
-            }]))
-        }
-        async fn reclaim() -> Json<serde_json::Value> {
-            Json(json!(null))
-        }
-
-        let app_router = axum::Router::new()
-            .route(maestro_protocol::routes::INFO, get(info))
-            .route(maestro_protocol::routes::WORKTREES, get(worktrees))
-            .route("/api/worktrees/:id", delete(reclaim));
-        let listener = tokio::net::TcpListener::bind("127.0.0.1:0").await.unwrap();
-        let base = format!("http://{}", listener.local_addr().unwrap());
-        tokio::spawn(async move { axum::serve(listener, app_router).await.unwrap() });
-
-        let mut app = make_app_at(&base);
-        let effects = app.dispatch(Action::ShowWorktrees);
-        app.perform(effects).await;
-        let update = tokio::time::timeout(Duration::from_secs(5), app.draft_rx.recv())
-            .await
-            .expect("load completed")
-            .expect("channel open");
-        app.apply_draft_update(update).await;
-
-        assert_eq!(app.state.overlay(), Some(&OverlayKind::Worktrees));
-        assert_eq!(app.worktrees.items.len(), 1);
-        assert_eq!(app.worktrees.items[0].branch, "feat/login");
-        assert_eq!(app.worktrees.home_dir, "/home/naps");
-
-        // Reclaiming drops the entry from the open view.
-        app.submit_reclaim(Uuid::from_u128(42)).await;
-        assert!(app.worktrees.items.is_empty());
-    }
 }

crates/maestro-tui/src/config.rs

@@ -54,7 +54,6 @@ const DEFAULT_NORMAL: &[(&str, &str)] = &[
     ("S", "FastForwardSpawn"),
     ("x", "KillHighlighted"),
     ("r", "RenameHighlighted"),
-    ("w", "ShowWorktrees"),
     // Focus
     ("ctrl-w h", "FocusSidebar"),
     // Session scrollback
@@ -211,10 +210,6 @@ mod tests {
             cfg.keymap.resolve(Mode::Normal, "S"),
             Some(Action::FastForwardSpawn),
         );
-        assert_eq!(
-            cfg.keymap.resolve(Mode::Normal, "w"),
-            Some(Action::ShowWorktrees),
-        );
     }
 
     #[test]

crates/maestro-tui/src/draft.rs

@@ -699,12 +699,13 @@ impl DraftPane {
         order.push(DraftField::DirPicker);
         if self.cwd_raw_visible {
             order.push(DraftField::CwdRaw);
-        } else if self.selected_group_key.is_some() {
-            order.push(DraftField::WorktreeChoice);
         }
         order.push(DraftField::Name);
         order.push(DraftField::Advanced);
         if !self.advanced_collapsed {
+            if !self.cwd_raw_visible && self.selected_group_key.is_some() {
+                order.push(DraftField::WorktreeChoice);
+            }
             order.push(DraftField::WorktreeToggle);
             if self.worktree_body_visible() {
                 order.push(DraftField::BranchMode);
@@ -1082,14 +1083,13 @@ impl DraftPane {
 
         let show_custom = self.is_custom();
         let show_raw = self.cwd_raw_visible;
-        // The worktree toggle and its body stay behind the "Advanced options"
-        // disclosure (#93): collapsed hides both, regardless of what they'd
-        // otherwise show. The worktree *choice* step is not part of that —
-        // it belongs to the directory picker (#95 rule 3), so it stays
-        // visible once a project is picked.
+        // The worktree choice step, the worktree toggle and its body all stay
+        // behind the "Advanced options" disclosure (#93): collapsed hides
+        // them, regardless of what they'd otherwise show.
         let show_wt_toggle = !self.advanced_collapsed;
         let show_wt_body = show_wt_toggle && self.worktree_body_visible();
-        let show_wt_choice = self.selected_group_key.is_some() && !show_raw;
+        let show_wt_choice =
+            !self.advanced_collapsed && self.selected_group_key.is_some() && !show_raw;
         let wt_choice_rows_data = if show_wt_choice {
             self.worktree_choice_rows()
         } else {
@@ -1118,11 +1118,12 @@ impl DraftPane {
         constraints.push(Constraint::Length(list_capacity as u16));
         if show_raw {
             constraints.push(Constraint::Length(1));
-        } else if show_wt_choice {
-            constraints.push(Constraint::Length(wt_choice_capacity));
         }
         constraints.push(Constraint::Length(1)); // name
         constraints.push(Constraint::Length(1)); // advanced header
+        if show_wt_choice {
+            constraints.push(Constraint::Length(wt_choice_capacity));
+        }
         if show_wt_toggle {
             constraints.push(Constraint::Length(1)); // worktree toggle
         }
@@ -1268,7 +1269,52 @@ impl DraftPane {
             if self.focused_field == DraftField::CwdRaw {
                 cursor = Some((col, row.y));
             }
-        } else if show_wt_choice {
+        }
+
+        // -- name + reroll ---------------------------------------------------------
+        let name_row = next_row();
+        draw_gutter(f, name_row, "name:");
+        let col = draw_text_field(f, &self.name, name_row, field_style(DraftField::Name));
+        if self.focused_field == DraftField::Name {
+            cursor = Some((col, name_row.y));
+        }
+        // Dim reroll hint, tucked after the typed name without covering it.
+        let hint_offset = (GUTTER as u16 + self.name.value().chars().count() as u16 + 1)
+            .min(name_row.width.saturating_sub(10));
+        f.render_widget(
+            Paragraph::new(Line::from(Span::styled(
+                "^R reroll",
+                Style::default().add_modifier(Modifier::DIM),
+            ))),
+            Rect {
+                x: name_row.x + hint_offset,
+                y: name_row.y,
+                width: name_row.width.saturating_sub(hint_offset),
+                height: 1,
+            },
+        );
+
+        // -- advanced disclosure (#93) ------------------------------------------
+        let advanced_row = next_row();
+        f.render_widget(
+            Paragraph::new(Line::from(vec![
+                Span::raw(format!("{:<GUTTER$}", "adv:")),
+                Span::styled(
+                    format!(
+                        "[{}] Advanced options",
+                        if self.advanced_collapsed { "+" } else { "-" }
+                    ),
+                    field_style(DraftField::Advanced),
+                ),
+                Span::styled(
+                    format!(" — {}", self.advanced_summary()),
+                    Style::default().add_modifier(Modifier::DIM),
+                ),
+            ])),
+            advanced_row,
+        );
+
+        if show_wt_choice {
             // Step two (#96 rule 3): "+ new" first, then the repository's
             // worktrees newest-first.
             let choice_rows = wt_choice_rows_data;
@@ -1317,49 +1363,6 @@ impl DraftPane {
             }
         }
 
-        // -- name + reroll ---------------------------------------------------------
-        let name_row = next_row();
-        draw_gutter(f, name_row, "name:");
-        let col = draw_text_field(f, &self.name, name_row, field_style(DraftField::Name));
-        if self.focused_field == DraftField::Name {
-            cursor = Some((col, name_row.y));
-        }
-        // Dim reroll hint, tucked after the typed name without covering it.
-        let hint_offset = (GUTTER as u16 + self.name.value().chars().count() as u16 + 1)
-            .min(name_row.width.saturating_sub(10));
-        f.render_widget(
-            Paragraph::new(Line::from(Span::styled(
-                "^R reroll",
-                Style::default().add_modifier(Modifier::DIM),
-            ))),
-            Rect {
-                x: name_row.x + hint_offset,
-                y: name_row.y,
-                width: name_row.width.saturating_sub(hint_offset),
-                height: 1,
-            },
-        );
-
-        // -- advanced disclosure (#93) ------------------------------------------
-        let advanced_row = next_row();
-        f.render_widget(
-            Paragraph::new(Line::from(vec![
-                Span::raw(format!("{:<GUTTER$}", "adv:")),
-                Span::styled(
-                    format!(
-                        "[{}] Advanced options",
-                        if self.advanced_collapsed { "+" } else { "-" }
-                    ),
-                    field_style(DraftField::Advanced),
-                ),
-                Span::styled(
-                    format!(" — {}", self.advanced_summary()),
-                    Style::default().add_modifier(Modifier::DIM),
-                ),
-            ])),
-            advanced_row,
-        );
-
         // -- worktree (#65) -----------------------------------------------------
         if show_wt_toggle {
             let wt_row = next_row();
@@ -2207,9 +2210,14 @@ mod tests {
         assert_eq!(draft.worktree_choice, WorktreeChoice::New);
         assert_eq!(draft.picked_cwd.as_deref(), Some(Path::new("/repos/main")));
         assert!(draft.worktree_enabled, "\"+ new\" means create-worktree on");
+        assert!(
+            !draft.field_order().contains(&DraftField::WorktreeChoice),
+            "step two lives behind the collapsed \"Advanced options\" disclosure"
+        );
+        draft.toggle_advanced();
         assert!(
             draft.field_order().contains(&DraftField::WorktreeChoice),
-            "step two is now reachable"
+            "step two is reachable once advanced is open"
         );
         assert!(
             draft.take_gitroot_probe(probe_due()).is_none(),
@@ -2622,6 +2630,13 @@ mod tests {
         );
         draft.focused_field = DraftField::DirPicker;
         draft.on_input(&InputKey::Enter); // picks the "main" box
+        let collapsed = render_to_string(100, 24, |f| draft.render(f, f.area(), true));
+        assert!(
+            !collapsed.contains("+ new"),
+            "step two hides with \"Advanced options\" closed:\n{collapsed}"
+        );
+
+        draft.toggle_advanced();
         let rendered = render_to_string(100, 24, |f| draft.render(f, f.area(), true));
 
         assert!(

crates/maestro-tui/src/lib.rs

@@ -22,7 +22,6 @@ pub mod sidebar;
 pub mod state;
 pub mod terminal;
 pub mod text_input;
-pub mod worktrees;
 
 use std::path::PathBuf;
 use std::time::Duration;

crates/maestro-tui/src/overlays.rs

@@ -33,7 +33,6 @@ impl KillConfirm {
         f.render_widget(block, area);
         let mut lines = vec![Line::from(format!("Kill session {}?", self.label))];
         if let Some(blockers) = &self.blockers {
-            // Same phrasing the reclaim confirmation uses for its own guard.
             lines.push(Line::from(format!("Currently in the way: {blockers}")));
             lines.push(Line::from("The kill will be refused unless forced."));
         }
@@ -45,9 +44,8 @@ impl KillConfirm {
 
 /// Rename prompt for the highlighted session (#125).
 ///
-/// Borrows its text buffer rather than owning it: the buffer lives on `App`
-/// alongside the worktrees view's entries, so [`crate::state::OverlayKind`]
-/// stays a plain comparable enum.
+/// Borrows its text buffer rather than owning it: the buffer lives on `App`,
+/// so [`crate::state::OverlayKind`] stays a plain comparable enum.
 #[derive(Debug)]
 pub struct RenamePrompt<'a> {
     /// What the card shows today — the session's name, or the command it

crates/maestro-tui/src/render.rs

@@ -15,7 +15,6 @@ use crate::palette;
 use crate::pane::Pane;
 use crate::sidebar::Sidebar;
 use crate::state::{AppState, Focus, OverlayKind, SidebarItem};
-use crate::worktrees::{ReclaimConfirm, WorktreesView};
 
 /// Tagged regions for mouse hit-testing. The input router resolves a mouse
 /// coordinate to one of these via [`RegionMap::region_at`].
@@ -82,8 +81,6 @@ pub struct RenderInputs<'a> {
     pub draft: Option<&'a crate::draft::DraftPane>,
     pub state: &'a AppState,
     pub help: &'a HelpOverlay,
-    /// Retained-worktree entries for the worktrees view (#85).
-    pub worktrees: Option<&'a WorktreesView>,
     /// Edit buffer behind the rename prompt (#125).
     pub rename_input: &'a crate::text_input::TextInput,
 }
@@ -193,22 +190,6 @@ pub fn draw(f: &mut Frame, inputs: RenderInputs<'_>) -> RegionMap {
             inputs.help.render(f, area);
             map.modal = Some(area);
         }
-        Some(OverlayKind::Worktrees) => {
-            if let Some(view) = inputs.worktrees {
-                view.render(f, area);
-                map.modal = Some(area);
-            }
-        }
-        Some(OverlayKind::ReclaimConfirm {
-            branch, blockers, ..
-        }) => {
-            let rc = ReclaimConfirm {
-                branch: branch.clone(),
-                blockers: blockers.clone(),
-            };
-            rc.render(f, area);
-            map.modal = Some(area);
-        }
         Some(OverlayKind::Toast(text)) => {
             let t = Toast { text: text.clone() };
             t.render(f, area);
@@ -372,7 +353,6 @@ mod tests {
             draft: Some(draft),
             state: &state,
             help: &help,
-            worktrees: None,
             rename_input: &crate::text_input::TextInput::default(),
         };
 
@@ -426,7 +406,6 @@ mod tests {
                 draft: None,
                 state: &state,
                 help: &help,
-                worktrees: None,
                 rename_input: &crate::text_input::TextInput::default(),
             };
             let backend = ratatui::backend::TestBackend::new(80, 24);
@@ -466,7 +445,6 @@ mod tests {
                 draft: None,
                 state: &state,
                 help: &help,
-                worktrees: None,
                 rename_input: &crate::text_input::TextInput::default(),
             };
             let backend = ratatui::backend::TestBackend::new(80, 24);

crates/maestro-tui/src/sidebar.rs

@@ -9,7 +9,7 @@ use std::path::PathBuf;
 
 use chrono::Utc;
 use maestro_protocol::display::{abbreviate_dir, abbreviate_folder_name, session_tree_path};
-use maestro_protocol::{ActivityState, Session, SessionStatus};
+use maestro_protocol::{ActivityState, Session, SessionStatus, StatKind};
 use ratatui::layout::Rect;
 use ratatui::style::{Color, Modifier, Style};
 use ratatui::text::{Line, Span};
@@ -283,7 +283,16 @@ impl Sidebar {
         }
         let mut tail: Vec<Span<'static>> = Vec::new();
         if let Some(b) = blocked {
-            tail.push(Span::styled(format!(" ⚠ {b}"), muted));
+            let palette = palette::active();
+            let surface = palette.surface(highlighted && focused);
+            for (kind, text) in b.stats() {
+                let style = match kind {
+                    StatKind::Added => muted.fg(surface.active),
+                    StatKind::Removed => muted.fg(surface.error),
+                    _ => muted,
+                };
+                tail.push(Span::styled(format!(" {text}"), style));
+            }
         }
         if dead {
             tail.push(Span::styled(" exited", muted));
@@ -627,14 +636,18 @@ mod tests {
     }
 
     /// #83 via docs/SIDEBAR.md: blocked work lives on the meta line now, not
-    /// on the name row; a clear or unprobed worktree never warns.
+    /// on the name row, and reads as diff notation; a clear or unprobed
+    /// worktree says nothing.
     #[test]
-    fn blocked_session_warns_on_its_meta_line() {
+    fn blocked_session_states_its_diff_on_the_meta_line() {
         let mut blocked = session_named(1, "claude");
         blocked.activity_since = Utc::now() - Duration::seconds(5);
         blocked.blocked_work = Some(InTheWay {
+            uncommitted_files: 2,
+            untracked_files: 3,
             unpushed_commits: 4,
-            ..InTheWay::default()
+            insertions: 124,
+            deletions: 18,
         });
         let mut clear = session_named(2, "bash");
         clear.blocked_work = Some(InTheWay::default());
@@ -644,31 +657,66 @@ mod tests {
             highlight: None,
             attached: Default::default(),
         };
-        let all = rows(&draw(&sb, 40, 10));
+        let all = rows(&draw(&sb, 44, 10));
 
         let meta = all
             .iter()
-            .find(|l| l.contains("⚠"))
-            .expect("blocked card must warn somewhere");
-        assert!(
-            meta.contains("4 unpushed commits"),
-            "warning must carry InTheWay's summary: {meta:?}"
-        );
+            .find(|l| l.contains("+124"))
+            .expect("blocked card must state its diff somewhere");
+        for fragment in ["-18", "?3", "↑4"] {
+            assert!(
+                meta.contains(fragment),
+                "meta line must carry {fragment}: {meta:?}"
+            );
+        }
         let name_row = row_containing(&all, "claude");
         assert!(
-            !name_row.contains('⚠'),
-            "warning moved off the name row: {name_row:?}"
+            !name_row.contains("+124"),
+            "the diff stays off the name row: {name_row:?}"
         );
         for name in ["bash", "vim"] {
             let idx = all.iter().position(|l| l.contains(name)).unwrap();
             assert!(
-                !all[idx + 1].contains('⚠'),
-                "{name} must not warn: {:?}",
+                !all[idx + 1].contains('+') && !all[idx + 1].contains('?'),
+                "{name} must not state a diff: {:?}",
                 all[idx + 1]
             );
         }
     }
 
+    /// The additions and deletions are the only coloured fragments: they are
+    /// what a reader scans for, and colouring the rest would make the line
+    /// compete with the activity dot.
+    #[test]
+    fn the_diff_colours_only_its_additions_and_deletions() {
+        let mut s = session_named(1, "claude");
+        s.blocked_work = Some(InTheWay {
+            uncommitted_files: 1,
+            untracked_files: 0,
+            unpushed_commits: 2,
+            insertions: 7,
+            deletions: 3,
+        });
+        let sb = Sidebar {
+            sessions: vec![s],
+            highlight: None,
+            attached: Default::default(),
+        };
+        let buf = draw(&sb, 44, 8);
+        let palette = palette::active();
+        let surface = palette.surface(false);
+        let colour_of = |needle: char| {
+            (0..buf.area.width)
+                .flat_map(|x| (0..buf.area.height).map(move |y| (x, y)))
+                .find(|&(x, y)| buf[(x, y)].symbol() == needle.to_string())
+                .map(|(x, y)| buf[(x, y)].fg)
+                .expect("fragment must be drawn")
+        };
+        assert_eq!(colour_of('+'), surface.active);
+        assert_eq!(colour_of('-'), surface.error);
+        assert_eq!(colour_of('↑'), surface.muted);
+    }
+
     /// A fresh session still gets its unconditional second line, but no
     /// duration until it has been held in state for 30 seconds.
     #[test]
@@ -929,7 +977,7 @@ mod tests {
         let meta = content(&all[idx + 1]);
         assert!(meta.contains('…'), "truncated detail ends with …: {meta:?}");
         assert!(
-            meta.contains("2 unpushed commits"),
+            meta.contains("↑2"),
             "blocked work stays on the line: {meta:?}"
         );
         assert!(
@@ -1062,7 +1110,7 @@ mod tests {
             attached: Default::default(),
         };
         let all = rows(&draw(&sb, 29, 6));
-        assert!(all[3].contains("⚠ 1 uncommitted file"), "{:?}", all[3]);
+        assert!(all[3].contains("~1"), "{:?}", all[3]);
         let meta = content(&all[3]);
         assert!(meta.ends_with('…'), "command must truncate: {meta:?}");
     }
@@ -1081,7 +1129,7 @@ mod tests {
             attached: Default::default(),
         };
         let all = rows(&draw(&sb, 40, 6));
-        assert!(all[3].contains("⚠ 1 uncommitted file"), "{:?}", all[3]);
+        assert!(all[3].contains("~1"), "{:?}", all[3]);
         assert!(
             all[3].contains("bash"),
             "command must survive: {:?}",

crates/maestro-tui/src/state.rs

@@ -61,21 +61,10 @@ impl Tab {
 pub enum OverlayKind {
     KillConfirm(Uuid),
     /// Rename prompt for this session (#125). The edit buffer lives on `App`,
-    /// like the worktrees view's entries, so this stays comparable.
+    /// so this stays comparable.
     Rename(Uuid),
     Help,
     Toast(String),
-    /// The retained-worktrees view (#85); its entries live on `App` because
-    /// they are daemon data, not UI state.
-    Worktrees,
-    /// Reclaim confirmation for the worktree with this id (#85), guarding
-    /// the destructive key like [`OverlayKind::KillConfirm`] does.
-    ReclaimConfirm {
-        id: Uuid,
-        branch: String,
-        /// [`maestro_protocol::InTheWay`] as of when the confirm was opened.
-        blockers: String,
-    },
 }
 
 #[derive(Debug)]
@@ -176,21 +165,42 @@ impl AppState {
         self.active_tab().session()
     }
 
-    /// Move the sidebar highlight by `delta` (positive = down) within the
-    /// rendered item sequence. Wraps. No-op on empty.
-    pub fn move_sidebar_highlight(&mut self, order: &[SidebarItem], delta: i32) {
-        if order.is_empty() {
+    /// Move the sidebar highlight one step through the rendered item
+    /// sequence, in the direction of `delta` (positive = down), landing only
+    /// on items `is_stop` accepts. Only the sign of `delta` matters. Wraps.
+    /// No-op when `order` holds no stop.
+    ///
+    /// Anchoring walks the *full* rendered order, not just the stops, so a
+    /// highlight parked on a skipped row — an expanded group header, which a
+    /// mouse click can still select — steps to its neighbour instead of
+    /// jumping to the far end of the list.
+    pub fn move_sidebar_highlight(
+        &mut self,
+        order: &[SidebarItem],
+        delta: i32,
+        is_stop: impl Fn(&SidebarItem) -> bool,
+    ) {
+        if order.is_empty() || delta == 0 {
             return;
         }
         let len = order.len() as i32;
+        let step = delta.signum();
         let cur = self
             .sidebar_highlight
             .as_ref()
             .and_then(|id| order.iter().position(|x| x == id))
             .map(|i| i as i32)
-            .unwrap_or(-1);
-        let next = (cur + delta).rem_euclid(len) as usize;
-        self.sidebar_highlight = Some(order[next].clone());
+            // Nothing highlighted: start one off the end we walk from, so the
+            // first step lands on the first (or last) item.
+            .unwrap_or(if step > 0 { -1 } else { 0 });
+        let mut idx = cur;
+        for _ in 0..len {
+            idx = (idx + step).rem_euclid(len);
+            if is_stop(&order[idx as usize]) {
+                self.sidebar_highlight = Some(order[idx as usize].clone());
+                return;
+            }
+        }
     }
 
     /// Toggle a group. When collapsing a group containing the current card,
@@ -243,28 +253,86 @@ mod tests {
         ];
 
         // No prior highlight + delta +1 lands on first item.
-        state.move_sidebar_highlight(&order, 1);
+        state.move_sidebar_highlight(&order, 1, |_| true);
         assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(a)));
 
-        state.move_sidebar_highlight(&order, 1);
+        state.move_sidebar_highlight(&order, 1, |_| true);
         assert_eq!(
             state.sidebar_highlight,
             Some(SidebarItem::Group("repo".into()))
         );
-        state.move_sidebar_highlight(&order, 1);
+        state.move_sidebar_highlight(&order, 1, |_| true);
         assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(b)));
-        // Wraps forward.
-        state.move_sidebar_highlight(&order, 1);
+        state.move_sidebar_highlight(&order, 1, |_| true);
         assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(c)));
+        // Wraps forward.
+        state.move_sidebar_highlight(&order, 1, |_| true);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(a)));
         // Wraps backward.
-        state.move_sidebar_highlight(&order, -1);
+        state.move_sidebar_highlight(&order, -1, |_| true);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(c)));
+    }
+
+    #[test]
+    fn move_sidebar_highlight_skips_rows_the_filter_rejects() {
+        let mut state = AppState::new();
+        let a = Uuid::from_u128(1);
+        let b = Uuid::from_u128(2);
+        let order = vec![
+            SidebarItem::Group("repo".into()),
+            SidebarItem::Session(a),
+            SidebarItem::Group("other".into()),
+            SidebarItem::Session(b),
+        ];
+        let sessions_only = |item: &SidebarItem| matches!(item, SidebarItem::Session(_));
+
+        state.move_sidebar_highlight(&order, 1, sessions_only);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(a)));
+        state.move_sidebar_highlight(&order, 1, sessions_only);
         assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(b)));
+        state.move_sidebar_highlight(&order, -1, sessions_only);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(a)));
+
+        // Backwards from nothing lands on the last stop, not the last row.
+        state.sidebar_highlight = None;
+        state.move_sidebar_highlight(&order, -1, sessions_only);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(b)));
+    }
+
+    /// A highlight sitting on a skipped row still moves to its neighbour.
+    #[test]
+    fn move_sidebar_highlight_anchors_on_a_skipped_row() {
+        let mut state = AppState::new();
+        let a = Uuid::from_u128(1);
+        let b = Uuid::from_u128(2);
+        let order = vec![
+            SidebarItem::Session(a),
+            SidebarItem::Group("repo".into()),
+            SidebarItem::Session(b),
+        ];
+        let sessions_only = |item: &SidebarItem| matches!(item, SidebarItem::Session(_));
+
+        state.sidebar_highlight = Some(SidebarItem::Group("repo".into()));
+        state.move_sidebar_highlight(&order, 1, sessions_only);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(b)));
+
+        state.sidebar_highlight = Some(SidebarItem::Group("repo".into()));
+        state.move_sidebar_highlight(&order, -1, sessions_only);
+        assert_eq!(state.sidebar_highlight, Some(SidebarItem::Session(a)));
+    }
+
+    #[test]
+    fn move_sidebar_highlight_is_noop_when_nothing_is_a_stop() {
+        let mut state = AppState::new();
+        let order = vec![SidebarItem::Group("repo".into())];
+        state.move_sidebar_highlight(&order, 1, |_| false);
+        assert_eq!(state.sidebar_highlight, None);
     }
 
     #[test]
     fn move_sidebar_highlight_is_noop_on_empty_order() {
         let mut state = AppState::new();
-        state.move_sidebar_highlight(&[], 1);
+        state.move_sidebar_highlight(&[], 1, |_| true);
         assert_eq!(state.sidebar_highlight, None);
     }
 

crates/maestro-tui/src/worktrees.rs

@@ -1,259 +0,0 @@
-//! Retained-worktrees view (#85): an overlay listing every retained worktree
-//! — branch, abbreviated path, reason, what is in the way — with j/k to move,
-//! `r` to reclaim behind a confirmation, esc to close.
-//!
-//! The in-the-way summary comes straight from [`InTheWay`]'s `Display` impl,
-//! the same source the kill refusal and the web client use, so no two clients
-//! can describe the same tree differently.
-
-use ratatui::layout::Rect;
-use ratatui::text::Line;
-use ratatui::widgets::{Block, Borders, Clear, Paragraph};
-use ratatui::Frame;
-
-use maestro_protocol::{RetainedReason, RetainedWorktree};
-
-use crate::draft::abbreviate_path;
-
-/// The open worktrees overlay: its entries and which one is highlighted.
-#[derive(Debug, Clone, Default)]
-pub struct WorktreesView {
-    pub items: Vec<RetainedWorktree>,
-    pub highlight: usize,
-    /// Daemon-reported home dir, for `~` abbreviation of paths.
-    pub home_dir: String,
-}
-
-impl WorktreesView {
-    /// Replace the listing and keep the highlight pointing at something real.
-    pub fn apply(&mut self, items: Vec<RetainedWorktree>) {
-        self.items = items;
-        if !self.items.is_empty() {
-            self.highlight = self.highlight.min(self.items.len() - 1);
-        } else {
-            self.highlight = 0;
-        }
-    }
-
-    pub fn remove(&mut self, id: uuid::Uuid) {
-        let idx = self.items.iter().position(|w| w.id == id);
-        self.items.retain(|w| w.id != id);
-        if let Some(i) = idx {
-            self.highlight = self.highlight.min(self.items.len().saturating_sub(1));
-            let _ = i;
-        }
-    }
-
-    /// Move the highlight by `delta` (positive = down). Wraps. No-op on empty.
-    pub fn move_highlight(&mut self, delta: i32) {
-        if self.items.is_empty() {
-            return;
-        }
-        let len = self.items.len() as i32;
-        let next = (self.highlight as i32 + delta).rem_euclid(len);
-        self.highlight = next as usize;
-    }
-
-    pub fn selected(&self) -> Option<&RetainedWorktree> {
-        self.items.get(self.highlight)
-    }
-
-    pub fn render(&self, f: &mut Frame, parent: Rect) {
-        let h = (self.items.len() as u16 + 6).clamp(4, parent.height);
-        let area = super::overlays::centered(parent, parent.width.saturating_sub(4), h);
-        f.render_widget(Clear, area);
-        let block = Block::default()
-            .title(" retained worktrees ")
-            .borders(Borders::ALL);
-        let inner = block.inner(area);
-        f.render_widget(block, area);
-
-        let mut lines: Vec<Line> = Vec::new();
-        if self.items.is_empty() {
-            // A drained store is the normal state — say so plainly, not like
-            // something went wrong.
-            lines.push(Line::from("no retained worktrees"));
-        } else {
-            lines.push(Line::from(format!(
-                "  {:<26}  {:<34}  {:<24}  IN THE WAY",
-                "BRANCH", "PATH", "REASON"
-            )));
-            for (i, w) in self.items.iter().enumerate() {
-                let marker = if i == self.highlight { ">" } else { " " };
-                lines.push(Line::from(format!(
-                    "{marker} {:<25}  {:<34}  {:<24}  {}",
-                    w.branch,
-                    abbreviate_path(&w.path, &self.home_dir),
-                    reason_label(w.reason),
-                    w.in_the_way,
-                )));
-            }
-        }
-        lines.push(Line::from(""));
-        lines.push(Line::from("j/k: select  r: reclaim  esc: close"));
-        f.render_widget(Paragraph::new(lines), inner);
-    }
-}
-
-/// The confirmation guarding reclaim, mirroring [`super::overlays::KillConfirm`].
-/// It names the branch and says out loud what reclaim does not take with it —
-/// docs/WORKTREES.md: "reclaim" otherwise sounds like it takes the commits too.
-#[derive(Debug, Clone)]
-pub struct ReclaimConfirm {
-    pub branch: String,
-    /// [`maestro_protocol::InTheWay`] rendered via its `Display` impl.
-    pub blockers: String,
-}
-
-impl ReclaimConfirm {
-    pub fn render(&self, f: &mut Frame, parent: Rect) {
-        let area = super::overlays::centered(parent, 70, 7);
-        f.render_widget(Clear, area);
-        let block = Block::default().title(" reclaim ").borders(Borders::ALL);
-        let inner = block.inner(area);
-        f.render_widget(block, area);
-        f.render_widget(
-            ratatui::widgets::Paragraph::new(vec![
-                Line::from(format!("Reclaim worktree {}?", self.branch)),
-                Line::from(format!("Currently in the way: {}", self.blockers)),
-                Line::from("The directory is removed; the branch is never deleted."),
-                Line::from(""),
-                Line::from("[y] yes   [n] no"),
-            ]),
-            inner,
-        );
-    }
-}
-
-fn reason_label(reason: RetainedReason) -> &'static str {
-    match reason {
-        RetainedReason::ForceKilled => "force-killed",
-        RetainedReason::ExitedWithUnsavedWork => "exited-with-unsaved-work",
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use chrono::{TimeZone, Utc};
-    use maestro_protocol::{InTheWay, RetainedReason};
-    use ratatui::backend::TestBackend;
-    use ratatui::Terminal;
-
-    fn render_to_string<F>(w: u16, h: u16, draw: F) -> String
-    where
-        F: FnOnce(&mut Frame),
-    {
-        let backend = TestBackend::new(w, h);
-        let mut terminal = Terminal::new(backend).unwrap();
-        terminal.draw(draw).unwrap();
-        let buf = terminal.backend().buffer().clone();
-        let mut out = String::new();
-        for y in 0..buf.area.height {
-            for x in 0..buf.area.width {
-                out.push_str(buf[(x, y)].symbol());
-            }
-            out.push('\n');
-        }
-        out
-    }
-
-    fn worktree(branch: &str, path: &str, id: u128) -> RetainedWorktree {
-        RetainedWorktree {
-            id: uuid::Uuid::from_u128(id),
-            path: path.into(),
-            base_repo: "/home/naps/tea/maestro".into(),
-            branch: branch.into(),
-            reason: RetainedReason::ForceKilled,
-            retained_at: Utc.with_ymd_and_hms(2026, 8, 24, 10, 0, 0).unwrap(),
-            in_the_way: InTheWay::default(),
-        }
-    }
-
-    #[test]
-    fn view_lists_branch_path_reason_and_what_is_in_the_way() {
-        let mut view = WorktreesView {
-            home_dir: "/home/naps".into(),
-            ..Default::default()
-        };
-        let mut wt = worktree(
-            "feat/login",
-            "/home/naps/tea/maestro/worktrees/feat/login",
-            1,
-        );
-        wt.in_the_way = InTheWay {
-            uncommitted_files: 2,
-            untracked_files: 1,
-            unpushed_commits: 3,
-        };
-        view.apply(vec![wt]);
-
-        let rendered = render_to_string(160, 20, |f| view.render(f, f.area()));
-        assert!(rendered.contains("feat/login"), "{rendered}");
-        // Path abbreviated against the daemon's home dir, same helper as the
-        // draft pane's project list.
-        assert!(rendered.contains("~/t/m/w/f/login"), "{rendered}");
-        assert!(rendered.contains("force-killed"), "{rendered}");
-        // Phrasing comes from InTheWay's Display impl — shared with the kill
-        // refusal and the web client.
-        assert!(
-            rendered.contains("2 uncommitted files, 1 untracked file, 3 unpushed commits"),
-            "{rendered}"
-        );
-    }
-
-    #[test]
-    fn empty_store_looks_like_a_normal_state_not_an_error() {
-        let view = WorktreesView::default();
-        let rendered = render_to_string(80, 12, |f| view.render(f, f.area()));
-        assert!(rendered.contains("no retained worktrees"), "{rendered}");
-        assert!(!rendered.contains("error"), "{rendered}");
-        assert!(!rendered.contains("failed"), "{rendered}");
-    }
-
-    #[test]
-    fn highlight_moves_and_wraps() {
-        let mut view = WorktreesView::default();
-        view.apply(vec![
-            worktree("a", "/r/a", 1),
-            worktree("b", "/r/b", 2),
-            worktree("c", "/r/c", 3),
-        ]);
-        assert_eq!(view.selected().unwrap().branch, "a");
-        view.move_highlight(1);
-        assert_eq!(view.selected().unwrap().branch, "b");
-        view.move_highlight(-1);
-        assert_eq!(view.selected().unwrap().branch, "a");
-        view.move_highlight(-1);
-        // Wraps backward to the last entry.
-        assert_eq!(view.selected().unwrap().branch, "c");
-        view.move_highlight(1);
-        assert_eq!(view.selected().unwrap().branch, "a");
-    }
-
-    #[test]
-    fn apply_clamps_a_stale_highlight() {
-        let mut view = WorktreesView::default();
-        view.apply(vec![worktree("a", "/r/a", 1), worktree("b", "/r/b", 2)]);
-        view.move_highlight(1);
-        view.apply(vec![worktree("c", "/r/c", 3)]);
-        assert_eq!(view.selected().unwrap().branch, "c");
-    }
-
-    #[test]
-    fn reclaim_confirm_names_the_branch_and_says_it_survives() {
-        let confirm = ReclaimConfirm {
-            branch: "feat/login".into(),
-            blockers: "2 uncommitted files".into(),
-        };
-        let rendered = render_to_string(80, 14, |f| confirm.render(f, f.area()));
-        assert!(
-            rendered.contains("Reclaim worktree feat/login?"),
-            "{rendered}"
-        );
-        assert!(rendered.contains("2 uncommitted files"), "{rendered}");
-        assert!(rendered.contains("never deleted"), "{rendered}");
-        assert!(rendered.contains("yes"), "{rendered}");
-        assert!(rendered.contains("no"), "{rendered}");
-    }
-}

docs/SIDEBAR.md

@@ -37,7 +37,7 @@ itself is not.
 Line 2 reads left to right:
 
 ```
-<dot> [<duration>] [<detail>] [<blocked work> | exited] <command>
+<dot> [<duration>] [<detail>] [<diff stat> | exited] <command>
 ```
 
 Its minimum content is therefore the dot and the command.
@@ -64,12 +64,46 @@ truncates with `…` ahead of the trailing fragments — a long detail never
 pushes `<blocked work>`, `exited`, or `<command>` off the line, and neither
 client ever lets it wrap.
 
+`<diff stat>` is the blocked-work breakdown (#83) in diff notation, replacing
+the spelled-out `⚠ 3 uncommitted files, 2 unpushed commits`. It renders only
+while something would block an unforced kill, and reads:
+
+```
++<insertions> -<deletions> ?<untracked files> ↑<unpushed commits>
+```
+
+Each fragment is omitted when its count is zero. The line counts cover
+**tracked** changes only — staged and unstaged together, measured against
+`HEAD` — so a `git add` never makes the counter drop. Untracked files are
+counted as files, not lines: counting their lines means reading every one off
+disk, and `?3` already says what is there. A tree whose only tracked change
+carries no line delta — a mode change, a rename — shows `~<changed files>`
+instead of the additions and deletions, since `+0 -0` would describe it
+wrongly.
+
+Only `+` and `-` take colour (`--live` / `--danger` in the web client, the
+surface's `active` / `error` roles in the TUI); `~`, `?` and `↑` stay muted, so
+the fragment reads as a diff rather than as an alarm. The spelled-out phrasing
+of `maestro_protocol::InTheWay`'s `Display` does not disappear: it is the web
+card's tooltip and the wording of the kill refusal, so no two clients describe
+the same tree differently.
+
 `<command>` is unconditional and muted: the session's command line, shown on
 every card in both clients — no omission in the TUI, no name-vs-command
 comparison gating it. A session named by its command line then shows the same
 string on both lines; that is accepted, the price of a row that never changes
 height.
 
+In the **web client** the command's leading program is replaced by that tool's
+icon when it is one we have a mark for — Claude Code, Codex, opencode, Gemini
+CLI, Aider, Cursor, pi, and any shell — and only the remaining words of the
+command line follow as text. Recognition looks through a path, leading
+`FOO=bar` assignments and wrappers (`sudo`, `env`, `npx`, `pnpm dlx`, …); an
+unrecognised program keeps its full text, so the card never shows a mark
+standing for the wrong tool. The full command line stays in the card's tooltip
+either way. The **TUI** keeps the command as text: a terminal cannot be assumed
+to have a font carrying tool glyphs.
+
 ## Colour per activity state
 
 The state word is never spelled out — the colour carries it.
@@ -95,9 +129,10 @@ surface; a colour chosen for the card background is never painted onto the
 selection background. Contrast is a property of the palette (values are RGB)
 and is asserted in the palette's tests, not left to the terminal theme.
 
-The duration text inherits its state's colour; the blocked-work, exited and
-command fragments use the surface's `muted` role, since the ⚠ glyph already
-signals severity.
+The duration text inherits its state's colour; the exited and command
+fragments use the surface's `muted` role, as do the diff stat's `~`, `?` and
+`↑` fragments. Only `+` and `-` step outside the surface's muted role, and they
+take the `active` and `error` roles of the same surface.
 
 ## Duration format
 
@@ -146,3 +181,22 @@ dropping the whole session would be worse than dimming one card. A report
 arriving at `POST /api/sessions/:id/activity` is parsed strictly and an unknown
 state is a 4xx: an agent posting a state the daemon does not know is a bug in
 the agent, and answering 204 would hide it.
+
+## Keyboard walk
+
+`j`/`k` (and the arrow keys) move a highlight through the sidebar. Both clients
+follow one rule for what the walk stops on:
+
+- **Session cards, always.**
+- **Folder headers, only while collapsed.** An expanded folder's cards are one
+  row away, so its header is nothing to land on; a collapsed folder's header is
+  the only row it has, and skipping it would leave the folder unreachable.
+
+Enter opens the highlighted session, or expands the highlighted folder and
+lands on the first row it was hiding. A highlight parked on a skipped row —
+reachable by clicking a folder header — still steps to its neighbour rather
+than jumping to the end of the list.
+
+The web client needs a chord to reach the list at all (`Ctrl+Shift+H`, and the
+same chord back): an attached terminal owns every plain keystroke. The TUI uses
+`ctrl-w h`.

docs/WORKTREES.md

@@ -12,9 +12,9 @@ have to reconstruct it.
 
 ## Nothing is ever adopted
 
-The daemon only ever removes a worktree it created in this session's lifetime or
-recorded in its retained store. It never takes ownership of a directory it finds
-on disk, whatever that directory looks like.
+The daemon only ever removes a worktree it created in this session's lifetime.
+It never takes ownership of a directory it finds on disk, whatever that
+directory looks like.
 
 There is no reliable way to ask "is this worktree mine?". A git worktree carries
 no marker distinguishing one a daemon made from one a human made ten minutes
@@ -69,75 +69,48 @@ its worktree still matters leaves a directory nobody is looking at.
 
 A consequence, accepted deliberately: a feature branch almost always has unpushed
 commits, so `--force` is the ordinary way to end a session, not an exception.
-That is only safe because force retains.
+That is only safe because force never removes what blocked.
 
 ### `--force`
 
 `--force` overrides the refusal and nothing else. It ends the session and
-**retains** the worktree, which enters the retained store with reason
-`force-killed`. Force never removes a tree that would have blocked. A tree
-whose guard is clear never needed the override, so a forced kill removes it as
-usual — force changes what happens to blocked work, not ordinary cleanup.
+**leaves the worktree on disk**, untouched. Force never removes a tree that
+would have blocked. A tree whose guard is clear never needed the override, so a
+forced kill removes it as usual — force changes what happens to blocked work,
+not ordinary cleanup.
 
-Ending a session and destroying its work are two separate destructive acts.
-Each is chosen on its own, and the second one — reclaim — is chosen while
-looking at exactly what would be lost.
-
-A forced kill answers with what was retained and why, so a client can say where
-the work went rather than leaving the operator to wonder.
+Ending a session and destroying its work are two separate acts. Maestro only
+performs the first one. Removing a tree it left behind is the operator's own
+`git worktree remove`, chosen while looking at exactly what would be lost.
 
 ### Exiting on its own
 
 A session that ends by itself — the command finished, the operator typed `exit` —
 never reaches the kill path. When the watcher notices the tmux session is gone it
-applies the same rule: a pristine worktree is removed, anything else is retained
-with reason `exited-with-unsaved-work`.
+applies the same rule: a pristine worktree is removed, anything else is left on
+disk.
 
 Nobody is waiting for an answer here, so there is nothing to refuse; the choice is
-only between removing and retaining. It uses the same predicate as the kill, from
+only between removing and leaving. It uses the same predicate as the kill, from
 the same place, so the two paths cannot drift apart.
 
 The session itself stays in the catalog marked dead until it is dismissed — the
 replay buffer of a crashed agent outlives it — but its worktree is settled at
-once. Once settled, the session drops its worktree reference, so dismissing it
-afterwards neither refuses (there is no guard left to trip) nor records a second
-store entry.
+once. A removed tree drops off the record, so dismissing the session afterwards
+does not refuse: there is no guard left to trip.
 
-If settling is impossible — the probe fails, or the store cannot take the entry —
-nothing is destroyed. The tree stays on disk *and* stays attached to the dead
-record, where dismissing it later still goes through `--force`'s
-retain-before-teardown path. Losing the store must never mean losing the work.
+A tree that was left alone stays attached to the dead record, where dismissing it
+later goes through `--force`, which leaves it on disk too. The same holds when
+the probe itself fails: nothing is destroyed.
 
-## The retained store
+## Nothing is removed on the operator's behalf
 
-A `state_dir`-backed record of worktrees the daemon created that no longer have a
-session. For each one: the path, the base repository, the branch, why it was
-retained, and when.
+There is no store of worktrees the daemon is holding, no list endpoint, and no
+reclaim command. A tree left behind by a forced kill or a dirty exit is an
+ordinary git worktree on disk; `git worktree list` finds it and
+`git worktree remove` removes it.
 
-It survives a daemon restart. It is the recovery path for everything above — a
-store that is lost on restart is worse than no store, because the worktrees are
-still on disk with nothing pointing at them.
-
-Entries leave when the worktree is reclaimed, and when the daemon notices the
-directory is gone because somebody removed it by hand.
-
-**It is not an ownership record.** It must never be consulted to decide whether a
-path may be adopted — see the first rule. Its only job is to remember what the
-daemon is holding so that it can be listed and reclaimed.
-
-## Reclaim
-
-`GET /api/worktrees` lists the store: path, base repository, branch, reason, when,
-and what is currently in the way. The in-the-way breakdown is recomputed on read,
-not stored, so a tree whose work has since been pushed says so.
-
-`DELETE /api/worktrees/<id>` reclaims one. It removes the directory and drops the
-store entry. It does **not** consult the guard — reclaim is the deliberate
-destructive act, and the operator chose it while looking at what was in the way.
-
-It never deletes the branch. `git worktree remove` leaves the ref, and reclaiming
-a tree must not throw away commits on it. This is worth saying out loud in every
-client, because "reclaim" otherwise sounds like it takes the commits too.
-
-Nothing is reclaimed automatically. There is no age-based expiry and no bulk
-reclaim.
+That is deliberate. A tracked list of trees pending deletion is a second place
+the truth can live, and it goes stale the moment somebody removes a directory by
+hand. Git already knows which worktrees exist, and removing one is not a
+decision Maestro is better placed to make.

web/package.json

@@ -13,6 +13,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
+    "@fontsource/jetbrains-mono": "^5.3.0",
     "@xterm/addon-fit": "^0.10.0",
     "@xterm/xterm": "^5.5.0"
   },

web/pnpm-lock.yaml

@@ -8,6 +8,9 @@ importers:
 
   .:
     dependencies:
+      '@fontsource/jetbrains-mono':
+        specifier: ^5.3.0
+        version: 5.3.0
       '@xterm/addon-fit':
         specifier: ^0.10.0
         version: 0.10.0(@xterm/xterm@5.5.0)
@@ -168,6 +171,9 @@ packages:
     cpu: [x64]
     os: [win32]
 
+  '@fontsource/jetbrains-mono@5.3.0':
+    resolution: {integrity: sha512-fqDfB5I9f1p1TV486aUgB9t8zP84P0O1FtQR5Ol9vjwPy+S+EIGlVYm1cvj2W5shcZMTg2nZFdVMoH5wFu8a1A==}
+
   '@jridgewell/sourcemap-codec@1.5.5':
     resolution: {integrity: sha512-cYQ9310grqxueWbl+WuIUIaiUaDcj7WOq5fVhEljNVgRfOUhY9fy2zTvfoqWsnebh8Sl70VScFbICvJnLKB0Og==}
 
@@ -631,6 +637,8 @@ snapshots:
   '@esbuild/win32-x64@0.21.5':
     optional: true
 
+  '@fontsource/jetbrains-mono@5.3.0': {}
+
   '@jridgewell/sourcemap-codec@1.5.5': {}
 
   '@rollup/rollup-android-arm-eabi@4.60.4':

web/src/api.ts

@@ -9,7 +9,6 @@ import type {
   NameBatch,
   Projects,
   RecentDirs,
-  RetainedWorktree,
   Session,
   Uuid,
 } from "./types.js";
@@ -112,22 +111,4 @@ export async function fetchBranches(path: string): Promise<Branches> {
   return jsonOrError(await fetch(url, { credentials: "same-origin" }));
 }
 
-/** The daemon's retained worktrees (#80): trees kept after a session ended
- * with unsaved work, with what is currently in the way. Drives the
- * retained-worktrees view (#84). */
-export async function listWorktrees(): Promise<RetainedWorktree[]> {
-  return jsonOrError(await fetch(routes.WORKTREES, { credentials: "same-origin" }));
-}
-
-/** Reclaim a retained worktree: removes its directory and drops the store
- * entry. Never deletes the branch (docs/WORKTREES.md). */
-export async function reclaimWorktree(id: Uuid): Promise<void> {
-  await jsonOrError(
-    await fetch(routes.worktree(id), {
-      method: "DELETE",
-      credentials: "same-origin",
-    }),
-  );
-}
-
 export { eventsWsUrl } from "./routes.js";

web/src/attach.ts

@@ -27,6 +27,8 @@ export interface AttachConnection {
   onMount(): void;
   /** Parked into the cache: keep streaming, just stop driving focus/resize. */
   onUnmount(): void;
+  /** Put the keyboard back into this terminal — what leaving the sidebar does. */
+  focusTerminal(): void;
   /** Fully disconnect: close the WS, dispose the terminal, remove the element. */
   dispose(): void;
 }
@@ -97,7 +99,7 @@ export function createAttachConnection(
   const term = new Terminal({
     cursorBlink: true,
     fontFamily:
-      'ui-monospace, SFMono-Regular, "SF Mono", Menlo, Consolas, "Liberation Mono", monospace',
+      '"JetBrains Mono", ui-monospace, SFMono-Regular, "SF Mono", Menlo, Consolas, "Liberation Mono", monospace',
     fontSize: 13,
     convertEol: false,
     theme: {
@@ -382,6 +384,9 @@ export function createAttachConnection(
     onUnmount() {
       mounted = false;
     },
+    focusTerminal() {
+      term.focus();
+    },
     dispose() {
       if (disposed) return;
       disposed = true;

web/src/catalog.test.ts

@@ -47,15 +47,33 @@ function rowFor(el: HTMLElement, id: string): HTMLElement | null {
 }
 
 describe("blocked-work indicator (#83)", () => {
-  it("marks the row of a blocked session with the shared phrasing", () => {
+  it("marks the row of a blocked session in diff notation", () => {
     const blocked = session({
       id: "aaaaaaaa-0000-0000-0000-000000000001",
-      blocked_work: { uncommitted_files: 3, unpushed_commits: 2 },
+      blocked_work: {
+        uncommitted_files: 3,
+        unpushed_commits: 2,
+        insertions: 124,
+        deletions: 18,
+      },
     });
     const el = renderSidebar(storeWith([blocked])).el;
     const badge = rowFor(el, blocked.id)?.querySelector(".leaf-blocked");
-    expect(badge?.textContent).toContain("⚠");
-    expect(badge?.textContent).toContain("3 uncommitted files, 2 unpushed commits");
+    expect(badge?.textContent).toBe("+124-18↑2");
+    // The spelled-out breakdown survives in the tooltip.
+    expect(badge?.getAttribute("title")).toContain(
+      "3 uncommitted files, 2 unpushed commits",
+    );
+  });
+
+  it("falls back to the file count when tracked changes carry no lines", () => {
+    const blocked = session({
+      id: "aaaaaaaa-0000-0000-0000-00000000001a",
+      blocked_work: { uncommitted_files: 2, untracked_files: 1 },
+    });
+    const el = renderSidebar(storeWith([blocked])).el;
+    const badge = rowFor(el, blocked.id)?.querySelector(".leaf-blocked");
+    expect(badge?.textContent).toBe("~2?1");
   });
 
   it("does not mark clear or unprobed sessions", () => {
@@ -161,7 +179,7 @@ describe("session cards (#107)", () => {
     expect(line2).not.toBeNull();
     expect(line2?.querySelector(".activity-dot")).not.toBeNull();
     expect(line2?.querySelector(".activity-duration")).toBeNull();
-    expect(line2?.querySelector(".leaf-cmd")?.textContent).toBe("claude");
+    expect(line2?.querySelector(".leaf-cmd")).not.toBeNull();
   });
 
   it("shows dot + floored duration once the state has been held past 30s", () => {
@@ -177,14 +195,40 @@ describe("session cards (#107)", () => {
     expect(line2?.querySelector(".activity-duration")?.textContent).toBe("4m");
   });
 
-  it("renders the command when the session name equals it", () => {
+  it("renders the command as its tool icon when the tool is known", () => {
     const s = session({
       id: "aaaaaaaa-0000-0000-0000-00000000001b",
       command: "claude",
       metadata: { name: "claude" },
     });
     const el = renderSidebar(storeWith([s])).el;
-    expect(rowFor(el, s.id)?.querySelector(".card-line2 .leaf-cmd")?.textContent).toBe("claude");
+    const cmd = rowFor(el, s.id)?.querySelector<HTMLElement>(".card-line2 .leaf-cmd");
+    expect(cmd?.querySelector(".tool-icon")?.getAttribute("data-tool")).toBe("claude");
+    expect(cmd?.getAttribute("title")).toBe("claude");
+    // The icon said it all: no text left over.
+    expect(cmd?.querySelector(".leaf-cmd-rest")).toBeNull();
+  });
+
+  it("keeps the command's arguments as text beside the icon", () => {
+    const s = session({
+      id: "aaaaaaaa-0000-0000-0000-00000000001c",
+      command: "claude --resume",
+    });
+    const el = renderSidebar(storeWith([s])).el;
+    const cmd = rowFor(el, s.id)?.querySelector<HTMLElement>(".card-line2 .leaf-cmd");
+    expect(cmd?.querySelector(".tool-icon")?.getAttribute("data-tool")).toBe("claude");
+    expect(cmd?.querySelector(".leaf-cmd-rest")?.textContent).toBe("--resume");
+  });
+
+  it("keeps an unknown command as plain text", () => {
+    const s = session({
+      id: "aaaaaaaa-0000-0000-0000-00000000001d",
+      command: "hx src/main.rs",
+    });
+    const el = renderSidebar(storeWith([s])).el;
+    const cmd = rowFor(el, s.id)?.querySelector<HTMLElement>(".card-line2 .leaf-cmd");
+    expect(cmd?.querySelector(".tool-icon")).toBeNull();
+    expect(cmd?.querySelector(".leaf-cmd-rest")?.textContent).toBe("hx src/main.rs");
   });
 
   it("maps each activity state to its contract slug", () => {
@@ -222,15 +266,20 @@ describe("session cards (#107)", () => {
     expect(line2?.querySelector(".activity-duration")?.textContent).toBe("2m");
   });
 
-  it("moves blocked work onto line 2 in the shared phrasing", () => {
+  it("moves blocked work onto line 2 as diff notation", () => {
     const s = session({
       id: "aaaaaaaa-0000-0000-0000-000000000017",
-      blocked_work: { uncommitted_files: 3, unpushed_commits: 2 },
+      blocked_work: {
+        uncommitted_files: 3,
+        unpushed_commits: 2,
+        insertions: 124,
+        deletions: 18,
+      },
     });
     const el = renderSidebar(storeWith([s])).el;
     const line2 = rowFor(el, s.id)?.querySelector<HTMLElement>(".card-line2");
-    expect(line2?.textContent).toContain("⚠");
-    expect(line2?.textContent).toContain("3 uncommitted files, 2 unpushed commits");
+    expect(line2?.textContent).toContain("+124");
+    expect(line2?.textContent).toContain("-18");
     // Line 1 carries no badges.
     expect(rowFor(el, s.id)?.querySelector(".leaf-badge")).toBeNull();
   });

web/src/catalog.ts

@@ -1,11 +1,12 @@
 import { escapeHtml } from "./html.js";
 import { getDaemonInfo, killSession, renameSession } from "./api.js";
 import { openContextMenu } from "./contextMenu.js";
-import { HUD_TOGGLE_EVENT, isHudEnabled, toggleHud } from "./debug.js";
+import { isTypingTarget, toggleHelp } from "./help.js";
 import { CatalogStore } from "./events.js";
-import type { Session } from "./types.js";
+import type { InTheWay, Session } from "./types.js";
 import {
   abbreviateFolderName,
+  blockerStats,
   customSessionName,
   formatInTheWay,
   MAX_SESSION_NAME_LEN,
@@ -13,6 +14,7 @@ import {
   sessionBlockers,
   sessionName,
 } from "./types.js";
+import { splitCommand, toolIconSvg } from "./toolIcon.js";
 import {
   activityColorSlug,
   activityDurationLabel,
@@ -30,6 +32,18 @@ export function requestOpenSpawn(): void {
   document.dispatchEvent(new CustomEvent(OPEN_SPAWN_EVENT));
 }
 
+/** Ask the shell to hand the keyboard back to the focused pane's terminal —
+ *  what Escape in the sidebar list does. Handled in main.ts, which owns the
+ *  connection pool. */
+export const FOCUS_PANE_EVENT = "catalog-focus-pane";
+
+/** The rows a keyboard walk stops on, in visual (document) order. Expanded
+ *  folder headers are skipped: a directory is not something to focus, and the
+ *  cards under it are one row away anyway. A *collapsed* folder still counts —
+ *  its header is the only row it has, so skipping it would leave the folder
+ *  unreachable. Same rule as the TUI sidebar. */
+const NAV_SELECTOR = '.tree-leaf, .tree-folder-header[aria-expanded="false"]';
+
 const collapsed = new Set<string>();
 
 /** Session whose card is currently being renamed inline (#125). The tree
@@ -61,15 +75,33 @@ function metaLine(s: Session, now: number): string {
     s.activity_detail
       ? `<span class="activity-detail">${escapeHtml(s.activity_detail)}</span>`
       : "",
-    blockers
-      ? `<span class="card-flag leaf-blocked" title="Kill will be refused unless forced">⚠ ${escapeHtml(formatInTheWay(blockers))}</span>`
-      : "",
+    blockers ? blockedFragment(blockers) : "",
     dead ? `<span class="card-flag">exited</span>` : "",
-    `<span class="card-flag leaf-cmd" title="${escapeHtml(s.command)}">${escapeHtml(s.command)}</span>`,
+    commandFragment(s.command),
   ];
   return `<span class="card-line2" data-activity="${slug}">${fragments.join("")}</span>`;
 }
 
+/** docs/SIDEBAR.md: blocked work renders as diff notation — `+124 -18 ?3 ↑2`
+ * — with the spelled-out breakdown kept in the tooltip. */
+function blockedFragment(blockers: InTheWay): string {
+  const stats = blockerStats(blockers)
+    .map((st) => `<span class="blocked-stat" data-stat="${st.kind}">${escapeHtml(st.text)}</span>`)
+    .join("");
+  const title = `Kill will be refused unless forced — ${formatInTheWay(blockers)}`;
+  return `<span class="card-flag leaf-blocked" title="${escapeHtml(title)}">${stats}</span>`;
+}
+
+/** docs/SIDEBAR.md: the command's leading program becomes its tool icon when
+ * we have a mark for it, and only the remaining words stay as text. The full
+ * command line stays in the tooltip either way. */
+function commandFragment(command: string): string {
+  const { tool, rest } = splitCommand(command);
+  const icon = tool ? toolIconSvg(tool) : null;
+  const text = icon ? rest : command;
+  return `<span class="card-flag leaf-cmd" title="${escapeHtml(command)}">${icon ?? ""}${text ? `<span class="leaf-cmd-rest">${escapeHtml(text)}</span>` : ""}</span>`;
+}
+
 function renderNode(node: TreeNode, depth: number): HTMLElement {
   if (node.kind === "folder") {
     const wrap = document.createElement("div");
@@ -82,6 +114,7 @@ function renderNode(node: TreeNode, depth: number): HTMLElement {
       header.style.paddingLeft = `${depth * 16}px`;
       const isCollapsed = collapsed.has(node.path);
       header.setAttribute("aria-expanded", String(!isCollapsed));
+      header.dataset.navKey = `folder:${node.path}`;
       // Single down-chevron; CSS rotates it -90° (→ right) when collapsed, so the
       // glyph and the rotation never fight each other.
       // A deep path (`~/git.naps.pt/naps62/maestro`) collapses its leading
@@ -111,6 +144,7 @@ function renderNode(node: TreeNode, depth: number): HTMLElement {
   row.href = `#/attach/${encodeURIComponent(s.id)}`;
   row.style.paddingLeft = `${depth * 16 + 8}px`;
   row.dataset.sessionId = s.id;
+  row.dataset.navKey = `session:${s.id}`;
   const running = s.status === "Running";
   row.dataset.status = running ? "running" : "dead";
   const name = sessionName(s);
@@ -269,25 +303,6 @@ function renderEmpty(filtering: boolean, onSpawn: () => void): HTMLElement {
  * the app-level actions (spawn, theme toggle). Kept out of the sidebar so these
  * stay put and reachable regardless of which pane is showing.
  */
-/** Navbar toggle for the debug HUD (#20), next to the design-kit shortcut.
- * Reflects the current state and stays in sync with the Ctrl+Shift+D keybind. */
-function createDebugToggle(): HTMLButtonElement {
-  const btn = document.createElement("button");
-  btn.type = "button";
-  btn.className = "btn btn-ghost debug-toggle";
-  btn.textContent = "debug";
-  btn.title = "Toggle the debug HUD (Ctrl+Shift+D)";
-  const sync = () => {
-    const on = isHudEnabled();
-    btn.setAttribute("aria-pressed", String(on));
-    btn.classList.toggle("is-active", on);
-  };
-  btn.addEventListener("click", () => toggleHud());
-  window.addEventListener(HUD_TOGGLE_EVENT, sync);
-  sync();
-  return btn;
-}
-
 export function renderTopbar(store: CatalogStore, onToggleSidebar?: () => void): HTMLElement {
   const header = document.createElement("header");
   header.className = "app-header";
@@ -315,15 +330,6 @@ export function renderTopbar(store: CatalogStore, onToggleSidebar?: () => void):
   renderStatus();
 
   const actions = header.querySelector(".app-actions") as HTMLElement;
-  // Retained worktrees (#84): a section reachable from the header, not a
-  // sidebar fixture — PRODUCT.md's scan is the product, so this must not
-  // compete with the session catalog for attention.
-  const worktreesLink = document.createElement("a");
-  worktreesLink.className = "btn btn-ghost";
-  worktreesLink.href = "#/worktrees";
-  worktreesLink.textContent = "worktrees";
-  worktreesLink.title = "Retained worktrees";
-  actions.appendChild(worktreesLink);
   // DEV-only shortcut to the standalone design kit (?kit). Lets you reach it by
   // clicking inside a proxied preview (e.g. crit live) that has no address bar.
   if (import.meta.env.DEV) {
@@ -334,7 +340,14 @@ export function renderTopbar(store: CatalogStore, onToggleSidebar?: () => void):
     kit.title = "Open the design kit (DEV)";
     actions.appendChild(kit);
   }
-  actions.appendChild(createDebugToggle());
+  const helpBtn = document.createElement("button");
+  helpBtn.type = "button";
+  helpBtn.className = "btn btn-ghost btn-icon help-btn";
+  helpBtn.textContent = "?";
+  helpBtn.title = "Keyboard shortcuts (?)";
+  helpBtn.setAttribute("aria-label", "Keyboard shortcuts");
+  helpBtn.addEventListener("click", () => toggleHelp());
+  actions.appendChild(helpBtn);
   actions.appendChild(createThemeToggle());
   const spawnBtn = document.createElement("button");
   spawnBtn.type = "button";
@@ -348,12 +361,10 @@ export function renderTopbar(store: CatalogStore, onToggleSidebar?: () => void):
 }
 
 /** Two-tier sidebar highlight state (issue #33): which sessions are open in some
- *  pane (quiet), which one is focused (loud), and a stable per-session colour so
- *  a row maps to its pane. */
+ *  pane (quiet), and which one is focused (loud). */
 export interface SidebarHighlights {
   open: Set<string>;
   focused: string | null;
-  colors: Map<string, string>;
 }
 
 /** Live handle to the persistent sidebar (built once, survives navigation). */
@@ -363,8 +374,12 @@ export interface SidebarHandle {
   /** Highlight the row for `id` (or clear it with `null`) as the focused session.
    *  Back-compat shim over {@link setHighlights}. */
   setActive(id: string | null): void;
-  /** Apply the full open/focused/colour highlight state. */
+  /** Apply the full open/focused highlight state. */
   setHighlights(h: SidebarHighlights): void;
+  /** Hand the keyboard to the session list, highlighting a row to walk from. */
+  focusList(): void;
+  /** Whether the session list currently holds DOM focus. */
+  hasListFocus(): boolean;
   /** Tear down the store subscription + listeners. */
   dispose(): void;
 }
@@ -472,7 +487,7 @@ export function renderSidebar(store: CatalogStore, opts: SidebarOptions = {}): S
     refresh();
   });
 
-  let highlights: SidebarHighlights = { open: new Set(), focused: null, colors: new Map() };
+  let highlights: SidebarHighlights = { open: new Set(), focused: null };
   const applyHighlights = () => {
     for (const n of treeHost.querySelectorAll<HTMLElement>(".tree-leaf")) {
       const id = n.dataset.sessionId;
@@ -481,12 +496,80 @@ export function renderSidebar(store: CatalogStore, opts: SidebarOptions = {}): S
       const open = highlights.open.has(id);
       n.classList.toggle("is-active", focused);
       n.classList.toggle("is-open", open && !focused);
-      const color = highlights.colors.get(id);
-      if (color && (open || focused)) n.style.setProperty("--mv-color", color);
-      else n.style.removeProperty("--mv-color");
     }
   };
 
+  // ---- keyboard navigation ----
+  // The walk reads the rendered DOM rather than a parallel model, so it can
+  // never disagree with what is on screen. Rows carry a `data-nav-key`
+  // (`session:<id>` / `folder:<path>`) that survives a re-render.
+  let navKey: string | null = null;
+
+  const navRows = (): HTMLElement[] => [
+    ...treeHost.querySelectorAll<HTMLElement>(NAV_SELECTOR),
+  ];
+
+  const rowFor = (key: string | null): HTMLElement | undefined =>
+    key === null ? undefined : navRows().find((r) => r.dataset.navKey === key);
+
+  const applyNav = () => {
+    const rows = navRows();
+    // A row that went away (session killed, folder collapsed) drops the
+    // highlight rather than leaving it pointing at nothing.
+    if (navKey !== null && !rows.some((r) => r.dataset.navKey === navKey)) navKey = null;
+    for (const r of rows) r.classList.toggle("is-highlighted", r.dataset.navKey === navKey);
+  };
+
+  const moveNav = (delta: number) => {
+    const rows = navRows();
+    if (rows.length === 0) return;
+    const cur = rows.findIndex((r) => r.dataset.navKey === navKey);
+    // Nothing highlighted: start one off the end we are walking from, so the
+    // first step lands on the first (or last) row.
+    const from = cur === -1 ? (delta > 0 ? -1 : 0) : cur;
+    const next = rows[(((from + delta) % rows.length) + rows.length) % rows.length];
+    navKey = next.dataset.navKey ?? null;
+    applyNav();
+    next.scrollIntoView({ block: "nearest" });
+  };
+
+  /** Enter: open the highlighted session, or expand the highlighted folder. */
+  const activateNav = () => {
+    const row = rowFor(navKey);
+    if (!row) return;
+    if (row.classList.contains("tree-leaf")) {
+      row.click();
+      return;
+    }
+    // Expanding replaces the folder's header in the walk with the rows it was
+    // hiding, so land on the first of them instead of losing the highlight.
+    const key = navKey;
+    row.click();
+    const header = [...treeHost.querySelectorAll<HTMLElement>(".tree-folder-header")].find(
+      (h) => h.dataset.navKey === key,
+    );
+    const revealed = header?.parentElement
+      ? [...header.parentElement.querySelectorAll<HTMLElement>(NAV_SELECTOR)][0]
+      : undefined;
+    navKey = revealed?.dataset.navKey ?? null;
+    applyNav();
+  };
+
+  // Focusable as a unit: the walk moves a highlight, not DOM focus, so the
+  // keyboard stays here while j/k runs.
+  treeHost.tabIndex = -1;
+  treeHost.addEventListener("keydown", (e) => {
+    if (e.ctrlKey || e.altKey || e.metaKey) return;
+    // The inline rename field lives inside a row and owns its own keys.
+    if (isTypingTarget(e.target)) return;
+    if (e.key === "j" || e.key === "ArrowDown") moveNav(1);
+    else if (e.key === "k" || e.key === "ArrowUp") moveNav(-1);
+    else if (e.key === "Enter") activateNav();
+    else if (e.key === "Escape") document.dispatchEvent(new CustomEvent(FOCUS_PANE_EVENT));
+    else return;
+    e.preventDefault();
+  });
+
   const refresh = () => {
     // Rebuilding the tree now would blow away the field being typed into.
     if (renamingId !== null) return;
@@ -504,6 +587,7 @@ export function renderSidebar(store: CatalogStore, opts: SidebarOptions = {}): S
       for (const child of tree.children) treeHost.appendChild(renderNode(child, 0));
     }
     applyHighlights();
+    applyNav();
   };
 
   void getDaemonInfo()
@@ -536,7 +620,6 @@ export function renderSidebar(store: CatalogStore, opts: SidebarOptions = {}): S
       highlights = {
         open: id ? new Set([id]) : new Set(),
         focused: id,
-        colors: new Map(),
       };
       applyHighlights();
     },
@@ -544,6 +627,19 @@ export function renderSidebar(store: CatalogStore, opts: SidebarOptions = {}): S
       highlights = h;
       applyHighlights();
     },
+    focusList() {
+      // Start from the session on screen — that is where the user's attention
+      // already is — falling back to the standing highlight, then the top row.
+      const preferred = highlights.focused ? `session:${highlights.focused}` : null;
+      const start = rowFor(preferred) ?? rowFor(navKey) ?? navRows()[0];
+      navKey = start?.dataset.navKey ?? null;
+      applyNav();
+      treeHost.focus();
+      start?.scrollIntoView({ block: "nearest" });
+    },
+    hasListFocus() {
+      return treeHost.contains(document.activeElement);
+    },
     dispose() {
       clearInterval(clock);
       // An unfinished rename would otherwise leave every later sidebar

web/src/connections.ts

@@ -140,6 +140,13 @@ export class ConnectionPool {
     this.setFocus(null);
   }
 
+  /** Put the keyboard back into the focused session's terminal. No-op when
+   *  nothing is focused (welcome pane) or its connection is gone. */
+  focusTerminal(): void {
+    if (this.focusedId === null) return;
+    this.cache.get(this.focusedId)?.focusTerminal();
+  }
+
   /** Fully disconnect + forget a session. */
   close(id: Uuid): void {
     const conn = this.cache.get(id);

web/src/help.test.ts

@@ -0,0 +1,112 @@
+// @vitest-environment happy-dom
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
+
+import {
+  SHORTCUTS,
+  closeHelp,
+  installHelpKeybind,
+  isHelpOpen,
+  isTypingTarget,
+  openHelp,
+} from "./help.js";
+
+beforeEach(() => {
+  installHelpKeybind();
+});
+
+afterEach(() => {
+  closeHelp();
+  document.body.replaceChildren();
+});
+
+function press(key: string, init: KeyboardEventInit = {}): void {
+  window.dispatchEvent(new KeyboardEvent("keydown", { key, bubbles: true, ...init }));
+}
+
+function rows(): string[] {
+  return [...document.querySelectorAll(".help-row")].map((r) =>
+    [...r.querySelectorAll("kbd")].map((k) => k.textContent).join(" "),
+  );
+}
+
+describe("help overlay", () => {
+  it("renders every registered shortcut, grouped", () => {
+    openHelp();
+    const groups = [...document.querySelectorAll(".help-group-title")].map((h) => h.textContent);
+    expect(groups).toEqual(SHORTCUTS.map((g) => g.title));
+    expect(rows()).toEqual(SHORTCUTS.flatMap((g) => g.items.map((s) => s.keys.join(" "))));
+    expect(document.querySelectorAll(".help-desc")).toHaveLength(
+      SHORTCUTS.reduce((n, g) => n + g.items.length, 0),
+    );
+  });
+
+  it("opens on ? and closes on a second ?", () => {
+    press("?", { shiftKey: true });
+    expect(isHelpOpen()).toBe(true);
+    press("?", { shiftKey: true });
+    expect(isHelpOpen()).toBe(false);
+  });
+
+  it("closes on Escape, on the close button, and on a click outside the panel", () => {
+    openHelp();
+    press("Escape");
+    expect(isHelpOpen()).toBe(false);
+
+    openHelp();
+    (document.querySelector(".help-close") as HTMLButtonElement).click();
+    expect(isHelpOpen()).toBe(false);
+
+    openHelp();
+    (document.querySelector(".help-overlay") as HTMLElement).dispatchEvent(
+      new MouseEvent("click", { bubbles: true }),
+    );
+    expect(isHelpOpen()).toBe(false);
+  });
+
+  it("stays open for a click inside the panel", () => {
+    openHelp();
+    (document.querySelector(".help-panel") as HTMLElement).dispatchEvent(
+      new MouseEvent("click", { bubbles: true }),
+    );
+    expect(isHelpOpen()).toBe(true);
+  });
+
+  it("opening twice leaves one overlay", () => {
+    openHelp();
+    openHelp();
+    expect(document.querySelectorAll(".help-overlay")).toHaveLength(1);
+  });
+
+  it("ignores ? while typing and when a non-shift modifier is held", () => {
+    const input = document.createElement("input");
+    document.body.appendChild(input);
+    input.dispatchEvent(new KeyboardEvent("keydown", { key: "?", bubbles: true, shiftKey: true }));
+    expect(isHelpOpen()).toBe(false);
+
+    press("?", { ctrlKey: true });
+    expect(isHelpOpen()).toBe(false);
+  });
+
+  it("installing the keybind twice still toggles once", () => {
+    installHelpKeybind();
+    press("?", { shiftKey: true });
+    expect(isHelpOpen()).toBe(true);
+    expect(document.querySelectorAll(".help-overlay")).toHaveLength(1);
+  });
+});
+
+describe("isTypingTarget", () => {
+  it("is true for form fields and editable nodes, false otherwise", () => {
+    const el = (html: string) => {
+      const d = document.createElement("div");
+      d.innerHTML = html;
+      return d.firstElementChild as HTMLElement;
+    };
+    expect(isTypingTarget(el("<input>"))).toBe(true);
+    expect(isTypingTarget(el("<textarea></textarea>"))).toBe(true);
+    expect(isTypingTarget(el("<select></select>"))).toBe(true);
+    expect(isTypingTarget(el('<div contenteditable="true"></div>'))).toBe(true);
+    expect(isTypingTarget(el("<button></button>"))).toBe(false);
+    expect(isTypingTarget(null)).toBe(false);
+  });
+});

web/src/help.ts

@@ -0,0 +1,172 @@
+/** Keyboard-shortcut overlay (`?`), the web counterpart of the TUI's help
+ *  overlay. One registry drives both the rendering and what we claim is bound,
+ *  so a binding added to `main.ts` without an entry here reads as missing. */
+
+export interface Shortcut {
+  /** Keycaps, rendered left to right as separate <kbd>. */
+  keys: string[];
+  description: string;
+}
+
+export interface ShortcutGroup {
+  title: string;
+  items: Shortcut[];
+}
+
+export const SHORTCUTS: ShortcutGroup[] = [
+  {
+    title: "Sessions",
+    items: [
+      { keys: ["n"], description: "Open a spawn draft pane" },
+      { keys: ["N"], description: "Spawn a session with every default resolved" },
+    ],
+  },
+  {
+    title: "Session list",
+    items: [
+      {
+        keys: ["Ctrl", "Shift", "H"],
+        description: "Move the keyboard between the session list and the pane",
+      },
+      { keys: ["j"], description: "Highlight the next session" },
+      { keys: ["k"], description: "Highlight the previous session" },
+      { keys: ["Enter"], description: "Open the highlighted session, or expand a folder" },
+      { keys: ["Esc"], description: "Hand the keyboard back to the pane" },
+    ],
+  },
+  {
+    title: "Help",
+    items: [
+      { keys: ["?"], description: "Show or hide this help" },
+      { keys: ["Esc"], description: "Close the help overlay or an open menu" },
+    ],
+  },
+  {
+    title: "Debug",
+    items: [{ keys: ["Ctrl", "Shift", "D"], description: "Toggle the debug HUD" }],
+  },
+];
+
+/** True when the event target swallows plain keystrokes — a form field, or the
+ *  hidden textarea xterm types into. Shared with `main.ts` so every global
+ *  single-key binding is deaf in the same places. */
+export function isTypingTarget(target: EventTarget | null): boolean {
+  const el = target as HTMLElement | null;
+  if (!el || typeof el.tagName !== "string") return false;
+  return (
+    el.tagName === "INPUT" ||
+    el.tagName === "TEXTAREA" ||
+    el.tagName === "SELECT" ||
+    el.isContentEditable
+  );
+}
+
+let open: { el: HTMLElement; dispose: () => void } | null = null;
+
+export function isHelpOpen(): boolean {
+  return open !== null;
+}
+
+export function closeHelp(): void {
+  open?.dispose();
+}
+
+export function toggleHelp(): void {
+  if (open) closeHelp();
+  else openHelp();
+}
+
+/** Mount the overlay. Opening while already open is a no-op re-render, so a
+ *  stray second call can never stack two dialogs. */
+export function openHelp(): HTMLElement {
+  closeHelp();
+
+  const el = document.createElement("div");
+  el.className = "help-overlay";
+  el.setAttribute("role", "dialog");
+  el.setAttribute("aria-modal", "true");
+  el.setAttribute("aria-label", "Keyboard shortcuts");
+
+  const panel = document.createElement("div");
+  panel.className = "help-panel";
+
+  const head = document.createElement("header");
+  head.className = "help-head";
+  const title = document.createElement("h2");
+  title.className = "help-title";
+  title.textContent = "Keyboard shortcuts";
+  const close = document.createElement("button");
+  close.type = "button";
+  close.className = "help-close";
+  close.setAttribute("aria-label", "Close");
+  close.textContent = "✕";
+  close.addEventListener("click", closeHelp);
+  head.append(title, close);
+  panel.appendChild(head);
+
+  for (const group of SHORTCUTS) {
+    const section = document.createElement("section");
+    section.className = "help-group";
+    const heading = document.createElement("h3");
+    heading.className = "help-group-title";
+    heading.textContent = group.title;
+    const list = document.createElement("dl");
+    list.className = "help-list";
+    for (const item of group.items) {
+      const row = document.createElement("div");
+      row.className = "help-row";
+      const keys = document.createElement("dt");
+      keys.className = "help-keys";
+      for (const key of item.keys) {
+        const cap = document.createElement("kbd");
+        cap.textContent = key;
+        keys.appendChild(cap);
+      }
+      const desc = document.createElement("dd");
+      desc.className = "help-desc";
+      desc.textContent = item.description;
+      row.append(keys, desc);
+      list.appendChild(row);
+    }
+    section.append(heading, list);
+    panel.appendChild(section);
+  }
+
+  // A press on the scrim closes; one inside the panel is left alone.
+  el.addEventListener("click", (ev) => {
+    if (!panel.contains(ev.target as Node)) closeHelp();
+  });
+  el.appendChild(panel);
+  document.body.appendChild(el);
+
+  const dispose = () => {
+    if (open?.el !== el) return;
+    open = null;
+    el.remove();
+  };
+  open = { el, dispose };
+  close.focus();
+  return el;
+}
+
+let installed = false;
+
+/** Global keybind: `?` toggles the overlay, Escape closes it. Idempotent, so a
+ *  second call during a hot reload doesn't double-toggle. */
+export function installHelpKeybind(): void {
+  if (installed) return;
+  installed = true;
+  window.addEventListener("keydown", (e) => {
+    if (e.key === "Escape" && open) {
+      e.preventDefault();
+      closeHelp();
+      return;
+    }
+    // Shift is how `?` is typed on most layouts; the others would be a
+    // different binding entirely.
+    if (e.key !== "?" || e.ctrlKey || e.altKey || e.metaKey) return;
+    if (isTypingTarget(e.target)) return;
+    e.preventDefault();
+    toggleHelp();
+  });
+}

web/src/main.ts

@@ -1,15 +1,18 @@
+import "@fontsource/jetbrains-mono/400.css";
+import "@fontsource/jetbrains-mono/500.css";
+import "@fontsource/jetbrains-mono/600.css";
+import "@fontsource/jetbrains-mono/700.css";
 import { renderSidebar, renderTopbar } from "./catalog.js";
 import { ConnectionPool } from "./connections.js";
 import { installHudKeybind } from "./debug.js";
+import { installHelpKeybind, isHelpOpen, isTypingTarget } from "./help.js";
 import { CatalogStore } from "./events.js";
 import { shouldRetryClose } from "./reconnect.js";
 import {
   createSession,
   fetchNames,
   getDaemonInfo,
-  listWorktrees,
   projects as fetchProjects,
-  reclaimWorktree,
 } from "./api.js";
 import {
   buildFastForwardRequest,
@@ -17,20 +20,21 @@ import {
   resolveFastForward,
 } from "./fastForward.js";
 import { FALLBACK_PRESETS } from "./spawnForm.js";
+import { spawnSimilarSeed } from "./spawnSimilar.js";
 import { mountHud, type Hud } from "./hud.js";
 import { emptyLayout, type Layout, openSessions } from "./layout.js";
 import { createMultiView, type MultiView } from "./multiview.js";
 import { loadLayout, saveLayout } from "./persistence.js";
-import { OPEN_SPAWN_EVENT } from "./catalog.js";
+import { FOCUS_PANE_EVENT, OPEN_SPAWN_EVENT } from "./catalog.js";
 import { initTheme } from "./theme.js";
 import { sessionName, type Project, type Session, type SpawnPreset } from "./types.js";
-import { createWorktreesView, type WorktreesViewHandle } from "./worktreesView.js";
 import { installViewportSizing } from "./viewport.js";
 import "./styles.css";
 
 initTheme();
 installViewportSizing();
 installHudKeybind();
+installHelpKeybind();
 registerServiceWorker();
 
 // Register the PWA service worker so the app is installable and the shell loads
@@ -222,6 +226,7 @@ function renderWelcome(): HTMLElement {
       Pick a session from the list, or start a new one. Open sessions keep
       streaming in the background — switching back is instant.
     </p>
+    <p class="empty-hint">Press <kbd>?</kbd> for keyboard shortcuts.</p>
     <div class="empty-actions"></div>`;
   const spawn = document.createElement("button");
   spawn.type = "button";
@@ -268,8 +273,8 @@ function showMultiView(): void {
 }
 
 // Spawn affordances (#62): the header button, the sidebar's empty state, and
-// `?mock&spawn` all open a draft pane; "Spawn similar" seeds one with the
-// cloned session's dir + command.
+// `?mock&spawn` all open a draft pane; "Spawn similar" seeds one from the
+// cloned session (see spawnSimilar.ts).
 function requestOpenSpawn(): void {
   showMultiView();
   // A new tab, not a split of the pane you were reading (#100). "Spawn
@@ -282,7 +287,7 @@ document.addEventListener(OPEN_SPAWN_EVENT, () => requestOpenSpawn());
 document.addEventListener("catalog-spawn-similar", (ev) => {
   const s = (ev as CustomEvent<Session>).detail;
   showMultiView();
-  mv.openDraft({ cwd: s.cwd, command: s.command });
+  mv.openDraft(spawnSimilarSeed(s));
 });
 
 // Fast-forward spawn (#67): the draft pane's keystroke neighbour spawns
@@ -330,16 +335,9 @@ async function requestFastForward(): Promise<void> {
 // textarea, search) so an attached session's keys are never hijacked.
 document.addEventListener("keydown", (e) => {
   if (e.ctrlKey || e.altKey || e.metaKey) return;
-  const target = e.target as HTMLElement | null;
-  if (
-    target &&
-    (target.tagName === "INPUT" ||
-      target.tagName === "TEXTAREA" ||
-      target.tagName === "SELECT" ||
-      target.isContentEditable)
-  ) {
-    return;
-  }
+  if (isTypingTarget(e.target)) return;
+  // The help overlay owns the keyboard while it is up.
+  if (isHelpOpen()) return;
   if (e.key === "n") {
     e.preventDefault();
     requestOpenSpawn();
@@ -349,50 +347,38 @@ document.addEventListener("keydown", (e) => {
   }
 });
 
+// Ctrl+Shift+H moves the keyboard between the session list and the pane, the
+// web counterpart of the TUI's `ctrl-w h`. It has to be a chord: an attached
+// terminal owns every plain keystroke, so nothing unmodified could ever get
+// the focus back out of it.
+window.addEventListener("keydown", (e) => {
+  if (!e.ctrlKey || !e.shiftKey || e.altKey || e.metaKey) return;
+  if (e.key !== "H" && e.key !== "h") return;
+  e.preventDefault();
+  if (sidebar.hasListFocus()) {
+    pool.focusTerminal();
+  } else {
+    openDrawer(); // no-op on desktop; on mobile the list is off-canvas
+    sidebar.focusList();
+  }
+});
+
+// Escape in the session list hands the keyboard back to the pane.
+document.addEventListener(FOCUS_PANE_EVENT, () => pool.focusTerminal());
+
 type Route =
   | { kind: "catalog" }
   | { kind: "attach"; sessionId: string }
-  | { kind: "worktrees" }
   | { kind: "not-found"; raw: string };
 
 function parseRoute(): Route {
   const hash = location.hash.replace(/^#/, "");
   if (hash === "" || hash === "/") return { kind: "catalog" };
-  if (hash === "/worktrees") return { kind: "worktrees" };
   const m = hash.match(/^\/attach\/([^/?#]+)$/);
   if (m) return { kind: "attach", sessionId: decodeURIComponent(m[1]) };
   return { kind: "not-found", raw: hash };
 }
 
-// Retained worktrees (#84): built lazily on first visit, then kept so a
-// revisit re-enters instantly. The daemon's home dir abbreviates paths once
-// `/api/info` answers.
-let worktreesView: WorktreesViewHandle | undefined;
-let homeDir: string | undefined;
-void getDaemonInfo()
-  .then((info) => {
-    homeDir = info.home_dir;
-  })
-  .catch(() => {
-    /* absolute paths are a fine fallback */
-  });
-
-function showWorktrees(): void {
-  pool.setVisible([]);
-  pool.setFocus(null);
-  contentShowsMv = false;
-  if (!worktreesView) {
-    worktreesView = createWorktreesView({
-      load: listWorktrees,
-      reclaim: reclaimWorktree,
-      homeDir: () => homeDir,
-    });
-  }
-  content.replaceChildren(worktreesView.el);
-  shellEl.dataset.route = "worktrees";
-  void worktreesView.refresh();
-}
-
 function dispatch(): void {
   const route = parseRoute();
   switch (route.kind) {
@@ -414,9 +400,6 @@ function dispatch(): void {
       // tears panes down.
       showMultiView();
       return;
-    case "worktrees":
-      showWorktrees();
-      return;
   }
 }
 

web/src/mock.ts

@@ -14,6 +14,7 @@ function sess(
   status: "Running" | "Dead",
   cwd: string,
   name?: string,
+  blocked_work?: Session["blocked_work"],
 ): Session {
   const seed = parseInt(id.slice(-1), 16) || 0;
   return {
@@ -30,6 +31,7 @@ function sess(
     last_activity_at: new Date(Date.now() - seed * 11_000).toISOString(),
     // The session tree nests by current_cwd; default it to the spawn dir.
     current_cwd: cwd,
+    blocked_work,
   };
 }
 
@@ -109,16 +111,26 @@ function stubSpawnApi(): void {
 export function seedMock(store: CatalogStore): void {
   const sessions: Session[] = [
     // ~/git/maestro/web/* — deep dir chain; "~/git/maestro" collapses to one node.
-    sess("a1b2c3d4-0000-0000-0000-000000000001", "claude", "Running", "/home/dev/git/maestro/web", "redesign"),
+    sess("a1b2c3d4-0000-0000-0000-000000000001", "claude", "Running", "/home/dev/git/maestro/web", "redesign", {
+      uncommitted_files: 4,
+      untracked_files: 2,
+      unpushed_commits: 1,
+      insertions: 312,
+      deletions: 47,
+    }),
     sess("a1b2c3d4-0000-0000-0000-000000000002", "vite dev", "Running", "/home/dev/git/maestro/web", "dev-server"),
     sess("a1b2c3d4-0000-0000-0000-000000000003", "bash", "Dead", "/home/dev/git/maestro/web", "scratch"),
-    sess("a1b2c3d4-0000-0000-0000-000000000004", "claude", "Running", "/home/dev/git/maestro/web/tests", "e2e-fixes"),
+    sess("a1b2c3d4-0000-0000-0000-000000000004", "codex --resume", "Running", "/home/dev/git/maestro/web/tests", "e2e-fixes", {
+      uncommitted_files: 1,
+    }),
     // ~/git/maestro/crates/* — sibling subtree under the same repo.
     sess("a1b2c3d4-0000-0000-0000-000000000005", "claude", "Running", "/home/dev/git/maestro/crates/daemon", "daemon-work"),
     sess("a1b2c3d4-0000-0000-0000-000000000006", "cargo watch -x test", "Running", "/home/dev/git/maestro/crates/daemon", "test-loop"),
     sess("a1b2c3d4-0000-0000-0000-000000000007", "cargo run -- daemon", "Running", "/home/dev/git/maestro/crates/daemon", "daemon"),
     // ~/work/acme/* — a separate top-level dir on the same host.
-    sess("a1b2c3d4-0000-0000-0000-000000000008", "claude", "Running", "/home/dev/work/acme/api", "billing-api"),
+    sess("a1b2c3d4-0000-0000-0000-000000000008", "gemini", "Running", "/home/dev/work/acme/api", "billing-api", {
+      untracked_files: 3,
+    }),
     sess("a1b2c3d4-0000-0000-0000-000000000009", "pnpm dev", "Dead", "/home/dev/work/acme/web", "dashboard"),
     // ~/git/dotfiles — single session, fully collapsed chain.
     sess("a1b2c3d4-0000-0000-0000-000000000010", "opencode", "Running", "/home/dev/git/dotfiles", "dotfiles"),

web/src/multiview.test.ts

@@ -1,7 +1,7 @@
 // @vitest-environment happy-dom
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 
-import { emptyLayout, leaves, type Layout } from "./layout.js";
+import { addTab, emptyLayout, leaves, type Layout } from "./layout.js";
 import { splitLeaf } from "./layout.js";
 import { createMultiView, type MultiView } from "./multiview.js";
 import type { ConnectionPool } from "./connections.js";
@@ -36,6 +36,10 @@ function fakeSidebar(): SidebarHandle {
     el: document.createElement("aside"),
     setActive() {},
     setHighlights() {},
+    focusList() {},
+    hasListFocus() {
+      return false;
+    },
     dispose() {},
   };
 }
@@ -333,3 +337,98 @@ describe("openDraft newTab (#100)", () => {
     expect(leaves(tab.root)).toHaveLength(2);
   });
 });
+
+describe("multiview closeSession tab cleanup (#137)", () => {
+  beforeEach(() => {
+    stubFetch();
+    document.body.replaceChildren();
+  });
+
+  afterEach(() => {
+    vi.unstubAllGlobals();
+  });
+
+  it("closes the tab when its last session goes away", () => {
+    const layout = emptyLayout();
+    const first = layout.tabs[0];
+    first.focusedLeafId = splitLeaf(first, "", "s1", "row", true);
+    const second = addTab(layout);
+    second.focusedLeafId = splitLeaf(second, "", "s2", "row", true);
+    const mv = makeMv(layout);
+    document.body.appendChild(mv.el);
+    mv.render();
+
+    mv.closeSession("s2");
+
+    expect(layout.tabs).toHaveLength(1);
+    expect(layout.tabs[0].id).toBe(first.id);
+    expect(layout.activeTabId).toBe(first.id);
+  });
+
+  it("closes a background tab without stealing focus from the active one", () => {
+    const layout = emptyLayout();
+    const background = layout.tabs[0];
+    background.focusedLeafId = splitLeaf(background, "", "s1", "row", true);
+    const active = addTab(layout);
+    active.focusedLeafId = splitLeaf(active, "", "s2", "row", true);
+    const mv = makeMv(layout);
+    document.body.appendChild(mv.el);
+    mv.render();
+
+    mv.closeSession("s1");
+
+    expect(layout.tabs).toHaveLength(1);
+    expect(layout.tabs[0].id).toBe(active.id);
+    expect(layout.activeTabId).toBe(active.id);
+  });
+
+  it("keeps a tab that still holds another pane", () => {
+    const layout = emptyLayout();
+    const tab = layout.tabs[0];
+    const first = splitLeaf(tab, "", "s1", "row", true);
+    tab.focusedLeafId = splitLeaf(tab, first, "s2", "row", true);
+    addTab(layout);
+    layout.activeTabId = tab.id;
+    const mv = makeMv(layout);
+    document.body.appendChild(mv.el);
+    mv.render();
+
+    mv.closeSession("s2");
+
+    expect(layout.tabs).toHaveLength(2);
+    expect(leaves(tab.root)).toHaveLength(1);
+  });
+
+  it("keeps a tab holding a draft pane", async () => {
+    const layout = emptyLayout();
+    const tab = layout.tabs[0];
+    const s1 = splitLeaf(tab, "", "s1", "row", true);
+    splitLeaf(tab, s1, null, "row", true);
+    addTab(layout);
+    layout.activeTabId = tab.id;
+    const mv = makeMv(layout);
+    document.body.appendChild(mv.el);
+    mv.render();
+    await flush();
+
+    mv.closeSession("s1");
+
+    expect(layout.tabs).toHaveLength(2);
+    expect(leaves(tab.root)).toHaveLength(1);
+  });
+
+  it("never closes the only tab", () => {
+    const layout = emptyLayout();
+    const tab = layout.tabs[0];
+    tab.focusedLeafId = splitLeaf(tab, "", "s1", "row", true);
+    const mv = makeMv(layout);
+    document.body.appendChild(mv.el);
+    mv.render();
+
+    mv.closeSession("s1");
+
+    expect(layout.tabs).toHaveLength(1);
+    expect(layout.tabs[0].root).toBeNull();
+    expect(layout.activeTabId).toBe(tab.id);
+  });
+});

web/src/multiview.ts

@@ -22,7 +22,6 @@ import {
   removeLeaf,
   splitLeaf,
 } from "./layout.js";
-import { sessionColor } from "./sessioncolor.js";
 import type { AttachInfo } from "./attach.js";
 import { createSpawnForm, type SpawnFormHandle } from "./spawnForm.js";
 
@@ -97,12 +96,6 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
 
   // ---- highlight helpers ----
 
-  function colorMap(): Map<string, string> {
-    const m = new Map<string, string>();
-    for (const id of openSessions(layout)) m.set(id, sessionColor(id));
-    return m;
-  }
-
   function focusedLeafSession(): string | null {
     const tab = activeTab(layout);
     const fl = tab.focusedLeafId ? findLeaf(tab.root, tab.focusedLeafId) : null;
@@ -120,7 +113,6 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
     sidebar.setHighlights({
       open: openSessions(layout),
       focused: focusedLeafSession(),
-      colors: colorMap(),
     });
   }
 
@@ -142,10 +134,7 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
     const pane = document.createElement("div");
     pane.className = "mv-pane";
     pane.dataset.leafId = leaf.id;
-    if (leaf.sessionId) {
-      pane.dataset.sessionId = leaf.sessionId;
-      pane.style.setProperty("--mv-color", sessionColor(leaf.sessionId));
-    }
+    if (leaf.sessionId) pane.dataset.sessionId = leaf.sessionId;
 
     const host = document.createElement("div");
     host.className = "mv-pane-host";
@@ -482,7 +471,7 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
       pool.setVisible([]);
       pool.setFocus(null);
       canvas.appendChild(renderWelcome());
-      sidebar.setHighlights({ open: openSessions(layout), focused: null, colors: colorMap() });
+      sidebar.setHighlights({ open: openSessions(layout), focused: null });
       persist();
       return;
     }
@@ -523,7 +512,6 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
         pill.type = "button";
         pill.className = "mv-pill" + (l.id === focused.id ? " is-active" : "");
         pill.textContent = l.sessionId ? (infoFor(l.sessionId)?.name ?? l.sessionId.slice(0, 8)) : "empty";
-        if (l.sessionId) pill.style.setProperty("--mv-color", sessionColor(l.sessionId));
         pill.addEventListener("click", () => {
           tab.focusedLeafId = l.id;
           render();
@@ -543,7 +531,7 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
       focused.sessionId ? [{ id: focused.sessionId, pane: pane.host, info: infoFor(focused.sessionId) }] : [],
     );
     pool.setFocus(focused.sessionId ?? null);
-    sidebar.setHighlights({ open: openSessions(layout), focused: focused.sessionId ?? null, colors: colorMap() });
+    sidebar.setHighlights({ open: openSessions(layout), focused: focused.sessionId ?? null });
   }
 
   function closeSession(sessionId: string): void {
@@ -553,6 +541,10 @@ export function createMultiView(opts: MultiViewOptions): MultiView {
     if (loc.tab.focusedLeafId === loc.leaf.id) {
       loc.tab.focusedLeafId = leaves(loc.tab.root)[0]?.id ?? null;
     }
+    // A tab that held nothing but the dead session goes with it (#137): a killed
+    // session should not leave an empty tab behind for the operator to close by
+    // hand. `closeTab` keeps the last tab (emptied) so there is always one.
+    if (!loc.tab.root) closeTab(layout, loc.tab.id);
     render();
   }
 

web/src/routes.ts

@@ -13,7 +13,6 @@ export const PROJECTS = "/api/projects";
 export const GITROOT = "/api/gitroot";
 export const NAMES = "/api/names";
 export const BRANCHES = "/api/branches";
-export const WORKTREES = "/api/worktrees";
 export const EVENTS = "/api/events";
 
 /** Name of the query parameter `DIRS` and `GITROOT` both take. */
@@ -30,11 +29,6 @@ export function attach(id: Uuid): string {
   return `${API_PREFIX}/attach/${encodeURIComponent(id)}`;
 }
 
-/** `DELETE /api/worktrees/:id` — reclaim one retained worktree (#80). */
-export function worktree(id: Uuid): string {
-  return `${WORKTREES}/${encodeURIComponent(id)}`;
-}
-
 /** Append the `path` query parameter `DIRS` and `GITROOT` accept. */
 export function withPathQuery(route: string, value: string): string {
   return `${route}?${PATH_QUERY}=${encodeURIComponent(value)}`;

web/src/sessioncolor.ts

@@ -1,23 +0,0 @@
-// Stable per-session accent colour, used to map a sidebar row to the pane
-// showing that session (issue #33). Hash the session id into a fixed OKLCH
-// palette tuned to the maestro tokens; collisions are tolerated when there are
-// more open sessions than hues — the colour is a hint, not an identity.
-
-import type { Uuid } from "./types.js";
-
-const PALETTE = [
-  "oklch(0.80 0.13 152)", // green (the house accent)
-  "oklch(0.80 0.12 90)", // amber
-  "oklch(0.76 0.13 30)", // coral
-  "oklch(0.78 0.11 210)", // teal
-  "oklch(0.74 0.13 265)", // blue
-  "oklch(0.76 0.14 320)", // magenta
-  "oklch(0.80 0.11 60)", // gold
-  "oklch(0.78 0.12 185)", // cyan
-];
-
-export function sessionColor(id: Uuid): string {
-  let h = 0;
-  for (let i = 0; i < id.length; i++) h = (h * 31 + id.charCodeAt(i)) >>> 0;
-  return PALETTE[h % PALETTE.length];
-}

web/src/sidebarNav.test.ts

@@ -0,0 +1,128 @@
+// @vitest-environment happy-dom
+import { beforeAll, describe, expect, it } from "vitest";
+
+import { FOCUS_PANE_EVENT, renderSidebar } from "./catalog.js";
+import type { CatalogListener, CatalogStore } from "./events.js";
+import type { Session } from "./types.js";
+
+beforeAll(() => {
+  Object.defineProperty(window, "localStorage", {
+    value: { getItem: () => null, setItem: () => {} },
+  });
+  window.fetch = () => Promise.reject(new Error("offline"));
+});
+
+function session(id: string, cwd: string): Session {
+  return {
+    id,
+    cwd,
+    command: "claude",
+    env: {},
+    metadata: {},
+    status: "Running",
+    created_at: "2026-08-24T10:00:00Z",
+  } as Session;
+}
+
+function storeWith(sessions: Session[]): CatalogStore {
+  const state = { sessions: new Map(sessions.map((s) => [s.id, s])) };
+  return {
+    snapshot: () => state,
+    subscribe: (fn: CatalogListener) => {
+      fn(state);
+      return () => {};
+    },
+  } as unknown as CatalogStore;
+}
+
+const A = "aaaaaaaa-0000-0000-0000-00000000000a";
+const B = "aaaaaaaa-0000-0000-0000-00000000000b";
+
+/** Two sessions in sibling directories: two folder headers, two cards. */
+function twoFolders(): { el: HTMLElement; handle: ReturnType<typeof renderSidebar> } {
+  const handle = renderSidebar(storeWith([session(A, "/one"), session(B, "/two")]));
+  // Mounted: `focusList` moves real DOM focus, which needs a live document.
+  document.body.replaceChildren(handle.el);
+  return { el: handle.el, handle };
+}
+
+function press(el: HTMLElement, key: string, init: KeyboardEventInit = {}): void {
+  el.dispatchEvent(new KeyboardEvent("keydown", { key, bubbles: true, ...init }));
+}
+
+function highlighted(el: HTMLElement): string | null {
+  return el.querySelector<HTMLElement>(".is-highlighted")?.dataset.navKey ?? null;
+}
+
+function tree(el: HTMLElement): HTMLElement {
+  return el.querySelector<HTMLElement>(".tree")!;
+}
+
+describe("sidebar keyboard navigation", () => {
+  it("walks the cards and never stops on an expanded folder", () => {
+    const { el } = twoFolders();
+    // The folders are there — the walk just refuses to land on them.
+    expect(el.querySelectorAll(".tree-folder-header")).toHaveLength(2);
+
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe(`session:${A}`);
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe(`session:${B}`);
+    // Wraps.
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe(`session:${A}`);
+    press(tree(el), "k");
+    expect(highlighted(el)).toBe(`session:${B}`);
+  });
+
+  it("accepts the arrow keys as well as j/k", () => {
+    const { el } = twoFolders();
+    press(tree(el), "ArrowDown");
+    expect(highlighted(el)).toBe(`session:${A}`);
+    press(tree(el), "ArrowUp");
+    // Backwards from the first card wraps to the last.
+    expect(highlighted(el)).toBe(`session:${B}`);
+  });
+
+  it("stops on a collapsed folder, since its cards are gone", () => {
+    const { el } = twoFolders();
+    const header = el.querySelectorAll<HTMLElement>(".tree-folder-header")[0];
+    header.click(); // collapse /one
+
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe("folder:one");
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe(`session:${B}`);
+
+    // Enter expands it again and lands on the card it was hiding.
+    press(tree(el), "j");
+    expect(highlighted(el)).toBe("folder:one");
+    press(tree(el), "Enter");
+    expect(highlighted(el)).toBe(`session:${A}`);
+  });
+
+  it("focusList starts from the session on screen", () => {
+    const { el, handle } = twoFolders();
+    handle.setHighlights({ open: new Set([B]), focused: B });
+    handle.focusList();
+    expect(highlighted(el)).toBe(`session:${B}`);
+    expect(handle.hasListFocus()).toBe(true);
+  });
+
+  it("Escape asks the shell for the pane back", () => {
+    const { el } = twoFolders();
+    let asked = 0;
+    document.addEventListener(FOCUS_PANE_EVENT, () => (asked += 1));
+    press(tree(el), "Escape");
+    expect(asked).toBe(1);
+  });
+
+  it("leaves the keyboard alone while a row is being renamed", () => {
+    const { el } = twoFolders();
+    press(tree(el), "j");
+    const input = document.createElement("input");
+    tree(el).appendChild(input);
+    input.dispatchEvent(new KeyboardEvent("keydown", { key: "j", bubbles: true }));
+    expect(highlighted(el)).toBe(`session:${A}`);
+  });
+});

web/src/spawnForm.ts

@@ -140,10 +140,6 @@ export function createSpawnForm(opts: SpawnFormOptions): SpawnFormHandle {
                placeholder="/home/user/project" inputmode="text" autocomplete="off" />
       </div>
     </div>
-    <div class="field wt-step" hidden>
-      <span class="field-label">Worktree</span>
-      <div class="picker wt-choices"></div>
-    </div>
     <div class="field">
       <span class="field-label">Name</span>
       <div class="name-row">
@@ -158,6 +154,10 @@ export function createSpawnForm(opts: SpawnFormOptions): SpawnFormHandle {
         <span class="wt-summary"></span>
       </button>
       <div class="wt-collapsible" hidden>
+        <div class="field wt-step" hidden>
+          <span class="field-label">Worktree</span>
+          <div class="picker wt-choices"></div>
+        </div>
         <label class="row-inline">
           <input type="checkbox" name="worktree_enabled" />
           Create git worktree
@@ -589,7 +589,10 @@ export function createSpawnForm(opts: SpawnFormOptions): SpawnFormHandle {
         cwd = seedCwd;
         cwdRaw.value = seedCwd;
         // Restoring must not touch the worktree toggle — the seed carries it.
-        if (!restoreSelection(seedCwd)) clearProjectSelection();
+        // A path no box matches (the project is gone, or the scan never saw
+        // it) falls back to the raw input showing that path: an empty picker
+        // with an invisible cwd reads as a form that lost the directory.
+        if (!restoreSelection(seedCwd)) showRawCwd(seedCwd, false);
       } else {
         showRawCwd(seedCwd, false);
       }

web/src/spawnSimilar.test.ts

@@ -0,0 +1,45 @@
+import { describe, expect, it } from "vitest";
+import { spawnSimilarSeed } from "./spawnSimilar.js";
+import type { Session } from "./types.js";
+
+function session(overrides: Partial<Session> = {}): Session {
+  return {
+    id: "s1",
+    cwd: "/repos/app",
+    command: "claude",
+    env: {},
+    metadata: {},
+    status: "Running",
+    created_at: "2026-01-01T00:00:00Z",
+    ...overrides,
+  } as Session;
+}
+
+describe("spawnSimilarSeed (#28)", () => {
+  it("seeds the project picker, not the free-text hatch", () => {
+    const seed = spawnSimilarSeed(session());
+    expect(seed.cwd).toBe("/repos/app");
+    expect(seed.command).toBe("claude");
+    expect(seed.cwdRawVisible).toBe(false);
+  });
+
+  it("keeps the worktree toggle off for a session that has no worktree", () => {
+    expect(spawnSimilarSeed(session()).worktreeEnabled).toBe(false);
+  });
+
+  it("clones a worktree session as a new worktree of its base repo", () => {
+    const seed = spawnSimilarSeed(
+      session({
+        cwd: "/repos/app-worktrees/feature",
+        worktree: { path: "/repos/app-worktrees/feature", base_repo: "/repos/app" },
+      }),
+    );
+    expect(seed.cwd).toBe("/repos/app");
+    expect(seed.worktreeEnabled).toBe(true);
+  });
+
+  it("never carries the cloned session's name", () => {
+    const seed = spawnSimilarSeed(session({ metadata: { name: "amber-falcon" } }));
+    expect(seed.name).toBeUndefined();
+  });
+});

web/src/spawnSimilar.ts

@@ -0,0 +1,30 @@
+// "Spawn similar" (#28): the seed a cloned session hands to a draft pane.
+// Pure — the caller owns opening the pane.
+
+import type { Draft } from "./layout.js";
+import type { Session } from "./types.js";
+
+/** Clone a session into draft values: same project, same command, same
+ *  worktree decision, fresh name.
+ *
+ *  A worktree session seeds its *base repository*, not the checkout the daemon
+ *  made for it: "similar" is another session of that project, and the daemon
+ *  never adopts an existing worktree (docs/WORKTREES.md), so seeding the
+ *  checkout would ask it to build a worktree out of a worktree. The picker
+ *  then lands on that repository's box with "+ new" active.
+ *
+ *  `cwdRawVisible: false` keeps the two-step project picker on screen with the
+ *  cloned project selected, instead of dropping the path into the free-text
+ *  escape hatch.
+ *
+ *  The name is left out on purpose: a clone is a new session and takes its own
+ *  generated handle. */
+export function spawnSimilarSeed(s: Session): Partial<Draft> {
+  const wt = s.worktree ?? null;
+  return {
+    command: s.command,
+    cwd: wt ? wt.base_repo : s.cwd,
+    worktreeEnabled: wt != null,
+    cwdRawVisible: false,
+  };
+}

web/src/styles.css

@@ -8,7 +8,7 @@
 :root {
   /* ---- type ---- */
   --font-mono:
-    ui-monospace, "SF Mono", "SFMono-Regular", "Cascadia Code", "JetBrains Mono",
+    "JetBrains Mono", ui-monospace, "SF Mono", "SFMono-Regular", "Cascadia Code",
     "Roboto Mono", Menlo, Consolas, "Liberation Mono", monospace;
   --font-sans:
     system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto,
@@ -764,6 +764,18 @@ textarea {
   box-shadow: inset 0 0 0 1px var(--accent-ring);
 }
 
+/* Where the keyboard walk (j/k) is standing. Distinct from `.is-active`: this
+   is a cursor, not a statement about what is mounted, so it reads as a ring
+   rather than a fill and the two can sit on the same row. */
+.tree-leaf.is-highlighted,
+.tree-folder-header.is-highlighted {
+  box-shadow: inset 0 0 0 2px var(--accent-ring);
+}
+
+.tree {
+  outline: none;
+}
+
 /* The session currently mounted in the content pane. */
 .tree-leaf.is-active,
 .tree-leaf.is-active[data-status="running"] {
@@ -876,6 +888,113 @@ input.leaf-rename:focus {
   background: var(--danger-tint);
 }
 
+/* ---------- help overlay (`?`) ---------- */
+.help-overlay {
+  position: fixed;
+  inset: 0;
+  z-index: var(--z-modal);
+  display: grid;
+  place-items: center;
+  padding: var(--sp-4);
+  background: oklch(0 0 0 / 0.5);
+  backdrop-filter: blur(4px);
+}
+
+.help-panel {
+  width: min(440px, 100%);
+  max-height: min(80vh, 640px);
+  overflow-y: auto;
+  padding: var(--sp-4);
+  background: var(--surface);
+  border: 1px solid var(--border-strong);
+  border-radius: var(--r-3);
+  box-shadow: var(--shadow-2);
+}
+
+.help-head {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  gap: var(--sp-2);
+  margin-bottom: var(--sp-3);
+}
+
+.help-title {
+  margin: 0;
+  font-family: var(--font-mono);
+  font-size: var(--fs-15);
+  color: var(--ink);
+}
+
+.help-close {
+  appearance: none;
+  background: none;
+  border: 0;
+  border-radius: var(--r-2);
+  color: var(--muted);
+  font: inherit;
+  line-height: 1;
+  padding: var(--sp-1) var(--sp-2);
+  cursor: pointer;
+}
+
+.help-close:hover,
+.help-close:focus-visible {
+  background: var(--surface-3);
+  color: var(--ink);
+  outline: none;
+}
+
+.help-group + .help-group {
+  margin-top: var(--sp-4);
+}
+
+.help-group-title {
+  margin: 0 0 var(--sp-2);
+  font-size: var(--fs-11);
+  font-weight: 600;
+  letter-spacing: 0.08em;
+  text-transform: uppercase;
+  color: var(--faint);
+}
+
+.help-list {
+  margin: 0;
+}
+
+/* Keycaps in a fixed column so the descriptions line up down the list. */
+.help-row {
+  display: grid;
+  grid-template-columns: 14ch 1fr;
+  align-items: baseline;
+  gap: var(--sp-3);
+  padding: var(--sp-1) 0;
+}
+
+.help-keys {
+  display: flex;
+  flex-wrap: nowrap;
+  gap: var(--sp-1);
+  margin: 0;
+}
+
+.help-keys kbd {
+  font-family: var(--font-mono);
+  font-size: var(--fs-11);
+  background: var(--surface-2);
+  border: 1px solid var(--border);
+  border-bottom-width: 2px;
+  border-radius: var(--r-1);
+  padding: 1px 6px;
+  color: var(--ink);
+}
+
+.help-desc {
+  margin: 0;
+  font-size: var(--fs-13);
+  color: var(--muted);
+}
+
 /* Line 2 — dot + duration in the state's colour, then muted fragments. The
    state word itself is never spelled out here; the colour carries it. */
 .card-line2 {
@@ -940,6 +1059,46 @@ input.leaf-rename:focus {
    truncation target when even the complete trailing set cannot fit. */
 .leaf-cmd {
   flex: 0 1 auto;
+  display: inline-flex;
+  align-items: center;
+  gap: 4px;
+  min-width: 0;
+}
+
+/* The tool's mark stands in for the program the command line starts with
+   (docs/SIDEBAR.md). It keeps its own colours where the tool has brand
+   colours, and inherits the muted fragment colour where it does not. */
+.tool-icon {
+  flex: 0 0 auto;
+  opacity: 0.9;
+}
+
+.leaf-cmd-rest {
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+/* Blocked work as diff notation: `+124 -18 ?3 ↑2` (docs/SIDEBAR.md). The
+   spelled-out breakdown is the tooltip; only the additions and deletions take
+   colour, so the ⚠-less fragment still reads as a diff and not as an alarm. */
+.leaf-blocked {
+  display: inline-flex;
+  align-items: center;
+  gap: 4px;
+  font-variant-numeric: tabular-nums;
+}
+
+.blocked-stat[data-stat="add"] {
+  color: var(--live);
+}
+.blocked-stat[data-stat="del"] {
+  color: var(--danger);
+}
+.blocked-stat[data-stat="files"],
+.blocked-stat[data-stat="untracked"],
+.blocked-stat[data-stat="unpushed"] {
+  color: var(--faint);
 }
 
 /* Actions live in line 1's flow at its right edge — never overlaid on the
@@ -2015,12 +2174,6 @@ input.leaf-rename:focus {
   color: var(--muted);
 }
 
-/* debug-HUD navbar toggle: lit when the HUD is on. */
-.debug-toggle.is-active {
-  color: var(--accent);
-  border-color: var(--accent);
-}
-
 /* =========================================================================
    multi-view (#30 splits, #31 tabs, #33 highlights)
    Tabs of nested split trees; each session leaf hosts a live .attach. The
@@ -2112,7 +2265,7 @@ input.leaf-rename:focus {
 .mv-canvas > .mv-split,
 .mv-canvas > .mv-pane {
   position: absolute;
-  inset: var(--sp-2);
+  inset: 0;
 }
 .mv-canvas > .content-welcome {
   position: absolute;
@@ -2200,20 +2353,10 @@ input.leaf-rename:focus {
   min-width: 0;
   min-height: 0;
 }
-/* per-session colour strip — maps a pane to its sidebar row */
-.mv-pane::before {
-  content: "";
-  position: absolute;
-  inset: 0 0 auto 0;
-  height: 2px;
-  background: var(--mv-color, transparent);
-  z-index: 3;
-  pointer-events: none;
-}
-/* focused pane: accent ring + glow, full strength */
+/* focused pane: accent ring, full strength */
 .mv-pane.is-focused {
   border-color: var(--accent);
-  box-shadow: inset 0 0 0 1px var(--accent), 0 0 22px -8px var(--accent-ring);
+  box-shadow: inset 0 0 0 1px var(--accent);
 }
 /* unfocused panes recede so the eye finds the live one */
 .mv-pane:not(.is-focused) {
@@ -2316,31 +2459,12 @@ input.leaf-rename:focus {
   border-radius: var(--r-1);
 }
 
-/* sidebar two-tier highlight + colour dot (issue #33).
+/* sidebar two-tier highlight (issue #33).
    .is-active = focused pane (loud, from existing rule); .is-open = open
-   elsewhere (quiet). The dot uses the session's colour so a row maps to a pane. */
+   elsewhere (quiet). */
 .tree-leaf.is-open {
   background: color-mix(in oklab, var(--accent) 6%, transparent);
 }
-.tree-leaf.is-open::after,
-.tree-leaf.is-active::after {
-  content: "";
-  position: absolute;
-  right: 9px;
-  top: 50%;
-  width: 7px;
-  height: 7px;
-  margin-top: -3.5px;
-  border-radius: 50%;
-  background: var(--mv-color, var(--accent));
-  pointer-events: none;
-}
-.tree-leaf.is-open::after {
-  opacity: 0.7;
-}
-.tree-leaf.is-active::after {
-  box-shadow: 0 0 8px var(--mv-color, var(--accent-ring));
-}
 
 /* =========================================================================
    multi-view mobile (#35): below 860px the split tree projects to a pill strip
@@ -2352,7 +2476,6 @@ input.leaf-rename:focus {
 
 .mv-mobile {
   position: absolute;
-  /* Flush to the edges — a single full-screen pane needs no canvas inset. */
   inset: 0;
   display: flex;
   flex-direction: column;
@@ -2386,14 +2509,6 @@ input.leaf-rename:focus {
   cursor: pointer;
   border: 1px solid transparent;
 }
-.mv-pill::before {
-  content: "";
-  width: 7px;
-  height: 7px;
-  border-radius: 50%;
-  background: var(--mv-color, var(--faint));
-  flex: none;
-}
 .mv-pill.is-active {
   background: var(--accent-tint);
   color: var(--accent-strong);
@@ -2439,148 +2554,3 @@ input.leaf-rename:focus {
     background: oklch(0 0 0 / 0.5);
   }
 }
-
-/* ---------- retained worktrees (#84) ---------- */
-
-.worktrees {
-  padding: var(--sp-4);
-  max-width: 960px;
-  margin: 0 auto;
-}
-
-.wt-head {
-  margin-bottom: var(--sp-4);
-}
-
-.wt-title {
-  font-family: var(--font-mono);
-  font-size: var(--fs-15);
-  font-weight: 600;
-  color: var(--ink);
-  margin: 0 0 var(--sp-1);
-}
-
-.wt-sub {
-  color: var(--muted);
-  font-size: var(--fs-13);
-  margin: 0;
-}
-
-.wt-row {
-  display: flex;
-  align-items: baseline;
-  gap: var(--sp-3);
-  padding: var(--sp-2) var(--sp-2);
-  border-bottom: 1px solid var(--surface-2);
-  font-family: var(--font-mono);
-  font-size: var(--fs-13);
-  flex-wrap: wrap;
-}
-
-.wt-branch {
-  color: var(--ink);
-  font-weight: 600;
-}
-
-.wt-path {
-  color: var(--muted);
-  overflow: hidden;
-  text-overflow: ellipsis;
-  white-space: nowrap;
-}
-
-.wt-reason {
-  color: var(--faint);
-}
-
-/* Status never rides on color alone: the label itself says what blocks. */
-.wt-way[data-clear] {
-  color: var(--faint);
-}
-
-.wt-way:not([data-clear]) {
-  color: var(--warn);
-}
-
-.wt-row .wt-reclaim-btn {
-  margin-left: auto;
-}
-
-.wt-row .wt-reclaim-btn::before {
-  content: "[";
-  color: var(--danger);
-}
-
-.wt-row .wt-reclaim-btn::after {
-  content: "]";
-  color: var(--danger);
-}
-
-.wt-row .wt-reclaim-btn:hover {
-  color: var(--danger);
-  background: var(--danger-tint);
-}
-
-.btn-danger {
-  color: var(--danger);
-  font-weight: 600;
-}
-
-.btn-danger:hover {
-  background: var(--danger-tint);
-}
-
-/* The destructive confirmation, centered over the section. */
-.wt-confirm {
-  position: fixed;
-  inset: 0;
-  z-index: var(--z-backdrop);
-  display: flex;
-  align-items: center;
-  justify-content: center;
-  background: oklch(0 0 0 / 0.5);
-  padding: var(--sp-4);
-}
-
-.wt-confirm[hidden] {
-  display: none;
-}
-
-.wt-confirm-panel {
-  pointer-events: auto;
-  background: var(--bg);
-  border: 1px solid var(--surface-2);
-  padding: var(--sp-4);
-  max-width: 480px;
-  font-family: var(--font-sans);
-  display: flex;
-  flex-direction: column;
-  gap: var(--sp-2);
-}
-
-.wt-confirm-q {
-  margin: 0;
-  font-family: var(--font-mono);
-  font-weight: 600;
-  color: var(--ink);
-}
-
-.wt-confirm-way {
-  margin: 0;
-  font-family: var(--font-mono);
-  font-size: var(--fs-13);
-  color: var(--warn);
-}
-
-.wt-confirm-note {
-  margin: 0;
-  font-size: var(--fs-13);
-  color: var(--muted);
-}
-
-.wt-confirm-actions {
-  display: flex;
-  gap: var(--sp-2);
-  justify-content: flex-end;
-  margin-top: var(--sp-2);
-}

web/src/toolIcon.test.ts

@@ -0,0 +1,58 @@
+import { describe, expect, it } from "vitest";
+import { splitCommand, toolIconSvg } from "./toolIcon.js";
+
+describe("tool icons on the command fragment (docs/SIDEBAR.md)", () => {
+  it("recognises the agents and shells sessions are spawned with", () => {
+    const cases: Array<[string, string]> = [
+      ["claude", "claude"],
+      ["codex", "codex"],
+      ["opencode", "opencode"],
+      ["gemini", "gemini"],
+      ["aider", "aider"],
+      ["cursor-agent", "cursor"],
+      ["pi", "pi"],
+      ["zsh", "shell"],
+      ["bash", "shell"],
+      ["fish", "shell"],
+    ];
+    for (const [command, tool] of cases) {
+      expect(splitCommand(command).tool, command).toBe(tool);
+    }
+  });
+
+  it("keeps everything after the program as text", () => {
+    expect(splitCommand("claude --resume -p hi")).toEqual({
+      tool: "claude",
+      rest: "--resume -p hi",
+    });
+    expect(splitCommand("  zsh   -l  ")).toEqual({ tool: "shell", rest: "-l" });
+  });
+
+  it("looks through paths, env assignments and wrappers", () => {
+    expect(splitCommand("/usr/local/bin/claude").tool).toBe("claude");
+    expect(splitCommand("FOO=1 BAR=2 claude").tool).toBe("claude");
+    expect(splitCommand("npx opencode").tool).toBe("opencode");
+    expect(splitCommand("pnpm dlx opencode run").tool).toBe("opencode");
+    expect(splitCommand("pnpm dlx opencode run").rest).toBe("run");
+  });
+
+  it("leaves an unrecognised program as text, mark-less", () => {
+    expect(splitCommand("hx src/main.rs")).toEqual({
+      tool: null,
+      rest: "hx src/main.rs",
+    });
+    // A wrapper with nothing after it is not a tool either.
+    expect(splitCommand("sudo").tool).toBeNull();
+    expect(splitCommand("").tool).toBeNull();
+  });
+
+  it("gives every recognised tool a labelled mark", () => {
+    for (const tool of ["claude", "codex", "opencode", "gemini", "aider", "cursor", "pi", "shell"]) {
+      const svg = toolIconSvg(tool);
+      expect(svg, tool).toContain("<svg");
+      expect(svg, tool).toContain(`data-tool="${tool}"`);
+      expect(svg, tool).toContain("aria-label=");
+    }
+    expect(toolIconSvg("nonesuch")).toBeNull();
+  });
+});

web/src/toolIcon.ts

@@ -0,0 +1,141 @@
+/** Tool icons for the sidebar's command fragment (docs/SIDEBAR.md).
+ *
+ * A session's command line starts with the program that runs it, and that
+ * program is nearly always one of a handful of agents or a shell. Rendering it
+ * as its own mark buys back most of the fragment's width, which the sidebar
+ * spends on the rest of the command line instead.
+ *
+ * Recognition is deliberately narrow: an unrecognised program keeps its text,
+ * so the fragment never renders a mark that stands for the wrong tool.
+ */
+
+/** Command wrappers that run another program: the tool is what follows. */
+const WRAPPERS = new Set(["sudo", "env", "npx", "bunx", "uvx", "pnpm", "yarn", "command"]);
+
+/** Program name (already basenamed and lowercased) → icon id. */
+const BY_PROGRAM: Record<string, string> = {
+  claude: "claude",
+  "claude-code": "claude",
+  codex: "codex",
+  opencode: "opencode",
+  gemini: "gemini",
+  aider: "aider",
+  cursor: "cursor",
+  "cursor-agent": "cursor",
+  pi: "pi",
+  bash: "shell",
+  zsh: "shell",
+  sh: "shell",
+  fish: "shell",
+  dash: "shell",
+  ksh: "shell",
+  nu: "shell",
+  xonsh: "shell",
+};
+
+/** Human name behind an icon id, for the tooltip and the aria label. */
+export const TOOL_LABELS: Record<string, string> = {
+  claude: "Claude Code",
+  codex: "Codex",
+  opencode: "opencode",
+  gemini: "Gemini CLI",
+  aider: "Aider",
+  cursor: "Cursor",
+  pi: "pi",
+  shell: "Shell",
+};
+
+export interface CommandParts {
+  /** Icon id, or `null` when the program is not one we have a mark for. */
+  tool: string | null;
+  /** What is left of the command line once the icon stands for the program.
+   * Empty when the icon says everything the command line said. */
+  rest: string;
+}
+
+/** Split a command line into the tool its icon stands for and the remainder.
+ *
+ * `claude --resume` → `{tool: "claude", rest: "--resume"}`; an unrecognised
+ * `hx src/` keeps the whole line as `rest` with no tool. */
+export function splitCommand(command: string): CommandParts {
+  const words = command.trim().split(/\s+/).filter(Boolean);
+  let i = 0;
+  // Leading `FOO=bar` assignments and wrappers stand in front of the real
+  // program; `pnpm dlx x` / `yarn dlx x` need one more word skipped.
+  while (i < words.length) {
+    const word = words[i];
+    if (/^[A-Za-z_][A-Za-z0-9_]*=/.test(word)) {
+      i += 1;
+      continue;
+    }
+    const program = basename(word);
+    if (WRAPPERS.has(program)) {
+      i += 1;
+      if ((program === "pnpm" || program === "yarn") && words[i] === "dlx") i += 1;
+      continue;
+    }
+    break;
+  }
+  if (i >= words.length) return { tool: null, rest: command.trim() };
+  const tool = BY_PROGRAM[basename(words[i])] ?? null;
+  if (!tool) return { tool: null, rest: command.trim() };
+  return { tool, rest: words.slice(i + 1).join(" ") };
+}
+
+function basename(word: string): string {
+  const cut = word.lastIndexOf("/");
+  return (cut >= 0 ? word.slice(cut + 1) : word).toLowerCase();
+}
+
+/** Anthropic's burst: tapered rays around a centre, long ones on the axes. */
+function burst(): string {
+  const rays = Array.from({ length: 8 }, (_, k) => {
+    const long = k % 2 === 0;
+    const tip = long ? 1.1 : 3.4;
+    const waist = long ? 6.3 : 6.6;
+    const half = long ? 1.05 : 0.85;
+    return `<path d="M8 ${tip} L${8 + half} ${waist} L8 8 L${8 - half} ${waist} Z" transform="rotate(${k * 45} 8 8)"/>`;
+  }).join("");
+  return `<g fill="#d97757">${rays}</g>`;
+}
+
+const SVGS: Record<string, string> = {
+  claude: burst(),
+  // A hexafoil knot in the shape of OpenAI's mark. The petals stay thin and
+  // wide apart: three fatter loops merge into a blob at 13px.
+  codex: `<g fill="none" stroke="currentColor" stroke-width="1.05">
+    <ellipse cx="8" cy="8" rx="2.5" ry="6.7"/>
+    <ellipse cx="8" cy="8" rx="2.5" ry="6.7" transform="rotate(60 8 8)"/>
+    <ellipse cx="8" cy="8" rx="2.5" ry="6.7" transform="rotate(120 8 8)"/>
+  </g>`,
+  gemini: `<path fill="#4285f4" d="M8 0.8C8.6 4.6 11.4 7.4 15.2 8C11.4 8.6 8.6 11.4 8 15.2C7.4 11.4 4.6 8.6 0.8 8C4.6 7.4 7.4 4.6 8 0.8Z"/>`,
+  // An isometric cube, the shape Cursor's mark is built from.
+  cursor: `<g fill="currentColor">
+    <path d="M8 1 L14.5 4.75 L8 8.5 L1.5 4.75 Z" opacity="0.95"/>
+    <path d="M14.5 4.75 L14.5 12.25 L8 16 L8 8.5 Z" opacity="0.5"/>
+    <path d="M1.5 4.75 L8 8.5 L8 16 L1.5 12.25 Z" opacity="0.75"/>
+  </g>`,
+  shell: `<g fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round" stroke-linejoin="round">
+    <path d="M2.6 4 L6.4 8 L2.6 12"/>
+    <path d="M8.4 12.6 H13.4"/>
+  </g>`,
+  opencode: monogram("oc"),
+  aider: monogram("ai"),
+  pi: monogram("π"),
+};
+
+/** Tools whose own mark is a wordmark get their initials instead — inventing
+ * a logo for them would be worse than saying which letters they start with.
+ * No chip around the letters: a border this small eats the height the letters
+ * need to stay apart. */
+function monogram(text: string): string {
+  return `<text x="8" y="8.6" fill="currentColor" font-family="ui-monospace, SFMono-Regular, Menlo, monospace" font-size="11" font-weight="700" letter-spacing="-0.6" text-anchor="middle" dominant-baseline="central">${text}</text>`;
+}
+
+/** The `<svg>` for an icon id, or `null` when the id has no mark. */
+export function toolIconSvg(tool: string): string | null {
+  const body = SVGS[tool];
+  if (!body) return null;
+  const label = TOOL_LABELS[tool] ?? tool;
+  return `<svg class="tool-icon" data-tool="${tool}" viewBox="0 0 16 16" width="13" height="13" role="img" aria-label="${label}"><title>${label}</title>${body}</svg>`;
+}

web/src/types.ts

@@ -157,10 +157,6 @@ export interface BranchInfo {
 /** `GET /api/branches?path=` — the local branches of one repository (#55). */
 export type Branches = BranchInfo[];
 
-/** Why a retained worktree outlived its session (#80). New values are
- * additive; unknown reasons render generically. */
-export type RetainedReason = "force-killed" | "exited-with-unsaved-work";
-
 /** What currently blocks an unforced kill of a worktree's session (#80),
  * recomputed by the daemon on every read. All counts default to 0 on the
  * wire for older daemons. */
@@ -168,19 +164,42 @@ export interface InTheWay {
   uncommitted_files?: number;
   untracked_files?: number;
   unpushed_commits?: number;
+  /** Lines added across the tracked changes, staged and unstaged together. */
+  insertions?: number;
+  /** Lines removed across the same tracked changes. */
+  deletions?: number;
 }
 
-/** One entry of `GET /api/worktrees` (#80): a retained worktree with its
- * in-the-way state as of this read. Reclaiming it removes the directory but
- * never the branch. Mirrors `maestro_protocol::RetainedWorktree`. */
-export interface RetainedWorktree {
-  id: Uuid;
-  path: string;
-  base_repo: string;
-  branch: string;
-  reason: RetainedReason;
-  retained_at: IsoTimestamp;
-  in_the_way?: InTheWay;
+/** One fragment of the sidebar's compact blocked-work stat (docs/SIDEBAR.md).
+ * `kind` picks the colour; `text` is what the client prints. */
+export interface BlockerStat {
+  kind: "add" | "del" | "files" | "untracked" | "unpushed";
+  text: string;
+}
+
+/** The blocked-work breakdown as diff notation: `+124 -18 ?3 ↑2`.
+ *
+ * The line counts cover tracked changes only, so a tree whose only tracked
+ * change carries no line delta — a mode change, a rename — falls back to the
+ * file count (`~2`) rather than claiming `+0 -0`. Both clients render exactly
+ * these fragments; the spelled-out {@link formatInTheWay} survives in the
+ * tooltip and in the kill refusal. */
+export function blockerStats(w: InTheWay): BlockerStat[] {
+  const stats: BlockerStat[] = [];
+  const added = w.insertions ?? 0;
+  const removed = w.deletions ?? 0;
+  const changed = w.uncommitted_files ?? 0;
+  if (added > 0 || removed > 0) {
+    if (added > 0) stats.push({ kind: "add", text: `+${added}` });
+    if (removed > 0) stats.push({ kind: "del", text: `-${removed}` });
+  } else if (changed > 0) {
+    stats.push({ kind: "files", text: `~${changed}` });
+  }
+  if ((w.untracked_files ?? 0) > 0)
+    stats.push({ kind: "untracked", text: `?${w.untracked_files}` });
+  if ((w.unpushed_commits ?? 0) > 0)
+    stats.push({ kind: "unpushed", text: `\u2191${w.unpushed_commits}` });
+  return stats;
 }
 
 /** The phrasing `maestro_protocol::InTheWay`'s Display impl uses — shared by
@@ -199,12 +218,6 @@ export function formatInTheWay(w: InTheWay): string {
   return parts.length > 0 ? parts.join(", ") : "no unsaved work";
 }
 
-/** Same reason labels the CLI table and the TUI overlay use; unknown future
- * reasons render their raw wire value. */
-export function reasonLabel(reason: RetainedReason): string {
-  return reason;
-}
-
 /** #83: this session's blocked-work breakdown when something actually blocks
  * an unforced kill; `null` while clear or not yet probed. Render it with
  * {@link formatInTheWay} so every client phrases it identically. */

web/src/worktreeStep.test.ts

@@ -108,6 +108,16 @@ describe("spawnForm worktree step (#95)", () => {
     expect(step(form).hidden).toBe(false);
   });
 
+  it("keeps the worktree step inside the collapsed Advanced options section", async () => {
+    const form = await mount([MAESTRO, WT_A]);
+    box(form, MAESTRO.path).click();
+    await drain();
+
+    const collapsible = form.querySelector<HTMLElement>(".wt-collapsible")!;
+    expect(collapsible.contains(step(form))).toBe(true);
+    expect(collapsible.hidden).toBe(true);
+  });
+
   it("lists '+ new' first and selected by default, then the project's worktrees", async () => {
     const form = await mount([MAESTRO, WT_A, WT_B]);
     box(form, MAESTRO.path).click();
@@ -230,4 +240,35 @@ describe("spawnForm worktree step (#95)", () => {
     expect(box(form, MAESTRO.path).classList.contains("is-active")).toBe(true);
     expect(toggle(form).checked).toBe(false);
   });
+
+  it("lands a 'spawn similar' seed on the cloned project with '+ new' active", async () => {
+    urls = stubFetch([MAESTRO, WT_A, WT_B]);
+    handle = createSpawnForm({
+      onSuccess() {},
+      seed: { cwd: MAESTRO.path, cwdRawVisible: false, worktreeEnabled: true },
+    });
+    document.body.appendChild(handle.el);
+    await drain();
+    const form = handle.el;
+
+    expect(box(form, MAESTRO.path).classList.contains("is-active")).toBe(true);
+    expect(activeChoice(form)!.dataset.kind).toBe("new");
+    expect(toggle(form).checked).toBe(true);
+    expect(form.querySelector<HTMLElement>(".cwd-raw-wrap")!.hidden).toBe(true);
+  });
+
+  it("falls back to the raw input when the seeded directory matches no project", async () => {
+    urls = stubFetch([MAESTRO, WT_A]);
+    handle = createSpawnForm({
+      onSuccess() {},
+      seed: { cwd: "/home/user/gone", cwdRawVisible: false, worktreeEnabled: false },
+    });
+    document.body.appendChild(handle.el);
+    await drain();
+    const form = handle.el;
+
+    const raw = form.querySelector<HTMLInputElement>(".cwd-raw")!;
+    expect(form.querySelector<HTMLElement>(".cwd-raw-wrap")!.hidden).toBe(false);
+    expect(raw.value).toBe("/home/user/gone");
+  });
 });

web/src/worktrees.test.ts

@@ -1,114 +0,0 @@
-// @vitest-environment happy-dom
-import { describe, expect, it } from "vitest";
-
-import { createWorktreesView } from "./worktreesView.js";
-import type { RetainedWorktree } from "./types.js";
-
-function worktree(overrides: Partial<RetainedWorktree>): RetainedWorktree {
-  return {
-    id: "11111111-2222-3333-4444-555555555555",
-    path: "/home/naps/tea/maestro/worktrees/feat/login",
-    base_repo: "/home/naps/tea/maestro",
-    branch: "feat/login",
-    reason: "force-killed",
-    retained_at: "2026-08-24T10:00:00Z",
-    in_the_way: {
-      uncommitted_files: 2,
-      untracked_files: 1,
-      unpushed_commits: 3,
-    },
-    ...overrides,
-  };
-}
-
-/** Mount a view over a scripted store and return (el, reclaimed-ids). */
-function mount(items: RetainedWorktree[], homeDir = "/home/naps") {
-  const reclaimed: string[] = [];
-  const view = createWorktreesView({
-    load: async () => items,
-    reclaim: async (id) => {
-      reclaimed.push(id);
-      items.splice(
-        items.findIndex((w) => w.id === id),
-        1,
-      );
-    },
-    homeDir: () => homeDir,
-  });
-  document.body.appendChild(view.el);
-  return { view, reclaimed };
-}
-
-describe("retained-worktrees view (#84)", () => {
-  it("lists branch, abbreviated path, reason and what is in the way", async () => {
-    const { view } = mount([
-      worktree({
-        path: "/home/naps/tea/maestro/worktrees/feat/login",
-      }),
-    ]);
-    await view.refresh();
-    const text = view.el.textContent ?? "";
-    expect(text).toContain("feat/login");
-    // Sidebar-style abbreviation of the path against the daemon's home dir,
-    // full path kept in the tooltip.
-    expect(text).toContain("t/m/w/f/login");
-    const pathEl = view.el.querySelector<HTMLElement>(".wt-path")!;
-    expect(pathEl.title).toBe("/home/naps/tea/maestro/worktrees/feat/login");
-    // Same reason labels as `maestro worktrees list` and the TUI overlay.
-    expect(text).toContain("force-killed");
-    // Same phrasing as InTheWay's Display impl — shared with the kill refusal.
-    expect(text).toContain("2 uncommitted files, 1 untracked file, 3 unpushed commits");
-  });
-
-  it("says a tree whose work has since been pushed is clear", async () => {
-    const { view } = mount([worktree({ in_the_way: {} })]);
-    await view.refresh();
-    expect(view.el.textContent).toContain("no unsaved work");
-  });
-
-  it("renders an empty store as a normal state, not an error", async () => {
-    const { view } = mount([]);
-    await view.refresh();
-    const text = view.el.textContent ?? "";
-    expect(text).toContain("no retained worktrees");
-    expect(text.toLowerCase()).not.toContain("error");
-    expect(text.toLowerCase()).not.toContain("failed");
-  });
-
-  it("guards reclaim with a confirmation that names what is lost and that the branch survives", async () => {
-    const wt = worktree({ branch: "feat/login" });
-    const { view, reclaimed } = mount([wt]);
-    await view.refresh();
-
-    view.el.querySelector<HTMLButtonElement>(".wt-reclaim-btn")!.click();
-    const confirmText = view.el.querySelector<HTMLElement>(".wt-confirm")!.textContent ?? "";
-    expect(confirmText).toContain("Reclaim worktree feat/login?");
-    // Names what will be lost before it happens…
-    expect(confirmText).toContain(
-      "2 uncommitted files, 1 untracked file, 3 unpushed commits",
-    );
-    // …and says reclaim does not take the commits too.
-    expect(confirmText).toContain("never deleted");
-
-    view.el.querySelector<HTMLButtonElement>(".wt-confirm-cancel")!.click();
-    expect(reclaimed).toEqual([]);
-
-    view.el.querySelector<HTMLButtonElement>(".wt-reclaim-btn")!.click();
-    view.el.querySelector<HTMLButtonElement>(".wt-confirm-go")!.click();
-    await Promise.resolve();
-    await Promise.resolve();
-    expect(reclaimed).toEqual(["11111111-2222-3333-4444-555555555555"]);
-    // The row is gone once the refresh lands.
-    expect(view.el.textContent).toContain("no retained worktrees");
-  });
-
-  it("sorts oldest-retained first, like every other client", async () => {
-    const { view } = mount([
-      worktree({ branch: "newer", retained_at: "2026-08-24T12:00:00Z" }),
-      worktree({ branch: "older", retained_at: "2026-08-24T08:00:00Z" }),
-    ]);
-    await view.refresh();
-    const branches = [...view.el.querySelectorAll(".wt-branch")].map((n) => n.textContent);
-    expect(branches).toEqual(["older", "newer"]);
-  });
-});

web/src/worktreesView.ts

@@ -1,166 +0,0 @@
-// Retained-worktrees view (#84): every worktree the daemon is holding that no
-// session uses, listed from `GET /api/worktrees` with a reclaim action.
-//
-// Wording is shared with the CLI (`maestro worktrees list`) and the TUI
-// overlay: same reason labels, same in-the-way phrasing via
-// `formatInTheWay` (the mirror of `InTheWay`'s Display impl). Reclaim reads
-// as what it is — destructive — naming what will be lost before it happens
-// and saying out loud that the branch survives (docs/WORKTREES.md).
-
-import { escapeHtml } from "./html.js";
-import {
-  abbreviateCwd,
-  abbreviateFolderName,
-  formatInTheWay,
-  reasonLabel,
-  type RetainedWorktree,
-  type Uuid,
-} from "./types.js";
-
-export interface WorktreesDeps {
-  /** `GET /api/worktrees`. */
-  load(): Promise<RetainedWorktree[]>;
-  /** `DELETE /api/worktrees/:id`. */
-  reclaim(id: Uuid): Promise<void>;
-  /** The daemon's home dir, for `~` abbreviation of paths. */
-  homeDir(): string | undefined;
-}
-
-export interface WorktreesViewHandle {
-  el: HTMLElement;
-  /** Re-fetch the store and re-render. Safe to call repeatedly. */
-  refresh(): Promise<void>;
-  dispose(): void;
-}
-
-/** Oldest-retained first, matching `maestro worktrees list`'s ordering. */
-function sorted(items: RetainedWorktree[]): RetainedWorktree[] {
-  return [...items].sort(
-    (a, b) => Date.parse(a.retained_at) - Date.parse(b.retained_at),
-  );
-}
-
-export function createWorktreesView(deps: WorktreesDeps): WorktreesViewHandle {
-  const el = document.createElement("section");
-  el.className = "worktrees";
-  el.innerHTML = `
-    <header class="wt-head">
-      <h2 class="wt-title">retained worktrees</h2>
-      <p class="wt-sub">
-        Kept when a session ended with unsaved work. Reclaiming removes the
-        directory only; branches are never deleted.
-      </p>
-    </header>
-    <div class="wt-list"></div>
-    <div class="wt-confirm" role="alertdialog" aria-modal="true" hidden></div>
-  `;
-  const listHost = el.querySelector<HTMLElement>(".wt-list")!;
-  const confirmEl = el.querySelector<HTMLElement>(".wt-confirm")!;
-
-  let pendingId: string | null = null;
-
-  const render = (items: RetainedWorktree[]) => {
-    listHost.replaceChildren();
-    if (items.length === 0) {
-      // A drained store is the normal state — say so plainly, not like
-      // something went wrong.
-      const empty = document.createElement("div");
-      empty.className = "empty";
-      empty.innerHTML = `
-        <div class="empty-glyph" aria-hidden="true">⌂</div>
-        <div class="empty-title">no retained worktrees</div>
-        <p class="empty-hint">Nothing is being held. Worktrees kept after a force-kill or an unsaved exit show up here.</p>`;
-      listHost.appendChild(empty);
-      return;
-    }
-    const home = deps.homeDir();
-    for (const w of sorted(items)) {
-      listHost.appendChild(renderRow(w, home));
-    }
-  };
-
-  const renderRow = (w: RetainedWorktree, home: string | undefined): HTMLElement => {
-    const row = document.createElement("div");
-    row.className = "wt-row";
-    const clear = formatInTheWay(w.in_the_way ?? {}) === "no unsaved work";
-    row.innerHTML = `
-      <span class="wt-branch">${escapeHtml(w.branch)}</span>
-      <span class="wt-path" title="${escapeHtml(w.path)}">${abbreviateFolderName(abbreviateCwd(w.path, home))}</span>
-      <span class="wt-reason">${escapeHtml(reasonLabel(w.reason))}</span>
-      <span class="wt-way"${clear ? ' data-clear="true"' : ""}>${escapeHtml(formatInTheWay(w.in_the_way ?? {}))}</span>
-      <button class="btn wt-reclaim-btn" type="button">Reclaim</button>
-    `;
-    row.querySelector(".wt-reclaim-btn")!.addEventListener("click", () => {
-      showConfirm(w);
-    });
-    return row;
-  };
-
-  const showConfirm = (w: RetainedWorktree) => {
-    pendingId = w.id;
-    // Mirrors the TUI's ReclaimConfirm line for line: what goes away, what is
-    // in the way right now, and that the branch survives reclaim.
-    confirmEl.innerHTML = `
-      <div class="wt-confirm-panel">
-        <p class="wt-confirm-q">Reclaim worktree ${escapeHtml(w.branch)}?</p>
-        <p class="wt-confirm-way">Currently in the way: ${escapeHtml(formatInTheWay(w.in_the_way ?? {}))}</p>
-        <p class="wt-confirm-note">The directory is removed; the branch is never deleted.</p>
-        <div class="wt-confirm-actions">
-          <button class="btn btn-danger wt-confirm-go" type="button">Reclaim</button>
-          <button class="btn btn-ghost wt-confirm-cancel" type="button">Cancel</button>
-        </div>
-      </div>`;
-    confirmEl.hidden = false;
-    confirmEl.querySelector<HTMLButtonElement>(".wt-confirm-go")!.focus();
-  };
-
-  const hideConfirm = () => {
-    pendingId = null;
-    confirmEl.hidden = true;
-    confirmEl.replaceChildren();
-  };
-
-  confirmEl.addEventListener("click", async (e) => {
-    const target = e.target as HTMLElement;
-    if (target.closest(".wt-confirm-cancel")) {
-      hideConfirm();
-      return;
-    }
-    if (!target.closest(".wt-confirm-go") || !pendingId) return;
-    const id = pendingId;
-    hideConfirm();
-    try {
-      await deps.reclaim(id);
-    } catch (err) {
-      alert(`Failed to reclaim: ${(err as Error).message}`);
-    }
-    await refresh();
-  });
-
-  const onKey = (e: KeyboardEvent) => {
-    if (e.key === "Escape" && !confirmEl.hidden) hideConfirm();
-  };
-  document.addEventListener("keydown", onKey);
-
-  const refresh = async () => {
-    try {
-      render(await deps.load());
-    } catch (err) {
-      listHost.replaceChildren();
-      const fail = document.createElement("div");
-      fail.className = "empty";
-      fail.innerHTML = `<div class="empty-title">Couldn't load retained worktrees</div><p class="empty-hint"></p>`;
-      fail.querySelector(".empty-hint")!.textContent = (err as Error).message;
-      listHost.appendChild(fail);
-    }
-  };
-
-  return {
-    el,
-    refresh,
-    dispose() {
-      document.removeEventListener("keydown", onKey);
-      el.remove();
-    },
-  };
-}