fix(models): add request sanitization

[waleedlatif1]

Summary

• add request sanitization

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Feb 8, 2026 3:04am

[greptile-apps]

Greptile Overview

Greptile Summary

Extended request sanitization to handle model-specific parameters (reasoningEffort, verbosity, thinkingLevel) in addition to existing temperature handling. The sanitizeRequest function now removes these parameters when models don't support them, preventing API errors when unsupported parameters are sent to provider APIs.

Key changes:

• Added three new capability check functions in utils.ts that follow the existing pattern (checking against model lists with case-insensitive comparison)
• Extended sanitizeRequest in index.ts to filter out reasoningEffort, verbosity, and thinkingLevel for models that don't support these features
• Added comprehensive test coverage for all three new capability functions

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The implementation follows established patterns in the codebase (consistent with existing supportsTemperature function), includes comprehensive test coverage for all new functions, and the changes are straightforward parameter filtering that prevents API errors without affecting logic
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/providers/index.ts	Extended sanitizeRequest to remove unsupported model parameters (reasoningEffort, verbosity, thinkingLevel) based on model capabilities
apps/sim/providers/utils.ts	Added three new capability check functions (supportsReasoningEffort, supportsVerbosity, supportsThinking) following existing patterns
apps/sim/providers/utils.test.ts	Added comprehensive test coverage for the three new capability functions with positive, negative, and case-insensitive test cases

Sequence Diagram

sequenceDiagram
    participant Client
    participant ProvidersIndex as providers/index.ts
    participant SanitizeRequest as sanitizeRequest()
    participant Utils as providers/utils.ts
    participant Provider as Provider Executor

    Client->>ProvidersIndex: executeRequest(request)
    ProvidersIndex->>SanitizeRequest: sanitizeRequest(request)
    
    SanitizeRequest->>Utils: supportsTemperature(model)
    Utils-->>SanitizeRequest: boolean
    alt !supportsTemperature
        SanitizeRequest->>SanitizeRequest: temperature = undefined
    end
    
    SanitizeRequest->>Utils: supportsReasoningEffort(model)
    Utils-->>SanitizeRequest: boolean
    alt !supportsReasoningEffort
        SanitizeRequest->>SanitizeRequest: reasoningEffort = undefined
    end
    
    SanitizeRequest->>Utils: supportsVerbosity(model)
    Utils-->>SanitizeRequest: boolean
    alt !supportsVerbosity
        SanitizeRequest->>SanitizeRequest: verbosity = undefined
    end
    
    SanitizeRequest->>Utils: supportsThinking(model)
    Utils-->>SanitizeRequest: boolean
    alt !supportsThinking
        SanitizeRequest->>SanitizeRequest: thinkingLevel = undefined
    end
    
    SanitizeRequest-->>ProvidersIndex: sanitizedRequest
    ProvidersIndex->>Provider: executeRequest(sanitizedRequest)
    Provider-->>Client: response

Loading

[greptile-apps]

_{3 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}