Skip to content

Add Claude-on-Bedrock MPS integration test - #742

Open
masonc08 wants to merge 1 commit into
mainfrom
masonc08/claude-bedrock-mps-integration
Open

masonc08 wants to merge 1 commit into
mainfrom
masonc08/claude-bedrock-mps-integration

Conversation

@masonc08

Copy link
Copy Markdown
Collaborator

What

Adds test_ug_configure_claude_bedrock_mps (tests/integration/test_ug_configure_claude.py), mirroring test_ug_configure_claude_anthropic_mps: ug configure's picker selects an AWS Bedrock (amazon_bedrock) MPS, then a real Claude Code session completes a file-reading task on it. Adds --claude-bedrock-provider (run_integration.py, default main.ucode.bedrock_e2e) and the claude_bedrock_provider fixture.

Why

Claude-on-Bedrock had solid unit coverage (region-slug pinning, provider-type allowlist, family resolution) but nothing exercised a real Bedrock MPS end-to-end. This is the live check that the real ug + Claude Code binary resolve a region-prefixed Bedrock slug (us.anthropic.claude-...) into a working launch — including the opus-less tier fallback, since the CI MPS has Sonnet + Haiku but no Opus (launch tier preference sonnet > opus > haiku picks Sonnet).

No allowlist dependency: amazon_bedrock is already in Claude's provider-type allowlist on main, so this stands alone.

Test

Backed by main.ucode.bedrock_e2e on the CI workspace (Sonnet + Haiku). ruff + py_compile clean. Gateway routing verified live against the MPS (Haiku /v1/messages → 200); the full session launch runs in the integration CI container.

This pull request and its description were written by Isaac.

Mirror test_ug_configure_claude_anthropic_mps for an amazon_bedrock MPS: the ug
configure picker selects the Bedrock MPS, then a real Claude Code session
completes a file task. Adds --claude-bedrock-provider (run_integration.py,
default main.ucode.bedrock_e2e) plus the claude_bedrock_provider fixture.

Closes the live gap for Claude-on-Bedrock, which had unit coverage but nothing
exercising a real Bedrock MPS end-to-end (region-slug pinning and launch-model
resolution against a live service).

Co-authored-by: Isaac <no-reply@databricks.com>
@masonc08
masonc08 force-pushed the masonc08/claude-bedrock-mps-integration branch from 1181155 to 6598ba4 Compare September 21, 2026 14:06
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