Skip to content

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

Open
DT1000101 wants to merge 1 commit into
acheronfail:masterfrom
DT1000101:floaty-csv-support
Open

Add Floaty CSV support (written with AI)#13
DT1000101 wants to merge 1 commit into
acheronfail:masterfrom
DT1000101:floaty-csv-support

Conversation

@DT1000101

Copy link
Copy Markdown

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

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>
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