Skip to content

feat: add OPENAI_API_KEY to frontEnd module configuration#283

Merged
vmlcode merged 1 commit intodevelopfrom
Add-OpenAI-api-key-to-frontend-repo
Mar 25, 2026
Merged

feat: add OPENAI_API_KEY to frontEnd module configuration#283
vmlcode merged 1 commit intodevelopfrom
Add-OpenAI-api-key-to-frontend-repo

Conversation

@vmlcode
Copy link
Copy Markdown
Collaborator

@vmlcode vmlcode commented Mar 25, 2026

No description provided.

@vmlcode vmlcode merged commit c5b08c9 into develop Mar 25, 2026
5 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an OPENAI_API_KEY application setting to the frontEnd App Service module so the frontend deployment can receive an OpenAI key via configuration.

Changes:

  • Add OPENAI_API_KEY to the frontEnd module appSettings in infra/main.bicep.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

}
{
name: 'OPENAI_API_KEY'
value: mcpOpenAiApiKey
Copy link

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This app setting uses mcpOpenAiApiKey directly, but the rest of the template consistently uses the normalized *Var value (e.g., mcpOpenAiApiKeyVar for OPENAI_API_KEY in the orchestrator settings). Using the *Var here would keep conventions consistent and avoids confusion given the parameter is described as "used by the MCP server" while it's now also consumed by the frontend; consider switching to mcpOpenAiApiKeyVar and/or clarifying the parameter naming/description to reflect shared use.

Suggested change
value: mcpOpenAiApiKey
value: mcpOpenAiApiKeyVar

Copilot uses AI. Check for mistakes.
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.

3 participants