Repository navigation
docs(llm-commits): describe the Codex model the command actually pins - #3867
Conversation
worktrunk-bot
left a comment
There was a problem hiding this comment.
Reviewed the prose change — it matches #3430's own framing of gpt-5.6-luna, the primary/mirror split is right (llm-commits is a non-command doc, so docs/content/llm-commits.md is PRIMARY per the sync taxonomy in docs/CLAUDE.md), and a repo-wide grep confirms no other prose describes the pin. Not approving only because this is a self-authored PR.
One cross-PR note: #3866 relocates this same file (docs/content/llm-commits.md → docs/src/content/docs/llm-commits.md, plus a new docs/public/ copy) without touching this sentence, so whichever lands second needs a rebase — and if #3866 goes first, this edit has to be reapplied at the new primary path rather than replayed as-is.
2694224 to
45536ce
Compare
|
The cross-PR note in my review landed: #3866 merged at 07:01Z, ~8 minutes after that review, so this branch was editing a path that no longer exists. Rebased onto Rebase detailsOld head
CI on |
The Codex setup section in the LLM-commits docs describes the pinned model as "the fast mini model", but the command it sits under pins
gpt-5.6-luna— which is not a-minimodel. The prose was written in #837 when the pin was a mini variant and was left behind when #3430 bumpedgpt-5.4-mini→gpt-5.6-luna. This rewords the sentence to describe what the command actually pins, following #3430's own framing ofgpt-5.6-lunaas "the fast/low-cost variant of the current recommended (5.6) family".docs/src/content/docs/llm-commits.mdis the primary (non-command doc, per the sync taxonomy indocs/CLAUDE.md); the two mirrors underskills/andplugins/were regenerated bycargo test --test integration test_docs_are_in_sync, which now passes. The branch was rebased ontomainafter #3866 relocated the docs tree, so the edit lands at the post-#3866 path rather than the olddocs/content/one.No regression test: the change is a single prose sentence with no behavior behind it. The sync test already pins the mirrors to the primary, which is the only mechanical invariant here.
Nightly sweep context
Found during the rolling survey (bucket 23/28), reviewing
skills/worktrunk/reference/llm-commits.md.Provenance:
A repo-wide
grep -i miniturned up no other stale reference to the old pin — the model string itself is consistent acrosssrc/cli/mod.rs,dev/config.example.toml,Taskfile.yaml, andsrc/output/commit_generation.rs; only this one sentence describes it.