feat: surface hook metadata & tool result JSON; fix PreToolUse attachment dropped by live-chain filter; skip empty redacted thinking items

- Fix: attachment entries (hook results) were silently dropped by the live-chain UUID
  filter in read_session_incremental — their uuid is never referenced as anyone's
  parentUuid, so they never entered the live set. Now passed through when parentUuid
  is on the live chain.

- feat: extract all key-value pairs from hook attachments as hook_metadata (Option<Value>)
  and from tool results as tool_result_json (Option<Value>); propagated through
  HookMsg → ContentBlock → DisplayItem → FrontendDisplayItem → shared/types.ts

- fix: skip Thinking DisplayItems when thinking text is empty (extended thinking
  blocks store only an encrypted signature in JSONL; thinking_count badge still reflects
  the correct count)

- chore: bump version 0.4.12 → 0.4.13

Author: delexw

AGENTS.md

@@ -2,7 +2,7 @@
 
 # GitNexus — Code Intelligence
 
-This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 relationships, 146 execution flows). Use the GitNexus MCP tools to understand code, assess impact, and navigate safely.
+This project is indexed by GitNexus as **tail-claude-gui** (1766 symbols, 4194 relationships, 148 execution flows). Use the GitNexus MCP tools to understand code, assess impact, and navigate safely.
 
 > If any GitNexus tool warns the index is stale, run `npx gitnexus analyze` in terminal first.
 
@@ -18,7 +18,7 @@ This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 rela
 
 1. `gitnexus_query({query: "<error or symptom>"})` — find execution flows related to the issue
 2. `gitnexus_context({name: "<suspect function>"})` — see all callers, callees, and process participation
-3. `READ gitnexus://repo/cct-626ddc17/process/{processName}` — trace the full execution flow step by step
+3. `READ gitnexus://repo/tail-claude-gui/process/{processName}` — trace the full execution flow step by step
 4. For regressions: `gitnexus_detect_changes({scope: "compare", base_ref: "main"})` — see what your branch changed
 
 ## When Refactoring
@@ -55,12 +55,12 @@ This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 rela
 
 ## Resources
 
-| Resource                                      | Use for                                  |
-| --------------------------------------------- | ---------------------------------------- |
-| `gitnexus://repo/cct-626ddc17/context`        | Codebase overview, check index freshness |
-| `gitnexus://repo/cct-626ddc17/clusters`       | All functional areas                     |
-| `gitnexus://repo/cct-626ddc17/processes`      | All execution flows                      |
-| `gitnexus://repo/cct-626ddc17/process/{name}` | Step-by-step execution trace             |
+| Resource                                         | Use for                                  |
+| ------------------------------------------------ | ---------------------------------------- |
+| `gitnexus://repo/tail-claude-gui/context`        | Codebase overview, check index freshness |
+| `gitnexus://repo/tail-claude-gui/clusters`       | All functional areas                     |
+| `gitnexus://repo/tail-claude-gui/processes`      | All execution flows                      |
+| `gitnexus://repo/tail-claude-gui/process/{name}` | Step-by-step execution trace             |
 
 ## Self-Check Before Finishing
 

CLAUDE.md

@@ -38,7 +38,7 @@ npx oxfmt && npx oxlint && npx tsc --noEmit && npx vitest run && cargo fmt --man
 
 # GitNexus — Code Intelligence
 
-This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 relationships, 146 execution flows). Use the GitNexus MCP tools to understand code, assess impact, and navigate safely.
+This project is indexed by GitNexus as **tail-claude-gui** (1766 symbols, 4194 relationships, 148 execution flows). Use the GitNexus MCP tools to understand code, assess impact, and navigate safely.
 
 > If any GitNexus tool warns the index is stale, run `npx gitnexus analyze` in terminal first.
 
@@ -54,7 +54,7 @@ This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 rela
 
 1. `gitnexus_query({query: "<error or symptom>"})` — find execution flows related to the issue
 2. `gitnexus_context({name: "<suspect function>"})` — see all callers, callees, and process participation
-3. `READ gitnexus://repo/cct-626ddc17/process/{processName}` — trace the full execution flow step by step
+3. `READ gitnexus://repo/tail-claude-gui/process/{processName}` — trace the full execution flow step by step
 4. For regressions: `gitnexus_detect_changes({scope: "compare", base_ref: "main"})` — see what your branch changed
 
 ## When Refactoring
@@ -91,12 +91,12 @@ This project is indexed by GitNexus as **cct-626ddc17** (1747 symbols, 4138 rela
 
 ## Resources
 
-| Resource                                      | Use for                                  |
-| --------------------------------------------- | ---------------------------------------- |
-| `gitnexus://repo/cct-626ddc17/context`        | Codebase overview, check index freshness |
-| `gitnexus://repo/cct-626ddc17/clusters`       | All functional areas                     |
-| `gitnexus://repo/cct-626ddc17/processes`      | All execution flows                      |
-| `gitnexus://repo/cct-626ddc17/process/{name}` | Step-by-step execution trace             |
+| Resource                                         | Use for                                  |
+| ------------------------------------------------ | ---------------------------------------- |
+| `gitnexus://repo/tail-claude-gui/context`        | Codebase overview, check index freshness |
+| `gitnexus://repo/tail-claude-gui/clusters`       | All functional areas                     |
+| `gitnexus://repo/tail-claude-gui/processes`      | All execution flows                      |
+| `gitnexus://repo/tail-claude-gui/process/{name}` | Step-by-step execution trace             |
 
 ## Self-Check Before Finishing
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-trace",
-  "version": "0.4.12",
+  "version": "0.4.13",
   "private": true,
   "bin": {
     "cctrace": "./bin/cctrace.mjs"

package.json

@@ -55,6 +55,10 @@ export interface DisplayItem {
   hook_event: string;
   hook_name: string;
   hook_command: string;
+  /** All key-value pairs from the hook attachment JSON (pretty-printed). */
+  hook_metadata: string;
+  /** Tool result as pretty-printed JSON when the content is an object or array. */
+  tool_result_json: string;
   is_orphan: boolean;
 }
 

shared/types.ts

@@ -1,6 +1,6 @@
 [package]
 name = "claude-code-trace"
-version = "0.4.12"
+version = "0.4.13"
 edition = "2021"
 rust-version = "1.77.2"
 

src-tauri/Cargo.lock

@@ -39,6 +39,10 @@ pub struct FrontendDisplayItem {
     pub hook_event: String,
     pub hook_name: String,
     pub hook_command: String,
+    /// All key-value pairs from the hook attachment JSON (pretty-printed).
+    pub hook_metadata: String,
+    /// Tool result as pretty-printed JSON when the content is an object or array.
+    pub tool_result_json: String,
     pub is_orphan: bool,
     pub subagent_prompt: String,
     pub is_deferred: bool,
@@ -247,6 +251,8 @@ fn convert_display_items(
                 hook_event: it.hook_event.clone(),
                 hook_name: it.hook_name.clone(),
                 hook_command: it.hook_command.clone(),
+                hook_metadata: pretty_json(&it.hook_metadata),
+                tool_result_json: pretty_json(&it.tool_result_json),
                 is_orphan: it.is_orphan,
                 subagent_prompt: String::new(),
                 is_deferred: it.is_deferred,

src-tauri/Cargo.toml

@@ -42,6 +42,10 @@ pub struct DisplayItem {
     pub hook_event: String,
     pub hook_name: String,
     pub hook_command: String,
+    /// All key-value pairs from the hook attachment JSON.
+    pub hook_metadata: Option<serde_json::Value>,
+    /// Tool result parsed as a JSON value when the content is an object or array.
+    pub tool_result_json: Option<serde_json::Value>,
     pub is_orphan: bool,
     /// True when the session was suspended via a "defer" PreToolUse permission decision
     /// before a tool_result arrived for this tool_use block.
@@ -70,6 +74,8 @@ impl Default for DisplayItem {
             hook_event: String::new(),
             hook_name: String::new(),
             hook_command: String::new(),
+            hook_metadata: None,
+            tool_result_json: None,
             is_orphan: false,
             is_deferred: false,
         }
@@ -198,6 +204,7 @@ pub fn build_chunks(msgs: &[ClassifiedMsg]) -> Vec<Chunk> {
                         text: m.command.clone(),
                         tool_name: m.hook_name.clone(),
                         tool_id: m.hook_event.clone(),
+                        hook_metadata: m.metadata.clone(),
                         ..Default::default()
                     }],
                     usage: Usage::default(),
@@ -258,11 +265,17 @@ fn merge_ai_buffer(buf: &[AIMsg]) -> Chunk {
             for b in &m.blocks {
                 match b.block_type.as_str() {
                     "thinking" => {
-                        items.push(DisplayItem {
-                            item_type: DisplayItemType::Thinking,
-                            text: b.text.clone(),
-                            ..Default::default()
-                        });
+                        // Extended thinking blocks store an encrypted signature in the JSONL
+                        // but redact the actual text (thinking field is ""). Only emit a
+                        // Thinking DisplayItem when there is real content to show; thinking_count
+                        // already tracks the presence of thinking blocks regardless.
+                        if !b.text.is_empty() {
+                            items.push(DisplayItem {
+                                item_type: DisplayItemType::Thinking,
+                                text: b.text.clone(),
+                                ..Default::default()
+                            });
+                        }
                     }
                     "text" => {
                         items.push(DisplayItem {
@@ -318,6 +331,7 @@ fn merge_ai_buffer(buf: &[AIMsg]) -> Chunk {
                     "tool_result" => {
                         if let Some(p) = pending.remove(&b.tool_id) {
                             items[p.index].tool_result = b.content.clone();
+                            items[p.index].tool_result_json = b.content_json.clone();
                             items[p.index].tool_error = b.is_error;
                             let dur = m.timestamp.signed_duration_since(p.timestamp);
                             items[p.index].duration_ms = dur.num_milliseconds();
@@ -344,6 +358,7 @@ fn merge_ai_buffer(buf: &[AIMsg]) -> Chunk {
                             hook_event: b.tool_id.clone(),
                             hook_name: b.tool_name.clone(),
                             hook_command: b.text.clone(),
+                            hook_metadata: b.hook_metadata.clone(),
                             ..Default::default()
                         });
                     }

src-tauri/src/convert.rs

@@ -40,6 +40,10 @@ pub struct ContentBlock {
     pub is_error: bool,
     pub teammate_id: String,
     pub teammate_color: String,
+    /// For hook_event blocks: all key-value pairs from the hook attachment JSON.
+    pub hook_metadata: Option<Value>,
+    /// For tool_result blocks: raw content value before stringification.
+    pub content_json: Option<Value>,
 }
 
 /// Classified message types.
@@ -100,6 +104,9 @@ pub struct HookMsg {
     pub hook_event: String,
     pub hook_name: String,
     pub command: String,
+    /// All key-value pairs from the hook attachment JSON (stdout, stderr, command,
+    /// exitCode, durationMs, additionalContext, decision, reason, …).
+    pub metadata: Option<Value>,
 }
 
 pub const SYSTEM_OUTPUT_TAGS: &[&str] = &[
@@ -172,6 +179,7 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                     hook_event,
                     hook_name,
                     command,
+                    metadata: None,
                 }));
             }
         }
@@ -197,6 +205,7 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                     hook_event: "Stop".to_string(),
                     hook_name,
                     command: String::new(),
+                    metadata: None,
                 }));
             }
             // hook_progress: written in verbose/stream-json mode for mid-session hooks.
@@ -206,6 +215,7 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                     hook_event: e.hook_event.clone(),
                     hook_name: e.hook_name.clone(),
                     command: String::new(),
+                    metadata: None,
                 }));
             }
             // hookEvent present on any system entry (forward-compat for future hook types).
@@ -215,6 +225,7 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                     hook_event: e.hook_event.clone(),
                     hook_name: e.hook_name.clone(),
                     command: String::new(),
+                    metadata: None,
                 }));
             }
             _ => {}
@@ -253,11 +264,17 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                 }
                 .trim()
                 .to_string();
+                // Store the whole attachment object so all key-value pairs are available:
+                // type, hookName, hookEvent, command, stdout, stderr, exitCode, durationMs,
+                // additionalContext (when stdout is parsed JSON), decision, reason, etc.
+                // Any nested JSON strings (e.g. stdout) are left for the frontend to expand.
+                let metadata = Some(att.clone());
                 return Some(ClassifiedMsg::Hook(HookMsg {
                     timestamp: ts,
                     hook_event: hook_event.to_string(),
                     hook_name,
                     command,
+                    metadata,
                 }));
             }
         }
@@ -300,6 +317,7 @@ pub fn classify(e: Entry) -> Option<ClassifiedMsg> {
                 hook_event: "Stop".to_string(),
                 hook_name,
                 command,
+                metadata: None,
             }));
         }
     }
@@ -677,14 +695,19 @@ fn extract_meta_blocks(content: &Option<Value>, text_fallback: &str) -> Vec<Cont
                 .and_then(|v| v.as_str())
                 .unwrap_or("")
                 .to_string();
-            let raw_content = stringify_content(&b.get("content").cloned());
+            let raw_content_val = b.get("content").cloned();
+            let raw_content = stringify_content(&raw_content_val);
             let content = resolve_persisted_output(&raw_content);
             let is_error = b.get("is_error").and_then(|v| v.as_bool()).unwrap_or(false);
+            // Keep the raw Value so callers can access all key-value pairs when the
+            // tool result is a JSON object (not just the stringified form).
+            let content_json = raw_content_val.filter(|v| v.is_object() || v.is_array());
             Some(ContentBlock {
                 block_type: "tool_result".to_string(),
                 tool_id,
                 content,
                 is_error,
+                content_json,
                 ..Default::default()
             })
         })