Skip to content

Add CodeMirror extension for Operations#9245

Merged
franknoirot merged 7 commits intomainfrom
franknoirot/7564/operations-extension
Feb 13, 2026
Merged

Add CodeMirror extension for Operations#9245
franknoirot merged 7 commits intomainfrom
franknoirot/7564/operations-extension

Conversation

@franknoirot
Copy link
Copy Markdown
Contributor

Towards #8885, stacked on top of #9237 and very much in the same vein. Registers a StateField and StateEffect pair to keep the AST in sync within the CodeMirror state. Broken out from #9190 so we can take safer baby steps towards that.

Another aspect I'm excited about for this one is that once we have a reliable binding between operations and code source ranges right within the CodeMirror editor I can basically get rid of featureTreeMachine. Most of that state machine is dedicated to rigamarole surrounding getting the current editor, opening it, and updating its selection, etc.

This is a refactor only, and should not be felt by users. For now this extension is unused, but it allows us to test the validity of this state, and move towards making all editor-related state being tracked within the EditorView class itself.

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 9, 2025

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

Project Deployment Actions Updated (UTC)
modeling-app Ready Ready Preview, Comment Feb 13, 2026 7:59pm

Request Review

@franknoirot franknoirot marked this pull request as ready for review February 13, 2026 19:45
@franknoirot franknoirot enabled auto-merge (squash) February 13, 2026 19:45
@franknoirot franknoirot merged commit e25a098 into main Feb 13, 2026
58 checks passed
@franknoirot franknoirot deleted the franknoirot/7564/operations-extension branch February 13, 2026 19:59
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