trekoon 0.2.9 → 0.3.1

package/src/domain/mutation-service.ts

@@ -1,8 +1,9 @@
 import { type Database } from "bun:sqlite";
 
+import { writeTransaction } from "../storage/database";
 import { appendEventWithGitContext } from "../sync/event-writes";
 import { ENTITY_OPERATIONS } from "./mutation-operations";
-import { TrackerDomain } from "./tracker-domain";
+import { TrackerDomain, validateStatusTransition } from "./tracker-domain";
 import {
   type CompactEpicCreateResult,
   type CompactEpicExpandResult,
@@ -104,7 +105,7 @@ export class MutationService {
   }
 
   createEpic(input: { title: string; description: string; status?: string | undefined }): EpicRecord {
-    return this.#db.transaction((): EpicRecord => {
+    return writeTransaction(this.#db, (): EpicRecord => {
       const epic = this.#domain.createEpic(input);
       this.#appendEntityEvent("epic", epic.id, ENTITY_OPERATIONS.epic.created, {
         title: epic.title,
@@ -112,7 +113,7 @@ export class MutationService {
         status: epic.status,
       });
       return epic;
-    })();
+    });
   }
 
   createEpicGraph(input: {
@@ -123,7 +124,7 @@ export class MutationService {
     subtaskSpecs: readonly CompactSubtaskSpec[];
     dependencySpecs: readonly CompactDependencySpec[];
   }): CompactEpicCreateResult {
-    return this.#db.transaction((): CompactEpicCreateResult => {
+    return writeTransaction(this.#db, (): CompactEpicCreateResult => {
       const epic = this.#domain.createEpic(input);
       const created = this.#domain.expandEpic({
         epicId: epic.id,
@@ -172,14 +173,18 @@ export class MutationService {
         dependencies: created.dependencies,
         result: created.result,
       };
-    })();
+    });
   }
 
   updateEpic(
     id: string,
     input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
   ): EpicRecord {
-    return this.#db.transaction((): EpicRecord => {
+    return writeTransaction(this.#db, (): EpicRecord => {
+      if (input.status !== undefined) {
+        const existing = this.#domain.getEpicOrThrow(id);
+        validateStatusTransition(existing.status, input.status, "epic", id);
+      }
       const epic = this.#domain.updateEpic(id, input);
       this.#appendEntityEvent("epic", epic.id, ENTITY_OPERATIONS.epic.updated, {
         title: epic.title,
@@ -187,27 +192,27 @@ export class MutationService {
         status: epic.status,
       });
       return epic;
-    })();
+    });
   }
 
   updateEpicStatusCascade(id: string, status: string): StatusCascadePlan {
-    return this.#db.transaction((): StatusCascadePlan => {
+    return writeTransaction(this.#db, (): StatusCascadePlan => {
       const plan = this.#domain.planStatusCascade("epic", id, status);
       this.#assertCascadeNotBlocked(plan);
       this.#applyStatusCascadePlan(plan);
       return plan;
-    })();
+    });
   }
 
   deleteEpic(id: string): void {
-    this.#db.transaction((): void => {
+    writeTransaction(this.#db, (): void => {
       this.#domain.deleteEpic(id);
       this.#appendEntityEvent("epic", id, ENTITY_OPERATIONS.epic.deleted, {});
-    })();
+    });
   }
 
   createTask(input: { epicId: string; title: string; description: string; status?: string | undefined }): TaskRecord {
-    return this.#db.transaction((): TaskRecord => {
+    return writeTransaction(this.#db, (): TaskRecord => {
       const task = this.#domain.createTask(input);
       this.#appendEntityEvent("task", task.id, ENTITY_OPERATIONS.task.created, {
         epic_id: task.epicId,
@@ -216,11 +221,11 @@ export class MutationService {
         status: task.status,
       });
       return task;
-    })();
+    });
   }
 
   createTaskBatch(input: { epicId: string; specs: readonly CompactTaskSpec[] }): CompactTaskBatchCreateResult {
-    return this.#db.transaction((): CompactTaskBatchCreateResult => {
+    return writeTransaction(this.#db, (): CompactTaskBatchCreateResult => {
       const created = this.#domain.createTaskBatch(input);
       for (const task of created.tasks) {
         this.#appendEntityEvent("task", task.id, ENTITY_OPERATIONS.task.created, {
@@ -231,7 +236,7 @@ export class MutationService {
         });
       }
       return created;
-    })();
+    });
   }
 
   expandEpic(input: {
@@ -240,7 +245,7 @@ export class MutationService {
     subtaskSpecs: readonly CompactSubtaskSpec[];
     dependencySpecs: readonly CompactDependencySpec[];
   }): CompactEpicExpandResult {
-    return this.#db.transaction((): CompactEpicExpandResult => {
+    return writeTransaction(this.#db, (): CompactEpicExpandResult => {
       const created = this.#domain.expandEpic(input);
       for (const task of created.tasks) {
         this.#appendEntityEvent("task", task.id, ENTITY_OPERATIONS.task.created, {
@@ -270,39 +275,44 @@ export class MutationService {
       }
 
       return created;
-    })();
+    });
   }
 
   updateTask(
     id: string,
-    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
+    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): TaskRecord {
-    return this.#db.transaction((): TaskRecord => {
+    return writeTransaction(this.#db, (): TaskRecord => {
+      if (input.status !== undefined) {
+        const existing = this.#domain.getTaskOrThrow(id);
+        validateStatusTransition(existing.status, input.status, "task", id);
+      }
       const task = this.#domain.updateTask(id, input);
       this.#appendEntityEvent("task", task.id, ENTITY_OPERATIONS.task.updated, {
         epic_id: task.epicId,
         title: task.title,
         description: task.description,
         status: task.status,
+        owner: task.owner,
       });
       return task;
-    })();
+    });
   }
 
   updateTaskStatusCascade(id: string, status: string): StatusCascadePlan {
-    return this.#db.transaction((): StatusCascadePlan => {
+    return writeTransaction(this.#db, (): StatusCascadePlan => {
       const plan = this.#domain.planStatusCascade("task", id, status);
       this.#assertCascadeNotBlocked(plan);
       this.#applyStatusCascadePlan(plan);
       return plan;
-    })();
+    });
   }
 
   deleteTask(id: string): void {
-    this.#db.transaction((): void => {
+    writeTransaction(this.#db, (): void => {
       this.#domain.deleteTask(id);
       this.#appendEntityEvent("task", id, ENTITY_OPERATIONS.task.deleted, {});
-    })();
+    });
   }
 
   createSubtask(input: {
@@ -311,7 +321,7 @@ export class MutationService {
     description?: string | undefined;
     status?: string | undefined;
   }): SubtaskRecord {
-    return this.#db.transaction((): SubtaskRecord => {
+    return writeTransaction(this.#db, (): SubtaskRecord => {
       const subtask = this.#domain.createSubtask(input);
       this.#appendEntityEvent("subtask", subtask.id, ENTITY_OPERATIONS.subtask.created, {
         task_id: subtask.taskId,
@@ -320,11 +330,11 @@ export class MutationService {
         status: subtask.status,
       });
       return subtask;
-    })();
+    });
   }
 
   createSubtaskBatch(input: { taskId: string; specs: readonly CompactSubtaskSpec[] }): CompactSubtaskBatchCreateResult {
-    return this.#db.transaction((): CompactSubtaskBatchCreateResult => {
+    return writeTransaction(this.#db, (): CompactSubtaskBatchCreateResult => {
       const created = this.#domain.createSubtaskBatch(input);
       for (const subtask of created.subtasks) {
         this.#appendEntityEvent("subtask", subtask.id, ENTITY_OPERATIONS.subtask.created, {
@@ -335,34 +345,39 @@ export class MutationService {
         });
       }
       return created;
-    })();
+    });
   }
 
   updateSubtask(
     id: string,
-    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
+    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): SubtaskRecord {
-    return this.#db.transaction((): SubtaskRecord => {
+    return writeTransaction(this.#db, (): SubtaskRecord => {
+      if (input.status !== undefined) {
+        const existing = this.#domain.getSubtaskOrThrow(id);
+        validateStatusTransition(existing.status, input.status, "subtask", id);
+      }
       const subtask = this.#domain.updateSubtask(id, input);
       this.#appendEntityEvent("subtask", subtask.id, ENTITY_OPERATIONS.subtask.updated, {
         task_id: subtask.taskId,
         title: subtask.title,
         description: subtask.description,
         status: subtask.status,
+        owner: subtask.owner,
       });
       return subtask;
-    })();
+    });
   }
 
   deleteSubtask(id: string): void {
-    this.#db.transaction((): void => {
+    writeTransaction(this.#db, (): void => {
       this.#domain.deleteSubtask(id);
       this.#appendEntityEvent("subtask", id, ENTITY_OPERATIONS.subtask.deleted, {});
-    })();
+    });
   }
 
   addDependency(sourceId: string, dependsOnId: string): DependencyRecord {
-    return this.#db.transaction((): DependencyRecord => {
+    return writeTransaction(this.#db, (): DependencyRecord => {
       const dependency = this.#domain.addDependency(sourceId, dependsOnId);
       this.#appendEntityEvent("dependency", dependency.id, ENTITY_OPERATIONS.dependency.added, {
         source_id: dependency.sourceId,
@@ -371,11 +386,11 @@ export class MutationService {
         depends_on_kind: dependency.dependsOnKind,
       });
       return dependency;
-    })();
+    });
   }
 
   addDependencyBatch(input: { specs: readonly CompactDependencySpec[] }): CompactDependencyBatchAddResult {
-    return this.#db.transaction((): CompactDependencyBatchAddResult => {
+    return writeTransaction(this.#db, (): CompactDependencyBatchAddResult => {
       const created = this.#domain.addDependencyBatch(input);
       for (const dependency of created.dependencies) {
         this.#appendEntityEvent("dependency", dependency.id, ENTITY_OPERATIONS.dependency.added, {
@@ -386,11 +401,11 @@ export class MutationService {
         });
       }
       return created;
-    })();
+    });
   }
 
   removeDependency(sourceId: string, dependsOnId: string): number {
-    return this.#db.transaction((): number => {
+    return writeTransaction(this.#db, (): number => {
       const removed = this.#domain.removeDependency(sourceId, dependsOnId);
       if (removed > 0) {
         this.#appendEntityEvent("dependency", `${sourceId}->${dependsOnId}`, ENTITY_OPERATIONS.dependency.removed, {
@@ -399,11 +414,19 @@ export class MutationService {
         });
       }
       return removed;
-    })();
+    });
   }
 
   describeError(error: unknown): string | undefined {
-    if (!(error instanceof DomainError) || error.code !== "dependency_blocked") {
+    if (!(error instanceof DomainError)) {
+      return undefined;
+    }
+
+    if (error.code === "status_transition_invalid") {
+      return error.message;
+    }
+
+    if (error.code !== "dependency_blocked") {
       return undefined;
     }
 
@@ -560,6 +583,7 @@ export class MutationService {
           title: task.title,
           description: task.description,
           status: task.status,
+          owner: task.owner,
         });
         continue;
       }
@@ -570,6 +594,7 @@ export class MutationService {
         title: subtask.title,
         description: subtask.description,
         status: subtask.status,
+        owner: subtask.owner,
       });
     }
   }
@@ -582,7 +607,7 @@ export class MutationService {
   ): ScopeReplacementResult {
     const result = this.#buildScopeReplacementResult(nodes, searchText, replacementText, fields, "apply");
 
-    this.#db.transaction((): void => {
+    writeTransaction(this.#db, (): void => {
       for (const node of nodes) {
         const nextTitle = fields.includes("title") ? replaceMatches(node.title, searchText, replacementText) : node.title;
         const nextDescription = fields.includes("description")
@@ -610,6 +635,7 @@ export class MutationService {
             title: task.title,
             description: task.description,
             status: task.status,
+            owner: task.owner,
           });
           continue;
         }
@@ -620,9 +646,10 @@ export class MutationService {
           title: subtask.title,
           description: subtask.description,
           status: subtask.status,
+          owner: subtask.owner,
         });
       }
-    })();
+    });
 
     return result;
   }

package/src/domain/tracker-domain.ts

@@ -32,10 +32,13 @@ import {
   type SubtaskRecord,
   type TaskTreeDetailed,
   type TaskRecord,
+  VALID_STATUSES,
+  VALID_TRANSITIONS,
+  type ValidStatus,
 } from "./types";
 
 const DEFAULT_STATUS = "todo";
-const DEPENDENCY_GATED_STATUSES = new Set<string>(["in_progress", "in-progress", "done"]);
+const DEPENDENCY_GATED_STATUSES = new Set<string>(["in_progress", "done"]);
 
 interface EpicRow {
   id: string;
@@ -48,10 +51,12 @@ interface EpicRow {
 
 interface TaskRow extends EpicRow {
   epic_id: string;
+  owner: string | null;
 }
 
 interface SubtaskRow extends EpicRow {
   task_id: string;
+  owner: string | null;
 }
 
 interface DependencyRow {
@@ -146,6 +151,45 @@ function normalizeSubtaskDescription(value: string | undefined): string {
   return value.trim();
 }
 
+function isValidStatus(status: string): status is ValidStatus {
+  return (VALID_STATUSES as readonly string[]).includes(status);
+}
+
+export function validateStatusTransition(fromStatus: string, toStatus: string, entityKind: string, entityId: string): void {
+  if (fromStatus === toStatus) {
+    return;
+  }
+
+  if (!isValidStatus(toStatus)) {
+    throw new DomainError({
+      code: "status_transition_invalid",
+      message: `invalid status '${toStatus}' for ${entityKind} ${entityId}; allowed statuses: ${VALID_STATUSES.join(", ")}`,
+      details: { entity: entityKind, id: entityId, fromStatus, toStatus, allowedStatuses: [...VALID_STATUSES] },
+    });
+  }
+
+  if (!isValidStatus(fromStatus)) {
+    // Legacy/custom status from pre-0.3.1 data; allow transition to any valid
+    // status so existing databases can migrate forward without manual fixups.
+    return;
+  }
+
+  const allowed = VALID_TRANSITIONS.get(fromStatus);
+  if (!allowed || !allowed.has(toStatus)) {
+    throw new DomainError({
+      code: "status_transition_invalid",
+      message: `cannot transition ${entityKind} ${entityId} from '${fromStatus}' to '${toStatus}'`,
+      details: {
+        entity: entityKind,
+        id: entityId,
+        fromStatus,
+        toStatus,
+        allowedTransitions: allowed ? [...allowed] : [],
+      },
+    });
+  }
+}
+
 function mapEpic(row: EpicRow): EpicRecord {
   return {
     id: row.id,
@@ -164,6 +208,7 @@ function mapTask(row: TaskRow): TaskRecord {
     title: row.title,
     description: row.description,
     status: row.status,
+    owner: row.owner ?? null,
     createdAt: row.created_at,
     updatedAt: row.updated_at,
   };
@@ -176,6 +221,7 @@ function mapSubtask(row: SubtaskRow): SubtaskRecord {
     title: row.title,
     description: row.description,
     status: row.status,
+    owner: row.owner ?? null,
     createdAt: row.created_at,
     updatedAt: row.updated_at,
   };
@@ -376,21 +422,21 @@ export class TrackerDomain {
       this.getEpicOrThrow(epicId);
       const rows = this.#db
         .query(
-          "SELECT id, epic_id, title, description, status, created_at, updated_at FROM tasks WHERE epic_id = ? ORDER BY created_at ASC, id ASC;",
+          "SELECT id, epic_id, title, description, status, owner, created_at, updated_at FROM tasks WHERE epic_id = ? ORDER BY created_at ASC, id ASC;",
         )
         .all(epicId) as TaskRow[];
       return rows.map(mapTask);
     }
 
     const rows = this.#db
-      .query("SELECT id, epic_id, title, description, status, created_at, updated_at FROM tasks ORDER BY created_at ASC, id ASC;")
+      .query("SELECT id, epic_id, title, description, status, owner, created_at, updated_at FROM tasks ORDER BY created_at ASC, id ASC;")
       .all() as TaskRow[];
     return rows.map(mapTask);
   }
 
   getTask(id: string): TaskRecord | null {
     const row = this.#db
-      .query("SELECT id, epic_id, title, description, status, created_at, updated_at FROM tasks WHERE id = ?;")
+      .query("SELECT id, epic_id, title, description, status, owner, created_at, updated_at FROM tasks WHERE id = ?;")
       .get(id) as TaskRow | null;
     return row ? mapTask(row) : null;
   }
@@ -410,19 +456,20 @@ export class TrackerDomain {
 
   updateTask(
     id: string,
-    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
+    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): TaskRecord {
     const existing: TaskRecord = this.getTaskOrThrow(id);
     const nextTitle: string = input.title !== undefined ? assertNonEmpty("title", input.title) : existing.title;
     const nextDescription: string =
       input.description !== undefined ? assertNonEmpty("description", input.description) : existing.description;
     const nextStatus: string = input.status !== undefined ? assertNonEmpty("status", input.status) : existing.status;
+    const nextOwner: string | null = input.owner !== undefined ? input.owner : existing.owner;
     this.assertNoUnresolvedDependenciesForStatusTransition(id, "task", existing.status, nextStatus);
     const now: number = Date.now();
 
     this.#db
-      .query("UPDATE tasks SET title = ?, description = ?, status = ?, updated_at = ?, version = version + 1 WHERE id = ?;")
-      .run(nextTitle, nextDescription, nextStatus, now, id);
+      .query("UPDATE tasks SET title = ?, description = ?, status = ?, owner = ?, updated_at = ?, version = version + 1 WHERE id = ?;")
+      .run(nextTitle, nextDescription, nextStatus, nextOwner, now, id);
 
     return this.getTaskOrThrow(id);
   }
@@ -541,7 +588,7 @@ export class TrackerDomain {
       this.getTaskOrThrow(taskId);
       const rows = this.#db
         .query(
-          "SELECT id, task_id, title, description, status, created_at, updated_at FROM subtasks WHERE task_id = ? ORDER BY created_at ASC, id ASC;",
+          "SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE task_id = ? ORDER BY created_at ASC, id ASC;",
         )
         .all(taskId) as SubtaskRow[];
       return rows.map(mapSubtask);
@@ -549,15 +596,25 @@ export class TrackerDomain {
 
     const rows = this.#db
       .query(
-        "SELECT id, task_id, title, description, status, created_at, updated_at FROM subtasks ORDER BY created_at ASC, id ASC;",
+        "SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks ORDER BY created_at ASC, id ASC;",
       )
       .all() as SubtaskRow[];
     return rows.map(mapSubtask);
   }
 
+  getOpenSubtasks(taskId: string): readonly SubtaskRecord[] {
+    this.getTaskOrThrow(taskId);
+    const rows = this.#db
+      .query(
+        "SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE task_id = ? AND status != 'done' ORDER BY created_at ASC, id ASC;",
+      )
+      .all(taskId) as SubtaskRow[];
+    return rows.map(mapSubtask);
+  }
+
   getSubtask(id: string): SubtaskRecord | null {
     const row = this.#db
-      .query("SELECT id, task_id, title, description, status, created_at, updated_at FROM subtasks WHERE id = ?;")
+      .query("SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE id = ?;")
       .get(id) as SubtaskRow | null;
     return row ? mapSubtask(row) : null;
   }
@@ -577,19 +634,20 @@ export class TrackerDomain {
 
   updateSubtask(
     id: string,
-    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
+    input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): SubtaskRecord {
     const existing: SubtaskRecord = this.getSubtaskOrThrow(id);
     const nextTitle: string = input.title !== undefined ? assertNonEmpty("title", input.title) : existing.title;
     const nextDescription: string =
       input.description !== undefined ? normalizeSubtaskDescription(input.description) : existing.description;
     const nextStatus: string = input.status !== undefined ? assertNonEmpty("status", input.status) : existing.status;
+    const nextOwner: string | null = input.owner !== undefined ? input.owner : existing.owner;
     this.assertNoUnresolvedDependenciesForStatusTransition(id, "subtask", existing.status, nextStatus);
     const now: number = Date.now();
 
     this.#db
-      .query("UPDATE subtasks SET title = ?, description = ?, status = ?, updated_at = ?, version = version + 1 WHERE id = ?;")
-      .run(nextTitle, nextDescription, nextStatus, now, id);
+      .query("UPDATE subtasks SET title = ?, description = ?, status = ?, owner = ?, updated_at = ?, version = version + 1 WHERE id = ?;")
+      .run(nextTitle, nextDescription, nextStatus, nextOwner, now, id);
 
     return this.getSubtaskOrThrow(id);
   }
@@ -605,7 +663,7 @@ export class TrackerDomain {
     const taskIds = new Set(tasks.map((task) => task.id));
     const subtasks = this.#db
       .query(
-        "SELECT id, task_id, title, description, status, created_at, updated_at FROM subtasks WHERE task_id IN (SELECT id FROM tasks WHERE epic_id = ?) ORDER BY created_at ASC, id ASC;",
+        "SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE task_id IN (SELECT id FROM tasks WHERE epic_id = ?) ORDER BY created_at ASC, id ASC;",
       )
       .all(epicId) as SubtaskRow[];
 
@@ -921,6 +979,65 @@ export class TrackerDomain {
     return rows.map(mapDependency);
   }
 
+  /**
+   * Resolves dependency statuses for multiple tasks using a single prepared
+   * statement executed once per task ID.  This avoids the previous N+1 pattern
+   * where each task required separate getTaskOrThrow/getSubtaskOrThrow calls
+   * per dependency.
+   */
+  batchResolveDependencyStatuses(
+    taskIds: readonly string[],
+  ): Map<string, { totalDependencies: number; blockers: Array<{ id: string; kind: "task" | "subtask"; status: string }> }> {
+    const result = new Map<string, { totalDependencies: number; blockers: Array<{ id: string; kind: "task" | "subtask"; status: string }> }>();
+
+    if (taskIds.length === 0) {
+      return result;
+    }
+
+    // Use a static parameterised query per task ID rather than interpolating
+    // a dynamic IN-list into the SQL string.  This is consistent with every
+    // other query in TrackerDomain and avoids any placeholder-count confusion.
+    const stmt = this.#db.query(
+      `SELECT d.source_id, d.depends_on_id, d.depends_on_kind, COALESCE(t.status, s.status) AS dep_status
+       FROM dependencies d
+       LEFT JOIN tasks t ON d.depends_on_kind = 'task' AND d.depends_on_id = t.id
+       LEFT JOIN subtasks s ON d.depends_on_kind = 'subtask' AND d.depends_on_id = s.id
+       WHERE d.source_id = ?
+       ORDER BY d.created_at ASC, d.id ASC;`,
+    );
+
+    for (const taskId of taskIds) {
+      const entry = { totalDependencies: 0, blockers: [] as Array<{ id: string; kind: "task" | "subtask"; status: string }> };
+      result.set(taskId, entry);
+
+      const rows = stmt.all(taskId) as Array<{
+        source_id: string;
+        depends_on_id: string;
+        depends_on_kind: "task" | "subtask";
+        dep_status: string | null;
+      }>;
+
+      for (const row of rows) {
+        entry.totalDependencies += 1;
+
+        // Skip orphaned dependency rows (target deleted).
+        if (row.dep_status === null) {
+          continue;
+        }
+
+        if (row.dep_status !== "done") {
+          entry.blockers.push({
+            id: row.depends_on_id,
+            kind: row.depends_on_kind,
+            status: row.dep_status,
+          });
+        }
+      }
+    }
+
+    return result;
+  }
+
   listReverseDependencies(nodeId: string): readonly ReverseDependencyNode[] {
     const normalizedNodeId: string = assertNonEmpty("nodeId", nodeId);
     this.resolveNodeKind(normalizedNodeId);
@@ -1492,10 +1609,17 @@ export class TrackerDomain {
       }
 
       for (const dependency of this.listDependencies(change.id)) {
-        const dependencyStatus =
+        const dependencyNode =
           dependency.dependsOnKind === "task"
-            ? this.getTaskOrThrow(dependency.dependsOnId).status
-            : this.getSubtaskOrThrow(dependency.dependsOnId).status;
+            ? this.getTask(dependency.dependsOnId)
+            : this.getSubtask(dependency.dependsOnId);
+
+        // Skip orphaned dependency rows where the referenced node no longer exists.
+        if (!dependencyNode) {
+          continue;
+        }
+
+        const dependencyStatus = dependencyNode.status;
         const inScope = scopeIdSet.has(dependency.dependsOnId);
         const willCascade = targetStatus === "done" && changedIdSet.has(dependency.dependsOnId);
         if (dependencyStatus === "done" || willCascade) {
@@ -1560,19 +1684,24 @@ export class TrackerDomain {
     const unresolved: UnresolvedDependencyBlocker[] = [];
 
     for (const dependency of dependencies) {
-      const dependencyStatus =
+      const dependencyNode =
         dependency.dependsOnKind === "task"
-          ? this.getTaskOrThrow(dependency.dependsOnId).status
-          : this.getSubtaskOrThrow(dependency.dependsOnId).status;
+          ? this.getTask(dependency.dependsOnId)
+          : this.getSubtask(dependency.dependsOnId);
+
+      // Skip orphaned dependency rows where the referenced node no longer exists.
+      if (!dependencyNode) {
+        continue;
+      }
 
-      if (dependencyStatus === "done") {
+      if (dependencyNode.status === "done") {
         continue;
       }
 
       unresolved.push({
         id: dependency.dependsOnId,
         kind: dependency.dependsOnKind,
-        status: dependencyStatus,
+        status: dependencyNode.status,
       });
     }
 

package/src/domain/types.ts

@@ -1,5 +1,15 @@
 export type NodeKind = "epic" | "task" | "subtask";
 
+export const VALID_STATUSES = ["todo", "in_progress", "done", "blocked"] as const;
+export type ValidStatus = (typeof VALID_STATUSES)[number];
+
+export const VALID_TRANSITIONS: ReadonlyMap<ValidStatus, ReadonlySet<ValidStatus>> = new Map<ValidStatus, ReadonlySet<ValidStatus>>([
+  ["todo", new Set<ValidStatus>(["in_progress", "blocked"])],
+  ["in_progress", new Set<ValidStatus>(["done", "blocked"])],
+  ["blocked", new Set<ValidStatus>(["in_progress", "todo"])],
+  ["done", new Set<ValidStatus>(["in_progress"])],
+]);
+
 export const COMPACT_TEMP_KEY_PREFIX = "@";
 
 export type CompactTempKey = string;
@@ -95,6 +105,7 @@ export interface TaskRecord {
   readonly title: string;
   readonly description: string;
   readonly status: string;
+  readonly owner: string | null;
   readonly createdAt: number;
   readonly updatedAt: number;
 }
@@ -105,6 +116,7 @@ export interface SubtaskRecord {
   readonly title: string;
   readonly description: string;
   readonly status: string;
+  readonly owner: string | null;
   readonly createdAt: number;
   readonly updatedAt: number;
 }