CU-86evwqnf4 Add provider options support

[june-tawk]

Summary by CodeRabbit

• New Features

  • Added support for provider-specific configuration options in model settings, enabling enhanced customization of AI model behavior based on the selected provider.

• Chores

  • Removed debug initialization log.
  • Optimized internal content transformation handling.

[coderabbitai]

Walkthrough

Added optional providerOptions field to agent model settings configuration and propagated it through the model invocation pipeline. Modified tool-call content assembly to use object spreading instead of explicit field mapping. Removed a startup log statement from Langfuse initialization.

Changes

Cohort / File(s)	Summary
Agent Configuration src/core/agent/types.ts	Added optional providerOptions field to modelSettings to accept provider-specific configuration as nested key-value pairs.
Model Execution Pipeline src/core/execution.ts, src/core/runner.ts	Updated tool-call content transformation to spread original part properties; passed providerOptions from agent model settings into generateText invocation.
Logging src/lifecycle/langfuse/index.ts	Removed runtime log message on successful Langfuse initialization.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A config field springs forth with care,
Through pipelines flowing, options fair,
The spread unfolds what once was bound,
While logs fade silent, not a sound.
Provider magic now takes flight! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Add provider options support' clearly and concisely describes the main change: adding a new providerOptions field to enable provider-specific configuration options across the codebase.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch CU-86evwqnf4-add-support-for-provider-options

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/core/agent/types.ts (1)

132-132: Consider adding documentation for providerOptions.

The type definition is correct. For consistency with other modelSettings fields documented in the JSDoc block (lines 84-89), consider adding a brief description:

📝 Suggested documentation addition

Add to the JSDoc block around line 89:

 * `@property` {number} [modelSettings.frequencyPenalty] - Frequency penalty (-2 to 2)
 * `@property` {Record<string, Record<string, unknown>>} [modelSettings.providerOptions] - Provider-specific options (e.g., { openai: { ... } })

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/core/agent/types.ts` at line 132, Add a short JSDoc entry for the
existing providerOptions field under the modelSettings block to match the
documented style for other fields: document providerOptions as {Record<string,
Record<string, unknown>>} with a brief description like "Provider-specific
options (e.g., { openai: { ... } })" and place it alongside the existing
`@property` lines for modelSettings so the type providerOptions is clearly
explained; reference the providerOptions symbol in src/core/agent/types.ts and
the surrounding modelSettings JSDoc block when making the update.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/core/agent/types.ts`:
- Line 132: Add a short JSDoc entry for the existing providerOptions field under
the modelSettings block to match the documented style for other fields: document
providerOptions as {Record<string, Record<string, unknown>>} with a brief
description like "Provider-specific options (e.g., { openai: { ... } })" and
place it alongside the existing `@property` lines for modelSettings so the type
providerOptions is clearly explained; reference the providerOptions symbol in
src/core/agent/types.ts and the surrounding modelSettings JSDoc block when
making the update.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3c20bee and 0863d84.

⛔ Files ignored due to path filters (17)

• dist/core/agent/agent-class.d.ts is excluded by !**/dist/**
• dist/core/agent/agent-class.d.ts.map is excluded by !**/dist/**, !**/*.map
• dist/core/agent/types.d.ts is excluded by !**/dist/**
• dist/core/agent/types.d.ts.map is excluded by !**/dist/**, !**/*.map
• dist/core/execution.d.ts.map is excluded by !**/dist/**, !**/*.map
• dist/core/execution.js is excluded by !**/dist/**
• dist/core/execution.js.map is excluded by !**/dist/**, !**/*.map
• dist/core/runner.d.ts.map is excluded by !**/dist/**, !**/*.map
• dist/core/runner.js is excluded by !**/dist/**
• dist/core/runner.js.map is excluded by !**/dist/**, !**/*.map
• dist/guardrails/index.js is excluded by !**/dist/**
• dist/guardrails/index.js.map is excluded by !**/dist/**, !**/*.map
• dist/guardrails/utils.js is excluded by !**/dist/**
• dist/guardrails/utils.js.map is excluded by !**/dist/**, !**/*.map
• dist/lifecycle/langfuse/index.d.ts.map is excluded by !**/dist/**, !**/*.map
• dist/lifecycle/langfuse/index.js is excluded by !**/dist/**
• dist/lifecycle/langfuse/index.js.map is excluded by !**/dist/**, !**/*.map

📒 Files selected for processing (4)

• src/core/agent/types.ts
• src/core/execution.ts
• src/core/runner.ts
• src/lifecycle/langfuse/index.ts

💤 Files with no reviewable changes (1)

• src/lifecycle/langfuse/index.ts