Files
Michael Neale a2f9658c1e fix: only skip builtins the user turned off from default-on
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.
2026-07-03 14:47:29 +10:00
..