@memfork/cli 0.1.52 → 0.1.54

package/dist/cli.js

@@ -182,9 +182,29 @@ program.parseAsync(process.argv).catch((e) => {
     process.exit(1);
 });
 // ─── Helpers ──────────────────────────────────────────────────────────────────
+/**
+ * Flush stdout/stderr, then exit with `code`.
+ *
+ * Commands like `commit` open keep-alive sockets to the relayer / Sui RPC that
+ * hold Node's event loop open long after the work is done — so the process
+ * would otherwise hang for ~minutes instead of returning. Agents (Codex) read
+ * this as a stuck command and retry, producing duplicate commits. We exit
+ * explicitly once the command resolves.
+ *
+ * We can't use a bare `process.exit()` because it truncates buffered output
+ * when stdout is a pipe (which it always is under an agent). Writing an empty
+ * chunk with a callback guarantees the prior writes have drained first.
+ */
+function flushAndExit(code) {
+    let pending = 2;
+    const done = () => { if (--pending === 0)
+        process.exit(code); };
+    process.stdout.write("", done);
+    process.stderr.write("", done);
+}
 function wrap(fn) {
     return (...args) => {
-        fn(...args).catch((e) => {
+        fn(...args).then(() => flushAndExit(0), (e) => {
             if (e.name === "ConfigError") {
                 console.error(chalk.red("\n  " + String(e.message)));
                 console.error(chalk.cyan("  → Run `memfork init` to configure.\n"));
@@ -192,7 +212,7 @@ function wrap(fn) {
             else {
                 console.error(chalk.red("\nError: " + String(e)));
             }
-            process.exit(1);
+            flushAndExit(1);
         });
     };
 }

package/dist/commands/ops.js

@@ -161,25 +161,26 @@ export async function cmdCommit(opts) {
         ...(artifacts.length > 0 ? { artifacts } : {}),
     });
     const out = { blobId, branch, artifacts: refs };
-    if (process.stdout.isTTY) {
+    // Always print the human-readable confirmation so agents and CI can recognise
+    // success without parsing JSON. In non-TTY contexts (subprocesses, pipes) we
+    // additionally emit the JSON on the following line for machine consumption.
+    console.log("");
+    console.log(chalk.green("✓") + " Committed to " + chalk.bold(branch));
+    console.log(chalk.dim(`  blob: ${blobId}`));
+    if (refs.length > 0) {
         console.log("");
-        console.log(chalk.green("✓") + " Committed to " + chalk.bold(branch));
-        console.log(chalk.dim(`  blob: ${blobId}`));
-        if (refs.length > 0) {
-            console.log("");
-            console.log(chalk.bold("  Artifacts:"));
-            for (const ref of refs) {
-                console.log(`    ${chalk.cyan(ref.path)}  ` +
-                    chalk.dim(`${(ref.size / 1024).toFixed(1)} KiB  `) +
-                    chalk.dim(`blob: ${ref.blobId.slice(0, 16)}…  `) +
-                    chalk.dim(`sha256: ${ref.sha256.slice(0, 16)}…`));
-                console.log(chalk.dim(`    Retrieve with: `) +
-                    chalk.white(`memfork cat ${ref.blobId} --output ${ref.path} --sha256 ${ref.sha256}`));
-            }
+        console.log(chalk.bold("  Artifacts:"));
+        for (const ref of refs) {
+            console.log(`    ${chalk.cyan(ref.path)}  ` +
+                chalk.dim(`${(ref.size / 1024).toFixed(1)} KiB  `) +
+                chalk.dim(`blob: ${ref.blobId.slice(0, 16)}…  `) +
+                chalk.dim(`sha256: ${ref.sha256.slice(0, 16)}…`));
+            console.log(chalk.dim(`    Retrieve with: `) +
+                chalk.white(`memfork cat ${ref.blobId} --output ${ref.path} --sha256 ${ref.sha256}`));
         }
-        console.log("");
     }
-    else {
+    console.log("");
+    if (!process.stdout.isTTY) {
         console.log(JSON.stringify(out));
     }
 }
@@ -695,12 +696,29 @@ export async function cmdBranch(name, opts = {}) {
     const { client, cfg } = await getClient();
     const from = resolveBranch({ explicit: opts.from, configDefault: cfg.defaultBranch });
     process.stdout.write(chalk.dim(`Creating branch ${chalk.green(name)} from ${chalk.green(from)} …  `));
-    const digest = await client.branch(name, { from });
-    console.log(chalk.green("done"));
-    console.log("");
-    console.log(chalk.dim(`  tx: ${digest}`));
-    console.log(chalk.dim(`  Run ${chalk.white("memfork checkout " + name)} to switch to it.`));
-    console.log("");
+    try {
+        const digest = await client.branch(name, { from });
+        console.log(chalk.green("done"));
+        console.log("");
+        console.log(chalk.dim(`  tx: ${digest}`));
+        console.log(chalk.dim(`  Run ${chalk.white("memfork checkout " + name)} to switch to it.`));
+        console.log("");
+    }
+    catch (err) {
+        // E_BRANCH_EXISTS (Move abort code 7) — treat as success so `memfork branch`
+        // is safe to run idempotently in scripts and runbooks.
+        const msg = err instanceof Error ? err.message : String(err);
+        if (msg.includes("MoveAbort") && msg.match(/,\s*7\)/)) {
+            console.log(chalk.dim("already exists"));
+            console.log("");
+            console.log(chalk.dim(`  Branch "${name}" is already on this tree — no action needed.`));
+            console.log(chalk.dim(`  Run ${chalk.white("memfork checkout " + name)} to switch to it.`));
+            console.log("");
+        }
+        else {
+            throw err;
+        }
+    }
 }
 // ─── checkout ─────────────────────────────────────────────────────────────────
 export async function cmdCheckout(name, opts = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memfork/cli",
-  "version": "0.1.52",
+  "version": "0.1.54",
   "description": "MemForks CLI — init, commit, recall, merge, install plugins",
   "repository": {
     "type": "git",