Skip to content

🔧 fix: Apply Convo Export filename sanitization at export, not input #8779

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

azamara
Copy link
Contributor

@azamara azamara commented Jul 31, 2025

Summary

This PR fixes the Korean character composition issue in the export filename input. Previously, filenamify() was called on every keystroke, which prevented

The fix moves filename sanitization from input time to export time, allowing proper character composition while still ensuring safe filenames.

Change Type

  • Bug fix (non-breaking change which fixes an issue)

Testing

Test Configuration:

  • Tested with Korean input to verify proper character composition (받침 조합)
  • Confirmed that special characters and spaces are still sanitized on export
  • Verified functionality across different export formats

Before

Jul-31-2025.17-26-06.mp4

After

Jul-31-2025.17-26-21.mp4

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • My changes do not introduce new warnings

@danny-avila danny-avila changed the base branch from main to dev July 31, 2025 11:27
@danny-avila danny-avila changed the title 🔧 fix: Apply filename sanitization at export time instead of input 🔧 fix: Apply Convo Export filename sanitization at export, not input Jul 31, 2025
@danny-avila danny-avila merged commit e192c99 into danny-avila:dev Jul 31, 2025
6 checks passed
@azamara azamara deleted the refactor/export-filename-validation branch July 31, 2025 11:30
xycjscs pushed a commit to xycjscs/LibreChat that referenced this pull request Aug 9, 2025
miguelwon pushed a commit to miguelwon/LibreChat that referenced this pull request Aug 10, 2025
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