Skip to content

Conversation

OskarStark
Copy link
Contributor

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

We should not use classes from other platforms/stores, which will be standalone packages in the future.

@OskarStark OskarStark self-assigned this Sep 27, 2025
@carsonbot carsonbot added Feature New feature Platform Issues & PRs about the AI Platform component Status: Needs Review labels Sep 27, 2025
@OskarStark OskarStark merged commit b39138c into symfony:main Sep 27, 2025
17 checks passed
@OskarStark OskarStark deleted the dedicated-model-classes branch September 28, 2025 20:48
OskarStark added a commit that referenced this pull request Sep 28, 2025
…(OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Platform] Revert "[Bedrock] Use dedicated  model class"

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| Docs?         | no
| Issues        | Reverts #687
| License       | MIT

This cannot work, as the normalizer from the contract only support the `Claude` model class from src/Bridge/Anthropic...

Commits
-------

31ef4e3 Revert "[Platform][Bedrock] Use dedicated  model class"
devmatt000 added a commit to devmatt000/dev_ai that referenced this pull request Oct 2, 2025
…(OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Platform] Revert "[Bedrock] Use dedicated  model class"

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| Docs?         | no
| Issues        | Reverts symfony/ai#687
| License       | MIT

This cannot work, as the normalizer from the contract only support the `Claude` model class from src/Bridge/Anthropic...

Commits
-------

31ef4e35 Revert "[Platform][Bedrock] Use dedicated  model class"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature Platform Issues & PRs about the AI Platform component Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants