Skip to content

feat: enhance OpenAI and Azure OpenAI authentication support#10

Open
BingqingLyu wants to merge 5 commits into
mainfrom
fork-pr-294-feat-azure-openai-support
Open

feat: enhance OpenAI and Azure OpenAI authentication support#10
BingqingLyu wants to merge 5 commits into
mainfrom
fork-pr-294-feat-azure-openai-support

Conversation

@BingqingLyu

@BingqingLyu BingqingLyu commented Apr 27, 2026

Copy link
Copy Markdown
Owner
  • 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.

TLDR

Dive Deeper

Reviewer Test Plan

Testing Matrix

🍏 🪟 🐧
npm run
npx
Docker
Podman - -
Seatbelt - -

Linked issues / bugs

KrtinShet and others added 5 commits August 16, 2025 19:34
- 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.
@BingqingLyu

BingqingLyu commented May 7, 2026

Copy link
Copy Markdown
Owner Author

Conflict Group 1

This 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.

Function File Also modified by
hasApiKeyForAuth auth.ts #7
parseApprovalModeValue config.ts #112, #113, #114, #117, #21, #22, #36, #46, #86, #88
parseArguments config.ts #112, #113, #114, #117, #14, #17, #18, #21, #22, #31, #36, #46, #7, #86, #88
setSource contentGenerator.ts #25, #30
start_sandbox sandbox.ts #112, #113, #114, #117, #26, #7, #88, #94
validateModelConfig contentGenerator.ts #46
validateNonInteractiveAuth validateNonInterActiveAuth.ts #14, #17
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
Loading

Posted by codegraph-ai conflict detection.

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

This pull request has had no activity for 60 days and is being marked as stale. It will be closed in another 30 days if no further activity occurs. To keep it open, push a new commit or leave a comment. Maintainers may apply pinned, status/blocked, status/on-hold, or status/ready-for-merge to exempt it from auto-close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicting-group-1 Conflicting PR group 1 — review as a batch conflicting-pr Shares at least one cross-PR dependency with other PRs status/stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants