@odla-ai/cli 0.30.1 → 0.30.3

package/README.md

@@ -147,7 +147,11 @@ Use `pm task ref <id>` (or the equivalent goal/decision/bug command) to print
 copy-ready structured markup for Discussion. The same markup may be pasted into
 `pm <entity> comment <id> --body "…"`; PM stores it as a structured reference,
 and `pm <entity> comments <id>` prints copy-ready markup instead of flattening
-the link back to a title.
+the link back to a title. For ordinary chat and handoffs, `pm task link <id>`
+prints a normal Markdown link to the record's durable Studio route. Successful
+create and lifecycle commands print that title-first link directly. Lead with
+the PM type and linked title, explain the state or next action, and keep the id
+hidden in the URL; a bare id list is not a useful handoff.
 
 When diagnosing a deployed app, agents can request one parseable observability
 snapshot instead of scraping Studio or composing collector routes themselves:

package/bin/odla-ai.js

@@ -0,0 +1,33 @@
+#!/usr/bin/env node
+import { existsSync, readFileSync } from "node:fs";
+import { spawnSync } from "node:child_process";
+import { dirname, join } from "node:path";
+import { fileURLToPath, pathToFileURL } from "node:url";
+import { workspaceBuildOrder } from "./workspace-build.js";
+
+const packageRoot = join(dirname(fileURLToPath(import.meta.url)), "..");
+const builtEntry = join(packageRoot, "dist", "bin.js");
+
+if (!existsSync(builtEntry)) {
+  const workspaceRoot = join(packageRoot, "..", "..");
+  const rootManifestPath = join(workspaceRoot, "package.json");
+  const rootManifest = existsSync(rootManifestPath)
+    ? JSON.parse(readFileSync(rootManifestPath, "utf8"))
+    : null;
+  if (rootManifest?.name !== "odla-ai" || rootManifest.private !== true) {
+    throw new Error("odla-ai CLI build is missing; reinstall @odla-ai/cli from npm");
+  }
+
+  console.error("odla-ai: workspace CLI is not built; building the local monorepo first");
+  const npm = process.platform === "win32" ? "npm.cmd" : "npm";
+  const workspaces = workspaceBuildOrder(workspaceRoot);
+  const child = spawnSync(
+    npm,
+    ["run", "build", ...workspaces.map((name) => `--workspace=${name}`)],
+    { cwd: workspaceRoot, stdio: "inherit" },
+  );
+  if (child.error) throw child.error;
+  if (child.status !== 0) process.exit(child.status ?? 1);
+}
+
+await import(pathToFileURL(builtEntry).href);

package/bin/workspace-build.d.ts

@@ -0,0 +1,2 @@
+/** Return buildable internal dependencies in dependency-first order. */
+export declare function workspaceBuildOrder(root: string): string[];

package/bin/workspace-build.js

@@ -0,0 +1,36 @@
+import { existsSync, readFileSync, readdirSync } from "node:fs";
+import { join } from "node:path";
+
+const manifestAt = (path) => JSON.parse(readFileSync(path, "utf8"));
+
+/** Return buildable internal dependencies in dependency-first order. */
+export function workspaceBuildOrder(root) {
+  const rootManifest = manifestAt(join(root, "package.json"));
+  const workspaces = new Map();
+  for (const pattern of rootManifest.workspaces ?? []) {
+    if (!pattern.endsWith("/*")) continue;
+    const parent = join(root, pattern.slice(0, -2));
+    if (!existsSync(parent)) continue;
+    for (const entry of readdirSync(parent, { withFileTypes: true })) {
+      if (!entry.isDirectory()) continue;
+      const manifestPath = join(parent, entry.name, "package.json");
+      if (!existsSync(manifestPath)) continue;
+      const manifest = manifestAt(manifestPath);
+      if (manifest.name) workspaces.set(manifest.name, manifest);
+    }
+  }
+
+  const visited = new Set();
+  const order = [];
+  const visit = (name) => {
+    if (visited.has(name)) return;
+    visited.add(name);
+    const manifest = workspaces.get(name);
+    if (!manifest) return;
+    const dependencies = { ...manifest.dependencies, ...manifest.devDependencies };
+    for (const dependency of Object.keys(dependencies)) visit(dependency);
+    if (manifest.scripts?.build) order.push(name);
+  };
+  visit("@odla-ai/cli");
+  return order;
+}

package/dist/bin.cjs

@@ -9608,6 +9608,7 @@ Usage:
   odla-ai pm bug add --app <id> --title <t> (--description <text>|--body <text>) [--status <s>] [--severity <s>] [--goal <id>] [--assignee <id>] [--decision <id>] [--mutation-id <id>] [--json]
   odla-ai bug report --app <id> --title <t> (--description <text>|--body <text>) [--severity <s>] [--json]
   odla-ai pm <goal|task|decision|bug> get <id> [--json]
+  odla-ai pm <goal|task|decision|bug> link <id> [--json]
   odla-ai pm <goal|task|decision|bug> ref <id> [--json]
   odla-ai pm goal set <id> [--title <t>|--status <s>|--proof <text>|--no-proof|--target <pct>|--no-target] [--mutation-id <id>] [--json]
   odla-ai pm task set <id> [--title <t>|--column <backlog|ready|doing|review|done>|--rank <n>|--goal <id>|--no-goal|--alignment-decision <id>|--no-alignment-decision|--execution <human|agent|either>|--assignee <id>|--no-assignee|--description <text>|--body <text>|--acceptance <text>|--no-acceptance|--due <epoch-ms>|--no-due|--expected-revision <n>] [--mutation-id <id>] [--json]
@@ -10509,16 +10510,26 @@ function referenceMarkup(entity, record10) {
   const label = (record10.title?.trim() || `${entity} ${record10.id}`).replaceAll("]", ")");
   return `@[${label}](pm:${entity}/${record10.id})`;
 }
+function studioRecordUrl(ctx, entity, id) {
+  return new URL(
+    `/studio/pm/${STUDIO_SECTION[entity]}/${encodeURIComponent(id)}`,
+    ctx.platformUrl
+  ).href;
+}
+function studioRecordLink(ctx, entity, record10) {
+  const label = (record10.title?.trim() || `${entity} ${record10.id}`).replaceAll("]", ")");
+  return `[${label}](${studioRecordUrl(ctx, entity, record10.id)})`;
+}
 function printRecord(ctx, entity, record10) {
   ctx.out.log(
-    `${record10.id}	[${statusCol(entity, record10)}]	${record10.appId}	${record10.title ?? ""}`
+    `${record10.id}	[${statusCol(entity, record10)}]	${record10.appId}	${studioRecordLink(ctx, entity, record10)}`
   );
 }
 function emit2(ctx, value2, human) {
   if (ctx.json) ctx.out.log(JSON.stringify(value2, null, 2));
   else human();
 }
-var DONE, writeMutationId2, FIELD_MAP;
+var DONE, writeMutationId2, FIELD_MAP, STUDIO_SECTION;
 var init_pm_action_core = __esm({
   "src/pm-action-core.ts"() {
     "use strict";
@@ -10551,6 +10562,12 @@ var init_pm_action_core = __esm({
       execution: { key: "executionMode" },
       "expected-revision": { key: "expectedRevision", num: true }
     };
+    STUDIO_SECTION = {
+      goal: "goals",
+      task: "board",
+      decision: "decisions",
+      bug: "bugs"
+    };
   }
 });
 
@@ -10596,7 +10613,8 @@ async function pmAdd(ctx, entity, parsed) {
     input,
     mutationId: writeMutationId2(parsed)
   });
-  emit2(ctx, res, () => ctx.out.log(`created ${entity} ${res.id}`));
+  const record10 = { id: res.id, appId, title: String(input.title) };
+  emit2(ctx, res, () => ctx.out.log(`created ${entity}: ${studioRecordLink(ctx, entity, record10)}`));
 }
 async function pmGet(ctx, entity, id) {
   const { record: record10 } = await pmRequest(ctx, "GET", `/${entity}/${encodeURIComponent(id)}`);
@@ -10621,7 +10639,10 @@ async function pmSet(ctx, entity, id, parsed) {
     patch: patch2,
     mutationId: writeMutationId2(parsed)
   });
-  emit2(ctx, res, () => ctx.out.log(res.record ? `${res.record.id}	[${statusCol(entity, res.record)}]	${res.record.appId}	${res.record.title ?? ""}` : `updated ${entity} ${id}`));
+  emit2(ctx, res, () => {
+    if (!res.record) return ctx.out.log(`updated ${entity} ${id}`);
+    ctx.out.log(`${entity}: ${studioRecordLink(ctx, entity, res.record)} \u2192 ${statusCol(entity, res.record)}`);
+  });
 }
 async function pmDone(ctx, entity, id, parsed) {
   const decisionId = stringOpt(parsed.options.decision);
@@ -10631,7 +10652,11 @@ async function pmDone(ctx, entity, id, parsed) {
     patch: patch2,
     mutationId: writeMutationId2(parsed)
   });
-  emit2(ctx, res, () => ctx.out.log(`${entity} ${id} \u2192 done`));
+  emit2(ctx, res, () => {
+    const label = res.record ? studioRecordLink(ctx, entity, res.record) : id;
+    const state2 = res.record ? statusCol(entity, res.record) : "done";
+    ctx.out.log(`${entity}: ${label} \u2192 ${state2}`);
+  });
 }
 async function pmTaskLifecycle(ctx, id, action2, parsed) {
   const rawRevision = stringOpt(parsed.options["expected-revision"]);
@@ -10660,7 +10685,8 @@ async function pmTaskLifecycle(ctx, id, action2, parsed) {
   );
   emit2(ctx, res, () => {
     const state2 = res.record ? statusCol("task", res.record) : action2;
-    ctx.out.log(`task ${id} \u2192 ${state2}`);
+    const label = res.record ? studioRecordLink(ctx, "task", res.record) : id;
+    ctx.out.log(`task: ${label} \u2192 ${state2}`);
   });
 }
 async function allRecords(ctx, entity, appId) {
@@ -10765,6 +10791,27 @@ var init_pm_actions = __esm({
   }
 });
 
+// src/pm-links.ts
+async function pmLink(ctx, entity, id) {
+  const { record: record10 } = await pmRequest(
+    ctx,
+    "GET",
+    `/${entity}/${encodeURIComponent(id)}`
+  );
+  const url = studioRecordUrl(ctx, entity, record10.id);
+  const markdown = studioRecordLink(ctx, entity, record10);
+  emit2(ctx, { kind: entity, id: record10.id, label: record10.title ?? "", url, markdown }, () => {
+    ctx.out.log(markdown);
+  });
+}
+var init_pm_links = __esm({
+  "src/pm-links.ts"() {
+    "use strict";
+    init_cjs_shims();
+    init_pm_action_core();
+  }
+});
+
 // src/pm-comments.ts
 async function pmComment(ctx, entity, id, parsed) {
   const body = stringOpt(parsed.options.body);
@@ -11091,7 +11138,7 @@ async function pmCommand(parsed, deps = {}) {
   if (!entity) throw new Error(`unknown pm entity "${word}". Try "odla-ai pm bug list" (goal|task|decision|bug).`);
   const requestedAction = parsed.positionals[2] ?? "list";
   const action2 = canonicalAction(requestedAction);
-  if (!action2) throw new Error(`unknown pm action "${requestedAction}". Try list|add|get|set|done|comment|comments|rm.`);
+  if (!action2) throw new Error(`unknown pm action "${requestedAction}". Try list|add|get|set|done|link|ref|comment|comments|rm.`);
   assertArgs(parsed, allowedOptions(entity, action2), 4);
   if ((action2 === "ready" || action2 === "claim" || action2 === "release") && entity !== "task") {
     throw new Error(`pm ${action2} is only valid for tasks`);
@@ -11115,6 +11162,8 @@ async function pmCommand(parsed, deps = {}) {
       return pmComments(ctx, entity, requireId2(id, action2));
     case "rm":
       return pmRemove(ctx, entity, requireId2(id, action2));
+    case "link":
+      return pmLink(ctx, entity, requireId2(id, action2));
     case "ref":
       return pmReference(ctx, entity, requireId2(id, action2));
     case "ready":
@@ -11131,6 +11180,7 @@ var init_pm_command = __esm({
     init_argv();
     init_operator_context();
     init_pm_actions();
+    init_pm_links();
     init_pm_comments();
     init_token();
     init_pm_watch();
@@ -11155,6 +11205,7 @@ var init_pm_command = __esm({
       ready: ["goal", "alignment-decision", "execution", "description", "desc", "body", "acceptance", "expected-revision", "mutation-id"],
       claim: ["expected-revision", "mutation-id"],
       release: ["expected-revision", "mutation-id"],
+      link: [],
       ref: []
     };
     ENTITY_OPTIONS = {