fix(models): preserve --all and --provider browse in replace mode

Monkey-wusky · Monkey-wusky · commit a537beed138b · 2026-07-17T23:24:40.000+08:00
The enableSourcePlanCascade guard was incorrectly blocked by
modelsModeReplace, which prevented --all and --provider-filtered
catalog browsing when models.mode=replace. The replace-mode
restriction is now scoped to appendAuthenticatedCatalogRows only,
matching the documented CLI contract.
diff --git a/src/commands/models/list.list-command.ts b/src/commands/models/list.list-command.ts
@@ -126,8 +126,8 @@ export async function modelsListCommand(
   const configuredByKey = new Map(entries.map((entry) => [entry.key, entry]));
   // When models.mode is "replace", only show explicitly configured providers.
   const modelsModeReplace = cfg.models?.mode === "replace";
-  const enableSourcePlanCascade =
-    !modelsModeReplace && (Boolean(opts.all) || Boolean(providerFilter));
+  // --all and --provider-filtered browse work regardless of models.mode.
+  const enableSourcePlanCascade = Boolean(opts.all) || Boolean(providerFilter);
   // Full/provider-filtered lists may need runtime, manifest, and registry rows.
   // Defer that planning so default configured-only output stays cheap.
   const sourcePlanModule = enableSourcePlanCascade ? await loadSourcePlanModule() : undefined;
