Fix WorkBench export file extension for selected delimiter

[acwhite211]

Fixes #8047

Fixes WorkBench data set exports so the downloaded filename extension matches the selected Export file delimiter preference.

Previously, exports used the configured delimiter for file content, but the filename defaulted to .tsv unless the caller had already supplied .csv. This caused comma delimited exports to download with the wrong extension.

Checklist

• Self-review the PR after opening it to make sure the changes look good and
  self-explanatory (or properly documented)
• Add relevant issue to release milestone
• Add pr to documentation list
• Add automated tests

Testing instructions

• Open User Preferences > WorkBench.
• Set Export file delimiter to Comma.
• Open a WorkBench data set and click Export.
• Confirm the downloaded file ends in .csv.
• Confirm the exported content is comma-delimited.
• Change Export file delimiter to Tab.
• Export the same data set again.
• Confirm the downloaded file ends in .tsv.
• Optionally repeat with pipe, semicolon, and space delimiters to confirm the extension matches the selected delimiter behavior.

Summary by CodeRabbit

• Bug Fixes

  • Fixed data export file naming. Downloaded files now receive the correct extension (.csv, .tsv, .psv, or .txt) based on their actual delimiter format, improving file compatibility and clarity.

• Tests

  • Added comprehensive test coverage for delimited file naming scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e43dcea8-796b-4852-b5a7-949738b8d3d3

📥 Commits

Reviewing files that changed from the base of the PR and between b4b1602 and 3893bf6.

📒 Files selected for processing (2)

• specifyweb/frontend/js_src/lib/components/WorkBench/__tests__/helpers.test.ts
• specifyweb/frontend/js_src/lib/components/WorkBench/helpers.ts

---

📝 Walkthrough

Walkthrough

The changes introduce a new getDelimitedFileName helper function that dynamically maps export delimiters to appropriate file extensions (.csv for comma, .tsv for tab, .psv for pipe, .txt as fallback), replacing hardcoded extension logic that previously ignored delimiter preferences.

Changes

Cohort / File(s)	Summary
Delimiter Extension Mapping specifyweb/frontend/js_src/lib/components/WorkBench/helpers.ts	Added exported getDelimitedFileName function that strips known delimited file extensions from filenames and appends the correct extension based on the provided delimiter value. Updated downloadDataSet to use this new helper instead of forcing .tsv extensions.
Test Coverage specifyweb/frontend/js_src/lib/components/WorkBench/__tests__/helpers.test.ts	Introduced comprehensive test suite using data-driven theories to verify delimiter-to-extension mappings (.csv/.tsv/.psv/.txt), known extension replacement, and preservation of non-delimited periods in filenames.

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: fixing WorkBench export filenames to match the selected delimiter preference.
Linked Issues check	✅ Passed	The PR implementation fully addresses issue #8047 by adding getDelimitedFileName function that maps delimiters to correct extensions (.csv, .tsv, .psv, .txt) and includes comprehensive test coverage.
Out of Scope Changes check	✅ Passed	All changes are directly related to fixing the delimiter-to-extension mapping issue; no unrelated modifications detected.
Automatic Tests	✅ Passed	PR includes comprehensive automatic tests using data-driven approach covering new getDelimitedFileName helper functionality with 7 well-structured test cases validating delimiter-to-extension mapping and filename handling.
Testing Instructions	✅ Passed	Testing instructions clearly cover affected components with manual and automated test cases validating the new getDelimitedFileName function and downloadDataSet integration across all 5 delimiters.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch issue-8047

---

_{Review rate limit: 9/10 reviews remaining, refill in 6 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[emenslin]

• Confirm the downloaded file ends in .csv.
• Confirm the exported content is comma-delimited.
• Confirm the downloaded file ends in .tsv.

Looks good, files seem to be exported correctly.

[Iwantexpresso]

• Confirm the downloaded file ends in .csv.
• Confirm the exported content is comma-delimited.
• Confirm the downloaded file ends in .tsv.

-functionality looks great!

[bhumikaguptaa]

• Confirm the downloaded file ends in .csv.
• Confirm the exported content is comma-delimited.
• Confirm the downloaded file ends in .tsv.

Ran into no errors!