handoff-mcp-server 0.24.5 → 0.24.6

package/Cargo.lock

@@ -145,7 +145,7 @@ dependencies = [
 
 [[package]]
 name = "handoff-mcp"
-version = "0.24.5"
+version = "0.24.6"
 dependencies = [
  "anyhow",
  "chrono",

package/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "handoff-mcp"
-version = "0.24.5"
+version = "0.24.6"
 edition = "2021"
 description = "MCP server that gives AI coding agents persistent memory across sessions"
 license = "MIT"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "handoff-mcp-server",
-  "version": "0.24.5",
+  "version": "0.24.6",
   "description": "MCP server that gives AI coding agents persistent memory across sessions",
   "license": "MIT",
   "author": "AlphaElements <66808803+alphaelements@users.noreply.github.com>",

package/src/mcp/handlers/save_context.rs

@@ -130,11 +130,16 @@ pub fn handle(arguments: &Value) -> Result<String> {
         if let Some(active_session) = target {
             let sid = active_session.id.clone().unwrap_or_default();
             if keep_active {
-                let updated_path = update_active_session(&sessions_dir, &sid, &handoff_updates)?;
+                let updated_path =
+                    update_active_session(&sessions_dir, &sid, &handoff_updates, Some(arguments))?;
                 (0, updated_path, Some(sid))
             } else {
-                let closed_path =
-                    update_and_close_active_session(&sessions_dir, &sid, &handoff_updates)?;
+                let closed_path = update_and_close_active_session(
+                    &sessions_dir,
+                    &sid,
+                    &handoff_updates,
+                    Some(arguments),
+                )?;
                 (
                     if closed_path.is_some() { 1 } else { 0 },
                     closed_path,

package/src/mcp/handlers/update_session.rs

@@ -153,11 +153,13 @@ pub fn handle(arguments: &Value) -> Result<String> {
     Ok(msg)
 }
 
-fn truncate(s: &str, max: usize) -> String {
-    if s.len() <= max {
-        s.to_string()
+fn truncate(s: &str, max_chars: usize) -> String {
+    let mut chars = s.chars();
+    let truncated: String = chars.by_ref().take(max_chars).collect();
+    if chars.next().is_some() {
+        format!("{truncated}...")
     } else {
-        format!("{}...", &s[..max])
+        truncated
     }
 }
 

package/src/storage/sessions.rs

@@ -384,18 +384,37 @@ pub fn read_latest_closed_session(sessions_dir: &Path) -> Result<Option<SessionD
     Ok(sessions.into_iter().last())
 }
 
-fn apply_session_updates(data: &mut SessionData, updates: &SessionData) {
+fn apply_session_updates(
+    data: &mut SessionData,
+    updates: &SessionData,
+    provided_keys: Option<&Value>,
+) {
     data.summary = updates.summary.clone();
     data.ended_at = updates.ended_at.clone();
     data.branch = updates.branch.clone();
     data.commit = updates.commit.clone();
     data.dirty_files = updates.dirty_files.clone();
-    data.decisions = updates.decisions.clone();
-    data.blockers = updates.blockers.clone();
-    data.checklist = updates.checklist.clone();
-    data.handoff_notes = updates.handoff_notes.clone();
-    data.references = updates.references.clone();
-    data.context_pointers = updates.context_pointers.clone();
+
+    let was_provided = |key: &str| provided_keys.and_then(|v| v.get(key)).is_some();
+
+    if was_provided("decisions") {
+        data.decisions = updates.decisions.clone();
+    }
+    if was_provided("blockers") {
+        data.blockers = updates.blockers.clone();
+    }
+    if was_provided("checklist") {
+        data.checklist = updates.checklist.clone();
+    }
+    if was_provided("handoff_notes") {
+        data.handoff_notes = updates.handoff_notes.clone();
+    }
+    if was_provided("references") {
+        data.references = updates.references.clone();
+    }
+    if was_provided("context_pointers") {
+        data.context_pointers = updates.context_pointers.clone();
+    }
     if updates.environment.is_some() {
         data.environment = updates.environment.clone();
     }
@@ -405,7 +424,7 @@ fn apply_session_updates(data: &mut SessionData, updates: &SessionData) {
     if updates.label.is_some() {
         data.label = updates.label.clone();
     }
-    if !updates.related_task_ids.is_empty() {
+    if was_provided("related_task_ids") && !updates.related_task_ids.is_empty() {
         data.related_task_ids = updates.related_task_ids.clone();
     }
 }
@@ -415,6 +434,7 @@ fn find_and_update_active_session(
     session_id: &str,
     updates: &SessionData,
     transition_to: Option<&str>,
+    provided_keys: Option<&Value>,
 ) -> Result<Option<PathBuf>> {
     let suffix = ".active.json";
 
@@ -464,7 +484,7 @@ fn find_and_update_active_session(
         let mut data: SessionData = serde_json::from_str(&content)
             .with_context(|| format!("Failed to parse session: {}", path.display()))?;
 
-        apply_session_updates(&mut data, updates);
+        apply_session_updates(&mut data, updates, provided_keys);
 
         let updated_content =
             serde_json::to_string_pretty(&data).context("Failed to serialize session")?;
@@ -495,16 +515,24 @@ pub fn update_and_close_active_session(
     sessions_dir: &Path,
     session_id: &str,
     updates: &SessionData,
+    provided_keys: Option<&Value>,
 ) -> Result<Option<PathBuf>> {
-    find_and_update_active_session(sessions_dir, session_id, updates, Some("closed"))
+    find_and_update_active_session(
+        sessions_dir,
+        session_id,
+        updates,
+        Some("closed"),
+        provided_keys,
+    )
 }
 
 pub fn update_active_session(
     sessions_dir: &Path,
     session_id: &str,
     updates: &SessionData,
+    provided_keys: Option<&Value>,
 ) -> Result<Option<PathBuf>> {
-    find_and_update_active_session(sessions_dir, session_id, updates, None)
+    find_and_update_active_session(sessions_dir, session_id, updates, None, provided_keys)
 }
 
 pub fn read_session_by_id(sessions_dir: &Path, session_id: &str) -> Result<Option<SessionData>> {