Skip to content

Revert "fix(ai-review): default opus-model to the bare "opus" alias" #48

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Hicham Zinalabdin (hichamza) merged 1 commit into from
Aug 12, 2026
Merged

Revert "fix(ai-review): default opus-model to the bare "opus" alias" #48

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ai-review/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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` |
Expand Down
4 changes: 2 additions & 2 deletions ai-review/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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}"
Expand Down
Loading