refactor: Refactor codebase by removing unused scripts and configurations#909
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates infrastructure and documentation to reflect a shift away from Azure Container Apps/ACR-focused deployment guidance and toward Azure App Service–based hosting, while removing legacy Container Apps IaC artifacts.
Changes:
- Updated docs (README, DeploymentGuide, next-steps, troubleshooting) to reference Azure App Service and revise security/troubleshooting guidance accordingly.
- Removed legacy Container Apps infrastructure templates/modules (
infra/resources.bicep,infra/modules/fetch-container-image.bicep) and related naming data (infra/abbreviations.json). - Regenerated/updated compiled ARM template output (
infra/main.json) and made small infra template refactors/comments.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/App/package-lock.json | Lockfile update (dependency tree entry removed). |
| README.md | Resource list/security guidance updated to reference App Service. |
| next-steps.md | Updated “what was added” + troubleshooting steps for App Service and main.bicep. |
| infra/resources.bicep | Deleted legacy Container Apps + ACR-based resource deployment template. |
| infra/modules/fetch-container-image.bicep | Deleted legacy helper for querying existing Container App images. |
| infra/main.json | Updated generated ARM template (hash + variable/template content changes). |
| infra/main.bicep | Comment/refactor tweaks and minor AI Foundry deployment gating simplification. |
| infra/main_custom.bicep | Removed container image/ACR-related parameters; updated telemetry comments and outputs. |
| infra/abbreviations.json | Deleted legacy abbreviations JSON used by the removed template. |
| documents/TroubleShootingSteps.md | Removed Container Apps troubleshooting entries; added App Service capacity guidance and updated identity wording. |
| documents/DeploymentGuide.md | Updated prerequisites and configurable-parameters text to remove Container Apps/ACR references. |
Files not reviewed (1)
- src/App/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <copilot@github.com>
Avijit-Microsoft
approved these changes
May 13, 2026
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
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.
Purpose
This pull request updates documentation to reflect a migration from Azure Container Apps and Azure Container Registry to Azure App Service for hosting the application's web frontend and backend. It also updates related security guidance, quota and troubleshooting documentation, and deployment parameter descriptions. The most important changes are summarized below:
Resource and Deployment Documentation Updates:
Azure Container AppsandAzure Container RegistrywithAzure App Servicein both the mainREADME.mdanddocuments/DeploymentGuide.md, including updated descriptions, pricing, and links. [1] [2]Security Guidance:
Troubleshooting and Quota Documentation:
documents/TroubleShootingSteps.md. [1] [2]These updates ensure the documentation accurately reflects the new hosting architecture and provides relevant guidance for deployment, security, and troubleshooting.
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information