From e4438d1a6eb680a5de56c9d67d73312ae5b6f6b0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?m=C3=BCde?= Date: Sun, 17 May 2026 14:27:03 +0200 Subject: [PATCH] todo: phase 6 event-covered redirects converted --- TODO.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/TODO.md b/TODO.md index 355bd8c..54d5442 100644 --- a/TODO.md +++ b/TODO.md @@ -24,7 +24,7 @@ - Per-agent reminder status (pending, delivered) - Reminder query interface for debugging - Display reminder delivery errors (failed sends, mark failures) -- **Phase 6 leftovers**: convert the remaining redirect-and-refetch endpoints whose mutations now fire dashboard events. `/approve`, `/deny`, `/answer-question`, `/cancel-question`, `/request-spawn` are 5-line `Redirect::to("/")` → `(StatusCode::OK, "ok")` changes; the client already updates derived state from the events. Container-lifecycle endpoints (`/restart`, `/destroy`, `/kill`, `/rebuild`, `/start`, `/api/{cancel,compact,model,new-session}`, `/meta-update`, `/purge-tombstone`) need a `ContainerListChanged` event first — `ContainerView` is currently sourced from external `nixos-container list`, so the 5s poll still drives that section. +- **Phase 6 leftovers** — event-covered endpoints (`/approve`, `/deny`, `/answer-question`, `/cancel-question`, `/request-spawn`) now return 200 (f559441); the matching forms carry `data-no-refresh` so the post-submit `/api/state` refetch is skipped (the SSE event delivers the update). Container-lifecycle endpoints (`/restart`, `/destroy`, `/kill`, `/rebuild`, `/start`, `/api/{cancel,compact,model,new-session}`, `/meta-update`, `/purge-tombstone`) still need a `ContainerListChanged` event before their redirects can drop — `ContainerView` is currently sourced from external `nixos-container list`, so the 5s poll continues to drive that section. ## Bugs