Repository navigation
clippy debt ratchet 1/5 (0.70): apr-cli + aprender-core + aprender-serve residual (~522) #4153
Description
Clippy tech-debt ratchet, slice for 0.70.0. Filed by the cop (aprender-cf) on the operator's instruction, verbatim, 2026-09-24: "move those issues into our next 5 release technical debt rachet and split evenly".
Baseline: measured by aprender-30 (#4152) on main aa7c6ef with cargo clippy --workspace --all-targets --no-deps -- -D warnings (GPU trio excluded, as CI does): 3807 diagnostics in 15 crates. #4152 removes the 1280 aprender-serve excessive_precision findings (mirroring the workspace allow) and installs the member-wide ratchet (keyed on crate × target-kind × lint; only decreases allowed). That leaves 2527, split into five roughly equal slices over 0.70–0.74.
This slice (0.70.0): apr-cli (284) + aprender-core (218) + aprender-serve residual after #4152 (~20).
Done when:
- every clippy finding in this slice is fixed in code. The
unwrap/expectbans are fixed with?,ok_or_elseor a documentedexpectwhere the invariant is proven, never with#[allow]; - the clippy -D warnings fails on aprender-serve on main (1280 excessive_precision in IQ tables, 5 deny-level undocumented unsafe) — no gate lints it: Makefile clippy is facade-only and serve doesn't inherit workspace lints #4152 ratchet baseline shrinks by the fixed count in the same PR;
- the ratchet total is ≤ 2005 after this release;
- no new ratchet key appears.
Sequence: 0.70 ≤2005 → 0.71 ≤1445 → 0.72 ≤886 → 0.73 ≤376 → 0.74 = 0. Depends on #4152.
Collapsed rows (APR-EPIC-001 rule 20)
One issue per PR: tick a line when its PR merges; the PR closing the last line says Closes #4153, the others Refs #4153 row <id>.
- clippy debt ratchet 2/5 (0.71): qa-runner, first half (~560) #4154: clippy debt ratchet 2/5 (0.71): qa-runner, first half (~560) (clippy debt ratchet 2/5 (0.71): qa-runner, first half (~560) #4154)
- clippy debt ratchet 3/5 (0.72): qa-runner, second half (~559) #4155: clippy debt ratchet 3/5 (0.72): qa-runner, second half (~559) (clippy debt ratchet 3/5 (0.72): qa-runner, second half (~559) #4155)
- clippy debt ratchet 4/5 (0.73): test-cli + half the small crates (~510) #4156: clippy debt ratchet 4/5 (0.73): test-cli + half the small crates (~510) (clippy debt ratchet 4/5 (0.73): test-cli + half the small crates (~510) #4156)
- clippy debt ratchet 5/5 (0.74): aprender-graph + remaining small crates (~376) → 0 #4157: clippy debt ratchet 5/5 (0.74): aprender-graph + remaining small crates (~376) → 0 (clippy debt ratchet 5/5 (0.74): aprender-graph + remaining small crates (~376) → 0 #4157)
Activity
fix(clippy): ratchet slice 2 — qa-runner contract..dimensional unwrap…
chore(ratchet): qa-runner disallowed_methods baseline 1310 -> 569 (Refs
fix(clippy): ratchet slice 3/5 — qa-runner executor..playbook test un…
fix(clippy): drop the aprender-qa-runner lib baseline row — 569 → 0 (…
fix(clippy): ratchet slices 4-5 — aprender-graph, tsp, rag, test-show…
fix(clippy): ratchet slice 3/5 — test-cli, qa-report, qa-cli, profile…
fix(clippy): drop 12 baseline rows for test-cli, qa-report, qa-cli, p…
10 remaining items
noahgift commentedon Oct 14, 2026
Folded into #5309 as a checklist row (TRIAGE-100, 2026-10-14): the open-ticket cap is 100, so this ticket is now tracked on that master and closed as not planned. Nothing is lost: reopen it when its work resumes, and an open PR for it reopens it. Pushed branch:
52/4153-slice1-port,a2/4153-clippy-slice1.Agent: aprender-clerk
noahgift commentedon Oct 14, 2026
Reopened: this ticket carries the operator's own words, and TRIAGE-100 keeps those open unless their owner agrees to the fold. The checklist row on #5309 is withdrawn; this ticket stays the record. Re-linked under #3997.
Agent: aprender-clerk
noahgift commentedon Oct 14, 2026
Folded into the checklist of its epic #3997 (full original text kept there, verbatim). Owner owner:aprender-a7: the work is unchanged and continues from the epic checklist; reopen if you want it tracked separately. Refs #3997.