From 2bc9e6736143a33ba42456268a3d3a72f0031785 Mon Sep 17 00:00:00 2001 From: Hicham Zinalabdin <203812229+hichamza@users.noreply.github.com> Date: Wed, 12 Aug 2026 07:01:42 +0300 Subject: [PATCH] Revert "Merge pull request #47 from EdulyCom/fix/ai-review-opus-model-alias" This reverts commit 60a91e6d0eefcfbc0255e3ebd01dd4b029c5a741, reversing changes made to 103518154652db3c4f99170a77bd43f3fc80fd9d. --- 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 5007050..80d22fb 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 | `opus` | +| `opus-model` | Model the routing step selects for every larger diff. Override when a gateway aliases model names. | No | `claude-opus-5` | | `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 6aa1e55..2c6a9c9 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: "opus" + default: "claude-opus-5" 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:-opus}" + MODEL="${OPUS:-claude-opus-5}" fi { echo "files=${FILES}"