tools (volt): ruff -F,E9 3 -> 0 -- an unused import, a placeholder-free f-string and a dead variable out of the instruments; 30 captured outputs unchanged (429 cells still green)

ruff check --select F,E9 tools/ -> 3 errors at 960b852:
  checkin_check.py:421 F541  f-string with no placeholder
  checkin_check.py:556 F841  jd assigned, never read (dead residue of an unwritten
                             malformed-JSON refusal cell)
  queue_append_audit.py:73 F401  re imported, never used (no 're.' in the file)
Fix: 3 lines removed, 1 line changed, nothing added (numstat 1/2 + 0/1).

Behaviour claim measured, not asserted: all 14 tools' --selftest stdout+stderr
captured before and after (28 files) plus checkin_check.py --dir checkins and
queue_append_audit.py --guard --to HEAD (2 files) -- 30 artifacts, 0 differ after
normalising only checkin_check's mkdtemp suffix. 14 tools / 429 cells / 429 pass
on both sides; verify.py 3/3 PASS rc 0.

Not enforced by the gate: verify.py has no static-check leg, so this property is
observed here only. Adding ruff to the gate would make it a hard dependency
(host without ruff -> UNGRADABLE -> rc 2), so no leg and no new tool.
This commit is contained in:
timmy committed 2026-10-19 07:34:05 +00:00
1 parent 960b852331
commit 9f7bc3b49e
2 files changed
+1 -3

No files matched your search

+1 -2
View File
@@ -418,7 +418,7 @@ def print_report(rep, show_gaps: bool, fail: bool):
adv = rep["advisory"] adv = rep["advisory"]
print(f"\nadvisory (never rc 1): {adv['missing_headings']} of {rep['checked']} " print(f"\nadvisory (never rc 1): {adv['missing_headings']} of {rep['checked']} "
f"files lack a Did/Blocked/Next/Ask heading") f"files lack a Did/Blocked/Next/Ask heading")
print(f" -- A1: the four-heading rule postdates the archive; grade it where it applies") print(" -- A1: the four-heading rule postdates the archive; grade it where it applies")
if show_gaps: if show_gaps:
print(f"\n{'handle':<12} {'files':>5} {'days':>5} {'first':>10} {'last':>10} " print(f"\n{'handle':<12} {'files':>5} {'days':>5} {'first':>10} {'last':>10} "
f"{'span':>4} {'miss':>4} {'gap':>4} {'idle':>4}") f"{'span':>4} {'miss':>4} {'gap':>4} {'idle':>4}")
@@ -553,7 +553,6 @@ def selftest():
cell("c20 --fail exits 1 when a fault exists", rc_fail == 1, str(rc_fail)) cell("c20 --fail exits 1 when a fault exists", rc_fail == 1, str(rc_fail))
# refusals: rc 2, never a traceback, never 1 # refusals: rc 2, never a traceback, never 1
jd = os.path.join(td, "j.json")
miss = os.path.join(td, "no-such-dir") miss = os.path.join(td, "no-such-dir")
rc_missing = main(["--dir", miss, "--roster", "volt"]) rc_missing = main(["--dir", miss, "--roster", "volt"])
cell("c21 a missing checkins dir refuses at rc 2", rc_missing == 2, str(rc_missing)) cell("c21 a missing checkins dir refuses at rc 2", rc_missing == 2, str(rc_missing))
-1
View File
@@ -70,7 +70,6 @@ import argparse
import io import io
import json import json
import os import os
import re
import subprocess import subprocess
import sys import sys
import tempfile import tempfile