sapper-iq 1.2.3 → 1.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sapper-iq",
-  "version": "1.2.3",
+  "version": "1.2.5",
   "description": "AI-powered development assistant that executes commands and builds projects",
   "main": "sapper.mjs",
   "bin": {

package/sapper.mjs

@@ -1016,6 +1016,9 @@ project/
 // Global flag — set after model selection, read in buildSystemPrompt
 let _useNativeToolsFlag = false;
 
+// Models known to reject `think:true` (populated lazily on first failure).
+const modelsWithoutThinking = new Set();
+
 function buildSystemPrompt(agentContent = null, skillContents = []) {
   const now = new Date();
   const dateStr = now.toLocaleDateString('en-US', { weekday: 'long', year: 'numeric', month: 'long', day: 'numeric' });
@@ -8621,8 +8624,9 @@ async function runSapper() {
         }
         return { result, success: true };
       }
-      const turnThinkingEnabled = shouldUseThinkingForInput(input);
-      
+      let turnThinkingEnabled = shouldUseThinkingForInput(input);
+      if (modelsWithoutThinking.has(selectedModel)) turnThinkingEnabled = false;
+
       let active = true;
       while (active) {
         if (stepMode) await safeQuestion(chalk.gray(promptLabel('questions.stepContinue', '[STEP] Press Enter to let AI think...')));
@@ -8630,38 +8634,55 @@ async function runSapper() {
         spinner.start('Thinking...');
         const aiStartTime = Date.now();
         let response;
-        try {
-          // Build chat options — pass native tools when supported
-          const chatOpts = { model: selectedModel, messages, stream: true };
-          if (effectiveContextLength()) {
-            chatOpts.options = { num_ctx: effectiveContextLength() };
-          }
-          // Thinking can be forced on, forced off, or auto-disabled for simple prompts.
-          chatOpts.think = turnThinkingEnabled;
-          if (useNativeTools) {
-            // Filter tool defs by agent restrictions if any
-            if (currentAgentTools) {
-              const toolNameMap = {
-                list_directory: 'LIST', read_file: 'READ', search_files: 'SEARCH',
-                write_file: 'WRITE', patch_file: 'PATCH', create_directory: 'MKDIR',
-                ls: 'LS', cat: 'CAT', head: 'HEAD', tail: 'TAIL', grep: 'GREP', find: 'FIND',
-                pwd: 'PWD', cd: 'CD', rmdir: 'RMDIR', changes: 'CHANGES',
-                fetch_web: 'FETCH', recall_memory: 'MEMORY', open_url: 'OPEN', run_shell: 'SHELL'
-              };
-              chatOpts.tools = nativeToolDefs.filter(t => 
-                isToolAllowedForAgent(currentAgentTools, toolNameMap[t.function.name])
-              );
-            } else {
-              chatOpts.tools = nativeToolDefs;
-            }
+        // Build chat options — pass native tools when supported
+        const chatOpts = { model: selectedModel, messages, stream: true };
+        if (effectiveContextLength()) {
+          chatOpts.options = { num_ctx: effectiveContextLength() };
+        }
+        // Thinking can be forced on, forced off, or auto-disabled for simple prompts.
+        if (turnThinkingEnabled) chatOpts.think = true;
+        if (useNativeTools) {
+          // Filter tool defs by agent restrictions if any
+          if (currentAgentTools) {
+            const toolNameMap = {
+              list_directory: 'LIST', read_file: 'READ', search_files: 'SEARCH',
+              write_file: 'WRITE', patch_file: 'PATCH', create_directory: 'MKDIR',
+              ls: 'LS', cat: 'CAT', head: 'HEAD', tail: 'TAIL', grep: 'GREP', find: 'FIND',
+              pwd: 'PWD', cd: 'CD', rmdir: 'RMDIR', changes: 'CHANGES',
+              fetch_web: 'FETCH', recall_memory: 'MEMORY', open_url: 'OPEN', run_shell: 'SHELL'
+            };
+            chatOpts.tools = nativeToolDefs.filter(t => 
+              isToolAllowedForAgent(currentAgentTools, toolNameMap[t.function.name])
+            );
+          } else {
+            chatOpts.tools = nativeToolDefs;
           }
+        }
+        try {
           response = await ollama.chat(chatOpts);
         } catch (ollamaError) {
-          spinner.stop();
-          console.error(chalk.red('\n❌ Ollama error:'), ollamaError.message);
-          logEntry('error', { message: `Ollama error: ${ollamaError.message}` });
-          active = false;
-          continue;
+          const errMsg = ollamaError && ollamaError.message ? ollamaError.message : String(ollamaError);
+          if (/does not support thinking/i.test(errMsg) && chatOpts.think) {
+            modelsWithoutThinking.add(selectedModel);
+            turnThinkingEnabled = false;
+            delete chatOpts.think;
+            console.log(chalk.yellow(`  ⚠ "${selectedModel}" doesn't support thinking — retrying without it.`));
+            try {
+              response = await ollama.chat(chatOpts);
+            } catch (retryErr) {
+              spinner.stop();
+              console.error(chalk.red('\n❌ Ollama error:'), retryErr.message);
+              logEntry('error', { message: `Ollama error: ${retryErr.message}` });
+              active = false;
+              continue;
+            }
+          } else {
+            spinner.stop();
+            console.error(chalk.red('\n❌ Ollama error:'), errMsg);
+            logEntry('error', { message: `Ollama error: ${errMsg}` });
+            active = false;
+            continue;
+          }
         }
         spinner.stop();