mirror of
https://github.com/aaif-goose/goose.git
synced 2026-07-03 14:10:03 +02: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.