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

[triage-generated] [bug] [P0] timmy chat crashes — ImportError: memory_write missing from memory_system #1249

New Issue
Closed
opened 2026-03-23 23:35:22 +00:00 by Timmy · 1 comment
Timmy commented 2026-03-23 23:35:22 +00:00
Owner
Copy Link

Bug: timmy chat crashes on startup — ImportError in memory_system

Reproduction

cd ~/Timmy-Time-dashboard && .venv/bin/timmy chat --session-id test "hello"

Error

ImportError: cannot import name memory_write from timmy.memory_system
(src/timmy/memory_system.py)

Root Cause

src/timmy/tools/_registry.py:93 imports memory_write from timmy.memory_system, but the module no longer exports that name. Likely broken by the tools.py breakup (#1215).

Fix

Either:

  1. Add memory_write back to memory_system.py, or
  2. Update the import in _registry.py to use the current function name

Acceptance Criteria

  • timmy chat --session-id test "hello" runs without ImportError
  • All tool registration completes successfully
  • Existing tests pass

Scope

Files: src/timmy/tools/_registry.py, src/timmy/memory_system.py

[triage-generated] [bug]

## Bug: `timmy chat` crashes on startup — ImportError in memory_system ### Reproduction ``` cd ~/Timmy-Time-dashboard && .venv/bin/timmy chat --session-id test "hello" ``` ### Error ``` ImportError: cannot import name memory_write from timmy.memory_system (src/timmy/memory_system.py) ``` ### Root Cause `src/timmy/tools/_registry.py:93` imports `memory_write` from `timmy.memory_system`, but the module no longer exports that name. Likely broken by the tools.py breakup (#1215). ### Fix Either: 1. Add `memory_write` back to `memory_system.py`, or 2. Update the import in `_registry.py` to use the current function name ### Acceptance Criteria - `timmy chat --session-id test "hello"` runs without ImportError - All tool registration completes successfully - Existing tests pass ### Scope Files: `src/timmy/tools/_registry.py`, `src/timmy/memory_system.py` [triage-generated] [bug]
claude self-assigned this 2026-03-23 23:35:29 +00:00
claude commented 2026-03-23 23:37:15 +00:00
Collaborator
Copy Link

PR created: #1251

Root cause: _registry.py imported memory_write from timmy.memory_system, but that name no longer exists after the tools.py breakup (#1215). The function was renamed to memory_store.

Fix: Updated the import in _register_memory_tools() to use memory_store. The tool is still registered under the name "memory_write" for backward compatibility with any callers.

All 435 unit tests pass.

PR created: #1251 **Root cause:** `_registry.py` imported `memory_write` from `timmy.memory_system`, but that name no longer exists after the tools.py breakup (#1215). The function was renamed to `memory_store`. **Fix:** Updated the import in `_register_memory_tools()` to use `memory_store`. The tool is still registered under the name `"memory_write"` for backward compatibility with any callers. All 435 unit tests pass.
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 claude
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#1249
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?