Skip to content

Fix issues with undefined/null trivia items#1531

Merged
TwitchBronBron merged 1 commit intov1from
fix-trivia-crash
Jul 16, 2025
Merged

Fix issues with undefined/null trivia items#1531
TwitchBronBron merged 1 commit intov1from
fix-trivia-crash

Conversation

@TwitchBronBron
Copy link
Member

Fixes several issues related to trivia arrays containing undefined. If plugins decide to use editor.removeProperty insead of editor.arraySplice, then those array entries are turned into undefined, and that was crashing the transpiler. This makes those functions more resilient.

Also updates the plugin docs to have a more accurate comment stripping plugin example.

@TwitchBronBron TwitchBronBron added the create-package create a temporary npm package on every commit label Jul 15, 2025
@rokucommunity-bot
Copy link
Contributor

Hey there! I just built a new temporary npm package based on e88b420. You can download it here or install it by running the following command:

npm install https://github.com/rokucommunity/brighterscript/releases/download/v0.0.0-packages/brighterscript-1.0.0-alpha.47-fix-trivia-crash.20250715203545.tgz

@TwitchBronBron TwitchBronBron merged commit a90c682 into v1 Jul 16, 2025
8 checks passed
@TwitchBronBron TwitchBronBron deleted the fix-trivia-crash branch July 16, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

create-package create a temporary npm package on every commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant