Skip to content

[release/v7.6.7] Simplify Microsoft Store listing metadata - #28103

Merged
Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/27968-8e1675f17
Sep 24, 2026
Merged

Dongbo Wang (daxian-dbw) merged 1 commit into
PowerShell:release/v7.6.7from
daxian-dbw:backport/release/v7.6.7/27968-8e1675f17

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

Backport of #27968 to release/v7.6.7

Triggered by Dongbo Wang (@daxian-dbw) on behalf of Justin Chung (@jshigetomi)

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Simplifies the Microsoft Store listing pipeline by keeping localized release-note language in the PDP and removing the now-redundant PDP-Media tree and AppStoreName mutation.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Verified that the Store release pipeline correctly uses the PDP as the static source of listing language and only replaces VERSION/CHANGELOGLINK placeholders during publishing. Backport verified by confirming the cherry-pick applied cleanly with no conflicts on the current release/v7.6.7 (after syncing to the latest upstream, which includes the revert of a previous mis-ordered backport of a dependent PR).

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Low risk: Store release pipeline/packaging change only. Removes redundant PDP media tree and AppStoreName mutation, and limits release-time text replacement to version-specific placeholders. Does not affect product runtime behavior. This exact commit was previously backported to v7.6.7 (PR #28079) and later reverted for unrelated reasons; re-applying it now.

Co-authored-by: Justin Chung <[email protected]>
Co-authored-by: Copilot App <[email protected]>
Copilot-Session: 15d5aa39-2adb-4337-80ba-a1a8568d8fba
@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Sep 24, 2026
Copilot AI lite review requested due to automatic review settings September 24, 2026 22:36
@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Sep 24, 2026
@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.

Copilot review overview

🟡 Changes recommended

Critical publishing and localized PDP metadata issues remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 High severity

Open (2)
What changed in this PR

Backports a simplification of Microsoft Store listing metadata and publishing for release/v7.6.7.

Changes:

  • Adds localized manifest resource declarations.
  • Uses placeholder-based release notes.
  • Removes redundant media configuration and AppStoreName mutation.
File Summary
assets/​AppxManifest.xml Adds localized Store resource declarations; localized PDPs still require AppStoreName removal.
.pipelines/​templates/​release-MSIX-Publish.yml Adds release-note substitution; localized PDPs lack the required placeholders.
.pipelines/​templates/​package-store-package.yml Simplifies StoreBroker packaging inputs.
.pipelines/​store/​SBConfig.json Removes obsolete media settings.
.pipelines/​store/​PDPs/​PDP/​en-US/​PDP.xml Adds placeholder-based release notes.
.pipelines/​store/​PDPs/​PDP-Media/​en-US/​.gitkeep Removes the obsolete media-tree marker.

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

Comment on lines +77 to +80
foreach ($placeholder in $releaseNoteVariables.Keys) {
if (-not $releaseNotes.Contains($placeholder)) {
throw "Store release notes for '$language' do not contain the required placeholder '$placeholder'."
}
Comment thread assets/AppxManifest.xml
Comment on lines +28 to +32
<Resource Language="cs"/>
<Resource Language="de"/>
<Resource Language="es"/>
<Resource Language="fr"/>
<Resource Language="it"/>
@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 10414fb into PowerShell:release/v7.6.7 Sep 24, 2026
36 checks passed
@daxian-dbw
Dongbo Wang (daxian-dbw) deleted the backport/release/v7.6.7/27968-8e1675f17 branch September 24, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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