Skip to content

fix(eval): accept both quoted and unquoted CSV in json_to_csv_export#299

Merged
chaliy merged 1 commit intomainfrom
claude/issue-283-eval-csv-76a2F
Feb 26, 2026
Merged

fix(eval): accept both quoted and unquoted CSV in json_to_csv_export#299
chaliy merged 1 commit intomainfrom
claude/issue-283-eval-csv-76a2F

Conversation

@chaliy
Copy link
Contributor

@chaliy chaliy commented Feb 26, 2026

Summary

  • jq @csv produces RFC 4180 quoted strings ("Alice Chen","Engineering",120000)
  • Changed file_contains checks to verify individual field values instead of full row match
  • This accepts both quoted and unquoted CSV output

Test plan

  • Expectations now pass whether model uses @csv (quoted) or manual string concat (unquoted)

Closes #283

@chaliy chaliy merged commit f86345d into main Feb 26, 2026
10 checks 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.

fix(eval): json_to_csv_export expects unquoted CSV but @csv correctly quotes strings

2 participants