fix: use conservative Ollama settings for reliability #83

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

Merged

nicholaspsmith merged 1 commit into main from fix/ollama-conservative-batching

Jan 27, 2026

Merged

fix: use conservative Ollama settings for reliability #83

fix: use conservative Ollama settings for reliability

Annotations

4 errors and 10 warnings

Test

Process completed with exit code 1.

Test: src/embeddings/retry.ts#L34

TypeError: Cannot read properties of undefined (reading 'headers') ❯ checkResponseSize src/embeddings/retry.ts:34:34 ❯ fetchWithRetry src/embeddings/retry.ts:85:7 ❯ src/embeddings/ollama.ts:122:28 ❯ OllamaBackend.embedBatch src/embeddings/ollama.ts:144:28 ❯ src/__tests__/embeddings/ollama.test.ts:262:22

Test: src/__tests__/embeddings/ollama.test.ts#L208

AssertionError: expected "vi.fn()" to be called with arguments: [ …(2) ] Received: 1st vi.fn() call: [ "http://localhost:11434/api/embed", - ObjectContaining { - "body": "{\"model\":\"qwen3-embedding:0.6b\",\"input\":[\"text1\",\"text2\",\"text3\"]}", + { + "body": "{\"model\":\"qwen3-embedding:0.6b\",\"input\":[\"text1\",\"text2\",\"text3\"],\"keep_alive\":\"10m\"}", + "headers": { + "Content-Type": "application/json", + }, + "method": "POST", + "signal": AbortSignal { + Symbol(kEvents): Map {}, + Symbol(events.maxEventTargetListeners): 10, + Symbol(events.maxEventTargetListenersWarned): false, + Symbol(kHandlers): Map {}, + Symbol(kAborted): false, + Symbol(kReason): undefined, + Symbol(kComposite): false, + }, }, ] Number of calls: 1 ❯ src/__tests__/embeddings/ollama.test.ts:208:25

Test: src/__tests__/embeddings/ollama.test.ts#L141

AssertionError: expected "vi.fn()" to be called with arguments: [ …(2) ] Received: 1st vi.fn() call: [ "http://localhost:11434/api/embed", - ObjectContaining { - "body": "{\"model\":\"qwen3-embedding:0.6b\",\"input\":[\"test text\"]}", + { + "body": "{\"model\":\"qwen3-embedding:0.6b\",\"input\":[\"test text\"],\"keep_alive\":\"10m\"}", "headers": { "Content-Type": "application/json", }, "method": "POST", + "signal": AbortSignal { + Symbol(kEvents): Map {}, + Symbol(events.maxEventTargetListeners): 10, + Symbol(events.maxEventTargetListenersWarned): false, + Symbol(kHandlers): Map {}, + Symbol(kAborted): false, + Symbol(kReason): undefined, + Symbol(kComposite): false, + }, }, ] Number of calls: 1 ❯ src/__tests__/embeddings/ollama.test.ts:141:25

Lint: src/__tests__/search/indexer.test.ts#L339