Extend chain creation to accept per-task description and priority

CreateChain and POST /api/widget/chains previously only took bare
title strings, with priority hardcoded to 1 -- narrower than the
spec's "an ordered list of task titles/descriptions" API line. Now
accepts []models.ChainTaskInput{Content, Description, Priority} per
position, priority defaulting to 1 when omitted.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EZ7ikw2ukGJFTHE3bJS7zL

1 files changed, 39 insertions, 5 deletions

diff --git a/internal/store/chains_test.go b/internal/store/chains_test.go
index 6d3e450..1d16508 100644
--- a/internal/store/chains_test.go
+++ b/internal/store/chains_test.go
@@ -2,12 +2,24 @@ package store
 
 import (
 	"testing"
+
+	"task-dashboard/internal/models"
 )
 
+// chainTasks builds bare-content ChainTaskInputs (no description/priority)
+// for tests that only care about title ordering and lock state.
+func chainTasks(titles ...string) []models.ChainTaskInput {
+	tasks := make([]models.ChainTaskInput, len(titles))
+	for i, t := range titles {
+		tasks[i] = models.ChainTaskInput{Content: t}
+	}
+	return tasks
+}
+
 func TestCreateChain_SeedsPositionsCorrectly(t *testing.T) {
 	s := newNativeTasksTestStore(t)
 
-	chain, err := s.CreateChain("Ham Radio Track", []string{"Study Technician", "Pass Technician exam", "Study General"})
+	chain, err := s.CreateChain("Ham Radio Track", chainTasks("Study Technician", "Pass Technician exam", "Study General"))
 	if err != nil {
 		t.Fatalf("CreateChain: %v", err)
 	}
@@ -38,7 +50,7 @@ func TestCreateChain_SeedsPositionsCorrectly(t *testing.T) {
 func TestCompleteNativeTask_AdvancesChain(t *testing.T) {
 	s := newNativeTasksTestStore(t)
 
-	chain, err := s.CreateChain("Track", []string{"Step 1", "Step 2"})
+	chain, err := s.CreateChain("Track", chainTasks("Step 1", "Step 2"))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -71,7 +83,7 @@ func TestCompleteNativeTask_AdvancesChain(t *testing.T) {
 func TestCompleteNativeTask_LastPosition_MarksChainCompleted(t *testing.T) {
 	s := newNativeTasksTestStore(t)
 
-	chain, err := s.CreateChain("Track", []string{"Only step"})
+	chain, err := s.CreateChain("Track", chainTasks("Only step"))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -96,7 +108,7 @@ func TestCompleteNativeTask_LastPosition_MarksChainCompleted(t *testing.T) {
 func TestCompleteNativeTask_PausedChain_DoesNotAdvance(t *testing.T) {
 	s := newNativeTasksTestStore(t)
 
-	chain, err := s.CreateChain("Track", []string{"Step 1", "Step 2"})
+	chain, err := s.CreateChain("Track", chainTasks("Step 1", "Step 2"))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -139,7 +151,7 @@ func TestCompleteNativeTask_PausedChain_DoesNotAdvance(t *testing.T) {
 func TestGetUndatedNativeTasks_ExcludesLockedChainTasks(t *testing.T) {
 	s := newNativeTasksTestStore(t)
 
-	chain, err := s.CreateChain("Track", []string{"Step 1", "Step 2", "Step 3"})
+	chain, err := s.CreateChain("Track", chainTasks("Step 1", "Step 2", "Step 3"))
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -156,6 +168,28 @@ func TestGetUndatedNativeTasks_ExcludesLockedChainTasks(t *testing.T) {
 	}
 }
 
+func TestCreateChain_PersistsDescriptionAndPriority(t *testing.T) {
+	s := newNativeTasksTestStore(t)
+
+	chain, err := s.CreateChain("Track", []models.ChainTaskInput{
+		{Content: "Step 1", Description: "do the thing", Priority: 4},
+		{Content: "Step 2"}, // no priority given -- should default to 1
+	})
+	if err != nil {
+		t.Fatal(err)
+	}
+	tasks, err := s.GetChainTasks(chain.ID)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if tasks[0].Description != "do the thing" || tasks[0].Priority != 4 {
+		t.Errorf("tasks[0] = %+v, want description=%q priority=4", tasks[0], "do the thing")
+	}
+	if tasks[1].Priority != 1 {
+		t.Errorf("tasks[1].Priority = %d, want default of 1", tasks[1].Priority)
+	}
+}
+
 func TestGetChain_UnknownID_ReturnsErrNotFound(t *testing.T) {
 	s := newNativeTasksTestStore(t)