Skip to content

Fix WinGet deployment conflicts and RPC recovery - #133

Merged
AmirMS (AmelBawa-msft) merged 2 commits into
mainfrom
user/amelbawa/fix-warn
Oct 2, 2026
Merged

AmirMS (AmelBawa-msft) merged 2 commits into
mainfrom
user/amelbawa/fix-warn

Conversation

@AmelBawa-msft

@AmelBawa-msft AmirMS (AmelBawa-msft) commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Fix three WinGet setup issues in src\windows-dev-config\steps\_winget.ps1:

  • Skip installed dependencies that satisfy the required name, publisher, architecture, and version to avoid unnecessary deployments and 0x80073D02 conflicts.
  • Use Write-Error in Invoke-DevConfigWinGetDeployment so CLIXML parsing failures no longer hide the real AppX error.
  • Handle 0x800706BA by HRESULT in Confirm-DevConfigWinGetReady, including generic exceptions from Windows PowerShell 5.1. Preserve the existing registration retry and verified CLI fallback, and rethrow unrelated errors.

Document dependency reuse in the Windows Dev Config README.

Validation

Tested in Windows x64 VMs with Windows PowerShell 5.1 and PowerShell 7:

  • Reproduced the original dependency conflict, CLIXML reporting failure, and PS5.1 RPC failure.
  • Verified successful WinGet updates and no-op reruns.
  • Passed two fixed PS5.1 recovery runs using verified CLI fallback and two PS7 runs retaining module access, including real package queries.
  • Passed focused dependency-selection, error-reporting, and RPC recovery checks in both editions.

No PR or issue templates were found; this draft follows  CONTRIBUTING.md .

@AmelBawa-msft AmirMS (AmelBawa-msft) changed the title User/amelbawa/fix warn Fix WinGet deployment conflicts and RPC recovery Oct 2, 2026
@AmelBawa-msft
AmirMS (AmelBawa-msft) requested a balanced review from Copilot October 2, 2026 19:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

Review effort: Lite
Findings: 2 Medium severity · 1 Low severity

Open (3)
What changed in this PR

Fixes WinGet/App Installer setup robustness in Windows Dev Config by avoiding unnecessary dependency deployments, surfacing real deployment errors, and improving RPC recovery behavior across PowerShell editions.

Changes:

  • Filter dependency packages to skip redeploying already-satisfied AppX dependencies (avoids 0x80073D02 conflicts).
  • Switch deployment failure reporting to Write-Error to avoid CLIXML masking of the underlying AppX error.
  • Broaden WinGet readiness RPC recovery to trigger by HRESULT across exception types (including Windows PowerShell 5.1), and document dependency reuse.
File Description
src/​windows-dev-config/​steps/​_winget.ps1 Filters dependency installs, improves error emission, and refines RPC-ready checks based on HRESULT.
src/​windows-dev-config/​README.md Documents that setup will reuse installed dependencies meeting identity+version requirements.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/windows-dev-config/steps/_winget.ps1
Comment thread src/windows-dev-config/steps/_winget.ps1
Comment thread src/windows-dev-config/steps/_winget.ps1
@AmelBawa-msft
AmirMS (AmelBawa-msft) merged commit a40a9c1 into main Oct 2, 2026
14 checks passed
@AmelBawa-msft
AmirMS (AmelBawa-msft) deleted the user/amelbawa/fix-warn branch October 2, 2026 22:21
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