search

ArchGPT / insomnium

Insomnium is a fast local API testing tool that is privacy-focused and 100% local. For testing GraphQL, REST, WebSockets and gRPC. This is a fork of Kong/insomnia
MIT License
3.36k stars 139 forks source link

Windows: package name matches Insomnia #37

Closed paulius-valiunas closed 10 months ago

paulius-valiunas commented 10 months ago

Expected Behavior

Insomnium should be either updated as a separate app or not touched at all by winget

Actual Behavior

Insomnium is "updated" to Insomnia 8.2.0 by winget

Reproduction Steps

  1. install Insomnium
  2. run winget upgrade
  3. winget shows Insomium with the id insomnia.insomnia and "latest available" version 8.2.0
  4. run winget upgrade --all
  5. winget replaces Insomnium with Insomnia

Is there an existing issue for this?

Additional Information

Insomnium should have its own package name, but right now the package name matches that of Insomnia: insomnium.insomnium. I think this name is derived from the package name in ./packages/insomnia/package.json. More info on how the package name is determined can be found here: https://github.com/microsoft/winget-cli/discussions/3033

Insomnium Version

0.2.1-b

What operating system are you using?

Windows

Operating System Version

Windows 11 22H2

Installation method

GitHub releases

Last Known Working Insomnium version

No response

TCB13 commented 10 months ago

Insomnium should have its own package name

+1.

This also seems to cause issues / makes it impossible to have both applications installed at the same time.

PSA: if you’re on Insomnia export your data BEFORE installing Insomnium as it will obliterate your Insomnia installation and you’re going to lose your collections. Insomnium won't pick on the previous stored data.

archywillhe commented 10 months ago

I think this name is derived from the package name in ./packages/insomnia/package.json. More info on how the package name is determined can be found here: https://github.com/microsoft/winget-cli/discussions/3033

thanks! good find! I will patch this asap in the next release; looks like a windows-only bug (The name in the package.json at root has long been updated to Insomnium. if the fix is about changing pacakges/insomnia, then I need to test it out thoroughly as it may break other stuff)

ShayBox commented 10 months ago

Could this also be added to winget?