trekoon 0.4.2 → 0.4.3

package/src/domain/mutation-service.ts

@@ -71,32 +71,32 @@ function normalizeOwnerInput(owner: string | null | undefined): string | null |
 
 /**
  * Thrown by the *WithIfMatch CAS variants when the supplied `If-Match`
- * `updatedAt` does not match the row currently in the database.
+ * version token does not match the row currently in the database.
  *
  * The error is **not** a `DomainError` so the generic `toBoardRouteError`
  * fall-through doesn't accidentally surface it as a 400 — route handlers
  * catch it explicitly and emit the canonical 409 `precondition_failed`
- * payload (with `currentUpdatedAt` fetched inside the same transaction
+ * payload (with `currentVersion` fetched inside the same transaction
  * that observed the mismatch).
  */
 export class PreconditionFailedError extends Error {
   readonly entityKind: "epic" | "task" | "subtask";
   readonly entityId: string;
-  readonly currentUpdatedAt: number;
-  readonly providedUpdatedAt: number;
+  readonly currentVersion: number;
+  readonly providedVersion: number;
 
   constructor(input: {
     entityKind: "epic" | "task" | "subtask";
     entityId: string;
-    currentUpdatedAt: number;
-    providedUpdatedAt: number;
+    currentVersion: number;
+    providedVersion: number;
   }) {
-    super("If-Match version does not match current updatedAt");
+    super("If-Match version does not match current version");
     this.name = "PreconditionFailedError";
     this.entityKind = input.entityKind;
     this.entityId = input.entityId;
-    this.currentUpdatedAt = input.currentUpdatedAt;
-    this.providedUpdatedAt = input.providedUpdatedAt;
+    this.currentVersion = input.currentVersion;
+    this.providedVersion = input.providedVersion;
   }
 }
 
@@ -132,11 +132,9 @@ const BOARD_IDEMPOTENCY_RETENTION_MS = 7 * 24 * 60 * 60 * 1000;
 // than keeps up with expiration.
 const BOARD_IDEMPOTENCY_PRUNE_INTERVAL_MS = 60 * 1000;
 
-// Module-level timestamp guards the prune sweep. Per-process (not
-// per-MutationService) so that a long-lived process making frequent claims
-// does not re-run the DELETE on every transaction. Reset to 0 by tests via
-// `__resetIdempotencyPruneThrottleForTests`.
-let lastIdempotencyPruneAt = 0;
+const idempotencyPruneByDatabase = new Map<string, number>();
+let memoryDatabasePruneKeys = new WeakMap<Database, string>();
+let nextMemoryDatabasePruneId = 0;
 
 /**
  * Test hook: resets the module-level prune throttle so a fresh test run
@@ -144,7 +142,26 @@ let lastIdempotencyPruneAt = 0;
  * never invoke this.
  */
 export function __resetIdempotencyPruneThrottleForTests(): void {
-  lastIdempotencyPruneAt = 0;
+  idempotencyPruneByDatabase.clear();
+  memoryDatabasePruneKeys = new WeakMap<Database, string>();
+  nextMemoryDatabasePruneId = 0;
+}
+
+function idempotencyPruneKeyForDatabase(db: Database): string {
+  const rows = db.query("PRAGMA database_list;").all() as Array<{ name: string; file: string }>;
+  const main = rows.find((row) => row.name === "main");
+  if (main?.file) {
+    return main.file;
+  }
+
+  const existing = memoryDatabasePruneKeys.get(db);
+  if (existing) {
+    return existing;
+  }
+  nextMemoryDatabasePruneId += 1;
+  const next = `:memory:${nextMemoryDatabasePruneId}`;
+  memoryDatabasePruneKeys.set(db, next);
+  return next;
 }
 
 interface ScopeReplacementResult {
@@ -296,10 +313,10 @@ export class MutationService {
    * Atomic If-Match CAS variant of {@link updateEpic}.
    *
    * The `If-Match` precondition is enforced INSIDE the write transaction
-   * via a SQL compare-and-swap (`UPDATE ... WHERE id = ? AND updated_at = ?`).
+   * via a SQL compare-and-swap (`UPDATE ... WHERE id = ? AND version = ?`).
    * If zero rows are affected we determine whether the row is missing
    * (→ `DomainError(not_found)`) or merely stale (→ {@link PreconditionFailedError}
-   * with the freshly-fetched `currentUpdatedAt`).
+   * with the freshly-fetched `currentVersion`).
    *
    * This eliminates the read-check-then-write race the previous route-level
    * check had: a concurrent writer could land between `parseIfMatchHeader`'s
@@ -312,7 +329,7 @@ export class MutationService {
    */
   updateEpicWithIfMatch(
     id: string,
-    ifMatchUpdatedAt: number,
+    ifMatchVersion: number,
     input: { title?: string | undefined; description?: string | undefined; status?: string | undefined },
   ): EpicRecord {
     return this.#writeTransaction((): EpicRecord => {
@@ -343,22 +360,22 @@ export class MutationService {
           `UPDATE epics
               SET title = ?, description = ?, status = ?, updated_at = ?, version = version + 1
             WHERE id = ?
-              AND updated_at = ?
+              AND version = ?
            RETURNING id`,
         )
-        .get(nextTitle, nextDescription, nextStatus, now, id, ifMatchUpdatedAt) as { id: string } | null;
+        .get(nextTitle, nextDescription, nextStatus, now, id, ifMatchVersion) as { id: string } | null;
 
       if (result === null) {
         // Zero rows changed. We already proved the row exists via
         // getEpicOrThrow, so the only remaining failure mode is a stale
-        // precondition. Re-fetch updatedAt inside the same tx so the
+        // precondition. Re-fetch version inside the same tx so the
         // caller's 409 carries the freshest value.
         const current = this.#domain.getEpicOrThrow(id);
         throw new PreconditionFailedError({
           entityKind: "epic",
           entityId: id,
-          currentUpdatedAt: current.updatedAt,
-          providedUpdatedAt: ifMatchUpdatedAt,
+          currentVersion: current.version,
+          providedVersion: ifMatchVersion,
         });
       }
 
@@ -389,17 +406,17 @@ export class MutationService {
    */
   updateEpicStatusCascadeWithIfMatch(
     id: string,
-    ifMatchUpdatedAt: number,
+    ifMatchVersion: number,
     status: string,
   ): StatusCascadePlan {
     return this.#writeTransaction((): StatusCascadePlan => {
       const existing = this.#domain.getEpicOrThrow(id);
-      if (existing.updatedAt !== ifMatchUpdatedAt) {
+      if (existing.version !== ifMatchVersion) {
         throw new PreconditionFailedError({
           entityKind: "epic",
           entityId: id,
-          currentUpdatedAt: existing.updatedAt,
-          providedUpdatedAt: ifMatchUpdatedAt,
+          currentVersion: existing.version,
+          providedVersion: ifMatchVersion,
         });
       }
       const plan = this.#domain.planStatusCascade("epic", id, status);
@@ -513,7 +530,7 @@ export class MutationService {
    */
   updateTaskWithIfMatch(
     id: string,
-    ifMatchUpdatedAt: number,
+    ifMatchVersion: number,
     input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): TaskRecord {
     return this.#writeTransaction((): TaskRecord => {
@@ -546,18 +563,18 @@ export class MutationService {
           `UPDATE tasks
               SET title = ?, description = ?, status = ?, owner = ?, updated_at = ?, version = version + 1
             WHERE id = ?
-              AND updated_at = ?
+              AND version = ?
            RETURNING id`,
         )
-        .get(nextTitle, nextDescription, nextStatus, nextOwner, now, id, ifMatchUpdatedAt) as { id: string } | null;
+        .get(nextTitle, nextDescription, nextStatus, nextOwner, now, id, ifMatchVersion) as { id: string } | null;
 
       if (result === null) {
         const current = this.#domain.getTaskOrThrow(id);
         throw new PreconditionFailedError({
           entityKind: "task",
           entityId: id,
-          currentUpdatedAt: current.updatedAt,
-          providedUpdatedAt: ifMatchUpdatedAt,
+          currentVersion: current.version,
+          providedVersion: ifMatchVersion,
         });
       }
 
@@ -686,7 +703,7 @@ export class MutationService {
       const nextStatus = input.status ?? existing.status;
       this.#db
         .query(
-          "UPDATE epics SET description = description || ?, status = ?, updated_at = ? WHERE id = ?;",
+          "UPDATE epics SET description = description || ?, status = ?, updated_at = ?, version = version + 1 WHERE id = ?;",
         )
         .run(separator + input.append, nextStatus, now, input.epicId);
       const epic = this.#domain.getEpicOrThrow(input.epicId);
@@ -1032,7 +1049,7 @@ export class MutationService {
    */
   updateSubtaskWithIfMatch(
     id: string,
-    ifMatchUpdatedAt: number,
+    ifMatchVersion: number,
     input: { title?: string | undefined; description?: string | undefined; status?: string | undefined; owner?: string | null | undefined },
   ): SubtaskRecord {
     return this.#writeTransaction((): SubtaskRecord => {
@@ -1063,18 +1080,18 @@ export class MutationService {
           `UPDATE subtasks
               SET title = ?, description = ?, status = ?, owner = ?, updated_at = ?, version = version + 1
             WHERE id = ?
-              AND updated_at = ?
+              AND version = ?
            RETURNING id`,
         )
-        .get(nextTitle, nextDescription, nextStatus, nextOwner, now, id, ifMatchUpdatedAt) as { id: string } | null;
+        .get(nextTitle, nextDescription, nextStatus, nextOwner, now, id, ifMatchVersion) as { id: string } | null;
 
       if (result === null) {
         const current = this.#domain.getSubtaskOrThrow(id);
         throw new PreconditionFailedError({
           entityKind: "subtask",
           entityId: id,
-          currentUpdatedAt: current.updatedAt,
-          providedUpdatedAt: ifMatchUpdatedAt,
+          currentVersion: current.version,
+          providedVersion: ifMatchVersion,
         });
       }
 
@@ -1581,12 +1598,13 @@ export class MutationService {
 
   #pruneExpiredIdempotencyKeys(now: number = Date.now()): void {
     // Skip if we swept recently — see BOARD_IDEMPOTENCY_PRUNE_INTERVAL_MS
-    // for rationale. Module-level throttle so it applies per-process even
-    // across MutationService instances.
-    if (now - lastIdempotencyPruneAt < BOARD_IDEMPOTENCY_PRUNE_INTERVAL_MS) {
+    // for rationale. The throttle is per database file so daemon processes
+    // serving several workspaces do not suppress each other's cleanup.
+    const pruneKey = idempotencyPruneKeyForDatabase(this.#db);
+    const lastPrunedAt = idempotencyPruneByDatabase.get(pruneKey) ?? 0;
+    if (now - lastPrunedAt < BOARD_IDEMPOTENCY_PRUNE_INTERVAL_MS) {
       return;
     }
-    lastIdempotencyPruneAt = now;
 
     const cutoff: number = now - BOARD_IDEMPOTENCY_RETENTION_MS;
     this.#db.query(
@@ -1596,6 +1614,7 @@ export class MutationService {
         AND created_at < ?;
       `,
     ).run(cutoff);
+    idempotencyPruneByDatabase.set(pruneKey, now);
   }
 
   #previewScopeReplacement(

package/src/domain/tracker-domain.ts

@@ -56,6 +56,7 @@ interface EpicRow {
   status: string;
   created_at: number;
   updated_at: number;
+  version: number;
 }
 
 interface TaskRow extends EpicRow {
@@ -208,6 +209,7 @@ function mapEpic(row: EpicRow): EpicRecord {
     status: row.status,
     createdAt: row.created_at,
     updatedAt: row.updated_at,
+    version: row.version,
   };
 }
 
@@ -221,6 +223,7 @@ function mapTask(row: TaskRow): TaskRecord {
     owner: row.owner ?? null,
     createdAt: row.created_at,
     updatedAt: row.updated_at,
+    version: row.version,
   };
 }
 
@@ -234,6 +237,7 @@ function mapSubtask(row: SubtaskRow): SubtaskRecord {
     owner: row.owner ?? null,
     createdAt: row.created_at,
     updatedAt: row.updated_at,
+    version: row.version,
   };
 }
 
@@ -299,7 +303,7 @@ export class TrackerDomain {
 
   listEpics(): readonly EpicRecord[] {
     const rows = this.#db
-      .query("SELECT id, title, description, status, created_at, updated_at FROM epics ORDER BY created_at ASC, id ASC;")
+      .query("SELECT id, title, description, status, created_at, updated_at, version FROM epics ORDER BY created_at ASC, id ASC;")
       .all() as EpicRow[];
     return rows.map(mapEpic);
   }
@@ -326,7 +330,7 @@ export class TrackerDomain {
   findActiveEpic(): EpicRecord | null {
     const inProgress = this.#db
       .query(
-        "SELECT id, title, description, status, created_at, updated_at FROM epics WHERE status = 'in_progress' LIMIT 1;",
+        "SELECT id, title, description, status, created_at, updated_at, version FROM epics WHERE status = 'in_progress' LIMIT 1;",
       )
       .get() as EpicRow | null;
     if (inProgress) {
@@ -335,7 +339,7 @@ export class TrackerDomain {
 
     const todo = this.#db
       .query(
-        "SELECT id, title, description, status, created_at, updated_at FROM epics WHERE status = 'todo' ORDER BY updated_at DESC LIMIT 1;",
+        "SELECT id, title, description, status, created_at, updated_at, version FROM epics WHERE status = 'todo' ORDER BY updated_at DESC LIMIT 1;",
       )
       .get() as EpicRow | null;
     if (todo) {
@@ -345,7 +349,7 @@ export class TrackerDomain {
     // Fallback: oldest epic regardless of status (mirrors epics[0] from listEpics).
     const oldest = this.#db
       .query(
-        "SELECT id, title, description, status, created_at, updated_at FROM epics ORDER BY created_at ASC, id ASC LIMIT 1;",
+        "SELECT id, title, description, status, created_at, updated_at, version FROM epics ORDER BY created_at ASC, id ASC LIMIT 1;",
       )
       .get() as EpicRow | null;
     return oldest ? mapEpic(oldest) : null;
@@ -353,7 +357,7 @@ export class TrackerDomain {
 
   getEpic(id: string): EpicRecord | null {
     const row = this.#db
-      .query("SELECT id, title, description, status, created_at, updated_at FROM epics WHERE id = ?;")
+      .query("SELECT id, title, description, status, created_at, updated_at, version FROM epics WHERE id = ?;")
       .get(id) as EpicRow | null;
     return row ? mapEpic(row) : null;
   }
@@ -469,7 +473,7 @@ export class TrackerDomain {
       const inPlaceholders: string = chunkIds.map(() => "?").join(", ");
       const chunkRows = this.#db
         .query(
-          `SELECT id, epic_id, title, description, status, owner, created_at, updated_at FROM tasks WHERE id IN (${inPlaceholders});`,
+          `SELECT id, epic_id, title, description, status, owner, created_at, updated_at, version FROM tasks WHERE id IN (${inPlaceholders});`,
         )
         .all(...chunkIds) as TaskRow[];
       fetchedRows.push(...chunkRows);
@@ -501,21 +505,21 @@ export class TrackerDomain {
       this.getEpicOrThrow(epicId);
       const rows = this.#db
         .query(
-          "SELECT id, epic_id, title, description, status, owner, 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, version 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, owner, 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, version 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, owner, created_at, updated_at FROM tasks WHERE id = ?;")
+      .query("SELECT id, epic_id, title, description, status, owner, created_at, updated_at, version FROM tasks WHERE id = ?;")
       .get(id) as TaskRow | null;
     return row ? mapTask(row) : null;
   }
@@ -661,7 +665,7 @@ export class TrackerDomain {
       const inPlaceholders: string = chunkIds.map(() => "?").join(", ");
       const chunkRows = this.#db
         .query(
-          `SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE id IN (${inPlaceholders});`,
+          `SELECT id, task_id, title, description, status, owner, created_at, updated_at, version FROM subtasks WHERE id IN (${inPlaceholders});`,
         )
         .all(...chunkIds) as SubtaskRow[];
       fetchedRows.push(...chunkRows);
@@ -737,7 +741,7 @@ export class TrackerDomain {
       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 = ? ORDER BY created_at ASC, id ASC;",
+          "SELECT id, task_id, title, description, status, owner, created_at, updated_at, version FROM subtasks WHERE task_id = ? ORDER BY created_at ASC, id ASC;",
         )
         .all(taskId) as SubtaskRow[];
       return rows.map(mapSubtask);
@@ -745,7 +749,7 @@ export class TrackerDomain {
 
     const rows = this.#db
       .query(
-        "SELECT id, task_id, title, description, status, owner, 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, version FROM subtasks ORDER BY created_at ASC, id ASC;",
       )
       .all() as SubtaskRow[];
     return rows.map(mapSubtask);
@@ -755,7 +759,7 @@ export class TrackerDomain {
     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;",
+        "SELECT id, task_id, title, description, status, owner, created_at, updated_at, version FROM subtasks WHERE task_id = ? AND status != 'done' ORDER BY created_at ASC, id ASC;",
       )
       .all(taskId) as SubtaskRow[];
     return rows.map(mapSubtask);
@@ -763,7 +767,7 @@ export class TrackerDomain {
 
   getSubtask(id: string): SubtaskRecord | null {
     const row = this.#db
-      .query("SELECT id, task_id, title, description, status, owner, created_at, updated_at FROM subtasks WHERE id = ?;")
+      .query("SELECT id, task_id, title, description, status, owner, created_at, updated_at, version FROM subtasks WHERE id = ?;")
       .get(id) as SubtaskRow | null;
     return row ? mapSubtask(row) : null;
   }
@@ -862,7 +866,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, 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;",
+        "SELECT id, task_id, title, description, status, owner, created_at, updated_at, version FROM subtasks WHERE task_id IN (SELECT id FROM tasks WHERE epic_id = ?) ORDER BY created_at ASC, id ASC;",
       )
       .all(epicId) as SubtaskRow[];
 
@@ -1266,7 +1270,7 @@ export class TrackerDomain {
       const inPlaceholders: string = chunkIds.map(() => "?").join(", ");
       const rows = this.#db
         .query(
-          `SELECT id, task_id, title, description, status, owner, created_at, updated_at
+          `SELECT id, task_id, title, description, status, owner, created_at, updated_at, version
            FROM subtasks
            WHERE task_id IN (${inPlaceholders})
            ORDER BY created_at ASC, id ASC;`,

package/src/domain/types.ts

@@ -97,6 +97,7 @@ export interface EpicRecord {
   readonly status: string;
   readonly createdAt: number;
   readonly updatedAt: number;
+  readonly version: number;
 }
 
 export interface TaskRecord {
@@ -108,6 +109,7 @@ export interface TaskRecord {
   readonly owner: string | null;
   readonly createdAt: number;
   readonly updatedAt: number;
+  readonly version: number;
 }
 
 export interface SubtaskRecord {
@@ -119,6 +121,7 @@ export interface SubtaskRecord {
   readonly owner: string | null;
   readonly createdAt: number;
   readonly updatedAt: number;
+  readonly version: number;
 }
 
 export interface DependencyRecord {

package/src/export/render-markdown.ts

@@ -124,8 +124,7 @@ function renderTaskIndex(lines: string[], bundle: ExportBundle): void {
   lines.push("| # | Title | Status | Subtasks |");
   lines.push("|---|-------|--------|----------|");
 
-  for (let i = 0; i < bundle.tasks.length; i++) {
-    const task = bundle.tasks[i];
+  for (const [i, task] of bundle.tasks.entries()) {
     const subtaskCount = bundle.subtasks.filter((s) => s.taskId === task.id).length;
     const anchor = taskAnchor(task);
     lines.push(`| ${i + 1} | [${escapeTableCell(escapeInlineText(task.title))}](#${anchor}) | ${task.status} | ${subtaskCount} |`);