Fix ollama support with AI SDK

[shrey150]

Summary

Ollama models now work with the AI SDK provider format (e.g., ollama/qwen3:1.7b). Previously, using ollama would fail with UnsupportedAISDKModelProviderError. Closes #1164

Changes

• Filter out undefined values when checking if clientOptions has meaningful data in getAISDKLanguageModel
• Skip error logging for providers that intentionally don't require API keys (like ollama)

Testing

Tested with model: "ollama/qwen3:1.7b" and verified Stagehand initializes successfully without errors.

---

Summary by cubic

Enable Ollama models to work with the AI SDK provider format (e.g., ollama/qwen3:1.7b). Fixes initialization errors and removes unnecessary API key warnings.

• Bug Fixes
  • Treat clientOptions as present only when values are non-null/non-undefined.
  • Skip "unknown env var" logging for providers that don’t require API keys (ollama).
  • Use createOllama to support custom baseURL and cases where an apiKey is provided.

^{Written for commit d3001d5. Summary will update on new commits.}

[changeset-bot]

⚠️ No Changeset found

Latest commit: d3001d5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[cubic-dev-ai]

No issues found across 2 files

[greptile-apps]

Greptile Summary

Fixes Ollama support by properly detecting when clientOptions contains meaningful configuration data and suppressing unnecessary API key warnings for providers that don't require them.

Key Changes:

• LLMProvider.ts: Changed validation from checking if clientOptions has keys to checking if it has actual non-null/undefined values using Object.values().some(), preventing the function from incorrectly routing to the API-key provider path when all values are undefined
• utils.ts: Added providersWithoutApiKey set containing "ollama" to skip warning logs for providers that intentionally don't require API keys

The root cause was that when using format like ollama/qwen3:1.7b, the code would pass a clientOptions object with undefined values, and the old check (Object.keys(clientOptions).length > 0) would incorrectly return true, attempting to use the non-existent AISDKProvidersWithAPIKey["ollama"] instead of the correct AISDKProviders["ollama"].

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The changes are minimal, well-targeted, and fix a clear bug without introducing new logic complexity. Both fixes address the exact root cause (undefined values being treated as meaningful data), and the implementation is straightforward with no edge cases or side effects.
• No files require special attention

Important Files Changed

Filename	Overview
packages/core/lib/v3/llm/LLMProvider.ts	Fixed clientOptions validation to check for meaningful values instead of just keys, preventing undefined values from triggering the wrong provider path
packages/core/lib/utils.ts	Added providersWithoutApiKey set to suppress warning logs for providers like ollama that don't require API keys

Sequence Diagram

sequenceDiagram
    participant User
    participant LLMProvider
    participant getAISDKLanguageModel
    participant AISDKProviders
    participant AISDKProvidersWithAPIKey
    participant loadApiKeyFromEnv
    
    User->>LLMProvider: getClient("ollama/qwen3:1.7b", clientOptions)
    LLMProvider->>getAISDKLanguageModel: call with subProvider="ollama", clientOptions
    
    Note over getAISDKLanguageModel: NEW: Check if clientOptions<br/>has meaningful values
    getAISDKLanguageModel->>getAISDKLanguageModel: Object.values(clientOptions)<br/>.some(v !== undefined)
    
    alt clientOptions has meaningful values
        getAISDKLanguageModel->>AISDKProvidersWithAPIKey: lookup provider with API key
        AISDKProvidersWithAPIKey-->>getAISDKLanguageModel: throw error (ollama not in map)
    else no meaningful values (ollama case)
        getAISDKLanguageModel->>AISDKProviders: lookup default provider
        AISDKProviders-->>getAISDKLanguageModel: return ollama provider
    end
    
    getAISDKLanguageModel-->>LLMProvider: return language model
    
    User->>loadApiKeyFromEnv: load API key for "ollama"
    Note over loadApiKeyFromEnv: NEW: Check if provider<br/>needs API key
    loadApiKeyFromEnv->>loadApiKeyFromEnv: providersWithoutApiKey.has("ollama")
    alt provider needs API key
        loadApiKeyFromEnv->>loadApiKeyFromEnv: log warning if env var not found
    else provider doesn't need API key (ollama)
        loadApiKeyFromEnv->>loadApiKeyFromEnv: skip warning log
    end
    loadApiKeyFromEnv-->>User: return undefined (no error)

Loading