Skip to content

fix: preserve CSV whitespace and CR records - #21

Merged
rogerchappel merged 2 commits into
mainfrom
agent/oss-f2df25afe0da-csv-roundtrip
Aug 30, 2026
Merged

fix: preserve CSV whitespace and CR records#21
rogerchappel merged 2 commits into
mainfrom
agent/oss-f2df25afe0da-csv-roundtrip

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • preserve leading and trailing whitespace in unquoted CSV fields and through toCsv round trips
  • accept CR-only record separators while retaining CR, LF, and CRLF inside quoted multiline fields
  • quote carriage-return values during serialization and retain stable unterminated-record diagnostics

Commits

  • bcd9057 test: cover CSV record and whitespace compatibility
  • 4f67f9f fix: preserve CSV whitespace and CR records

Verification

  • npm ci
  • npm run release:check (47/47 tests, syntax, CLI smoke, documentation install smoke, package dry-run)
  • git diff --check origin/main...HEAD

All commits are authored and committed by Roger Chappel miscanalysis@gmail.com.

@rogerchappel

Copy link
Copy Markdown
Owner Author

Automated merge note

Triage class: auto-merge

Summary: Preserves unquoted CSV field edge whitespace, accepts CR-only records, retains embedded CR/LF/CRLF in quoted fields, and quotes CR values; includes focused regression coverage (2 files, +38/-6).

Checks run: GitHub CI Repository hygiene completed SUCCESS; worker verification npm ci, npm run release:check (47/47 tests, syntax, CLI smoke, documentation install smoke, package dry-run), and git diff --check origin/main...HEAD passed. Full diff and commit identities reviewed.

Rebased/CI-repaired: No.

Verified head SHA: 4f67f9f91943fb88ab7f374e5d41801341cd5b26.

@rogerchappel
rogerchappel merged commit b7e0700 into main Aug 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant