chore: dev to main merge - #693
Merged
Merged
Conversation
fix: Codespace ACR build and push script failure
fix: updated the nanoid package
fix: Route dependency restoration through Microsoft Package Feed Proxy
fix: ACR build and push script failing
Contributor
There was a problem hiding this comment.
Pull request overview
This PR standardizes Microsoft Package Feed Proxy usage across dependencies, CI/CD, containers, and documentation while refactoring ACR build tooling and adding deployment utilities.
Changes:
- Configures Python, npm, and NuGet package sources through the proxy.
- Refactors Bash and PowerShell ACR build scripts.
- Adds POSIX post-deployment and upload scripts.
- Updates frontend dependency and container configuration.
Reviewed changes
Copilot reviewed 25 out of 31 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Summary |
|---|---|
tests/e2e-test/requirements.txt |
Uses the Microsoft PyPI proxy. |
src/ContentProcessorWorkflow/pyproject.toml |
Adds the uv proxy index. |
src/ContentProcessorWeb/package.json |
Updates nanoid. |
src/ContentProcessorWeb/Dockerfile |
Uses the npm proxy during builds. |
src/ContentProcessorWeb/.npmrc |
Sets the npm proxy registry. |
src/ContentProcessorWeb/.dockerignore |
Adds build-context exclusions. |
src/ContentProcessorWeb/.devcontainer/Dockerfile |
Updates npm proxy guidance. |
src/ContentProcessorWeb/.devcontainer/base.Dockerfile |
Uses the npm proxy for global packages. |
src/ContentProcessorAPI/samples/upload_files.sh |
Adds a Bash upload utility. Critical (4 votes): the manual Content-Type header omits curl’s multipart boundary and causes uploads to be rejected. |
src/ContentProcessorAPI/requirements.txt |
Uses the Microsoft PyPI proxy. |
src/ContentProcessorAPI/pyproject.toml |
Adds the uv proxy index. |
src/ContentProcessor/requirements.txt |
Uses the Microsoft PyPI proxy. |
src/ContentProcessor/pyproject.toml |
Adds the uv proxy index. |
nuget.config |
Configures the NuGet proxy source. |
infra/vscode_web/requirements.txt |
Uses the Microsoft PyPI proxy. |
infra/vscode_web/endpoint-requirements.txt |
Uses the Microsoft PyPI proxy. |
infra/scripts/post_deployment.sh |
Adds POSIX post-deployment automation. Moderate (2 votes): it is not wired into the documented azd up flow. Moderate (3 votes): GNU realpath is unavailable by default on macOS. |
infra/scripts/acr_build_push.sh |
Refactors staged ACR builds. Critical (2 votes): the shebang is not the first line, so direct execution may use a non-Bash shell and fail. |
infra/scripts/acr_build_push.ps1 |
Refactors PowerShell ACR builds with staged contexts. |
docs/LocalDevelopmentSetup.md |
Updates proxied Python setup commands. |
docs/AVMPostDeploymentGuide.md |
Updates proxied installation instructions. |
.github/workflows/test.yml |
Sets the Python package proxy. |
.github/workflows/test-automation.yml |
Sets the automation Python proxy. |
.github/workflows/test-automation-v2.yml |
Sets the automation Python proxy. |
.github/workflows/pylint.yml |
Sets the lint Python proxy. |
.devcontainer/Dockerfile |
Uses the npm proxy. |
.devcontainer/devcontainer.json |
Adds package-proxy environment variables. |
Suppressed comments (1)
src/ContentProcessorWeb/.npmrc:3
- This registry setting does not control the dependency artifacts pinned in the frozen lockfile:
pnpm-lock.yamlcontains explicit tarball URLs onms-feed-*.pkgs.visualstudio.com(for example thenanoidentry).pnpm install --frozen-lockfilewill continue downloading those URLs, so existing dependencies are not actually routed through the new proxy and can fail where that feed is inaccessible. Regenerate the lockfile against the proxy or otherwise remove/replace the embedded tarball URLs.
registry=https://packagefeedproxy.microsoft.io/npm/
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Avijit-Microsoft
approved these changes
Aug 25, 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 introduces significant changes to centralize and standardize the use of the Microsoft Package Feed Proxy for Python, Node.js, and .NET dependencies across development, CI/CD, and documentation. It also refactors ACR build scripts for improved maintainability and reliability. The most important changes are grouped below:
Standardizing Package Feeds
https://packagefeedproxy.microsoft.io/pypi/simple/) to ensure consistent and reliable dependency resolution. This includes updates indocs/LocalDevelopmentSetup.md,docs/AVMPostDeploymentGuide.md,infra/vscode_web/requirements.txt,infra/vscode_web/endpoint-requirements.txt, andsrc/ContentProcessor/pyproject.toml[1] [2] [3] [4] [5] [6] [7] [8] [9].https://packagefeedproxy.microsoft.io/npm/), as reflected in.devcontainer/Dockerfileand.devcontainer/devcontainer.json[1] [2].nuget.configfile is added to direct .NET package restores to the Microsoft NuGet proxy (https://packagefeedproxy.microsoft.io/nuget/v3/index.json).CI/CD Pipeline Updates
PIP_INDEX_URLenvironment variable to the Microsoft proxy, ensuring all automated builds and tests use the same package source (.github/workflows/pylint.yml,.github/workflows/test.yml,.github/workflows/test-automation.yml,.github/workflows/test-automation-v2.yml) [1] [2] [3] [4].Build Script Refactoring
infra/scripts/acr_build_push.ps1) and Bash (infra/scripts/acr_build_push.sh) are refactored to use helper functions (Build-Imageandbuild_image) that stage build contexts more reliably using git, improving maintainability and reducing duplication [1] [2] [3] [4].Other Improvements
These changes collectively improve dependency management, build reliability, and maintainability across the project.
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information