From 773dad256e260f90d1d3113e1135f1f8362c7c1f Mon Sep 17 00:00:00 2001 From: Al Date: Thu, 22 Jan 2026 15:22:20 -0500 Subject: [PATCH] fix(session-memory): suppress user-visible confirmation message The session-memory hook saves session context to memory files when /new is run, which is useful internal housekeeping. However, the confirmation message that was displayed to users (showing the file path) leaked implementation details. This change removes the user-visible message while keeping the console.log for debugging purposes. The hook continues to save session context silently. --- src/hooks/bundled/session-memory/handler.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/hooks/bundled/session-memory/handler.ts b/src/hooks/bundled/session-memory/handler.ts index bcce11cea..14b486fcc 100644 --- a/src/hooks/bundled/session-memory/handler.ts +++ b/src/hooks/bundled/session-memory/handler.ts @@ -160,11 +160,9 @@ const saveSessionToMemory: HookHandler = async (event) => { await fs.writeFile(memoryFilePath, entry, "utf-8"); console.log("[session-memory] Memory file written successfully"); - // Send confirmation message to user with filename + // Log completion (but don't send user-visible confirmation - it's internal housekeeping) const relPath = memoryFilePath.replace(os.homedir(), "~"); - const confirmMsg = `šŸ’¾ Session context saved to memory before reset.\nšŸ“„ ${relPath}`; - event.messages.push(confirmMsg); - console.log("[session-memory] Confirmation message queued:", confirmMsg); + console.log(`[session-memory] Session context saved to ${relPath}`); } catch (err) { console.error( "[session-memory] Failed to save session memory:",