fix: set a correct node location for static blocks

[TrickyPi]

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

List any relevant issue numbers:
resolves #4895

Description

[codecov]

Codecov Report

Merging #4898 (1e224d5) into master (7bff18c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4898   +/-   ##
=======================================
  Coverage   99.01%   99.01%           
=======================================
  Files         221      221           
  Lines        8025     8026    +1     
  Branches     2209     2209           
=======================================
+ Hits         7946     7947    +1     
  Misses         26       26           
  Partials       53       53           

Impacted Files	Coverage Δ
src/ast/nodes/StaticBlock.ts	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[github-actions]

Thank you for your contribution! ❤️

You can try out this pull request locally by installing Rollup via

npm install TrickyPi/rollup#fix/4895

or load it into the REPL:
https://deploy-preview-4898--rollupjs.netlify.app/repl/?pr=4898

[lukastaegert]

Nice catch!

One thing to note: Could you use findFirstOccurrenceOutsideComment instead of indexOf? Because otherwise, this might happen:

https://deploy-preview-4898--rollupjs.netlify.app/repl/?pr=4898&shareable=JTdCJTIyZXhhbXBsZSUyMiUzQW51bGwlMkMlMjJtb2R1bGVzJTIyJTNBJTVCJTdCJTIyY29kZSUyMiUzQSUyMmNsYXNzJTIwRm9vJTIwJTdCJTVDbiUyMCUyMHN0YXRpYyUyMCUyRiolMjBEbyUyMG5vdCUyMHVzZSUyMCU3QiUyMGhlcmUlMjAqJTJGJTIwJTdCJTIwY29uc3QlMjB1bnVzZWQlMjAlM0QlMjAncmVtb3ZlZCclM0IlMjAlN0QlNUNuJTdEJTVDbiU1Q25jb25zb2xlLmxvZyhuZXclMjBGb28oKSklM0IlMjIlMkMlMjJpc0VudHJ5JTIyJTNBdHJ1ZSUyQyUyMm5hbWUlMjIlM0ElMjJtYWluLmpzJTIyJTdEJTVEJTJDJTIyb3B0aW9ucyUyMiUzQSU3QiUyMm91dHB1dCUyMiUzQSU3QiUyMmZvcm1hdCUyMiUzQSUyMmVzJTIyJTdEJTJDJTIydHJlZXNoYWtlJTIyJTNBdHJ1ZSU3RCU3RA==

[TrickyPi]

Nice catch!

One thing to note: Could you use findFirstOccurrenceOutsideComment instead of indexOf? Because otherwise, this might happen:

https://deploy-preview-4898--rollupjs.netlify.app/repl/?pr=4898&shareable=JTdCJTIyZXhhbXBsZSUyMiUzQW51bGwlMkMlMjJtb2R1bGVzJTIyJTNBJTVCJTdCJTIyY29kZSUyMiUzQSUyMmNsYXNzJTIwRm9vJTIwJTdCJTVDbiUyMCUyMHN0YXRpYyUyMCUyRiolMjBEbyUyMG5vdCUyMHVzZSUyMCU3QiUyMGhlcmUlMjAqJTJGJTIwJTdCJTIwY29uc3QlMjB1bnVzZWQlMjAlM0QlMjAncmVtb3ZlZCclM0IlMjAlN0QlNUNuJTdEJTVDbiU1Q25jb25zb2xlLmxvZyhuZXclMjBGb28oKSklM0IlMjIlMkMlMjJpc0VudHJ5JTIyJTNBdHJ1ZSUyQyUyMm5hbWUlMjIlM0ElMjJtYWluLmpzJTIyJTdEJTVEJTJDJTIyb3B0aW9ucyUyMiUzQSU3QiUyMm91dHB1dCUyMiUzQSU3QiUyMmZvcm1hdCUyMiUzQSUyMmVzJTIyJTdEJTJDJTIydHJlZXNoYWtlJTIyJTNBdHJ1ZSU3RCU3RA==

Oh, I didn't take this into account, thanks for your review!

[lukastaegert]

Perfect 👍

[rollup-bot]

This PR has been released as part of rollup@3.19.1. You can test it via npm install rollup.