refactor(commands): remove yes confirmation gates

Author: XXPermanentXX

docs/plans/finetune-deploy-mvp.md

@@ -190,7 +190,6 @@ describe.skipIf(!isDashScopeE2EReady())("e2e: finetune (offline)", () => {
       "qwen3-8b",
       "--datasets",
       localPath,
-      "--yes",
       "--output",
       "json",
     ]);
@@ -214,7 +213,6 @@ describe.skipIf(!isDashScopeE2EReady())("e2e: finetune (offline)", () => {
       localPath,
       "--batch-size",
       "1",
-      "--yes",
       "--output",
       "json",
     ]);

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

@@ -22,7 +22,6 @@ describe("e2e: quota", () => {
     expect(exitCode, stderr).toBe(0);
     expect(stderr).toContain("--model");
     expect(stderr).toContain("--tpm");
-    expect(stderr).toContain("--yes");
   });
 
   test("quota history --help 正常退出", async () => {

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

@@ -1,11 +1,4 @@
-import {
-  defineCommand,
-  detectOutputFormat,
-  deleteDataset,
-  BailianError,
-  ExitCode,
-  type FlagsDef,
-} from "bailian-cli-core";
+import { defineCommand, detectOutputFormat, deleteDataset, type FlagsDef } from "bailian-cli-core";
 import { emitResult, emitBare } from "bailian-cli-runtime";
 
 const DELETE_FLAGS = {
@@ -15,15 +8,14 @@ const DELETE_FLAGS = {
     description: "Dataset file ID (required)",
     required: true,
   },
-  yes: { type: "switch", description: "Confirm the deletion (required to delete)" },
 } satisfies FlagsDef;
 
 export default defineCommand({
   description: "Delete a dataset file by ID",
   auth: "apiKey",
-  usageArgs: "--file-id <id> --yes",
+  usageArgs: "--file-id <id>",
   flags: DELETE_FLAGS,
-  exampleArgs: ["--file-id file-id-xxx --yes", "--file-id file-id-xxx --dry-run"],
+  exampleArgs: ["--file-id file-id-xxx", "--file-id file-id-xxx --dry-run"],
   async run(ctx) {
     const { settings, flags } = ctx;
     const fileId = flags.fileId;
@@ -34,14 +26,6 @@ export default defineCommand({
       return;
     }
 
-    if (!flags.yes) {
-      throw new BailianError(
-        `Refusing to permanently delete ${fileId} without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm the deletion.",
-      );
-    }
-
     const response = await deleteDataset(ctx.client, fileId);
 
     if (settings.quiet || format === "text") {

packages/commands/src/commands/dataset/delete.ts

@@ -2,8 +2,6 @@ import {
   defineCommand,
   detectOutputFormat,
   createDeployment,
-  BailianError,
-  ExitCode,
   type CreateDeploymentRequest,
   type FlagsDef,
 } from "bailian-cli-core";
@@ -58,16 +56,15 @@ const CREATE_FLAGS = {
     valueHint: "<n>",
     description: "PTU max thinking-output tokens/min (optional, some models)",
   },
-  yes: { type: "switch", description: "Confirm deployment creation (required to create)" },
 } satisfies FlagsDef;
 
 /**
  * `bl deploy create` — create a model deployment.
  *
  * Plan-specific behaviour (required flags / body assembly / auto-pick) lives
  * in `plans.ts` (`PlanStrategy` + `STRATEGIES`). This file only handles the
- * shared envelope: flag validation, dispatch, dry-run, the --yes gate, and
- * result formatting. Adding a new plan = one entry in the strategy table;
+ * shared envelope: flag validation, dispatch, dry-run, and result
+ * formatting. Adding a new plan = one entry in the strategy table;
  * nothing here changes.
  *
  * `--model` (model identifier) and `--name` (console display name) are required.
@@ -76,13 +73,13 @@ export default defineCommand({
   description: "Create a model deployment",
   auth: "apiKey",
   usageArgs:
-    "--model <model_name> --name <display_name> --yes [--plan <plan>] [--template-id <id>] [--capacity <n>] [--billing-method <m>] [--input-tpm <n>] [--output-tpm <n>] [--thinking-output-tpm <n>]",
+    "--model <model_name> --name <display_name> [--plan <plan>] [--template-id <id>] [--capacity <n>] [--billing-method <m>] [--input-tpm <n>] [--output-tpm <n>] [--thinking-output-tpm <n>]",
   flags: CREATE_FLAGS,
   exampleArgs: [
-    "--model my-qwen-sft --name my-sft-test --yes",
-    "--model qwen3.6-flash-2026-04-16 --name my-flash --plan ptu --input-tpm 10000 --output-tpm 1000 --yes",
-    "--model qwen3-8b --name my-qwen3-mu --plan mu --yes",
-    "--model qwen3-8b --name my-qwen3 --plan mu --template-id MU1 --capacity 2 --yes",
+    "--model my-qwen-sft --name my-sft-test",
+    "--model qwen3.6-flash-2026-04-16 --name my-flash --plan ptu --input-tpm 10000 --output-tpm 1000",
+    "--model qwen3-8b --name my-qwen3-mu --plan mu",
+    "--model qwen3-8b --name my-qwen3 --plan mu --template-id MU1 --capacity 2",
   ],
   notes: [
     "Plan defaults to `lora` (Token-billed). Pass --plan to override.",
@@ -124,15 +121,6 @@ export default defineCommand({
     // already rejected by `validate` above.
     const strategy = pickPlanStrategy(plan);
 
-    // Gate before any side-effecting resolution (mu hits the catalog API).
-    if (!settings.dryRun && !flags.yes) {
-      throw new BailianError(
-        `Refusing to create deployment (model=${model}, name=${name}, plan=${plan}) without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm deployment creation, or use --dry-run to preview the request.",
-      );
-    }
-
     const resolved = await strategy.resolve({
       client: ctx.client,
       dryRun: settings.dryRun,

packages/commands/src/commands/deploy/create.ts

@@ -16,7 +16,6 @@ const DELETE_FLAGS = {
     description: "Deployed model identifier (required)",
     required: true,
   },
-  yes: { type: "switch", description: "Confirm the deletion (required to delete)" },
   skipPrecheck: {
     type: "switch",
     description: "Skip the local STOPPED/FAILED status precheck",
@@ -33,9 +32,9 @@ const DELETE_FLAGS = {
 export default defineCommand({
   description: "Delete a model deployment (must be STOPPED or FAILED)",
   auth: "apiKey",
-  usageArgs: "--deployed-model <id> --yes [--skip-precheck]",
+  usageArgs: "--deployed-model <id> [--skip-precheck]",
   flags: DELETE_FLAGS,
-  exampleArgs: ["--deployed-model dep-... --yes", "--deployed-model dep-... --dry-run"],
+  exampleArgs: ["--deployed-model dep-...", "--deployed-model dep-... --dry-run"],
   async run(ctx) {
     const { settings, flags } = ctx;
     const deployedModel = flags.deployedModel;
@@ -46,14 +45,6 @@ export default defineCommand({
       return;
     }
 
-    if (!flags.yes) {
-      throw new BailianError(
-        `Refusing to delete deployment ${deployedModel} without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm the deletion.",
-      );
-    }
-
     // Precheck status unless skipped — surface a clear hint instead of letting
     // the server return a generic precondition error.
     if (!flags.skipPrecheck) {

packages/commands/src/commands/deploy/delete.ts

@@ -2,8 +2,6 @@ import {
   defineCommand,
   detectOutputFormat,
   scaleDeployment,
-  BailianError,
-  ExitCode,
   type FlagsDef,
 } from "bailian-cli-core";
 import { emitResult, emitBare } from "bailian-cli-runtime";
@@ -30,7 +28,6 @@ const SCALE_FLAGS = {
     valueHint: "<n>",
     description: "PTU only — output tokens per minute",
   },
-  yes: { type: "switch", description: "Confirm the scaling (required to scale)" },
 } satisfies FlagsDef;
 
 /**
@@ -42,11 +39,11 @@ const SCALE_FLAGS = {
 export default defineCommand({
   description: "Scale a deployment's capacity",
   auth: "apiKey",
-  usageArgs: "--deployed-model <id> --capacity <n> --yes [--input-tpm <n>] [--output-tpm <n>]",
+  usageArgs: "--deployed-model <id> --capacity <n> [--input-tpm <n>] [--output-tpm <n>]",
   flags: SCALE_FLAGS,
   exampleArgs: [
-    "--deployed-model qwen-plus-...-b6d61c71 --capacity 8 --yes",
-    "--deployed-model dep-... --capacity 2 --yes",
+    "--deployed-model qwen-plus-...-b6d61c71 --capacity 8",
+    "--deployed-model dep-... --capacity 2",
   ],
   validate: (flags) =>
     flags.capacity === undefined && flags.inputTpm === undefined && flags.outputTpm === undefined
@@ -67,18 +64,6 @@ export default defineCommand({
       return;
     }
 
-    if (!flags.yes) {
-      const parts: string[] = [];
-      if (flags.capacity !== undefined) parts.push(`capacity=${flags.capacity}`);
-      if (flags.inputTpm !== undefined) parts.push(`input_tpm=${flags.inputTpm}`);
-      if (flags.outputTpm !== undefined) parts.push(`output_tpm=${flags.outputTpm}`);
-      throw new BailianError(
-        `Refusing to scale deployment ${deployedModel} (${parts.join(", ")}) without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm the scaling.",
-      );
-    }
-
     const response = await scaleDeployment(ctx.client, deployedModel, body);
     const deployment = response.output ?? response.data;
 

packages/commands/src/commands/deploy/scale.ts

@@ -2,8 +2,6 @@ import {
   defineCommand,
   detectOutputFormat,
   updateDeployment,
-  BailianError,
-  ExitCode,
   type FlagsDef,
 } from "bailian-cli-core";
 import { emitResult, emitBare } from "bailian-cli-runtime";
@@ -25,7 +23,6 @@ const UPDATE_FLAGS = {
     valueHint: "<n>",
     description: "Tokens per minute",
   },
-  yes: { type: "switch", description: "Confirm the rate-limit update (required to update)" },
 } satisfies FlagsDef;
 
 /**
@@ -37,11 +34,11 @@ const UPDATE_FLAGS = {
 export default defineCommand({
   description: "Update a deployment's rate limits (rpm_limit / tpm_limit)",
   auth: "apiKey",
-  usageArgs: "--deployed-model <id> --yes [--rpm-limit <n>] [--tpm-limit <n>]",
+  usageArgs: "--deployed-model <id> [--rpm-limit <n>] [--tpm-limit <n>]",
   flags: UPDATE_FLAGS,
   exampleArgs: [
-    "--deployed-model dep-... --rpm-limit 1000 --yes",
-    "--deployed-model dep-... --rpm-limit 1000 --tpm-limit 200000 --yes",
+    "--deployed-model dep-... --rpm-limit 1000",
+    "--deployed-model dep-... --rpm-limit 1000 --tpm-limit 200000",
   ],
   notes: ["At least one of --rpm-limit / --tpm-limit must be provided."],
   validate: (flags) =>
@@ -62,17 +59,6 @@ export default defineCommand({
       return;
     }
 
-    if (!flags.yes) {
-      const parts: string[] = [];
-      if (flags.rpmLimit !== undefined) parts.push(`rpm_limit=${flags.rpmLimit}`);
-      if (flags.tpmLimit !== undefined) parts.push(`tpm_limit=${flags.tpmLimit}`);
-      throw new BailianError(
-        `Refusing to update rate limits for ${deployedModel} (${parts.join(", ")}) without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm the rate-limit update.",
-      );
-    }
-
     const response = await updateDeployment(ctx.client, deployedModel, body);
     const deployment = response.output ?? response.data;
 

packages/commands/src/commands/deploy/update.ts

@@ -1,11 +1,4 @@
-import {
-  defineCommand,
-  detectOutputFormat,
-  cancelFineTune,
-  BailianError,
-  ExitCode,
-  type FlagsDef,
-} from "bailian-cli-core";
+import { defineCommand, detectOutputFormat, cancelFineTune, type FlagsDef } from "bailian-cli-core";
 import { emitResult, emitBare } from "bailian-cli-runtime";
 
 const CANCEL_FLAGS = {
@@ -15,15 +8,14 @@ const CANCEL_FLAGS = {
     description: "Fine-tune job ID (required)",
     required: true,
   },
-  yes: { type: "switch", description: "Confirm the cancellation (required to cancel)" },
 } satisfies FlagsDef;
 
 export default defineCommand({
   description: "Cancel a running fine-tune job",
   auth: "apiKey",
-  usageArgs: "--job-id <id> --yes",
+  usageArgs: "--job-id <id>",
   flags: CANCEL_FLAGS,
-  exampleArgs: ["--job-id ft-xxx --yes", "--job-id ft-xxx --dry-run"],
+  exampleArgs: ["--job-id ft-xxx", "--job-id ft-xxx --dry-run"],
   notes: [
     "Only PENDING / RUNNING jobs can be cancelled. Completed / failed / already-",
     "cancelled jobs return a server-side error (passed through verbatim).",
@@ -38,14 +30,6 @@ export default defineCommand({
       return;
     }
 
-    if (!flags.yes) {
-      throw new BailianError(
-        `Refusing to cancel fine-tune job ${jobId} without --yes.`,
-        ExitCode.USAGE,
-        "Pass --yes to confirm the cancellation.",
-      );
-    }
-
     const response = await cancelFineTune(ctx.client, jobId);
     const job = response.output ?? response.data;
 

packages/commands/src/commands/finetune/cancel.ts

@@ -250,31 +250,27 @@ const CREATE_FLAGS = {
     valueHint: "<n>",
     description: "Max sequence length",
   },
-  yes: {
-    type: "switch",
-    description: "Confirm job creation (required to submit; uploads data and consumes quota)",
-  },
 } satisfies FlagsDef;
 
 export default defineCommand({
   description: "Create a fine-tune job (sft | sft-lora | dpo | dpo-lora | cpt)",
   auth: "apiKey",
   usageArgs:
-    "--model <model> --datasets <id|path,...> [--validations <id|path,...>] [--model-name <name>] [--suffix <text>] [--n-epochs <n>] [--batch-size <n>] [--learning-rate <str>] [--max-length <n>] [--training-type <sft|sft-lora|dpo|dpo-lora|cpt>] --yes",
+    "--model <model> --datasets <id|path,...> [--validations <id|path,...>] [--model-name <name>] [--suffix <text>] [--n-epochs <n>] [--batch-size <n>] [--learning-rate <str>] [--max-length <n>] [--training-type <sft|sft-lora|dpo|dpo-lora|cpt>]",
   flags: CREATE_FLAGS,
   exampleArgs: [
-    "--model qwen3-8b --datasets file-xxx --yes",
-    "--model qwen3-8b --datasets ./train.jsonl --yes",
-    "--model qwen3-8b --datasets ./train.jsonl --validations ./eval.jsonl --yes",
-    "--model qwen3-8b --datasets file-aaa,./extra.jsonl --yes",
-    "--model qwen3-8b --datasets ./train.jsonl --training-type sft --yes",
-    '--model qwen3-8b --datasets file-xxx --learning-rate "1.6e-5" --n-epochs 4 --yes',
-    "--model qwen3-8b --datasets file-xxx --yes --output json",
+    "--model qwen3-8b --datasets file-xxx",
+    "--model qwen3-8b --datasets ./train.jsonl",
+    "--model qwen3-8b --datasets ./train.jsonl --validations ./eval.jsonl",
+    "--model qwen3-8b --datasets file-aaa,./extra.jsonl",
+    "--model qwen3-8b --datasets ./train.jsonl --training-type sft",
+    '--model qwen3-8b --datasets file-xxx --learning-rate "1.6e-5" --n-epochs 4',
+    "--model qwen3-8b --datasets file-xxx --output json",
     "--model qwen3-8b --datasets file-xxx --dry-run",
   ],
   notes: [
-    "Creating a job consumes training quota, so --yes is required to submit",
-    "(use --dry-run to preview the request body without --yes).",
+    "Creating a job uploads any local datasets and consumes training quota.",
+    "Use --dry-run to preview the request body without submitting.",
     "Training-type values use the `<method>` / `<method>-lora` convention:",
     "sft (full) | sft-lora (LoRA) | dpo (full) | dpo-lora (LoRA) | cpt. These map",
     "to the server's training_type at the interface boundary, so the rest of the",
@@ -443,18 +439,8 @@ export default defineCommand({
       }
     }
 
-    // --yes gate — BEFORE upload: without it we must not silently consume
-    // quota OR upload any file. (Local validation is still allowed to run.)
-    if (!settings.dryRun && !flags.yes) {
-      throw new BailianError(
-        "Refusing to create a fine-tune job without --yes.",
-        ExitCode.USAGE,
-        "Pass --yes to confirm creation (uploads datasets and consumes training quota), or --dry-run to preview the request.",
-      );
-    }
-
     // Upload local paths now that pre-flight (validation, batch-size gate,
-    // capability check, --yes gate) has cleared them. This swaps the
+    // capability check) has cleared them. This swaps the
     // placeholder path entries in `training.fileIds` / `validation?.fileIds`
     // for real file-ids, so the body below sees ids.
     if (!settings.dryRun) {