Skip to content

Conversation

OskarStark
Copy link
Contributor

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

Another apporach to #708

@OskarStark OskarStark self-assigned this Oct 1, 2025
@carsonbot carsonbot added Status: Needs Review Feature New feature Platform Issues & PRs about the AI Platform component labels Oct 1, 2025
@OskarStark OskarStark force-pushed the OskarStark/model-size-query-support branch from a6bfc46 to c401326 Compare October 1, 2025 14:02
@OskarStark OskarStark merged commit 2efbd12 into symfony:main Oct 1, 2025
OskarStark added a commit that referenced this pull request Oct 2, 2025
…on in config (sonnymilton)

This PR was squashed before being merged into the main branch.

Discussion
----------

[AI Bundle] Add parsing of model names with colon notation in config

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | - <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

In #712, support for Ollama models with colon notation was added.
However, in the bundle configuration, `parse_url()` incorrectly treated the part before the colon as a URI scheme and discarded it.
As a result, model names were normalized to an empty or incomplete string, causing configuration errors.

```php
$parsed = parse_url('qwen3:0.6b'); // ['path' => '0.6b', 'scheme' => 'qwen' ]
$model = $parsed['path']; // "0.6b" ❌
```

This change ensures that model names written in colon notation are preserved correctly when parsing bundle configuration, allowing proper usage of Ollama models in Symfony projects.

Commits
-------

56714f3 [AI Bundle] Add parsing of model names with colon notation in config
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