recursive-agent 0.8.3

//! Per-session log of turn-level checkpoint records.
//!
//! Each session writes a `checkpoints.jsonl` file alongside its
//! transcript. One line = one turn, capturing the pre/post checkpoint
//! ids and the set of files this turn touched (via structured tool
//! calls and/or fallback shell-diff).
//!
//! This metadata is what `recursive sessions rewind` reads to:
//!  1. find the right `pre` checkpoint to restore to,
//!  2. compute the union of files touched in the rewound-away turns,
//!  3. detect conflicts where the workspace's current state diverges
//!     from this session's last known post-snapshot.

use serde::{Deserialize, Serialize};
use std::fs::{File, OpenOptions};
use std::io::{BufRead, BufReader, BufWriter, Write};
use std::path::{Path, PathBuf};

use crate::checkpoint::CheckpointId;
use crate::error::{Error, Result};

/// One record per checkpoint.
///
/// **Goal 284**: With on-demand checkpoints (no automatic pre/post
/// per turn), the schema is:
/// - `turn` — which turn the agent was in when it saved.
/// - `id` — the checkpoint id (was `post`).
/// - `message` — agent-supplied label (optional).
/// - `touched_files` — files touched up to this point.
/// - `touched_via` — attribution method.
/// - `saved_at` — Unix timestamp of the save.
///
/// **Backwards compatibility**: old `pre`/`post`/`started_at`/
/// `finished_at` fields are accepted on deserialization but ignored
/// in favour of the new fields. New records always use the new names.
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq)]
pub struct CheckpointRecord {
    /// 0-indexed turn within the session.
    pub turn: usize,
    /// Deprecated (auto-snapshot era). Present on old records;
    /// ignored when `id` is available.
    #[serde(default, skip_serializing_if = "Option::is_none")]
    pub pre: Option<CheckpointId>,
    /// Checkpoint id (was `post` in auto-snapshot era).
    #[serde(alias = "post")]
    pub id: CheckpointId,
    /// Agent-supplied label for this checkpoint. Defaults to empty
    /// for auto-snapshot records that lack it.
    #[serde(default, skip_serializing_if = "Option::is_none")]
    pub message: Option<String>,
    /// Workspace-relative paths the agent touched this turn.
    #[serde(default, skip_serializing_if = "Vec::is_empty")]
    pub touched_files: Vec<String>,
    /// "structured" if all touched files came from typed tool args;
    /// "shell-diff" if at least one file was attributed via fallback
    /// pre/post tree diff after a `run_shell` call.
    pub touched_via: TouchedVia,
    /// Deprecated (auto-snapshot era). Present on old records.
    #[serde(default, skip_serializing_if = "is_zero_i64")]
    pub started_at: i64,
    /// Deprecated (auto-snapshot era). Present on old records.
    #[serde(default, skip_serializing_if = "is_zero_i64")]
    pub finished_at: i64,
    /// Unix timestamp of when this checkpoint was saved.
    #[serde(default, skip_serializing_if = "is_zero_i64")]
    pub saved_at: i64,
}

fn is_zero_i64(v: &i64) -> bool {
    *v == 0
}

#[derive(Debug, Clone, Copy, Serialize, Deserialize, PartialEq, Eq)]
#[serde(rename_all = "kebab-case")]
pub enum TouchedVia {
    Structured,
    ShellDiff,
}

/// Append-only writer.
#[derive(Debug, Clone)]
pub struct CheckpointLogWriter {
    path: PathBuf,
}

impl CheckpointLogWriter {
    /// Open or create the log at `path`. Parent directory must exist.
    pub fn open(path: impl Into<PathBuf>) -> Result<Self> {
        let path = path.into();
        if let Some(parent) = path.parent() {
            std::fs::create_dir_all(parent).map_err(Error::Io)?;
        }
        // Touch the file if missing.
        let _ = OpenOptions::new()
            .append(true)
            .create(true)
            .open(&path)
            .map_err(Error::Io)?;
        Ok(Self { path })
    }

    /// Append a record. Each call performs an O_APPEND fsync-less write;
    /// records are durable on flush.
    pub fn append(&self, rec: &CheckpointRecord) -> Result<()> {
        let mut f = OpenOptions::new()
            .append(true)
            .create(true)
            .open(&self.path)
            .map_err(Error::Io)?;
        let mut w = BufWriter::new(&mut f);
        serde_json::to_writer(&mut w, rec).map_err(Error::Json)?;
        w.write_all(b"\n").map_err(Error::Io)?;
        w.flush().map_err(Error::Io)?;
        Ok(())
    }
}

/// Read all records from `path`. Empty / missing file → empty vec.
pub fn read_log(path: &Path) -> Result<Vec<CheckpointRecord>> {
    if !path.exists() {
        return Ok(vec![]);
    }
    let f = File::open(path).map_err(Error::Io)?;
    let r = BufReader::new(f);
    let mut out = Vec::new();
    for (i, line) in r.lines().enumerate() {
        let line = line.map_err(Error::Io)?;
        if line.trim().is_empty() {
            continue;
        }
        let rec: CheckpointRecord = serde_json::from_str(&line).map_err(|e| Error::Tool {
            name: "checkpoint-log".into(),
            call_id: None,
            message: format!("malformed log line {}: {e}", i + 1),
        })?;
        out.push(rec);
    }
    Ok(out)
}

/// Truncate the log to records with `turn < cutoff`. Atomic via temp
/// file + rename.
pub fn truncate_to_turn(path: &Path, cutoff: usize) -> Result<()> {
    let recs = read_log(path)?;
    let kept: Vec<&CheckpointRecord> = recs.iter().filter(|r| r.turn < cutoff).collect();

    let tmp = path.with_extension("jsonl.tmp");
    {
        let f = File::create(&tmp).map_err(Error::Io)?;
        let mut w = BufWriter::new(f);
        for r in &kept {
            serde_json::to_writer(&mut w, r).map_err(Error::Json)?;
            w.write_all(b"\n").map_err(Error::Io)?;
        }
        w.flush().map_err(Error::Io)?;
    }
    std::fs::rename(&tmp, path).map_err(Error::Io)?;
    Ok(())
}

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

    fn rec(turn: usize, id: &str) -> CheckpointRecord {
        CheckpointRecord {
            turn,
            pre: None,
            id: CheckpointId(id.to_string()),
            message: None,
            touched_files: vec!["a.txt".into()],
            touched_via: TouchedVia::Structured,
            started_at: 0,
            finished_at: 0,
            saved_at: 0,
        }
    }

    #[test]
    fn write_then_read_round_trip() {
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("checkpoints.jsonl");
        let w = CheckpointLogWriter::open(&path).unwrap();
        w.append(&rec(0, "aaa")).unwrap();
        w.append(&rec(1, "bbb")).unwrap();
        let read = read_log(&path).unwrap();
        assert_eq!(read.len(), 2);
        assert_eq!(read[0].turn, 0);
        assert_eq!(read[1].id.0, "bbb");
    }

    #[test]
    fn read_missing_file_returns_empty() {
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("does-not-exist.jsonl");
        assert!(read_log(&path).unwrap().is_empty());
    }

    #[test]
    fn truncate_drops_later_turns() {
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("c.jsonl");
        let w = CheckpointLogWriter::open(&path).unwrap();
        for t in 0..5 {
            w.append(&rec(t, &format!("p{t}"))).unwrap();
        }
        truncate_to_turn(&path, 3).unwrap();
        let kept = read_log(&path).unwrap();
        assert_eq!(kept.len(), 3);
        assert!(kept.iter().all(|r| r.turn < 3));
    }

    #[test]
    fn truncate_to_zero_empties_file() {
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("c.jsonl");
        let w = CheckpointLogWriter::open(&path).unwrap();
        w.append(&rec(0, "x")).unwrap();
        truncate_to_turn(&path, 0).unwrap();
        assert!(read_log(&path).unwrap().is_empty());
    }

    #[test]
    fn is_zero_i64_returns_true_for_zero() {
        assert!(is_zero_i64(&0_i64));
    }

    #[test]
    fn is_zero_i64_returns_false_for_positive() {
        assert!(!is_zero_i64(&1_i64));
        assert!(!is_zero_i64(&1000_i64));
    }

    #[test]
    fn is_zero_i64_returns_false_for_negative() {
        assert!(!is_zero_i64(&-1_i64));
    }

    #[test]
    fn nonzero_timestamps_survive_round_trip() {
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("ts.jsonl");
        let w = CheckpointLogWriter::open(&path).unwrap();
        let mut r = rec(0, "ts-test");
        r.started_at = 1_700_000_000;
        r.finished_at = 1_700_000_001;
        r.saved_at = 1_700_000_002;
        w.append(&r).unwrap();
        let rows = read_log(&path).unwrap();
        assert_eq!(rows[0].started_at, 1_700_000_000);
        assert_eq!(rows[0].finished_at, 1_700_000_001);
        assert_eq!(rows[0].saved_at, 1_700_000_002);
    }

    #[test]
    fn zero_timestamps_are_skipped_in_serialization() {
        let r = rec(0, "z");
        let json = serde_json::to_string(&r).unwrap();
        assert!(
            !json.contains("started_at"),
            "zero started_at should be skipped"
        );
        assert!(
            !json.contains("finished_at"),
            "zero finished_at should be skipped"
        );
        assert!(
            !json.contains("saved_at"),
            "zero saved_at should be skipped"
        );
    }

    #[test]
    fn nonzero_timestamps_are_included_in_serialization() {
        // kills `replace is_zero_i64 -> bool with true` and `replace == with !=` mutations.
        // If is_zero_i64 always returns true, non-zero timestamps would be omitted too.
        let mut r = rec(0, "z");
        r.started_at = 1_700_000_000;
        r.finished_at = 1_700_001_000;
        r.saved_at = 1_700_002_000;
        let json = serde_json::to_string(&r).unwrap();
        assert!(
            json.contains("started_at"),
            "non-zero started_at must appear in serialization; got: {json}"
        );
        assert!(
            json.contains("finished_at"),
            "non-zero finished_at must appear in serialization; got: {json}"
        );
        assert!(
            json.contains("saved_at"),
            "non-zero saved_at must appear in serialization; got: {json}"
        );
    }

    #[test]
    fn read_log_skips_blank_lines() {
        // kills the `if line.trim().is_empty() { continue }` guard removal mutation
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("ck.jsonl");
        let writer = CheckpointLogWriter::open(&path).unwrap();
        writer.append(&rec(1, "a")).unwrap();

        // Manually insert a blank line between records
        use std::io::Write;
        let mut f = std::fs::OpenOptions::new()
            .append(true)
            .open(&path)
            .unwrap();
        writeln!(f).unwrap(); // blank line
        drop(f);

        writer.append(&rec(2, "b")).unwrap();

        let recs = read_log(&path).unwrap();
        assert_eq!(recs.len(), 2, "blank line must be silently skipped");
        assert_eq!(recs[0].turn, 1);
        assert_eq!(recs[1].turn, 2);
    }

    #[test]
    fn read_log_returns_error_for_malformed_json() {
        // kills function-level replacement of the parse error path
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("bad.jsonl");
        std::fs::write(&path, "not-valid-json\n").unwrap();
        let result = read_log(&path);
        assert!(result.is_err(), "malformed JSON must return an Err");
    }

    #[test]
    fn truncate_keeps_records_below_cutoff_exactly() {
        // kills off-by-one mutations in `r.turn < cutoff`
        let dir = tempfile::tempdir().unwrap();
        let path = dir.path().join("ck.jsonl");
        let writer = CheckpointLogWriter::open(&path).unwrap();
        for turn in 0..5 {
            writer.append(&rec(turn, "x")).unwrap();
        }
        // cutoff=3 => keep turns 0,1,2; drop turns 3,4
        truncate_to_turn(&path, 3).unwrap();
        let recs = read_log(&path).unwrap();
        assert_eq!(recs.len(), 3);
        assert_eq!(recs[2].turn, 2, "turn 2 must be kept (< 3)");
    }
}