@dennisrongo/dsh-todo 0.4.0 → 0.5.0

package/lib/cli.js

@@ -22,6 +22,14 @@ function normalizeLabel(raw) {
   const text = raw.replace(/\s+/g, " ").trim().slice(0, MAX_LABEL);
   return text.length > 0 ? text : void 0;
 }
+var RELEASE_LABEL_RE = /^\d+(\.\d+){0,2}$/;
+var SPRINT_LABEL_RE = /^\d+(\.\d+)?$/;
+function normalizeVersionLabel(raw, field) {
+  const label = normalizeLabel(raw);
+  if (label === void 0) return void 0;
+  const pattern = field === "release" ? RELEASE_LABEL_RE : SPRINT_LABEL_RE;
+  return pattern.test(label) ? label : void 0;
+}
 var DATE_RE = /^\d{4}-\d{2}-\d{2}$/;
 function normalizeDueDate(raw) {
   if (typeof raw !== "string") return void 0;
@@ -34,6 +42,8 @@ function normalizeDueDate(raw) {
 var MAX_TEXT = 500;
 var MAX_DESC = 5e3;
 var MAX_LABEL = 60;
+var SUGGESTIONS_DIR = ".dsh";
+var SUGGESTIONS_FILE = `${SUGGESTIONS_DIR}/suggestions.json`;
 
 // src/db.ts
 var DOT_DSH = ".dsh";
@@ -76,6 +86,7 @@ function migrateSchema(db) {
   add("release", "release TEXT");
   add("sprint", "sprint TEXT");
   add("due_date", "due_date TEXT");
+  add("session_id", "session_id TEXT");
   if (addedTitle && columns.has("text")) {
     db.exec("UPDATE todo SET title = text WHERE title IS NULL");
   }
@@ -91,7 +102,7 @@ function readList(db) {
   const updatedAt = Number(db.prepare("SELECT value FROM meta WHERE key = 'updatedAt'").get()?.value ?? 0);
   const rows = db.prepare(
     `SELECT id, title, description, status, priority, release, sprint, due_date,
-              created_at, completed_at, archived_at
+              session_id, created_at, completed_at, archived_at
        FROM todo ORDER BY position ASC`
   ).all();
   const text = (v) => v === null || v === void 0 ? void 0 : String(v);
@@ -106,6 +117,7 @@ function readList(db) {
       ...normalizeLabel(row.release) !== void 0 ? { release: normalizeLabel(row.release) } : {},
       ...normalizeLabel(row.sprint) !== void 0 ? { sprint: normalizeLabel(row.sprint) } : {},
       ...normalizeDueDate(row.due_date) !== void 0 ? { dueDate: normalizeDueDate(row.due_date) } : {},
+      ...text(row.session_id) !== void 0 ? { sessionId: text(row.session_id) } : {},
       createdAt: Number(row.created_at),
       ...row.completed_at !== null && row.completed_at !== void 0 ? { completedAt: Number(row.completed_at) } : {},
       ...row.archived_at !== null && row.archived_at !== void 0 ? { archivedAt: Number(row.archived_at) } : {}
@@ -119,8 +131,8 @@ function writeList(db, items, revision, updatedAt = Date.now()) {
     db.prepare("DELETE FROM todo").run();
     const insert = db.prepare(
       `INSERT INTO todo (id, title, description, status, priority, release, sprint, due_date,
-                         text, done, created_at, completed_at, archived_at, position)
-       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
+                         session_id, text, done, created_at, completed_at, archived_at, position)
+       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
     );
     items.forEach((item, index) => {
       insert.run(
@@ -132,6 +144,7 @@ function writeList(db, items, revision, updatedAt = Date.now()) {
         item.release ?? null,
         item.sprint ?? null,
         item.dueDate ?? null,
+        item.sessionId ?? null,
         item.title,
         item.status === "done" ? 1 : 0,
         item.createdAt,
@@ -189,10 +202,13 @@ var CliError = class extends Error {
   /**
    * @param message - human-readable reason.
    * @param code - process exit code, from {@link EXIT}.
+   * @param details - extra machine-readable fields merged into the `--json`
+   * error payload, so an agent can correct itself without parsing the sentence.
    */
-  constructor(message, code = EXIT.usage) {
+  constructor(message, code = EXIT.usage, details = {}) {
     super(message);
     this.code = code;
+    this.details = details;
     this.name = "CliError";
   }
 };
@@ -210,6 +226,16 @@ function oneOf(options, key, allowed) {
   }
   return raw;
 }
+function assertLabel(field, raw) {
+  if (raw === void 0 || raw === "") return;
+  if (normalizeVersionLabel(raw, field) !== void 0) return;
+  const shape = field === "release" ? "a version number like 1.5 or 0.5.1 (up to three numbers)" : "a decimal number like 1.5 (one dot at most)";
+  throw new CliError(
+    `--${field} must be ${shape} (got "${raw}") \u2014 nothing was saved`,
+    EXIT.usage,
+    { field, expected: shape, got: raw }
+  );
+}
 function resolveWorkspace(options, cwd) {
   return resolve2(str(options, "workspace") ?? cwd);
 }
@@ -272,6 +298,7 @@ function formatItem(item) {
   if (item.release) meta.push(`release=${item.release}`);
   if (item.sprint) meta.push(`sprint=${item.sprint}`);
   if (item.dueDate) meta.push(`due=${item.dueDate}`);
+  if (item.sessionId) meta.push(`session=${item.sessionId}`);
   if (isArchived(item)) meta.push("archived");
   return bits.join(" ") + (meta.length ? `  (${meta.join(" ")})` : "");
 }
@@ -297,9 +324,11 @@ Options
 
   --status <s>               ${STATUSES.join("|")}
   --priority <p>             ${PRIORITIES.join("|")}
-  --release <label>          e.g. v1.2.0        (empty string clears)
-  --sprint <label>           e.g. "Sprint 24"   (empty string clears)
+  --release <n[.n[.n]]>      e.g. 1.5 or 0.5.1   (empty string clears)
+  --sprint <n[.n]>           e.g. 24             (empty string clears)
   --due <YYYY-MM-DD>         Calendar day       (empty string clears)
+  --session <id>             Harness session working the task (update only;
+                             empty string clears)
   --description <text>       Body text          (empty string clears)
   --title <text>             Rename (update only)
 
@@ -309,8 +338,8 @@ Ids may be given as any unambiguous prefix.
 
 Examples
   dsh-todo list --open --json
-  dsh-todo add "Fix token refresh" --priority p0 --release v1.2.0 --due 2026-03-14
-  dsh-todo update t1a2 --status in-progress --sprint "Sprint 24"
+  dsh-todo add "Fix token refresh" --priority p0 --release 1.5 --due 2026-03-14
+  dsh-todo update t1a2 --status in-progress --sprint 24
   dsh-todo done t1a2
 `;
 function run(parsed, cwd, now = Date.now, rand = Math.random) {
@@ -347,6 +376,7 @@ function run(parsed, cwd, now = Date.now, rand = Math.random) {
         `release     ${item.release ?? "-"}`,
         `sprint      ${item.sprint ?? "-"}`,
         `due         ${item.dueDate ?? "-"}`,
+        `session     ${item.sessionId ?? "-"}`,
         `created     ${new Date(item.createdAt).toISOString()}`,
         ...item.completedAt ? [`completed   ${new Date(item.completedAt).toISOString()}`] : [],
         ...item.archivedAt ? [`archived    ${new Date(item.archivedAt).toISOString()}`] : [],
@@ -358,8 +388,12 @@ function run(parsed, cwd, now = Date.now, rand = Math.random) {
       const title = positional.join(" ").trim();
       if (!title) throw new CliError("add needs a title");
       const description = str(options, "description");
-      const release = normalizeLabel(str(options, "release"));
-      const sprint = normalizeLabel(str(options, "sprint"));
+      const releaseRaw = str(options, "release");
+      const sprintRaw = str(options, "sprint");
+      assertLabel("release", releaseRaw);
+      assertLabel("sprint", sprintRaw);
+      const release = normalizeVersionLabel(releaseRaw, "release");
+      const sprint = normalizeVersionLabel(sprintRaw, "sprint");
       const dueRaw = str(options, "due");
       if (dueRaw !== void 0 && dueRaw !== "" && normalizeDueDate(dueRaw) === void 0) {
         throw new CliError(`--due must be a real calendar date as YYYY-MM-DD (got "${dueRaw}")`);
@@ -388,10 +422,13 @@ function run(parsed, cwd, now = Date.now, rand = Math.random) {
       const release = str(options, "release");
       const sprint = str(options, "sprint");
       const due = str(options, "due");
+      const session = str(options, "session");
       if (due !== void 0 && due !== "" && normalizeDueDate(due) === void 0) {
         throw new CliError(`--due must be a real calendar date as YYYY-MM-DD (got "${due}")`);
       }
-      if (status === void 0 && priority === void 0 && title === void 0 && description === void 0 && release === void 0 && sprint === void 0 && due === void 0) {
+      assertLabel("release", release);
+      assertLabel("sprint", sprint);
+      if (status === void 0 && priority === void 0 && title === void 0 && description === void 0 && release === void 0 && sprint === void 0 && due === void 0 && session === void 0) {
         throw new CliError("update needs at least one field to change");
       }
       let updated;
@@ -413,7 +450,7 @@ function run(parsed, cwd, now = Date.now, rand = Math.random) {
           }
           for (const [key, raw] of [["release", release], ["sprint", sprint]]) {
             if (raw === void 0) continue;
-            const label = normalizeLabel(raw);
+            const label = normalizeVersionLabel(raw, key);
             if (label !== void 0) next[key] = label;
             else delete next[key];
           }
@@ -422,6 +459,10 @@ function run(parsed, cwd, now = Date.now, rand = Math.random) {
             if (value !== void 0) next.dueDate = value;
             else delete next.dueDate;
           }
+          if (session !== void 0) {
+            if (session) next.sessionId = session.slice(0, MAX_LABEL);
+            else delete next.sessionId;
+          }
           updated = next;
           return next;
         });
@@ -487,12 +528,14 @@ function main(argv, cwd = process.cwd()) {
   const wantsJson = parsed.options.json === true;
   try {
     const outcome = run(parsed, cwd);
-    console.log(wantsJson ? JSON.stringify(outcome.json, null, 2) : outcome.text);
+    const json = outcome.json !== null && typeof outcome.json === "object" && !Array.isArray(outcome.json) ? { ok: true, ...outcome.json } : { ok: true, result: outcome.json };
+    console.log(wantsJson ? JSON.stringify(json, null, 2) : outcome.text);
     return EXIT.ok;
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     const code = error instanceof CliError ? error.code : 1;
-    if (wantsJson) console.log(JSON.stringify({ error: message, code }, null, 2));
+    const details = error instanceof CliError ? error.details : {};
+    if (wantsJson) console.log(JSON.stringify({ ok: false, error: message, code, ...details }, null, 2));
     else console.error(`dsh-todo: ${message}`);
     return code;
   }