feat: add description field to tasks

Amp-Thread-ID:
https://ampcode.com/threads/T-79499d9e-f4f4-40de-893c-524c32a45483
Co-authored-by: Amp <amp@ampcode.com>

1 files changed, 34 insertions, 24 deletions

diff --git a/Omni/Task.hs b/Omni/Task.hs
index 65e5c42..36b318b 100644
--- a/Omni/Task.hs
+++ b/Omni/Task.hs
@@ -82,6 +82,7 @@ Options:
   --dep-type=<type>             Dependency type: blocks, discovered-from, parent-child, related (default: blocks)
   --discovered-from=<id>        Shortcut for --deps=<id> --dep-type=discovered-from
   --namespace=<ns>              Optional namespace (e.g., Omni/Task, Biz/Cloud)
+  --description=<desc>          Task description
   --flush                       Force immediate export
   --json                        Output in JSON format (for agent use)
   --quiet                       Non-interactive mode (for agents)
@@ -159,7 +160,12 @@ move args
           let validNs = Namespace.fromHaskellModule ns
               nsPath = T.pack <| Namespace.toPath validNs
           pure <| Just nsPath
-      createdTask <- createTask title taskType parent namespace priority deps
+
+      description <- case Cli.getArg args (Cli.longOption "description") of
+        Nothing -> pure Nothing
+        Just d -> pure <| Just (T.pack d)
+
+      createdTask <- createTask title taskType parent namespace priority deps description
       if isJsonMode args
         then outputJson createdTask
         else putStrLn <| "Created task: " <> T.unpack (taskId createdTask)
@@ -301,60 +307,63 @@ unitTests =
         initTaskDb
         True Test.@?= True,
       Test.unit "can create task" <| do
-        task <- createTask "Test task" WorkTask Nothing Nothing P2 []
+        task <- createTask "Test task" WorkTask Nothing Nothing P2 [] Nothing
         taskTitle task Test.@?= "Test task"
         taskType task Test.@?= WorkTask
         taskStatus task Test.@?= Open
         taskPriority task Test.@?= P2
         null (taskDependencies task) Test.@?= True,
+      Test.unit "can create task with description" <| do
+        task <- createTask "Test task" WorkTask Nothing Nothing P2 [] (Just "My description")
+        taskDescription task Test.@?= Just "My description",
       Test.unit "can list tasks" <| do
-        _ <- createTask "Test task for list" WorkTask Nothing Nothing P2 []
+        _ <- createTask "Test task for list" WorkTask Nothing Nothing P2 [] Nothing
         tasks <- listTasks Nothing Nothing Nothing Nothing
         not (null tasks) Test.@?= True,
       Test.unit "ready tasks exclude blocked ones" <| do
-        task1 <- createTask "First task" WorkTask Nothing Nothing P2 []
+        task1 <- createTask "First task" WorkTask Nothing Nothing P2 [] Nothing
         let blockingDep = Dependency {depId = taskId task1, depType = Blocks}
-        task2 <- createTask "Blocked task" WorkTask Nothing Nothing P2 [blockingDep]
+        task2 <- createTask "Blocked task" WorkTask Nothing Nothing P2 [blockingDep] Nothing
         ready <- getReadyTasks
         (taskId task1 `elem` map taskId ready) Test.@?= True
         (taskId task2 `notElem` map taskId ready) Test.@?= True,
       Test.unit "discovered-from dependencies don't block" <| do
-        task1 <- createTask "Original task" WorkTask Nothing Nothing P2 []
+        task1 <- createTask "Original task" WorkTask Nothing Nothing P2 [] Nothing
         let discDep = Dependency {depId = taskId task1, depType = DiscoveredFrom}
-        task2 <- createTask "Discovered work" WorkTask Nothing Nothing P2 [discDep]
+        task2 <- createTask "Discovered work" WorkTask Nothing Nothing P2 [discDep] Nothing
         ready <- getReadyTasks
         -- Both should be ready since DiscoveredFrom doesn't block
         (taskId task1 `elem` map taskId ready) Test.@?= True
         (taskId task2 `elem` map taskId ready) Test.@?= True,
       Test.unit "related dependencies don't block" <| do
-        task1 <- createTask "Task A" WorkTask Nothing Nothing P2 []
+        task1 <- createTask "Task A" WorkTask Nothing Nothing P2 [] Nothing
         let relDep = Dependency {depId = taskId task1, depType = Related}
-        task2 <- createTask "Task B" WorkTask Nothing Nothing P2 [relDep]
+        task2 <- createTask "Task B" WorkTask Nothing Nothing P2 [relDep] Nothing
         ready <- getReadyTasks
         -- Both should be ready since Related doesn't block
         (taskId task1 `elem` map taskId ready) Test.@?= True
         (taskId task2 `elem` map taskId ready) Test.@?= True,
       Test.unit "child task gets sequential ID" <| do
-        parent <- createTask "Parent" Epic Nothing Nothing P2 []
-        child1 <- createTask "Child 1" WorkTask (Just (taskId parent)) Nothing P2 []
-        child2 <- createTask "Child 2" WorkTask (Just (taskId parent)) Nothing P2 []
+        parent <- createTask "Parent" Epic Nothing Nothing P2 [] Nothing
+        child1 <- createTask "Child 1" WorkTask (Just (taskId parent)) Nothing P2 [] Nothing
+        child2 <- createTask "Child 2" WorkTask (Just (taskId parent)) Nothing P2 [] Nothing
         taskId child1 Test.@?= taskId parent <> ".1"
         taskId child2 Test.@?= taskId parent <> ".2",
       Test.unit "grandchild task gets sequential ID" <| do
-        parent <- createTask "Grandparent" Epic Nothing Nothing P2 []
-        child <- createTask "Parent" Epic (Just (taskId parent)) Nothing P2 []
-        grandchild <- createTask "Grandchild" WorkTask (Just (taskId child)) Nothing P2 []
+        parent <- createTask "Grandparent" Epic Nothing Nothing P2 [] Nothing
+        child <- createTask "Parent" Epic (Just (taskId parent)) Nothing P2 [] Nothing
+        grandchild <- createTask "Grandchild" WorkTask (Just (taskId child)) Nothing P2 [] Nothing
         taskId grandchild Test.@?= taskId parent <> ".1.1",
       Test.unit "siblings of grandchild task get sequential ID" <| do
-        parent <- createTask "Grandparent" Epic Nothing Nothing P2 []
-        child <- createTask "Parent" Epic (Just (taskId parent)) Nothing P2 []
-        grandchild1 <- createTask "Grandchild 1" WorkTask (Just (taskId child)) Nothing P2 []
-        grandchild2 <- createTask "Grandchild 2" WorkTask (Just (taskId child)) Nothing P2 []
+        parent <- createTask "Grandparent" Epic Nothing Nothing P2 [] Nothing
+        child <- createTask "Parent" Epic (Just (taskId parent)) Nothing P2 [] Nothing
+        grandchild1 <- createTask "Grandchild 1" WorkTask (Just (taskId child)) Nothing P2 [] Nothing
+        grandchild2 <- createTask "Grandchild 2" WorkTask (Just (taskId child)) Nothing P2 [] Nothing
         taskId grandchild1 Test.@?= taskId parent <> ".1.1"
         taskId grandchild2 Test.@?= taskId parent <> ".1.2",
       Test.unit "child ID generation skips gaps" <| do
-        parent <- createTask "Parent with gaps" Epic Nothing Nothing P2 []
-        child1 <- createTask "Child 1" WorkTask (Just (taskId parent)) Nothing P2 []
+        parent <- createTask "Parent with gaps" Epic Nothing Nothing P2 [] Nothing
+        child1 <- createTask "Child 1" WorkTask (Just (taskId parent)) Nothing P2 [] Nothing
         -- Manually create a task with .3 suffix to simulate a gap (or deleted task)
         let child3Id = taskId parent <> ".3"
             child3 =
@@ -368,15 +377,16 @@ unitTests =
                   taskPriority = P2,
                   taskDependencies = [],
                   taskCreatedAt = taskCreatedAt child1,
-                  taskUpdatedAt = taskUpdatedAt child1
+                  taskUpdatedAt = taskUpdatedAt child1,
+                  taskDescription = Nothing
                 }
         saveTask child3
 
         -- Create a new child, it should get .4, not .2
-        child4 <- createTask "Child 4" WorkTask (Just (taskId parent)) Nothing P2 []
+        child4 <- createTask "Child 4" WorkTask (Just (taskId parent)) Nothing P2 [] Nothing
         taskId child4 Test.@?= taskId parent <> ".4",
       Test.unit "task lookup is case insensitive" <| do
-        task <- createTask "Case sensitive" WorkTask Nothing Nothing P2 []
+        task <- createTask "Case sensitive" WorkTask Nothing Nothing P2 [] Nothing
         let tid = taskId task
             upperTid = T.toUpper tid
         tasks <- loadTasks