Skip to content

Conversation

@Yorisoft
Copy link
Contributor

@Yorisoft Yorisoft commented Jan 13, 2026

Summary

Add a condition in ExtensionInstallModal.tsx to check existing extension when installing new extensions.

  • Ignore deeplink if extension is already installed.
  • Prompt to install extension if it is not already installed.

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Manually - Observed expected changes.

Related Issues

Relates to: fixes #5130

Before:
Screencast_20260113_154618

After:
Screencast_20260113_151508

@Yorisoft Yorisoft marked this pull request as draft January 13, 2026 15:50
@Yorisoft Yorisoft force-pushed the yorisoft/fix_deeplink_install_refresh branch from 8a5db7e to f22c81a Compare January 13, 2026 20:07
Yorisoft and others added 3 commits January 13, 2026 15:27
Signed-off-by: Yelsin Sepulveda <[email protected]>
Signed-off-by: Yelsin Sepulveda <[email protected]>
@Yorisoft Yorisoft marked this pull request as ready for review January 15, 2026 05:19
Copy link
Collaborator

@zanesq zanesq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM the ref is unfortunately needed in its current state

@zanesq zanesq merged commit 5e815e2 into block:main Jan 15, 2026
17 checks passed
zanesq added a commit that referenced this pull request Jan 15, 2026
…ased

* 'main' of github.com:block/goose:
  Fix popular topics not starting chat when clicked (#6508)
  fix[desktop]: deeplink ui repeat on refresh (#6469)
  fixed test compilation on main branch (#6512)
  fix: correctly parse extension name from tool call for MCP apps (#6482)
  fixed 0 token in openrouter steaming (#6493)
  feat(goose-acp): enable parallel sessions with isolated agent state (#6392)
  copilot instruction to flag prelease docs (#6504)
  docs: acp mcp support (#6491)
  feat: add flatpak support for linux (#6387)
@Yorisoft
Copy link
Contributor Author

@zanesq Thanks for the help!

lifeizhou-ap added a commit that referenced this pull request Jan 15, 2026
* main:
  don't add escaping to the command field (#6519)
  Fix popular topics not starting chat when clicked (#6508)
  fix[desktop]: deeplink ui repeat on refresh (#6469)
  fixed test compilation on main branch (#6512)
  fix: correctly parse extension name from tool call for MCP apps (#6482)
zanesq added a commit that referenced this pull request Jan 16, 2026
* 'main' of github.com:block/goose: (28 commits)
  chore(deps): bump aiohttp from 3.13.0 to 3.13.3 in /scripts/provider-error-proxy (#6539)
  chore(deps): bump brotli from 1.1.0 to 1.2.0 in /scripts/provider-error-proxy (#6538)
  docs: temp correction for agent directory (#6544)
  chore: upgrade rmcp (#6516)
  docs: clarify directory in /documentation readme (#6541)
  Release 1.20.0
  Standalone mcp apps (#6458)
  don't add escaping to the command field (#6519)
  Fix popular topics not starting chat when clicked (#6508)
  fix[desktop]: deeplink ui repeat on refresh (#6469)
  fixed test compilation on main branch (#6512)
  fix: correctly parse extension name from tool call for MCP apps (#6482)
  fixed 0 token in openrouter steaming (#6493)
  feat(goose-acp): enable parallel sessions with isolated agent state (#6392)
  copilot instruction to flag prelease docs (#6504)
  docs: acp mcp support (#6491)
  feat: add flatpak support for linux (#6387)
  fix(code_execution): serialize record_result output as JSON (#6495)
  perf(google): avoid accumulating thoughtSignatures across conversation history (#6462)
  fix(openai): make tool_call arguments optional and fix silent stream termination (#6309)
  ...
fbalicchia pushed a commit to fbalicchia/goose that referenced this pull request Jan 23, 2026
Signed-off-by: Yelsin Sepulveda <[email protected]>
Co-authored-by: Zane Staggs <[email protected]>
Signed-off-by: fbalicchia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refreshing after extension installation deeplink shows installation dialog again

2 participants