busa-cli 0.51.0 → 0.52.0

package/dist/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { i as runCli, r as pkgVersion, t as checkForUpdate } from "./update-check-BWyS6qOd.js";
+import { i as runCli, r as pkgVersion, t as checkForUpdate } from "./update-check-CNYcPsfm.js";
 //#region src/cli.ts
 const exitCode = await runCli(process.argv.slice(2));
 const notice = await checkForUpdate("busabase-cli", pkgVersion());

package/dist/index.js

@@ -1,3 +1,3 @@
-import { a as render, i as runCli, n as HELP, r as pkgVersion, t as checkForUpdate } from "./update-check-BWyS6qOd.js";
+import { a as render, i as runCli, n as HELP, r as pkgVersion, t as checkForUpdate } from "./update-check-CNYcPsfm.js";
 import { DEFAULT_BASE_URL, createBusabaseClient, normalizeBaseUrl } from "busabase-sdk";
 export { DEFAULT_BASE_URL, HELP, checkForUpdate, createBusabaseClient, normalizeBaseUrl, pkgVersion, render, runCli };

package/dist/{update-check-BWyS6qOd.js → update-check-CNYcPsfm.js}

@@ -1621,6 +1621,90 @@ const rejectMismatchedPayload = (input) => {
 	if ((input.files !== void 0 || input.mergeMode !== void 0) && !FILE_TREE_TYPES.has(type)) wrong.push(input.files !== void 0 ? "files" : "mergeMode");
 	if (wrong.length > 0) throw new Error(`${wrong.join(", ")} ${wrong.length > 1 ? "are" : "is"} not valid for --type ${type}.`);
 };
+/**
+* Validate `agentPrompts` BEFORE the node is created, against the same schema
+* the dedicated endpoint and `nodes set-agent-prompts` use.
+*
+* Order matters: prompts are written in a second call, so a malformed list
+* discovered afterwards would leave a created node behind and still fail. The
+* caller then has a node they did not expect and an error that reads like the
+* creation failed. Validating first makes the whole task all-or-nothing for the
+* one failure mode the caller can actually fix.
+*/
+const parseAgentPrompts = (input) => {
+	if (input.agentPrompts === void 0 || input.agentPrompts === null) return void 0;
+	const parsed = customAgentPromptsSchema.safeParse(input.agentPrompts);
+	if (!parsed.success) {
+		const detail = parsed.error.issues.map((issue) => `${issue.path.length > 0 ? `${issue.path.join(".")}: ` : ""}${issue.message}`).join("; ");
+		throw new Error(`Invalid agentPrompts — ${detail}`);
+	}
+	return parsed.data;
+};
+const isRecord = (value) => typeof value === "object" && value !== null;
+/**
+* The node id to hang prompts on, or `undefined` when this call did not
+* materialize a node.
+*
+* Every create endpoint returns a `materialized` discriminator, but the
+* materialized half is shaped per type: a Base VO carries the owning `nodeId`,
+* a Doc/File/Skill/Drive/AirApp wraps its node under `node`, and the generic
+* tree endpoint returns a ChangeRequest whose merged create operation names it.
+*/
+const materializedNodeId = (type, result) => {
+	if (!isRecord(result)) return void 0;
+	if (result.materialized === false) return void 0;
+	if (type === "base") return typeof result.nodeId === "string" ? result.nodeId : void 0;
+	if (isRecord(result.node) && typeof result.node.id === "string") return result.node.id;
+	if (result.status !== "merged") return void 0;
+	const created = (Array.isArray(result.operations) ? result.operations : []).find((op) => isRecord(op) && typeof op.nodeId === "string");
+	return isRecord(created) ? created.nodeId : void 0;
+};
+/**
+* Write the prompts onto the node just created, and report what happened in the
+* task's own result.
+*
+* Two deliberate choices:
+*
+* - **A pending ChangeRequest is reported, never silently dropped.** Review-first
+*   callers get a node id only after a human merges, so the prompts genuinely
+*   cannot be written here. Saying so (with the command that finishes the job)
+*   is the difference between "not yet" and a caller believing prompts exist.
+* - **A server without the dedicated endpoint falls back to the metadata key.**
+*   The CLI and the server ship separately, so a current caller routinely talks
+*   to an older server; `nodes set-agent-prompts` already carries this fallback
+*   for exactly that reason and this path must not be the one that regresses.
+*/
+const writeAgentPrompts = async (client, type, result, agentPrompts) => {
+	const nodeId = materializedNodeId(type, result);
+	if (!nodeId) return {
+		...isRecord(result) ? result : { result },
+		agentPromptsWrite: {
+			written: false,
+			reason: "Prompts are addressed by node id, and this call proposed the node for review instead of creating it — there is no node id yet.",
+			nextStep: "Once the change request merges: busabase-cli nodes set-agent-prompts --node-id <nodeId> --file prompts.json"
+		}
+	};
+	try {
+		await client.nodes.updateAgentPrompts({
+			nodeId,
+			agentPrompts
+		});
+	} catch (error) {
+		if ((isRecord(error) ? error.status : void 0) !== 404) throw error;
+		await client.nodes.updateMetadata({
+			nodeId,
+			metadata: { agentPrompts }
+		});
+	}
+	return {
+		...isRecord(result) ? result : { result },
+		agentPromptsWrite: {
+			written: true,
+			nodeId,
+			count: agentPrompts.length
+		}
+	};
+};
 /** `autoMerge` is tri-state: unset (permission-aware default), forced on, forced off. */
 const mergeIntent$3 = (input) => {
 	if (input.requireReview) return { autoMerge: false };
@@ -1635,6 +1719,66 @@ const mergeIntent$3 = (input) => {
 * new field type added.
 */
 const asFieldArray = (value) => Array.isArray(value) ? value : void 0;
+/**
+* Dispatch to the endpoint that can actually build this node type. Split out
+* of `execute` so the prompt write that follows reads as its own step.
+*/
+const createNode = async (client, input) => {
+	const type = input.type;
+	const common = {
+		slug: input.slug,
+		name: input.name,
+		description: input.description,
+		parentNodeId: input.parentNodeId,
+		...mergeIntent$3(input)
+	};
+	if (FILE_TREE_TYPES.has(type)) {
+		const payload = {
+			...common,
+			type,
+			...Array.isArray(input.files) ? { files: input.files } : {},
+			...input.mergeMode ? { mergeMode: input.mergeMode } : {},
+			...input.visibility ? { visibility: input.visibility } : {},
+			...input.version ? { version: input.version } : {}
+		};
+		return client.fileTrees.create(payload);
+	}
+	if (type === "doc") {
+		const payload = {
+			...common,
+			body: input.body ?? ""
+		};
+		return client.docs.create(payload);
+	}
+	if (type === "base") {
+		const payload = {
+			...common,
+			fields: asFieldArray(input.fields) ?? []
+		};
+		return client.bases.create(payload);
+	}
+	if (type === "file") {
+		if (!input.assetId) throw new Error("assetId is required for type \"file\".");
+		const payload = {
+			...common,
+			assetId: input.assetId
+		};
+		return client.files.create(payload);
+	}
+	return client.nodes.createChangeRequest({
+		message: input.message ?? `Create ${type} ${input.name}`,
+		submittedBy: input.submittedBy,
+		...mergeIntent$3(input),
+		operations: [{
+			kind: "create",
+			nodeType: type,
+			slug: input.slug,
+			name: input.name,
+			description: input.description,
+			parentNodeId: input.parentNodeId
+		}]
+	});
+};
 const nodeCreateTask = {
 	name: "node_create",
 	cliPath: ["nodes", "create"],
@@ -1672,7 +1816,7 @@ const nodeCreateTask = {
 		}
 	],
 	summary: "Create a workspace node of any type",
-	guidance: "One call creates any of the 11 node types with its type-specific payload: `fields` for a Base, `body` for a Doc, `files` for a Skill/Drive/AirApp, `assetId` for a File. Review is permission-aware, decided server-side: this merges immediately when you already have write access on the parent node, and proposes a ChangeRequest for a human otherwise. Pass requireReview to always propose instead of merging.",
+	guidance: "One call creates any of the 11 node types with its type-specific payload: `fields` for a Base, `body` for a Doc, `files` for a Skill/Drive/AirApp, `assetId` for a File. Review is permission-aware, decided server-side: this merges immediately when you already have write access on the parent node, and proposes a ChangeRequest for a human otherwise. Pass requireReview to always propose instead of merging. Before you finish: if the person will come back to this node to do the same job again, pass agentPrompts so the node opens with THEIR job on it instead of the node type's generic list.",
 	annotations: {
 		readOnly: false,
 		destructive: false
@@ -1818,6 +1962,11 @@ const nodeCreateTask = {
 			name: "requireReview",
 			kind: "boolean",
 			description: "Always propose a pending ChangeRequest, even with write access."
+		},
+		{
+			name: "agentPrompts",
+			kind: "json",
+			description: "Scenario prompts shown when someone opens this node and asks an agent for something: [{\"key\":\"log-visit\",\"label\":\"Log a customer visit\",\"body\":\"{target}\\n\\nAdd a visit record with today's date, the contact I name, and a one-line summary.\",\"intent\":\"change\"}]. Write what the PERSON wants in their own words (\"Log a customer visit\"), not the operation (\"Create a record in Visits\") — the node type already covers the operations. These REPLACE the node type's default scenario prompts, so 2-5 real recurring jobs help and a generic pair is worse than none: omit this when you cannot name one. `{target}` expands to a complete sentence naming the node and space, so give it its own line. Stored in a second call after the node exists, so they are skipped (and reported) when this call proposes a ChangeRequest instead of creating the node; add them afterwards with `nodes set-agent-prompts`."
 		}
 	],
 	examples: [
@@ -1825,64 +1974,14 @@ const nodeCreateTask = {
 		"busabase-cli nodes create --type base --slug blog --name \"Blog Posts\" --field title:Title:text --field body:Body:markdown",
 		"busabase-cli nodes create --type doc --slug readme --name \"README\" --body \"# Hello\"",
 		"busabase-cli nodes create --type skill --slug my-skill --name \"My Skill\" --files-json @files.json",
-		"busabase-cli nodes create --type folder --slug cms --name \"CMS\" --require-review"
+		"busabase-cli nodes create --type folder --slug cms --name \"CMS\" --require-review",
+		"busabase-cli nodes create --type base --slug visits --name \"Visits\" --field title:Title:text --agent-prompts-json @prompts.json"
 	],
 	execute: async (client, input) => {
 		rejectMismatchedPayload(input);
-		const type = input.type;
-		const common = {
-			slug: input.slug,
-			name: input.name,
-			description: input.description,
-			parentNodeId: input.parentNodeId,
-			...mergeIntent$3(input)
-		};
-		if (FILE_TREE_TYPES.has(type)) {
-			const payload = {
-				...common,
-				type,
-				...Array.isArray(input.files) ? { files: input.files } : {},
-				...input.mergeMode ? { mergeMode: input.mergeMode } : {},
-				...input.visibility ? { visibility: input.visibility } : {},
-				...input.version ? { version: input.version } : {}
-			};
-			return client.fileTrees.create(payload);
-		}
-		if (type === "doc") {
-			const payload = {
-				...common,
-				body: input.body ?? ""
-			};
-			return client.docs.create(payload);
-		}
-		if (type === "base") {
-			const payload = {
-				...common,
-				fields: asFieldArray(input.fields) ?? []
-			};
-			return client.bases.create(payload);
-		}
-		if (type === "file") {
-			if (!input.assetId) throw new Error("assetId is required for type \"file\".");
-			const payload = {
-				...common,
-				assetId: input.assetId
-			};
-			return client.files.create(payload);
-		}
-		return client.nodes.createChangeRequest({
-			message: input.message ?? `Create ${type} ${input.name}`,
-			submittedBy: input.submittedBy,
-			...mergeIntent$3(input),
-			operations: [{
-				kind: "create",
-				nodeType: type,
-				slug: input.slug,
-				name: input.name,
-				description: input.description,
-				parentNodeId: input.parentNodeId
-			}]
-		});
+		const agentPrompts = parseAgentPrompts(input);
+		const created = await createNode(client, input);
+		return agentPrompts === void 0 ? created : writeAgentPrompts(client, input.type, created, agentPrompts);
 	}
 };
 //#endregion
@@ -6983,10 +7082,19 @@ const PackageManifestSchema = z.object({
 	* iString: none of those call sites expect a locale-keyed object, and
 	* "the directory name is the package's identity, not a label beside it"
 	* (audit.ts) is exactly the argument against ever making it one. A
-	* human-facing translated title belongs in `description` below, not here.
+	* human-facing translated title belongs in `displayName` below, not here.
 	*/
 	name: z.string().min(1),
 	/**
+	* Optional human-facing card title, shown instead of `name` wherever a
+	* template is presented to a reader (never for identity — routing, install
+	* folder, CLI sort, and SKILL.md frontmatter matching all still key off
+	* `name`, untouched by this field). Absent by default: a package written
+	* before this field, or whose author didn't bother, falls back to `name`
+	* and renders exactly as before — this is additive, not a migration.
+	*/
+	displayName: iStringSchema.optional(),
+	/**
 	* The one-line blurb shown on a template's catalog card, and (via
 	* `layout-read.ts`) the description on the Folder node created at install
 	* time — Folder/node descriptions are already iString-capable text
@@ -10221,6 +10329,7 @@ const discoverPackages = (files) => {
 			subdir,
 			name: tree.manifest.name,
 			description: tree.manifest.description,
+			...tree.manifest.displayName ? { displayName: tree.manifest.displayName } : {},
 			isTemplate: validation.ok,
 			templateErrors: validation.ok || !claimsTemplate ? [] : validation.errors,
 			counts: countTree(tree),
@@ -10448,6 +10557,7 @@ const TEMPLATE_INDEX_FORMAT = "busabase-template-index@1";
 const toEntry = (found, airapps) => ({
 	subdir: found.subdir,
 	name: found.name,
+	...found.displayName ? { displayName: found.displayName } : {},
 	description: found.description,
 	category: found.category ?? "uncategorized",
 	...found.risk ? { risk: found.risk } : {},
@@ -10523,6 +10633,7 @@ const compact = (value) => Object.fromEntries(Object.entries(value).filter(([, i
 const serializeManifest = (manifest) => toJsonFile(compact({
 	format: manifest.format,
 	name: manifest.name,
+	displayName: manifest.displayName,
 	description: manifest.description || void 0,
 	version: manifest.version,
 	author: manifest.author,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "busa-cli",
-  "version": "0.51.0",
+  "version": "0.52.0",
   "description": "Command-line client for the Busabase OpenAPI REST API. Talks to a local or remote `busabase server`. Short-name alias for busabase-cli.",
   "license": "MIT",
   "homepage": "https://github.com/busabase/busabase/tree/main/apps/busabase-cli",
@@ -33,7 +33,7 @@
   "dependencies": {
     "@zip.js/zip.js": "^2.8.20",
     "busabase-package": "workspace:*",
-    "busabase-sdk": "0.51.0",
+    "busabase-sdk": "0.52.0",
     "commander": "^14.0.0",
     "qrcode": "^1.5.4",
     "tar-stream": "^3.1.7",