From b515d70de7d6225a2c06a90f2e14edd99e1fdcaa Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Tue, 1 Jul 2025 11:10:26 +0100
Subject: [PATCH] fix: Properly track session.json debounce timer for cleanup

- Update session.sessionJsonDebounceTimer when creating new timers
- Clear timer reference after callback execution
- Remove incorrect initial null assignment
- Fixes memory leak where timers continued after session cleanup

The bug occurred because the timer ID was only stored in a local variable,
preventing proper cleanup during session termination.
---
 web/src/server/pty/pty-manager.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/server/pty/pty-manager.ts b/web/src/server/pty/pty-manager.ts
index 11324715..114481bd 100644
--- a/web/src/server/pty/pty-manager.ts
+++ b/web/src/server/pty/pty-manager.ts
@@ -818,7 +818,7 @@ export class PtyManager extends EventEmitter {
             // Clear the timer reference after execution
             session.sessionJsonDebounceTimer = null;
           }, 100);
-          
+
           // Update the session's timer reference
           session.sessionJsonDebounceTimer = debounceTimer;
         }