omniharness-cli 0.1.53 → 0.1.55

package/dist/cli.js

@@ -24,6 +24,15 @@ function dumpTranscript(messages) {
     }
     process.stdout.write('\n');
 }
+// A crash anywhere below would otherwise surface as a raw Node stack trace, or
+// as an unhandled rejection that terminates the process without saying why.
+// A CLI should fail with a sentence.
+function die(error) {
+    console.error(`omniharness: ${error instanceof Error ? error.message : String(error)}`);
+    process.exit(1);
+}
+process.on('unhandledRejection', die);
+process.on('uncaughtException', die);
 const command = process.argv[2];
 if (command === 'update') {
     process.exitCode = runUpdate();
@@ -43,15 +52,31 @@ else if (command === 'doctor') {
 }
 else if (command === 'models') {
     void (async () => {
+        const client = new OmniRouteClient();
         try {
-            console.log((await models(new OmniRouteClient())).join('\n'));
+            console.log((await models(client)).join('\n'));
         }
         catch (error) {
-            console.error(`omniharness models: ${error instanceof Error ? error.message : String(error)}`);
+            // "fetch failed" on its own tells the user nothing they can act on.
+            // Name the endpoint that was tried and what to check, the way doctor
+            // does — a connection failure here almost always means the gateway is
+            // not running or OMNIROUTE_URL points somewhere else.
+            const reason = error instanceof Error ? error.message : String(error);
+            console.error(`omniharness models: ${reason}`);
+            console.error(`  endpoint: ${client.endpoint}`);
+            console.error('  check that OmniRoute is running, and that OMNIROUTE_URL points at it');
+            console.error("  run 'omniharness doctor' for a full connection report");
             process.exitCode = 1;
         }
     })();
 }
+else if (command !== undefined) {
+    // Anything else is a mistake, not an invitation to open the TUI: `omniharness
+    // doctr` used to start an interactive session instead of naming the typo.
+    console.error(`omniharness: unknown command '${command}'`);
+    console.error("run 'omniharness --help' for usage");
+    process.exitCode = 1;
+}
 else {
     void (async () => {
         const saved = await readActiveCombo();

package/dist/config/omniRoute.js

@@ -7,6 +7,22 @@ export class OmniRouteError extends Error {
     }
 }
 const DEFAULT_ENDPOINT = 'http://localhost:20128';
+/**
+ * First value that is actually set, treating blank strings as unset.
+ *
+ * `??` only falls back on null/undefined, so `OMNIROUTE_URL=` — routine in a
+ * .env file, a docker-compose entry, or CI where a variable is declared but
+ * left empty — produced an empty endpoint and every request failed with
+ * "Failed to parse URL". The Go side has always treated empty as unset; this
+ * matches it.
+ */
+function firstSet(...values) {
+    for (const value of values) {
+        if (value !== undefined && value.trim() !== '')
+            return value.trim();
+    }
+    return undefined;
+}
 export class OmniRouteClient {
     endpoint;
     timeoutMs;
@@ -19,7 +35,7 @@ export class OmniRouteClient {
         requestCount: 0,
     };
     constructor(config = {}) {
-        this.endpoint = (config.endpoint ?? process.env.OMNIROUTE_URL ?? DEFAULT_ENDPOINT).replace(/\/$/, '');
+        this.endpoint = (firstSet(config.endpoint, process.env.OMNIROUTE_URL) ?? DEFAULT_ENDPOINT).replace(/\/$/, '');
         this.apiKey = config.apiKey ?? process.env.OMNIROUTE_API_KEY ?? '';
         this.mgmtToken = config.mgmtToken ?? process.env.OMNIROUTE_MGMT_TOKEN ?? '';
         this.timeoutMs = config.timeoutMs ?? 120_000;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omniharness-cli",
-  "version": "0.1.53",
+  "version": "0.1.55",
   "description": "OmniHarness — local-first agent orchestration harness for OmniRoute.",
   "license": "MIT",
   "type": "module",