@bpmnkit/cli 0.0.37 → 0.1.0

package/README.md

@@ -9,7 +9,7 @@
   [![ai-assisted](https://img.shields.io/badge/AI--assisted-claude-8b5cf6?style=flat-square)](https://github.com/bpmnkit/monorepo)
   [![experimental](https://img.shields.io/badge/status-experimental-f59e0b?style=flat-square)](https://github.com/bpmnkit/monorepo)
 
-  [Website](https://bpmnkit.com) · [Documentation](https://docs.bpmnkit.com) · [GitHub](https://github.com/bpmnkit/monorepo) · [Changelog](https://github.com/bpmnkit/monorepo/blob/main/apps/cli/CHANGELOG.md)
+  [Website](https://bpmnkit.com) · [Documentation](https://bpmnkit.com/docs) · [GitHub](https://github.com/bpmnkit/monorepo) · [Changelog](https://github.com/bpmnkit/monorepo/blob/main/apps/cli/CHANGELOG.md)
 </div>
 
 ---

package/dist/commands/generate.js

@@ -1,5 +1,6 @@
 import { readFile, writeFile } from "node:fs/promises";
-import { Bpmn, compactify, expand } from "@bpmnkit/core";
+import { resolve } from "node:path";
+import { Bpmn, applyAutoLayout, applyBpmnOperations, compactify, expand } from "@bpmnkit/core";
 // ── JSON schema reference ─────────────────────────────────────────────────────
 const SCHEMA_HELP = `CompactDiagram JSON schema — for --definition and stdin input
 ==============================================================
@@ -455,6 +456,35 @@ async function readStdin() {
     }
     return Buffer.concat(chunks).toString("utf-8").trim();
 }
+// ── Output path guard ─────────────────────────────────────────────────────────
+/**
+ * Decide where `--input` mode writes, refusing to replace the input file unless
+ * the caller asked for it explicitly.
+ *
+ * The patch itself is applied to the full model, so nothing outside it is lost.
+ * Replacing the source is still destructive — the diagram is re-laid out, and a
+ * mistaken patch has nowhere to be compared against — so in-place replacement
+ * has to be asked for by name.
+ *
+ * @param inputFile - The `--input` path.
+ * @param outputFlag - The `--output` path, if any. `-` (stdout) is handled by the caller.
+ * @param force - Whether `--force` was passed.
+ * @returns The path to write to.
+ * @throws If the write would replace `inputFile` and `force` is false.
+ */
+export function resolveModifyOutputPath({ inputFile, outputFlag, force, }) {
+    const output = typeof outputFlag === "string" && outputFlag.length > 0 ? outputFlag : undefined;
+    const inPlace = output === undefined || resolve(output) === resolve(inputFile);
+    if (inPlace && !force) {
+        throw new Error([
+            `Refusing to overwrite ${inputFile}.`,
+            "The patch applies to the full model, so nothing is dropped, but the diagram is " +
+                "re-laid out and the original is gone once it is replaced.",
+            `Write elsewhere with --output <file>, or pass --force to replace ${inputFile} anyway.`,
+        ].join("\n"));
+    }
+    return output ?? inputFile;
+}
 // ── Command ───────────────────────────────────────────────────────────────────
 const generateBpmnCmd = {
     name: "bpmn",
@@ -500,7 +530,9 @@ const generateBpmnCmd = {
         {
             name: "input",
             short: "f",
-            description: "Existing .bpmn file to load and modify",
+            description: "Existing .bpmn file to load and modify. The patch is applied to the full model, so " +
+                "pools, lanes, data wiring and Zeebe detail are preserved; the diagram is re-laid " +
+                "out. Requires --output, or --force to replace it in place.",
             type: "string",
         },
         {
@@ -513,6 +545,11 @@ const generateBpmnCmd = {
             description: "Print the CompactDiagram JSON of --input and exit (for AI inspection of existing files)",
             type: "boolean",
         },
+        {
+            name: "force",
+            description: "Allow --input to be replaced in place. Prefer --output <file>.",
+            type: "boolean",
+        },
     ],
     examples: [
         {
@@ -553,11 +590,11 @@ const generateBpmnCmd = {
         },
         {
             description: "Add a new gateway path to an existing file",
-            command: 'casen generate bpmn --input order.bpmn --patch \'{"elements":[{"id":"notify","type":"serviceTask","name":"Notify","jobType":"notify-worker"},{"id":"end2","type":"endEvent","name":"Notified"}],"flows":[{"id":"fn1","from":"gw","to":"notify","condition":"= urgent"},{"id":"fn2","from":"notify","to":"end2"}]}\'',
+            command: 'casen generate bpmn --input order.bpmn --output order.patched.bpmn --patch \'{"elements":[{"id":"notify","type":"serviceTask","name":"Notify","jobType":"notify-worker"},{"id":"end2","type":"endEvent","name":"Notified"}],"flows":[{"id":"fn1","from":"gw","to":"notify","condition":"= urgent"},{"id":"fn2","from":"notify","to":"end2"}]}\'',
         },
         {
             description: "Pipe a patch from AI output",
-            command: 'echo \'{"elements":[...],"flows":[...]}\' | casen generate bpmn --input order.bpmn',
+            command: 'echo \'{"elements":[...],"flows":[...]}\' | casen generate bpmn --input order.bpmn --output order.patched.bpmn',
         },
         {
             description: "Re-apply auto-layout to an existing file",
@@ -580,12 +617,20 @@ const generateBpmnCmd = {
         if (inputFile) {
             const xml = await readFile(inputFile, "utf-8");
             const defs = Bpmn.parse(xml);
-            const compact = compactify(defs);
             // --dump-compact: print JSON for AI inspection and exit
             if (ctx.flags["dump-compact"]) {
-                process.stdout.write(`${JSON.stringify(compact, null, 2)}\n`);
+                process.stdout.write(`${JSON.stringify(compactify(defs), null, 2)}\n`);
                 return;
             }
+            // Settle the destination before reading stdin or building the patch, so an
+            // unwritable target fails immediately instead of after the work is done.
+            const outputPath = outputFlag === "-"
+                ? null
+                : resolveModifyOutputPath({
+                    inputFile,
+                    outputFlag,
+                    force: ctx.flags.force === true,
+                });
             // Resolve patch from --patch flag or stdin
             let patch = null;
             if (patchFlag) {
@@ -607,22 +652,40 @@ const generateBpmnCmd = {
                     }
                 }
             }
-            // Apply patch to first process (covers all single-process cases)
+            // Apply the patch to the full model, not to the compact view of it: an
+            // element added this way leaves the document's pools, lanes, data
+            // wiring and Zeebe detail exactly where they were.
+            let edited = defs;
             if (patch) {
-                const proc = compact.processes[0];
-                if (!proc)
+                const process = defs.processes[0];
+                if (!process)
                     throw new Error("Input BPMN has no processes");
-                if (patch.elements?.length)
-                    proc.elements.push(...patch.elements);
-                if (patch.flows?.length)
-                    proc.flows.push(...patch.flows);
+                const operations = [
+                    ...(patch.elements ?? []).map((element) => ({ op: "insert", element, parent: process.id })),
+                    ...(patch.flows ?? []).map((flow) => ({
+                        op: "add_flow",
+                        id: flow.id,
+                        parent: process.id,
+                        from: flow.from,
+                        to: flow.to,
+                        name: flow.name,
+                        condition: flow.condition,
+                    })),
+                ];
+                try {
+                    edited = applyBpmnOperations(defs, operations).definitions;
+                }
+                catch (error) {
+                    // Unresolved ids used to be skipped in silence, so a patch naming a
+                    // misspelled element reported success and changed nothing.
+                    throw new Error(`Patch could not be applied: ${error instanceof Error ? error.message : String(error)}`);
+                }
             }
-            const patched = Bpmn.export(expand(compact));
-            if (outputFlag === "-") {
+            const patched = Bpmn.export(applyAutoLayout(edited));
+            if (outputPath === null) {
                 process.stdout.write(patched);
                 return;
             }
-            const outputPath = typeof outputFlag === "string" && outputFlag.length > 0 ? outputFlag : inputFile;
             await writeFile(outputPath, patched, "utf-8");
             ctx.output.ok(patch ? `Patched and written to ${outputPath}` : `Re-laid-out and written to ${outputPath}`);
             return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bpmnkit/cli",
-  "version": "0.0.37",
+  "version": "0.1.0",
   "description": "Command-line interface for Camunda 8 — deploy, manage, and monitor processes from the terminal",
   "type": "module",
   "bin": {
@@ -17,14 +17,14 @@
   },
   "dependencies": {
     "@bpmnkit/api": "0.0.20",
-    "@bpmnkit/ascii": "0.0.30",
+    "@bpmnkit/ascii": "0.0.31",
     "@bpmnkit/connector-gen": "0.0.15",
-    "@bpmnkit/connectors": "0.0.2",
-    "@bpmnkit/core": "0.1.2",
-    "@bpmnkit/engine": "0.1.30",
+    "@bpmnkit/connectors": "0.0.3",
+    "@bpmnkit/core": "0.2.0",
+    "@bpmnkit/engine": "0.1.31",
     "@bpmnkit/patterns": "0.0.5",
     "@bpmnkit/profiles": "0.0.18",
-    "@bpmnkit/proxy": "0.0.33"
+    "@bpmnkit/proxy": "0.1.0"
   },
   "publishConfig": {
     "access": "public"