test(push): coverage wave 2 (batch 1-2: category helpers + CSV-validator errors)

[LukasWodka]

Wave-2 coverage for internal/push (87.7% → 89.0%): pure category/family helpers (were 0%) + the CSV-validator file-error arms. First installment of the push grind — the package's remaining gap is spread across ~50 functions (preflight validators, pod/stage lifecycle, orphan/list), so push→95% is a multi-PR effort. make ci green.

🤖 Generated with Claude Code

---

Note

Low Risk
No production code changes—only new unit tests for existing helpers and validators.

Overview
Adds test-only coverage in internal/push as the first wave toward higher package coverage (~87.7% → ~89%).

Category helpers (category_pure_coverage_test.go): exercises FamilyNoun / FamilyFromNoun round-trips and unknown-input fallbacks to tabular, non-empty FamilyNouns(), SupportedCategoriesList() including known task ids, and TextSidecarDir() for text task categories.

Preflight CSV validators (preflight_csverrors_coverage_test.go): asserts missing-file paths error for HasBOM, ReadCSVHeader, CheckTabularBOM, CheckHasDataRows, and CheckCSVEncoding; empty files fail header read; sub-3-byte files return no BOM without error.

^{Reviewed by Cursor Bugbot for commit 3dd13b7. Bugbot is set up for automated code reviews on this repo. Configure here.}

[LukasWodka]

bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 3dd13b7. Configure here.}

[LukasWodka]

👋 Heads-up — Code review queue is at 34 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

• averaging-service#181 — feat(weights): normalize-on-read — mixed cycles average instead of rejecting (SEC-03 §8 step 2) · author: @shujaatTracebloc · no reviewer assigned
• averaging-service#182 — feat(weights): averaging writes SafeTensors (TF + PyTorch) — SEC-03 §8 step 4 · author: @shujaatTracebloc · no reviewer assigned
• backend#1079 — feat(global_meta): edge dataset_meta exposure + attributes contract at ingest (#924 G4a) · author: @divyasinghds · no reviewer assigned
• backend#1086 — docs(rfc): SafeTensors weight-format migration — SEC-03 Phase 1 (RFC 0004) · author: @shujaatTracebloc · no reviewer assigned
• backend#1093 — chore(deps): bump django from 5.2.14 to 5.2.15 · author: @dependabot · no reviewer assigned
• backend#1095 — feat(experiment): configurable preprocessing knobs incl. tabular scaler — RFC 0003 L1 + L1b (#1094) · author: @LukasWodka · no reviewer assigned
• backend#1100 — fix(boot): pin SDK install to tracebloc==0.11.2, drop 404 dev line (#1098) · author: @LukasWodka · reviewer: @saqlainsyed007
• cli#266 — main - > enhance CLI features and tests · author: @saadqbal · no reviewer assigned
• cli#267 — fix(cli): home screen labels the env by remembered client name (bugbot main - > enhance CLI features and tests #266) · author: @saadqbal · no reviewer assigned
• cli#268 — test: coverage wave 2 — utility packages toward 95% · author: @LukasWodka · no reviewer assigned

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

[saadqbal]

LGTM 👍 Verified FamilyNoun/FamilyFromNoun defaults + round-trip, TextSidecarDir reading PrimarySubdir from the contract (MLM→sequences), and HasBOM's short-file n<3→(false,nil) arm vs the missing-file open error. Good coverage of the CSV-validator error paths. Clean.