From d0009200119bcc1e27d3aa9fc1307c5b31993549 Mon Sep 17 00:00:00 2001 From: Hicham Zinalabdin <203812229+hichamza@users.noreply.github.com> Date: Wed, 12 Aug 2026 06:35:31 +0300 Subject: [PATCH] fix(ai-review): default opus-model to the bare "opus" tier alias Drops the pinned version from the review stage's Opus default so the gate always runs the current model in that tier instead of going stale against a hardcoded ID. --- ai-review/README.md | 2 +- ai-review/action.yml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/ai-review/README.md b/ai-review/README.md index 80d22fb..5007050 100644 --- a/ai-review/README.md +++ b/ai-review/README.md @@ -146,7 +146,7 @@ injection-safety rule. | `sonnet-files-threshold` | Max changed-file count for a diff to still route to `sonnet-model` (must hold together with `sonnet-churn-threshold`); larger diffs route to `opus-model`. | No | `25` | | `sonnet-churn-threshold` | Max changed-line count (adds + deletes) for a diff to still route to `sonnet-model`. | No | `800` | | `sonnet-model` | Model the routing step selects for diffs within **both** thresholds. Override when a gateway aliases model names. | No | `claude-sonnet-5` | -| `opus-model` | Model the routing step selects for every larger diff. Override when a gateway aliases model names. | No | `claude-opus-5` | +| `opus-model` | Model the routing step selects for every larger diff. Override when a gateway aliases model names. | No | `opus` | | `haiku-model` | Model used by the context stage. Note: Haiku 4.5 does not accept the `effort` parameter, so no stage running it passes `--effort`. | No | `claude-haiku-4-5` | | `enable-context-stage` | When `false`, skips the Haiku context stage (and its `context.md` verification) entirely. The stage is best-effort and its output optional, so disabling it removes a wall-clock risk without changing the gate contract. | No | `true` | | `api-timeout-ms` | Per-request timeout (ms) for every Claude stage, passed as `API_TIMEOUT_MS` (CLI default `600000`). **Does not bound the ~27.5-min stall** — a run with this set to `180000` still stalled 27m36s. It is a genuine per-request bound and fails a wedged request faster than the default, nothing more. | No | `180000` | diff --git a/ai-review/action.yml b/ai-review/action.yml index 2c6a9c9..6aa1e55 100644 --- a/ai-review/action.yml +++ b/ai-review/action.yml @@ -111,7 +111,7 @@ inputs: routing step, whose output feeds the Review stage as the model to run. Override when a gateway aliases model names. required: false - default: "claude-opus-5" + default: "opus" sonnet-files-threshold: description: > Max changed-file count for a diff to still be considered "tiny" and @@ -484,7 +484,7 @@ runs: if [ "${FILES}" -le "${FILES_MAX}" ] && [ "${CHURN}" -le "${CHURN_MAX}" ]; then MODEL="${SONNET:-claude-sonnet-5}" else - MODEL="${OPUS:-claude-opus-5}" + MODEL="${OPUS:-opus}" fi { echo "files=${FILES}"