mirror of
https://github.com/block/goose.git
synced 2026-07-10 00:20:17 +00:00
a2f9658c1e
CI (test_prompt_codemode) exposed a flaw in the first fix. run_read_migrations synthesizes a config entry for every platform extension using its default_enabled value, so a default-off extension (e.g. code_execution, chatrecall) ends up with enabled: false in config even when the user never touched it. The previous is_extension_explicitly_disabled treated that as a user disable and skipped code_execution even when the ACP server explicitly requested it via builtins (as code mode does). Replace with is_builtin_disabled_by_user, which only skips a builtin when it is disabled AND default_enabled. This still fixes the developer/desktop bug (#10221, developer is default-on) while letting explicit builtins requests load default-off extensions. Add regression tests: default-off builtin loads when explicitly requested. |
||
|---|---|---|
| .. | ||
| goose | ||
| goose-acp-macros | ||
| goose-cli | ||
| goose-download-manager | ||
| goose-local-inference | ||
| goose-mcp | ||
| goose-provider-types | ||
| goose-providers | ||
| goose-sdk | ||
| goose-sdk-types | ||
| goose-server | ||
| goose-test | ||
| goose-test-support | ||