@walknpm/todo-cli 1.2.0 → 2.0.0

package/src/todo.js

@@ -4,6 +4,8 @@
 export const PRIORITIES = ["high", "medium", "low"];
 const PRIORITY_RANK = { high: 0, medium: 1, low: 2 };
 
+const RECURRENCE_FREQS = ["daily", "weekly", "monthly", "yearly"];
+
 /** Normalise and validate a priority string. Defaults to "medium". */
 export function normalizePriority(p) {
   const v = String(p ?? "").toLowerCase().trim();
@@ -52,16 +54,73 @@ export function dueStatus(due) {
   return null;
 }
 
+/** Add days to a YYYY-MM-DD date string and return a new YYYY-MM-DD. */
+export function addDaysToDate(dateStr, days) {
+  const d = new Date(dateStr + "T00:00:00");
+  d.setDate(d.getDate() + days);
+  const y = d.getFullYear();
+  const m = String(d.getMonth() + 1).padStart(2, "0");
+  const day = String(d.getDate()).padStart(2, "0");
+  return `${y}-${m}-${day}`;
+}
+
+/** Add months to a YYYY-MM-DD date string. */
+export function addMonthsToDate(dateStr, months) {
+  const d = new Date(dateStr + "T00:00:00");
+  d.setMonth(d.getMonth() + months);
+  const y = d.getFullYear();
+  const m = String(d.getMonth() + 1).padStart(2, "0");
+  const day = String(d.getDate()).padStart(2, "0");
+  return `${y}-${m}-${day}`;
+}
+
+/** Add years to a YYYY-MM-DD date string. */
+export function addYearsToDate(dateStr, years) {
+  const d = new Date(dateStr + "T00:00:00");
+  d.setFullYear(d.getFullYear() + years);
+  const y = d.getFullYear();
+  const m = String(d.getMonth() + 1).padStart(2, "0");
+  const day = String(d.getDate()).padStart(2, "0");
+  return `${y}-${m}-${day}`;
+}
+
 function nextId(todos) {
   return todos.reduce((max, t) => Math.max(max, t.id), 0) + 1;
 }
 
+/** Ensure a todo has all new fields (backward-compat migration). */
+function migrate(todo) {
+  if (!todo.subtasks) todo.subtasks = [];
+  if (todo.recurring === undefined) todo.recurring = null;
+  if (!todo.dependsOn) todo.dependsOn = [];
+  if (todo.timeLogged === undefined) todo.timeLogged = 0;
+  if (todo.timerStartedAt === undefined) todo.timerStartedAt = null;
+  if (todo.pinned === undefined) todo.pinned = false;
+  if (todo.estimate === undefined) todo.estimate = null;
+  if (todo.assignee === undefined) todo.assignee = "";
+  if (todo.color === undefined) todo.color = "";
+  if (todo.order === undefined) todo.order = todo.id;
+  if (todo.completedAt === undefined) todo.completedAt = null;
+  return todo;
+}
+
+/** Migrate all todos in place. */
+export function migrateAll(todos) {
+  for (const t of todos) migrate(t);
+  return todos;
+}
+
+// ─── Core CRUD ─────────────────────────────────────────────────────────────
+
 /** Add a new todo. Returns the created todo object. */
-export function addTodo(todos, { title, priority, tags, category, due, note } = {}) {
+export function addTodo(todos, {
+  title, priority, tags, category, due, note,
+  estimate, assignee, color, pinned,
+} = {}) {
   if (!title || !title.trim()) {
     throw new Error("Title is required.");
   }
-  const todo = {
+  const todo = migrate({
     id: nextId(todos),
     title: title.trim(),
     done: false,
@@ -71,7 +130,18 @@ export function addTodo(todos, { title, priority, tags, category, due, note } =
     due: parseDueDate(due),
     note: note ? String(note).trim() : "",
     createdAt: new Date().toISOString(),
-  };
+    subtasks: [],
+    recurring: null,
+    dependsOn: [],
+    timeLogged: 0,
+    timerStartedAt: null,
+    pinned: Boolean(pinned),
+    estimate: estimate != null ? Math.max(0, Number(estimate) || 0) : null,
+    assignee: assignee ? String(assignee).trim() : "",
+    color: color ? String(color).trim() : "",
+    order: nextId(todos),
+    completedAt: null,
+  });
   todos.push(todo);
   return todo;
 }
@@ -81,25 +151,41 @@ function indexOf(todos, id) {
   return todos.findIndex((t) => t.id === Number(id));
 }
 
+/** Get a todo by id. Returns the todo or null. */
+export function getTodo(todos, id) {
+  const i = indexOf(todos, id);
+  return i === -1 ? null : migrate(todos[i]);
+}
+
 /** Delete a todo by id. Returns true if something was removed. */
 export function deleteTodo(todos, id) {
   const i = indexOf(todos, id);
   if (i === -1) return false;
   todos.splice(i, 1);
+  // Remove this id from any dependency lists.
+  for (const t of todos) {
+    t.dependsOn = t.dependsOn.filter((d) => d !== Number(id));
+  }
   return true;
 }
 
 /** Edit an existing todo by id with partial fields. Returns the updated todo or null. */
-export function editTodo(todos, id, { title, priority, tags, category, due, note } = {}) {
+export function editTodo(todos, id, {
+  title, priority, tags, category, due, note,
+  estimate, assignee, color,
+} = {}) {
   const i = indexOf(todos, id);
   if (i === -1) return null;
-  const todo = todos[i];
+  const todo = migrate(todos[i]);
   if (title != null) todo.title = title.trim();
   if (priority != null) todo.priority = normalizePriority(priority);
   if (tags != null) todo.tags = parseTags(tags);
   if (category != null) todo.category = String(category).trim();
   if (due !== undefined) todo.due = parseDueDate(due);
   if (note !== undefined) todo.note = String(note).trim();
+  if (estimate !== undefined) todo.estimate = estimate != null ? Math.max(0, Number(estimate) || 0) : null;
+  if (assignee !== undefined) todo.assignee = String(assignee).trim();
+  if (color !== undefined) todo.color = String(color).trim();
   todo.updatedAt = new Date().toISOString();
   return todo;
 }
@@ -108,20 +194,400 @@ export function editTodo(todos, id, { title, priority, tags, category, due, note
 export function toggleTodo(todos, id) {
   const i = indexOf(todos, id);
   if (i === -1) return null;
-  todos[i].done = !todos[i].done;
-  todos[i].updatedAt = new Date().toISOString();
-  return todos[i];
+  const todo = migrate(todos[i]);
+  todo.done = !todo.done;
+  todo.updatedAt = new Date().toISOString();
+  if (todo.done) {
+    todo.completedAt = new Date().toISOString();
+    // Handle recurring: create a new instance and reset this one.
+    if (todo.recurring) {
+      _rescheduleRecurring(todos, todo);
+    }
+  } else {
+    todo.completedAt = null;
+  }
+  return todo;
 }
 
 /** Mark a todo done by id. Returns the updated todo or null. */
 export function completeTodo(todos, id) {
   const i = indexOf(todos, id);
   if (i === -1) return null;
-  todos[i].done = true;
-  todos[i].updatedAt = new Date().toISOString();
-  return todos[i];
+  const todo = migrate(todos[i]);
+  if (!todo.done) {
+    todo.done = true;
+    todo.completedAt = new Date().toISOString();
+    todo.updatedAt = new Date().toISOString();
+    if (todo.recurring) {
+      _rescheduleRecurring(todos, todo);
+    }
+  }
+  return todo;
+}
+
+/** Mark ALL pending todos as done. Returns the count completed. */
+export function completeAll(todos) {
+  let n = 0;
+  const now = new Date().toISOString();
+  for (const t of todos) {
+    migrate(t);
+    if (!t.done) {
+      t.done = true;
+      t.completedAt = now;
+      t.updatedAt = now;
+      n++;
+      if (t.recurring) _rescheduleRecurring(todos, t);
+    }
+  }
+  return n;
+}
+
+// ─── Recurring ─────────────────────────────────────────────────────────────
+
+/** Set recurrence on a todo. freq: daily|weekly|monthly|yearly, interval: n. */
+export function setRecurring(todos, id, freq, interval = 1) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  const f = String(freq).toLowerCase().trim();
+  if (!RECURRENCE_FREQS.includes(f)) {
+    throw new Error(`Invalid recurrence frequency "${freq}". Use ${RECURRENCE_FREQS.join(", ")}.`);
+  }
+  const n = Math.max(1, Number(interval) || 1);
+  todo.recurring = { freq: f, interval: n };
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Remove recurrence from a todo. */
+export function removeRecurring(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.recurring = null;
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Internal: when a recurring todo is completed, create a new pending clone and reopen the original. */
+function _rescheduleRecurring(todos, todo) {
+  const newTodo = migrate({
+    ...structuredClone(todo),
+    id: nextId(todos),
+    done: false,
+    completedAt: null,
+    timerStartedAt: null,
+    createdAt: new Date().toISOString(),
+    updatedAt: new Date().toISOString(),
+    order: nextId(todos),
+  });
+  // Advance the due date.
+  if (todo.due) {
+    const { freq, interval } = todo.recurring;
+    if (freq === "daily") newTodo.due = addDaysToDate(todo.due, interval);
+    else if (freq === "weekly") newTodo.due = addDaysToDate(todo.due, interval * 7);
+    else if (freq === "monthly") newTodo.due = addMonthsToDate(todo.due, interval);
+    else if (freq === "yearly") newTodo.due = addYearsToDate(todo.due, interval);
+  }
+  // Reset subtasks.
+  newTodo.subtasks = newTodo.subtasks.map((s) => ({ ...s, done: false }));
+  todos.push(newTodo);
+}
+
+// ─── Subtasks ──────────────────────────────────────────────────────────────
+
+/** Add a subtask to a todo. Returns the subtask. */
+export function addSubtask(todos, id, title) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  if (!title || !title.trim()) throw new Error("Subtask title is required.");
+  const todo = migrate(todos[i]);
+  const sid = todo.subtasks.reduce((max, s) => Math.max(max, s.id), 0) + 1;
+  const subtask = { id: sid, title: title.trim(), done: false };
+  todo.subtasks.push(subtask);
+  todo.updatedAt = new Date().toISOString();
+  return subtask;
+}
+
+/** Toggle a subtask's done status. Returns the subtask or null. */
+export function toggleSubtask(todos, id, subtaskId) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  const st = todo.subtasks.find((s) => s.id === Number(subtaskId));
+  if (!st) return null;
+  st.done = !st.done;
+  todo.updatedAt = new Date().toISOString();
+  return st;
+}
+
+/** Delete a subtask. Returns true if removed. */
+export function deleteSubtask(todos, id, subtaskId) {
+  const i = indexOf(todos, id);
+  if (i === -1) return false;
+  const todo = migrate(todos[i]);
+  const idx = todo.subtasks.findIndex((s) => s.id === Number(subtaskId));
+  if (idx === -1) return false;
+  todo.subtasks.splice(idx, 1);
+  todo.updatedAt = new Date().toISOString();
+  return true;
+}
+
+/** Edit a subtask title. Returns the subtask or null. */
+export function editSubtask(todos, id, subtaskId, title) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  const st = todo.subtasks.find((s) => s.id === Number(subtaskId));
+  if (!st) return null;
+  st.title = title.trim();
+  todo.updatedAt = new Date().toISOString();
+  return st;
+}
+
+// ─── Dependencies ──────────────────────────────────────────────────────────
+
+/** Add a dependency: todo `id` depends on todo `depId`. */
+export function addDependency(todos, id, depId) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  const d = Number(depId);
+  if (d === todo.id) throw new Error("A todo cannot depend on itself.");
+  if (!todos.some((t) => t.id === d)) throw new Error(`Todo #${d} does not exist.`);
+  if (!todo.dependsOn.includes(d)) todo.dependsOn.push(d);
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Remove a dependency. */
+export function removeDependency(todos, id, depId) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.dependsOn = todo.dependsOn.filter((d) => d !== Number(depId));
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Check if a todo is blocked by incomplete dependencies. Returns array of blocking todo IDs. */
+export function getBlockedBy(todos, id) {
+  const todo = getTodo(todos, id);
+  if (!todo) return [];
+  return todo.dependsOn.filter((depId) => {
+    const dep = getTodo(todos, depId);
+    return dep && !dep.done;
+  });
+}
+
+// ─── Time Tracking ─────────────────────────────────────────────────────────
+
+/** Start a timer on a todo. Throws if one is already running. */
+export function startTimer(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  // Stop any other running timer.
+  for (const t of todos) {
+    if (t.timerStartedAt && t.id !== todo.id) {
+      _stopTimer(t);
+    }
+  }
+  todo.timerStartedAt = new Date().toISOString();
+  todo.updatedAt = todo.timerStartedAt;
+  return todo;
+}
+
+/** Stop the timer on a todo and log elapsed time. Returns the todo or null. */
+export function stopTimer(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  if (!todo.timerStartedAt) return todo;
+  _stopTimer(todo);
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+function _stopTimer(todo) {
+  if (!todo.timerStartedAt) return;
+  const elapsed = Math.round((Date.now() - new Date(todo.timerStartedAt).getTime()) / 60000);
+  if (elapsed > 0) todo.timeLogged = (todo.timeLogged || 0) + elapsed;
+  todo.timerStartedAt = null;
 }
 
+/** Manually log time on a todo. */
+export function logTime(todos, id, minutes) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  const m = Number(minutes);
+  if (isNaN(m) || m <= 0) throw new Error("Minutes must be a positive number.");
+  todo.timeLogged = (todo.timeLogged || 0) + m;
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Get the currently running timer todo, if any. */
+export function getActiveTimer(todos) {
+  return todos.find((t) => t.timerStartedAt) || null;
+}
+
+// ─── Pinning ───────────────────────────────────────────────────────────────
+
+/** Toggle pin status. Returns the todo or null. */
+export function togglePin(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.pinned = !todo.pinned;
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Pin a todo. */
+export function pinTodo(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.pinned = true;
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+/** Unpin a todo. */
+export function unpinTodo(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.pinned = false;
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+// ─── Duplicate ─────────────────────────────────────────────────────────────
+
+/** Duplicate a todo. Returns the new todo or null. */
+export function duplicateTodo(todos, id) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const orig = migrate(todos[i]);
+  const now = new Date().toISOString();
+  const copy = migrate({
+    ...structuredClone(orig),
+    id: nextId(todos),
+    done: false,
+    completedAt: null,
+    timerStartedAt: null,
+    timeLogged: 0,
+    createdAt: now,
+    updatedAt: now,
+    order: nextId(todos),
+    title: orig.title + " (copy)",
+  });
+  copy.subtasks = copy.subtasks.map((s) => ({ ...s, done: false }));
+  todos.push(copy);
+  return copy;
+}
+
+// ─── Bulk Operations ───────────────────────────────────────────────────────
+
+/** Complete multiple todos by IDs. Returns count completed. */
+export function bulkComplete(todos, ids) {
+  let n = 0;
+  const now = new Date().toISOString();
+  for (const id of ids) {
+    const todo = getTodo(todos, id);
+    if (todo && !todo.done) {
+      todo.done = true;
+      todo.completedAt = now;
+      todo.updatedAt = now;
+      n++;
+      if (todo.recurring) _rescheduleRecurring(todos, todo);
+    }
+  }
+  return n;
+}
+
+/** Delete multiple todos by IDs. Returns count deleted. */
+export function bulkDelete(todos, ids) {
+  let n = 0;
+  const idSet = new Set(ids.map(Number));
+  for (let i = todos.length - 1; i >= 0; i--) {
+    if (idSet.has(todos[i].id)) {
+      todos.splice(i, 1);
+      n++;
+    }
+  }
+  // Clean up dependency references.
+  for (const t of todos) {
+    t.dependsOn = t.dependsOn.filter((d) => !idSet.has(d));
+  }
+  return n;
+}
+
+/** Set priority on multiple todos. Returns count updated. */
+export function bulkPriority(todos, ids, priority) {
+  const p = normalizePriority(priority);
+  let n = 0;
+  const now = new Date().toISOString();
+  for (const id of ids) {
+    const todo = getTodo(todos, id);
+    if (todo) {
+      todo.priority = p;
+      todo.updatedAt = now;
+      n++;
+    }
+  }
+  return n;
+}
+
+/** Set category on multiple todos. Returns count updated. */
+export function bulkCategory(todos, ids, category) {
+  let n = 0;
+  const now = new Date().toISOString();
+  const c = category ? String(category).trim() : "";
+  for (const id of ids) {
+    const todo = getTodo(todos, id);
+    if (todo) {
+      todo.category = c;
+      todo.updatedAt = now;
+      n++;
+    }
+  }
+  return n;
+}
+
+/** Set assignee on multiple todos. Returns count updated. */
+export function bulkAssign(todos, ids, assignee) {
+  let n = 0;
+  const now = new Date().toISOString();
+  const a = assignee ? String(assignee).trim() : "";
+  for (const id of ids) {
+    const todo = getTodo(todos, id);
+    if (todo) {
+      todo.assignee = a;
+      todo.updatedAt = now;
+      n++;
+    }
+  }
+  return n;
+}
+
+// ─── Reorder ───────────────────────────────────────────────────────────────
+
+/** Move a todo to a new position in manual order. */
+export function reorderTodo(todos, id, newOrder) {
+  const i = indexOf(todos, id);
+  if (i === -1) return null;
+  const todo = migrate(todos[i]);
+  todo.order = Number(newOrder);
+  todo.updatedAt = new Date().toISOString();
+  return todo;
+}
+
+// ─── Listing & Filtering ───────────────────────────────────────────────────
+
 /**
  * Filter and sort the todos.
  * @param {Array}  todos
@@ -130,11 +596,13 @@ export function completeTodo(todos, id) {
  * @param {string} [filter.category] Exact category match.
  * @param {string} [filter.tag]      Exact tag match.
  * @param {string} [filter.priority] "high" | "medium" | "low"
- * @param {string} [filter.sort]     "priority" | "created" | "id"
+ * @param {string} [filter.assignee] Filter by assignee.
+ * @param {string} [filter.sort]     "priority" | "created" | "id" | "due" | "title" | "updated"
+ * @param {boolean} [filter.pinnedFirst] Pin items to top.
  */
 export function listTodos(todos, filter = {}) {
   const status = filter.status ?? "all";
-  let result = todos.slice();
+  let result = todos.slice().map(migrate);
 
   if (status === "done") result = result.filter((t) => t.done);
   else if (status === "pending") result = result.filter((t) => !t.done);
@@ -151,6 +619,16 @@ export function listTodos(todos, filter = {}) {
     const p = normalizePriority(filter.priority);
     result = result.filter((t) => t.priority === p);
   }
+  if (filter.assignee) {
+    const a = String(filter.assignee).toLowerCase().trim();
+    result = result.filter((t) => t.assignee.toLowerCase() === a);
+  }
+  if (filter.overdue) {
+    result = result.filter((t) => !t.done && dueStatus(t.due) === "overdue");
+  }
+  if (filter.dueToday) {
+    result = result.filter((t) => !t.done && dueStatus(t.due) === "today");
+  }
 
   const sort = filter.sort ?? "priority";
   switch (sort) {
@@ -168,6 +646,15 @@ export function listTodos(todos, filter = {}) {
         return a.due.localeCompare(b.due);
       });
       break;
+    case "title":
+      result.sort((a, b) => a.title.localeCompare(b.title));
+      break;
+    case "updated":
+      result.sort((a, b) => (b.updatedAt || "").localeCompare(a.updatedAt || ""));
+      break;
+    case "order":
+      result.sort((a, b) => (a.order || 0) - (b.order || 0));
+      break;
     case "priority":
     default:
       result.sort((a, b) => {
@@ -176,6 +663,12 @@ export function listTodos(todos, filter = {}) {
       });
       break;
   }
+
+  // Pin items to top if requested.
+  if (filter.pinnedFirst) {
+    result.sort((a, b) => (b.pinned ? 1 : 0) - (a.pinned ? 1 : 0));
+  }
+
   return result;
 }
 
@@ -186,20 +679,24 @@ export function clearTodos(todos) {
   return n;
 }
 
-/** Search todos by keyword across title, tags, category, and note. */
+/** Search todos by keyword across title, tags, category, note, and assignee. */
 export function searchTodos(todos, keyword) {
   if (!keyword) return [];
   const kw = String(keyword).toLowerCase().trim();
   return todos.filter((t) => {
+    migrate(t);
     return (
       t.title.toLowerCase().includes(kw) ||
       t.category.toLowerCase().includes(kw) ||
       t.note.toLowerCase().includes(kw) ||
+      t.assignee.toLowerCase().includes(kw) ||
       t.tags.some((tag) => tag.includes(kw))
     );
   });
 }
 
+// ─── Statistics ────────────────────────────────────────────────────────────
+
 /** Compute summary statistics over all todos. */
 export function getStats(todos) {
   const stats = {
@@ -207,35 +704,34 @@ export function getStats(todos) {
     done: 0,
     pending: 0,
     overdue: 0,
+    dueToday: 0,
+    pinned: 0,
+    totalTimeLogged: 0,
     byPriority: { high: 0, medium: 0, low: 0 },
     byCategory: {},
+    byAssignee: {},
+    completionRate: 0,
   };
   for (const t of todos) {
+    migrate(t);
     if (t.done) stats.done++;
     else stats.pending++;
     stats.byPriority[t.priority] = (stats.byPriority[t.priority] ?? 0) + 1;
     if (t.category) {
       stats.byCategory[t.category] = (stats.byCategory[t.category] ?? 0) + 1;
     }
+    if (t.assignee) {
+      stats.byAssignee[t.assignee] = (stats.byAssignee[t.assignee] ?? 0) + 1;
+    }
+    if (t.pinned) stats.pinned++;
+    stats.totalTimeLogged += t.timeLogged || 0;
     if (!t.done && dueStatus(t.due) === "overdue") stats.overdue++;
+    if (!t.done && dueStatus(t.due) === "today") stats.dueToday++;
   }
+  stats.completionRate = stats.total > 0 ? Math.round((stats.done / stats.total) * 100) : 0;
   return stats;
 }
 
-/** Mark ALL pending todos as done. Returns the count completed. */
-export function completeAll(todos) {
-  let n = 0;
-  const now = new Date().toISOString();
-  for (const t of todos) {
-    if (!t.done) {
-      t.done = true;
-      t.updatedAt = now;
-      n++;
-    }
-  }
-  return n;
-}
-
 /** Remove all completed todos. Returns the count archived (removed). */
 export function archiveCompleted(todos) {
   let n = 0;
@@ -248,6 +744,8 @@ export function archiveCompleted(todos) {
   return n;
 }
 
+// ─── Export ────────────────────────────────────────────────────────────────
+
 /** Export todos to a string in the given format: "json" | "csv" | "md". */
 export function exportTodos(todos, format = "json") {
   switch (format) {
@@ -257,20 +755,21 @@ export function exportTodos(todos, format = "json") {
       return exportMarkdown(todos);
     case "json":
     default:
-      return JSON.stringify(todos, null, 2);
+      return JSON.stringify(todos.map(migrate), null, 2);
   }
 }
 
 function exportCSV(todos) {
-  const cols = ["id", "title", "done", "priority", "category", "tags", "due", "note"];
+  const cols = ["id", "title", "done", "priority", "category", "tags", "due", "note", "assignee", "estimate", "timeLogged", "pinned"];
   const esc = (v) => {
     const s = String(v ?? "");
-    return /[",\n]/.test(s) ? `"${s.replace(/"/g, '""')}"` : s;
+    return /[",\n]/.test(s) ? `\"${s.replace(/"/g, '""')}\"` : s;
   };
   const lines = [cols.join(",")];
   for (const t of todos) {
+    migrate(t);
     lines.push(
-      [t.id, t.title, t.done, t.priority, t.category, t.tags.join(";"), t.due, t.note]
+      [t.id, t.title, t.done, t.priority, t.category, t.tags.join(";"), t.due, t.note, t.assignee, t.estimate ?? "", t.timeLogged, t.pinned]
         .map(esc)
         .join(",")
     );
@@ -280,12 +779,22 @@ function exportCSV(todos) {
 
 function exportMarkdown(todos) {
   if (todos.length === 0) return "_(no todos)_";
-  const lines = ["| ID | Status | Priority | Category | Tags | Due | Title |", 
-    "|----|--------|----------|----------|------|-----|-------|"];
+  const lines = ["| ID | Status | Priority | Category | Tags | Due | Assignee | Title |",
+    "|----|--------|----------|----------|------|-----|----------|-------|"];
   for (const t of todos) {
+    migrate(t);
     lines.push(
-      `| ${t.id} | ${t.done ? "✓" : "○"} | ${t.priority} | ${t.category || "-"} | ${t.tags.join(", ") || "-"} | ${t.due || "-"} | ${t.title} |`
+      `| ${t.id} | ${t.done ? "✓" : "○"} | ${t.priority} | ${t.category || "-"} | ${t.tags.join(", ") || "-"} | ${t.due || "-"} | ${t.assignee || "-"} | ${t.title} |`
     );
   }
   return lines.join("\n");
 }
+
+/** Format minutes as "Xh Ym". */
+export function formatTime(minutes) {
+  const m = Math.round(Number(minutes) || 0);
+  if (m < 60) return `${m}m`;
+  const h = Math.floor(m / 60);
+  const rem = m % 60;
+  return rem ? `${h}h ${rem}m` : `${h}h`;
+}