@vincentt-xr/cli 0.15.1 → 0.15.2

package/dist/cli/index.js

@@ -21,6 +21,7 @@ import { ensureLoggedIn } from "../login/ensure.js";
 import { clearMachineToken, loadMachineConfig, locateProjectBinding, resolveConfig, writeDxEnablement, MACHINE_CONFIG_PATH, } from "../shared/config.js";
 import { fileLocationStore } from "../shared/locationStore.js";
 import { getMe, listProjects, ProjectUnreachableError } from "../lib/backend.js";
+import { EXIT } from "../lib/exitCodes.js";
 import { listProjectsCommand } from "../lib/listProjects.js";
 import { initCommand } from "../lib/initCommand.js";
 import { linkCommand, printUnreachable, unlinkCommand } from "../lib/projects.js";
@@ -813,7 +814,7 @@ async function feedbackWait(cwd, timeoutMs, verbose) {
                 break;
             for (const l of feedbackCopy.windowElapsed(timeoutMs))
                 console.error(l);
-            return 64;
+            return EXIT.WINDOW_ELAPSED;
         }
         if (result.kind === "closed") {
             if (handed > 0)
@@ -825,7 +826,7 @@ async function feedbackWait(cwd, timeoutMs, verbose) {
                     console.error(l);
                 }
             }
-            return 65;
+            return EXIT.CHANNEL_ENDED;
         }
         await handOver(cwd, result.annotation);
         after = result.annotation.seq;
@@ -882,7 +883,7 @@ async function feedbackWatch(cwd, timeoutMs, verbose) {
                     console.error(l);
                 }
             }
-            return 65;
+            return EXIT.CHANNEL_ENDED;
         }
         const a = result.annotation;
         await handOver(cwd, a);
@@ -1110,7 +1111,7 @@ async function assets() {
         for (const l of unreachableLines("auth", 'vincentt assets search "<what you want>"')) {
             console.error(l);
         }
-        process.exitCode = 4;
+        process.exitCode = EXIT.NOT_AUTHENTICATED;
         return;
     }
     const deps = {

package/dist/dx/clusters.js

@@ -15,6 +15,7 @@
  * Detection is deliberately loose. A false positive costs a line in a report a
  * human is already reading; a miss costs a DX gap nobody ever hears about.
  */
+import { EXIT } from "../lib/exitCodes.js";
 import { verbOf } from "./skeleton.js";
 /**
  * Non-zero exits a verb DEFINES as ordinary outcomes, not failures.
@@ -26,11 +27,19 @@ import { verbOf } from "./skeleton.js";
  * fight-with-the-tool that never happened, and the nudge narrates the intended
  * loop as friction — on stderr, which in this loop is a caller's own input.
  *
+ * `create` and `init` exit 3 when the scaffold landed but no account was on the
+ * machine to reserve an address. The creator holds a working project, so
+ * counting it as a fault would make the nudge narrate a delivered project as a
+ * fight with the tool. `LOOPING_VERBS` deliberately does NOT gain these two:
+ * re-running create without signing in first IS real repetition.
+ *
  * A verb's neutral exits are ITS OWN. Nothing here generalizes 64/65 across the
  * CLI; another verb adding a non-zero-but-fine exit adds a row.
  */
 const NEUTRAL_EXITS = {
-    feedback: new Set([64, 65]),
+    feedback: new Set([EXIT.WINDOW_ELAPSED, EXIT.CHANNEL_ENDED]),
+    create: new Set([EXIT.DELIVERED_NOT_REGISTERED]),
+    init: new Set([EXIT.DELIVERED_NOT_REGISTERED]),
 };
 /**
  * Verbs whose contract is "run it again". Repeating them is not a caller failing

package/dist/lib/exitCodes.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Every exit code the CLI returns. B-F12-1: APPEND-ONLY — a code may be added,
+ * an existing code's meaning never changes. Removing or repurposing one breaks
+ * grounding that already shipped into a creator's repository and is never read
+ * back, so the platform cannot warn about it or repair it.
+ */
+export declare const EXIT: {
+    /** The command did what it was asked. */
+    readonly OK: 0;
+    /** The command failed and nothing lasting was produced. */
+    readonly FAILED: 1;
+    /**
+     * The project source was written by this invocation, but no address was
+     * reserved: no account on this machine, and no browser to get one. The
+     * creator holds a working local project; only registration is deferred.
+     */
+    readonly DELIVERED_NOT_REGISTERED: 3;
+    /** A platform call needed an account and the command did not answer for it. */
+    readonly NOT_AUTHENTICATED: 4;
+    /** A bounded wait elapsed with nothing handed over. */
+    readonly WINDOW_ELAPSED: 64;
+    /** The channel ended before anything was handed over. */
+    readonly CHANNEL_ENDED: 65;
+};

package/dist/lib/exitCodes.js

@@ -0,0 +1,28 @@
+/**
+ * Every exit code the CLI returns. B-F12-1: APPEND-ONLY — a code may be added,
+ * an existing code's meaning never changes. Removing or repurposing one breaks
+ * grounding that already shipped into a creator's repository and is never read
+ * back, so the platform cannot warn about it or repair it.
+ */
+export const EXIT = {
+    /** The command did what it was asked. */
+    OK: 0,
+    /** The command failed and nothing lasting was produced. */
+    FAILED: 1,
+    /**
+     * The project source was written by this invocation, but no address was
+     * reserved: no account on this machine, and no browser to get one. The
+     * creator holds a working local project; only registration is deferred.
+     */
+    DELIVERED_NOT_REGISTERED: 3,
+    /** A platform call needed an account and the command did not answer for it. */
+    NOT_AUTHENTICATED: 4,
+    /** A bounded wait elapsed with nothing handed over. */
+    WINDOW_ELAPSED: 64,
+    /** The channel ended before anything was handed over. */
+    CHANNEL_ENDED: 65,
+};
+// 2 is deliberately unallocated. Shells conventionally use 2 for misuse of a
+// builtin, and a CLI that returns it invites a caller to misread a shell error
+// as ours. The gap is recorded here so a later change does not fill it by
+// looking for the lowest free number.

package/dist/lib/initFrom.js

@@ -10,6 +10,7 @@
 // be wrong.
 import path from "node:path";
 import { SlugTakenError } from "./backend.js";
+import { EXIT } from "./exitCodes.js";
 import { slugify } from "./slug.js";
 import { formatSourceRef } from "./sourceRef.js";
 /** How many times the address prompt re-asks before falling back to the printed block. */
@@ -70,7 +71,9 @@ export async function registerAfterScaffold(deps, opts = {}) {
     if (!token) {
         if (!deps.ask) {
             deps.print("");
-            deps.print("✗ Not signed in, and signing in needs a browser.");
+            deps.print(scaffolded
+                ? "→ Not signed in, so no address is reserved yet."
+                : "✗ Not signed in, and signing in needs a browser.");
             deps.print("");
             if (scaffolded)
                 reassure(deps.print, deps.dirLabel);
@@ -81,9 +84,16 @@ export async function registerAfterScaffold(deps, opts = {}) {
             deps.print("");
             deps.print("  Then finish:");
             resume(deps.print, deps.dirLabel, resumeCommand, needsCd);
+            if (scaffolded) {
+                deps.print("");
+                // B-F2-9(a): the open route must be named in something the creator's
+                // agent reads. On a non-TTY this output IS that artifact — the TTY
+                // branch's login prompt, which carries the same line, is unreachable here.
+                deps.print("  Or don't — the local dev loop needs no account.");
+            }
             deps.print("");
             deps.print(closing);
-            return 1;
+            return scaffolded ? EXIT.DELIVERED_NOT_REGISTERED : EXIT.FAILED;
         }
         deps.print("");
         deps.print("→ Now registering the project so it gets an address.");

package/dist/lib/lifecycle.d.ts

@@ -36,7 +36,11 @@ export interface CreateOptions {
 export interface CreateResult {
     /** The full transcript, exactly as the CLI prints it. */
     message: string;
-    /** Non-zero when nothing was created (already linked, address taken, no org). */
+    /**
+     * Non-zero when the address was not reserved. Code 1 means nothing was created
+     * (already linked, address taken, no org); code 3 means the project source was
+     * written but registration was deferred for want of an account.
+     */
     exitCode: number;
 }
 /**

package/dist/lib/projects.js

@@ -8,6 +8,7 @@ import { promises as fs } from "node:fs";
 import path from "node:path";
 import { BackendError, SlugTakenError, createProject, getProject, } from "./backend.js";
 import { classifyTarget } from "./classifyTarget.js";
+import { EXIT } from "./exitCodes.js";
 import { registerAfterScaffold } from "./initFrom.js";
 import { slugify } from "./slug.js";
 import { detectPackageManager, installDependencies, scaffoldFromTemplate, } from "../scaffold/index.js";
@@ -169,7 +170,10 @@ export async function createCommand(deps, opts = {}) {
             name: opts.name ?? path.basename(target),
             slug: opts.slug ?? slugify(opts.name ?? path.basename(target)),
         });
-    if (code !== 0)
+    // The gate is DELIVERY, not zero. A run that wrote the source and deferred
+    // only the address still delivered a project, so it still prints how to run
+    // it. The endings that stay exit 1 delivered nothing and correctly skip this.
+    if (code !== 0 && code !== EXIT.DELIVERED_NOT_REGISTERED)
         return code;
     if (scaffolded) {
         // This reader has never run the dev server, so the block that fits is the one
@@ -187,7 +191,10 @@ export async function createCommand(deps, opts = {}) {
         for (const line of alignedSteps(steps))
             deps.print(line);
     }
-    return 0;
+    // Printing the steps must not launder the ending: a delivered-not-registered
+    // run still reports 3, so a caller never reads success for an unregistered
+    // project.
+    return code;
 }
 /** Two columns, aligned on the widest command, so the block reads as a table. */
 function alignedSteps(steps) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vincentt-xr/cli",
-  "version": "0.15.1",
+  "version": "0.15.2",
   "description": "Vincentt CLI — the vincentt command-line dev loop (login, create, publish, preview, diagnostics, feedback, relay)",
   "license": "UNLICENSED",
   "type": "module",