Go: implement provider: Xinference (#14938)

### What problem does this PR solve? Closes #14808. Adds a Go model driver for Xinference so self-hosted Xinference chat models can be used through the Go provider layer instead of falling through to the dummy driver. Xinference exposes an OpenAI-compatible API under `/v1`; the driver accepts either a root endpoint such as `http://127.0.0.1:9997` or an OpenAI-compatible endpoint such as `http://127.0.0.1:9997/v1` and normalizes it before calling chat or model-listing routes. ### What is changed? - Add `internal/entity/models/xinference.go` implementing `ModelDriver` for Xinference chat. - Route provider name `xinference` in `internal/entity/models/factory.go`. - Add `conf/models/xinference.json` as a local provider config. - Add focused unit tests in `internal/entity/models/xinference_test.go`. Initial method coverage: - `ChatWithMessages`: POST `/v1/chat/completions`. - `ChatStreamlyWithSender`: SSE streaming from `/v1/chat/completions`. - `ListModels`: GET `/v1/models`. - `CheckConnection`: lightweight `ListModels` probe. - Optional auth: send `Authorization: Bearer <api_key>` only when a non-empty key is configured, matching Xinference no-auth and auth-enabled deployments. - `Balance`, `Embed`, `Rerank`, ASR, TTS, and OCR return `no such method` for this initial chat-provider PR. ### Type of change - [x] New Feature (non-breaking change which adds functionality) - [x] Bug Fix (non-breaking change which fixes an issue) ### Tests - `go test -vet=off -run TestXinference -count=1 ./internal/entity/models/...` - `go test -vet=off -count=1 ./internal/entity/models/...` ### References - Xinference docs: https://inference.readthedocs.io/zh-cn/latest/index.html - OpenAI-compatible chat usage: https://inference.readthedocs.io/zh-cn/latest/getting_started/using_xinference.html - API key auth: https://inference.readthedocs.io/zh-cn/latest/user_guide/auth_system.html --------- Co-authored-by: Jin Hai <haijin.chn@gmail.com>
2026-07-08 20:34:48 +08:00 · 2026-05-18 21:10:13 -10:00
parent 7edabdf7c3
commit 09a06f1b00
4 changed files with 799 additions and 0 deletions
--- a/conf/models/xinference.json
+++ b/conf/models/xinference.json
@@ -0,0 +1,8 @@
+{
+  "name": "xinference",
+  "url_suffix": {
+    "chat": "v1/chat/completions",
+    "models": "v1/models"
+  },
+  "class": "local"
+}
--- a/internal/entity/models/factory.go
+++ b/internal/entity/models/factory.go
@@ -89,6 +89,8 @@ func (f *ModelFactory) CreateModelDriver(providerName string, baseURL map[string
 		return NewJinaModel(baseURL, urlSuffix), nil
 	case "localai":
 		return NewLocalAIModel(baseURL, urlSuffix), nil
+	case "xinference":
+		return NewXinferenceModel(baseURL, urlSuffix), nil
 	case "longcat":
 		return NewLongCatModel(baseURL, urlSuffix), nil
 	case "novita":
--- a/internal/entity/models/xinference.go
+++ b/internal/entity/models/xinference.go
@@ -0,0 +1,476 @@
+//
+//  Copyright 2026 The InfiniFlow Authors. All Rights Reserved.
+//
+//  Licensed under the Apache License, Version 2.0 (the "License");
+//  you may not use this file except in compliance with the License.
+//  You may obtain a copy of the License at
+//
+//      http://www.apache.org/licenses/LICENSE-2.0
+//
+//  Unless required by applicable law or agreed to in writing, software
+//  distributed under the License is distributed on an "AS IS" BASIS,
+//  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+//  See the License for the specific language governing permissions and
+//  limitations under the License.
+//
+
+package models
+
+import (
+	"bufio"
+	"bytes"
+	"context"
+	"encoding/json"
+	"fmt"
+	"io"
+	"net/http"
+	"strings"
+	"sync"
+	"time"
+)
+
+// xinferenceStreamIdleTimeout bounds how long a stream can go without
+// receiving any SSE line. Self-hosted models can be slow, but a stream
+// that stays silent for a full minute is more useful as a surfaced error
+// than as a stuck goroutine.
+var xinferenceStreamIdleTimeout = 60 * time.Second
+
+// XinferenceModel implements ModelDriver for Xinference chat models.
+//
+// Xinference exposes an OpenAI-compatible API under <endpoint>/v1. The
+// tenant may configure either the root endpoint (http://127.0.0.1:9997)
+// or the OpenAI-compatible endpoint (http://127.0.0.1:9997/v1); the
+// driver normalizes both to the root endpoint before adding URLSuffix
+// values that match Xinference docs, such as v1/chat/completions.
+// Authentication is optional: no-auth deployments ignore API keys, while
+// auth-enabled deployments require Authorization: Bearer <api_key>.
+type XinferenceModel struct {
+	BaseURL    map[string]string
+	URLSuffix  URLSuffix
+	httpClient *http.Client
+}
+
+type xinferenceChatChoice struct {
+	Message struct {
+		Content          string `json:"content"`
+		ReasoningContent string `json:"reasoning_content"`
+		Reasoning        string `json:"reasoning"`
+		Thinking         string `json:"thinking"`
+	} `json:"message"`
+}
+
+type xinferenceChatResponse struct {
+	Choices []xinferenceChatChoice `json:"choices"`
+}
+
+type xinferenceModelListResponse struct {
+	Data []struct {
+		ID string `json:"id"`
+	} `json:"data"`
+}
+
+// NewXinferenceModel creates a new Xinference model instance.
+func NewXinferenceModel(baseURL map[string]string, urlSuffix URLSuffix) *XinferenceModel {
+	transport := http.DefaultTransport.(*http.Transport).Clone()
+	transport.MaxIdleConns = 100
+	transport.MaxIdleConnsPerHost = 10
+	transport.IdleConnTimeout = 90 * time.Second
+	transport.DisableCompression = false
+	transport.ResponseHeaderTimeout = 60 * time.Second
+
+	return &XinferenceModel{
+		BaseURL:   baseURL,
+		URLSuffix: urlSuffix,
+		httpClient: &http.Client{
+			Transport: transport,
+		},
+	}
+}
+
+func (x *XinferenceModel) NewInstance(baseURL map[string]string) ModelDriver {
+	return NewXinferenceModel(baseURL, x.URLSuffix)
+}
+
+func (x *XinferenceModel) Name() string {
+	return "xinference"
+}
+
+func (x *XinferenceModel) baseURLForRegion(region string) (string, error) {
+	if base, ok := x.BaseURL[region]; ok && strings.TrimSpace(base) != "" {
+		return normalizeXinferenceBaseURL(base), nil
+	}
+	if base, ok := x.BaseURL["default"]; ok && strings.TrimSpace(base) != "" {
+		return normalizeXinferenceBaseURL(base), nil
+	}
+	return "", fmt.Errorf("xinference: missing base URL, configure the Xinference endpoint (e.g., http://127.0.0.1:9997 or http://127.0.0.1:9997/v1)")
+}
+
+func normalizeXinferenceBaseURL(base string) string {
+	trimmed := strings.TrimRight(strings.TrimSpace(base), "/")
+	if trimmed == "" {
+		return trimmed
+	}
+	if strings.HasSuffix(trimmed, "/v1") {
+		return strings.TrimSuffix(trimmed, "/v1")
+	}
+	return trimmed
+}
+
+func setXinferenceAuth(req *http.Request, apiConfig *APIConfig) {
+	if apiConfig == nil || apiConfig.ApiKey == nil || *apiConfig.ApiKey == "" {
+		return
+	}
+	req.Header.Set("Authorization", fmt.Sprintf("Bearer %s", *apiConfig.ApiKey))
+}
+
+func xinferenceRegion(apiConfig *APIConfig) string {
+	if apiConfig != nil && apiConfig.Region != nil && *apiConfig.Region != "" {
+		return *apiConfig.Region
+	}
+	return "default"
+}
+
+func xinferenceReasoningFromStrings(reasoningContent string, reasoning string, thinking string) string {
+	switch {
+	case reasoningContent != "":
+		return reasoningContent
+	case reasoning != "":
+		return reasoning
+	case thinking != "":
+		return thinking
+	default:
+		return ""
+	}
+}
+
+func xinferenceReasoningFromMap(value map[string]interface{}) string {
+	for _, field := range []string{"reasoning_content", "reasoning", "thinking"} {
+		if text, ok := value[field].(string); ok && text != "" {
+			return text
+		}
+	}
+	return ""
+}
+
+func buildXinferenceChatBody(modelName string, messages []Message, stream bool, chatModelConfig *ChatConfig) map[string]interface{} {
+	apiMessages := make([]map[string]interface{}, len(messages))
+	for i, msg := range messages {
+		apiMessages[i] = map[string]interface{}{
+			"role":    msg.Role,
+			"content": msg.Content,
+		}
+	}
+
+	reqBody := map[string]interface{}{
+		"model":    modelName,
+		"messages": apiMessages,
+		"stream":   stream,
+	}
+
+	if chatModelConfig != nil {
+		if chatModelConfig.MaxTokens != nil {
+			reqBody["max_tokens"] = *chatModelConfig.MaxTokens
+		}
+		if chatModelConfig.Temperature != nil {
+			reqBody["temperature"] = *chatModelConfig.Temperature
+		}
+		if chatModelConfig.TopP != nil {
+			reqBody["top_p"] = *chatModelConfig.TopP
+		}
+		if chatModelConfig.Stop != nil {
+			reqBody["stop"] = *chatModelConfig.Stop
+		}
+	}
+
+	return reqBody
+}
+
+// ChatWithMessages sends multiple messages with roles and returns the response.
+func (x *XinferenceModel) ChatWithMessages(modelName string, messages []Message, apiConfig *APIConfig, chatModelConfig *ChatConfig) (*ChatResponse, error) {
+	if len(messages) == 0 {
+		return nil, fmt.Errorf("messages is empty")
+	}
+
+	baseURL, err := x.baseURLForRegion(xinferenceRegion(apiConfig))
+	if err != nil {
+		return nil, err
+	}
+	url := fmt.Sprintf("%s/%s", baseURL, x.URLSuffix.Chat)
+
+	reqBody := buildXinferenceChatBody(modelName, messages, false, chatModelConfig)
+	jsonData, err := json.Marshal(reqBody)
+	if err != nil {
+		return nil, fmt.Errorf("failed to marshal request: %w", err)
+	}
+
+	ctx, cancel := context.WithTimeout(context.Background(), nonStreamCallTimeout)
+	defer cancel()
+
+	req, err := http.NewRequestWithContext(ctx, "POST", url, bytes.NewBuffer(jsonData))
+	if err != nil {
+		return nil, fmt.Errorf("failed to create request: %w", err)
+	}
+	req.Header.Set("Content-Type", "application/json")
+	setXinferenceAuth(req, apiConfig)
+
+	resp, err := x.httpClient.Do(req)
+	if err != nil {
+		return nil, fmt.Errorf("failed to send request: %w", err)
+	}
+	defer resp.Body.Close()
+
+	body, err := io.ReadAll(resp.Body)
+	if err != nil {
+		return nil, fmt.Errorf("failed to read response: %w", err)
+	}
+	if resp.StatusCode != http.StatusOK {
+		return nil, fmt.Errorf("API request failed with status %d: %s", resp.StatusCode, string(body))
+	}
+
+	var result xinferenceChatResponse
+	if err = json.Unmarshal(body, &result); err != nil {
+		return nil, fmt.Errorf("failed to parse response: %w", err)
+	}
+	if len(result.Choices) == 0 {
+		return nil, fmt.Errorf("no choices in response")
+	}
+
+	content := result.Choices[0].Message.Content
+	reasonContent := xinferenceReasoningFromStrings(
+		result.Choices[0].Message.ReasoningContent,
+		result.Choices[0].Message.Reasoning,
+		result.Choices[0].Message.Thinking,
+	)
+
+	return &ChatResponse{
+		Answer:        &content,
+		ReasonContent: &reasonContent,
+	}, nil
+}
+
+// ChatStreamlyWithSender sends messages and streams response via sender.
+func (x *XinferenceModel) ChatStreamlyWithSender(modelName string, messages []Message, apiConfig *APIConfig, chatModelConfig *ChatConfig, sender func(*string, *string) error) error {
+	if sender == nil {
+		return fmt.Errorf("sender is required")
+	}
+	if len(messages) == 0 {
+		return fmt.Errorf("messages is empty")
+	}
+	if chatModelConfig != nil && chatModelConfig.Stream != nil && !*chatModelConfig.Stream {
+		return fmt.Errorf("stream must be true in ChatStreamlyWithSender")
+	}
+
+	baseURL, err := x.baseURLForRegion(xinferenceRegion(apiConfig))
+	if err != nil {
+		return err
+	}
+	url := fmt.Sprintf("%s/%s", baseURL, x.URLSuffix.Chat)
+
+	reqBody := buildXinferenceChatBody(modelName, messages, true, chatModelConfig)
+	jsonData, err := json.Marshal(reqBody)
+	if err != nil {
+		return fmt.Errorf("failed to marshal request: %w", err)
+	}
+
+	ctx, cancel := context.WithCancel(context.Background())
+	defer cancel()
+
+	req, err := http.NewRequestWithContext(ctx, "POST", url, bytes.NewBuffer(jsonData))
+	if err != nil {
+		return fmt.Errorf("failed to create request: %w", err)
+	}
+	req.Header.Set("Content-Type", "application/json")
+	setXinferenceAuth(req, apiConfig)
+
+	resp, err := x.httpClient.Do(req)
+	if err != nil {
+		return fmt.Errorf("failed to send request: %w", err)
+	}
+	defer resp.Body.Close()
+	if resp.StatusCode != http.StatusOK {
+		body, _ := io.ReadAll(resp.Body)
+		return fmt.Errorf("API request failed with status %d: %s", resp.StatusCode, string(body))
+	}
+
+	lastActive := time.Now()
+	var lastActiveMu sync.Mutex
+	done := make(chan struct{})
+	defer close(done)
+	go func() {
+		ticker := time.NewTicker(xinferenceStreamIdleTimeout / 4)
+		defer ticker.Stop()
+		for {
+			select {
+			case <-done:
+				return
+			case now := <-ticker.C:
+				lastActiveMu.Lock()
+				idle := now.Sub(lastActive)
+				lastActiveMu.Unlock()
+				if idle >= xinferenceStreamIdleTimeout {
+					cancel()
+					return
+				}
+			}
+		}
+	}()
+
+	scanner := bufio.NewScanner(resp.Body)
+	scanner.Buffer(make([]byte, 64*1024), 1024*1024)
+	sawTerminal := false
+	for scanner.Scan() {
+		lastActiveMu.Lock()
+		lastActive = time.Now()
+		lastActiveMu.Unlock()
+
+		line := scanner.Text()
+		if !strings.HasPrefix(line, "data:") {
+			continue
+		}
+		data := strings.TrimSpace(line[5:])
+		if data == "[DONE]" {
+			sawTerminal = true
+			break
+		}
+
+		var event map[string]interface{}
+		if err = json.Unmarshal([]byte(data), &event); err != nil {
+			continue
+		}
+
+		choices, ok := event["choices"].([]interface{})
+		if !ok || len(choices) == 0 {
+			continue
+		}
+		firstChoice, ok := choices[0].(map[string]interface{})
+		if !ok {
+			continue
+		}
+
+		if delta, ok := firstChoice["delta"].(map[string]interface{}); ok {
+			if reasoning := xinferenceReasoningFromMap(delta); reasoning != "" {
+				if err := sender(nil, &reasoning); err != nil {
+					return err
+				}
+			}
+			if content, ok := delta["content"].(string); ok && content != "" {
+				if err := sender(&content, nil); err != nil {
+					return err
+				}
+			}
+		}
+
+		if finishReason, ok := firstChoice["finish_reason"].(string); ok && finishReason != "" {
+			sawTerminal = true
+			break
+		}
+	}
+
+	if err := scanner.Err(); err != nil {
+		if ctx.Err() != nil {
+			return fmt.Errorf("xinference: stream idle for more than %s, aborted", xinferenceStreamIdleTimeout)
+		}
+		return fmt.Errorf("failed to scan response body: %w", err)
+	}
+	if !sawTerminal {
+		return fmt.Errorf("xinference: stream ended before [DONE] or finish_reason")
+	}
+
+	endOfStream := "[DONE]"
+	return sender(&endOfStream, nil)
+}
+
+func (x *XinferenceModel) Embed(modelName *string, texts []string, apiConfig *APIConfig, embeddingConfig *EmbeddingConfig) ([]EmbeddingData, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) Rerank(modelName *string, query string, documents []string, apiConfig *APIConfig, rerankConfig *RerankConfig) (*RerankResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) TranscribeAudio(modelName *string, file *string, apiConfig *APIConfig, asrConfig *ASRConfig) (*ASRResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) TranscribeAudioWithSender(modelName *string, file *string, apiConfig *APIConfig, asrConfig *ASRConfig, sender func(*string, *string) error) error {
+	return fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) AudioSpeech(modelName *string, audioContent *string, apiConfig *APIConfig, asrConfig *TTSConfig) (*TTSResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) AudioSpeechWithSender(modelName *string, audioContent *string, apiConfig *APIConfig, ttsConfig *TTSConfig, sender func(*string, *string) error) error {
+	return fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) OCRFile(modelName *string, content []byte, url *string, apiConfig *APIConfig, ocrConfig *OCRConfig) (*OCRFileResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) ParseFile(modelName *string, content []byte, url *string, apiConfig *APIConfig, parseFileConfig *ParseFileConfig) (*ParseFileResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+// ListModels returns the model IDs exposed by Xinference's OpenAI-compatible
+// /v1/models endpoint.
+func (x *XinferenceModel) ListModels(apiConfig *APIConfig) ([]string, error) {
+	baseURL, err := x.baseURLForRegion(xinferenceRegion(apiConfig))
+	if err != nil {
+		return nil, err
+	}
+	url := fmt.Sprintf("%s/%s", baseURL, x.URLSuffix.Models)
+
+	ctx, cancel := context.WithTimeout(context.Background(), nonStreamCallTimeout)
+	defer cancel()
+
+	req, err := http.NewRequestWithContext(ctx, "GET", url, nil)
+	if err != nil {
+		return nil, fmt.Errorf("failed to create request: %w", err)
+	}
+	setXinferenceAuth(req, apiConfig)
+
+	resp, err := x.httpClient.Do(req)
+	if err != nil {
+		return nil, fmt.Errorf("failed to send request: %w", err)
+	}
+	defer resp.Body.Close()
+
+	body, err := io.ReadAll(resp.Body)
+	if err != nil {
+		return nil, fmt.Errorf("failed to read response: %w", err)
+	}
+	if resp.StatusCode != http.StatusOK {
+		return nil, fmt.Errorf("API request failed with status %d: %s", resp.StatusCode, string(body))
+	}
+
+	var result xinferenceModelListResponse
+	if err = json.Unmarshal(body, &result); err != nil {
+		return nil, fmt.Errorf("failed to parse response: %w", err)
+	}
+
+	models := make([]string, 0, len(result.Data))
+	for _, model := range result.Data {
+		if model.ID != "" {
+			models = append(models, model.ID)
+		}
+	}
+	return models, nil
+}
+
+func (x *XinferenceModel) Balance(apiConfig *APIConfig) (map[string]interface{}, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) CheckConnection(apiConfig *APIConfig) error {
+	_, err := x.ListModels(apiConfig)
+	return err
+}
+
+func (x *XinferenceModel) ListTasks(apiConfig *APIConfig) ([]ListTaskStatus, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
+
+func (x *XinferenceModel) ShowTask(taskID string, apiConfig *APIConfig) (*TaskResponse, error) {
+	return nil, fmt.Errorf("%s, no such method", x.Name())
+}
--- a/internal/entity/models/xinference_test.go
+++ b/internal/entity/models/xinference_test.go
@@ -0,0 +1,313 @@
+package models
+
+import (
+	"encoding/json"
+	"io"
+	"net/http"
+	"net/http/httptest"
+	"strings"
+	"testing"
+	"time"
+)
+
+func newXinferenceForTest(baseURL string) *XinferenceModel {
+	return NewXinferenceModel(
+		map[string]string{"default": baseURL},
+		URLSuffix{
+			Chat:   "v1/chat/completions",
+			Models: "v1/models",
+		},
+	)
+}
+
+func withXinferenceIdleTimeout(t *testing.T, d time.Duration) {
+	t.Helper()
+	original := xinferenceStreamIdleTimeout
+	xinferenceStreamIdleTimeout = d
+	t.Cleanup(func() {
+		xinferenceStreamIdleTimeout = original
+	})
+}
+
+func TestXinferenceName(t *testing.T) {
+	x := newXinferenceForTest("http://unused")
+	if got := x.Name(); got != "xinference" {
+		t.Errorf("Name()=%q, want %q", got, "xinference")
+	}
+}
+
+func TestNormalizeXinferenceBaseURL(t *testing.T) {
+	cases := []struct {
+		in   string
+		want string
+	}{
+		{"http://127.0.0.1:9997", "http://127.0.0.1:9997"},
+		{"http://127.0.0.1:9997/", "http://127.0.0.1:9997"},
+		{"http://127.0.0.1:9997/v1", "http://127.0.0.1:9997"},
+		{" http://127.0.0.1:9997/v1/ ", "http://127.0.0.1:9997"},
+	}
+	for _, tc := range cases {
+		if got := normalizeXinferenceBaseURL(tc.in); got != tc.want {
+			t.Errorf("normalizeXinferenceBaseURL(%q)=%q, want %q", tc.in, got, tc.want)
+		}
+	}
+}
+
+func TestXinferenceFactoryRoute(t *testing.T) {
+	driver, err := NewModelFactory().CreateModelDriver("xinference", map[string]string{"default": "http://unused"}, URLSuffix{})
+	if err != nil {
+		t.Fatalf("CreateModelDriver: %v", err)
+	}
+	if driver.Name() != "xinference" {
+		t.Errorf("driver.Name()=%q, want xinference", driver.Name())
+	}
+}
+
+func TestXinferenceChatHappyPathNormalizesBaseURLAndOmitsEmptyAuth(t *testing.T) {
+	var seen map[string]interface{}
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/v1/chat/completions" {
+			t.Errorf("path=%s, want /v1/chat/completions", r.URL.Path)
+		}
+		if got := r.Header.Get("Authorization"); got != "" {
+			t.Errorf("expected no Authorization header, got %q", got)
+		}
+		raw, err := io.ReadAll(r.Body)
+		if err != nil {
+			t.Errorf("read body: %v", err)
+			return
+		}
+		if err := json.Unmarshal(raw, &seen); err != nil {
+			t.Errorf("unmarshal request: %v", err)
+			return
+		}
+		_, _ = io.WriteString(w, `{"choices":[{"message":{"content":"pong"}}]}`)
+	}))
+	defer srv.Close()
+
+	x := newXinferenceForTest(srv.URL)
+	maxTokens := 32
+	temp := 0.2
+	resp, err := x.ChatWithMessages("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "ping"}},
+		&APIConfig{},
+		&ChatConfig{MaxTokens: &maxTokens, Temperature: &temp})
+	if err != nil {
+		t.Fatalf("ChatWithMessages: %v", err)
+	}
+	if resp.Answer == nil || *resp.Answer != "pong" {
+		t.Fatalf("Answer=%v, want pong", resp.Answer)
+	}
+	if seen["stream"] != false {
+		t.Errorf("stream=%v, want false", seen["stream"])
+	}
+	if seen["max_tokens"] != float64(32) {
+		t.Errorf("max_tokens=%v, want 32", seen["max_tokens"])
+	}
+	if seen["temperature"] != 0.2 {
+		t.Errorf("temperature=%v, want 0.2", seen["temperature"])
+	}
+}
+
+func TestXinferenceChatSendsAuthHeaderWhenKeyProvided(t *testing.T) {
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if got := r.Header.Get("Authorization"); got != "Bearer sk-test" {
+			t.Errorf("Authorization=%q, want Bearer sk-test", got)
+		}
+		_, _ = io.WriteString(w, `{"choices":[{"message":{"content":"ok"}}]}`)
+	}))
+	defer srv.Close()
+
+	x := newXinferenceForTest(srv.URL + "/v1")
+	key := "sk-test"
+	_, err := x.ChatWithMessages("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "x"}},
+		&APIConfig{ApiKey: &key}, nil)
+	if err != nil {
+		t.Fatalf("ChatWithMessages: %v", err)
+	}
+}
+
+func TestXinferenceChatExtractsReasoningFields(t *testing.T) {
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+		_, _ = io.WriteString(w, `{"choices":[{"message":{
+			"content":"12",
+			"reasoning_content":"0.15 * 80 = 12"
+		}}]}`)
+	}))
+	defer srv.Close()
+
+	x := newXinferenceForTest(srv.URL)
+	resp, err := x.ChatWithMessages("qwen3",
+		[]Message{{Role: "user", Content: "15% of 80?"}},
+		&APIConfig{}, nil)
+	if err != nil {
+		t.Fatalf("ChatWithMessages: %v", err)
+	}
+	if resp.ReasonContent == nil || *resp.ReasonContent != "0.15 * 80 = 12" {
+		t.Errorf("ReasonContent=%v", resp.ReasonContent)
+	}
+}
+
+func TestXinferenceStreamHappyPath(t *testing.T) {
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/v1/chat/completions" {
+			t.Errorf("path=%s", r.URL.Path)
+		}
+		var seen map[string]interface{}
+		raw, _ := io.ReadAll(r.Body)
+		_ = json.Unmarshal(raw, &seen)
+		if seen["stream"] != true {
+			t.Errorf("stream=%v, want true", seen["stream"])
+		}
+		w.Header().Set("Content-Type", "text/event-stream")
+		_, _ = io.WriteString(w,
+			`data: {"choices":[{"delta":{"reasoning_content":"step. "}}]}`+"\n"+
+				`data: {"choices":[{"delta":{"content":"Hello"}}]}`+"\n"+
+				`data: {"choices":[{"delta":{"content":" world"},"finish_reason":"stop"}]}`+"\n"+
+				`data: [DONE]`+"\n",
+		)
+	}))
+	defer srv.Close()
+
+	x := newXinferenceForTest(srv.URL)
+	var content []string
+	var reasoning []string
+	var sawDone bool
+	err := x.ChatStreamlyWithSender("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "hi"}},
+		&APIConfig{}, nil,
+		func(c *string, r *string) error {
+			if r != nil && *r != "" {
+				reasoning = append(reasoning, *r)
+			}
+			if c != nil && *c == "[DONE]" {
+				sawDone = true
+			}
+			if c != nil && *c != "" && *c != "[DONE]" {
+				content = append(content, *c)
+			}
+			return nil
+		})
+	if err != nil {
+		t.Fatalf("ChatStreamlyWithSender: %v", err)
+	}
+	if strings.Join(reasoning, "") != "step. " {
+		t.Errorf("reasoning=%q", strings.Join(reasoning, ""))
+	}
+	if strings.Join(content, "") != "Hello world" {
+		t.Errorf("content=%q", strings.Join(content, ""))
+	}
+	if !sawDone {
+		t.Error("expected [DONE] callback")
+	}
+}
+
+func TestXinferenceStreamRejectsFalseStreamConfig(t *testing.T) {
+	x := newXinferenceForTest("http://unused")
+	stream := false
+	err := x.ChatStreamlyWithSender("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "x"}},
+		&APIConfig{},
+		&ChatConfig{Stream: &stream},
+		func(*string, *string) error { return nil })
+	if err == nil || !strings.Contains(err.Error(), "stream must be true") {
+		t.Errorf("expected stream-must-be-true error, got %v", err)
+	}
+}
+
+func TestXinferenceStreamCancelsOnIdle(t *testing.T) {
+	withXinferenceIdleTimeout(t, 200*time.Millisecond)
+
+	hold := make(chan struct{})
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "text/event-stream")
+		w.WriteHeader(http.StatusOK)
+		if f, ok := w.(http.Flusher); ok {
+			_, _ = io.WriteString(w, `data: {"choices":[{"delta":{"content":"hi"}}]}`+"\n")
+			f.Flush()
+		}
+		select {
+		case <-hold:
+		case <-r.Context().Done():
+		}
+	}))
+	t.Cleanup(srv.Close)
+	t.Cleanup(func() { close(hold) })
+
+	x := newXinferenceForTest(srv.URL)
+	err := x.ChatStreamlyWithSender("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "x"}},
+		&APIConfig{}, nil,
+		func(*string, *string) error { return nil })
+	if err == nil || !strings.Contains(err.Error(), "stream idle") {
+		t.Errorf("expected stream-idle error, got %v", err)
+	}
+}
+
+func TestXinferenceListModelsAndCheckConnection(t *testing.T) {
+	srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/v1/models" {
+			t.Errorf("path=%s, want /v1/models", r.URL.Path)
+		}
+		if got := r.Header.Get("Authorization"); got != "Bearer sk-test" {
+			t.Errorf("Authorization=%q, want Bearer sk-test", got)
+		}
+		_, _ = io.WriteString(w, `{"object":"list","data":[{"id":"qwen2.5-instruct"},{"id":"custom-chat"}]}`)
+	}))
+	defer srv.Close()
+
+	x := newXinferenceForTest(srv.URL)
+	key := "sk-test"
+	apiConfig := &APIConfig{ApiKey: &key}
+	models, err := x.ListModels(apiConfig)
+	if err != nil {
+		t.Fatalf("ListModels: %v", err)
+	}
+	if strings.Join(models, ",") != "qwen2.5-instruct,custom-chat" {
+		t.Errorf("models=%v", models)
+	}
+	if err := x.CheckConnection(apiConfig); err != nil {
+		t.Fatalf("CheckConnection: %v", err)
+	}
+}
+
+func TestXinferenceMissingBaseURLFailsClearly(t *testing.T) {
+	x := NewXinferenceModel(map[string]string{}, URLSuffix{Chat: "v1/chat/completions"})
+	_, err := x.ChatWithMessages("qwen2.5-instruct",
+		[]Message{{Role: "user", Content: "x"}},
+		&APIConfig{}, nil)
+	if err == nil || !strings.Contains(err.Error(), "missing base URL") {
+		t.Errorf("expected missing-base-URL error, got %v", err)
+	}
+}
+
+func TestXinferenceUnsupportedMethodsReturnNoSuchMethod(t *testing.T) {
+	x := newXinferenceForTest("http://unused")
+	model := "qwen2.5-instruct"
+
+	if _, err := x.Embed(&model, []string{"x"}, &APIConfig{}, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("Embed: expected no such method, got %v", err)
+	}
+	if _, err := x.Rerank(&model, "q", []string{"d"}, &APIConfig{}, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("Rerank: expected no such method, got %v", err)
+	}
+	if _, err := x.Balance(&APIConfig{}); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("Balance: expected no such method, got %v", err)
+	}
+	if _, err := x.TranscribeAudio(&model, nil, &APIConfig{}, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("TranscribeAudio: expected no such method, got %v", err)
+	}
+	if err := x.TranscribeAudioWithSender(&model, nil, &APIConfig{}, nil, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("TranscribeAudioWithSender: expected no such method, got %v", err)
+	}
+	if _, err := x.AudioSpeech(&model, nil, &APIConfig{}, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("AudioSpeech: expected no such method, got %v", err)
+	}
+	if err := x.AudioSpeechWithSender(&model, nil, &APIConfig{}, nil, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("AudioSpeechWithSender: expected no such method, got %v", err)
+	}
+	if _, err := x.OCRFile(&model, nil, nil, &APIConfig{}, nil); err == nil || !strings.Contains(err.Error(), "no such method") {
+		t.Errorf("OCRFile: expected no such method, got %v", err)
+	}
+}