Skip to content

Add Floaty CSV support (written with AI) - #13

Merged
acheronfail merged 3 commits into
acheronfail:masterfrom
DT1000101:floaty-csv-support
Aug 29, 2026
Merged

acheronfail merged 3 commits into
acheronfail:masterfrom
DT1000101:floaty-csv-support

Conversation

@DT1000101

Copy link
Copy Markdown
Contributor

Summary

  • Add support for Floaty CSV exports (previously only Floaty JSON worked; CSV was misrouted through the Float Control parser and hung on the loading screen).
  • Detect Float Control vs Floaty CSV by headers, and show a clear error for unrecognised CSV formats instead of spinning forever.
  • Update Welcome/README copy and add fixture + unit tests for routing and Floaty CSV mapping.

Test plan

  • Load a Floaty .csv export — ride opens (no infinite “Parsing your ride…”)
  • Load a Floaty .json export — still works as before
  • Load a Float Control CSV / ZIP — still works
  • Load a VESC Tool semicolon CSV — still works
  • Load a junk CSV with unknown headers — alert + return to file picker
  • npm run checks

Made with Cursor

DT1000101 and others added 2 commits August 2, 2026 17:03
Floaty CSV exports were misrouted through the Float Control parser, which flooded the console and hung on the loading screen. Detect Floaty CSV, map it like Floaty JSON, and show an error for unrecognised formats.

Co-authored-by: Cursor <cursoragent@cursor.com>
@acheronfail

Copy link
Copy Markdown
Owner

Thanks for this change! I've made a couple adjustments to it, but otherwise it seems good.

I'll merge it in and it'll be out in the next release.

@acheronfail
acheronfail merged commit 51b856e into acheronfail:master Aug 29, 2026
1 check 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.

2 participants