Skip to content

fixed openrouter#7

Merged
bmdavis419 merged 2 commits intomainfrom
staging
Dec 19, 2025
Merged

fixed openrouter#7
bmdavis419 merged 2 commits intomainfrom
staging

Conversation

@bmdavis419
Copy link
Collaborator

@bmdavis419 bmdavis419 commented Dec 19, 2025

Greptile Summary

Updated OpenRouter SDK configuration to wrap provider options in extraBody object, resolving API compatibility issues. Added Cerebras as an additional inference provider option alongside Groq.

  • Fixed OpenRouter model configuration by moving provider options into extraBody wrapper
  • Added cerebras to provider fallback list alongside existing groq provider
  • Removed TypeScript error suppressions that are no longer needed with correct API structure

Confidence Score: 4/5

  • This PR is safe to merge with low risk - it fixes a broken OpenRouter SDK configuration
  • The changes correctly update the OpenRouter SDK API structure from provider to extraBody.provider, which appears to be required for v1.5.4 compatibility. The commit message "fixed openrouter" indicates this addresses a broken state. Removal of TypeScript error suppressions confirms proper type alignment. The addition of Cerebras provider is additive and non-breaking. Score is 4/5 rather than 5/5 because there are no automated tests to verify the AI service still functions correctly with these changes, so manual testing is recommended to ensure comment classification and sponsor detection features work as expected.
  • Test packages/channel-sync/src/ai/index.ts to verify AI inference calls (comment classification and sponsor detection) work correctly with the new API structure

Important Files Changed

Filename Overview
packages/channel-sync/src/ai/index.ts Fixed OpenRouter SDK configuration by wrapping provider options in extraBody, added Cerebras provider, removed TypeScript error suppressions

@bmdavis419 bmdavis419 merged commit 57f49f3 into main Dec 19, 2025
2 of 3 checks passed
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.

1 participant