docs: sweep em/en dashes in README and LLM_PROVIDERS, add reranker-eval note - #172
docs: sweep em/en dashes in README and LLM_PROVIDERS, add reranker-eval note#172Coding-Dev-Tools wants to merge 4 commits into
Conversation
…al note - README.md: replace two en-dashes in the env-var table with `--` so the public-doc em-dash gate passes; add a brief evaluation note for the cross-encoder reranker knob (now reachable via MemoryEngine.create(rerank_model=...)) and document the measured ~15ms per-query cost on the bundled retrieval gates - docs/LLM_PROVIDERS.md: replace one em-dash in the Windows note with `--` These are the only doc changes shipped in this round; the rest of the work is in PR1 (core + MCP + integration).
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f25fc87d47
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Review pass 2Re-read both diffs in the worktree. PR2 is small, targeted, and ships cleanly: the em-dash sweep in 1. The reranker-eval note lands in a quiet sectionThe reranker-eval paragraph was added to the README env-var table area. That's correct content, but it sits in a section titled by the table above it. Consider giving it a subheading ( 2. The CHANGELOG cross-link is missingPR1 adds the cross-encoder reranker to the [Unreleased] Added section. The README now describes the same feature. There's no cross-link, so a reader hunting the rationale or evaluation numbers has to read both files. The README paragraph could close with a one-liner like "(see What's good
Approve
A note on what the changes do for Command Code usageNeither file in PR2 directly changes Command Code behavior, but the Windows |
…inter
The previous paragraph quoted model-specific MRR@5 / hit@5 / ~15 ms numbers
for the optional cross-encoder reranker without identifying the model,
revision, hardware, exact command, or evidence artifact. Per AGENTS.md
L177-L178 ("Prove 'better' with a number. No retrieval/quality claim ships
without an eval."), such numbers must not ship until the evidence is
registered.
The README now contains a short note that reranker benchmarks are
intentionally unpublished and points operators at the local reproduction
commands (`python -m eval.ablation`, `python -m eval.reinforcement`).
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 32195d1ff2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
PR #172 introduced a single em-dash on README.md:751 inside the new reranker-eval blockquote. The repo's no-em-dashes-in-public-docs policy (tests/test_benchmark_evidence.py::test_public_facing_docs_do_not_use_em_dashes) blocks on every Python matrix plus the core floor and coverage gate jobs. Replace U+2014 with --, mirroring the convention used elsewhere in the same blockquote.
1 participant
docs: sweep em/en dashes in README and LLM_PROVIDERS, add reranker-eval note
--so the public-docem-dash gate passes; add a brief evaluation note for the cross-encoder reranker
knob (now reachable via MemoryEngine.create(rerank_model=...)) and document the
measured ~15ms per-query cost on the bundled retrieval gates
--These are the only doc changes shipped in this round; the rest of the work is in
PR1 (core + MCP + integration).