commonjs: this is incorrectly transformed in class field initializers

[connorjclark]

• Rollup Plugin Name: commonjs
• Rollup Plugin Version: 10.1.0
• Rollup Version: 2.52.7
• Operating System (or Browser): n/a
• Node Version: n/a
• Link to reproduction: https://replit.com/@hoten/rollup-repro

Expected Behavior

this in class fields should not be transformed.

Actual Behavior

this is transformed to commonjsGlobal.

Additional Information

A workaround is to set ignoreGlobal: true.

if the word require is in the module then this issue does not occur (even if it exists only in a comment). So another workaround is literally a "magic comment":

// require <<< DON'T TOUCH OR THE BUILD BREAKS

[stale]

Hey folks. This issue hasn't received any traction for 60 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it. ⓘ

[connorjclark]

Any thoughts?

[shellscape]

Folks, this is the not the place for discussion of issue management in this repo or org. Snark is not constructive. There are plenty of other mediums for venting. The message we've chosen for the bot is clear and concise. Please cease off-topic discussion in this issue.

[arackaf]

Fair enough but can we please reopen this ticket for visibility? It's manifesting in upstream tools like Vite.

vitejs/vite#7676