fix: correct syntax errors in model context window detection

bhouston · bhouston · commit a9fc083e8562 · 2025-03-21T21:04:57.000-04:00
diff --git a/packages/agent/CHANGELOG.md b/packages/agent/CHANGELOG.md
@@ -1,15 +1,13 @@
 # [mycoder-agent-v1.7.0](https://github.com/drivecore/mycoder/compare/mycoder-agent-v1.6.0...mycoder-agent-v1.7.0) (2025-03-21)
 
-
 ### Bug Fixes
 
-* Fix TypeScript errors and tests for message compaction feature ([d4f1fb5](https://github.com/drivecore/mycoder/commit/d4f1fb5d197e623bf98f2221352f9132dcb3e5de))
-
+- Fix TypeScript errors and tests for message compaction feature ([d4f1fb5](https://github.com/drivecore/mycoder/commit/d4f1fb5d197e623bf98f2221352f9132dcb3e5de))
 
 ### Features
 
-* Add automatic compaction of historical messages for agents ([a5caf46](https://github.com/drivecore/mycoder/commit/a5caf464a0a8dca925c7b46023ebde4727e211f8)), closes [#338](https://github.com/drivecore/mycoder/issues/338)
-* Improve message compaction with proactive suggestions ([6276bc0](https://github.com/drivecore/mycoder/commit/6276bc0bc5fa27c4f1e9be61ff4375690ad04c62))
+- Add automatic compaction of historical messages for agents ([a5caf46](https://github.com/drivecore/mycoder/commit/a5caf464a0a8dca925c7b46023ebde4727e211f8)), closes [#338](https://github.com/drivecore/mycoder/issues/338)
+- Improve message compaction with proactive suggestions ([6276bc0](https://github.com/drivecore/mycoder/commit/6276bc0bc5fa27c4f1e9be61ff4375690ad04c62))
 
 # [mycoder-agent-v1.6.0](https://github.com/drivecore/mycoder/compare/mycoder-agent-v1.5.0...mycoder-agent-v1.6.0) (2025-03-21)
 
diff --git a/packages/agent/src/core/llm/providers/anthropic.ts b/packages/agent/src/core/llm/providers/anthropic.ts
@@ -154,29 +154,69 @@ export class AnthropicProvider implements LLMProvider {
     });
 
     // Initialize model context window detection
-    this.initializeModelContextWindow();
+    // This is async but we don't need to await it here
+    // If it fails, we'll fall back to hardcoded limits
+    this.initializeModelContextWindow().catch((error) => {
+      console.warn(
+        `Failed to initialize model context window: ${error.message}`,
+      );
+    });
   }
 
   /**
    * Fetches the model context window size from the Anthropic API
+   *
+   * @returns The context window size if successfully fetched, otherwise undefined
    */
-  private async initializeModelContextWindow(): Promise<void> {
+  private async initializeModelContextWindow(): Promise<number | undefined> {
     try {
       const response = await this.client.models.list();
-      const model = response.data.find((m) => m.id === this.model);
+
+      if (!response?.data || !Array.isArray(response.data)) {
+        console.warn(`Invalid response from models.list() for ${this.model}`);
+        return undefined;
+      }
+
+      // Try to find the exact model
+      let model = response.data.find((m) => m.id === this.model);
+
+      // If not found, try to find a model that starts with the same name
+      // This helps with model aliases like 'claude-3-sonnet-latest'
+      if (!model) {
+        // Split by '-latest' or '-20' to get the base model name
+        const parts = this.model.split('-latest');
+        const modelPrefix =
+          parts.length > 1 ? parts[0] : this.model.split('-20')[0];
+
+        if (modelPrefix) {
+          model = response.data.find((m) => m.id.startsWith(modelPrefix));
+
+          if (model) {
+            console.info(
+              `Model ${this.model} not found, using ${model.id} for context window size`,
+            );
+          }
+        }
+      }
 
       // Using type assertion to access context_window property
       // The Anthropic API returns context_window but it may not be in the TypeScript definitions
       if (model && 'context_window' in model) {
-        this.modelContextWindow = (model as any).context_window;
+        const contextWindow = (model as any).context_window;
+        this.modelContextWindow = contextWindow;
         // Cache the result for future use
-        modelContextWindowCache[this.model] = (model as any).context_window;
+        modelContextWindowCache[this.model] = contextWindow;
+        return contextWindow;
+      } else {
+        console.warn(`No context window information found for ${this.model}`);
+        return undefined;
       }
     } catch (error) {
       console.warn(
         `Failed to fetch model context window for ${this.model}: ${(error as Error).message}`,
       );
       // Will fall back to hardcoded limits
+      return undefined;
     }
   }
 
