[Memory] Thought counter increments during idle periods without user input #317

Closed
opened 2026-03-18 22:54:34 +00:00 by Timmy · 1 comment
Owner

What's happening: The thought counter (e.g., 189 thoughts vs 8 messages) increments during extended idle periods (e.g., 53-minute silence), counting internal processing as thoughts.

Expected behavior: The counter should only increment when new user input or explicit thought triggers occur, not during silent internal processing.

Suggested fix: Modify memory_system.py's increment_counter() method to only increment when is_user_input flag is true (e.g., after message processing). Add guard clause:

if self.last_input_time < datetime.now() - timedelta(minutes=1):
    return  # Skip idle increments

Acceptance criteria: System remains idle for 60 minutes → thought counter remains unchanged. New user message arrives → counter increments by 1.


Auto-filed by Timmy's thinking engine

**What's happening:** The thought counter (e.g., 189 thoughts vs 8 messages) increments during extended idle periods (e.g., 53-minute silence), counting internal processing as thoughts. **Expected behavior:** The counter should only increment when new user input or explicit thought triggers occur, not during silent internal processing. **Suggested fix:** Modify `memory_system.py`'s `increment_counter()` method to only increment when `is_user_input` flag is true (e.g., after message processing). Add guard clause: ```python if self.last_input_time < datetime.now() - timedelta(minutes=1): return # Skip idle increments ``` **Acceptance criteria:** System remains idle for 60 minutes → thought counter remains unchanged. New user message arrives → counter increments by 1. --- *Auto-filed by Timmy's thinking engine*
Collaborator

Fixed via PR #319 (merged). Thought counter now pauses after 60min idle timeout.

Fixed via PR #319 (merged). Thought counter now pauses after 60min idle timeout.
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#317