Skip to content

Conversation

@michaelneale
Copy link
Collaborator

@michaelneale michaelneale commented Jan 18, 2026

trying this approach - fixes #6554

thanks to #6567 from @The-Best-Codes

fixes: #6554

@michaelneale michaelneale marked this pull request as ready for review January 19, 2026 00:27
Copilot AI review requested due to automatic review settings January 19, 2026 00:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to fix regressions in extension loading by modifying how bundled extensions are detected and initialized. The changes span both the TypeScript frontend (ConfigContext) and Rust backend (extensions config).

Changes:

  • Modified frontend to detect absence of bundled extensions and trigger sync from bundled-extensions.json on initial load
  • Changed Rust backend to unconditionally inject platform extensions instead of only when extensions map is non-empty
  • Added logic to reload extensions after syncing bundled extensions

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ui/desktop/src/components/ConfigContext.tsx Adds bundled extension detection and sync logic during initial config load
crates/goose/src/config/extensions.rs Removes conditional check so platform extensions are always injected

@michaelneale michaelneale merged commit f3c2f8e into main Jan 19, 2026
25 checks passed
@michaelneale michaelneale deleted the micn/hotfix-extension-init-new-install branch January 19, 2026 00:55
@michaelneale michaelneale changed the title fix: trying this approach to undo regressions for extension loading fix: extension loading, syncs platform/builtin configs when new install Jan 19, 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.

Built-in extensions are not built-in

4 participants