Enable() with AppX doesn't add a registry entry

[Oxalin]

OS: Windows
Context: AppX registration on enable()

According to #111 (comment), enable() with AppX doesn't register the application as intended.

Investigate and add appropriate test coverage.

[Oxalin]

If an electron app is converted to an AppX, both process.versions.electron and process.windowsStore will be defined. As of now, the logic for detecting an electron app takes precedence over the logic for an AppX in the enable() code. This needs to be revisited.

Need more info - If someone can tell me more: how is an AppX built on Electron updated? Does it update itself through update.exe or is it through the MS Store? And is there a "update.exe" file installed? @hovancik

[Oxalin]

Oh! I think I've found out why it is not working. 6.0.0-rcX tags were created, but they don't seem to contain the actual updated code... Tags were probably created to early.

Next 6.0.0-rc2 will be a major update, dropping CoffeeScript and all the other improvements.

[hovancik]

Thanks so much for your work! Let me know how can I test or add more information.

I updated my app via Windows Store: by creating a testing group and inviting myself, then pushing test version there.

[Oxalin]

@hovancik now would be a good time to test with the just released 6.0.0-rc2 :)

[hovancik]

Great, will try. But it will take some time as I need to wait for approvals and such. Will report.

[hovancik]

Hi @Oxalin , I do not see that version on NPM: https://www.npmjs.com/package/auto-launch?activeTab=versions

and installing via github errors when used (but I think it was mentioned elsewhere that you don't plan to fix that? ):

Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Program Files\WindowsApps\33881JanHovancik.stretchly_1.15.102.0_x64__24fg4m0zq65je\app\resources\app.asar\node_modules\auto-launch\src\index.js from C:\Program Files\WindowsApps\33881JanHovancik.stretchly_1.15.102.0_x64__24fg4m0zq65je\app\resources\app.asar\app\main.js not supported.
Instead change the require of index.js in C:\Program Files\WindowsApps\33881JanHovancik.stretchly_1.15.102.0_x64__24fg4m0zq65je\app\resources\app.asar\app\main.js to a dynamic import() which is available in all CommonJS modules.
    at c._load (node:electron/js2c/node_init:2:13672)
    at IpcMainImpl.<anonymous> (C:\Program Files\WindowsApps\33881JanHovancik.stretchly_1.15.102.0_x64__24fg4m0zq65je\app\resources\app.asar\app\main.js:1463:26)
    at IpcMainImpl.emit (node:events:517:28)
    at WebContents.<anonymous> (node:electron/js2c/browser_init:2:77585)
    at WebContents.emit (node:events:517:28)

Here's my PR hovancik/stretchly#1450

[Fefedu973]

An OS update caused the package to fail working ? Many user of a software i made that use this package complain that it dosen't start on boot even though it should and it was working before and it looks like it dosen't create the registry key. Weird. Does the script need to run as admin to work ?