fix(cli): combo create cannot accept models — add --models option (#10954)

[diegosouzapw]

Closes #10954

Root cause

bin/cli/commands/combo.mjs:

• the combo create subcommand only ever registered --strategy;
• the HTTP body (POST /api/combos, line ~292) and the local-db fallback
  (db.combos.createCombo, line ~308) both hardcoded models: [].

Every combo created via omniroute combo create came out empty regardless
of operator intent — there was no CLI flag to specify models at all.

Fix

• Added --models <spec> (comma-separated provider/model entries, or a
  JSON array of structured ComboStep-shaped entries) and a repeatable
  --model <spec> option to combo create.
• New bin/cli/commands/comboModels.mjs module parses and validates these
  options. The CLI ships as plain .mjs with relative-only imports (no
  @/ path aliases, no TS transpilation at runtime), so importing the real
  Zod schema from src/shared/validation/schemas/combo.ts directly is not
  viable — this module instead validates the same minimal shape by hand
  (string entries, { kind?: "model", model, providerId?, provider? }, or
  { kind: "combo-ref", comboName }), matching comboModelEntry /
  createComboSchema.models.
• runComboCreateCommand now propagates the parsed models array into
  both the HTTP body and the local-db fallback instead of the hardcoded
  [].

Tests (TDD, Hard Rule #18)

tests/unit/cli-combo-create-models-10954.test.ts:

• RED on the untouched code: --models/--model are unrecognized options
  — Commander (exitOverride) throws unknown option '--models'.
• GREEN after the fix: Commander parses both options; comboModels
  parser unit tests (CSV, JSON array, invalid-shape rejection, merge of
  --models + repeated --model); end-to-end through
  runComboCreateCommand for both the local-db fallback (models land in
  getComboByName(...).models, normalized by the repository layer) and
  the HTTP path (the POST /api/combos body carries the parsed models).

Sibling suites re-run clean: cli-combo-command.test.ts,
cli-combo-suggest-commands.test.ts, cli-route-unavailable-fallback-10081.test.ts.

Gates run locally

• node --import tsx/esm --test tests/unit/cli-combo-create-models-10954.test.ts tests/unit/cli-combo-command.test.ts tests/unit/cli-combo-suggest-commands.test.ts tests/unit/cli-route-unavailable-fallback-10081.test.ts — 22/22 green
• node scripts/check/check-file-size.mjs — OK
• node scripts/check/check-complexity.mjs — OK (2615 violations vs baseline 2774)
• node scripts/check/check-cognitive-complexity.mjs — OK (1175 violations vs baseline 1223)
• npm run typecheck:core — clean
• npx eslint bin/cli/commands/combo.mjs bin/cli/commands/comboModels.mjs tests/unit/cli-combo-create-models-10954.test.ts --suppressions-location config/quality/eslint-suppressions.json — clean (no new any)
• node scripts/check/check-changelog-integrity.mjs — OK

⚠️ base-red inherited: #9985 — public-creds(copilot-m365-web), docs-sync(PROVIDER_REFERENCE)