fix redirect to extensions page after deeplink install and show toast with success message #4863

zanesq · 2025-09-26T21:09:15Z

Noticed extension deeplinks weren't redirecting to the extensions page. Also noticed that the extension is not automatically activated simply by going back to chat route and only activated in a completely new session so we need to address that.

For now I added a toast with more info to start a completely new session to activate the extension. I also considered not enabling the extension and letting the user activate it manually but that adds another step.

NOTE: there is an issue currently where refreshing the app after installing from a deeplink starts the deeplink again (same with recipe deeplinks) we should follow up and fix that in another PR

… with success message

DOsinga · 2025-09-26T21:41:56Z

ui/desktop/src/App.tsx

  const [didSelectProvider, setDidSelectProvider] = useState<boolean>(false);

  const navigate = useNavigate();
+  const setView = useMemo(() => createNavigationHandler(navigate), [navigate]);


we have this pattern N times now, we can probably do something less repetitive

done but added your favorite a new hook unless you have better ideas

ui/desktop/src/components/ExtensionInstallModal.tsx

ui/desktop/src/components/settings/extensions/deeplink.ts

DOsinga · 2025-09-26T21:50:46Z

ui/desktop/src/components/settings/extensions/deeplink.ts

+    console.log('Environment variables or headers required, redirecting to extensions');
    console.log('Calling setView with:', { deepLinkConfig: config, showEnvVars: true });
-    setView('settings', { deepLinkConfig: config, showEnvVars: true });
+    setView('extensions', { deepLinkConfig: config, showEnvVars: true });


we're now going to extensions whether you had this set or no. but in the second case we provide the extensionId. what's the difference? and also can we unify the two paths?

Good catch, should probably send extensionId in both cases will update 👍

Changed to use the name from deepLinkConfig instead of passing extensionId separately since thats what extensionId was anyway :) not sure what you meant by unify the paths, in this case it returns after because it doesn't need to call addExtensionFn and the toast message is shown after the modal installation in this case

ui/desktop/src/components/settings/extensions/deeplink.ts

…-deeplink * 'main' of github.com:block/goose: (57 commits) Don't set agent props twice (#4872) fix: conversation fixer merges assistant text blocks and drops empty text messages (#4898) Batch fetch remaining issues for documentation updates fix: session timestamps (#4913) feat: lazy infinite scroller for session history view (#4922) chore: properly identify when to try oauth (#4918) Make token counter safer (#4924) Rename Hacktoberfest Blog to Hacktoberfest Content (#4926) Include Session ID appropriately in UI (#4901) fix mcp integration test flakiness (#4871) Add type field to empty schemas for anthropic (#4911) docs: remove subagents from experimental (#4907) CLI: dont show logs to user (#4902) patching the security scanner to redact environment variables (#4908) rmcp upgrade (#4792) feat: Use the screen, goose (#4905) added claude-sonnet-4-5 (#4906) Tiny: fix github casing (#4903) remove anyOf from create_task tool (#4897) chore(deps): bump tracing-subscriber from 0.3.19 to 0.3.20 (#4442) ...

* main: (30 commits) feat(nightly): build nightlies from main shas (block#4888) Add missing library for fedora/rhel/centos docs (block#4819) feat(process): Add GOVERNANCE and MAINTAINERS documents (block#4962) Pause test finder, have it run cargo fmt (block#4958) Disable the issue comment trigger on pr-comment-bundle (block#4961) fix(providers): update Claude Sonnet 4 model identifier (block#4884) fix redirect to extensions page after deeplink install and show toast with success message (block#4863) Remove wait-for-ready log (block#4956) docs: add a new goose tip (block#4940) Add PR template (block#4934) Using --resume with --name should still accept session IDs (block#4937) Fix auto scroll to bottom during chat (block#4923) Fix Typo, Add Description to Hacktoberfest Content Issue Template (block#4931) Don't set agent props twice (block#4872) fix: conversation fixer merges assistant text blocks and drops empty text messages (block#4898) Batch fetch remaining issues for documentation updates fix: session timestamps (block#4913) feat: lazy infinite scroller for session history view (block#4922) chore: properly identify when to try oauth (block#4918) Make token counter safer (block#4924) ...

…se into zane/create-recipe-unification-feedback * 'zane/create-recipe-unification' of github.com:block/goose: feat(nightly): build nightlies from main shas (#4888) Add missing library for fedora/rhel/centos docs (#4819) feat(process): Add GOVERNANCE and MAINTAINERS documents (#4962) Pause test finder, have it run cargo fmt (#4958) Disable the issue comment trigger on pr-comment-bundle (#4961) fix(providers): update Claude Sonnet 4 model identifier (#4884) fix redirect to extensions page after deeplink install and show toast with success message (#4863) Remove wait-for-ready log (#4956)

* main: (24 commits) Lifei/create save recipe to file (#4895) feat(nightly): build nightlies from main shas (#4888) Add missing library for fedora/rhel/centos docs (#4819) feat(process): Add GOVERNANCE and MAINTAINERS documents (#4962) Pause test finder, have it run cargo fmt (#4958) Disable the issue comment trigger on pr-comment-bundle (#4961) fix(providers): update Claude Sonnet 4 model identifier (#4884) fix redirect to extensions page after deeplink install and show toast with success message (#4863) Remove wait-for-ready log (#4956) docs: add a new goose tip (#4940) Add PR template (#4934) Using --resume with --name should still accept session IDs (#4937) Fix auto scroll to bottom during chat (#4923) Fix Typo, Add Description to Hacktoberfest Content Issue Template (#4931) Don't set agent props twice (#4872) fix: conversation fixer merges assistant text blocks and drops empty text messages (#4898) Batch fetch remaining issues for documentation updates fix: session timestamps (#4913) feat: lazy infinite scroller for session history view (#4922) chore: properly identify when to try oauth (#4918) ...

… with success message (block#4863) Co-authored-by: Amed Rodriguez <[email protected]> Signed-off-by: HikaruEgashira <[email protected]>

… with success message (#4863) Co-authored-by: Amed Rodriguez <[email protected]> # Conflicts: # ui/desktop/src/components/settings/extensions/ExtensionsSection.tsx # ui/desktop/src/components/settings/extensions/deeplink.ts

…orphaned-tool-calls * 'release/1.9.2' of github.com:block/goose: Fix auto scroll to bottom during chat (#4923) fix redirect to extensions page after deeplink install and show toast with success message (#4863) added claude-sonnet-4-5 (#4906) Upgrade electron for macOS Tahoe compatibility (#5015) chore(release): release version 1.9.2

zanesq added 2 commits September 26, 2025 12:04

fix redirect to extensions page after deeplink install and show toast…

f1c4858

… with success message

remove comment

240dad1

DOsinga approved these changes Sep 26, 2025

View reviewed changes

zanesq added 3 commits September 29, 2025 09:26

reuse createNavigationHandler and remove unnecessary memoization

e6cc7fb

make setView required

dc346a0

remove unnecessary try catch and log

1da8ca1

DOsinga self-assigned this Sep 29, 2025

zanesq added 2 commits September 30, 2025 14:54

remove extensionId and use config name instead

8cf7a93

amed-xyz approved these changes Oct 2, 2025

View reviewed changes

Fix Streaming HTTP extension parsing

d0578db

zanesq merged commit 031b8b7 into main Oct 2, 2025
11 checks passed

zanesq deleted the zane/fix-extension-deeplink branch October 2, 2025 17:50

katzdave mentioned this pull request Oct 7, 2025

chore(release): release version 1.9.2 #5025

Closed

This was referenced Oct 8, 2025

chore(release): release version 1.10.0 #5060

Closed

release/1.10.0 #5101

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix redirect to extensions page after deeplink install and show toast with success message #4863

fix redirect to extensions page after deeplink install and show toast with success message #4863

Uh oh!

zanesq commented Sep 26, 2025 •

edited

Loading

Uh oh!

DOsinga Sep 26, 2025

Uh oh!

zanesq Oct 1, 2025

Uh oh!

Uh oh!

Uh oh!

DOsinga Sep 26, 2025

Uh oh!

zanesq Sep 29, 2025 •

edited

Loading

Uh oh!

zanesq Oct 1, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

fix redirect to extensions page after deeplink install and show toast with success message #4863

fix redirect to extensions page after deeplink install and show toast with success message #4863

Uh oh!

Conversation

zanesq commented Sep 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

DOsinga Sep 26, 2025

Choose a reason for hiding this comment

Uh oh!

zanesq Oct 1, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

DOsinga Sep 26, 2025

Choose a reason for hiding this comment

Uh oh!

zanesq Sep 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

zanesq Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

zanesq commented Sep 26, 2025 •

edited

Loading

zanesq Sep 29, 2025 •

edited

Loading

zanesq Oct 1, 2025 •

edited

Loading