From aafaa412eee3569dd78903787583ab64cfe3cecc Mon Sep 17 00:00:00 2001
From: Tanguille <91473554+Tanguille@users.noreply.github.com>
Date: Tue, 4 Aug 2026 15:58:13 +0200
Subject: [PATCH] fix(ci): point ai_fallback_model at qwen-3.6-fast for
 pr-reviewer-action v2.1.10
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

v2.1.10 made ai_fallback_base_url inherit from ai_base_url when unset,
then added validation requiring ai_fallback_model whenever
ai_fallback_base_url is set (even inherited) — the action now fails
before making any model call.

litellm's routerSettings.fallbacks already retries omniroute failures
internally with qwen-3.6-fast, so the action's own fallback path only
fires if that whole chain fails. Point it at qwen-3.6-fast directly
(same litellm endpoint, distinct model) rather than duplicating
ai_model — a genuine second attempt, not a no-op retry of what just
failed.
---
 .github/workflows/agent-pr-review.yaml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/agent-pr-review.yaml b/.github/workflows/agent-pr-review.yaml
index 469e5b3231..ad5ad9f866 100644
--- a/.github/workflows/agent-pr-review.yaml
+++ b/.github/workflows/agent-pr-review.yaml
@@ -33,7 +33,7 @@ jobs:
           ref: ${{ github.event.pull_request.head.sha }}
 
       - name: Run reviewer
-        uses: misospace/pr-reviewer-action@1cb6bed7a29f0d528ff5ba56bcf57092bf096af1 # v2.1.9
+        uses: misospace/pr-reviewer-action@6f4fb724b222f6d9a83096d6b41cd636ef3a044e # v2.1.10
         with:
           github_token: ${{ github.token }}
           ai_base_url: http://litellm.ai.svc.cluster.local/v1
@@ -42,6 +42,14 @@ jobs:
           # (see litellm routerSettings.fallbacks); falls back to qwen-3.6-fast
           # automatically if the free provider fails or times out.
           ai_model: omniroute
+          # v2.1.10 inherits ai_fallback_base_url from ai_base_url when unset, then
+          # requires ai_fallback_model whenever ai_fallback_base_url is set (even
+          # inherited) — fails fast with no model call otherwise. Point this at
+          # qwen-3.6-fast directly (same litellm endpoint) rather than duplicating
+          # ai_model: litellm's routerSettings.fallbacks already retries omniroute
+          # failures internally, so this only fires if that whole chain fails —
+          # a real second attempt via a distinct model, not a no-op retry.
+          ai_fallback_model: qwen-3.6-fast
           # omniroute's proxied models don't share SGLang's sampling_defaults, so
           # an empty temperature has no known-good fallback here — pin explicitly.
           ai_temperature: "0.2"