Add priority-based task ordering and auto time tracking (v1.4.1)

- task_list now sorts by priority (critical first) by default, with
  sort_by parameter for created, due_date, and status ordering
- Auto-compute actual_hours when tasks move to done, based on time
  since last in_progress status change in activity log
- Works in both task_update and task_batch_update

Closes #7, closes #10

Author: spranab

manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": "0.3",
   "name": "saga-mcp",
   "display_name": "Saga — Project Tracker for AI Agents",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "A Jira-like project tracker MCP server for AI agents. SQLite-backed, per-project scoped, with full hierarchy and activity logging — so LLMs never lose track.",
   "long_description": "Saga gives your AI assistant a structured SQLite database to track projects, epics, tasks, subtasks, notes, and decisions across sessions. No more scattered markdown files — one `tracker_dashboard` call gives full project context to resume work.\n\n**Key features:**\n- Full hierarchy: Projects > Epics > Tasks > Subtasks\n- SQLite: Self-contained `.tracker.db` file per project — zero setup\n- Activity log: Every mutation is automatically tracked\n- Dashboard: One tool call gives full project overview\n- Notes system: Decisions, context, meeting notes, blockers\n- Batch operations: Create multiple subtasks or update multiple tasks in one call\n- 25 focused tools with safety annotations\n- Import/export: Full project backup and migration as JSON",
   "author": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "saga-mcp",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "A Jira-like project tracker MCP server for AI agents. SQLite-backed, per-project scoped, with full hierarchy (Projects > Epics > Tasks > Subtasks), activity logging, and a dashboard — so LLMs never lose track.",
   "type": "module",
   "main": "dist/index.js",

src/tools/activity.ts

@@ -203,6 +203,28 @@ function handleTaskBatchUpdate(args: Record<string, unknown>) {
         );
       }
 
+      // Auto time tracking
+      if (status === 'done' && oldRow.status !== 'done' && !newRow.actual_hours) {
+        const startEntry = db.prepare(
+          `SELECT created_at FROM activity_log
+           WHERE entity_type = 'task' AND entity_id = ? AND action = 'status_changed'
+             AND field_name = 'status' AND new_value = 'in_progress'
+           ORDER BY created_at DESC LIMIT 1`
+        ).get(id) as { created_at: string } | undefined;
+
+        if (startEntry) {
+          const startMs = new Date(startEntry.created_at + 'Z').getTime();
+          const nowMs = Date.now();
+          const hours = Math.round(((nowMs - startMs) / 3_600_000) * 10) / 10;
+          if (hours > 0) {
+            db.prepare('UPDATE tasks SET actual_hours = ? WHERE id = ?').run(hours, id);
+            newRow.actual_hours = hours;
+            logActivity(db, 'task', id, 'updated', 'actual_hours', null, String(hours),
+              `Task '${newRow.title}' auto-tracked: ${hours}h`);
+          }
+        }
+      }
+
       return newRow;
     });
   })();

src/tools/tasks.ts

@@ -58,6 +58,12 @@ export const definitions: Tool[] = [
         priority: { type: 'string', enum: ['low', 'medium', 'high', 'critical'] },
         assigned_to: { type: 'string', description: 'Filter by assignee' },
         tag: { type: 'string', description: 'Filter by tag' },
+        sort_by: {
+          type: 'string',
+          enum: ['priority', 'created', 'due_date', 'status'],
+          default: 'priority',
+          description: 'Sort order: priority (critical first), created (newest first), due_date (earliest first), status (actionable first)',
+        },
         limit: { type: 'integer', default: 50, description: 'Max results' },
       },
     },
@@ -137,13 +143,32 @@ function handleTaskCreate(args: Record<string, unknown>) {
   return task;
 }
 
+const PRIORITY_ORDER = "CASE t.priority WHEN 'critical' THEN 0 WHEN 'high' THEN 1 WHEN 'medium' THEN 2 WHEN 'low' THEN 3 END";
+const STATUS_ORDER = "CASE t.status WHEN 'blocked' THEN 0 WHEN 'in_progress' THEN 1 WHEN 'review' THEN 2 WHEN 'todo' THEN 3 WHEN 'done' THEN 4 END";
+
+function getTaskOrderClause(sortBy: string): string {
+  switch (sortBy) {
+    case 'priority':
+      return `${PRIORITY_ORDER}, ${STATUS_ORDER}, t.sort_order, t.created_at`;
+    case 'status':
+      return `${STATUS_ORDER}, ${PRIORITY_ORDER}, t.sort_order, t.created_at`;
+    case 'due_date':
+      return `t.due_date IS NULL, t.due_date ASC, ${PRIORITY_ORDER}, t.created_at`;
+    case 'created':
+      return `t.created_at DESC`;
+    default:
+      return `${PRIORITY_ORDER}, ${STATUS_ORDER}, t.sort_order, t.created_at`;
+  }
+}
+
 function handleTaskList(args: Record<string, unknown>) {
   const db = getDb();
   const epicId = args.epic_id as number | undefined;
   const status = args.status as string | undefined;
   const priority = args.priority as string | undefined;
   const assignedTo = args.assigned_to as string | undefined;
   const tag = args.tag as string | undefined;
+  const sortBy = (args.sort_by as string) ?? 'priority';
   const limit = (args.limit as number) ?? 50;
 
   const whereClauses: string[] = [];
@@ -181,7 +206,7 @@ function handleTaskList(args: Record<string, unknown>) {
     LEFT JOIN subtasks s ON s.task_id = t.id
     ${whereStr}
     GROUP BY t.id
-    ORDER BY t.sort_order, t.created_at
+    ORDER BY ${getTaskOrderClause(sortBy)}
     LIMIT ?
   `;
 
@@ -237,6 +262,29 @@ function handleTaskUpdate(args: Record<string, unknown>) {
     'status', 'priority', 'assigned_to', 'title',
   ]);
 
+  // Auto time tracking: when status changes to done and actual_hours wasn't manually set
+  const statusChanged = args.status && oldRow.status !== args.status;
+  if (statusChanged && args.status === 'done' && !args.actual_hours && !newRow.actual_hours) {
+    const startEntry = db.prepare(
+      `SELECT created_at FROM activity_log
+       WHERE entity_type = 'task' AND entity_id = ? AND action = 'status_changed'
+         AND field_name = 'status' AND new_value = 'in_progress'
+       ORDER BY created_at DESC LIMIT 1`
+    ).get(id) as { created_at: string } | undefined;
+
+    if (startEntry) {
+      const startMs = new Date(startEntry.created_at + 'Z').getTime();
+      const nowMs = Date.now();
+      const hours = Math.round(((nowMs - startMs) / 3_600_000) * 10) / 10; // 1 decimal
+      if (hours > 0) {
+        db.prepare('UPDATE tasks SET actual_hours = ? WHERE id = ?').run(hours, id);
+        (newRow as Record<string, unknown>).actual_hours = hours;
+        logActivity(db, 'task', id, 'updated', 'actual_hours', null, String(hours),
+          `Task '${newRow.title}' auto-tracked: ${hours}h`);
+      }
+    }
+  }
+
   return newRow;
 }