Chat history: confirm the agentic-path fixes are in the SHIPPED brain (tool-turns vanishing from history) #90

New Issue

2026-07-19T16:14:08Z

tim.lingo commented

2026-07-19 16:14:08 +00:00

Container-era diagnosis (2026-07-09, code-verified): three defects in the agentic chat path — (1) turns that use tools are never persisted to conversation history (bridge-turn hole: suspend returns empty reply, resume path never saves), (2) history load has no engram fallback after restart, (3) agentic writes conv:history: while the loader reads session:messages:. Full analysis: docs/specs/CHAT-RELIABILITY-DIAGNOSIS-2026-07-09.md + HONEST-TURNS-SPEC-2026-07-09.md in the docs repo.

Your main commits 615f0ce + 8db3c8c appear to fix exactly this class — but they predate our diagnosis and the shipped dist/soul.c regen may not carry them. Ask: confirm the fixes are in the brain the app ships (or fold into the #150 rebuild). This is the "Neuron forgets what it just did for you" bug — it directly shapes whether conversations feel continuous.

Findings B (approval hang) and C (fabricated success) from the same diagnosis are only partially covered by #78 + SPEC-DURABLE-APPROVALS — flagging so they're not lost with the container era.

Container-era diagnosis (2026-07-09, code-verified): three defects in the agentic chat path — (1) turns that use tools are never persisted to conversation history (bridge-turn hole: suspend returns empty reply, resume path never saves), (2) history load has no engram fallback after restart, (3) agentic writes conv:history:<sid> while the loader reads session:messages:<sid>. Full analysis: docs/specs/CHAT-RELIABILITY-DIAGNOSIS-2026-07-09.md + HONEST-TURNS-SPEC-2026-07-09.md in the docs repo. Your main commits 615f0ce + 8db3c8c appear to fix exactly this class — but they predate our diagnosis and the shipped dist/soul.c regen may not carry them. Ask: confirm the fixes are in the brain the app ships (or fold into the #150 rebuild). This is the "Neuron forgets what it just did for you" bug — it directly shapes whether conversations feel continuous. Findings B (approval hang) and C (fabricated success) from the same diagnosis are only partially covered by #78 + SPEC-DURABLE-APPROVALS — flagging so they're not lost with the container era.

will.anderson commented

2026-08-03 19:07:53 +00:00

GROOMING — 2026-08-03
Status: Unconfirmed — main commits 615f0ce + 8db3c8c look like they fix the agentic-path history holes (tool-turns never persisted, no engram fallback after restart, conv:history vs session:messages key mismatch), but it is not verified they are compiled into the shipped brain; findings B/C overlap #78.
Next action: Will to confirm the three fixes are in the brain the app ships, or fold them into the #150/#95 rebuild.
Owner: Will
Priority: BETA-CRITICAL

**GROOMING — 2026-08-03** **Status:** Unconfirmed — main commits 615f0ce + 8db3c8c look like they fix the agentic-path history holes (tool-turns never persisted, no engram fallback after restart, conv:history vs session:messages key mismatch), but it is not verified they are compiled into the shipped brain; findings B/C overlap #78. **Next action:** Will to confirm the three fixes are in the brain the app ships, or fold them into the #150/#95 rebuild. **Owner:** Will **Priority:** BETA-CRITICAL

will.anderson added the BETA-CRITICAL label 2026-08-03 19:07:56 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: neuron-technologies/neuron#90