@odla-ai/cli 0.22.1 → 0.24.0

package/README.md

@@ -43,12 +43,20 @@ app is registered, start each session by reading its doing tasks, open bugs and
 goals, and recent decisions:
 
 ```bash
+npx odla-ai pm handoff --app <appId>
 npx odla-ai pm task list --app <appId> --column doing
 npx odla-ai pm bug list --app <appId> --status open
 npx odla-ai pm goal list --app <appId> --status open
 npx odla-ai pm decision list --app <appId> --limit 20
 ```
 
+`pm handoff` is the authoritative unresolved-state view: unmet goals, every
+non-done task, and every open or triaged bug. File bugs with actionable context
+(`pm bug add ... --description "reproduction/evidence"`); a title-only bug is
+rejected. When an accepted decision resolves or explicitly retains a bug, link
+it with `pm bug done <id> --decision <decision-id>` or
+`pm bug set <id> --status triaged --decision <decision-id>`.
+
 Read the active task, linked goal, and comments before changing code. Without
 `--app`, list commands span every project the account co-owns, which is the
 right first view when coordinating several efforts; scope every write to the

package/dist/bin.cjs

@@ -1390,6 +1390,7 @@ function chooseIdMode(options, rows) {
 async function appImport(options) {
   const cfg = await loadProjectConfig(options.configPath);
   const out = options.stdout ?? console;
+  const say = options.json ? (line) => out.error(line) : (line) => out.log(line);
   const doFetch = options.fetch ?? fetch;
   const { tenant } = resolveTenant(cfg, options.env);
   const text = options.file === "-" ? (options.readStdin ?? (() => (0, import_node_fs6.readFileSync)(0, "utf8")))() : (0, import_node_fs6.readFileSync)(options.file, "utf8");
@@ -1411,10 +1412,10 @@ ${detail}${more}`);
   }
   const chunks = (0, import_import.planImportChunks)(ops);
   const result = { tenant, rows: ops.length, chunks: chunks.length, committed: 0, duplicate: 0, txIds: [], ignored };
-  if (ignored.length > 0) out.log(`ignoring server-maintained field(s): ${ignored.join(", ")} \u2014 odla-db stamps these on write`);
+  if (ignored.length > 0) say(`ignoring server-maintained field(s): ${ignored.join(", ")} \u2014 odla-db stamps these on write`);
   if (options.dryRun || options.yes !== true) {
     const why = options.dryRun ? "--dry-run" : "no --yes";
-    out.log(`${tenant}: would upsert ${ops.length} row(s) across ${chunks.length} transaction(s) \u2014 nothing written (${why})`);
+    say(`${tenant}: would upsert ${ops.length} row(s) across ${chunks.length} transaction(s) \u2014 nothing written (${why})`);
     if (options.json) out.log(JSON.stringify(result, null, 2));
     return result;
   }
@@ -1623,6 +1624,7 @@ function printPlan(out, verb, pre, opts) {
 async function appTransfer(options) {
   const cfg = await loadProjectConfig(options.configPath);
   const out = options.stdout ?? console;
+  const say = options.json ? (line) => out.error(line) : (line) => out.log(line);
   const doFetch = options.fetch ?? fetch;
   const tenants = bothTenants(cfg);
   const route2 = endpointsFor(options.verb, tenants);
@@ -1643,7 +1645,7 @@ async function appTransfer(options) {
     throw new Error(`pre-flight failed${pre.body.error?.code ? ` (${pre.body.error.code})` : ""}: ${pre.body.error?.message ?? pre.status}`);
   }
   const plan = pre.body;
-  printPlan(out, options.verb, plan, options);
+  printPlan({ log: say }, options.verb, plan, options);
   if (options.verb === "go-live" && !plan.targetEmpty) {
     throw new Error(
       `${route2.target} already has data \u2014 go-live has already happened. Use \`odla-ai app promote --yes\` to push schema and rules, or \`odla-ai app refresh-sandbox --yes\` to pull live back down.`
@@ -1651,7 +1653,7 @@ async function appTransfer(options) {
   }
   if (plan.blockers.length > 0) throw new Error(`cannot ${options.verb}: ${plan.blockers.map((b) => b.message).join("; ")}`);
   if (options.dryRun || options.yes !== true) {
-    out.log(`nothing written (${options.dryRun ? "--dry-run" : "no --yes"})`);
+    say(`nothing written (${options.dryRun ? "--dry-run" : "no --yes"})`);
     if (options.json) out.log(JSON.stringify(plan, null, 2));
     return { ok: true, plan };
   }
@@ -1665,7 +1667,7 @@ async function appTransfer(options) {
     );
     if (res2.status !== 200) throw new Error(`promote failed${res2.body.error?.code ? ` (${res2.body.error.code})` : ""}: ${res2.body.error?.message ?? res2.status}`);
     if (options.json) out.log(JSON.stringify(res2.body, null, 2));
-    else out.log(`${route2.target}: promoted ${(res2.body.applied ?? []).join(", ")}`);
+    else say(`${route2.target}: promoted ${(res2.body.applied ?? []).join(", ")}`);
     return { ok: true, plan, result: res2.body };
   }
   const res = await api(
@@ -1684,7 +1686,7 @@ async function appTransfer(options) {
     doFetch
   );
   if (res.status !== 200) throw new Error(`${options.verb} failed${res.body.error?.code ? ` (${res.body.error.code})` : ""}: ${res.body.error?.message ?? res.status}`);
-  out.log(`${route2.target}: replaced from ${route2.source} (tx ${res.body.destination?.maxTx}, epoch ${res.body.destination?.epoch}) \u2014 connected clients resync automatically`);
+  say(`${route2.target}: replaced from ${route2.source} (tx ${res.body.destination?.maxTx}, epoch ${res.body.destination?.epoch}) \u2014 connected clients resync automatically`);
   if (options.includeFiles) await copyFiles(cfg, token, route2, doFetch, out);
   if (options.json) out.log(JSON.stringify(res.body, null, 2));
   return { ok: true, plan, result: res.body };
@@ -6424,13 +6426,14 @@ Usage:
   odla-ai app owners add <email> [--email <odla-account>] [--json]
   odla-ai app owners remove <email> [--email <odla-account>] [--json]
   odla-ai pm <goal|task|decision|bug> list [--app <id>] [--status <s>] [--severity <s>] [--column <c>] [--q <text>] [--json]
-  odla-ai pm <goal|task|decision|bug> add --app <id> --title <t> [--severity high|--column doing|--goal <id>|--body <text>|--proof <text>]
+  odla-ai pm <goal|task|decision|bug> add --app <id> --title <t> [--severity high|--column doing|--goal <id>|--description <text>|--body <text>|--proof <text>]
   odla-ai pm <goal|task|decision|bug> get <id> [--json]
-  odla-ai pm <goal|task|decision|bug> set <id> [--status <s>|--column <c>|--assignee <id>|--no-assignee|--goal <id>]
-  odla-ai pm <goal|task|decision|bug> done <id>
+  odla-ai pm <goal|task|decision|bug> set <id> [--status <s>|--column <c>|--assignee <id>|--no-assignee|--goal <id>|--decision <id>]
+  odla-ai pm <goal|task|decision|bug> done <id> [--decision <accepted-decision-id>]
   odla-ai pm <goal|task|decision|bug> comment <id> --body "..."
   odla-ai pm <goal|task|decision|bug> comments <id> [--json]
   odla-ai pm <goal|task|decision|bug> rm <id>
+  odla-ai pm handoff --app <id> [--json]
   odla-ai whoami [--json]
   odla-ai runbook ask "<question>" [--app <id>] [--all] [--json]
   odla-ai runbook search "<question>" [--app <id>] [--all] [--limit <n>] [--json]
@@ -7189,6 +7192,7 @@ var FIELD_MAP = {
   column: { key: "column" },
   rank: { key: "rank", num: true },
   goal: { key: "goalId" },
+  decision: { key: "decisionId" },
   assignee: { key: "assigneeId" },
   due: { key: "dueAt", num: true },
   proof: { key: "proof" },
@@ -7227,6 +7231,14 @@ function collectFields(parsed, allowClear) {
   }
   return out;
 }
+function collectEntityFields(entity, parsed, allowClear) {
+  const fields = collectFields(parsed, allowClear);
+  if ((entity === "task" || entity === "bug") && fields.body !== void 0) {
+    if (fields.description === void 0) fields.description = fields.body;
+    delete fields.body;
+  }
+  return fields;
+}
 function statusCol(entity, r) {
   if (entity === "bug") return `${r.status ?? ""}/${r.severity ?? ""}`;
   if (entity === "task") return String(r.column ?? "");
@@ -7241,7 +7253,14 @@ function emit(ctx, value, human) {
 }
 async function pmList(ctx, entity, parsed) {
   const q = new URLSearchParams();
-  const filters = { status: "status", severity: "severity", column: "column", goal: "goalId", assignee: "assigneeId" };
+  const filters = {
+    status: "status",
+    severity: "severity",
+    column: "column",
+    goal: "goalId",
+    assignee: "assigneeId",
+    decision: "decisionId"
+  };
   const app = stringOpt(parsed.options.app);
   if (app) q.set("app", app);
   for (const [flag, param] of Object.entries(filters)) {
@@ -7263,8 +7282,10 @@ async function pmList(ctx, entity, parsed) {
 async function pmAdd(ctx, entity, parsed) {
   const appId = stringOpt(parsed.options.app);
   if (!appId) throw new Error("pm add needs --app <appId>");
-  const input = collectFields(parsed, false);
+  const input = collectEntityFields(entity, parsed, false);
   if (!input.title) throw new Error("pm add needs --title <title>");
+  if (entity === "bug" && !input.description)
+    throw new Error("pm bug add needs --description <context> (or --body <context>)");
   const res = await pmRequest(ctx, "POST", `/${entity}`, { appId, input });
   emit(ctx, res, () => ctx.out.log(`created ${entity} ${res.id}`));
 }
@@ -7273,16 +7294,71 @@ async function pmGet(ctx, entity, id) {
   emit(ctx, record5, () => printRecord(ctx, entity, record5));
 }
 async function pmSet(ctx, entity, id, parsed) {
-  const patch2 = collectFields(parsed, true);
+  const patch2 = collectEntityFields(entity, parsed, true);
   if (Object.keys(patch2).length === 0)
     throw new Error("pm set needs at least one field flag (e.g. --status doing, --assignee me, --no-assignee)");
   const res = await pmRequest(ctx, "PATCH", `/${entity}/${encodeURIComponent(id)}`, { patch: patch2 });
   emit(ctx, res, () => ctx.out.log(res.record ? `${res.record.id}	[${statusCol(entity, res.record)}]	${res.record.appId}	${res.record.title ?? ""}` : `updated ${entity} ${id}`));
 }
-async function pmDone(ctx, entity, id) {
-  const res = await pmRequest(ctx, "PATCH", `/${entity}/${encodeURIComponent(id)}`, { patch: DONE[entity] });
+async function pmDone(ctx, entity, id, parsed) {
+  const decisionId = stringOpt(parsed.options.decision);
+  if (decisionId && entity !== "bug") throw new Error("--decision is only valid when completing a bug");
+  const patch2 = { ...DONE[entity], ...decisionId ? { decisionId } : {} };
+  const res = await pmRequest(ctx, "PATCH", `/${entity}/${encodeURIComponent(id)}`, { patch: patch2 });
   emit(ctx, res, () => ctx.out.log(`${entity} ${id} \u2192 done`));
 }
+async function allRecords(ctx, entity, appId) {
+  const records = [];
+  for (; ; ) {
+    const q = new URLSearchParams({
+      app: appId,
+      limit: "100",
+      offset: String(records.length)
+    });
+    const page = await pmRequest(ctx, "GET", `/${entity}?${q}`);
+    records.push(...page.records);
+    if (records.length >= page.total || page.records.length === 0) return records;
+  }
+}
+async function pmHandoff(ctx, parsed) {
+  const appId = stringOpt(parsed.options.app);
+  if (!appId) throw new Error("pm handoff needs --app <appId>");
+  const [goals, tasks, bugs] = await Promise.all([
+    allRecords(ctx, "goal", appId),
+    allRecords(ctx, "task", appId),
+    allRecords(ctx, "bug", appId)
+  ]);
+  const handoff = {
+    appId,
+    unmetGoals: goals.filter((record5) => record5.status !== "met"),
+    activeTasks: tasks.filter((record5) => record5.column !== "done"),
+    openBugs: bugs.filter((record5) => record5.status !== "fixed" && record5.status !== "wontfix")
+  };
+  const result = {
+    ...handoff,
+    clean: !handoff.unmetGoals.length && !handoff.activeTasks.length && !handoff.openBugs.length
+  };
+  emit(ctx, result, () => {
+    if (result.clean) {
+      ctx.out.log(`${appId}: no unresolved PM work`);
+      return;
+    }
+    ctx.out.log(`${appId}: authoritative PM handoff`);
+    for (const [label, records] of [
+      ["unmet goals", result.unmetGoals],
+      ["active tasks", result.activeTasks],
+      ["open bugs", result.openBugs]
+    ]) {
+      ctx.out.log(`${label}:`);
+      if (!records.length) ctx.out.log("- (none)");
+      else for (const record5 of records) printRecord(
+        ctx,
+        label === "unmet goals" ? "goal" : label === "active tasks" ? "task" : "bug",
+        record5
+      );
+    }
+  });
+}
 async function pmComment(ctx, entity, id, parsed) {
   const body = stringOpt(parsed.options.body);
   if (!body) throw new Error('pm comment needs --body "..."');
@@ -7333,6 +7409,7 @@ var ALLOWED = [
   "target",
   "description",
   "desc",
+  "decision",
   "limit",
   "offset",
   "q"
@@ -7355,6 +7432,10 @@ async function buildContext(parsed, deps) {
 }
 async function pmCommand(parsed, deps = {}) {
   const word = parsed.positionals[1] ?? "";
+  if (word === "handoff") {
+    assertArgs(parsed, ALLOWED, 2);
+    return pmHandoff(await buildContext(parsed, deps), parsed);
+  }
   const entity = ALIASES[word];
   if (!entity) throw new Error(`unknown pm entity "${word}". Try "odla-ai pm bug list" (goal|task|decision|bug).`);
   const action = parsed.positionals[2] ?? "list";
@@ -7375,7 +7456,7 @@ async function pmCommand(parsed, deps = {}) {
     case "move":
       return pmSet(ctx, entity, requireId(id, action), parsed);
     case "done":
-      return pmDone(ctx, entity, requireId(id, action));
+      return pmDone(ctx, entity, requireId(id, action), parsed);
     case "comment":
       return pmComment(ctx, entity, requireId(id, action), parsed);
     case "comments":
@@ -7851,7 +7932,10 @@ var COMMAND_SURFACE = {
   doctor: {},
   help: {},
   init: {},
-  pm: PM_ENTITIES,
+  pm: {
+    ...PM_ENTITIES,
+    handoff: {}
+  },
   provision: {},
   runbook: {
     ask: {},