Skip to content

Allow "unsafe" refactoring code actions (with warning)#259

Merged
webspam merged 22 commits into
masterfrom
codeaction-semantics
Jun 16, 2026
Merged

Allow "unsafe" refactoring code actions (with warning)#259
webspam merged 22 commits into
masterfrom
codeaction-semantics

Conversation

@webspam

@webspam webspam commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Features

  • Standardised and improved side-effect detection for code actions (extract variable, inline variable, and split/join local var declarations)
  • When detected, the action is still offered but marked "(unsafe)", allowing an informed decision

Refactoring

  • Consolidates logic into the BodyModel abstraction
  • Completely rewrites and consolidates large tracts of disparate code action logic
  • Body block is now compiled once per event instead of partially-compiled for every action
  • Large clean up; deduplicates several functions
  • Merges three extract action handlers into one shared handler

@webspam webspam enabled auto-merge (squash) June 16, 2026 19:43
@webspam webspam merged commit b149262 into master Jun 16, 2026
2 checks passed
@webspam webspam deleted the codeaction-semantics branch June 16, 2026 19:46
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