[AI Bundle][Platform] Allow configure Bedrock platform via bundle config #1206

uerka · 2025-12-18T19:28:54Z

Q	A
Bug fix?	yes
New feature?	yes
Docs?	no
Issues
License	MIT

Currently it seems not possible to define bedrock as a platform.
Added configuration for this platform with support of multiple instances (for example per AWS region). Added tests for model clients. Changed Bedrock's PlatformFactory to have nullable BedrockRuntimeClient

So now the AWS bedrock can be configured together with other platforms as follows:

ai:
    platform:
        bedrock:
            default: ~
            us:
                bedrock_runtime_client: 'async_aws.client.bedrock_runtime_us'
    agent:
        default:
            platform: 'ai.platform.bedrock_default'
            model:
                name: 'claude-sonnet-4-5-20250929'

Also fixed some smaller bugs for Nova model (exception of sending model name).

uerka · 2025-12-18T20:01:36Z

It seems like to run tests successfully for ai-bundle link call would be needed (at least it helped for me locally). Is it possible or should i search for another approach?
Also not sure i understand why demo tests are failing - as far as i see there is nothing related bedrock.

could someone help me out with those ?

PS big thanks for quick merge of previous issue, it was my first one contribution, so pardon me for silly questions))

chr-hertel

Thanks @uerka for picking up here 👍

chr-hertel · 2025-12-19T11:40:01Z

src/ai-bundle/src/AiBundle.php

+            if (!ContainerBuilder::willBeAvailable('symfony/ai-bedrock-platform', BedrockFactory::class, ['symfony/ai-bundle'])) {
+                throw new RuntimeException('Bedrock platform configuration requires "symfony/ai-bedrock-platform" package. Try running "composer require symfony/ai-bedrock-platform".');
+            }


needs a check for [] === $type['bedrock'] or to be moved into foreach

see #1214 for similar

carsonbot · 2025-12-19T12:22:41Z

It looks like you unchecked the "Allow edits from maintainer" box. That is fine, but please note that if you have multiple commits, you'll need to squash your commits into one before this can be merged. Or, you can check the "Allow edits from maintainers" box and the maintainer can squash for you.

Cheers!

Carsonbot

OskarStark · 2025-12-19T12:26:20Z

docs/bundles/ai-bundle.rst

                model: 'text-to-speech'
                tools: false
+            nova:
+                platform: 'ai.platform.bedrock_default


Suggested change

platform: 'ai.platform.bedrock_default

platform: 'ai.platform.bedrock_default‚

guess it was about missing quote mark, fixed)

nicodemuz · 2025-12-19T13:18:04Z

I noticed your tests are failing cause of similar reasons as in my PR: #1139

Any idea how to fix these type of issues? Do we need to add new method parameters as last to avoid breaking BC?

src/platform/src/Bridge/Bedrock/PlatformFactory.php

OskarStark · 2025-12-20T07:54:03Z

src/ai-bundle/src/AiBundle.php

+                    throw new RuntimeException('Bedrock platform configuration requires "symfony/ai-bedrock-platform" package. Try running "composer require symfony/ai-bedrock-platform".');
+                }
+
+                $platformId = 'ai.platform.bedrock_'.$name;


@chr-hertel should we go with

Suggested change

$platformId = 'ai.platform.bedrock_'.$name;

$platformId = 'ai.platform.bedrock.'.$name;

?

yes, would be more consistent - see azure

then i guess we might want to change here to avoid platform named "ai.platform.bedrock.eu" to become "ai.traceable_platform.eu" that could collide with azure/generic or other platforms. should we go with that change or change azure/generic as atm it seems should handle same strange traceable platform name conversion?

I agree, we should change that to ai.traceable_platform.bedrock.eu, same for azure, it is more consistent, isn't it @chr-hertel ?

But thats subject to change in a follow up PR, lets merge it like this

I crafted a PR:

[AI Bundle] Fix traceable platform service naming to include platform type to avoid collisions #1230

OskarStark · 2025-12-20T07:54:32Z

src/platform/src/Bridge/Bedrock/Anthropic/ClaudeModelClient.php

        return new RawBedrockResult($this->bedrockRuntimeClient->invokeModel(new InvokeModelRequest($request)));
    }

-    public function convert(InvokeModelResponse $bedrockResponse): ToolCallResult|TextResult


why removing this?

because conversion atm handled with ResultConverter, so this code seems not supposed to be called like that.

Extracted to:

[Platform][Bedrock][Anthropic] Remove unused ClaudeModelClient convert() method #1229

OskarStark · 2025-12-20T07:55:53Z

src/platform/src/Bridge/Bedrock/Tests/Anthropic/ClaudeModelClientTest.php

+            ->method('invokeModel')
+            ->with($this->callback(function ($arg) {
+                $this->assertInstanceOf(InvokeModelRequest::class, $arg);
+                $this->assertEquals('application/json', $arg->getContentType());


assertSame where possible please

There's a rector rule that could automate (or at least detect) this:
https://getrector.com/rule-detail/assert-equals-to-same-rector

I created an issue

#1228

OskarStark · 2025-12-20T07:58:16Z

src/platform/src/Bridge/Bedrock/ModelCatalog.php

+            'claude-opus-4-5-20251101' => [
+                'class' => Claude::class,
+                'capabilities' => [
+                    Capability::INPUT_MESSAGES,
+                    Capability::INPUT_IMAGE,
+                    Capability::OUTPUT_TEXT,
+                    Capability::OUTPUT_STREAMING,
+                    Capability::TOOL_CALLING,
+                ],
+            ],
+            'claude-sonnet-4-5-20250929' => [
+                'class' => Claude::class,
+                'capabilities' => [
+                    Capability::INPUT_MESSAGES,
+                    Capability::INPUT_IMAGE,
+                    Capability::OUTPUT_TEXT,
+                    Capability::OUTPUT_STREAMING,
+                    Capability::TOOL_CALLING,
+                ],
+            ],


Please revert, I extracted it to:

[Platform][Bedrock] Add Claude Opus 4.5 and Sonnet 4.5 models to ModelCatalog #1225

…iple instances

…hrough response conversion)

…onfig

… one configured

OskarStark changed the title ~~[AI Bundle] [Platform] allow configure bedrock platform via bundle config~~ [AI Bundle][Platform] allow configure bedrock platform via bundle config Dec 18, 2025

uerka mentioned this pull request Dec 19, 2025

Feat/bedrock platform config dekkode/symfony-ai#2

Draft

uerka force-pushed the feat/bedrock-platform-config branch from 0fb4ea9 to 6b5ad59 Compare December 19, 2025 09:36

chr-hertel requested changes Dec 19, 2025

View reviewed changes

carsonbot added the Status: Needs Work label Dec 19, 2025

carsonbot changed the title ~~[AI Bundle][Platform] allow configure bedrock platform via bundle config~~ allow configure bedrock platform via bundle config Dec 19, 2025

chr-hertel added Platform Issues & PRs about the AI Platform component AI Bundle Issues & PRs about the AI integration bundle labels Dec 19, 2025

carsonbot changed the title ~~allow configure bedrock platform via bundle config~~ [AI Bundle][Platform] allow configure bedrock platform via bundle config Dec 19, 2025

uerka force-pushed the feat/bedrock-platform-config branch from 8ff4030 to fd18c26 Compare December 19, 2025 11:58

carsonbot added Status: Needs Review and removed Status: Needs Work labels Dec 19, 2025

uerka marked this pull request as ready for review December 19, 2025 12:22

uerka requested a review from OskarStark as a code owner December 19, 2025 12:22

carsonbot added Bug Something isn't working Feature New feature labels Dec 19, 2025

OskarStark reviewed Dec 19, 2025

View reviewed changes

chr-hertel reviewed Dec 19, 2025

View reviewed changes

src/platform/src/Bridge/Bedrock/PlatformFactory.php Show resolved Hide resolved

OskarStark reviewed Dec 20, 2025

View reviewed changes

OskarStark changed the title ~~[AI Bundle][Platform] allow configure bedrock platform via bundle config~~ [AI Bundle][Platform] Allow configure Bedrock platform via bundle config Dec 20, 2025

OskarStark mentioned this pull request Dec 20, 2025

Rector #1228

Open

feat(platform): allow configure bedrock platform with support of mult…

b7feaa0

…iple instances

uerka force-pushed the feat/bedrock-platform-config branch 2 times, most recently from d476f90 to 0230ee0 Compare December 20, 2025 14:54

uerka added 13 commits December 20, 2025 16:16

fix(nova): avoid populate model name to message payload

29e001f

chore(claude): removed unnesessary convert method (as its converted t…

d880a21

…hrough response conversion)

chore(bedrock): throw exception if client is missing

d81102c

chore(bedrock): ensure bedrock platform can be initiated via bundle c…

517693b

…onfig

chore(bedrock): cover model clients with tests

b892e5e

chore(bedrock): allow pass null as bedrock client

b892aef

chore(bedrock): rollback unnecessary change

fcd9546

chore(cs): fixing cs

ca32750

fix(demo): fixing typo in package name

e005a9a

chore(cs): strict comparison to null for passed bedrock runtime client

ffa4589

chore(cs): assertEquals -> assertSame

378c58f

chore(cs): missing quote

8f14ba0

fix(bundle): check for bedrock platform package only in case at least…

98e7ab2

… one configured

uerka force-pushed the feat/bedrock-platform-config branch from f4858e8 to 98e7ab2 Compare December 20, 2025 15:18

chore(cs): rollback assertEquals -> assertSame where was not intended

2f4c870

	platform: 'ai.platform.bedrock_default
	platform: 'ai.platform.bedrock_default‚

	$platformId = 'ai.platform.bedrock_'.$name;
	$platformId = 'ai.platform.bedrock.'.$name;

Uh oh!

[AI Bundle][Platform] Allow configure Bedrock platform via bundle config #1206

Are you sure you want to change the base?

[AI Bundle][Platform] Allow configure Bedrock platform via bundle config #1206

Uh oh!

Conversation

uerka commented Dec 18, 2025 • edited by OskarStark Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

uerka commented Dec 18, 2025

Uh oh!

chr-hertel left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

carsonbot commented Dec 19, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nicodemuz commented Dec 19, 2025

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

uerka Dec 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

OskarStark Dec 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

uerka Dec 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

OskarStark Dec 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

uerka commented Dec 18, 2025 •

edited by OskarStark

Loading

uerka Dec 20, 2025 •

edited

Loading

OskarStark Dec 21, 2025 •

edited

Loading

uerka Dec 20, 2025 •

edited

Loading

OskarStark Dec 21, 2025 •

edited

Loading