Skip to content

Do not allow users to enter sketches if there are parser errors#10849

Merged
franknoirot merged 7 commits intomainfrom
franknoirot/adhoc/cancel-sketch-with-errors-fix
Apr 7, 2026
Merged

Do not allow users to enter sketches if there are parser errors#10849
franknoirot merged 7 commits intomainfrom
franknoirot/adhoc/cancel-sketch-with-errors-fix

Conversation

@franknoirot
Copy link
Copy Markdown
Contributor

@franknoirot franknoirot commented Apr 6, 2026

It's possible to unrecoverably overwrite your file's contents by entering sketch mode when you have parser errors. This Codex-written PR prevents that, and adds an integration test for it.

Demo

Screenshare.-.2026-04-06.2_59_17.PM-compressed.mp4

It's possible to unrecoverably overwrite your file's contents by
entering sketch mode when you have parser errors. This Codex-written PR
prevents that, and adds a unit test for it.
@franknoirot franknoirot requested a review from a team as a code owner April 6, 2026 18:56
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
modeling-app Ready Ready Preview, Comment Apr 7, 2026 2:09pm

Request Review

franknoirot and others added 2 commits April 6, 2026 22:38
Co-authored-by: Jace Browning <[email protected]>
Co-authored-by: Jace Browning <[email protected]>
franknoirot and others added 2 commits April 7, 2026 09:54
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
@franknoirot franknoirot merged commit 041d698 into main Apr 7, 2026
61 of 63 checks passed
@franknoirot franknoirot deleted the franknoirot/adhoc/cancel-sketch-with-errors-fix branch April 7, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants