Skip to content

Conversation

@OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? no
Issues Replaces #1076
License MIT
  • Update codewithkyrian/chromadb-php requirement from ^0.2.1|^0.3|^0.4 to ^1.0
  • Update namespaces for v1.0 BC breaks:
    • QueryItemsResponse moved to Codewithkyrian\ChromaDB\Responses
    • CollectionResource replaced with Collection in Codewithkyrian\ChromaDB\Models
  • Remove queryImages parameter from query() call (removed in v1.0)
  • Add queryTexts option to query() method for text-based searches

@xprojects-de
Copy link
Contributor

nice! Like it!

@OskarStark OskarStark force-pushed the OskarStark/chromadb-php-v1 branch from bf9b4c9 to 135a219 Compare December 8, 2025 08:08
- Update codewithkyrian/chromadb-php requirement from ^0.2.1|^0.3|^0.4 to ^1.0
- Update namespaces for v1.0 BC breaks:
  - QueryItemsResponse moved to Codewithkyrian\ChromaDB\Responses
  - CollectionResource replaced with Collection in Codewithkyrian\ChromaDB\Models
- Remove queryImages parameter from query() call (removed in v1.0)
- Add queryTexts option to query() method for text-based searches
@OskarStark OskarStark force-pushed the OskarStark/chromadb-php-v1 branch from 135a219 to 537169f Compare December 8, 2025 08:09
@OskarStark OskarStark changed the title [Store] Update ChromaDB bridge to support v1.0 [Store] Use codewithkyrian/chromadb-php v1.0 Dec 8, 2025
@OskarStark OskarStark changed the title [Store] Use codewithkyrian/chromadb-php v1.0 [Store] Use codewithkyrian/chromadb-php v1.0 and drop 0.x Dec 8, 2025
@OskarStark OskarStark merged commit d8a3938 into symfony:main Dec 8, 2025
15 checks passed
@rvmourik
Copy link

rvmourik commented Dec 9, 2025

Could it be that this change causes an error when trying to index the stores, I just upgraded to the latest versions and changed the config.

image

The set-up is running with docker in combination with symfony cli, before upgrading the indexing worked

@chr-hertel
Copy link
Member

@rvmourik needed to fix that in the demo as well: https://github.com/symfony/ai/pull/1086/files

not sure if we need to change something here - i think it's just a breaking change we inherit from the package, see https://github.com/CodeWithKyrian/chromadb-php/releases/tag/1.0.0

@rvmourik
Copy link

rvmourik commented Dec 9, 2025

@rvmourik needed to fix that in the demo as well: https://github.com/symfony/ai/pull/1086/files

not sure if we need to change something here - i think it's just a breaking change we inherit from the package, see https://github.com/CodeWithKyrian/chromadb-php/releases/tag/1.0.0

@chr-hertel thanks for the quick reply, that works indeed.

Offtopic, loved your talk at Symfony Con in Amsterdam

@OskarStark OskarStark deleted the OskarStark/chromadb-php-v1 branch December 9, 2025 16:06
@chr-hertel
Copy link
Member

Happy to hear that @rvmourik & thanks for the feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature Status: Needs Review Store Issues & PRs about the AI Store component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants