Skip to content

[AIGTWY-4573] Configure branching: managed-config fetch and cache foundation - #641

Closed
david-siqi-liu wants to merge 1 commit into
mainfrom
david/AIGTWY-4573-1-configure-branch
Closed

david-siqi-liu wants to merge 1 commit into
mainfrom
david/AIGTWY-4573-1-configure-branch

Conversation

@david-siqi-liu

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

Copy link
Copy Markdown
Collaborator

Parse, fetch, and cache the managed CodingAgentConfig so the rest of the stack has a single normalized source. First layer of the AIGTWY-4573 carve: behavior-identical to the reviewed #593 + #597, reorganized into small reviewable pieces.

  • Frozen dataclasses (from_wire / to_internal) that normalize the admin manifest into the internal dict
  • Fetch and cache helpers, the ManagedConfigResult 3-tuple, and the newer-than-applied comparison

Integration coverage (existing): every configure CUJ fetches, parses, and caches the managed config as its first step, so test_ug_configure_claude_databricks and test_ug_configure_codex_databricks (smoke) exercise this end to end; test_ug_configure_{claude,codex}_repeat_and_revert covers cache reuse across repeated configure. Unit: tests/test_managed_config.py. No integration test added or modified (behavior-preserving).

This pull request and its description were written by Isaac.

…ndation

Add the launch/refresh-side managed-config data path (fetch, ManagedConfigResult,
version watermark comparison, and the OS-managed reconcile helpers) that the
configure and launch paths build on. No configure/launch wiring yet; that lands
on top of this in the same PR.

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