forge icon indicating copy to clipboard operation
forge copied to clipboard

Documentation for `name` property on Squirrel maker config is incorrect

Open RandomEngy opened this issue 3 years ago • 3 comments

Pre-flight checklist

  • [X] I have read the contribution documentation for this project.
  • [X] I agree to follow the code of conduct that this project uses.
  • [X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version

6.0.4

Electron version

22.0.0

Operating system

Windows 11 (22621.963)

Last known working Electron Forge version

No response

Expected behavior

The name config option for the Windows Squirrel maker is described accurately.

Actual behavior

This is the current description for name:

Windows Application Model ID (appId).

Defaults to the name field in your app's package.json file.

It looks like the AUMID actually gets set to com.squirrel.<makerConfigName>.<packageName> Where <makerConfigName> is the value given to the name property on the maker config, and <packageName> is the name field in package.json.

Steps to reproduce

Use the lnk-parser tool to check System.AppUserModel.ID on the shortcut generated by the Squirrel installer.

Additional information

No response

RandomEngy avatar Dec 20 '22 02:12 RandomEngy

The application name. If omitted, it will use the productName or name value from the nearest package.json.

On Windows particularly, the application User Model ID (AUMID) is derived from the application name. The AUMID format follows this pattern: com.squirrel.<makerConfigName>.<packageName>, where <makerConfigName> is the value given to the name property on the maker config, and <packageName> is the name field in package.json.

Regardless of source, characters in the Electron app name which are not allowed in all target platforms' filenames (e.g., /), will be replaced by hyphens (-).

Does this describe the name option in maker-squirrel more accurately?

DevanceJ avatar Apr 05 '24 09:04 DevanceJ

I think so. Though I've since migrated my project off of Electron, so I haven't checked lately.

RandomEngy avatar Apr 05 '24 16:04 RandomEngy