fix(config): restore --no-workflow as warn-and-ignore shim

Author: TheGreatAxios

docs/IMPLEMENTATION.md

@@ -444,6 +444,7 @@ Printed by `corbits --help` / `-h` from `CLI_HELP_TEXT` in `src/config/index.ts`
 | `--dangerously-skip-permissions`, `--yolo` | false                      | Process-only aliases that auto-allow anything not denied by the authorization layer (gate + pre-gate workspace sandboxes; secret-guard / authz hard denies remain). `/yolo [on\|off\|toggle]` persists to the active settings file instead.  |
 | `--auto`                                   | true (default)             | Force auto mode on (workspace writes + unconstrained shell without prompts)                                                                                                                                                                  |
 | `--no-auto`                                | false                      | Start with auto mode off (ask on every consequential action); no in-session key toggles it                                                                                                                                                   |
+| `--no-workflow`                            | —                          | Deprecated no-op: accepted with a stderr warning and ignored (kept for one release so existing wrappers keep starting; will become `unrecognized flag` after removal)                                                                        |
 | `--help`, `-h`                             | —                          | Show help (exit 0 via `CliHelpError`)                                                                                                                                                                                                        |
 | `--version`, `-V`                          | —                          | Print `corbits v` plus the running `package.json` version (exit 0 via `CliVersionError`)                                                                                                                                                     |
 

src/config/index.ts

@@ -957,6 +957,15 @@ export async function loadConfig(
       auto = false;
       continue;
     }
+    // CL-10755: deprecated no-op retained for one release as warn-and-ignore
+    // so existing `--no-workflow` wrappers keep starting instead of hitting
+    // the catch-all "unrecognized flag" error below.
+    if (arg === "--no-workflow") {
+      process.stderr.write(
+        "--no-workflow is deprecated and has no effect; ignoring.\n",
+      );
+      continue;
+    }
     if (arg === "-p") {
       continue;
     }

src/config/settings.test.ts

@@ -202,6 +202,31 @@ test("loadConfig --no-auto disables auto mode", async () => {
   });
 });
 
+// CL-10755: --no-workflow is a deprecated no-op. It must stay accepted for
+// one release (stderr warn-and-ignore) instead of hitting the catch-all
+// "unrecognized flag" error.
+test("loadConfig --no-workflow warns and is ignored", async () => {
+  await withSettings(async ({ cwd, globalSettingsPath }) => {
+    const { impl } = offlineFetch();
+    const stderr = captureStderr();
+    try {
+      const config = await loadConfig(
+        ["--cwd", cwd, "--no-workflow", "do something"],
+        {
+          globalSettingsPath,
+          pricing: { fetchImpl: impl },
+        },
+      );
+      expect(config.configured).toBe(true);
+      expect(stderr.output()).toContain(
+        "--no-workflow is deprecated and has no effect; ignoring.",
+      );
+    } finally {
+      stderr.restore();
+    }
+  });
+});
+
 test("loadConfig uses the injected pricing fetchImpl instead of the network", async () => {
   await withSettings(async ({ cwd, globalSettingsPath }) => {
     resetPricingMetadataRefreshForTests();