fix(#160): add regression test for SessionStore lifecycle (list_sessions, delete_session, session_exists)

Adds session_store_lifecycle_regression_160 test that verifies the full
SessionStore CRUD lifecycle. Also fixes pre-existing non-exhaustive match
errors in trident.rs for the ContentBlock::Thinking variant.

Author: code-yeongyu

rust/crates/runtime/src/session_control.rs

@@ -1230,4 +1230,35 @@ mod tests {
         );
         fs::remove_dir_all(base).expect("temp dir should clean up");
     }
+
+    /// #160 regression: store-level list_sessions/session_exists/delete_session
+    /// lifecycle works end-to-end.
+    #[test]
+    fn session_store_lifecycle_regression_160() {
+        // given
+        let base = temp_dir();
+        fs::create_dir_all(&base).expect("base dir should exist");
+        let store = SessionStore::from_cwd(&base).expect("store should build");
+        let session = persist_session_via_store(&store, "160 regression test");
+
+        // when/then — session exists and is listed before deletion
+        assert!(!store.list_sessions().expect("list").is_empty(),
+            "store should have at least one session");
+        assert!(store.session_exists(&session.session_id),
+            "session should exist before deletion");
+
+        // when — delete the session
+        let deleted = store.delete_session(&session.session_id)
+            .expect("delete should succeed");
+
+        // then — session is gone
+        assert_eq!(deleted.id, session.session_id);
+        assert!(!deleted.path.exists(), "session file should be removed");
+        assert!(!store.session_exists(&session.session_id),
+            "session should not exist after deletion");
+        assert!(store.list_sessions().expect("list").is_empty(),
+            "store should have no sessions after deletion");
+
+        fs::remove_dir_all(base).expect("temp dir should clean up");
+    }
 }

rust/crates/runtime/src/trident.rs

@@ -246,6 +246,7 @@ fn extract_file_operation(block: &ContentBlock) -> Option<(String, FileOp)> {
             Some((path, op_type))
         }
         ContentBlock::Text { .. } => None,
+        ContentBlock::Thinking { .. } => None,
     }
 }
 
@@ -344,6 +345,7 @@ fn is_chatty_message(msg: &ConversationMessage) -> bool {
         ContentBlock::Text { text } => text.len(),
         ContentBlock::ToolUse { input, .. } => input.len(),
         ContentBlock::ToolResult { output, .. } => output.len(),
+        ContentBlock::Thinking { thinking, .. } => thinking.len(),
     }).sum();
 
     let has_tool_use = msg.blocks.iter().any(|b| matches!(b, ContentBlock::ToolUse { .. }));
@@ -528,6 +530,9 @@ fn fingerprint_message(index: usize, msg: &ConversationMessage) -> Option<Messag
             ContentBlock::Text { text } => {
                 text_length += text.len();
             }
+            ContentBlock::Thinking { thinking, .. } => {
+                text_length += thinking.len();
+            }
         }
     }
 
@@ -598,6 +603,7 @@ fn generate_cluster_summary(messages: &[&ConversationMessage]) -> String {
                     }
                 }
                 ContentBlock::Text { .. } => {}
+                ContentBlock::Thinking { .. } => {}
             }
         }
     }
@@ -633,8 +639,9 @@ fn estimate_message_tokens(message: &ConversationMessage) -> usize {
             ContentBlock::ToolResult {
                 tool_name, output, ..
             } => (tool_name.len() + output.len()) / 4 + 1,
+            ContentBlock::Thinking { thinking, .. } => thinking.len() / 4 + 1,
         })
-        .sum()
+.sum()
 }
 
 fn truncate_text(text: &str, max_chars: usize) -> String {