[Platform][Ollama] Add support for generation endpoint by Guikingone · Pull Request #1762 · symfony/ai

Guikingone · 2026-03-13T12:16:58Z

Q	A
Bug fix?	no
New feature?	yes
Docs?	yes
Issues	Suggested by @WebMamba on Slack
License	MIT

Introduce support for /api/generate endpoint
Up PHPStan analysis to 10 (a few issues were detected)

chr-hertel · 2026-03-14T00:40:38Z

do the examples work for you?

that's what i'm currently getting after composer u && ../link:

Guikingone · 2026-03-14T18:15:44Z

Was working perfectly yesterday, I'll take a look at it tomorrow

Guikingone · 2026-03-16T12:02:09Z

Found the issue, the Capability::INPUT_TEXT is missing in ModelCatalog, IMHO, we should merge #1767 before this one, this way, we solve the issue for the hard-coded catalog and the generation capability.

Guikingone · 2026-03-17T14:32:01Z

@chr-hertel Fixed, examples are working like a charm now thanks to the new INPUT_TEXT capability 😅

…kingone) This PR was merged into the main branch. Discussion ---------- [Platform][Ollama] Remove prefix for ScopingHttpClient | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Docs? | yes | Issues | Discussed in #1762 (comment). | License | MIT Commits ------- 7c1024a refactor(ollama): remove prefix for ScopingHttpClient

Guikingone · 2026-04-09T09:10:11Z

@chr-hertel I updated the PR following the one you merged, same for tests, ready for review when you have time 🙂

Copilot

Pull request overview

This PR extends the Platform’s Ollama bridge to support the /api/generate text-generation endpoint (including streaming), updates capability detection from the Ollama model catalog, and adds new documentation/examples while increasing PHPStan strictness for the bridge.

Changes:

Add /api/generate handling in the Ollama client and result converter (plus related tests).
Expand model capability mapping (including new Capability::OUTPUT_EMBEDDINGS) and adjust catalog tests.
Add Ollama-specific docs and runnable examples; raise Ollama bridge PHPStan level to 10.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 13 comments.

Show a summary per file

File	Description
src/platform/src/Capability.php	Adds `OUTPUT_EMBEDDINGS` capability; minor comment normalization.
src/platform/src/Bridge/Ollama/phpstan.dist.neon	Raises PHPStan level from 6 to 10 for the Ollama bridge.
src/platform/src/Bridge/Ollama/TokenUsageExtractor.php	Tightens token usage extraction to require integer counts.
src/platform/src/Bridge/Ollama/OllamaResultConverter.php	Routes conversion based on endpoint URL; adds generate + improved streaming handling.
src/platform/src/Bridge/Ollama/OllamaClient.php	Introduces generation requests and refactors chat/embed request payload construction.
src/platform/src/Bridge/Ollama/OllamaPayload.php	New helper for distinguishing generation vs completion payload shapes.
src/platform/src/Bridge/Ollama/ModelCatalog.php	Revises mapping from Ollama API “capabilities” to Platform `Capability` set.
src/platform/src/Bridge/Ollama/Tests/OllamaResultConverterTest.php	Adds generation + endpoint-aware conversion tests; adjusts streaming/vector tests.
src/platform/src/Bridge/Ollama/Tests/OllamaClientTest.php	Updates request URL expectations and adds generation coverage.
src/platform/src/Bridge/Ollama/Tests/ModelCatalogTest.php	Updates/expands expected capabilities and renames the test class.
src/platform/src/Bridge/Ollama/CHANGELOG.md	Documents `/api/generate` support under 0.9.
examples/ollama/text-generation.php	New basic text generation example.
examples/ollama/text-generation-as-stream.php	New streaming text generation example.
examples/ollama/text-generation-structured-output-math.php	New structured output example for generation.
docs/components/platform/ollama.rst	New end-user documentation page for the Ollama bridge.

Guikingone · 2026-04-23T07:43:54Z

@OskarStark Fixes applied on Copilot review, totally forgot about the new public API for the platform, sorry 😅

…kingone) This PR was merged into the main branch. Discussion ---------- [Platform][Ollama] Remove prefix for ScopingHttpClient | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Docs? | yes | Issues | Discussed in symfony/ai#1762 (comment). | License | MIT Commits ------- 7c1024a0 refactor(ollama): remove prefix for ScopingHttpClient

chr-hertel · 2026-05-05T22:27:59Z

This really got me thinking ... do we want to provide all kind of endpoint per provider/bridge? yes, i agree nowadays on that :D

with all bridges, and especially ModelClient implementations, that gets messy tho. i was tinkering around a bit with claude about that topic, and we ended up with #2029

don't think all the naming is great, and the PR needs slicing down, but was curious what you think about that?

Guikingone · 2026-05-07T13:35:36Z

Hi @chr-hertel

#2029 make sense, especially the "per endpoint split" plus it could open the door for a better separation between HTTP handling and websockets (huge part for certain bridges), it could also ease the maintenance of "per endpoint" bugs / improvements, a huge win for future bridges / features IMHO 🤔

Guikingone · 2026-05-16T10:04:17Z

@chr-hertel Saw that 0.9 is out, don't know if we should wait for #2029 for this one, if so, let me know once it's merged, I'd rework this branch to support the new architecture 🙂

Guikingone force-pushed the refactoring/ollama branch 2 times, most recently from f387b67 to 9d53fb5 Compare March 13, 2026 13:41

Guikingone marked this pull request as ready for review March 13, 2026 13:46

Guikingone requested review from OskarStark and chr-hertel as code owners March 13, 2026 13:46

carsonbot added Feature New feature Platform Issues & PRs about the AI Platform component Status: Needs Review labels Mar 13, 2026

Guikingone mentioned this pull request Mar 14, 2026

[Platform][Ollama] Replace ModelCatalog by OllamaApiCatalog #1767

Merged

Guikingone force-pushed the refactoring/ollama branch from 9d53fb5 to 5e7d7df Compare March 17, 2026 14:27

Guikingone force-pushed the refactoring/ollama branch 5 times, most recently from 30ef6c1 to f32de29 Compare March 29, 2026 17:00

Guikingone force-pushed the refactoring/ollama branch 3 times, most recently from 5fdce7f to 12b01f7 Compare April 5, 2026 15:16

chr-hertel requested changes Apr 5, 2026

View reviewed changes

carsonbot added Status: Needs Work and removed Status: Needs Review labels Apr 5, 2026

Guikingone force-pushed the refactoring/ollama branch from 12b01f7 to fe20df7 Compare April 7, 2026 12:30

carsonbot added Status: Needs Review and removed Status: Needs Work labels Apr 7, 2026

Guikingone force-pushed the refactoring/ollama branch 2 times, most recently from e64d0ef to ea72901 Compare April 7, 2026 12:58

Guikingone requested a review from chr-hertel April 7, 2026 13:03

OskarStark reviewed Apr 7, 2026

View reviewed changes

Comment thread src/platform/src/Bridge/Ollama/ModelCatalog.php

Guikingone mentioned this pull request Apr 7, 2026

[Platform][Ollama] Remove prefix for ScopingHttpClient #1888

Merged

Guikingone force-pushed the refactoring/ollama branch from ea72901 to 7ebe685 Compare April 8, 2026 14:52

Guikingone requested a review from OskarStark April 8, 2026 15:25

Guikingone force-pushed the refactoring/ollama branch 2 times, most recently from 04b6699 to e56eb4a Compare April 9, 2026 09:04

Guikingone force-pushed the refactoring/ollama branch 3 times, most recently from ff9a46f to a034898 Compare April 13, 2026 07:08

Guikingone force-pushed the refactoring/ollama branch 5 times, most recently from b1f52ec to 2476e26 Compare April 21, 2026 07:35

OskarStark requested a review from Copilot April 23, 2026 06:03

Copilot started reviewing on behalf of OskarStark April 23, 2026 06:03 View session

Copilot AI reviewed Apr 23, 2026

View reviewed changes

Guikingone force-pushed the refactoring/ollama branch from 2476e26 to 2cb53f5 Compare April 23, 2026 07:27

Guikingone force-pushed the refactoring/ollama branch from 2cb53f5 to fa47ad7 Compare May 5, 2026 13:27

Guikingone force-pushed the refactoring/ollama branch from fa47ad7 to ad18673 Compare May 16, 2026 09:57

refactor(platform): add support for generation endpoint on Ollama

55ada32

Guikingone force-pushed the refactoring/ollama branch from ad18673 to 55ada32 Compare June 8, 2026 14:29

Uh oh!

Conversation

Guikingone commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chr-hertel commented Mar 14, 2026

Uh oh!

Guikingone commented Mar 14, 2026

Uh oh!

Guikingone commented Mar 16, 2026

Uh oh!

Guikingone commented Mar 17, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Guikingone commented Apr 9, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Guikingone commented Apr 23, 2026

Uh oh!

chr-hertel commented May 5, 2026

Uh oh!

Guikingone commented May 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Guikingone commented May 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Guikingone commented Mar 13, 2026 •

edited

Loading

Guikingone commented May 7, 2026 •

edited

Loading