handoff-mcp-server 0.7.2 → 0.7.3

package/Cargo.lock

@@ -144,7 +144,7 @@ dependencies = [
 
 [[package]]
 name = "handoff-mcp"
-version = "0.7.2"
+version = "0.7.3"
 dependencies = [
  "anyhow",
  "chrono",

package/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "handoff-mcp"
-version = "0.7.2"
+version = "0.7.3"
 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.7.2",
+  "version": "0.7.3",
   "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/import_context.rs

@@ -7,8 +7,7 @@ use crate::storage::config::read_config;
 use crate::storage::ensure_handoff_exists;
 use crate::storage::git::capture_git_state;
 use crate::storage::sessions::{
-    close_active_sessions, close_open_sessions, enforce_history_limit, write_open_session,
-    SessionData,
+    close_active_sessions, enforce_history_limit, write_open_session, SessionData,
 };
 use crate::storage::tasks::*;
 
@@ -60,7 +59,6 @@ pub fn handle(arguments: &Value) -> Result<String> {
             })?;
 
         close_active_sessions(&sessions_dir)?;
-        let closed = close_open_sessions(&sessions_dir)?;
         let git_state = capture_git_state(&project_dir)?;
         let now = Utc::now().to_rfc3339();
 
@@ -117,12 +115,10 @@ pub fn handle(arguments: &Value) -> Result<String> {
         };
         enforce_history_limit(&sessions_dir, history_limit)?;
 
-        let _ = closed;
         session_saved = true;
     } else if let Some(raw_notes) = arguments.get("raw_notes").and_then(|v| v.as_str()) {
         if !raw_notes.is_empty() {
             close_active_sessions(&sessions_dir)?;
-            let closed = close_open_sessions(&sessions_dir)?;
             let git_state = capture_git_state(&project_dir)?;
             let now = Utc::now().to_rfc3339();
 
@@ -162,7 +158,6 @@ pub fn handle(arguments: &Value) -> Result<String> {
             };
             enforce_history_limit(&sessions_dir, history_limit)?;
 
-            let _ = closed;
             session_saved = true;
         }
     }

package/src/mcp/handlers/save_context.rs

@@ -9,9 +9,9 @@ use crate::storage::config::read_config;
 use crate::storage::ensure_handoff_exists;
 use crate::storage::git::capture_git_state;
 use crate::storage::sessions::{
-    close_active_sessions, close_open_sessions, close_session_by_id, enforce_history_limit,
-    generate_session_id, pause_active_sessions, pause_session_by_id, read_active_sessions,
-    write_open_session, SessionData,
+    close_active_sessions, close_session_by_id, enforce_history_limit, generate_session_id,
+    pause_active_sessions, pause_session_by_id, read_active_sessions, write_session_with_status,
+    SessionData,
 };
 
 pub fn handle(arguments: &Value) -> Result<String> {
@@ -32,6 +32,17 @@ pub fn handle(arguments: &Value) -> Result<String> {
         .get("pause_active")
         .and_then(|v| v.as_bool())
         .unwrap_or(false);
+    let session_status = arguments
+        .get("session_status")
+        .and_then(|v| v.as_str())
+        .unwrap_or("open");
+
+    if session_status != "open" && session_status != "active" {
+        anyhow::bail!(
+            "Invalid session_status '{}': must be 'open' or 'active'",
+            session_status
+        );
+    }
 
     let mut total_paused = 0usize;
     if let Some(id) = pause_id {
@@ -50,8 +61,7 @@ pub fn handle(arguments: &Value) -> Result<String> {
             0
         }
     } else if pause_id.is_some() || pause_all {
-        let closed_open = close_open_sessions(&sessions_dir)?;
-        closed_open.len()
+        0
     } else {
         let active = read_active_sessions(&sessions_dir)?;
         if active.len() > 1 {
@@ -64,8 +74,7 @@ pub fn handle(arguments: &Value) -> Result<String> {
             );
         }
         let closed_active = close_active_sessions(&sessions_dir)?;
-        let closed_open = close_open_sessions(&sessions_dir)?;
-        closed_active.len() + closed_open.len()
+        closed_active.len()
     };
 
     let git_state = capture_git_state(&project_dir)?;
@@ -89,7 +98,7 @@ pub fn handle(arguments: &Value) -> Result<String> {
         environment: arguments.get("environment").cloned(),
     };
 
-    let path = write_open_session(&sessions_dir, &data)?;
+    let path = write_session_with_status(&sessions_dir, &data, session_status)?;
 
     let history_limit = if config_path.exists() {
         read_config(&config_path)

package/src/mcp/tools.rs

@@ -57,9 +57,15 @@ pub fn all_tool_definitions() -> Vec<ToolDefinition> {
                         "type": "string",
                         "description": "One-line summary of this session"
                     },
+                    "session_status": {
+                        "type": "string",
+                        "description": "Status of the new session: 'open' (default) = saved but not activated, allows multiple open sessions; 'active' = immediately active for this session.",
+                        "enum": ["open", "active"],
+                        "default": "open"
+                    },
                     "close_session_id": {
                         "type": "string",
-                        "description": "Session ID to close. If omitted (and no pause options set), all active and open sessions are closed."
+                        "description": "Session ID to close. If omitted (and no pause options set), active sessions are closed."
                     },
                     "pause_session_id": {
                         "type": "string",

package/src/storage/sessions.rs

@@ -321,8 +321,24 @@ pub fn enforce_history_limit(sessions_dir: &Path, limit: u32) -> Result<u32> {
     Ok(removed)
 }
 
-// Backward-compatible aliases for tests and migration
-#[doc(hidden)]
-pub fn write_active_session(sessions_dir: &Path, data: &SessionData) -> Result<PathBuf> {
-    write_open_session(sessions_dir, data)
+pub fn write_session_with_status(
+    sessions_dir: &Path,
+    data: &SessionData,
+    status: &str,
+) -> Result<PathBuf> {
+    let mut data = data.clone();
+    if data.id.is_none() {
+        data.id = Some(generate_session_id());
+    }
+
+    let ts_part = compact_timestamp(&data);
+    let base = generate_session_filename(&data.summary, &ts_part);
+    let filename = format!("{base}.{status}.json");
+    let path = sessions_dir.join(&filename);
+
+    let content = serde_json::to_string_pretty(&data).context("Failed to serialize session")?;
+    std::fs::write(&path, content)
+        .with_context(|| format!("Failed to write session: {}", path.display()))?;
+
+    Ok(path)
 }