Skip to content

feat: add HybridSearchInfo field to SearchResultHit#207

Closed
hungcs wants to merge 2 commits intotypesense:masterfrom
hungcs:feature/add-hybrid-search-info
Closed

feat: add HybridSearchInfo field to SearchResultHit#207
hungcs wants to merge 2 commits intotypesense:masterfrom
hungcs:feature/add-hybrid-search-info

Conversation

@hungcs
Copy link
Copy Markdown

@hungcs hungcs commented Jun 30, 2025

Add a new HybridSearchInfo field to the SearchResultHit struct to support hybrid search metadata. This field includes rank_fusion_score for combined scoring from keyword and vector search results.

Add a new HybridSearchInfo field to the SearchResultHit struct to support
hybrid search metadata. This field includes rank_fusion_score for combined
scoring from keyword and vector search results.
@haydenhoang
Copy link
Copy Markdown
Contributor

Hi @hungcs, updates must be made to the API spec in this repository.
Running go generate ./... will then pull those changes and generate the types.

@hungcs
Copy link
Copy Markdown
Author

hungcs commented Jul 28, 2025

thanks @haydenhoang, sorry for the delay - pr here: typesense/typesense-api-spec#95

@hungcs
Copy link
Copy Markdown
Author

hungcs commented Aug 1, 2025

@haydenhoang i ran go generate ./... and it ended up pulling in a lot of other api changes, since it runs on the latest openapi.yml of typesense-api-spe master - is that okay? thanks!

@haydenhoang
Copy link
Copy Markdown
Contributor

haydenhoang commented Aug 4, 2025

@hungcs that's completely normal as long as the tests pass. We actually run the command every now and then to keep the types updated. If the tests fail, it might be an indication that there is a breaking change in the api. Let me know if that happens.

@haydenhoang
Copy link
Copy Markdown
Contributor

@jasonbosco this PR looks good. Could you approve the workflow and merge it if the tests pass?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants