goose icon indicating copy to clipboard operation
goose copied to clipboard

Refreshing after extension installation deeplink shows installation dialog again

Open zanesq opened this issue 2 months ago • 2 comments

Describe the bug After installing an extension from a deeplink if you refresh the browser the installation dialog shows again. It appears to be holding on to the deeplink params in the url. We should add logic to clear these deeplink params from the url after its launched so it doesn't pick it up again on refresh.

To Reproduce Steps to reproduce the behavior:

  1. Install an extension from a deeplink
  2. See installation dialog and environment variable dialog if it has env variables to set.
  3. Cancel install or proceed successfully either is fine to close the modal and finish extension installation process.
  4. Refresh browser
  5. See installation dialogs reappear again.

Expected behavior After launching an extension install from a deeplink it shouldn't show again on refresh.

Please provide following information:

  • OS & Arch: Mac arm
  • Interface: UI
  • Version: v1.9.1 (goes back further)
  • Extensions enabled: Developer
  • Provider & Model: Claude

Additional context None

zanesq avatar Oct 10 '25 22:10 zanesq

I looked at the associated PR and it seems to solve a different deeplink issue maybe?

So for extensions and deeplinks I think we should check whether we have that extension installed and if so, always ignore the deeplink whether this is refresh or no. if the extension is not installed, I think reload showing the dialogue again if the extension is not installed is fine.

for recipes I think starting the same again is always fine. I started goose with a deeplink, maybe I want to do the same thing in a slightly different way.

DOsinga avatar Oct 11 '25 14:10 DOsinga

Yeah that was fixing a different issue where opening a new window was showing the installation dialog again from the deeplink (electron level not clearing deeplink), I think this is related to the current window and the url holding on to the deeplink after install.

Sounds good will take that into account when I work on it.

zanesq avatar Oct 14 '25 15:10 zanesq