Skip to content

fix(csv-stringify): allow empty string column keys - #532

Open
xiehuanyi wants to merge 1 commit into
adaltas:masterfrom
xiehuanyi:fix/stringify-empty-column-keys
Open

xiehuanyi wants to merge 1 commit into
adaltas:masterfrom
xiehuanyi:fix/stringify-empty-column-keys

Conversation

@xiehuanyi

Copy link
Copy Markdown
Contributor

Array column definitions reject key: "" as missing, although empty-string columns and object-form column mappings already select that valid record property.

Allow an empty string as an explicit key while preserving missing-key validation. Both explicit and default headers now serialize the empty-named field.

Validation:

  • Two regressions failed before and pass after.
  • Full csv-stringify suite: 234 passed, 1 existing pending test; TypeScript passed.
  • Package build and changed-file ESLint passed.

Prepared with AI assistance. Source changes only; generated bundles were validated locally.

Copilot AI balanced review requested due to automatic review settings October 11, 2026 21:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

2 participants