Conversation
- also allowed pre release of sarvam 0.1.26a2
📝 WalkthroughWalkthroughThe pipecat-ai dependency version in the call processing module's pyproject.toml was updated from 0.0.102 to 0.0.103. This is a patch-level version bump with all associated feature extras maintained. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
wavefront/server/apps/call_processing/pyproject.toml (1)
24-24: Transitive dependency on a pre-releasesarvampackage (0.1.26a2) in production.The PR description states that this upgrade "allows the pre-release of sarvam version 0.1.26a2." This means the
sarvamextra insidepipecat-ai==0.0.103now resolves to a pre-release (a2) build of the Sarvam SDK. Pre-release packages carry no stability guarantees and are generally unsuitable for production workloads. If thesarvamintegration is actively used, monitor for regressions and plan a follow-up pin once a stable0.1.26is available.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@wavefront/server/apps/call_processing/pyproject.toml` at line 24, The pipecat-ai extra list in pyproject.toml currently pulls a pre-release sarvam (0.1.26a2) via "pipecat-ai[...sarvam...]==0.0.103"; update the dependency resolution to avoid pre-releases by either removing the sarvam extra from the pipecat-ai extras or explicitly pining sarvam to a stable version (e.g., sarvam==0.1.26 when available) or adding a non-prerelease constraint (e.g., exclude alpha/beta/prerelease versions) so CI and production do not resolve to 0.1.26a2, and ensure package metadata (pyproject.toml) reflects that change.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
wavefront/server/uv.lockis excluded by!**/*.lock
📒 Files selected for processing (1)
wavefront/server/apps/call_processing/pyproject.toml
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@wavefront/server/apps/call_processing/pyproject.toml`:
- Line 24: The pipecat-ai extra list in pyproject.toml currently pulls a
pre-release sarvam (0.1.26a2) via "pipecat-ai[...sarvam...]==0.0.103"; update
the dependency resolution to avoid pre-releases by either removing the sarvam
extra from the pipecat-ai extras or explicitly pining sarvam to a stable version
(e.g., sarvam==0.1.26 when available) or adding a non-prerelease constraint
(e.g., exclude alpha/beta/prerelease versions) so CI and production do not
resolve to 0.1.26a2, and ensure package metadata (pyproject.toml) reflects that
change.
Summary by CodeRabbit