this is transpiled incorrectly in a class instance field

[arackaf]

Describe the bug

I have a library, which is transpiled (by Vite) to something like this

'use strict';

Object.defineProperties(exports, {
  __esModule: { value: true },
  [Symbol.toStringTag]: { value: 'Module' },
});

class MutationManager {
  runMutation = (variables) => {
    console.log('RUN MUTATION');
  };
  currentState = {
    runMutation: this.runMutation,
  };
}

exports.MutationManager = MutationManager;

Note the currentState property.

When this library is loaded into a Vite app, and built (ie npm run build), currentState is transformed incorrectly, where runMutation: this.runMutation becomes

__publicField(this, "currentState", {
  runMutation: commonjsGlobal.runMutation
});

The Vite.new link below reproduces this all in full (after running npm run build)

Reproduction

https://stackblitz.com/edit/vitejs-vite-f18nb2?file=vite.config.js&terminal=dev

System Info

Repro is the same on my machine vs the sandbox link above, but for whatever it's worth, mine is

System:
    OS: macOS 12.3
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 442.83 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.11.1 - /usr/local/Cellar/nvm/0.38.0/versions/node/v16.11.1/bin/node
    Yarn: 1.22.17 - /usr/local/Cellar/nvm/0.38.0/versions/node/v16.11.1/bin/yarn
    npm: 8.0.0 - /usr/local/Cellar/nvm/0.38.0/versions/node/v16.11.1/bin/npm
  Browsers:
    Chrome: 100.0.4896.75

Used Package Manager

npm

Logs

n/a

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[bluwy]

Looks like an upstream issue in @rollup/plugin-commonjs: rollup/plugins#922

[bluwy]

Fixed in rollup/plugins#1537