Skip to content

build(store): add an openscreen execution alias to the MSIX - #1311

Merged
EtienneLescot merged 1 commit into
mainfrom
chore/store-exec-alias
Oct 17, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
chore/store-exec-alias

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Summary

The Microsoft Store package lives in a versioned folder under WindowsApps, so an agent or a script had no stable path to register for the CLI or openscreen mcp. The MSIX now declares an app execution alias, openscreen.exe, which Windows puts in %LOCALAPPDATA%\Microsoft\WindowsApps, already on the PATH.

  • build/appx-extensions.xml holds the uap5:AppExecutionAlias extension, wired through appx.customExtensionsPath.
  • appx.minVersion moves to 10.0.18362.0: an alias needs 16299, and the app already requires 1903. Side effect: MaxVersionTested follows it, from electron-builder's default 14316 to 18362.

Once this ships, website/docs/cli.md (Microsoft Store row) and its translations can say openscreen. The demo skill already handles both cases.

Test plan

  • Built a test package from the installed 2.1.0 (electron-builder --win appx --prepackaged). MakeAppx accepts the manifest.
  • Registered it from the unpacked folder (unsigned packages cannot carry an alias). openscreen.exe appears in WindowsApps.
  • openscreen help exits 0 through the alias. openscreen mcp answers initialize (server 2.1.0-rc.6) and lists 55 tools, startRecording included.
  • Test package removed.
  • Store certification of the next submission.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Windows installations now support launching the app with the openscreen.exe command.
  • Compatibility
    • The Windows app package now requires Windows 10 version 1903 or newer.

The Store package lives in a versioned folder under WindowsApps, so an agent or a
script had no path to register for the CLI or for openscreen mcp. The alias puts
openscreen.exe in %LOCALAPPDATA%\Microsoft\WindowsApps, which is on the PATH.

An alias needs a MinVersion of 16299; it moves to 18362, the 1903 the app already
requires. Checked on a test package registered from the installed 2.1.0: openscreen
help exits 0 and openscreen mcp lists its 55 tools through the alias.
@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: efd8ded1-e7e9-4dc4-b07e-71c09b8f3081
📥 Commits

Reviewing files that changed from the base of the PR and between a16c782 and 94d6304.

📒 Files selected for processing: 2
  • build/appx-extensions.xml
  • electron-builder.json5

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 4 remain after this review.

📝 Walkthrough

The AppX package now declares an openscreen.exe execution alias for app\Openscreen.exe. The AppX build configuration includes the extension and sets the package minimum version to 10.0.18362.0.

Changes: AppX Execution Alias

Layer / File(s) Summary
Declare and include the execution alias
build/appx-extensions.xml, electron-builder.json5
The package extension maps openscreen.exe to app\Openscreen.exe. The AppX configuration references the extension and sets the minimum version to 10.0.18362.0.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Suggested reviewers: satwiksharma01

Merge Risk: ⚪ Minimal · up to 94d63

The reported installed-package test exercised the CLI alias successfully. Store certification remains untested, but no concrete issue in the available change context indicates a reason to block merging.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description provides a clear summary and detailed test results, but it omits the template sections for the related issue, type of change, release impact, and desktop impact. Add the missing template sections. State the related issue or confirm that none applies, select the applicable change type, release impact, and desktop impact, and use the required Testing heading or clearly map the existing Test plan to it…
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies the build change and the addition of an OpenScreen execution alias to the MSIX.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

Full details: Description check

Resolution

Add the missing template sections. State the related issue or confirm that none applies, select the applicable change type, release impact, and desktop impact, and use the required Testing heading or clearly map the existing Test plan to it.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@EtienneLescot
EtienneLescot merged commit 5ea72a2 into main Oct 17, 2026
20 checks passed
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.

1 participant