summaryrefslogtreecommitdiff
path: root/internal
diff options
context:
space:
mode:
authorClaude <noreply@anthropic.com>2026-05-25 05:05:27 +0000
committerClaude <noreply@anthropic.com>2026-05-25 05:05:27 +0000
commitb44c8277465c3b4e03fe4de4dd93850413023b69 (patch)
treef79131cb5f4d7444f2c358f35ad7749bad612bda /internal
parent6890bafa65a309b540cbe1562c39df137f202369 (diff)
feat(api,web): event-stream timeline (Phase 3)
Adds GET /api/tasks/{id}/events (seq-ordered, ?since_seq for incremental polling) as the timeline data source, and a per-task Timeline section in the web UI that fetches and renders the observability event stream. New pure, unit-tested JS helpers: formatEventText, renderEventTimeline, fetchTaskEvents. The timeline load is async and guarded on a global fetch so the synchronous renderTaskPanel path (and its DOM-mock unit tests) is unaffected. Verified via Go endpoint tests and node --test (272 JS tests pass). Note: the live in-browser panel render was not exercised in this environment — only unit-level coverage. https://claude.ai/code/session_01SESwn7kQ7oP62trWw6pc39
Diffstat (limited to 'internal')
-rw-r--r--internal/api/events.go38
-rw-r--r--internal/api/events_test.go94
-rw-r--r--internal/api/server.go1
3 files changed, 133 insertions, 0 deletions
diff --git a/internal/api/events.go b/internal/api/events.go
new file mode 100644
index 0000000..eefe064
--- /dev/null
+++ b/internal/api/events.go
@@ -0,0 +1,38 @@
+package api
+
+import (
+ "net/http"
+ "strconv"
+
+ "github.com/thepeterstone/claudomator/internal/event"
+)
+
+// handleListTaskEvents returns a task's observability event stream in seq order.
+// Pass ?since_seq=N to fetch only events newer than seq N (incremental polling).
+func (s *Server) handleListTaskEvents(w http.ResponseWriter, r *http.Request) {
+ id := r.PathValue("id")
+ if _, err := s.store.GetTask(id); err != nil {
+ writeJSON(w, http.StatusNotFound, map[string]string{"error": "task not found"})
+ return
+ }
+
+ var sinceSeq int64
+ if v := r.URL.Query().Get("since_seq"); v != "" {
+ n, err := strconv.ParseInt(v, 10, 64)
+ if err != nil || n < 0 {
+ writeJSON(w, http.StatusBadRequest, map[string]string{"error": "invalid since_seq"})
+ return
+ }
+ sinceSeq = n
+ }
+
+ events, err := s.store.ListEvents(id, sinceSeq)
+ if err != nil {
+ writeJSON(w, http.StatusInternalServerError, map[string]string{"error": err.Error()})
+ return
+ }
+ if events == nil {
+ events = []*event.Event{}
+ }
+ writeJSON(w, http.StatusOK, events)
+}
diff --git a/internal/api/events_test.go b/internal/api/events_test.go
new file mode 100644
index 0000000..456b6d8
--- /dev/null
+++ b/internal/api/events_test.go
@@ -0,0 +1,94 @@
+package api
+
+import (
+ "encoding/json"
+ "net/http"
+ "net/http/httptest"
+ "strconv"
+ "testing"
+
+ "github.com/thepeterstone/claudomator/internal/event"
+ "github.com/thepeterstone/claudomator/internal/task"
+)
+
+func TestListTaskEvents_ReturnsStreamInSeqOrder(t *testing.T) {
+ srv, store := testServer(t)
+ createTaskWithState(t, store, "ev-task", task.StateReady)
+
+ req := httptest.NewRequest("GET", "/api/tasks/ev-task/events", nil)
+ w := httptest.NewRecorder()
+ srv.Handler().ServeHTTP(w, req)
+
+ if w.Code != http.StatusOK {
+ t.Fatalf("status: want 200, got %d; body %s", w.Code, w.Body.String())
+ }
+ var events []*event.Event
+ if err := json.Unmarshal(w.Body.Bytes(), &events); err != nil {
+ t.Fatalf("unmarshal: %v", err)
+ }
+ if len(events) == 0 {
+ t.Fatal("expected state-change events from the walk to READY")
+ }
+ for i := 1; i < len(events); i++ {
+ if events[i].Seq <= events[i-1].Seq {
+ t.Errorf("events not in ascending seq order: %d then %d", events[i-1].Seq, events[i].Seq)
+ }
+ }
+}
+
+func TestListTaskEvents_SinceSeqFilters(t *testing.T) {
+ srv, store := testServer(t)
+ createTaskWithState(t, store, "ev-since", task.StateReady)
+
+ all := getEvents(t, srv, "ev-since", "")
+ if len(all) < 2 {
+ t.Skipf("need at least 2 events to test since_seq, got %d", len(all))
+ }
+ firstSeq := all[0].Seq
+ rest := getEvents(t, srv, "ev-since", "?since_seq="+itoa(firstSeq))
+ for _, e := range rest {
+ if e.Seq <= firstSeq {
+ t.Errorf("since_seq=%d returned event with seq %d", firstSeq, e.Seq)
+ }
+ }
+}
+
+func TestListTaskEvents_NotFound(t *testing.T) {
+ srv, _ := testServer(t)
+ req := httptest.NewRequest("GET", "/api/tasks/missing/events", nil)
+ w := httptest.NewRecorder()
+ srv.Handler().ServeHTTP(w, req)
+ if w.Code != http.StatusNotFound {
+ t.Errorf("want 404, got %d", w.Code)
+ }
+}
+
+func TestListTaskEvents_InvalidSinceSeq(t *testing.T) {
+ srv, store := testServer(t)
+ createTaskWithState(t, store, "ev-bad", task.StateReady)
+ req := httptest.NewRequest("GET", "/api/tasks/ev-bad/events?since_seq=abc", nil)
+ w := httptest.NewRecorder()
+ srv.Handler().ServeHTTP(w, req)
+ if w.Code != http.StatusBadRequest {
+ t.Errorf("want 400, got %d", w.Code)
+ }
+}
+
+func getEvents(t *testing.T, srv *Server, taskID, query string) []*event.Event {
+ t.Helper()
+ req := httptest.NewRequest("GET", "/api/tasks/"+taskID+"/events"+query, nil)
+ w := httptest.NewRecorder()
+ srv.Handler().ServeHTTP(w, req)
+ if w.Code != http.StatusOK {
+ t.Fatalf("getEvents %s%s: status %d", taskID, query, w.Code)
+ }
+ var events []*event.Event
+ if err := json.Unmarshal(w.Body.Bytes(), &events); err != nil {
+ t.Fatalf("getEvents unmarshal: %v", err)
+ }
+ return events
+}
+
+func itoa(n int64) string {
+ return strconv.FormatInt(n, 10)
+}
diff --git a/internal/api/server.go b/internal/api/server.go
index 6564280..a4b7ea1 100644
--- a/internal/api/server.go
+++ b/internal/api/server.go
@@ -144,6 +144,7 @@ func (s *Server) routes() {
s.mux.HandleFunc("DELETE /api/tasks/{id}", s.handleDeleteTask)
s.mux.HandleFunc("GET /api/tasks/{id}/subtasks", s.handleListSubtasks)
s.mux.HandleFunc("GET /api/tasks/{id}/executions", s.handleListExecutions)
+ s.mux.HandleFunc("GET /api/tasks/{id}/events", s.handleListTaskEvents)
s.mux.HandleFunc("GET /api/executions", s.handleListRecentExecutions)
s.mux.HandleFunc("GET /api/stats", s.handleGetDashboardStats)
s.mux.HandleFunc("GET /api/agents/status", s.handleGetAgentStatus)