Require minimum agent versions for all ug launches - #754
Merged
Merged
Conversation
Enforce Claude Code >= 2.1.259 and Codex >= 0.145.0 unconditionally instead of only when smart routing or gateway model discovery is enabled, and drop the feature-specific message casing. The launch-time recheck in _launch_smart_routing is subsumed by the bootstrap gate.
The 2.1.259 launch floor makes the managedMcpServers version probe always true, so managed_mcp_uses_managed_file no longer gates on the CLI version.
Below the version floor, ug again asks 'Upgrade <tool> if available?' before running the native upgrader (restores the flow removed in #638, now that the floor applies to every launch). Declining blocks the launch with the requirement error; EOF/piped runs take the default and upgrade so headless launches do not stall. npm-managed tools keep the silent auto-upgrade.
Drives the real installed CLI with below-floor agent pins (claude 2.1.258, codex 0.144.0): declining the required-upgrade prompt blocks the launch with the requirement restated; accepting runs the native upgrader and the floor clears only when the upgrade succeeds. The scenarios skip when the pinned agent already meets the floor, so the default CI version pins are unaffected. Verified with scripts/run_integration.py --installation-only under both pin sets.
CI rejected the skip-based selection: the integration contract bans pytest.skip and the runner fails when selected tests skip. Each agent now has a single below-floor scenario (decline blocks, accept runs the native upgrader and the floor clears only on success) that returns early when the pinned CLI already meets the floor.
andy-xu-db
approved these changes
Sep 18, 2026
lilly-luo
enabled auto-merge (squash)
September 19, 2026 00:24
lilly-luo
disabled auto-merge
September 19, 2026 00:26
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.
https://databricks.slack.com/archives/C0BT0VBHAPN/p1789766932687409
Summary
_launch_smart_routing; renameMINIMUM_ROUTING_CODEX_VERSION→MINIMUM_CODEX_VERSIONand the 0.134.0 constant →LEGACY_LAYOUT_CODEX_VERSION(layout detection only).Testing
the codex job keeps cancelling itself - it seems like we are on a bad runner? i ran the tests locally and they passed
https://github.com/databricks/unity-gateway/actions/runs/35405097543