steve/go-llm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add llm.Foreman() constructor for foreman daemon integration #4

Closed
steve wants to merge 2 commits from feat/foreman-constructor into main
pull from: feat/foreman-constructor
merge into: steve:main
Conversation 0 Commits 2 Files Changed 2
+10
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 0c4e0081d8 - Show all commits
v2/CLAUDE.md
+10
View File
@@ -32,3 +32,13 @@
6. Streaming via pull-based `StreamReader.Next()`
7. Middleware for logging, retry, timeout, usage tracking
8. Ollama uses the native `/api/chat` API rather than the OpenAI-compat `/v1` endpoint. Native API supports `think: false` for thinking-capable models, has more reliable tool calling, and is approximately 15-20% lower latency. Both local and cloud share the same provider; only the apiKey/baseURL differ. `llm.Ollama()` targets `http://localhost:11434` with no Authorization header; `llm.OllamaCloud(key)` targets `https://ollama.com` with `Authorization: Bearer <key>`.
### DD#9 — foreman integration via thin Foreman() constructor (2026-05-23)
**Context:** foreman is a private queued Ollama endpoint. It speaks native Ollama
on the wire, so no new provider is needed.
**Decision:** `llm.Foreman(baseURL, apiKey)` delegates to `ollamaProvider.New()`.
baseURL is positional and required (no sensible default for a private daemon).
apiKey can be empty for network-trusted deployments.
**Consequence:** Zero new code paths. Streaming, tool use, and think work through
the existing native ollama provider. A dedicated foreman provider (Level 2) is
deferred unless async /jobs or webhook integration is needed at the llm layer.