-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
🧠 feat: Azure AI Search vector search support with hybrid search #10441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
🧠 feat: Azure AI Search vector search support with hybrid search #10441
Conversation
1c20d19 to
6658e37
Compare
|
|
||
| if (action === 'install') { | ||
| // Get tool config to check for sensitive fields | ||
| const { manifestToolMap } = require('~/app/clients/tools'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no dynamic imports
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
api/server/controllers/tools.js
Outdated
| try { | ||
| const { pluginKey } = req.params; | ||
| const { manifestToolMap } = require('~/app/clients/tools'); | ||
| const { getAuthFields } = require('~/app/clients/tools/util/handleTools'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
refrain from dynamic imports
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
|
thanks, mostly looks good to me. possibly as a followup, I can see people asking for multiple vector search fields. possibly handle with 1 input, comma-delimited? can be in a followup PR after I test this a bit |
|
and just missing documentation PR |
I will create one later today! |
I'll add support for multiple vector fields as part of this PR, I also found a bug when loading pre-exisiting auth configs that don't have the newly added field yet. I'll fix that too. |
d351f73 to
187f1b6
Compare
|
@danny-avila I just fixed the merge conflict that was in The force push was because I needed to rebase. In a merge commit, some eslint errors emerged that I think were missed by contributers that don't have pre-commit running. After running |
Summary
This PR adds Azure AI Search vector search support, enabling semantic search capabilities in the Azure AI Search tool. The implementation includes:
Primary Feature - Azure AI Search Vector Search:
AZURE_AI_SEARCH_VECTOR_FIELDSconfiguration field to specify the vector field names in the Azure AI Search index2023-11-01to2025-09-01to support vector search featuresvectorQueryText: Text to be vectorized for vector search queriesfilter: OData $filter expression for search queriesquery: Text for keyword search (existing functionality)queryandvectorQueryTextare providedSupporting Features:
GET /agents/tools/:pluginKey/auth-valuesthat returns plugin authentication values with sensitive fields masked using a placeholder value (<redacted>).sensitive: trueflag to all API key fields in plugin manifest configurations. Sensitive fields are masked in the API response and can be updated without requiring the user to re-enter existing values.optionalflag in plugin auth configurations, allowing fields to be marked as optional in the UI.Dependencies:
@azure/search-documentsfrom^12.0.0to^12.2.0to support vector search featuresChange Type
Please delete any irrelevant options.
Testing
Test Configuration:
Primary Testing - Azure AI Search Vector Search:
Vector Search Configuration:
AZURE_AI_SEARCH_VECTOR_FIELDSset to the names of your vector fieldsVector Search Queries:
vectorQueryTextparameter onlyHybrid Search (Keyword + Vector):
query(keyword) andvectorQueryText(vector) parametersFilter Support:
filterparameter with OData $filter expressionsSupporting Features Testing:
Sensitive Field Handling:
GET /agents/tools/{pluginKey}/auth-values<redacted>placeholderAgent Tool Configuration UI:
Checklist
Please delete any irrelevant options.