Skip to content

feat: add history columns to CSV - #405

Open
tbuckley wants to merge 1 commit into
mainfrom
add-csv-history
Open

feat: add history columns to CSV#405
tbuckley wants to merge 1 commit into
mainfrom
add-csv-history

Conversation

@tbuckley

Copy link
Copy Markdown
Owner

No description provided.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying evals-pwa with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c740ce
Status: ✅  Deploy successful!
Preview URL: https://c47045ae.evals-pwa.pages.dev
Branch Preview URL: https://add-csv-history.evals-pwa.pages.dev

View logs

@cursor cursor Bot 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.

Bug: CSV Column Misalignment Due to Copy-Paste Error

A copy-paste error causes an empty cell for the history column to be pushed to providerRow instead of promptRow in the prompt row section. This results in CSV column misalignment, as the prompt row lacks the history column and the provider row gains an extra cell.

src/lib/utils/csvExport.ts#L77-L78

if (get(run.results[0][envIndex]).history) {
providerRow.push(''); // Empty cell for history column

Fix in CursorFix in Web


BugBot free trial expires on July 22, 2025
Learn more in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

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