busa-cli 0.16.1 → 0.16.2

package/dist/{chunk-FCF6L4Y2.js → chunk-AHIRHKUB.js}

@@ -1049,7 +1049,13 @@ var htmlNodeType = {
   label: "HTML",
   icon: "code-xml",
   capabilities: { hasDetail: true, creatable: true },
-  operations: []
+  operations: [
+    {
+      kind: "html_document_update",
+      label: "Update HTML",
+      tone: "border-blue-200 bg-blue-50 text-blue-800"
+    }
+  ]
 };
 
 // ../../packages/busabase-contract/src/domains/skill/definition.ts
@@ -1068,7 +1074,13 @@ var whiteboardNodeType = {
   label: "Whiteboard",
   icon: "pen-tool",
   capabilities: { hasDetail: true, creatable: true },
-  operations: []
+  operations: [
+    {
+      kind: "whiteboard_document_update",
+      label: "Update whiteboard",
+      tone: "border-blue-200 bg-blue-50 text-blue-800"
+    }
+  ]
 };
 
 // ../../packages/busabase-contract/src/domains/workflow/definition.ts
@@ -1077,7 +1089,13 @@ var workflowNodeType = {
   label: "Workflow",
   icon: "workflow",
   capabilities: { hasDetail: true, creatable: true },
-  operations: []
+  operations: [
+    {
+      kind: "workflow_document_update",
+      label: "Update workflow",
+      tone: "border-blue-200 bg-blue-50 text-blue-800"
+    }
+  ]
 };
 
 // ../../packages/busabase-contract/src/domains/registry.ts
@@ -3564,12 +3582,8 @@ var nodeSchema = z8.lazy(
     slug: z8.string(),
     name: z8.string(),
     description: z8.string(),
-    metadata: z8.object({
-      entryFile: z8.string().optional(),
-      visibility: z8.enum(["private", "workspace", "public"]).optional(),
-      version: z8.string().optional(),
-      assetId: z8.string().optional()
-    }).catchall(z8.unknown()).default({}),
+    metadata: z8.object({ version: z8.string().optional() }).catchall(z8.unknown()).default({}),
+    explicitVisibility: z8.enum(["private", "workspace", "public"]).nullable().default(null),
     position: z8.number(),
     createdAt: z8.string(),
     updatedAt: z8.string(),
@@ -3676,7 +3690,17 @@ var commitSchema = z8.object({
   nodeId: z8.string().nullable(),
   operationId: z8.string().nullable(),
   parentCommitId: z8.string().nullable(),
-  fields: z8.record(z8.string(), z8.unknown()),
+  // Deliberately loose (`z.record`), NOT a discriminated union keyed on `operation`.
+  //
+  // Do not "tighten" this. Commits already in the database were written before
+  // per-operation payload validation existed, so their shapes carry no guarantee.
+  // The write path (`insertCommit`) and the merge path (`parseCommitPayload`) are
+  // strict precisely because they only ever touch freshly-written payloads; this VO
+  // is also used to render *history* and approval detail pages, which read arbitrarily
+  // old commits. Making it strict would turn any legacy-shaped row into a 500 on a
+  // read-only screen. This is not a compatibility shim — it is the requirement not to
+  // break reading data that already exists.
+  payload: z8.record(z8.string(), z8.unknown()),
   operation: z8.enum(OPERATION_KINDS),
   message: z8.string(),
   author: z8.string(),
@@ -4216,7 +4240,7 @@ var PACKAGE_COMPUTED_FIELD_TYPES = [
   "auto_number",
   "formula",
   // `lookup` is derived from OTHER records and resolved at read time, so it
-  // shows up in `headCommit.fields` like any other value — but exporting it
+  // shows up in `headCommit.payload` like any other value — but exporting it
   // would ship derived data as if it were authored, and install would strip it
   // server-side anyway.
   "lookup",
@@ -4542,14 +4566,18 @@ var applyInstall = async (client, plan, options) => {
     options.serverUrl
   );
   progress("Pass 1/5: creating the node tree\u2026");
-  state.targetFolderNodeId = await createFolderNode(client, {
-    parentNodeId: void 0,
-    slug: plan.targetFolderSlug,
-    name: plan.tree.manifest.name,
-    description: plan.tree.manifest.description,
-    submittedBy: options.submittedBy
-  });
-  state.created.folders++;
+  if (plan.existingTargetFolderNodeId) {
+    state.targetFolderNodeId = plan.existingTargetFolderNodeId;
+  } else {
+    state.targetFolderNodeId = await createFolderNode(client, {
+      parentNodeId: void 0,
+      slug: plan.targetFolderSlug,
+      name: plan.tree.manifest.name,
+      description: plan.tree.manifest.description,
+      submittedBy: options.submittedBy
+    });
+    state.created.folders++;
+  }
   await createStructure(
     client,
     plan.tree.nodes,
@@ -5293,7 +5321,7 @@ var collectRecords = async (client, baseId, fields, options) => {
     for (const record of page.records) {
       if (record.status !== "active") continue;
       const values = {};
-      for (const [slug, value] of Object.entries(record.headCommit.fields ?? {})) {
+      for (const [slug, value] of Object.entries(record.headCommit.payload ?? {})) {
         const type = typeBySlug.get(slug);
         if (!type) continue;
         if (PACKAGE_COMPUTED_FIELD_TYPES.includes(type)) continue;
@@ -6059,27 +6087,39 @@ var writePackageFiles = async (files, outDir, options = {}) => {
 };
 
 // ../../packages/busabase-package/src/plan.ts
+var flattenExistingNodes = (nodes) => {
+  const result = [];
+  const visit = (node) => {
+    result.push(node);
+    for (const child of node.children ?? []) visit(child);
+  };
+  for (const node of nodes) visit(node);
+  return result;
+};
 var findTargetFolder = (nodes, slug) => {
-  const roots = nodes.length === 1 && nodes[0].children ? nodes[0].children : nodes;
-  return roots.find((node) => node.slug === slug && node.type === "folder");
+  return flattenExistingNodes(nodes).find((node) => node.slug === slug && node.type === "folder");
 };
 var resolveTargetState = async (client, targetFolderSlug) => {
-  const [nodes, bases] = await Promise.all([client.nodes.list({}), client.bases.list({})]);
+  const listedNodes = await client.nodes.list({});
+  const slugsByType = /* @__PURE__ */ new Map();
+  for (const node of flattenExistingNodes(listedNodes)) {
+    const slugs = slugsByType.get(node.type) ?? /* @__PURE__ */ new Set();
+    slugs.add(node.slug);
+    slugsByType.set(node.type, slugs);
+  }
   return {
-    targetFolder: findTargetFolder(nodes, targetFolderSlug),
-    existingBaseSlugs: new Set(bases.map((base) => base.slug))
+    targetFolder: findTargetFolder(listedNodes, targetFolderSlug),
+    existingNodeSlugsByType: slugsByType
   };
 };
 var buildInstallPlan = (tree, target, options = {}) => {
   const targetFolderSlug = options.intoFolder ?? suggestSlug(tree.manifest.name);
   const warnings = [];
-  const existingChildSlugs = new Set(
-    (target.targetFolder?.children ?? []).map((child) => child.slug)
-  );
-  const collisions = findCollisions(tree, targetFolderSlug, existingChildSlugs, target);
+  assertPackageNodeIdentitiesUnique(tree);
+  const collisions = findCollisions(tree, targetFolderSlug, target);
   let planned = tree;
   if (options.rename && collisions.length > 0) {
-    planned = applyRenames(tree, collisions, existingChildSlugs, target.existingBaseSlugs);
+    planned = applyRenames(tree, collisions, target.existingNodeSlugsByType);
   }
   collectWarnings(planned, warnings);
   const counts = countTree(planned);
@@ -6090,6 +6130,7 @@ var buildInstallPlan = (tree, target, options = {}) => {
   }
   return {
     targetFolderSlug,
+    existingTargetFolderNodeId: target.targetFolder?.id,
     tree: planned,
     collisions,
     warnings,
@@ -6107,24 +6148,33 @@ var hasUnlinkableRelationValues = (node) => {
     })
   );
 };
-var findCollisions = (tree, targetFolderSlug, existingChildSlugs, target) => {
+var findCollisions = (tree, targetFolderSlug, target) => {
   const collisions = [];
-  for (const node of tree.nodes) {
-    if (existingChildSlugs.has(node.slug)) {
-      collisions.push({ kind: "node", slug: node.slug, path: `${targetFolderSlug}/${node.slug}` });
-    }
-  }
-  for (const baseNode of collectBaseNodes(tree.nodes)) {
-    if (target.existingBaseSlugs.has(baseNode.slug)) {
+  for (const node of walkNodes(tree.nodes)) {
+    if (target.existingNodeSlugsByType.get(node.type)?.has(node.slug)) {
       collisions.push({
-        kind: "base",
-        slug: baseNode.slug,
-        path: `${targetFolderSlug}/\u2026 /${baseNode.slug} (base slugs are unique per space)`
+        kind: node.type === "base" ? "base" : "node",
+        nodeType: node.type,
+        slug: node.slug,
+        path: `${targetFolderSlug}/\u2026/${node.slug} (${node.type} slugs are unique per space)`
       });
     }
   }
   return collisions;
 };
+var nodeIdentityKey = (node) => `${node.type}\0${node.slug}`;
+var assertPackageNodeIdentitiesUnique = (tree) => {
+  const seen = /* @__PURE__ */ new Set();
+  for (const node of walkNodes(tree.nodes)) {
+    const key = nodeIdentityKey(node);
+    if (seen.has(key)) {
+      throw new Error(
+        `Package contains duplicate ${node.type} slug "${node.slug}". Node slugs must be unique by type within a space.`
+      );
+    }
+    seen.add(key);
+  }
+};
 var nextFreeSlug = (slug, taken) => {
   for (let suffix = 2; suffix < 1e3; suffix++) {
     const candidate = `${slug}-${suffix}`;
@@ -6132,38 +6182,42 @@ var nextFreeSlug = (slug, taken) => {
   }
   throw new Error(`Could not find a free slug for "${slug}" after 999 attempts.`);
 };
-var applyRenames = (tree, collisions, existingChildSlugs, existingBaseSlugs) => {
-  const packageBaseSlugs = new Set(collectBaseNodes(tree.nodes).map((node) => node.slug));
-  const takenNodeSlugs = /* @__PURE__ */ new Set([...existingChildSlugs, ...tree.nodes.map((node) => node.slug)]);
-  const takenBaseSlugs = /* @__PURE__ */ new Set([...existingBaseSlugs, ...packageBaseSlugs]);
+var applyRenames = (tree, collisions, existingNodeSlugsByType) => {
+  const takenByType = /* @__PURE__ */ new Map();
+  for (const [type, slugs] of existingNodeSlugsByType) {
+    takenByType.set(type, new Set(slugs));
+  }
+  for (const node of walkNodes(tree.nodes)) {
+    const taken = takenByType.get(node.type) ?? /* @__PURE__ */ new Set();
+    taken.add(node.slug);
+    takenByType.set(node.type, taken);
+  }
   const nodeRenames = /* @__PURE__ */ new Map();
   const baseRenames = /* @__PURE__ */ new Map();
   for (const collision of collisions) {
-    if (collision.kind === "node") {
-      const renamed = nextFreeSlug(collision.slug, takenNodeSlugs);
-      takenNodeSlugs.add(renamed);
-      nodeRenames.set(collision.slug, renamed);
-      collision.renamedTo = renamed;
-    } else {
-      const renamed = nextFreeSlug(collision.slug, takenBaseSlugs);
-      takenBaseSlugs.add(renamed);
-      baseRenames.set(collision.slug, renamed);
-      collision.renamedTo = renamed;
-    }
-  }
-  const rewriteNode = (node, isTopLevel) => {
+    const node = [...walkNodes(tree.nodes)].find(
+      (candidate) => candidate.type === collision.nodeType && candidate.slug === collision.slug
+    );
+    if (!node) continue;
+    const taken = takenByType.get(node.type) ?? /* @__PURE__ */ new Set();
+    const renamed = nextFreeSlug(collision.slug, taken);
+    taken.add(renamed);
+    takenByType.set(node.type, taken);
+    nodeRenames.set(nodeIdentityKey(node), renamed);
+    if (node.type === "base") baseRenames.set(node.slug, renamed);
+    collision.renamedTo = renamed;
+  }
+  const rewriteNode = (node) => {
+    const slug = nodeRenames.get(nodeIdentityKey(node)) ?? node.slug;
     if (node.type === "base") {
-      const renamed = baseRenames.get(node.slug) ?? (isTopLevel ? nodeRenames.get(node.slug) : void 0);
-      const slug2 = renamed ?? node.slug;
       return {
         ...node,
-        slug: slug2,
+        slug,
         base: { ...node.base, fields: node.base.fields.map(rewriteFieldTarget) }
       };
     }
-    const slug = (isTopLevel ? nodeRenames.get(node.slug) : void 0) ?? node.slug;
     if (node.type === "folder") {
-      return { ...node, slug, children: node.children.map((child) => rewriteNode(child, false)) };
+      return { ...node, slug, children: node.children.map(rewriteNode) };
     }
     return { ...node, slug };
   };
@@ -6174,7 +6228,7 @@ var applyRenames = (tree, collisions, existingChildSlugs, existingBaseSlugs) =>
     if (!renamed) return field;
     return { ...field, options: { ...field.options, targetBaseSlug: renamed } };
   };
-  return { ...tree, nodes: tree.nodes.map((node) => rewriteNode(node, true)) };
+  return { ...tree, nodes: tree.nodes.map(rewriteNode) };
 };
 var collectWarnings = (tree, warnings) => {
   for (const baseNode of collectBaseNodes(tree.nodes)) {

package/dist/cli.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import {
   runCli
-} from "./chunk-FCF6L4Y2.js";
+} from "./chunk-AHIRHKUB.js";
 
 // src/cli.ts
 process.exit(await runCli(process.argv.slice(2)));

package/dist/index.js

@@ -2,7 +2,7 @@ import {
   HELP,
   render,
   runCli
-} from "./chunk-FCF6L4Y2.js";
+} from "./chunk-AHIRHKUB.js";
 
 // src/index.ts
 import {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "busa-cli",
-  "version": "0.16.1",
+  "version": "0.16.2",
   "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.16.1",
+    "busabase-sdk": "0.16.2",
     "commander": "^14.0.0",
     "tar-stream": "^3.1.7",
     "yaml": "^2.9.0",