Skip to content

Update Appx manifest to not allow updates while PS7 is running - #28276

Closed
Steve Lee (SteveL-MSFT) wants to merge 1 commit into
PowerShell:masterfrom
SteveL-MSFT:msix-update-defer
Closed

Steve Lee (SteveL-MSFT) wants to merge 1 commit into
PowerShell:masterfrom
SteveL-MSFT:msix-update-defer

Conversation

@SteveL-MSFT

@SteveL-MSFT Steve Lee (SteveL-MSFT) commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

PR Summary

Customers have reported that pwsh.exe gets killed while they are using it due to a new version being available in the Store. The change here uses the UpdateWhileInUse property to defer updates while PS7 is in use.

PR Checklist

Copilot AI balanced review requested due to automatic review settings October 28, 2026 21:39
@SteveL-MSFT
Steve Lee (SteveL-MSFT) requested a review from a team as a code owner October 28, 2026 21:39
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Deferring updates causes non-Store deployments to ignore forced-shutdown options, but the compatibility impact and need to close running sessions are not documented.

1 open issue
What changed in this PR

This PR updates PowerShell’s MSIX manifest to defer app updates while PowerShell is running on Windows 11 24H2 and later, protecting active sessions from Store-update interruptions.

Changes:

  • Declares uap17 as an ignorable namespace for older Windows compatibility.
  • Sets UpdateWhileInUse to defer.
File Description
assets/​AppxManifest.xml Adds the namespace and policy for deferring updates while PowerShell is in use.

Review effort: Balanced

Comment thread assets/AppxManifest.xml Resolved

@kilasuit Ryan Yates (kilasuit) left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR is almost Identical to

with only differences being

  • the closing > added to line 12 instead of line 11
  • the ordering of lines 10 & 11

We need to decide whether it's this PR that gets taken or #28275 gets taken forward & as it looks like either will fix #28272

@SteveL-MSFT

Copy link
Copy Markdown
Member Author

I didn't see that someone else submitted essentially the same PR. I'll close this one as dupe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.4.x-Consider Backport-7.5.x-Consider Backport-7.6.x-Consider CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants