Skip to content

fix(inference): prefer inference credentials for interruption detector#1486

Open
GaureeshAnvekar wants to merge 2 commits into
livekit:mainfrom
GaureeshAnvekar:fix-inference-api-key
Open

fix(inference): prefer inference credentials for interruption detector#1486
GaureeshAnvekar wants to merge 2 commits into
livekit:mainfrom
GaureeshAnvekar:fix-inference-api-key

Conversation

@GaureeshAnvekar
Copy link
Copy Markdown

@GaureeshAnvekar GaureeshAnvekar commented May 13, 2026

Description

Fixes adaptive interruption detector credential precedence when using a self-hosted LiveKit SFU with LiveKit Cloud Inference. Issue #1353

Previously, adaptive interruption could pick up LIVEKIT_API_KEY / LIVEKIT_API_SECRET before LIVEKIT_INFERENCE_API_KEY / LIVEKIT_INFERENCE_API_SECRET as described in the issue, causing Cloud inference websocket auth to fail with 401 when the SFU credentials were local/self-hosted credentials.

Changes Made

  • Prefer inference-specific env vars for adaptive interruption auth.
  • Preserve fallback to regular LiveKit credentials when inference-specific credentials are not set.

Pre-Review Checklist

  • Build passes: All builds (lint, typecheck, tests) pass locally
  • AI-generated code reviewed: Removed unnecessary comments and ensured code quality
  • Changes explained: All changes are properly documented and justified above
  • Scope appropriate: All changes relate to the PR title, or explanations provided for why they're included

Testing

  • All tests pass

Additional Notes

Before change (local SFU):

Screenshot 2026-05-12 at 8 12 41 PM

After change (local SFU):

Screenshot 2026-05-12 at 8 23 41 PM

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 13, 2026

⚠️ No Changeset found

Latest commit: 479ffa1

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

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 13, 2026

CLA assistant check
All committers have signed the CLA.

devin-ai-integration[bot]

This comment was marked as resolved.

@GaureeshAnvekar GaureeshAnvekar force-pushed the fix-inference-api-key branch from 6dac940 to 46e8bc9 Compare May 17, 2026 02:44
@GaureeshAnvekar GaureeshAnvekar force-pushed the fix-inference-api-key branch from 46e8bc9 to 479ffa1 Compare May 18, 2026 02:37
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.

2 participants