fix: llm choice working well for openai, anthropic and ollama

bhouston · bhouston · commit 68d34abf8a73 · 2025-03-12T10:57:53.000-04:00
diff --git a/README.md b/README.md
@@ -59,23 +59,23 @@ Create a `mycoder.config.js` file in your project root:
 export default {
   // GitHub integration
   githubMode: true,
-  
+
   // Browser settings
   headless: true,
   userSession: false,
   pageFilter: 'none', // 'simple', 'none', or 'readability'
-  
+
   // Model settings
   provider: 'anthropic',
   model: 'claude-3-7-sonnet-20250219',
   maxTokens: 4096,
   temperature: 0.7,
-  
+
   // Custom settings
   customPrompt: '',
   profile: false,
   tokenCache: true,
-  
+
   // Ollama configuration (if using local models)
   ollamaBaseUrl: 'http://localhost:11434',
 };
diff --git a/mycoder.config.js b/mycoder.config.js
@@ -11,8 +11,10 @@ export default {
   // Model settings
   //provider: 'anthropic',
   //model: 'claude-3-7-sonnet-20250219',
-  provider: 'openai',
-  model: 'gpt-4o',
+  //provider: 'openai',
+  //model: 'gpt-4o',
+  provider: 'ollama',
+  model: 'medragondot/Sky-T1-32B-Preview:latest',
   maxTokens: 4096,
   temperature: 0.7,
 
diff --git a/packages/agent/src/core/llm/provider.ts b/packages/agent/src/core/llm/provider.ts
@@ -53,7 +53,6 @@ export function createProvider(
   model: string,
   options: ProviderOptions = {},
 ): LLMProvider {
-  console.log({ providerType, model, options });
   const factory = providerFactories[providerType.toLowerCase()];
 
   if (!factory) {
diff --git a/packages/agent/src/core/llm/providers/anthropic.ts b/packages/agent/src/core/llm/providers/anthropic.ts
@@ -110,12 +110,6 @@ export class AnthropicProvider implements LLMProvider {
       throw new Error('Anthropic API key is required');
     }
 
-    console.log({
-      provider: this.provider,
-      model,
-      apiKey: this.apiKey,
-    });
-
     // Initialize Anthropic client
     this.client = new Anthropic({
       apiKey: this.apiKey,
diff --git a/packages/agent/src/core/llm/providers/openai.ts b/packages/agent/src/core/llm/providers/openai.ts
@@ -50,12 +50,6 @@ export class OpenAIProvider implements LLMProvider {
       throw new Error('OpenAI API key is required');
     }
 
-    console.log({
-      provider: this.provider,
-      model,
-      apiKey: this.apiKey,
-    });
-
     // Initialize OpenAI client
     this.client = new OpenAI({
       apiKey: this.apiKey,
diff --git a/packages/agent/src/utils/errors.ts b/packages/agent/src/utils/errors.ts
@@ -7,12 +7,11 @@ export const providerConfig: Record<
     keyName: 'ANTHROPIC_API_KEY',
     docsUrl: 'https://mycoder.ai/docs/getting-started/anthropic',
   },
-  /*
   openai: {
     keyName: 'OPENAI_API_KEY',
     docsUrl: 'https://mycoder.ai/docs/getting-started/openai',
   },
-  xai: {
+  /*xai: {
     keyName: 'XAI_API_KEY',
     docsUrl: 'https://mycoder.ai/docs/getting-started/xai',
   },
diff --git a/packages/cli/README.md b/packages/cli/README.md
@@ -55,11 +55,13 @@ MyCoder includes a GitHub mode that enables the agent to work with GitHub issues
 To enable GitHub mode:
 
 1. Via CLI option (overrides config file):
+
 ```bash
 mycoder --githubMode true
 ```
 
 2. Via configuration file:
+
 ```js
 // mycoder.config.js
 export default {
@@ -71,11 +73,13 @@ export default {
 To disable GitHub mode:
 
 1. Via CLI option:
+
 ```bash
 mycoder --githubMode false
 ```
 
 2. Via configuration file:
+
 ```js
 // mycoder.config.js
 export default {
@@ -158,7 +162,7 @@ Or in your configuration file:
 export default {
   // Model settings
   provider: 'anthropic',
-  model: 'claude-3-7-sonnet-20250219',  // or any other Anthropic model
+  model: 'claude-3-7-sonnet-20250219', // or any other Anthropic model
   // other configuration options...
 };
 ```
@@ -186,14 +190,15 @@ Example configuration in `mycoder.config.js`:
 // mycoder.config.js
 export default {
   // Browser settings
-  headless: false,  // Show browser UI
-  userSession: true,  // Use existing browser session
-  pageFilter: 'readability',  // Use readability for webpage processing
-  
+  headless: false, // Show browser UI
+  userSession: true, // Use existing browser session
+  pageFilter: 'readability', // Use readability for webpage processing
+
   // Custom settings
-  customPrompt: "Always prioritize readability and simplicity in your code. Prefer TypeScript over JavaScript when possible.",
-  tokenCache: false,  // Disable token caching for LLM API calls
-  
+  customPrompt:
+    'Always prioritize readability and simplicity in your code. Prefer TypeScript over JavaScript when possible.',
+  tokenCache: false, // Disable token caching for LLM API calls
+
   // Other configuration options...
 };
 ```
diff --git a/packages/cli/src/commands/$default.ts b/packages/cli/src/commands/$default.ts
@@ -155,8 +155,6 @@ export const command: CommandModule<SharedOptions, DefaultArgs> = {
         temperature: config.temperature,
       };
 
-      console.log('agentConfig', agentConfig);
-
       const result = await toolAgent(prompt, tools, agentConfig, {
         logger,
         headless: config.headless,
diff --git a/packages/cli/src/settings/config.ts b/packages/cli/src/settings/config.ts
@@ -93,26 +93,18 @@ export async function loadConfig(
     searchStrategy: 'global',
   });
 
-  console.log({ cwd: process.cwd() });
-
   // Search for configuration file
   const result = await explorer.search();
-  console.log({ explorerResult: result });
 
   // Merge configurations with precedence: default < file < cli
   const fileConfig = result?.config || {};
 
-  console.log({ defaultConfig });
-  console.log({ fileConfig });
-  console.log({ cliOptions });
-
   // Return merged configuration
   const mergedConfig = {
     ...defaultConfig,
     ...removeUndefined(fileConfig),
     ...removeUndefined(cliOptions),
   };
-  console.log({ mergedConfig });
   return mergedConfig;
 }
 
