-
Notifications
You must be signed in to change notification settings - Fork 168
Dev Merge #90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dev Merge #90
Conversation
…ng effort and verbosity fields
Feature/gpt 5
…riable-support added support for vault keys usage inside NodeJS code component
Caution Review failedThe pull request is closed. WalkthroughVersion bumps in core and SDK packages. Core adds Node >=20 engine and updates OpenAI dependency. New verbosity setting and expanded reasoningEffort in GenAILLM. AWS Lambda helper and deploy flow now handle environment variables end-to-end and include them in hashing and packaging. LLM connectors add GPT-5 fields, reasoning validation, and streaming usage refactors; utilities and types updated; index re-exports utils. Changes
Sequence Diagram(s)sequenceDiagram
participant CM as ComputeManager
participant H as AWSLambdaCode.helper
participant AWS as AWS Lambda
CM->>H: getCurrentEnvironmentVariables(agentTeamId, code)
H-->>CM: envVariables (Record<string,string>)
CM->>H: getSortedObjectValues(envVariables)
H-->>CM: envValues[]
CM->>H: generateCodeHash(codeBody, codeInputs, envValues)
H-->>CM: codeHash
CM->>H: generateLambdaCode(code, parameters, envVariables)
H-->>CM: zipFilePath
CM->>H: createOrUpdateLambdaFunction(name, zipFilePath, awsConfigs, envVariables)
H->>AWS: Create/UpdateFunction + UpdateFunctionConfiguration (Environment)
AWS-->>H: ARNs/status
H-->>CM: result
sequenceDiagram
participant Client
participant API as OpenAI API Interface
participant Stream as processStream
participant Util as reportUsageStatistics
Client->>API: handleStream(emitter, context)
API->>Stream: processStream(responseStream)
Stream-->>API: {toolsData, finishReason, usageData}
API->>Util: reportUsageStatistics(usageData, context)
Util-->>API: reportedUsage[]
API-->>Client: emit final events (toolsData, reportedUsage, finishReason|stop)
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (12)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Summary by CodeRabbit
New Features
Chores