chore: Merge from main to demo#604
Merged
Merged
Conversation
fix: Dependent bot code changes
feat: Enhance email template, add RG owner tag, scalability input
fix: update event stream names and enforce security settings in Bicep
fix: fixed copilot comments
chore: Dev Merge to Main
Roopan-Microsoft
approved these changes
May 26, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Merges the latest main branch updates into the demo branch, bringing in dependency refreshes, infrastructure template updates, and CI/CD workflow improvements (notifications, tagging, and new scalability toggle support).
Changes:
- Bumped several pinned dependencies across Python and web packages (e.g.,
python-multipart,urllib3,postcss,axios,uuid). - Updated Bicep/ARM templates to adjust monitoring/data-collection rule configuration and strengthen defaults (e.g., infra encryption, disallow insecure ingress).
- Enhanced GitHub Actions workflows: richer HTML email notifications, added
enableScalabilitywiring/validation, and added/merged anOwnertag on resource groups.
Reviewed changes
Copilot reviewed 16 out of 20 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/ContentProcessorWorkflow/pyproject.toml | Bumps Python dependency (python-multipart) for the workflow service. |
| src/ContentProcessorWeb/pnpm-lock.yaml | Updates frontend lockfile with stricter overrides/pins (e.g., postcss, axios, uuid). |
| src/ContentProcessorWeb/package.json | Aligns postcss version and pnpm override pins with lockfile. |
| src/ContentProcessorAPI/uv.lock | Updates locked Python package versions (python-multipart, urllib3) for the API. |
| src/ContentProcessorAPI/requirements.txt | Updates pinned Python deps (python-multipart, urllib3) for the API. |
| src/ContentProcessorAPI/pyproject.toml | Updates Python dependency constraints (python-multipart, urllib3) for the API. |
| src/ContentProcessor/uv.lock | Updates locked Python package versions (python-multipart, urllib3) for the processor. |
| src/ContentProcessor/requirements.txt | Updates pinned urllib3 for the processor. |
| infra/main.json | Regenerated ARM template reflecting Bicep updates (monitoring destination naming, event stream changes, security settings). |
| infra/main.bicep | Adjusts monitoring/DCR config and hardens infra defaults (encryption, disallow insecure ingress); adds scalability-related naming variables. |
| infra/main_custom.bicep | Mirrors main.bicep infra/security and monitoring changes for the custom variant. |
| .github/workflows/job-send-notification.yml | Reworks notification emails into styled HTML with status “pills” and HTML escaping helper. |
| .github/workflows/job-deploy.yml | Adds enableScalability input wiring and applies Owner tagging on resource groups. |
| .github/workflows/job-deploy-windows.yml | Adds ENABLE_SCALABILITY input and injects enableScalability into parameters for WAF deployments (Windows path). |
| .github/workflows/job-deploy-linux.yml | Adds ENABLE_SCALABILITY input and injects enableScalability into parameters for WAF deployments (Linux path). |
| .github/workflows/deploy.yml | Adds Owner tag on RG create/update for the legacy deploy workflow. |
| .github/workflows/deploy-windows.yml | Adds enableScalability input + validation and passes through to the orchestration job. |
| .github/workflows/deploy-v2.yml | Clarifies input descriptions and adds enableScalability input/validation passthrough. |
| .github/workflows/deploy-orchestrator.yml | Adds enableScalability input passthrough to orchestrated deploy execution. |
Files not reviewed (1)
- src/ContentProcessorWeb/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Avijit-Microsoft
approved these changes
May 26, 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
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information