From 0760cf96d4d5f880e2ee66349ad1ddfb5d77337a Mon Sep 17 00:00:00 2001 From: Steve Dudenhoeffer Date: Wed, 12 Aug 2026 16:53:05 -0400 Subject: [PATCH] =?UTF-8?q?docs:=20gadfly=20round=204=20=E2=80=94=20two=20?= =?UTF-8?q?doc-comment=20fixes?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit DSN.Scheme's list named kimi but not qwen — the same sibling drift this PR keeps finding, in a doc comment this time (I updated the README's scheme list and not this one). Added qwen, and llama-swap/llama-swaps while there, since they were missing too. envKeyForProvider's example ran backwards: "LLM_M1 → m1" describes registry naming, not what the function does. Now reads name → variable in one direction. Not taking the third: the const block's alignment is gofmt's own output (gofmt -l is empty), and the uneven padding is forced by the doc comments that split the block into alignment groups. glm-5.2 reached that same conclusion in round 2 before flagging it here. Co-Authored-By: Claude Opus 5 (1M context) --- env.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/env.go b/env.go index 3397a9b..20c274b 100644 --- a/env.go +++ b/env.go @@ -26,8 +26,9 @@ var ErrUnknownProvider = errors.New("unknown provider") // authenticated with the bearer token "test-token". type DSN struct { // Scheme selects the provider implementation: "foreman", "ollama", - // "ollama-cloud", "openai", "kimi", "anthropic", "google"/"gemini", or - // any custom scheme registered with RegisterScheme. + // "ollama-cloud", "openai", "kimi", "qwen", "anthropic", + // "google"/"gemini", "llama-swap"/"llama-swaps", or any custom scheme + // registered with RegisterScheme. Scheme string // Token is the provider secret (bearer token or API key); empty = none. Token string @@ -41,7 +42,7 @@ type DSN struct { func (d DSN) BaseURL() string { return "https://" + d.Host } // envKeyForProvider returns the LLM_* variable that defines the provider named -// name (LLM_M1 → "m1", so "my-prov" → LLM_MY_PROV). +// name: "m1" → LLM_M1, "my-prov" → LLM_MY_PROV. // // This is the single definition on purpose. Two call sites need byte-identical // output and would drift apart in silence: lazy resolution reads this variable