Skip to content

feat(config): add baseURL parameter to Cohere reranker#558

Open
bevzzz wants to merge 1 commit intomainfrom
feat/cohere-reranker
Open

feat(config): add baseURL parameter to Cohere reranker#558
bevzzz wants to merge 1 commit intomainfrom
feat/cohere-reranker

Conversation

@bevzzz
Copy link
Copy Markdown
Collaborator

@bevzzz bevzzz commented Apr 8, 2026

Resolves #522

@bevzzz bevzzz requested a review from antas-marcin April 8, 2026 12:18
@bevzzz bevzzz self-assigned this Apr 8, 2026
@bevzzz bevzzz changed the title feat(gh-522): add baseURL parameter to cohere reranker feat(config): add baseURL parameter to cohere reranker Apr 8, 2026
@bevzzz bevzzz changed the title feat(config): add baseURL parameter to cohere reranker feat(config): add baseURL parameter to Cohere reranker Apr 8, 2026
Copy link
Copy Markdown

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@@ -1362,10 +1362,12 @@ public static Object[][] testCases() {
{
Reranker.class,
Reranker.cohere(rerank -> rerank
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can you add a test without the baseURL?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I can add that test case, but IMO it does not add any value.
CohereReranker is a plain Java record and handling "baseURL is null" is something Gson does, not us.

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.

Add baseURL to Cohere Reranker config

2 participants