From 54f091e1b920943967c6aebc9c1f3122ce52e267 Mon Sep 17 00:00:00 2001 From: Peter Stone Date: Fri, 23 Jan 2026 15:55:48 -1000 Subject: Fix critical resilience issues from code review - DB connection pool: Allow 5 connections instead of 1 for better concurrency - JSON unmarshal: Add error handling to prevent nil slice issues - Context cancellation: Check ctx.Done() in aggregateData goroutines - Migration path: Make configurable via MIGRATION_DIR env var Co-Authored-By: Claude Opus 4.5 --- internal/handlers/heuristic_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'internal/handlers/heuristic_test.go') diff --git a/internal/handlers/heuristic_test.go b/internal/handlers/heuristic_test.go index 2b70218..b03b664 100644 --- a/internal/handlers/heuristic_test.go +++ b/internal/handlers/heuristic_test.go @@ -33,7 +33,7 @@ func TestHandleTasks_Heuristic(t *testing.T) { defer os.Chdir(originalDir) // Initialize store (this runs migrations) - db, err := store.New(tmpFile.Name()) + db, err := store.New(tmpFile.Name(), "migrations") if err != nil { t.Fatalf("Failed to initialize store: %v", err) } -- cgit v1.2.3