tamada / oinkie / 28728293946

73%

main: 74%

Pull #15

github

fix: harden compare module against panics, NaN results, and silent I/O failures

- PairingStrategy::compare_count(All) underflowed (debug panic) when the
  target list was empty; use saturating_sub like the other strategies
- Comparison::similarity returned NaN when the aggregated similarity
  list was empty (reachable via an empty TopN selection); return 0.0
- Aggregator::from_str now rejects 'topn:0', which produced an empty
  selection and hence NaN similarities downstream
- Comparison::store discarded every write error with 'let _ =', so a
  full disk produced truncated result CSVs reported as success;
  propagate the errors and flush the buffer explicitly
- Fix the misleading comment in build_matrix claiming the matrix holds
  costs; it holds similarities, the cost conversion happens in
  hungarian_algorithm

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

87 of 98 new or added lines in 4 files covered. (88.78%)

1568 of 2143 relevant lines covered (73.17%)

12.34 hits per line