Fix #115: Remove hardcoded Anthropic API key check and add provider-specific error messages

Author: bhouston

packages/agent/src/core/toolAgent/toolAgentCore.ts

@@ -1,7 +1,5 @@
 import { CoreMessage, ToolSet, generateText, tool as makeTool } from 'ai';
 
-import { getAnthropicApiKeyError } from '../../utils/errors.js';
-
 import { DEFAULT_CONFIG } from './config.js';
 import {
   addCacheControlToMessages,
@@ -30,9 +28,6 @@ export const toolAgent = async (
 
   let interactions = 0;
 
-  const apiKey = process.env.ANTHROPIC_API_KEY;
-  if (!apiKey) throw new Error(getAnthropicApiKeyError());
-
   const messages: CoreMessage[] = [
     {
       role: 'user',

packages/agent/src/utils/errors.ts

@@ -1,10 +1,47 @@
 export const getAnthropicApiKeyError = () => `
 Error: ANTHROPIC_API_KEY environment variable is not set
 
-Before using MyCoder, you must have an ANTHROPIC_API_KEY specified either:
+Before using MyCoder with Anthropic models, you must have an ANTHROPIC_API_KEY specified either:
 
 - As an environment variable, "export ANTHROPIC_API_KEY=[your-api-key]" or
 - In a .env file in the folder you run "mycoder" from
 
 Get an API key from https://www.anthropic.com/api
+For setup instructions, visit: https://mycoder.ai/docs/getting-started/anthropic
+`;
+
+export const getOpenAIApiKeyError = () => `
+Error: OPENAI_API_KEY environment variable is not set
+
+Before using MyCoder with OpenAI models, you must have an OPENAI_API_KEY specified either:
+
+- As an environment variable, "export OPENAI_API_KEY=[your-api-key]" or
+- In a .env file in the folder you run "mycoder" from
+
+Get an API key from https://platform.openai.com/api-keys
+For setup instructions, visit: https://mycoder.ai/docs/getting-started/openai
+`;
+
+export const getXAIApiKeyError = () => `
+Error: XAI_API_KEY environment variable is not set
+
+Before using MyCoder with xAI models, you must have an XAI_API_KEY specified either:
+
+- As an environment variable, "export XAI_API_KEY=[your-api-key]" or
+- In a .env file in the folder you run "mycoder" from
+
+Get an API key from https://platform.xai.com
+For setup instructions, visit: https://mycoder.ai/docs/getting-started/xai
+`;
+
+export const getMistralApiKeyError = () => `
+Error: MISTRAL_API_KEY environment variable is not set
+
+Before using MyCoder with Mistral models, you must have a MISTRAL_API_KEY specified either:
+
+- As an environment variable, "export MISTRAL_API_KEY=[your-api-key]" or
+- In a .env file in the folder you run "mycoder" from
+
+Get an API key from https://console.mistral.ai/api-keys/
+For setup instructions, visit: https://mycoder.ai/docs/getting-started/mistral
 `;

packages/cli/src/commands/$default.ts

@@ -7,6 +7,9 @@ import {
   Logger,
   getTools,
   getAnthropicApiKeyError,
+  getOpenAIApiKeyError,
+  getXAIApiKeyError,
+  getMistralApiKeyError,
   userPrompt,
   LogLevel,
   subAgentTool,
@@ -102,25 +105,16 @@ export const command: CommandModule<SharedOptions, DefaultArgs> = {
         userModelProvider === 'openai' &&
         !process.env.OPENAI_API_KEY
       ) {
-        logger.error(
-          'No OpenAI API key found. Please set the OPENAI_API_KEY environment variable.',
-          'You can get an API key from https://platform.openai.com/api-keys',
-        );
+        logger.error(getOpenAIApiKeyError());
         throw new Error('OpenAI API key not found');
       } else if (userModelProvider === 'xai' && !process.env.XAI_API_KEY) {
-        logger.error(
-          'No xAI API key found. Please set the XAI_API_KEY environment variable.',
-          'You can get an API key from https://platform.xai.com',
-        );
+        logger.error(getXAIApiKeyError());
         throw new Error('xAI API key not found');
       } else if (
         userModelProvider === 'mistral' &&
         !process.env.MISTRAL_API_KEY
       ) {
-        logger.error(
-          'No Mistral API key found. Please set the MISTRAL_API_KEY environment variable.',
-          'You can get an API key from https://console.mistral.ai/api-keys/',
-        );
+        logger.error(getMistralApiKeyError());
         throw new Error('Mistral API key not found');
       }
       // No API key check needed for Ollama as it uses a local server