Skip to content

Testing AppX execution alias without yao-pkg#2599

Closed
fredrikekelund wants to merge 3 commits intotrunkfrom
f26d/appx-execution-alias-without-yao-pkg
Closed

Testing AppX execution alias without yao-pkg#2599
fredrikekelund wants to merge 3 commits intotrunkfrom
f26d/appx-execution-alias-without-yao-pkg

Conversation

@fredrikekelund
Copy link
Contributor

Related issues

Proposed Changes

I'm not familiar with AppX, but looking at #2547, I wondered whether we've tried an approach like this. In this PR, the Executable parameter points to the node runtime we already bundle, and a uap10:Parameters contains command line parameters passed to Node (in this case, simply the path to cli/main.js).

Testing Instructions

We need to build an AppX installer for the app to test this.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 17, 2026

📊 Performance Test Results

Comparing b06a80f vs trunk

site-editor

Metric trunk b06a80f Diff Change
load 2704.00 ms 2697.00 ms -7.00 ms ⚪ 0.0%

site-startup

Metric trunk b06a80f Diff Change
siteCreation 6075.00 ms 6073.00 ms -2.00 ms ⚪ 0.0%
siteStartup 3954.00 ms 3943.00 ms -11.00 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@fredrikekelund fredrikekelund marked this pull request as draft February 26, 2026 11:09
Copy link
Contributor

@gcsecsey gcsecsey left a comment

Choose a reason for hiding this comment

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

The approach looked good to me on the surface, but when I tried to test it, I couldn't access the studio command from any shell (Command Prompt or PowerShell).

After digging a bit into how Windows execution aliases work, I think this approach won't work because of a limitation of the alias mechanism. Based on this article, the AppExecutionAlias doesn't seem to support receiving parameters. I think the uap10:Parameters attribute is not being applied to the execution alias.

It'd be great to be able to pass a parameter to avoid packaging a standalone exe, but I don't have any better ideas at the moment for how to do that.

@fredrikekelund
Copy link
Contributor Author

Thanks for digging, @gcsecsey 🙏 I did some more testing with this last week, too, but also couldn't get it to work. I'm actually not even sure this approach had a very big effect on filesize… I'll just close this.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants