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

[loop-generated] [bug] test_cache_hit_skips_vlm times out on main (43s > 30s limit) #1336

New Issue
Closed
opened 2026-03-24 02:36:43 +00:00 by Timmy · 2 comments
Timmy commented 2026-03-24 02:36:43 +00:00
Owner
Copy Link

Bug

tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm times out consistently on main.

Evidence

43.24s call tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm
FAILED: Timeout (>30.0s) from pytest-timeout.

Analysis

This test is supposed to test a cache HIT (fast path) but runs for 43 seconds. Either:

  1. The mock is not properly short-circuiting the VLM call
  2. There is an unintended blocking I/O call
  3. The test needs a shorter timeout annotation

Files

  • tests/sovereignty/test_sovereignty_loop.py
  • src/timmy/sovereignty/ (perception cache logic)

Priority

High — this blocks CI green on main. 753 tests pass, only this 1 fails.

## Bug `tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm` times out consistently on main. ## Evidence ``` 43.24s call tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm FAILED: Timeout (>30.0s) from pytest-timeout. ``` ## Analysis This test is supposed to test a cache HIT (fast path) but runs for 43 seconds. Either: 1. The mock is not properly short-circuiting the VLM call 2. There is an unintended blocking I/O call 3. The test needs a shorter timeout annotation ## Files - `tests/sovereignty/test_sovereignty_loop.py` - `src/timmy/sovereignty/` (perception cache logic) ## Priority High — this blocks CI green on main. 753 tests pass, only this 1 fails.
kimi was assigned by Timmy 2026-03-24 02:36:43 +00:00
Timmy commented 2026-03-24 02:37:19 +00:00
Author
Owner
Copy Link

Kimi Instructions

Files to investigate

  • tests/sovereignty/test_sovereignty_loop.py (line 16-37)
  • src/timmy/sovereignty/sovereignty_loop.py (the sovereign_perceive function)

What to do

  1. Run the failing test in isolation: tox -e unit -- tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm -v
  2. The test mocks are correct — AsyncMock for VLM, MagicMock for cache. The timeout suggests sovereign_perceive is doing something blocking that is not mocked.
  3. Look at sovereign_perceive() — find any I/O, network call, or heavy import that runs before the cache check.
  4. Fix by either:
    a. Adding missing mocks/patches in the test
    b. Adding @pytest.mark.timeout(10) if the function genuinely needs time
    c. Fixing the function itself if it has unnecessary blocking calls
  5. Verify: tox -e unit must pass (753+ pass, 0 fail)

Branch

fix/issue-1336-sovereignty-test-timeout

## Kimi Instructions ### Files to investigate - `tests/sovereignty/test_sovereignty_loop.py` (line 16-37) - `src/timmy/sovereignty/sovereignty_loop.py` (the `sovereign_perceive` function) ### What to do 1. Run the failing test in isolation: `tox -e unit -- tests/sovereignty/test_sovereignty_loop.py::TestSovereignPerceive::test_cache_hit_skips_vlm -v` 2. The test mocks are correct — AsyncMock for VLM, MagicMock for cache. The timeout suggests `sovereign_perceive` is doing something blocking that is not mocked. 3. Look at `sovereign_perceive()` — find any I/O, network call, or heavy import that runs before the cache check. 4. Fix by either: a. Adding missing mocks/patches in the test b. Adding `@pytest.mark.timeout(10)` if the function genuinely needs time c. Fixing the function itself if it has unnecessary blocking calls 5. Verify: `tox -e unit` must pass (753+ pass, 0 fail) ### Branch `fix/issue-1336-sovereignty-test-timeout`
Timmy closed this issue 2026-03-24 02:56:20 +00:00
kimi commented 2026-03-24 02:58:02 +00:00
Collaborator
Copy Link

PR #1356 created to fix this issue.

Summary of changes:

  • Added "cv2" to the stubbed modules list in tests/conftest.py

Root cause: perception_cache.py imports cv2 (OpenCV) at the module level. When tests run in parallel with pytest-xdist, the cv2 import can hang/deadlock, causing the 30-second timeout.

Fix: By stubbing cv2 in conftest.py, tests get a MagicMock instead of the real cv2, avoiding the import hang.

Verification: All 812 unit tests now pass (previously 1 failed with timeout).

PR #1356 created to fix this issue. **Summary of changes:** - Added `"cv2"` to the stubbed modules list in `tests/conftest.py` **Root cause:** `perception_cache.py` imports `cv2` (OpenCV) at the module level. When tests run in parallel with pytest-xdist, the cv2 import can hang/deadlock, causing the 30-second timeout. **Fix:** By stubbing cv2 in conftest.py, tests get a MagicMock instead of the real cv2, avoiding the import hang. **Verification:** All 812 unit tests now pass (previously 1 failed with timeout).
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
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 kimi
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#1336
Delete Branch "%!s()"

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?