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.
This commit is contained in:
@@ -160,11 +160,9 @@ const saveSessionToMemory: HookHandler = async (event) => {
|
|||||||
await fs.writeFile(memoryFilePath, entry, "utf-8");
|
await fs.writeFile(memoryFilePath, entry, "utf-8");
|
||||||
console.log("[session-memory] Memory file written successfully");
|
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 relPath = memoryFilePath.replace(os.homedir(), "~");
|
||||||
const confirmMsg = `💾 Session context saved to memory before reset.\n📄 ${relPath}`;
|
console.log(`[session-memory] Session context saved to ${relPath}`);
|
||||||
event.messages.push(confirmMsg);
|
|
||||||
console.log("[session-memory] Confirmation message queued:", confirmMsg);
|
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
console.error(
|
console.error(
|
||||||
"[session-memory] Failed to save session memory:",
|
"[session-memory] Failed to save session memory:",
|
||||||
|
|||||||
Reference in New Issue
Block a user