Skip to content

Repair sweep-test-coverage state CSV#2679

Merged
brendancol merged 1 commit into
mainfrom
fix/repair-sweep-test-coverage-csv
May 29, 2026
Merged

Repair sweep-test-coverage state CSV#2679
brendancol merged 1 commit into
mainfrom
fix/repair-sweep-test-coverage-csv

Conversation

@brendancol

Copy link
Copy Markdown
Contributor

What

Repairs .claude/sweep-test-coverage-state.csv. Parallel-agent sweep appends had re-corrupted it under the merge=union gitattribute.

Why

The file keeps one record per sweep module. When a note contains an embedded newline, merge=union treats each physical line as its own record and unions them. That produced duplicate module rows, duplicate header lines, and phantom rows split out of the multi-line notes. It had grown to 31 physical lines with mixed CRLF/LF endings.

How

  • Kept the latest record per module (last-write-wins by last_inspected).
  • Collapsed each record back to a single physical line so future union appends stay record-aligned.
  • Normalized line endings to CRLF.

Result is 7 lines (header + 6 modules), each carrying its newest pass data: geotiff 19, polygonize 3, rasterize 4, resample 2, zonal 2, and the 2026-05-29 reproject entry (#2618).

Dedupe duplicate module rows (latest-by-last_inspected wins) and collapse
each record to a single physical line. The merge=union strategy in
.gitattributes splits any note containing an embedded newline
record-by-record, so parallel-agent appends had re-corrupted the file into
31 physical lines with phantom rows and duplicate header lines.

One record per line keeps union merges safe. Per-module rows reflect the
latest pass: geotiff 19, polygonize 3, rasterize 4, resample 2, zonal 2,
reproject (2026-05-29 entry, #2618).
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label May 29, 2026
@brendancol brendancol merged commit fa8a9da into main May 29, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant