[release/v7.4] Separate Store Automation Service Endpoints, Resolve AppID#26396
Merged
TravisEz13 merged 3 commits intoPowerShell:release/v7.4from Nov 5, 2025
Merged
Conversation
…lease/v7.4/pr/26210 # Conflicts: # .pipelines/templates/package-create-msix.yml
TravisEz13
added
the
CL-BuildPackaging
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the MSIX publishing pipeline to use conditional tasks instead of dynamic variable selection, simplifying the service endpoint and App ID configuration logic.
- Replaced dynamic channel configuration with separate conditional publish tasks for Stable/LTS and Preview channels
- Fixed regex pattern for URL generation to use separate replace operations
- Removed the channel configuration hash table in favor of explicit conditional logic
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .pipelines/templates/release-MSIX-Publish.yml | Refactored to use conditional tasks for Stable/LTS and Preview publishing, replaced dynamic service endpoint selection with hardcoded values per task |
| .pipelines/templates/package-create-msix.yml | Added channel variable exports and split store package creation into separate conditional tasks for Preview vs Stable/LTS |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The variables $IsLTS, $IsStable, and $IsPreview were being used but never initialized. In the main branch, these are set by the channelSelection.yml template (added in PR PowerShell#25725), but in release/v7.4 the backport PR PowerShell#26163 uses stageDependencies to access these values. The PowerShell variables need to be explicitly initialized from the pipeline variables $(LTS), $(STABLE), and $(PREVIEW) at the beginning of the script block.
Contributor
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SeeminglyScience
approved these changes
Nov 5, 2025
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.
Backport of #26210 to release/v7.4
Triggered by @TravisEz13 on behalf of @jshigetomi
Original CL Label: CL-BuildPackaging
/cc @PowerShell/powershell-maintainers
Impact
Tooling Impact
This PR separates the Microsoft Store automation service endpoints to use dedicated endpoints for Preview builds (
StoreAppPublish-Preview) and Stable/LTS builds (StoreAppPublish-Stable), replacing the previous singleStoreAppPublish-Privateendpoint. It also resolves AppID dynamically based on the channel (LTS/Stable/Preview).Regression
This change improves the Store publishing infrastructure and is not fixing a regression.
Testing
The original PR was tested in the main branch CI/CD pipeline. This backport includes merge conflict resolution where:
MS-RDX-MRO.windows-store-publish.package-task.store-package@3by PR [release/v7.4] Fix the task name to not use the pre-release task #26395Risk
Risk Level: High
This change modifies critical CI/CD infrastructure for Microsoft Store publishing. While the changes are well-tested in the main branch, they affect:
Not taking this change would create technical debt and make it difficult to apply future CI/CD changes that build on top of this infrastructure improvement. However, any issues with Store publishing could block releases.
Merge Conflicts Resolution
Conflicts occurred in two files during the backport:
.pipelines/templates/release-MSIX-Publish.yml$channelConfigshashtable$(AppID)variable instead of multiple intermediate variables