Skip to content

Propagate managed Claude defaults with model discovery - #753

Merged
andy-xu-db merged 6 commits into
mainfrom
andy/managed-claude-uc-defaults
Sep 18, 2026
Merged

andy-xu-db merged 6 commits into
mainfrom
andy/managed-claude-uc-defaults

Conversation

@andy-xu-db

@andy-xu-db andy-xu-db commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • propagate managed Claude default_model with both Unity Catalog and Model Provider Service discovery
  • write every explicitly configured Claude family default for both sources
  • preserve MPS target IDs verbatim while retaining UC [1m] behavior
  • keep local and automatically discovered family defaults suppressed for families the managed config omits
  • preserve model precedence: explicit --model, managed budget recommendation, managed default, then provider fallback
  • support Fable-only MPS configurations and relayed MPS private settings

Context

PR #716 intentionally skipped automatic model resolution for managed discovery sources, but its guards also skipped defaults explicitly supplied by Coding Agent Config. This restores the managed-policy values without changing PR #744 picker behavior or unmanaged provider behavior.

Testing

  • uv run --frozen pytest tests/test_agent_claude.py tests/test_cli.py tests/test_managed_resolve.py tests/test_databricks.py -q (938 passed)
  • Ruff check and format checks for all touched files
  • independent implementation, risk, and test reviews: no remaining findings
Screenshot 2026-09-18 at 6 26 46 PM
Screen.Recording.2026-09-18.at.6.17.36.PM.mov

@andy-xu-db andy-xu-db changed the title Propagate managed Claude defaults with UC discovery Propagate managed Claude defaults with model discovery Sep 18, 2026
Comment thread src/ucode/agents/claude.py Outdated
@andy-xu-db
andy-xu-db enabled auto-merge (squash) September 18, 2026 23:30
@andy-xu-db
andy-xu-db merged commit 9542311 into main Sep 18, 2026
20 checks passed
@andy-xu-db
andy-xu-db deleted the andy/managed-claude-uc-defaults branch September 18, 2026 23:37
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.

2 participants