Integration of LLM Chain #15

chr-hertel · 2025-06-06T21:56:56Z

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

Replaces #14 since this is coming with the git history.

Splits LLM Chain into three components as packages
Renames Chain component to Agent
Renames PhpLlm\LlmChain into Symfony\AI

Or what about naming it `merge()`?

* Helpful exceptions

* Introduce `Model` value object

* More specific exception

…implementations (#25) * Allow usage of gpt visions api * Add tests * Rename content classes * Move content casting of user message to message object * Simplify message interface with content classes * Rename variable * Fix image describer after latest interface change * Some review adjustments

* Change sentence in `README` * -

* Fix wrong namespace * -

* Read description for schema to gove more context to the LLM * -

No need to run cs-fixer, PHPStan etc. so often with different PHP versions

* [Store][Azure] Make vector field name configurable * -

* Add links for stores * - * -

* Better options handling * Update examples/chat-gpt-openai.php * Apply suggestions from code review

As we don't have support for it now

* Remove method `Vector::create1536` * -

Nyholm

This pr is too big and the project is too young. Let's merge so we can collaborate and improve from main

chr-hertel · 2025-06-07T08:41:50Z

Yes agree, in this case it makes sense not to squash and have a merge commit to preserve the history.

Would you still use gh or just hit the button?

fabpot · 2025-06-07T09:08:00Z

Just rebase first to get rid of the merge commits and gh will work fine.

chr-hertel · 2025-06-07T10:10:40Z

the reason for the merge commit here is that I integrated llm-chain with merge --unrelated-history on top of what we have here.

another option i see is to use the llm-chain history as base and rebase the history of symfony/ai on top of that - at least that would be easier than rebasing those 300 commits that conflict in structure 🙈

chr-hertel · 2025-06-07T10:24:27Z

@Nyholm any objections from your end if i wipe main for now and rebase the current status on top of this here?
at least i'd give it a try - should be easier than the other way around
not sure i'm able to do a force push on main tho :D

fabpot · 2025-06-07T13:10:07Z

If it’s too complicated, don’t bother and just hit the merge button.

* Rename `Registry` to `Toolbox` * refactor: rename tool registry to ToolBox --------- Co-authored-by: Christopher Hertel <mail@christopher-hertel.de>

chr-hertel · 2025-06-07T13:40:03Z

reworded the big library merge, squashed the Symfony AI changes on top and merged with no commit on main. the best way out for the history without too much of a stress.

done here :)

* demo-repo/restructure: (31 commits) refactor: restructure into demo folder chore: update to llm-chain 0.22 (#27) refactor: more error handling in vidoe example chore: symfony 7.3 update fix: pin chroma db version and clean up (#26) feat: add demo of GPT vision capabilities based on video stream (#22) chore: dependecy update (#23) feat: extend wikipedia system prompt by tools (#21) chore: update to lib v0.19 (#20) refactor: optimize audio chat ui (#19) feat: audio example (#18) chore: updating dependencies (#17) chore: composer update incl twig cve patch (#16) chore: llm chain update with system_prompt support (#15) chore: install bundle 0.12 (#14) chore: update to llm-chain 0.11.0 (#13) chore: library update (#12) refactor: follow up on example structure to have them cleaner and more separated (#11) fix: typed animation only on xhr responses (#10) refactor: moving classes to a more component like structure (#9) ...

* integrate-demo: (31 commits) refactor: restructure into demo folder chore: update to llm-chain 0.22 (#27) refactor: more error handling in vidoe example chore: symfony 7.3 update fix: pin chroma db version and clean up (#26) feat: add demo of GPT vision capabilities based on video stream (#22) chore: dependecy update (#23) feat: extend wikipedia system prompt by tools (#21) chore: update to lib v0.19 (#20) refactor: optimize audio chat ui (#19) feat: audio example (#18) chore: updating dependencies (#17) chore: composer update incl twig cve patch (#16) chore: llm chain update with system_prompt support (#15) chore: install bundle 0.12 (#14) chore: update to llm-chain 0.11.0 (#13) chore: library update (#12) refactor: follow up on example structure to have them cleaner and more separated (#11) fix: typed animation only on xhr responses (#10) refactor: moving classes to a more component like structure (#9) ...

chr-hertel and others added 30 commits September 22, 2024 22:59

fix: add composer.lock to ignore (#22)

4e361a5

fix: inconsistent readme namings (#23)

e704c55

feat: add MessageBag::withMessageBag() method (#21)

6dfaa7a

Or what about naming it `merge()`?

chore: helpful exceptions (#17)

dccc6dd

* Helpful exceptions

feat: introduce Version value object (#12)

14a9f1a

* Introduce `Model` value object

chore: more specific exception (#24)

1c1ed9a

* More specific exception

docs: readme rework (#33)

e443b48

docs: fix typo (#35)

83ff49b

docs: change sentence in README (#36)

a9d5831

* Change sentence in `README` * -

refactor: rename Runtime to Platform (#34)

ce89797

fix: wrong namespace (#40)

e9d22ad

* Fix wrong namespace * -

feat: Read description for schema to give more context to the LLM (#38)

f5accab

* Read description for schema to gove more context to the LLM * -

ci: Split pipeline (#39)

1db39a3

No need to run cs-fixer, PHPStan etc. so often with different PHP versions

feat: [Store][Azure] Make vector field name configurable (#43)

add34c8

* [Store][Azure] Make vector field name configurable * -

ci: More infos when running CS-Fixer (#44)

7e2cbca

docs: Add links for stores (#45)

af86cc2

* Add links for stores * - * -

chore: Update namespace (#46)

dadb0bd

feat: Better options handling (#41)

19fe1ea

* Better options handling * Update examples/chat-gpt-openai.php * Apply suggestions from code review

fix: Remove suggestion (#50)

6f3d4c6

As we don't have support for it now

chore: handle image support (#52)

98ad755

docs: add reasoning example (#53)

04b4abb

Add MongoDB vector store (#42)

9977e78

chore: improve example execution with make and dotenv (#54)

3ab2764

feat: introducing chain processors (#47)

2e8f434

feat: add pinecone store (#55)

ec118a0

fix: various fixes for bundle release 0.3 (#56)

1ac3431

chore: Remove method Vector::create1536 (#57)

3659701

* Remove method `Vector::create1536` * -

chore: streamline qa and phpunit config with bundle (#59)

34f6b13

fix: Metadata converting to array (#60)

8888a52

chr-hertel force-pushed the chain-integration branch from c61bd5f to 1e1f12d Compare June 6, 2025 23:24

chr-hertel marked this pull request as ready for review June 6, 2025 23:25

chr-hertel requested a review from Nyholm as a code owner June 6, 2025 23:25

chr-hertel force-pushed the chain-integration branch 3 times, most recently from 574f8f2 to 94ba24d Compare June 7, 2025 00:09

Nyholm approved these changes Jun 7, 2025

View reviewed changes

OskarStark approved these changes Jun 7, 2025

View reviewed changes

refactor: rework LLM Chain into Symfony AI

7b7dc8c

chr-hertel force-pushed the chain-integration branch from 94ba24d to 7b7dc8c Compare June 7, 2025 13:32

chr-hertel merged commit 7b7dc8c into main Jun 7, 2025
25 checks passed

chr-hertel added a commit that referenced this pull request Jun 7, 2025

refactor: rename Registry to ToolBox (#15)

06f3b62

* Rename `Registry` to `Toolbox` * refactor: rename tool registry to ToolBox --------- Co-authored-by: Christopher Hertel <mail@christopher-hertel.de>

chr-hertel deleted the chain-integration branch June 7, 2025 13:37

chr-hertel added a commit that referenced this pull request Jun 13, 2025

fix: adopt to lib changes and bring in pipeline (#15)

322011b

chr-hertel mentioned this pull request Jun 13, 2025

Integrate LLM Chain Bundle as AI Bundle #21

Merged

chr-hertel added a commit that referenced this pull request Jun 22, 2025

chore: llm chain update with system_prompt support (#15)

0afe1b9

chr-hertel added Platform Issues & PRs about the AI Platform component Agent Issues & PRs about the AI Agent component Store Issues & PRs about the AI Store component Run examples Issues & PRs about the example scripts labels Jul 12, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Integration of LLM Chain #15

Integration of LLM Chain #15

Uh oh!

chr-hertel commented Jun 6, 2025 •

edited

Loading

Uh oh!

Nyholm left a comment

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

fabpot commented Jun 7, 2025

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

fabpot commented Jun 7, 2025

Uh oh!

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

Uh oh!

Integration of LLM Chain #15

Integration of LLM Chain #15

Uh oh!

Conversation

chr-hertel commented Jun 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Nyholm left a comment

Choose a reason for hiding this comment

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

fabpot commented Jun 7, 2025

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

fabpot commented Jun 7, 2025

Uh oh!

Uh oh!

chr-hertel commented Jun 7, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

chr-hertel commented Jun 6, 2025 •

edited

Loading