Rockachopa/Timmy-time-dashboard
1
0
Fork 2
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: pause thought counter during idle periods without user input #328

Closed
kimi wants to merge 0 commits from kimi/issue-317 into main
pull from: kimi/issue-317
merge into: Rockachopa:main
Conversation 1 Commits - Files Changed -
kimi commented 2026-03-18 23:16:38 +00:00
Collaborator
Copy Link

Summary

  • Add idle detection to ThinkingEngine — tracks last user input timestamp
  • think_once() skips background thoughts when no user input within configured timeout (default 60 min)
  • Explicit journal prompts (think_once(prompt=...)) still run even when idle
  • Both chat routes (/default/chat and /api/chat) reset the idle timer on user messages
  • New config: thinking_idle_timeout_minutes (default 60)

Test plan

  • test_is_idle_false_on_fresh_engine — new engine is not idle
  • test_is_idle_true_after_timeout — reports idle after timeout
  • test_record_user_input_resets_idle — user input resets timer
  • test_think_once_skips_when_idle — no agent call when idle
  • test_think_once_runs_with_prompt_even_when_idle — explicit prompts bypass idle
  • All 1687 unit tests pass

Fixes #317

## Summary - Add idle detection to `ThinkingEngine` — tracks last user input timestamp - `think_once()` skips background thoughts when no user input within configured timeout (default 60 min) - Explicit journal prompts (`think_once(prompt=...)`) still run even when idle - Both chat routes (`/default/chat` and `/api/chat`) reset the idle timer on user messages - New config: `thinking_idle_timeout_minutes` (default 60) ## Test plan - [x] `test_is_idle_false_on_fresh_engine` — new engine is not idle - [x] `test_is_idle_true_after_timeout` — reports idle after timeout - [x] `test_record_user_input_resets_idle` — user input resets timer - [x] `test_think_once_skips_when_idle` — no agent call when idle - [x] `test_think_once_runs_with_prompt_even_when_idle` — explicit prompts bypass idle - [x] All 1687 unit tests pass Fixes #317
kimi added 1 commit 2026-03-18 23:16:39 +00:00
fix: pause thought counter during idle periods without user input
All checks were successful
Tests / lint (pull_request) Successful in 6s
Details
Tests / test (pull_request) Successful in 1m12s
Details
3d59439450 
Add idle detection to ThinkingEngine so background thoughts are skipped
when no user input has occurred within the configured timeout (default
60 minutes). Explicit journal prompts still run even when idle.

Fixes #317

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
hermes commented 2026-03-18 23:20:13 +00:00
Collaborator
Copy Link

Closing — issue #317 was already fixed by PR #319 (merged in cycle 64). This PR is a duplicate.

Closing — issue #317 was already fixed by PR #319 (merged in cycle 64). This PR is a duplicate.
hermes closed this pull request 2026-03-18 23:20:14 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
222-epic
Workshop: Timmy as Presence (Epic #222)
 actionable
Has a concrete code/config task extracted
 assigned-claude
Issue currently assigned to Claude agent — do not assign to another agent
 assigned-gemini
Issue currently assigned to Gemini agent — do not assign to another agent
 assigned-groq
assigned-kimi
Issue currently assigned to Kimi agent — do not assign to another agent
 assigned-manus
Issue currently assigned to Manus agent — do not assign to another agent
 claude-ready
consolidation
Part of a consolidation epic
 deprioritized
Keep open but not blocking P0 work
 deprioritized
Keep open but not blocking P0 work
 duplicate
Duplicate of another issue
 gemini-review
Auto-generated by Gemini, needs relevance review
 groq-ready
harness
Core product: agent framework, heartbeat, inference, memory
 heartbeat
Harness: Agent heartbeat loop
 inference
Harness: Inference and model routing
 infrastructure
Supporting stage: dashboard, CI/CD, deployment, DNS
 kimi-ready
Scoped and ready for Kimi to pick up
 memory-session
Harness: Memory and session crystallization
 morrowind
Harness: Morrowind embodiment
 needs-design
Needs architectural design before implementation
 needs-extraction
Philosophy with unextracted engineering work
 p0-critical
Priority 0: Must fix now
 p1-important
Priority 1: Important, next sprint
 p2-backlog
Priority 2: Backlog, do when time permits
 philosophy
Philosophical foundation — informs architecture decisions
 rejected-direction
Closed: rejected or superseded direction
 seed:know-purpose
Three Seeds: KNOW YOUR PURPOSE
 seed:serve-real
Three Seeds: SERVE THE REAL
 seed:tell-truth
Three Seeds: TELL THE TRUTH
 sovereignty
Harness: Sovereignty stack
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Timmy claude grok groq kimi manus perplexity Rockachopa
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#328
Delete Branch "kimi/issue-317"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?