reliary-agent 0.8.16

use std::path::PathBuf;
use std::fs;
use serde_json::{json, Value};
use std::process::Command;
use std::io::Write;

/// Show a spinner while a closure runs. Clears the line when done.
/// Uses a helper that prints a message, runs the closure, then clears.
/// Avoids threading (rusqlite Connection is !Sync).
pub fn with_spinner<F, T>(msg: &str, f: F) -> T
where
    F: FnOnce() -> T,
{
    // P1/P2: no fake animation, no ANSI when piped, suppressed under -q/json
    // by callers routing through here only on TTY. Plain stderr line otherwise.
    let tty = std::io::IsTerminal::is_terminal(&std::io::stderr());
    if tty {
        eprint!("{} ... ", msg);
        let _ = std::io::stderr().flush();
    }
    let result = f();
    if tty && colors_on() {
        eprint!("\r\x1b[K");
    }
    result
}

fn home_dir() -> Option<PathBuf> {
    dirs::home_dir()
}

fn colors_on() -> bool {
    if std::env::var("NO_COLOR").is_ok() { return false; }
    if std::env::var("TERM").map(|t| t == "dumb").unwrap_or(false) { return false; }
    if std::env::var("CLICOLOR").map(|v| v == "0").unwrap_or(false) { return false; }
    // Piped output is never colored unless explicitly forced.
    if std::env::var("CLICOLOR_FORCE").map(|v| v != "0").unwrap_or(false) { return true; }
    std::io::IsTerminal::is_terminal(&std::io::stderr())
}

fn color() -> &'static str { if colors_on() { "\x1b[1m\x1b[32m" } else { "" } }
fn reset() -> &'static str { if colors_on() { "\x1b[0m" } else { "" } }
fn dim() -> &'static str { if colors_on() { "\x1b[2m" } else { "" } }
fn blue() -> &'static str { if colors_on() { "\x1b[34m" } else { "" } }
fn red() -> &'static str { if colors_on() { "\x1b[31m" } else { "" } }
fn yellow() -> &'static str { if colors_on() { "\x1b[33m" } else { "" } }

/// Shared color gate used by `main.rs` and `init.rs` so every command honors
/// `NO_COLOR`/`CLICOLOR`+TTY identically (P1.1: two color systems, one gate).
pub fn colors_enabled() -> bool { colors_on() }

/// Names the binary can be installed under: the shipped name (`reliary`) and
/// the legacy crate/bin name (`reliary-agent`) that older releases used.
const BINARY_NAMES: [&str; 2] = ["reliary", "reliary-agent"];

/// Scan for multiple installations of the reliary binary.
pub struct InstallInfo {
    pub path: String,
    pub version: String,
    pub method: &'static str,
    pub active: bool,
}

pub fn find_installs() -> Vec<InstallInfo> {
    let mut installs: Vec<InstallInfo> = Vec::new();
    let mut seen_paths = std::collections::HashSet::new();

    // Find active binary via PATH (either name).
    let which = if cfg!(target_os = "windows") { "where" } else { "which" };
    for bin_name in BINARY_NAMES {
        if let Ok(output) = Command::new(which).arg("-a").arg(bin_name).output() {
            let stdout = String::from_utf8_lossy(&output.stdout);
            for path in stdout.lines() {
                let p = path.trim();
                if !p.is_empty() && seen_paths.insert(p.to_string()) {
                    let version = binary_version(p);
                    installs.push(InstallInfo {
                        path: p.to_string(),
                        version,
                        method: "PATH",
                        active: installs.is_empty(),
                    });
                }
            }
        }
    }

    // Check cargo bin
    if let Some(home) = home_dir() {
        for bin_name in BINARY_NAMES {
            let cargo_bin = home.join(".cargo/bin").join(bin_name);
            let cargo_path = cargo_bin.to_string_lossy().to_string();
            if cargo_bin.exists() && seen_paths.insert(cargo_path.clone()) {
                let version = binary_version(&cargo_path);
                installs.push(InstallInfo { path: cargo_path, version, method: "cargo", active: false });
            }

            // Check npm global
            let npm_bin = home.join(".local/share/io.npm/.npm-global/bin").join(bin_name);
            let npm_path = npm_bin.to_string_lossy().to_string();
            if npm_bin.exists() && seen_paths.insert(npm_path.clone()) {
                let version = binary_version(&npm_path);
                installs.push(InstallInfo { path: npm_path, version, method: "npm", active: false });
            }
            // Also check npm's common global dir
            let npm_bin2 = home.join("node_modules/.bin").join(bin_name);
            let npm_path2 = npm_bin2.to_string_lossy().to_string();
            if npm_bin2.exists() && seen_paths.insert(npm_path2.clone()) {
                let version = binary_version(&npm_path2);
                installs.push(InstallInfo { path: npm_path2, version, method: "npm", active: false });
            }
        }
    }

    // Check Homebrew paths
    for dir in &[
        "/opt/homebrew/bin",
        "/usr/local/bin",
        "/home/linuxbrew/.linuxbrew/bin",
    ] {
        for bin_name in BINARY_NAMES {
            let brew_path = format!("{}/{}", dir, bin_name);
            let p = std::path::Path::new(&brew_path);
            if p.exists() && seen_paths.insert(brew_path.clone()) {
                let version = binary_version(&brew_path);
                installs.push(InstallInfo { path: brew_path, version, method: "brew", active: false });
            }
        }
    }

    installs
}

fn binary_version(path: &str) -> String {
    let output = Command::new(path).arg("--version").output();
    match output {
        Ok(o) if o.status.success() => {
            let s = String::from_utf8_lossy(&o.stdout).trim().to_string();
            if s.is_empty() { "?".to_string() } else { s }
        }
        _ => "?".to_string(),
    }
}

/// Print install table to display alongside doctor
pub fn print_install_table(installs: &[InstallInfo]) {
    if installs.is_empty() { return; }
    println!("  {}", "─".repeat(50));
    for inst in installs {
        let marker = if inst.active { format!("{}→{}", blue(), reset()) } else { " ".to_string() };
        println!("  {} {} {}v{}  {}", marker, dim(), dim(), inst.version, inst.path);
        if inst.active {
            println!("    {}Active (from PATH){}", dim(), reset());
        }
    }
    println!("  {}", "─".repeat(50));
}

struct DoctorCheck {
    name: &'static str,
    ok: bool,
    detail: String,
    fixable: bool,
    optional: bool,
}

/// Return the names of reliary hook commands registered in
/// `~/.claude/settings.json` whose script file does not exist. Such an entry
/// fires a failing exec on every matching tool call, so surfacing it is worth
/// a doctor failure.
fn dangling_reliary_hooks(home: &std::path::Path) -> Vec<String> {
    let settings_path = home.join(".claude/settings.json");
    let content = match fs::read_to_string(&settings_path) {
        Ok(c) => c,
        Err(_) => return Vec::new(),
    };
    let v: serde_json::Value = match serde_json::from_str(&content) {
        Ok(v) => v,
        Err(_) => return Vec::new(),
    };
    let mut dangling = Vec::new();
    if let Some(hooks) = v.get("hooks").and_then(|h| h.as_object()) {
        for entries in hooks.values() {
            let Some(arr) = entries.as_array() else { continue };
            for entry in arr {
                let Some(hs) = entry.get("hooks").and_then(|h| h.as_array()) else { continue };
                for h in hs {
                    let Some(cmd) = h.get("command").and_then(|c| c.as_str()) else { continue };
                    if !cmd.contains("reliary") {
                        continue;
                    }
                    // Only the program token matters; drop any arguments and
                    // skip env-assignment prefixes (FOO=bar cmd ...).
                    let prog = cmd.split_whitespace()
                        .find(|tok| !tok.contains('='))
                        .unwrap_or("");
                    if !prog.contains("reliary") {
                        continue;
                    }
                    let expanded = if let Some(rest) = prog.strip_prefix("~/") {
                        home.join(rest)
                    } else {
                        std::path::PathBuf::from(prog)
                    };
                    if !expanded.exists() {
                        dangling.push(prog.to_string());
                    }
                }
            }
        }
    }
    dangling.sort();
    dangling.dedup();
    dangling
}

fn doctor_checks(installs: &[InstallInfo]) -> Vec<DoctorCheck> {
    let mut checks = Vec::new();
    // --- Binary reachability (critical: MCP server needs this) ---
    // P4: `current_exe().is_some()` was tautological (always true when it
    // returns). Resolve via PATH instead so a stale/broken install fails.
    let exe = std::env::current_exe().ok(); // GUARDED: intentional — None falls back to "reliary"
    let exe_name = exe.as_ref()
        .and_then(|p| p.file_name())
        .and_then(|n| n.to_str())
        .unwrap_or("reliary");
    let exe_on_path = std::env::var("PATH").map(|p| {
        std::env::split_paths(&p).any(|d| d.join(exe_name).exists() || d.join("reliary").exists())
    }).unwrap_or(false);
    let binary_ok = exe.is_some() && exe_on_path;
    checks.push(DoctorCheck {
        name: "binary",
        ok: binary_ok,
        detail: match &exe {
            Some(p) if exe_on_path => format!("{} at {}", exe_name, p.display()),
            Some(p) => format!("{} at {} (not on PATH — MCP clients may not find it)", exe_name, p.display()),
            None => "could not resolve current exe".into(),
        },
        fixable: false,
        optional: false,
    });

    // --- Index health (the core capability) ---
    let index_path = PathBuf::from(".reliary/index.sqlite");
    let index_exists = index_path.exists();
    let (index_detail, index_ok) = if index_exists {
        // Probe the index for freshness
        let probe: Option<(i64, i64, f64)> = (|| -> Option<(i64, i64, f64)> {
            let db = rusqlite::Connection::open(&index_path).ok()?;
            let file_count: i64 = db.query_row("SELECT COUNT(*) FROM file_map", [], |r| r.get(0))
                .inspect_err(|e| eprintln!("[doctor] file count query failed: {}", e)).ok()?;
            let phrase_count: i64 = db.query_row("SELECT COUNT(*) FROM phrases", [], |r| r.get(0))
                .inspect_err(|e| eprintln!("[doctor] phrase count query failed: {}", e)).ok()?;
            // V80: `file_map` has no `mtime` column, so the old query always
            // errored and `unwrap_or(0.0)` reported "0d old" forever — the
            // stale-index branch was dead. Use the index file's own mtime.
            let age_days: f64 = std::fs::metadata(&index_path)
                .and_then(|m| m.modified())
                .ok()
                .and_then(|t| t.elapsed().ok())
                .map(|d| d.as_secs_f64() / 86_400.0)
                .unwrap_or(0.0);
            Some((file_count, phrase_count, age_days))
        })();
        // A leftover `.bak` is an interrupted-rebuild artifact; it doubles disk
        // usage and is never used again. `run_index` removes it on the next
        // rebuild, but a project that is never re-trusted keeps it forever.
        let bak_note = if index_path.with_extension("sqlite.bak").exists() {
            " (+ stale index.sqlite.bak — rebuild with `reliary trust .` to reclaim)"
        } else {
            ""
        };
        // V86: an index built by a different indexer has the same schema and
        // different contents. Old classifiers stored type names and local
        // variables as definitions; nothing used to report the mismatch, so an
        // upgraded binary kept answering from the old extraction.
        let fp_mismatch = rusqlite::Connection::open(&index_path)
            .ok()
            .map(|db| {
                reliary_search::schema::indexer_fingerprint(&db)
                    .map(|fp| fp != reliary_search::warrant::INDEXER_FINGERPRINT)
                    .unwrap_or(true)
            })
            .unwrap_or(false);
        match probe {
            Some((files, phrases, age)) => {
                let stale_note = if fp_mismatch {
                    " (built by a different indexer — run `reliary trust .` to rebuild)"
                } else if age > 7.0 {
                    " (stale — run `reliary trust .`)"
                } else {
                    ""
                };
                if fp_mismatch || age > 7.0 {
                    (format!("{} files, {} phrases, {:.0}d old{}{}", files, phrases, age, stale_note, bak_note), true)
                } else {
                    (format!("{} files, {} phrases, {:.0}d old{}", files, phrases, age, bak_note), true)
                }
            }
            // The file exists but cannot be read as an index. Reporting this
            // as ok (as it did before) hides a corrupt or truncated database
            // behind a green tick.
            None => (format!(
                "{} exists but is not a usable index — run `reliary trust .` to rebuild",
                index_path.display()
            ), false),
        }
    } else {
        ("no index found — run `reliary trust .`".into(), false)
    };
    checks.push(DoctorCheck {
        name: "index",
        ok: index_ok,
        detail: index_detail,
        fixable: true,
        optional: false,
    });

    // --- Ignore safety (index/cache must not be committable) ---
    // `.reliary/` holds source-derived data and cache.sqlite (original command
    // output, often secrets). If the project is a git worktree and the
    // self-ignore file is missing, `git add .` stages multi-MB leaks. Detected
    // without shelling to git: presence of `.git` in the walk-up plus a missing
    // `.reliary/.gitignore`. P4: walks up to 4 levels (not just ../), checks
    // the file has content (an empty guard ignores nothing), and is fixable
    // via `doctor --fix` (writes the guard through the shared helper).
    if index_exists {
        let mut in_git = std::path::Path::new(".git").exists();
        let mut up = std::path::PathBuf::from("..");
        for _ in 0..3 {
            if up.join(".git").exists() { in_git = true; break; }
            up.push("..");
        }
        let ignore_file = PathBuf::from(".reliary/.gitignore");
        let guard_ok = std::fs::read_to_string(&ignore_file)
            .map(|c| c.lines().any(|l| { let t = l.trim(); !t.is_empty() && !t.starts_with('#') }))
            .unwrap_or(false);
        let (ignore_ok, ignore_detail) = if !in_git {
            (true, "not a git worktree (n/a)".to_string())
        } else if guard_ok {
            (true, ".reliary/ is git-ignored".to_string())
        } else {
            (false, ".reliary/ is NOT ignored — run `reliary doctor --fix` or `reliary trust .`".to_string())
        };
        checks.push(DoctorCheck {
            name: "ignore",
            ok: ignore_ok,
            detail: ignore_detail,
            fixable: true,
            optional: false,
        });
    }

    // --- Mode (configuration check) ---
    // P4: a typo'd RELIARY_MODE previously fell back to `reactive` and still
    // reported green. Surface the raw value when it is not a known mode.
    let mode = crate::config::resolve_mode(Some("."));
    let raw_mode = std::env::var("RELIARY_MODE").unwrap_or_default();
    let known = ["fast", "reactive", "strict"];
    let (mode_ok, mode_detail) = if !raw_mode.is_empty() && !known.contains(&raw_mode.as_str()) {
        (false, format!("unknown RELIARY_MODE={:?} — falling back to {} (expected fast|reactive|strict)", raw_mode, mode.as_str()))
    } else {
        (true, mode.as_str().into())
    };
    checks.push(DoctorCheck {
        name: "mode",
        ok: mode_ok,
        detail: mode_detail,
        fixable: false,
        optional: false,
    });

    // --- Claude Code integration (optional) ---
    // P4: hooks are optional — a user who skipped them is not broken.
    // Require 2/2 only when a registration exists; otherwise report wiring.
    let claude_cfg = home_dir().map(|h| h.join(".claude.json")).unwrap_or_default();
    let claude_ok = has_mcp_server(&claude_cfg, "reliary");
    let claude_hooks_dir = home_dir().map(|h| h.join(".claude/hooks")).unwrap_or_default();
    let hooks_count = ["reliary-session-reminder", "reliary-sift-pretooluse"]
        .iter()
        .filter(|name| claude_hooks_dir.join(name).exists())
        .count();
    // A registration whose command file is missing fires a failing exec on
    // every matching tool call. Detect it instead of counting files only.
    let dangling = home_dir()
        .map(|h| dangling_reliary_hooks(&h))
        .unwrap_or_default();
    let claude_detail = if !claude_ok {
        if claude_cfg.exists() {
            "config exists but not wired — run `reliary init`".into()
        } else {
            "not found (optional)".into()
        }
    } else if !dangling.is_empty() {
        format!(
            "MCP wired, but registered hook(s) missing on disk: {} — run `reliary init`",
            dangling.join(", ")
        )
    } else if hooks_count == 2 {
        "MCP wired, 2/2 hooks installed".into()
    } else if hooks_count == 0 {
        "MCP wired (hooks skipped — optional)".into()
    } else {
        format!("MCP wired, {}/2 hooks installed (hooks optional)", hooks_count)
    };
    // P4: hooks skipped is not a failure — only dangling registrations fail.
    let claude_ok_final = claude_ok && dangling.is_empty();
    checks.push(DoctorCheck {
        name: "claude",
        ok: claude_ok_final,
        detail: claude_detail,
        fixable: false,
        optional: true,
    });

    // --- OpenCode integration (optional) ---
    let opencode_cfg = find_opencode_config();
    let (opencode_ok, opencode_detail) = match &opencode_cfg {
        None => (false, "not found (optional)".to_string()),
        Some(p) if opencode_mcp_well_formed(p) => (true, "MCP wired".to_string()),
        Some(p) if has_mcp_server(p, "reliary") => (
            false,
            "mcp.reliary present but malformed (needs \"type\":\"local\" and an \
             array \"command\") — re-run `reliary init`"
                .to_string(),
        ),
        Some(_) => (false, "config exists but not wired — run `reliary init`".to_string()),
    };
    checks.push(DoctorCheck {
        name: "opencode",
        ok: opencode_ok,
        detail: opencode_detail,
        fixable: false,
        optional: true,
    });

    // --- Pi Agent integration (optional) ---
    let pi_gate = home_dir().map(|h| h.join(".local/share/reliary/gate.js")).unwrap_or_default();
    checks.push(DoctorCheck {
        name: "pi",
        ok: pi_gate.exists(),
        detail: if pi_gate.exists() { "gate.js installed".into() } else { "not found (optional)".into() },
        fixable: false,
        optional: true,
    });

    // --- Tee/recovery directory cleanup (low-priority hygiene) ---
    let tee_dir = std::env::temp_dir().join("reliary-tee");
    let tee_count = if tee_dir.exists() {
        std::fs::read_dir(&tee_dir).map(|d| d.count()).unwrap_or(0)
    } else {
        0
    };
    if tee_count > 100 {
        checks.push(DoctorCheck {
            name: "tee",
            ok: false,
            detail: format!("{} entries in {} — run `reliary clean --global` to free space", tee_count, tee_dir.display()),
            fixable: false,
            optional: true,
        });
    }

    // --- Multi-install check (installs passed in to avoid redundant find_installs call) ---
    if installs.len() > 1 {
        let active_version = installs.iter().find(|i| i.active).map(|i| i.version.clone()).unwrap_or_default();
        let stale_count = installs.iter().filter(|i| !i.active && i.version != active_version).count();
        if stale_count > 0 {
            checks.push(DoctorCheck {
                name: "installs",
                ok: false,
                detail: format!("{} installations, {} stale — remove old binaries from PATH", installs.len(), stale_count),
                fixable: false,
                optional: false,
            });
        }
    }
    let installs_count = installs.len();
    if installs_count > 2 {
        checks.push(DoctorCheck {
            name: "installs",
            ok: false,
            detail: format!("{} active copies — clutter", installs_count),
            fixable: false,
            optional: false,
        });
    }

    checks
}

fn doctor_json(checks: &[DoctorCheck]) -> Value {
    let all_good = checks.iter().all(|c| c.ok || c.optional);
    json!({
        "ready": all_good,
        "checks": checks.iter().map(|c| json!({
            "name": c.name,
            "ok": c.ok,
            "detail": c.detail,
        })).collect::<Vec<_>>(),
    })
}

pub fn doctor(fix: bool, format: &str) {
    let installs = find_installs();
    let mut checks = doctor_checks(&installs);

    if format == "json" {
        let mut j = doctor_json(&checks);
        let install_json: Vec<Value> = installs.iter().map(|i| json!({
            "path": i.path,
            "version": i.version,
            "method": i.method,
            "active": i.active,
        })).collect();
        j.as_object_mut().unwrap().insert("installations".into(), Value::Array(install_json));
        println!("{}", serde_json::to_string_pretty(&j).unwrap_or_else(|_| r#"{"ready":false,"checks":[]}"#.to_string()));
        return;
    }

    println!("\n{}| Reliary Doctor |{}\n", color(), reset());

    let mut needs_index = false;
    let mut needs_ignore = false;

    for c in &checks {
        let icon = if c.ok {
            format!("{}✓{}", color(), reset())
        } else if c.optional {
            format!("{}-{}", dim(), reset())
        } else {
            format!("{}✗{}", red(), reset())
        };
        println!("  {} {} {}{}", icon, c.name, dim(), c.detail);
        if !c.ok && c.fixable {
            if c.name == "index" { needs_index = true; }
            // P4: `doctor --fix` writes the missing self-ignore guard through
            // the shared helper (was documented fixable but never fixed).
            if c.name == "ignore" { needs_ignore = true; }
        }
    }

    if fix && (needs_index || needs_ignore) {
        println!();
        if needs_ignore {
            print!("  {} Writing .reliary/.gitignore... ", dim());
            crate::paths::ensure_reliary_ignored(std::path::Path::new(".reliary"));
            if PathBuf::from(".reliary/.gitignore").exists() { println!("{}done", color()); } else { println!("{}failed", red()); }
        }
        if needs_index {
            print!("  {} Building index... ", dim());
            let exe = std::env::current_exe().unwrap_or_else(|_| PathBuf::from("reliary"));
            let status = Command::new(exe).arg("trust").arg(".").stdout(std::process::Stdio::inherit()).stderr(std::process::Stdio::inherit()).status();
            if status.is_ok_and(|s| s.success()) { println!("{}done", color()); } else { println!("{}failed", red()); }
        }
        // Re-evaluate after fixes
        checks = doctor_checks(&installs);
    }

    if installs.len() > 1 {
        println!();
        println!("  {}| Installations |{}", color(), reset());
        print_install_table(&installs);
    }

    let all_good = checks.iter().all(|c| c.ok || c.optional);
    if all_good {
        println!("\n{}✓{} System ready.", color(), reset());
    } else if !fix {
        println!("\n  {}Tip: run 'reliary doctor --fix' to fix issues automatically.{}", dim(), reset());
    } else {
        println!("\n{}✓{} System ready after fix.", color(), reset());
        // Show remaining non-optional failures with hints
        for c in &checks {
            if !c.ok && !c.optional {
                let hint = match c.name {
                    "index" => "Run 'reliary trust .' to build it",
                    "installs" => "Remove stale binary paths manually",
                    _ => "Check the detail above",
                };
                println!("  {} {} {}", dim(), c.name, hint);
            }
        }
    }
}

struct StatusData {
    mode: String,
    index_files: i64,
    chronicle_events: i64,
    index_exists: bool,
    /// V70 P7: age of the index in seconds (None if no index).
    index_age_secs: Option<u64>,
    /// V70 P7: watcher availability (env-gated at spawn in the MCP server).
    watcher_enabled: bool,
}

fn status_data_at(root: &str) -> StatusData {
    let index_path = std::path::Path::new(root).join(".reliary/index.sqlite");
    let mut index_files = 0i64;
    let mut chronicle_events = 0i64;
    let index_exists = index_path.exists();

    if index_exists {
        // F8 hardening: read-only open — status must never create, lock, or
        // journal a project DB (the old read-write open could create a 0-table
        // stub via `Connection::open` semantics on some paths).
        let db = rusqlite::Connection::open_with_flags(
            &index_path,
            rusqlite::OpenFlags::SQLITE_OPEN_READ_ONLY,
        );
        if let Ok(db) = db {
            if let Ok(mut stmt) = db.prepare("SELECT COUNT(*) FROM file_map") {
                if let Ok(mut rows) = stmt.query([]) {
                    if let Ok(Some(row)) = rows.next() { index_files = row.get(0).unwrap_or(0); }
                }
            }
            if let Ok(mut stmt) = db.prepare("SELECT COUNT(*) FROM chronicle") {
                if let Ok(mut rows) = stmt.query([]) {
                    if let Ok(Some(row)) = rows.next() { chronicle_events = row.get(0).unwrap_or(0); }
                }
            }
        }
    }

    let index_age_secs = std::fs::metadata(&index_path)
        .and_then(|m| m.modified())
        .ok()
        .and_then(|t| t.elapsed().ok())
        .map(|d| d.as_secs());
    let watcher_enabled = std::env::var("NO_RELIARY_WATCHER").is_err();

    StatusData {
        mode: crate::config::resolve_mode(Some(".")).as_str().to_string(),
        index_files,
        chronicle_events,
        index_exists,
        index_age_secs,
        watcher_enabled,
    }
}

fn status_json(d: &StatusData) -> Value {
    json!({
        "mode": d.mode,
        "index": {
            "exists": d.index_exists,
            "files": d.index_files,
            "age_secs": d.index_age_secs,
        },
        "watcher": { "enabled": d.watcher_enabled },
        "chronicle": { "events": d.chronicle_events },
    })
}

pub fn status_at(root: &str, format: &str) {
    let d = status_data_at(root);

    if format == "json" {
        println!("{}", serde_json::to_string_pretty(&status_json(&d)).unwrap_or_else(|_| "{}".to_string()));
        return;
    }

    println!("\n{}| Reliary Agent Status |{}\n", color(), reset());
    println!("  {}•{} Mode: {}", blue(), reset(), d.mode);

    if d.index_exists {
        let age = match d.index_age_secs {
            Some(s) if s < 60 => format!("{}s ago", s),
            Some(s) if s < 3600 => format!("{}m ago", s / 60),
            Some(s) if s < 86400 => format!("{}h ago", s / 3600),
            Some(s) => format!("{}d ago", s / 86400),
            None => "unknown".to_string(),
        };
        println!("  {}•{} Index: {} files indexed (updated {})", blue(), reset(), d.index_files, age);
        println!("  {}•{} Watcher: {}", blue(), reset(),
            if d.watcher_enabled { "enabled (auto-reindex on save)" } else { "disabled (NO_RELIARY_WATCHER=1)" });
        println!("  {}•{} Memory: {} chronicle events", blue(), reset(), d.chronicle_events);
    } else {
        println!("  {}•{} Index: {}-{} No index found", blue(), reset(), yellow(), reset());
        println!("    {}→ Run 'reliary trust .' to build it{}", dim(), reset());
    }
}

pub fn clean(global: bool, all: bool) {
    let do_global = global || all;
    let do_local = !global || all;

    if do_local {
        let local_dir = PathBuf::from(".reliary");
        if local_dir.exists() {
            if fs::remove_dir_all(&local_dir).is_ok() {
                println!("{}✓{} Cleaned project state (.reliary)", color(), reset());
            } else {
                println!("{}✗{} Failed to clean project state", red(), reset());
            }
        } else {
            println!("{}-{} No project state found", yellow(), reset());
        }
    }

    if do_global {
        // V61: doctor flags >100 tee entries but clean --global never
        // removed them — /tmp/reliary-tee grew forever.
        match crate::tee::clean_tee() {
            Ok(bytes) if bytes > 0 => {
                println!("{}✓{} Cleaned {} bytes of tee artifacts (/tmp/reliary-tee)", color(), reset(), bytes);
            }
            _ => {
                println!("{}-{} No tee artifacts to clean", yellow(), reset());
            }
        }
        if let Some(home) = home_dir() {
            let global_dir = home.join(".reliary");
            if global_dir.exists() {
                if fs::remove_dir_all(&global_dir).is_ok() {
                    println!("{}✓{} Cleaned global state (~/.reliary)", color(), reset());
                } else {
                    println!("{}✗{} Failed to clean global state", red(), reset());
                }
            } else {
                println!("{}-{} No global state found", yellow(), reset());
            }
        }
    }
}

pub fn logs(tail: bool, level: Option<String>) {
    if let Ok(log_path_str) = std::env::var("RELIARY_LOG_FILE") {
        let log_path = std::path::Path::new(&log_path_str);
        if log_path.exists() {
            // F8 hardening: 10MB cap — an operator-pointed log file must not
            // OOM the reader. `tail -f` streams (no cap needed); the read
            // paths refuse oversize files with remediation.
            const LOG_CAP: u64 = 10 * 1024 * 1024;
            let too_big = std::fs::metadata(log_path).map(|m| m.len() > LOG_CAP).unwrap_or(false);
            if tail {
                println!("{} Tailing {}...{}", blue(), log_path_str, reset());
                let status = Command::new("tail").arg("-f").arg(&log_path_str).status();
                if status.is_err() {
                    if too_big {
                        eprintln!("{} Log file exceeds 10MB; refusing to dump. Use `tail` directly.{}{}", yellow(), log_path_str, reset());
                        std::process::exit(1);
                    }
                    if let Ok(content) = std::fs::read_to_string(log_path) { println!("{}", content); }
                }
            } else if let Some(lvl) = level {
                if too_big {
                    eprintln!("{} Log file exceeds 10MB; refusing to dump. Use `tail` directly.{}{}", yellow(), log_path_str, reset());
                    std::process::exit(1);
                }
                let lower_lvl = lvl.to_lowercase();
                let upper = format!(" [{}] ", lvl.to_uppercase());
                let lower = format!("[{}]", lower_lvl);
                if let Ok(content) = std::fs::read_to_string(log_path) {
                    for line in content.lines() {
                        if line.contains(&upper) || line.contains(&lower) { println!("{}", line); }
                    }
                }
            } else {
                if too_big {
                    eprintln!("{} Log file exceeds 10MB; refusing to dump. Use `tail` directly.{}{}", yellow(), log_path_str, reset());
                    std::process::exit(1);
                }
                if let Ok(content) = std::fs::read_to_string(log_path) { println!("{}", content); }
            }
            return;
        } else {
            eprintln!("{} Log file not found: {}{}", yellow(), log_path_str, reset());
            std::process::exit(1);
        }
    }
    eprintln!("{} No RELIARY_LOG_FILE env var set.{}", yellow(), reset());
    std::process::exit(1);
}

pub fn format_risk(path: &str, risk: &str, format: &str) {
    if format == "json" {
        let risk_lower = risk.to_lowercase();
        let (level, reason) = if risk_lower.contains("high") {
            ("high", "High blast radius — many callers or critical path")
        } else if risk_lower.contains("medium") {
            ("medium", "Moderate blast radius — some callers affected")
        } else {
            ("low", "Low risk: small file or few callers")
        };
        println!("{}", json!({
            "file": path,
            "risk": level,
            "reason": reason,
        }));
    } else {
        let risk_lower = risk.to_lowercase();
        let icon = if risk_lower.contains("high") {
            format!("{}⚠{}", red(), reset())
        } else if risk_lower.contains("medium") {
            format!("{}⚡{}", yellow(), reset())
        } else {
            format!("{}✓{}", color(), reset())
        };
        // V74: floor_char_boundary — slicing at byte 60 panicked on multibyte text.
        println!("  {} {} {}", icon, path, &risk[..risk.floor_char_boundary(60.min(risk.len()))]);
    }
}

pub fn format_dead(path: &str, entries: &[String], format: &str) {
    if format == "json" {
        println!("{}", json!({
            "path": path,
            "candidates": entries,
        }));
    } else {
        println!("\n{}| Dead Code: {} |{}\n", color(), path, reset());
        if entries.is_empty() {
            println!("  {}No dead code candidates found.{}", dim(), reset());
        } else {
            for entry in entries {
                println!("  {}•{} {}", yellow(), reset(), entry);
            }
            println!("\n  {}Found {} candidate(s){}", dim(), entries.len(), reset());
        }
    }
}

fn has_mcp_server(cfg_path: &PathBuf, server_name: &str) -> bool {
    if let Ok(content) = fs::read_to_string(cfg_path) {
        if let Ok(v) = serde_json::from_str::<Value>(&content) {
            for key in &["mcpServers", "mcp"] {
                if let Some(servers) = v.get(key).and_then(|m| m.as_object()) {
                    if servers.contains_key(server_name) {
                        return true;
                    }
                }
            }
        }
    }
    false
}

/// Whether an OpenCode config's `mcp.reliary` entry is *well-formed*.
///
/// OpenCode requires a local server to be `{ "type": "local", "command":
/// ["<exe>", ...] }`. The Claude/Cline shape (`{ "command": "<string>",
/// "args": [...] }`) decodes as invalid and the server never starts, so
/// checking only for the key's presence (as `has_mcp_server` does) reports a
/// false green.
///
/// Parsed with the JSONC parser because OpenCode's documented default config
/// (`opencode.jsonc`) contains comments that `serde_json` rejects.
fn opencode_mcp_well_formed(cfg_path: &std::path::Path) -> bool {
    let Ok(content) = fs::read_to_string(cfg_path) else { return false };
    let Ok(Some(v)) = jsonc_parser::parse_to_value(&content, &jsonc_parser::ParseOptions::default())
    else {
        return false;
    };
    let jsonc_parser::JsonValue::Object(root) = v else { return false };
    let Some(jsonc_parser::JsonValue::Object(mcp)) = root.get("mcp") else { return false };
    let Some(entry) = mcp.get("reliary") else { return false };
    let jsonc_parser::JsonValue::Object(entry) = entry else { return false };
    let is_local = matches!(entry.get("type"), Some(jsonc_parser::JsonValue::String(s)) if *s == "local");
    let command_ok = matches!(
        entry.get("command"),
        Some(jsonc_parser::JsonValue::Array(a)) if !a.is_empty()
    );
    is_local && command_ok
}

/// Locate any OpenCode global config, preferring the files OpenCode itself
/// checks (`opencode.jsonc`, then `opencode.json`, then `config.json`).
fn find_opencode_config() -> Option<PathBuf> {
    // V80: same resolution as `init::opencode_config_dir` —
    // `XDG_CONFIG_HOME || ~/.config/opencode` on every platform.
    let dir = if let Some(xdg) = std::env::var_os("XDG_CONFIG_HOME") {
        let p = PathBuf::from(xdg);
        if !p.as_os_str().is_empty() && p.is_absolute() {
            p.join("opencode")
        } else {
            home_dir()?.join(".config/opencode")
        }
    } else {
        home_dir()?.join(".config/opencode")
    };
    for name in ["opencode.jsonc", "opencode.json", "config.json"] {
        let p = dir.join(name);
        if p.exists() {
            return Some(p);
        }
    }
    None
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn test_with_spinner_runs_closure() {
        let result = with_spinner("testing", || 42);
        assert_eq!(result, 42);
    }

    #[test]
    fn test_with_spinner_no_side_effects() {
        let x = with_spinner("testing", || "hello world".to_string());
        assert_eq!(x, "hello world");
    }
}