feat(go-models): harden ListModels for FishAudio (#15853) by hunnyboy1217 · Pull Request #15957 · infiniflow/ragflow (original) (raw)

@hunnyboy1217

…iflow#15853)

Also includes the shared gitee_test.go DSModelList->ModelList compile fix (infiniflow#15900 rename); it auto-resolves against the sibling infiniflow#15853 PRs.

Tests: voyage static-catalog test asserts the catalog + metadata; fishaudio httptest covers the happy path, title->_id fallback, missing key, and HTTP errors. go build and go vet are clean.

@dosubot Bot added size:M

This PR changes 30-99 lines, ignoring generated files.

☯️ refactor

Pull request that refactor/refine code

💞 feature

Feature request, pull request that fullfill a new feature.

labels

Jun 11, 2026

[coderabbitai[bot]](/apps/coderabbitai)

Haruko386

Haruko386

@hunnyboy1217

… to stub (infiniflow#15853)

Per review: Voyage AI's public API exposes only /v1/embeddings and /v1/rerank, with no model-list endpoint (https://docs.voyageai.com/reference). The previous static-catalog ListModels misrepresented a capability the API does not have, so revert it to the not-supported stub used by other endpoint-less providers.

Also removes the static-catalog test (and its global ProviderManager mutation, which CodeRabbit flagged as order-dependent) in favour of a small test asserting ListModels returns a not-supported error. FishAudio (which does have a /model endpoint) is unchanged.

@dosubot dosubot Bot added size:S

This PR changes 10-29 lines, ignoring generated files.

and removed size:M

This PR changes 30-99 lines, ignoring generated files.

labels

Jun 16, 2026

[coderabbitai[bot]](/apps/coderabbitai)

@Haruko386

Removed outdated comments regarding Fish Audio model parsing.

@Haruko386

@Haruko386

Removed unsupported ListModels test for Voyage.

@Haruko386 Haruko386 changed the titlefeat(go-models): implement ListModels for Voyage and FishAudio (#15853) feat(go-models): harden ListModels for FishAudio (#15853)

Jun 17, 2026

Haruko386

@dosubot dosubot Bot added the lgtm

This PR has been approved by a maintainer

label

Jun 17, 2026

@Haruko386 Haruko386 marked this pull request as ready for review

June 17, 2026 01:59

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})