Don't install Databricks AI Tools on agent launch (configure-only) - #350
Merged
Conversation
`ucode claude` (and the other launch commands) auto-configure an unseen agent via configure_single_tool, which also installed Databricks AI Tools (skills + plugins). That put skill installation on the launch path, so a plain `ucode claude --provider ... --workspace ...` printed "Installing Databricks AI Tools for claude-code..." unprompted. Move the install out of the shared configure_single_tool and into the `ucode configure` callers only: - configure_single_tool no longer installs (launch auto-configures through it). - configure_workspace_command installs explicitly after it (single-agent path); the multi-agent path already installs via configure_selected_tools. Result: launch never installs skills; `ucode configure` still does (default on, `--disable-databricks-ai-tools` to opt out). Co-authored-by: Isaac
lilly-luo
reviewed
Aug 17, 2026
lilly-luo
reviewed
Aug 17, 2026
…_agents Clarify (per review) that these are Databricks' AI Tools, not the customer's. Co-authored-by: Isaac
lilly-luo
reviewed
Aug 17, 2026
lilly-luo
approved these changes
Aug 17, 2026
Co-authored-by: Isaac
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Running a launch command like:
prints "Installing Databricks AI Tools for claude-code..." unprompted. Launch commands auto-configure an unseen agent through
configure_single_tool, which also installed Databricks AI Tools (skills + plugins). That put skill installation on the launch path, not just the configure path.Fix
Move the install out of the shared
configure_single_tooland into theucode configurecallers only:configure_single_toolno longer installs — the launch path (_auto_configure_tool) auto-configures through it and must never install skills.configure_workspace_commandinstalls explicitly afterconfigure_single_toolon the single-agent path; the multi-agent path already installs viaconfigure_selected_tools.Result
ucode claude, etc.) never installs skills.ucode claude --provider ... --workspace ...now just configures the agent for the provider and launches — no install step.ucode configurestill installs, on by default, with--disable-databricks-ai-toolsto opt out (unchanged).Testing
TestConfigureWiresAiToolsInstall:configure_single_toolis now asserted to not install;configure_selected_toolsstill does.databricks aitoolssubprocess call).tests/test_agents_init.py+tests/test_cli.py: 288 passed. Lint + format clean.This pull request and its description were written by Isaac.