🔧 Simplify token limit handling across providers

Remove direct token limit parameter handling in provider configurations

This change streamlines how token limits are managed across different LLM
providers by:

- Removing token_limit injection in ProviderConfig to LLMProviderConfig
  conversion
- Adding explicit token_limit parameter filtering in OpenAI provider
- Removing special Claude 3.7 token limit handling logic

These changes make token limit handling more consistent and maintainable
while preserving the functionality through provider-specific implementations.

Author: hyperb1iss

src/config.rs

@@ -247,12 +247,7 @@ impl ProviderConfig {
 
     /// Convert to `LLMProviderConfig`
     pub fn to_llm_provider_config(&self) -> LLMProviderConfig {
-        let mut additional_params = self.additional_params.clone();
-
-        // Add token limit to additional params if set
-        if let Some(limit) = self.token_limit {
-            additional_params.insert("token_limit".to_string(), limit.to_string());
-        }
+        let additional_params = self.additional_params.clone();
 
         LLMProviderConfig {
             api_key: self.api_key.clone(),

src/llm_providers/claude.rs

@@ -35,10 +35,6 @@ impl LLMProvider for ClaudeProvider {
 
         // Add additional parameters from the configuration
         for (key, value) in &self.config.additional_params {
-            // Skip token_limit parameter for Claude 3.7 models
-            if key == "token_limit" && self.config.model.contains("claude-3-7") {
-                continue;
-            }
             request_body[key] = serde_json::Value::String(value.clone());
         }
 

src/llm_providers/openai.rs

@@ -34,6 +34,10 @@ impl LLMProvider for OpenAIProvider {
 
         // Add additional parameters from the configuration
         for (key, value) in &self.config.additional_params {
+            // Skip token_limit parameter as it's for internal use only
+            if key == "token_limit" {
+                continue;
+            }
             request_body[key] = serde_json::Value::String(value.clone());
         }