Skip to content

[AIGTWY-4573] Claude configuration: managed models, headers, discovery - #646

Closed
david-siqi-liu wants to merge 1 commit into
david/AIGTWY-4573-5-spendfrom
david/AIGTWY-4573-6-claude
Closed

david-siqi-liu wants to merge 1 commit into
david/AIGTWY-4573-5-spendfrom
david/AIGTWY-4573-6-claude

Conversation

@david-siqi-liu

@david-siqi-liu david-siqi-liu commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Apply managed default models, custom headers, and the discovery location to Claude's config, and curate the /model picker from the manifest. This is the content of #597.

  • agents/claude.py model/header/discovery rendering and picker curation
  • databricks.py extra custom headers
  • managed_resolve.py Claude family / static-list / discovery-location helpers

Integration coverage (existing): test_ug_configure_claude_databricks (managed models/headers/discovery applied, then a real Claude TUI file task), test_ug_configure_claude_anthropic_mps (provider picker + model), test_ug_claude_headless_prompt_{argument,stdin,after_separator} (launch on managed models), test_ug_claude_preserves_caller_settings_and_hook (custom-header/settings merge preserves caller settings and runs the real SessionStart hook), test_ug_claude_reports_unsupported_short_model_option. Unit: tests/test_agent_claude.py, tests/test_managed_resolve.py. No integration test added or modified (behavior-preserving).

This pull request and its description were written by Isaac.

Apply managed default models, custom headers, and the discovery location to
Claude's config through the managed flow, with the model picker curated from
the managed config.

Co-authored-by: Isaac <no-reply@databricks.com>
@david-siqi-liu
david-siqi-liu added this pull request to stack #650 September 15, 2026 17:55
@david-siqi-liu

Copy link
Copy Markdown
Collaborator Author

Closing: re-scoping into a smaller stack focused on making static model lists work for Claude Code and Codex. The rest (tracing, UC-location discovery, MCP, skills, spend defaults, smart routing) follows separately.

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.

1 participant