Skip to content

Conversation

junaidbinfarooq
Copy link
Contributor

@junaidbinfarooq junaidbinfarooq commented Sep 12, 2025

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

This PR adds support for Supabase as a vector store backend, leveraging Supabase's pgvector extension through the REST API.

Features

  • Supabase Store Bridge: Full integration with Supabase's REST API and pgvector extension
  • Bundle Configuration: Complete YAML configuration support for the AI Bundle
  • Batch Operations: Efficient document insertion with chunking (200 docs per request)
  • Vector Search: Similarity search with configurable match count and score thresholds
  • Manual Setup: Clear documentation for required database schema setup

Key Differences from Postgres Store

Unlike the direct Postgres store, the Supabase bridge:

  • Uses REST API instead of direct database connections
  • Requires manual database setup (tables, functions, indexes)
  • Supports Supabase's hosted PostgreSQL with pgvector

@carsonbot carsonbot added Bug Something isn't working Feature New feature Store Issues & PRs about the AI Store component Status: Needs Review labels Sep 12, 2025
@carsonbot carsonbot changed the title [Hackathon][Store]: Integrate supabase store into the store package [Store][Hackathon] : Integrate supabase store into the store package Sep 12, 2025
@chr-hertel chr-hertel added the Hackathon 2025 This issue or pull request was part of the Symfony AI Hackathon 2025 label Sep 12, 2025
@OskarStark OskarStark changed the title [Store][Hackathon] : Integrate supabase store into the store package [Store][Supabase] Add store Sep 12, 2025
@welcoMattic welcoMattic removed the Bug Something isn't working label Sep 12, 2025
@junaidbinfarooq junaidbinfarooq changed the title [Store][Supabase] Add store Hackathon: [Store] Integrate Supabase store into the store package Sep 12, 2025
@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch 2 times, most recently from 27c5049 to 76610d8 Compare September 12, 2025 18:22
@OskarStark OskarStark changed the title Hackathon: [Store] Integrate Supabase store into the store package [Store][Supabase] Add store Sep 12, 2025
@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch from 76610d8 to e962da1 Compare September 15, 2025 12:52
->useAttributeAsKey('name')
->arrayPrototype()
->children()
->scalarNode('url')->isRequired()->cannotBeEmpty()->end()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following #604, please make the http client configurable, you can barely copy paste from #604, thanks

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not see it done for any other store. Am I missing something here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a rebase you should

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still sclaraNode()

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not understand.
Could you help me here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, sure, please use stringNode() or integerNode()

@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch from e962da1 to 7758745 Compare September 17, 2025 17:51
@junaidbinfarooq
Copy link
Contributor Author

Addressed the review.
However, I'd like to mention that many of the comments I've seen in this and a few other PRs seem to be nitty-gritty IMO, and especially come at a later stage in the PR, unless there is a particular doc on coding standards in the said repo that I have missed so far.

@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch from 7758745 to f0bd639 Compare September 17, 2025 17:56
@OskarStark
Copy link
Contributor

We try to keep the code a little bit in sync. What coding standards do you mean?

@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch from f0bd639 to 390c3c1 Compare September 22, 2025 11:27
- Adds bundle integration
- Adds an entry into the stores factory for supabase
- Adds documentation for the new store integration
@junaidbinfarooq junaidbinfarooq force-pushed the feat/issue-320-support-for-supabase-store branch from 390c3c1 to f53dfe5 Compare September 22, 2025 13:32
@junaidbinfarooq
Copy link
Contributor Author

We try to keep the code a little bit in sync. What coding standards do you mean?

Something like a contributing guide or PSR that specifically details the coding standards to be followed when contributing to the repo. This will ensure we have a consistent coding style throughout the codebase without having the maintainers worry much about it IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature Hackathon 2025 This issue or pull request was part of the Symfony AI Hackathon 2025 Status: Needs Review Store Issues & PRs about the AI Store component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Store] Integrate Supabase as a vector store
5 participants