agent-dump 1.2.2__tar.gz → 1.2.4__tar.gz

{agent_dump-1.2.2 → agent_dump-1.2.4}/Cargo.lock

@@ -10,7 +10,7 @@ checksum = "320119579fcad9c21884f5c4861d16174d0e06250625266f50fe6898340abefa"
 
 [[package]]
 name = "agent-dump"
-version = "1.2.2"
+version = "1.2.4"
 dependencies = [
  "agent-dump-core",
  "caseless",
@@ -837,9 +837,9 @@ checksum = "8f42a60cbdf9a97f5d2305f08a87dc4e09308d1276d28c869c684d7777685682"
 
 [[package]]
 name = "jiff"
-version = "0.2.37"
+version = "0.2.38"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "0ab1baf72f08796de0260609515130699b890ac25f30e610ad894bc5856cafdb"
+checksum = "b2b005715dcbeb0089a3c0dab99f2ff1cc3b2525323552703d648585d342a383"
 dependencies = [
  "defmt",
  "jiff-core",
@@ -864,9 +864,9 @@ dependencies = [
 
 [[package]]
 name = "jiff-static"
-version = "0.2.37"
+version = "0.2.38"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "378268a1116ad67ae6228701118ac9f491d78fda38a40a1f1a9e1348de6f7212"
+checksum = "2cc9817253cf7c7ee4684451bd327e88d6f3658014e54a29198625590650695c"
 dependencies = [
  "jiff-core",
  "proc-macro2",
@@ -1859,9 +1859,9 @@ checksum = "06abde3611657adf66d383f00b093d7faecc7fa57071cce2578660c9f1010821"
 
 [[package]]
 name = "uuid"
-version = "1.26.1"
+version = "1.28.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "2ef6dac1e96601b4fb3acccccff2139741fcb757cb9a36089bf5be91cfb285ce"
+checksum = "7cc1186384beb7dd8eedea376413fd654937285ea6c9cfbb928dc3043ea4b606"
 dependencies = [
  "getrandom 0.4.3",
  "js-sys",

{agent_dump-1.2.2 → agent_dump-1.2.4}/Cargo.toml

@@ -46,7 +46,7 @@ nix = { version = "0.31", default-features = false, features = ["fs", "user"] }
 
 [package]
 name = "agent-dump"
-version = "1.2.2"
+version = "1.2.4"
 edition = "2024"
 rust-version = "1.90"
 publish = false

{agent_dump-1.2.2 → agent_dump-1.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-dump
-Version: 1.2.2
+Version: 1.2.4
 Classifier: Environment :: Console
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Programming Language :: Rust
@@ -462,7 +462,7 @@ File discovery failures include partially unreadable providers. Valid sessions r
 
 When some session reads or summaries fail, collection continues with successful sessions. The saved Markdown includes a fixed incomplete-report notice with failure and included-session counts; an entirely failed run still fails.
 
-Collect discovers candidate sessions without a creation-time cutoff and processes user/assistant visible text only within the requested local dates. A session spanning several days is split into separate daily units before summarization; session counts remain unique URI counts. Text without a reliable timestamp (including undated Kimi context and inferred Cursor times) belongs to no date and is excluded without being reported as a coverage gap. No session creation/update time is substituted. Query filters select candidates; limits apply to unique sessions after text-date filtering. Events, including long individual messages, are split into chunks of at most 3,200 Unicode characters including labels. Chunk summaries are merged in groups of at most eight; large final reports are rendered in separate source-preserving groups. Model inputs are limited to 64,000 characters per request. Oversized metadata or derived summaries fail explicitly; source text is never silently truncated. Partial reports list omitted session URIs. Full input coverage does not mean a summary preserves every detail.
+Collect discovers candidate sessions without a creation-time cutoff and processes user/assistant visible text only within the requested local dates. A session spanning several days is split into separate daily units before summarization; session counts remain unique URI counts. Text without a reliable timestamp (including undated Kimi context and inferred Cursor times) belongs to no date and is excluded without being reported as a coverage gap. No session creation/update time is substituted. Query filters select candidates; limits apply to unique sessions after text-date filtering. Events, including long individual messages, are split into chunks of at most 3,200 Unicode characters including labels. Chunk summaries are merged in groups of at most eight; large final reports are rendered in separate source-preserving groups. Model inputs are limited to 64,000 characters per request. Oversized metadata or derived summaries fail explicitly; source text is never silently truncated. A final report or `--summary` reply that stops at the model output token limit is treated as a failed request, not saved as complete. Partial reports list omitted session URIs. Full input coverage does not mean a summary preserves every detail.
 
 PM summaries merge sessions only within the same date and known working directory. Sessions without a working directory retain separate attribution.
 
@@ -479,6 +479,7 @@ provider = "openai" # openai | anthropic
 base_url = "https://api.openai.com/v1"
 model = "gpt-4.1-mini"
 api_key = "sk-..."
+reasoning = "off" # off | default
 
 [collect]
 summary_concurrency = 4 # 1-32
@@ -501,6 +502,23 @@ deny = [
 ]
 ```
 
+`[ai].reasoning` controls the model's thinking mode. `off` (the default) sends `enable_thinking: false` to OpenAI-compatible services and `thinking: {type: "disabled"}` to Anthropic. `default` sends neither and leaves the service default in place. Enabling reasoning or choosing an effort level uses vendor-specific parameters, so set them in `[ai.extra_body]`. The table is deep-merged into every request body: nested tables merge, other values replace.
+
+```toml
+# OpenAI reasoning models
+[ai.extra_body]
+reasoning_effort = "low"
+```
+
+```toml
+# Anthropic extended thinking; max_tokens must exceed budget_tokens
+[ai.extra_body]
+max_tokens = 8192
+thinking = { type = "enabled", budget_tokens = 2048 }
+```
+
+When an OpenAI-compatible service rejects the default `enable_thinking`, `temperature`, or `max_tokens`, agent-dump retries without it; `max_tokens` is renamed to `max_completion_tokens`. Parameters set in `extra_body` are never removed. `--config edit` does not prompt for `reasoning` or `extra_body` and keeps their values.
+
 `[agent.<name>].deny` only applies to `--collect`. When a session `cwd` matches one of the configured paths, or is inside that path, the session is ignored during collect.
 
 Collect, `--collect --dry-run`, and `--collect --emit-prompt` require valid TOML and exclusion arrays containing only nonempty path strings. Invalid configuration stops the command before session discovery or AI requests; compatibility parsing never disables exclusions for collect.
@@ -559,7 +577,7 @@ Pass `locations[].locator` from search unchanged to `--message`. Positions are o
 
 `--locate` requires `--search`. It returns messages containing any search term, respecting role filters, while all terms must still match the session. Title-only matches have empty locations; failed location reads have null locations and partial status. Search without --locate is unchanged.
 
-`--message` cannot combine with `--head` or `--summary`. Without `--format`, it prints context; `--json` writes a context envelope to stdout with kind=context and data containing uri, locator, total_messages, start/end and messages (position plus normalized message). Invalid or stale locators exit nonzero; JSON diagnostics go only to stderr.
+`--message` cannot combine with `--head` or `--summary`. Without `--format`, it prints context; `--json` writes a context envelope to stdout with kind=context, status (partial when recoverable source diagnostics occur) and data containing uri, locator, total_messages, start/end and messages (position plus normalized message). Invalid or stale locators exit nonzero; JSON diagnostics go only to stderr.
 
 Use `--format json,markdown` to export the selected context. Files are named `<id>.messages-<start>-<end>.json` / `.md` under the Provider's output directory, separate from full-session exports. Both formats retain the canonical Session URI, the locator (revision plus target position), the absolute message range, and status. JSON retains normalized message records; Markdown renders their searchable text. Recoverable source diagnostics mark the file `partial` and are reported on stderr. Stale locators are rejected before file creation. Provider format restrictions still apply; raw/print file formats and `--json` with file-export options are rejected. `--output` requires an explicit `--format`; output-directory defaults follow full-session export rules.
 

{agent_dump-1.2.2 → agent_dump-1.2.4}/README.md

@@ -438,7 +438,7 @@ File discovery failures include partially unreadable providers. Valid sessions r
 
 When some session reads or summaries fail, collection continues with successful sessions. The saved Markdown includes a fixed incomplete-report notice with failure and included-session counts; an entirely failed run still fails.
 
-Collect discovers candidate sessions without a creation-time cutoff and processes user/assistant visible text only within the requested local dates. A session spanning several days is split into separate daily units before summarization; session counts remain unique URI counts. Text without a reliable timestamp (including undated Kimi context and inferred Cursor times) belongs to no date and is excluded without being reported as a coverage gap. No session creation/update time is substituted. Query filters select candidates; limits apply to unique sessions after text-date filtering. Events, including long individual messages, are split into chunks of at most 3,200 Unicode characters including labels. Chunk summaries are merged in groups of at most eight; large final reports are rendered in separate source-preserving groups. Model inputs are limited to 64,000 characters per request. Oversized metadata or derived summaries fail explicitly; source text is never silently truncated. Partial reports list omitted session URIs. Full input coverage does not mean a summary preserves every detail.
+Collect discovers candidate sessions without a creation-time cutoff and processes user/assistant visible text only within the requested local dates. A session spanning several days is split into separate daily units before summarization; session counts remain unique URI counts. Text without a reliable timestamp (including undated Kimi context and inferred Cursor times) belongs to no date and is excluded without being reported as a coverage gap. No session creation/update time is substituted. Query filters select candidates; limits apply to unique sessions after text-date filtering. Events, including long individual messages, are split into chunks of at most 3,200 Unicode characters including labels. Chunk summaries are merged in groups of at most eight; large final reports are rendered in separate source-preserving groups. Model inputs are limited to 64,000 characters per request. Oversized metadata or derived summaries fail explicitly; source text is never silently truncated. A final report or `--summary` reply that stops at the model output token limit is treated as a failed request, not saved as complete. Partial reports list omitted session URIs. Full input coverage does not mean a summary preserves every detail.
 
 PM summaries merge sessions only within the same date and known working directory. Sessions without a working directory retain separate attribution.
 
@@ -455,6 +455,7 @@ provider = "openai" # openai | anthropic
 base_url = "https://api.openai.com/v1"
 model = "gpt-4.1-mini"
 api_key = "sk-..."
+reasoning = "off" # off | default
 
 [collect]
 summary_concurrency = 4 # 1-32
@@ -477,6 +478,23 @@ deny = [
 ]
 ```
 
+`[ai].reasoning` controls the model's thinking mode. `off` (the default) sends `enable_thinking: false` to OpenAI-compatible services and `thinking: {type: "disabled"}` to Anthropic. `default` sends neither and leaves the service default in place. Enabling reasoning or choosing an effort level uses vendor-specific parameters, so set them in `[ai.extra_body]`. The table is deep-merged into every request body: nested tables merge, other values replace.
+
+```toml
+# OpenAI reasoning models
+[ai.extra_body]
+reasoning_effort = "low"
+```
+
+```toml
+# Anthropic extended thinking; max_tokens must exceed budget_tokens
+[ai.extra_body]
+max_tokens = 8192
+thinking = { type = "enabled", budget_tokens = 2048 }
+```
+
+When an OpenAI-compatible service rejects the default `enable_thinking`, `temperature`, or `max_tokens`, agent-dump retries without it; `max_tokens` is renamed to `max_completion_tokens`. Parameters set in `extra_body` are never removed. `--config edit` does not prompt for `reasoning` or `extra_body` and keeps their values.
+
 `[agent.<name>].deny` only applies to `--collect`. When a session `cwd` matches one of the configured paths, or is inside that path, the session is ignored during collect.
 
 Collect, `--collect --dry-run`, and `--collect --emit-prompt` require valid TOML and exclusion arrays containing only nonempty path strings. Invalid configuration stops the command before session discovery or AI requests; compatibility parsing never disables exclusions for collect.
@@ -535,7 +553,7 @@ Pass `locations[].locator` from search unchanged to `--message`. Positions are o
 
 `--locate` requires `--search`. It returns messages containing any search term, respecting role filters, while all terms must still match the session. Title-only matches have empty locations; failed location reads have null locations and partial status. Search without --locate is unchanged.
 
-`--message` cannot combine with `--head` or `--summary`. Without `--format`, it prints context; `--json` writes a context envelope to stdout with kind=context and data containing uri, locator, total_messages, start/end and messages (position plus normalized message). Invalid or stale locators exit nonzero; JSON diagnostics go only to stderr.
+`--message` cannot combine with `--head` or `--summary`. Without `--format`, it prints context; `--json` writes a context envelope to stdout with kind=context, status (partial when recoverable source diagnostics occur) and data containing uri, locator, total_messages, start/end and messages (position plus normalized message). Invalid or stale locators exit nonzero; JSON diagnostics go only to stderr.
 
 Use `--format json,markdown` to export the selected context. Files are named `<id>.messages-<start>-<end>.json` / `.md` under the Provider's output directory, separate from full-session exports. Both formats retain the canonical Session URI, the locator (revision plus target position), the absolute message range, and status. JSON retains normalized message records; Markdown renders their searchable text. Recoverable source diagnostics mark the file `partial` and are reported on stderr. Stale locators are rejected before file creation. Provider format restrictions still apply; raw/print file formats and `--json` with file-export options are rejected. `--output` requires an explicit `--format`; output-directory defaults follow full-session export rules.
 

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/compat/json/diagnostic.rs

@@ -39,6 +39,7 @@ pub fn syntax(text: &str, original: &serde_json::Error) -> Error {
     let mut parser = Parser {
         chars: text.chars().collect(),
         index: 0,
+        depth: 0,
     };
     let failure = if text.starts_with('\u{feff}') {
         Some(("Unexpected UTF-8 BOM (decode using utf-8-sig)", 0))
@@ -49,7 +50,9 @@ pub fn syntax(text: &str, original: &serde_json::Error) -> Error {
                 .then_some(("Extra data", parser.index))
         })
     };
-    let message = if let Some((message, position)) = failure {
+    let message = if let Some((message, position)) =
+        failure.filter(|(message, _)| *message != TOO_DEEP)
+    {
         let before = &parser.chars[..position];
         let line = before.iter().filter(|c| **c == '\n').count() + 1;
         let column = position
@@ -70,9 +73,14 @@ pub fn syntax(text: &str, original: &serde_json::Error) -> Error {
 
 type Failure = (&'static str, usize);
 
+// Matches serde_json's recursion limit so deep input cannot overflow the stack.
+const MAX_DEPTH: usize = 128;
+const TOO_DEEP: &str = "recursion limit exceeded";
+
 struct Parser {
     chars: Vec<char>,
     index: usize,
+    depth: usize,
 }
 
 impl Parser {
@@ -107,8 +115,19 @@ impl Parser {
         self.whitespace();
         match self.chars.get(self.index).copied() {
             Some('"') => self.string(),
-            Some('{') => self.object(),
-            Some('[') => self.array(),
+            Some('{' | '[') if self.depth == MAX_DEPTH => {
+                Err((TOO_DEEP, self.index))
+            }
+            Some(open @ ('{' | '[')) => {
+                self.depth += 1;
+                let result = if open == '{' {
+                    self.object()
+                } else {
+                    self.array()
+                };
+                self.depth -= 1;
+                result
+            }
             _ => {
                 for word in
                     ["null", "true", "false", "NaN", "Infinity", "-Infinity"]

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/compat/json.rs

@@ -156,3 +156,15 @@ impl serde_json::ser::Formatter for Formatter {
         writer.write_all(output.as_bytes())
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn deep_nesting_reports_error_without_overflow() {
+        let error = from_str(&"[".repeat(100_000)).unwrap_err();
+
+        assert!(error.to_string().starts_with("recursion limit exceeded"));
+    }
+}

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/config.rs

@@ -10,6 +10,8 @@ pub struct AiConfig {
     pub base_url: String,
     pub model: String,
     pub api_key: String,
+    pub reasoning: String,
+    pub extra_body: Value,
 }
 
 pub struct CollectConfig {
@@ -183,6 +185,12 @@ impl Config {
             base_url: text(&self.data["ai"]["base_url"]),
             model: text(&self.data["ai"]["model"]),
             api_key: text(&self.data["ai"]["api_key"]),
+            reasoning: if self.data["ai"]["reasoning"].is_null() {
+                "off".into()
+            } else {
+                text(&self.data["ai"]["reasoning"])
+            },
+            extra_body: self.data["ai"]["extra_body"].clone(),
         })
     }
     pub fn output(&self) -> String {
@@ -436,6 +444,12 @@ pub fn validate_ai(
     if config.api_key.is_empty() {
         errors.push("api_key");
     }
+    if !matches!(config.reasoning.as_str(), "off" | "default") {
+        errors.push("reasoning");
+    }
+    if !config.extra_body.is_null() && !config.extra_body.is_object() {
+        errors.push("extra_body");
+    }
     if !config.base_url.is_empty() {
         let scheme = config
             .base_url

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/output/export.rs

@@ -59,25 +59,14 @@ fn filename(id: &str) -> crate::Result<String> {
     }
 }
 
-fn absolute_existing_ancestor(path: &Path) -> crate::Result<PathBuf> {
-    if path.exists() {
-        return Ok(path.canonicalize()?);
-    }
-    let parent = path.parent().ok_or("Cannot resolve output path")?;
-    Ok(absolute_existing_ancestor(parent)?
-        .join(path.file_name().ok_or("Invalid output path")?))
-}
-
 fn write(
     output: &Path,
     session_id: &str,
     suffix: &str,
-    source_root: &Path,
     contents: impl FnOnce(&mut fs::File) -> crate::Result<()>,
 ) -> crate::Result<PathBuf> {
     let directory = std::path::absolute(output)?;
-    let resolved = absolute_existing_ancestor(&directory)?;
-    if resolved.starts_with(source_root.canonicalize()?) {
+    if crate::providers::registry::inside_source(&directory)? {
         return Err(
             "Export output must be outside the Provider source directory"
                 .into(),
@@ -114,10 +103,9 @@ pub fn json(
     session_id: &str,
     data: &impl serde::Serialize,
     output: &Path,
-    source_root: &Path,
     suffix: &str,
 ) -> crate::Result<PathBuf> {
-    write(output, session_id, suffix, source_root, |file| {
+    write(output, session_id, suffix, |file| {
         let mut writer = io::BufWriter::new(file);
         data.serialize(&mut serde_json::Serializer::with_formatter(
             &mut writer,
@@ -132,19 +120,17 @@ pub fn markdown(
     session_id: &str,
     text: &str,
     output: &Path,
-    source_root: &Path,
 ) -> crate::Result<PathBuf> {
-    markdown_with_suffix(session_id, text, output, source_root, ".md")
+    markdown_with_suffix(session_id, text, output, ".md")
 }
 
 fn markdown_with_suffix(
     session_id: &str,
     text: &str,
     output: &Path,
-    source_root: &Path,
     suffix: &str,
 ) -> crate::Result<PathBuf> {
-    write(output, session_id, suffix, source_root, |file| {
+    write(output, session_id, suffix, |file| {
         if cfg!(windows) {
             file.write_all(text.replace('\n', "\r\n").as_bytes())?;
         } else {
@@ -158,9 +144,8 @@ pub fn raw(
     session_id: &str,
     source: &Path,
     output: &Path,
-    source_root: &Path,
 ) -> crate::Result<PathBuf> {
-    write(output, session_id, ".raw.jsonl", source_root, |file| {
+    write(output, session_id, ".raw.jsonl", |file| {
         io::copy(&mut fs::File::open(source)?, file)?;
         Ok(())
     })
@@ -179,7 +164,6 @@ impl Excerpt<'_> {
         &self,
         format: crate::output::formats::OutputFormat,
         output: &Path,
-        source_root: &Path,
     ) -> crate::Result<PathBuf> {
         use crate::output::formats::OutputFormat;
         let suffix =
@@ -197,7 +181,6 @@ impl Excerpt<'_> {
                     &self.data.id,
                     &document,
                     output,
-                    source_root,
                     &format!("{suffix}.json"),
                 )
             }
@@ -211,7 +194,6 @@ impl Excerpt<'_> {
                     self.incomplete,
                 ),
                 output,
-                source_root,
                 &format!("{suffix}.md"),
             ),
             OutputFormat::Raw | OutputFormat::Print => {
@@ -239,17 +221,13 @@ impl SessionExport<'_> {
         use crate::providers::contract::RawExport;
         match format {
             OutputFormat::Raw => match self.raw.as_ref().unwrap() {
-                RawExport::File(source) => raw(
-                    &self.session.id,
-                    source,
-                    output,
-                    self.provider.source_root(),
-                ),
+                RawExport::File(source) => {
+                    raw(&self.session.id, source, output)
+                }
                 RawExport::Session => json(
                     &self.session.id,
                     self.data.unwrap(),
                     output,
-                    self.provider.source_root(),
                     ".raw.json",
                 ),
             },
@@ -261,13 +239,7 @@ impl SessionExport<'_> {
                 {
                     object.insert("summary".into(), summary.into());
                 }
-                json(
-                    &self.session.id,
-                    &payload,
-                    output,
-                    self.provider.source_root(),
-                    ".json",
-                )
+                json(&self.session.id, &payload, output, ".json")
             }
             OutputFormat::Markdown => markdown(
                 &self.session.id,
@@ -276,7 +248,6 @@ impl SessionExport<'_> {
                     self.data.unwrap(),
                 ),
                 output,
-                self.provider.source_root(),
             ),
             OutputFormat::Print => unreachable!(),
         }

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/claude/mod.rs

@@ -1,10 +1,12 @@
 pub(super) mod transcript;
 
-use crate::compat::value::{field, text, truthy};
+use crate::compat::value::{field, text};
 use crate::providers::contract::{
     DiagnosticSink, Provider, RecoverableDiagnostic,
 };
-use crate::providers::files::{self as file_sessions, SourceRoots};
+use crate::providers::files::{
+    self as file_sessions, SourceFiles, SourceRoots,
+};
 use crate::providers::jsonl;
 use crate::providers::title::{basename, normalize_title};
 use crate::session::timestamp::Timestamp;
@@ -51,7 +53,7 @@ impl Claude {
         })
     }
 
-    fn files(&mut self) -> crate::Result<Vec<PathBuf>> {
+    fn files(&mut self) -> crate::Result<SourceFiles> {
         let files = self.roots.files(Some(2), |path, base| {
             path.extension().is_some_and(|ext| ext == "jsonl")
                 && path.parent() != Some(base)
@@ -112,11 +114,14 @@ impl Claude {
         let mut skipped = 0;
         for entry in value["entries"].as_array().into_iter().flatten() {
             let id = text(&entry["sessionId"]);
-            if id.trim().is_empty() {
+            let summary = &entry["summary"];
+            if id.trim().is_empty()
+                || !(summary.is_null() || summary.is_string())
+            {
                 skipped += 1;
                 continue;
             }
-            titles.insert(id.to_owned(), entry["summary"].clone());
+            titles.insert(id.to_owned(), summary.clone());
         }
         if skipped > 0 {
             diagnostics(RecoverableDiagnostic::TitleCacheEntriesSkipped {
@@ -149,21 +154,11 @@ impl Claude {
             .into_owned();
         let created_at = parse_timestamp(text(&header["timestamp"]))
             .unwrap_or(Timestamp::try_from(path.metadata()?.modified()?)?);
-        let explicit = match self
+        let explicit = self
             .titles(project, diagnostics)?
             .get(&id)
-            .filter(|value| truthy(value))
-        {
-            Some(Value::String(title)) => normalize_title(title),
-            Some(value) => {
-                let kind = crate::compat::value::type_name(value);
-                return Err(format!(
-                    "expected string or bytes-like object, got '{kind}'"
-                )
-                .into());
-            }
-            None => None,
-        };
+            .and_then(Value::as_str)
+            .and_then(normalize_title);
         let first_user = scan
             .records
             .iter()
@@ -244,7 +239,7 @@ impl Provider for Claude {
     ) -> crate::Result<crate::providers::contract::Discovery> {
         self.titles.clear();
         file_sessions::discover(
-            &self.files()?,
+            self.files()?,
             days,
             true,
             Self::scan,
@@ -261,7 +256,7 @@ impl Provider for Claude {
         let files = self.files()?;
         file_sessions::find(
             self.roots.base.clone().as_deref(),
-            &files,
+            files,
             id,
             |path| path.file_stem().is_some_and(|name| name == id),
             |path| self.parse(path, Self::scan(path)?, diagnostics),
@@ -296,6 +291,10 @@ impl Provider for Claude {
     fn source_root(&self) -> &Path {
         self.roots.owned()
     }
+
+    fn select_source(&mut self) -> crate::Result<Option<&Path>> {
+        self.roots.select()
+    }
 }
 
 #[cfg(test)]

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/claude/transcript.rs

@@ -255,12 +255,16 @@ impl Decoder {
                     continue;
                 }
                 let parts = output_parts(&item["content"], timestamp);
+                let mut updates = updates.clone();
+                if item["is_error"] == true {
+                    updates.insert("status".into(), "error".into());
+                }
                 if let Some(tool) = backfill(
                     &mut self.messages,
                     &self.pending,
                     &call_id,
                     &parts,
-                    Some(updates.clone()),
+                    Some(updates),
                 )? {
                     if !parts.is_empty() {
                         tool.state

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/codex/enrichment.rs

@@ -79,7 +79,10 @@ pub fn subagent_notification(
     if !payload.is_object() {
         return None;
     }
-    let agent_id = field(&payload, "agent_id").trim().to_owned();
+    let agent_id = match field(&payload, "agent_id").trim() {
+        "" => field(&payload, "agent_path").trim().to_owned(),
+        agent_id => agent_id.to_owned(),
+    };
     let nickname = field(&payload, "nickname").trim().to_owned();
     let completed = if payload["status"].is_object() {
         field(&payload["status"], "completed").trim().to_owned()

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/codex/mod.rs

@@ -6,7 +6,9 @@ use crate::compat::value::text;
 use crate::providers::contract::{
     DiagnosticSink, Provider, RecoverableDiagnostic,
 };
-use crate::providers::files::{self as file_sessions, SourceRoots};
+use crate::providers::files::{
+    self as file_sessions, SourceFiles, SourceRoots,
+};
 use crate::providers::jsonl;
 use crate::providers::title::{basename, normalize_title};
 use crate::session::timestamp::Timestamp;
@@ -87,7 +89,7 @@ impl Codex {
             .cloned())
     }
 
-    fn files(&mut self) -> crate::Result<Vec<PathBuf>> {
+    fn files(&mut self) -> crate::Result<SourceFiles> {
         self.roots.files(None, |path, _| {
             path.extension().is_some_and(|ext| ext == "jsonl")
         })
@@ -219,7 +221,7 @@ impl Provider for Codex {
     ) -> crate::Result<crate::providers::contract::Discovery> {
         self.titles = None;
         file_sessions::discover(
-            &self.files()?,
+            self.files()?,
             days,
             true,
             Self::scan,
@@ -237,7 +239,7 @@ impl Provider for Codex {
         let files = self.files()?;
         file_sessions::find(
             self.roots.base.clone().as_deref(),
-            &files,
+            files,
             id,
             |path| {
                 path.file_name().is_some_and(|name| {
@@ -280,6 +282,10 @@ impl Provider for Codex {
     fn source_root(&self) -> &Path {
         self.roots.owned()
     }
+
+    fn select_source(&mut self) -> crate::Result<Option<&Path>> {
+        self.roots.select()
+    }
 }
 
 #[cfg(test)]

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/codex/transcript.rs

@@ -154,13 +154,17 @@ impl Decoder {
             })
             .collect::<Vec<_>>()
             .join("\n\n");
+        let is_notification = user_text.starts_with("<subagent_notification>")
+            && user_text.ends_with("</subagent_notification>");
         if self.pending_plan.is_some()
             && role == "user"
             && !user_text.is_empty()
+            && !is_notification
         {
             if user_text
                 .trim_start()
                 .starts_with("PLEASE IMPLEMENT THIS PLAN")
+                || user_text == "Implement the plan."
             {
                 self.finish_plan("success", None);
             } else {
@@ -343,10 +347,13 @@ pub fn read(
 ) -> crate::Result<SessionData> {
     let mut decoder = Decoder::default();
     let mut stats = Stats::default();
+    let mut usage = Value::Null;
     jsonl::scan(&session.source_path, diagnostics, |record, diagnostics| {
-        let result = decoder.record(&record, zh).and_then(|()| {
-            let usage = &record["payload"]["info"]["total_token_usage"];
-            stats.add_tokens(&usage["input_tokens"], &usage["output_tokens"])
+        let result = decoder.record(&record, zh).map(|()| {
+            let snapshot = &record["payload"]["info"]["total_token_usage"];
+            if snapshot.is_object() {
+                usage = snapshot.clone();
+            }
         });
         if let Err(error) = result {
             diagnostics(RecoverableDiagnostic::MessageConvertFailed(
@@ -356,6 +363,7 @@ pub fn read(
         Ok(())
     })?;
     decoder.finish_plan("fail", None);
+    stats.add_tokens(&usage["input_tokens"], &usage["output_tokens"])?;
     stats.message_count = decoder.messages.len();
     let mut data = session.payload(decoder.messages, stats);
     data.directory = session.source_metadata["cwd"].clone();

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/contract.rs

@@ -98,6 +98,7 @@ pub trait Provider: Send + Sync {
         diagnostics: &mut DiagnosticSink<'_>,
     ) -> crate::Result<SessionData>;
     fn source_root(&self) -> &Path;
+    fn select_source(&mut self) -> crate::Result<Option<&Path>>;
     fn search_roots(&self) -> crate::Result<Vec<(&'static str, PathBuf)>>;
 
     fn change_sources(&self, session: &Session) -> Vec<PathBuf> {

{agent_dump-1.2.2 → agent_dump-1.2.4}/crates/agent-dump-core/src/providers/cursor/mod.rs

@@ -163,6 +163,11 @@ impl Provider for Cursor {
     fn source_root(&self) -> &Path {
         self.database.parent().unwrap()
     }
+
+    fn select_source(&mut self) -> crate::Result<Option<&Path>> {
+        self.database = (self.resolve_database)()?;
+        Ok(self.database.exists().then(|| self.source_root()))
+    }
     fn supports_format(&self, format: OutputFormat) -> bool {
         matches!(format, OutputFormat::Json | OutputFormat::Print)
     }