refactor(commands): route all exits through the central error handler

Commands no longer call process.exit() directly. Every failure now throws
UsageError (bad input → exit 2) or BailianError (runtime failure, with
AUTH/TIMEOUT codes), so the runtime's handleError stays the single exit
point and telemetry always flushes.

- convert 27 process.exit() sites across 15 commands to throws
- move cross-flag/value checks into validate(); use flag `choices` for
  --events / --sort; drop dead --model required check
- keep pipeline's process.exitCode for lint-style soft failures
- enforce with unicorn/no-process-exit, allowed only in runtime/tools/tests
- fix incidental lint: void floating run(), narrow console errorCode,
  align generate-reference type imports to source

Author: XXPermanentXX

packages/cli/src/main.ts

@@ -2,7 +2,7 @@ import { createCli } from "bailian-cli-runtime";
 import { commands } from "./commands.ts";
 import pkg from "../package.json" with { type: "json" };
 
-createCli(commands, {
+void createCli(commands, {
   binName: "bl",
   version: pkg.version,
   clientName: "bailian-cli",

packages/cli/tests/e2e/pipeline.e2e.test.ts

@@ -245,6 +245,6 @@ describe("e2e: pipeline", () => {
     ]);
     expect(exitCode).toBe(2);
     expect(stdout).toBe("");
-    expect(stderr).toMatch(/unsupported --events format: bogus/i);
+    expect(stderr).toMatch(/--events must be one of: jsonl/i);
   });
 });

packages/cli/tests/e2e/quota.e2e.test.ts

@@ -53,7 +53,7 @@ describe("e2e: quota", () => {
 
   test("quota check --period 0 报错最小值", async () => {
     const { stderr, exitCode } = await runCli(["quota", "check", "--period", "0.5"]);
-    expect(exitCode).toBe(1);
+    expect(exitCode).toBe(2);
     expect(stderr).toContain("at least 1 minute");
   });
 });
@@ -184,7 +184,7 @@ describe.skipIf(!isConsoleE2EReady())("e2e: quota（Console）", () => {
       "--tpm",
       "999",
     ]);
-    expect(exitCode).toBe(1);
+    expect(exitCode).toBe(2);
     expect(stderr).toContain("out of range");
     expect(stderr).toContain("Current");
     expect(stderr).toContain("Range");

packages/commands/src/commands/app/call.ts

@@ -1,5 +1,6 @@
 import {
   defineCommand,
+  UsageError,
   appCompletionPath,
   parseSSE,
   detectOutputFormat,
@@ -114,8 +115,7 @@ export default defineCommand({
       try {
         body.input.biz_params = JSON.parse(flags.bizParams);
       } catch {
-        process.stderr.write("Error: --biz-params must be valid JSON\n");
-        process.exit(1);
+        throw new UsageError("--biz-params must be valid JSON");
       }
     }
 

packages/commands/src/commands/console/call.ts

@@ -1,4 +1,9 @@
-import { defineCommand, effectiveConsoleGatewayConfig, detectOutputFormat } from "bailian-cli-core";
+import {
+  defineCommand,
+  UsageError,
+  effectiveConsoleGatewayConfig,
+  detectOutputFormat,
+} from "bailian-cli-core";
 import { emitResult } from "bailian-cli-runtime";
 
 export default defineCommand({
@@ -39,8 +44,7 @@ export default defineCommand({
     try {
       data = JSON.parse(dataRaw) as Record<string, unknown>;
     } catch {
-      process.stderr.write("Error: --data must be valid JSON\n");
-      process.exit(1);
+      throw new UsageError("--data must be valid JSON");
     }
 
     const format = detectOutputFormat(config.output);

packages/commands/src/commands/mcp/call.ts

@@ -1,13 +1,18 @@
-import { defineCommand, bailianMcpPath, detectOutputFormat } from "bailian-cli-core";
+import {
+  defineCommand,
+  UsageError,
+  BailianError,
+  bailianMcpPath,
+  detectOutputFormat,
+} from "bailian-cli-core";
 import { emitResult } from "bailian-cli-runtime";
 
 function parseArgFlags(raw: string[]): Record<string, unknown> {
   const out: Record<string, unknown> = {};
   for (const item of raw) {
     const idx = item.indexOf("=");
     if (idx <= 0) {
-      process.stderr.write(`Error: --arg must be in K=V form, got: ${item}\n`);
-      process.exit(1);
+      throw new UsageError(`--arg must be in K=V form, got: ${item}`);
     }
     const key = item.slice(0, idx).trim();
     const rawVal = item.slice(idx + 1);
@@ -64,25 +69,23 @@ export default defineCommand({
 
     const dot = target.indexOf(".");
     if (dot <= 0 || dot === target.length - 1) {
-      process.stderr.write(`Error: target must be <server-code>.<tool>, got "${target}".\n`);
-      process.exit(1);
+      throw new UsageError(`target must be <server-code>.<tool>, got "${target}".`);
     }
     const serverCode = target.slice(0, dot);
     const toolName = target.slice(dot + 1);
 
     let toolArgs: Record<string, unknown> = {};
     if (flags.json) {
+      let parsed: unknown;
       try {
-        const parsed = JSON.parse(flags.json);
-        if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
-          process.stderr.write("Error: --json must decode to an object.\n");
-          process.exit(1);
-        }
-        toolArgs = parsed as Record<string, unknown>;
+        parsed = JSON.parse(flags.json);
       } catch (err) {
-        process.stderr.write(`Error: --json is not valid JSON — ${(err as Error).message}\n`);
-        process.exit(1);
+        throw new UsageError(`--json is not valid JSON — ${(err as Error).message}`);
       }
+      if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+        throw new UsageError("--json must decode to an object.");
+      }
+      toolArgs = parsed as Record<string, unknown>;
     }
     Object.assign(toolArgs, parseArgFlags(flags.arg ?? []));
     if (flags.query !== undefined) toolArgs.query = flags.query;
@@ -109,8 +112,7 @@ export default defineCommand({
 
     if (result.isError) {
       const errText = result.content.map((c) => c.text || "").join("\n");
-      process.stderr.write(`Tool error: ${errText}\n`);
-      process.exit(1);
+      throw new BailianError(`Tool error: ${errText}`);
     }
 
     emitResult(result, format);

packages/commands/src/commands/memory/add.ts

@@ -1,5 +1,6 @@
 import {
   defineCommand,
+  UsageError,
   memoryAddPath,
   detectOutputFormat,
   type FlagsDef,
@@ -52,8 +53,7 @@ export default defineCommand({
       try {
         body.messages = JSON.parse(flags.messages);
       } catch {
-        process.stderr.write("Error: --messages must be valid JSON array\n");
-        process.exit(1);
+        throw new UsageError("--messages must be valid JSON array");
       }
     }
 

packages/commands/src/commands/memory/profile-create.ts

@@ -1,5 +1,6 @@
 import {
   defineCommand,
+  UsageError,
   profileSchemaPath,
   detectOutputFormat,
   type ProfileSchemaCreateRequest,
@@ -38,8 +39,7 @@ export default defineCommand({
     try {
       attributes = JSON.parse(attrStr);
     } catch {
-      process.stderr.write("Error: --attributes must be valid JSON array\n");
-      process.exit(1);
+      throw new UsageError("--attributes must be valid JSON array");
     }
 
     const body: ProfileSchemaCreateRequest = { name, attributes };

packages/commands/src/commands/memory/search.ts

@@ -1,5 +1,6 @@
 import {
   defineCommand,
+  UsageError,
   memorySearchPath,
   detectOutputFormat,
   type FlagsDef,
@@ -49,8 +50,7 @@ export default defineCommand({
       try {
         body.messages = JSON.parse(flags.messages);
       } catch {
-        process.stderr.write("Error: --messages must be valid JSON array\n");
-        process.exit(1);
+        throw new UsageError("--messages must be valid JSON array");
       }
     }
 

packages/commands/src/commands/pipeline/load-file.ts

@@ -1,11 +1,11 @@
 import { readFile } from "node:fs/promises";
 import { extname } from "node:path";
+import { UsageError } from "bailian-cli-core";
 import type { PipelineDefinition } from "bailian-cli-runtime";
 
 export async function loadPipelineFile(filePath: string): Promise<PipelineDefinition> {
   const raw = await readFile(filePath, "utf-8").catch((err: Error) => {
-    process.stderr.write(`Error: cannot read pipeline file: ${err.message}\n`);
-    process.exit(2);
+    throw new UsageError(`cannot read pipeline file: ${err.message}`);
   });
   const ext = extname(filePath).toLowerCase();
   let parsed: unknown;