Skip to content

Commit a36600b

Browse files
committed
ci: promote deepnsm clippy advisory -> gating (TD-DEEPNSM-CLIPPY-195 cleared)
With deepnsm now clippy-clean (commit bf95caa), move its CI clippy step from Tier-B advisory (continue-on-error) to Tier-A mandatory gating — the same posture as the zero-dep contract crate. deepnsm changes can no longer regress clippy unseen. Records the resolution of TD-DEEPNSM-CLIPPY-195 in the tech-debt ledger. https://claude.ai/code/session_014A4JuRCqKP2yNENrQ9Ha7H
1 parent bf95caa commit a36600b

2 files changed

Lines changed: 16 additions & 8 deletions

File tree

.claude/board/TECH_DEBT.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2638,3 +2638,14 @@ W6 entropy-ledger reframe of `DEEPNSM-NSM-1`.
26382638
## TD-DEEPNSM-CLIPPY-195 — 12 pre-existing default-clippy lints in deepnsm (clippy 1.95 bump)
26392639

26402640
`cargo clippy --manifest-path crates/deepnsm/Cargo.toml --all-targets -- -D warnings` reports 12 errors across 7 files (codebook 2, encoder 4, similarity 2, disambiguator_glue/nsm_primes/parser/quantum_mode 1 each) — newer lints (`manual_repeat_n`, `uninlined_format_args`, …) that were clean when written and fire only under clippy 1.95.0. Pre-existing (not from the E-ENGLISH-BIFURCATES slice; `arcs.rs` is clean at pedantic+nursery). Tests unaffected (94+4+8+1 green). Fix = a separate mechanical sweep across the 7 files; deliberately NOT bundled into the feature slice (7-file scope creep). Surfaced 2026-05-31.
2641+
2642+
**Resolved 2026-06-09** (PR #479, branch `claude/stoic-turing-M0Eiq`, commit `bf95caa`):
2643+
hand-reviewed clippy sweep landed. `cargo clippy --manifest-path
2644+
crates/deepnsm/Cargo.toml --all-targets -- -D warnings` is now clean (exit 0).
2645+
Cleared the original 7-file set plus the lints in PR #479's new reader modules
2646+
(window / reader_state / crystal_neighborhood / sentence_transformer64 /
2647+
signed_crystal / codebook) surfaced by `--all-targets` — 22 lints across 13
2648+
files; 217 tests green. Fixes are hand-applied (NOT `clippy --fix`, which mangled
2649+
`reader_state.rs` into stranded-comment match guards). The CI clippy step for
2650+
deepnsm was promoted Tier-B advisory → Tier-A gating in
2651+
`.github/workflows/style.yml`.

.github/workflows/style.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -71,14 +71,11 @@ jobs:
7171
- name: Clippy lance-graph (advisory)
7272
continue-on-error: true
7373
run: cargo clippy --manifest-path crates/lance-graph/Cargo.toml --lib --tests -- -D warnings
74-
# Tier B (advisory until incrementally cleaned, non-gating):
75-
# deepnsm is the standalone 0-dep codec crate, workspace-excluded, so it
76-
# was never clippy-checked. It carries ~12 pre-existing default-clippy
77-
# lints (TD-DEEPNSM-CLIPPY-195, clippy 1.95 bump) — mechanical idiom
78-
# lints to be paid down by a focused sweep, same policy as lance-graph
79-
# core. Advisory now; promote to gating once TD-DEEPNSM-CLIPPY-195 clears.
80-
- name: Clippy deepnsm (advisory)
81-
continue-on-error: true
74+
# Tier A (mandatory, gating): deepnsm is now clippy-clean — TD-DEEPNSM-CLIPPY-195
75+
# resolved 2026-06-09 by a hand-reviewed sweep. It's a standalone 0-dep codec
76+
# crate, workspace-excluded, so the lance-graph clippy steps don't cover it;
77+
# gate it explicitly (same posture as the contract crate) so it can't regress.
78+
- name: Clippy deepnsm (mandatory)
8279
run: cargo clippy --manifest-path crates/deepnsm/Cargo.toml --all-targets -- -D warnings
8380

8481
format:

0 commit comments

Comments
 (0)