Skip to content

fix extension selector syncing issues#7133

Merged
katzdave merged 2 commits intomainfrom
zane/extension-selector-syncing
Feb 11, 2026
Merged

fix extension selector syncing issues#7133
katzdave merged 2 commits intomainfrom
zane/extension-selector-syncing

Conversation

@zanesq
Copy link
Collaborator

@zanesq zanesq commented Feb 11, 2026

Extension Selector Sync Fix

Problem

The extension count/list in the bottom bar (puzzle icon) was out of sync with
the toast notifications that appear when extensions load at session start.

Root Cause

  • SESSION_EXTENSIONS_LOADED was dispatched in sessions.ts BEFORE resumeAgent
    was called, meaning extensions hadn't actually loaded yet
  • BottomMenuExtensionSelection compensated with a 500ms setTimeout hack
    listening to SESSION_CREATED and MESSAGE_STREAM_FINISHED, which was unreliable

Fix (frontend-only, 3 files, +1/-13 lines)

useChatStream.ts

  • Dispatch SESSION_EXTENSIONS_LOADED right next to showExtensionLoadResults()
    so the bottom bar refreshes at the exact moment toasts appear

sessions.ts

  • Removed premature SESSION_EXTENSIONS_LOADED dispatch that fired before
    extensions were actually loaded

BottomMenuExtensionSelection.tsx

  • Removed SESSION_CREATED and MESSAGE_STREAM_FINISHED listeners with 500ms
    delay hack
  • Now only listens for SESSION_EXTENSIONS_LOADED which fires at the correct time

Problem:

Starting a new chat from the sidebar "Chat" button loaded fewer extensions
than starting from the home screen (Hub).

Root Cause:

AppSidebar.tsx called startNewSession without passing the user's enabled
extensions list. The backend fell back to its own default config, which
could differ from what the UI had enabled.

Fix (AppSidebar.tsx, +4/-2):

  • Pass configContext.extensionsList as allExtensions to startNewSession,
    matching what the Hub already does
  • Added configContext.extensionsList to the useCallback dependency array

@zanesq zanesq requested a review from katzdave February 11, 2026 00:07
@katzdave katzdave added this pull request to the merge queue Feb 11, 2026
Merged via the queue into main with commit 45e84f9 Feb 11, 2026
17 of 18 checks passed
@katzdave katzdave deleted the zane/extension-selector-syncing branch February 11, 2026 01:11
zanesq added a commit that referenced this pull request Feb 11, 2026
tlongwell-block added a commit that referenced this pull request Feb 11, 2026
* origin/main: (107 commits)
  feat: Allow overriding default bat themes using environment variables (#7140)
  Make the system prompt smaller (#6991)
  Pre release script (#7145)
  Spelling (#7137)
  feat(mcp): upgrade rmcp to 0.15.0 and advertise MCP Apps UI extension capability (#6927)
  fix: ensure assistant messages with tool_calls include content field (#7076)
  fix(canonical): handle gcp_vertex_ai model mapping correctly (#6836)
  Group dependencies in root Cargo.toml (#6948)
  refactor: updated elevenLabs API module and `remove button` UX (#6781)
  fix: we were missing content from langfuse traces (#7135)
  docs: update username in authors.yml (#7132)
  fix extension selector syncing issues (#7133)
  fix(acp): per-session Agent for model isolation and load_session restore (#7115)
  fix(claude-code): defensive coding improvements for model switching (#7131)
  feat(claude-code): dynamic model listing and mid-session model switching (#7120)
  Inline worklet source (#7128)
  [docs] One shot prompting is dead - Blog Post (#7113)
  fix: correct spelling of Debbie O'Brien's name in authors.yml (#7127)
  docs: GCP Vertex AI org policy filtering & update OnboardingProviderSetup component (#7125)
  feat: replace subagent and skills with unified summon extension (#6964)
  ...

# Conflicts:
#	Cargo.lock
#	Cargo.toml
zanesq added a commit to Abhijay007/goose that referenced this pull request Feb 11, 2026
* upstream/main: (109 commits)
  [docs] Skills Marketplace UI Improvements (block#7158)
  More no-window flags (block#7122)
  feat: Allow overriding default bat themes using environment variables (block#7140)
  Make the system prompt smaller (block#6991)
  Pre release script (block#7145)
  Spelling (block#7137)
  feat(mcp): upgrade rmcp to 0.15.0 and advertise MCP Apps UI extension capability (block#6927)
  fix: ensure assistant messages with tool_calls include content field (block#7076)
  fix(canonical): handle gcp_vertex_ai model mapping correctly (block#6836)
  Group dependencies in root Cargo.toml (block#6948)
  refactor: updated elevenLabs API module and `remove button` UX (block#6781)
  fix: we were missing content from langfuse traces (block#7135)
  docs: update username in authors.yml (block#7132)
  fix extension selector syncing issues (block#7133)
  fix(acp): per-session Agent for model isolation and load_session restore (block#7115)
  fix(claude-code): defensive coding improvements for model switching (block#7131)
  feat(claude-code): dynamic model listing and mid-session model switching (block#7120)
  Inline worklet source (block#7128)
  [docs] One shot prompting is dead - Blog Post (block#7113)
  fix: correct spelling of Debbie O'Brien's name in authors.yml (block#7127)
  ...
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
Tyler-Hardin pushed a commit to Tyler-Hardin/goose that referenced this pull request Feb 11, 2026
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.

2 participants