[nevridge]

Problem

PUT /api/v1/JournalEntries/{id} returned 500 (UNIQUE constraint failed: JournalEntryTaskItem.JournalEntryId, JournalEntryTaskItem.TodosId) whenever the entry had planned tasks, so a journal summary or title could not be saved once a day had todos.

Cause

JournalEntryRepository.UpdateAsync called DbSet.Update() on the detached graph returned by GetByIdAsync (entry + todos + subtasks + log entries). EF re-inserted the todo join rows, and it also marked every loaded task and log row as modified, so stale loaded values would have been written back.

Fix

UpdateAsync now copies only Title, Summary and UpdatedAt onto the tracked row (found with FindAsync) and sets UpdatedAt on the instance passed in, which the controller returns. Date and CreatedAt are never changed by an update. A missing entry throws KeyNotFoundException (the controller already returns 404 before reaching this).

Tests

New JournalEntryRepositoryUpdateTests (real in-memory SQLite, one context per request as in production). They failed before the fix and pass now:

• a single planned task (reproduces the exact production error)

• a parent with a subtask

• stale task / log values on the loaded graph are not written back, and todos stay linked

• Date / CreatedAt preserved, UpdatedAt set on the returned instance

• an entry tracked by the same context still works; a missing entry throws

• dotnet test (325 passing), dotnet format --verify-no-changes

• Rebuilt the local Docker stack and repeated the failing flow against the live API: plan two tasks, update the summary twice, tasks stay planned

• CI

Found while testing the TaskFlow assistant (MCP server), which edits the journal summary.

🤖 Generated with Claude Code