Skip to content

Conversation

@0xApotheosis
Copy link
Member

@0xApotheosis 0xApotheosis commented Dec 10, 2025

Summary by CodeRabbit

  • Refactor

    • Improved AI provider configuration management for more consistent provider selection.
  • Bug Fixes

    • Updated Venice model to a newer, faster variant for better performance.
    • Adjusted Venice behavior to disable internal reasoning traces and skip its system prompt when Venice is the selected provider, improving output clarity.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Dec 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
shapeshift-agentic Ready Ready Preview Comment Dec 10, 2025 3:42am

@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Added getProviderName() to centralize provider selection (defaults to 'anthropic'); changed Venice model from claude-opus-45 to grok-41-fast; chat streaming now conditionally injects Venice-specific providerOptions (disable internal thinking, omit Venice system prompt) when provider is 'venice'.

Changes

Cohort / File(s) Summary
Provider configuration & model update
apps/agentic-server/src/models.ts
Added getProviderName(): AIProvider (reads AI_PROVIDER, defaults to 'anthropic'); refactored getModel() to call it; changed Venice model from claude-opus-45 to grok-41-fast.
Chat route — conditional Venice options
apps/agentic-server/src/routes/chat.ts
Import getProviderName() and conditionally pass providerOptions to the Venice streamText call to disable internal reasoning and omit the Venice system prompt only when provider is 'venice'.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify getProviderName() correctly reads AI_PROVIDER and default behavior.
  • Confirm getModel() mapping still returns intended models for all providers.
  • Ensure the Venice providerOptions are only applied when provider equals 'venice'.
  • Check the Venice model identifier change (grok-41-fast) is consistent with deployment/config.

Possibly related PRs

  • feat: add venice support #142 — Overlaps changes to apps/agentic-server/src/models.ts and routes/chat.ts around provider selection and Venice support.

Poem

🐰 I hopped through env vars, pulled out a name,
Venice now whispers, no internal game,
Models swapped shoes — a swift, bright new cast,
Streams tidy up, prompts tucked in fast,
I thump my foot and celebrate the change at last.

Pre-merge checks and finishing touches

✅ 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 'feat: use grok 4.1, no thinking' accurately reflects the main changes: updating the Venice model to grok-4.1-fast and adding Venice-specific options to disable reasoning.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 536ed51 and c416fc9.

📒 Files selected for processing (2)
  • apps/agentic-server/src/models.ts (1 hunks)
  • apps/agentic-server/src/routes/chat.ts (2 hunks)

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

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

@premiumjibles premiumjibles merged commit f4f80fb into main Dec 10, 2025
1 of 3 checks passed
@premiumjibles premiumjibles deleted the grok branch December 10, 2025 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants