Respect useTools=false for all tool types

Author: eypsilon

src/services/review-client.js

@@ -184,9 +184,16 @@ function createToolCallKey(toolCall) {
 }
 
 function createToolKit({ cwd, reviewer, priorReviews }) {
-  const fileTools = reviewer.useTools === false
-    ? { definitions: [], execute: null }
-    : createFileContextTools(cwd);
+  if (reviewer.useTools === false) {
+    return {
+      definitions: [],
+      execute: async () => {
+        throw new Error('Tools are disabled for this reviewer.');
+      }
+    };
+  }
+
+  const fileTools = createFileContextTools(cwd);
   const historyTools = Array.isArray(priorReviews) && priorReviews.length > 0 && reviewer.reviewSharing !== false
     ? createReviewHistoryTools(priorReviews)
     : { definitions: [], execute: null };

test/nvidia-client.test.js

@@ -405,3 +405,54 @@ test('requestReviewResult exposes prior reviews through get_available_reviews wh
   assert.ok(payloads[0].messages[0].content.includes('get_available_reviews'));
   assert.equal(payloads[0].tools.some((tool) => tool.function.name === 'get_available_reviews'), true);
 });
+
+test('requestReviewResult does not expose any tools when reviewer disables tools', async () => {
+  const payloads = [];
+  const client = {
+    chat: {
+      completions: {
+        create: async (payload) => {
+          payloads.push(payload);
+          return {
+            choices: [
+              {
+                message: {
+                  content: JSON.stringify({ findings: [] }),
+                  tool_calls: []
+                }
+              }
+            ]
+          };
+        }
+      }
+    }
+  };
+
+  const result = await requestReviewResult({
+    client,
+    cwd: __dirname,
+    reviewer: {
+      name: 'Reviewer',
+      model: 'gpt-5-mini',
+      prompt: 'Review this code.',
+      useTools: false
+    },
+    context: {
+      contextMode: 'diff',
+      entries: []
+    },
+    priorReviews: [
+      {
+        reviewer: 'Earlier Review',
+        model: 'example/model',
+        findings: ['something'],
+        structuredFindings: [],
+        error: null
+      }
+    ],
+    structuredOutputMode: 'json_object'
+  });
+
+  assert.equal(result.structured.findings.length, 0);
+  assert.equal(Array.isArray(payloads[0].tools), false);
+});