fix: ollama tool use

The code that added the PATH to the stdio transport environment was
mutating the original options.transport.env object, which can lead
to an infinite loop of reloading the continue config since
MCPMananagerSingletone.setConnections() compares the old and new
options and forces a refresh if any existing server changed transport
options. This triggers a config reload because in core.ts we have:

  MCPManagerSingleton.getInstance().onConnectionsRefreshed = async () => {
     await this.configHandler.reloadConfig();
  };
Which thens calls setConnections() and so forth...

core/config/ConfigHandler.ts

@@ -279,10 +279,7 @@ export class ConfigHandler {
     if (options.includeWorkspace) {
       const assistantFiles = await getAllDotContinueYamlFiles(
         this.ide,
-        {
-          ...options,
-          includeGlobal: false, // Because the global comes from above
-        },
+        options,
         ASSISTANTS,
       );
       const profiles = assistantFiles.map((assistant) => {

core/context/mcp/MCPConnection.ts

@@ -55,7 +55,9 @@ class MCPConnection {
   private constructTransport(options: MCPOptions): Transport {
     switch (options.transport.type) {
       case "stdio":
-        const env: Record<string, string> = options.transport.env || {};
+        const env: Record<string, string> = options.transport.env
+          ? { ...options.transport.env }
+          : {};
         if (process.env.PATH !== undefined) {
           env.PATH = process.env.PATH;
         }

core/llm/llms/Ollama.ts

@@ -1,5 +1,6 @@
 import { Mutex } from "async-mutex";
 import { JSONSchema7, JSONSchema7Object } from "json-schema";
+import { v4 as uuidv4 } from "uuid";
 
 import {
   ChatMessage,
@@ -90,10 +91,10 @@ type OllamaBaseResponse = {
   model: string;
   created_at: string;
 } & (
-    | {
+  | {
       done: false;
     }
-    | {
+  | {
       done: true;
       done_reason: string;
       total_duration: number; // Time spent generating the response in nanoseconds
@@ -104,7 +105,7 @@ type OllamaBaseResponse = {
       eval_duration: number; // Time spent generating the response in nanoseconds
       context: number[]; // An encoding of the conversation used in this response; can be sent in the next request to keep conversational memory
     }
-  );
+);
 
 type OllamaErrorResponse = {
   error: string;
@@ -113,14 +114,14 @@ type OllamaErrorResponse = {
 type OllamaRawResponse =
   | OllamaErrorResponse
   | (OllamaBaseResponse & {
-    response: string; // the generated response
-  });
+      response: string; // the generated response
+    });
 
 type OllamaChatResponse =
   | OllamaErrorResponse
   | (OllamaBaseResponse & {
-    message: OllamaChatMessage;
-  });
+      message: OllamaChatMessage;
+    });
 
 interface OllamaTool {
   type: "function";
@@ -370,7 +371,7 @@ class Ollama extends BaseLLM implements ModelInstaller {
             if ("error" in j) {
               throw new Error(j.error);
             }
-            j.response ??= ''
+            j.response ??= "";
             yield j.response;
           } catch (e) {
             throw new Error(`Error parsing Ollama response: ${e} ${chunk}`);
@@ -439,6 +440,7 @@ class Ollama extends BaseLLM implements ModelInstaller {
           // But ollama returns the full object in one response with no streaming
           chatMessage.toolCalls = res.message.tool_calls.map((tc) => ({
             type: "function",
+            id: `tc_${uuidv4()}`, // Generate a proper UUID with a prefix
             function: {
               name: tc.function.name,
               arguments: JSON.stringify(tc.function.arguments),

core/util/logger.ts

@@ -7,7 +7,9 @@ export const logger = winston.createLogger({
   format: winston.format.json(),
   transports: [
     // Write all logs with importance level of `info` or higher to `info.log`
-    new winston.transports.File({ filename: "e2e.log", level: "info" }),
+    ...(process.env.NODE_ENV === "test"
+      ? [new winston.transports.File({ filename: "e2e.log", level: "info" })]
+      : []),
     // Normal console.log behavior
     new winston.transports.Console(),
   ],

extensions/vscode/package.json

@@ -2,7 +2,7 @@
   "name": "continue",
   "icon": "media/icon.png",
   "author": "Continue Dev, Inc",
-  "version": "1.1.31",
+  "version": "1.0.9",
   "repository": {
     "type": "git",
     "url": "https://github.com/continuedev/continue"