feat: enhance OpenAI and Azure OpenAI authentication support#10
Open
BingqingLyu wants to merge 5 commits into
Open
feat: enhance OpenAI and Azure OpenAI authentication support#10BingqingLyu wants to merge 5 commits into
BingqingLyu wants to merge 5 commits into
Conversation
- Updated documentation in README.md and authentication.md to include Azure OpenAI configuration options. - Modified CLI to accept Azure OpenAI parameters and handle them appropriately. - Enhanced OpenAIKeyPrompt component to support Azure OpenAI input fields. - Updated OpenAIContentGenerator to differentiate between Standard OpenAI and Azure OpenAI configurations. - Added environment variable checks for Azure OpenAI in the sandbox and configuration files.
- Implemented Azure OpenAI authentication in validateNonInterActiveAuth.ts and auth.ts. - Updated AuthDialog and OpenAIKeyPrompt components to handle Azure OpenAI input fields and prompts. - Introduced AzureOpenAIContentGenerator for Azure-specific content generation. - Enhanced content generator configuration to support Azure OpenAI parameters. - Updated logging to exclude Azure OpenAI responses and errors for better security.
… to match tests and expected UX
This was referenced Apr 28, 2026
5 tasks
Owner
Author
Conflict Group 1This PR shares modified functions with 19 other PR(s): #112, #113, #114, #117, #14, #17, #18, #21, #22, #25, #26, #30, #31, #36, #46, #7, #86, #88, #94. These PRs should be reviewed as a batch — merging one may affect the others.
graph LR
PR10["PR #10"]
FhasApiKeyForAuth_8617["hasApiKeyForAuth<br>auth.ts"]
PR10 -->|modifies| FhasApiKeyForAuth_8617
PR7["PR #7"]
PR7 -->|modifies| FhasApiKeyForAuth_8617
FparseApprovalModeValue_6977["parseApprovalModeValue<br>config.ts"]
PR10 -->|modifies| FparseApprovalModeValue_6977
PR112["PR #112"]
PR112 -->|modifies| FparseApprovalModeValue_6977
PR113["PR #113"]
PR113 -->|modifies| FparseApprovalModeValue_6977
PR114["PR #114"]
PR114 -->|modifies| FparseApprovalModeValue_6977
PR117["PR #117"]
PR117 -->|modifies| FparseApprovalModeValue_6977
PR21["PR #21"]
PR21 -->|modifies| FparseApprovalModeValue_6977
PR22["PR #22"]
PR22 -->|modifies| FparseApprovalModeValue_6977
PR36["PR #36"]
PR36 -->|modifies| FparseApprovalModeValue_6977
PR46["PR #46"]
PR46 -->|modifies| FparseApprovalModeValue_6977
PR86["PR #86"]
PR86 -->|modifies| FparseApprovalModeValue_6977
PR88["PR #88"]
PR88 -->|modifies| FparseApprovalModeValue_6977
FparseArguments_6977["parseArguments<br>config.ts"]
PR10 -->|modifies| FparseArguments_6977
PR112 -->|modifies| FparseArguments_6977
PR113 -->|modifies| FparseArguments_6977
PR114 -->|modifies| FparseArguments_6977
PR117 -->|modifies| FparseArguments_6977
PR14["PR #14"]
PR14 -->|modifies| FparseArguments_6977
PR17["PR #17"]
PR17 -->|modifies| FparseArguments_6977
PR18["PR #18"]
PR18 -->|modifies| FparseArguments_6977
PR21 -->|modifies| FparseArguments_6977
PR22 -->|modifies| FparseArguments_6977
PR31["PR #31"]
PR31 -->|modifies| FparseArguments_6977
PR36 -->|modifies| FparseArguments_6977
PR46 -->|modifies| FparseArguments_6977
PR7 -->|modifies| FparseArguments_6977
PR86 -->|modifies| FparseArguments_6977
PR88 -->|modifies| FparseArguments_6977
FsetSource_5386["setSource<br>contentGenerator.ts"]
PR10 -->|modifies| FsetSource_5386
PR25["PR #25"]
PR25 -->|modifies| FsetSource_5386
PR30["PR #30"]
PR30 -->|modifies| FsetSource_5386
Fstart_sandbox_9815["start_sandbox<br>sandbox.ts"]
PR10 -->|modifies| Fstart_sandbox_9815
PR112 -->|modifies| Fstart_sandbox_9815
PR113 -->|modifies| Fstart_sandbox_9815
PR114 -->|modifies| Fstart_sandbox_9815
PR117 -->|modifies| Fstart_sandbox_9815
PR26["PR #26"]
PR26 -->|modifies| Fstart_sandbox_9815
PR7 -->|modifies| Fstart_sandbox_9815
PR88 -->|modifies| Fstart_sandbox_9815
PR94["PR #94"]
PR94 -->|modifies| Fstart_sandbox_9815
FvalidateModelConfig_5386["validateModelConfig<br>contentGenerator.ts"]
PR10 -->|modifies| FvalidateModelConfig_5386
PR46 -->|modifies| FvalidateModelConfig_5386
FvalidateNonInteractiveAuth_1241["validateNonInteractiveAuth<br>validateNonInterActiveAuth.ts"]
PR10 -->|modifies| FvalidateNonInteractiveAuth_1241
PR14 -->|modifies| FvalidateNonInteractiveAuth_1241
PR17 -->|modifies| FvalidateNonInteractiveAuth_1241
Posted by codegraph-ai conflict detection. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR
Dive Deeper
Reviewer Test Plan
Testing Matrix
Linked issues / bugs