@notis_ai/cli 0.2.0-beta.16.1 → 0.2.0-beta.160.1

package/src/runtime/output.js

@@ -22,6 +22,13 @@ function formatTable(rows, columns) {
   return [header, divider, ...body].join('\n');
 }
 
+function formatHint(hint) {
+  if (typeof hint?.message === 'string' && hint.message) {
+    return `  ${hint.message}`;
+  }
+  return `  ${hint?.command || ''}  ${hint?.reason || ''}`.trimEnd();
+}
+
 function yamlScalar(value) {
   if (value === null || value === undefined) {
     return 'null';
@@ -94,7 +101,30 @@ export class OutputManager {
     process.stderr.write(`${message}\n`);
   }
 
+  note(message) {
+    if (this.isMachineMode()) {
+      return;
+    }
+    process.stderr.write(`${message}\n`);
+  }
+
+  notice(message) {
+    // Some pre-result instructions are required to make progress. They belong
+    // on stderr so stdout stays one valid machine envelope, but unlike ordinary
+    // table notes they must remain visible in JSON/YAML/NDJSON modes.
+    process.stderr.write(`${message}\n`);
+  }
+
+  emitProgress({ phase, message, requestId = null }) {
+    if (this.runtime.quiet) {
+      return;
+    }
+    const suffix = requestId ? ` (${requestId})` : '';
+    process.stderr.write(`[${phase}] ${message}${suffix}\n`);
+  }
+
   emitSuccess({
+    ok = true,
     command,
     data = {},
     humanSummary,
@@ -105,7 +135,7 @@ export class OutputManager {
     renderHuman,
   }) {
     const envelope = {
-      ok: true,
+      ok: Boolean(ok),
       command,
       data,
       human_summary: humanSummary,
@@ -115,7 +145,10 @@ export class OutputManager {
       meta: {
         profile: this.runtime.profileName,
         api_base: this.runtime.apiBase,
-        mutating: Boolean(meta.mutating),
+        mutating:
+          meta.mutating === true
+            ? true
+            : (meta.mutating === false ? false : null),
         ...meta,
       },
     };
@@ -141,7 +174,7 @@ export class OutputManager {
     }
 
     if (hints.length) {
-      const lines = hints.map((hint) => `  ${hint.command}  ${hint.reason}`);
+      const lines = hints.map(formatHint);
       process.stdout.write(`\nNext:\n${lines.join('\n')}\n`);
     }
 
@@ -170,11 +203,10 @@ export class OutputManager {
 
     process.stderr.write(`Error: ${error.message}\n`);
     for (const hint of error.hints || []) {
-      process.stderr.write(`  ${hint.command}  ${hint.reason}\n`);
+      process.stderr.write(`${formatHint(hint)}\n`);
     }
     return error.exitCode || EXIT_CODES.unexpected;
   }
 }
 
 export { formatTable };
-

package/src/runtime/ports.js

@@ -0,0 +1,31 @@
+import { createServer } from 'node:http';
+
+export async function getAvailablePortPreferring(preferredPort) {
+  if (Number.isInteger(preferredPort) && preferredPort > 0 && preferredPort <= 65535) {
+    const preferredIsFree = await new Promise((resolvePromise) => {
+      const server = createServer();
+      server.once('error', () => resolvePromise(false));
+      server.listen(preferredPort, '127.0.0.1', () => {
+        server.close(() => resolvePromise(true));
+      });
+    });
+    if (preferredIsFree) {
+      return preferredPort;
+    }
+  }
+  return getAvailablePort();
+}
+
+export async function getAvailablePort() {
+  const server = createServer();
+  await new Promise((resolvePromise, rejectPromise) => {
+    server.once('error', rejectPromise);
+    server.listen(0, '127.0.0.1', () => {
+      server.off('error', rejectPromise);
+      resolvePromise();
+    });
+  });
+  const { port } = server.address();
+  await new Promise((resolvePromise) => server.close(resolvePromise));
+  return port;
+}