Conversation
Co-authored-by: Alex Tapper <alex.tapper@gmail.com>
|
Cursor Agent can help with this pull request. Just |
|
@cursor review |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
| if (!isPlainPaste && data.text) { | ||
| if (this.insertTableFromText(data.text, sceneX, sceneY)) { | ||
| return; | ||
| } |
There was a problem hiding this comment.
Table insertion completely preempts spreadsheet chart dialog
High Severity
insertTableFromText runs before tryParseSpreadsheet, and parseTabularText accepts any consistent ≥2×2 text grid (no numeric requirement), while tryParseSpreadsheet requires numeric value columns. Every dataset that would have opened the chart dialog (e.g. "time,value\n01:00,61\n02:00,85") now silently creates a table instead, making the paste-to-chart feature completely unreachable via paste. This is a regression for existing chart users.
| return { ok: false, reason: "Less than 2 rows" }; | ||
| } | ||
|
|
||
| return { ok: true, data: bestCandidate.parsed }; |
There was a problem hiding this comment.
Aggressive comma detection creates tables from natural text
Medium Severity
parseTabularText treats any multi-line text with a consistent number of commas per line as tabular data. Natural English text like "I need eggs, milk\nAnd bread, butter" gets detected as a 2×2 table and rendered as rectangles with text elements instead of being pasted as plain text. The minimum threshold of 2 columns with comma as delimiter is too low given how common commas are in prose.
Additional Locations (1)
2 participants
Add support for creating editable tables on the canvas by dragging/dropping CSV files or pasting tabular data.