Worker build #3157
Worker build #3157
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
Added support for X.ai (formerly Twitter) provider across multiple files, integrating cost tracking and API handling capabilities.
- Added X.ai provider configuration in
/bifrost/packages/cost/providers/mappings.tswith regex pattern^https:\/\/api\.x\.ai/and associated costs - Added four new X.ai models (grok-beta, grok-vision-beta, grok-2-1212, grok-2-vision-1212) with pricing in cost calculations
- Updated database queries in
/worker/src/lib/RequestWrapper.tsto use booleanfalseinstead of string"false"forsoft_deletefield - Extended API specification in swagger.json and routes.ts to include X provider in ProviderName enum
8 file(s) reviewed, 8 comment(s)
Edit PR Review Bot Settings | Greptile
| @@ -52,6 +53,8 @@ const firecrawl = /^https:\/\/api\.firecrawl\.dev/; | |||
| const awsBedrock = /^https:\/\/bedrock-runtime\.[a-z0-9-]+\.amazonaws\.com\/.*/; | |||
| // https://api.deepseek.com | |||
| const deepseek = /^https:\/\/api\.deepseek\.com/; | |||
| // https://api.x.ai | |||
| const x = /^https:\/\/api\.x\.ai/; | |||
There was a problem hiding this comment.
style: regex pattern for x.ai should include optional 'api.' subdomain for consistency with other patterns like openRouter
| @@ -52,6 +53,8 @@ const firecrawl = /^https:\/\/api\.firecrawl\.dev/; | |||
| const awsBedrock = /^https:\/\/bedrock-runtime\.[a-z0-9-]+\.amazonaws\.com\/.*/; | |||
| // https://api.deepseek.com | |||
| const deepseek = /^https:\/\/api\.deepseek\.com/; | |||
| // https://api.x.ai | |||
| const x = /^https:\/\/api\.x\.ai/; | |||
There was a problem hiding this comment.
style: regex pattern for x.ai should include optional 'api.' prefix for consistency with openRouter pattern
| { | ||
| pattern: x, | ||
| provider: "X", | ||
| costs: xCosts, | ||
| }, |
There was a problem hiding this comment.
style: X provider configuration should be placed after AWS and before DEEPSEEK to maintain alphabetical ordering with other providers
| { | ||
| pattern: x, | ||
| provider: "X", | ||
| costs: xCosts, | ||
| }, |
There was a problem hiding this comment.
style: X provider entry should be placed at the end of the providers array to maintain alphabetical ordering with other providers
| @@ -52,6 +53,8 @@ const firecrawl = /^https:\/\/api\.firecrawl\.dev/; | |||
| const awsBedrock = /^https:\/\/bedrock-runtime\.[a-z0-9-]+\.amazonaws\.com\/.*/; | |||
| // https://api.deepseek.com | |||
| const deepseek = /^https:\/\/api\.deepseek\.com/; | |||
| // https://api.x.ai | |||
| const x = /^https:\/\/api\.x\.ai/; | |||
There was a problem hiding this comment.
logic: regex pattern should include optional 'api.' prefix for consistency with other patterns like openRouter
| { | ||
| pattern: x, | ||
| provider: "X", | ||
| costs: xCosts, | ||
| }, |
There was a problem hiding this comment.
style: X provider entry should be placed at the end of the providers array for consistency with providersNames order
| @@ -52,6 +53,8 @@ const firecrawl = /^https:\/\/api\.firecrawl\.dev/; | |||
| const awsBedrock = /^https:\/\/bedrock-runtime\.[a-z0-9-]+\.amazonaws\.com\/.*/; | |||
| // https://api.deepseek.com | |||
| const deepseek = /^https:\/\/api\.deepseek\.com/; | |||
| // https://api.x.ai | |||
| const x = /^https:\/\/api\.x\.ai/; | |||
There was a problem hiding this comment.
logic: URL pattern regex should include end-of-string anchor ($) to prevent partial matches
| { | ||
| pattern: x, | ||
| provider: "X", | ||
| costs: xCosts, | ||
| }, |
There was a problem hiding this comment.
style: X provider configuration should include modelDetails property for consistency with other major providers like OpenAI and Anthropic
No description provided.