nirpa/granthi-review
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: send tools:false to the router so skill fast-paths cannot hijack reviews

Browse Source 
The router's current-info briefing fast-path (enableToolsEffective gate)
was intercepting review prompts whose diffs mention GitHub-like terms and
returning a trending-repos digest instead of the model's review. Reviews
need pure model reasoning; body.tools=false disables the tool layer.

Co-Authored-By: Claude Fable 5 <[email protected]>
This commit is contained in:
Nirav Patel
2026-08-18 23:41:09 -04:00
co-authored by Claude Fable 5
parent 74222a1631
commit ed4b347cab
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/lib/router.js
+4 -1
View File
@@ -6,7 +6,10 @@ export async function chat({ routerUrl, model, messages, agentId, sessionId, ten
const res = await fetch(`${routerUrl.replace(/\/$/, '')}/v1/chat`, { const res = await fetch(`${routerUrl.replace(/\/$/, '')}/v1/chat`, {
method: 'POST', method: 'POST',
headers: { 'Content-Type': 'application/json' }, headers: { 'Content-Type': 'application/json' },
body: JSON.stringify({ model, messages, agentId, sessionId, tenantId, stream: false }), // tools:false — reviews need pure model reasoning; it also keeps the
// router's tool/skill fast-paths (e.g. current-info briefing) from
// hijacking prompts whose diffs mention GitHub/news-like terms.
body: JSON.stringify({ model, messages, agentId, sessionId, tenantId, stream: false, tools: false }),
signal: AbortSignal.timeout(timeoutMs) signal: AbortSignal.timeout(timeoutMs)
}); });
if (!res.ok) { if (!res.ok) {