self reference in static block is not working well on rename

[nicolashenry]

Rollup Version

3.10.0

Operating System (or Browser)

Chrome

Node Version (if applicable)

No response

Link To Reproduction

https://rollupjs.org/repl/?version=3.10.0&shareable=JTdCJTIybW9kdWxlcyUyMiUzQSU1QiU3QiUyMm5hbWUlMjIlM0ElMjJtYWluLmpzJTIyJTJDJTIyY29kZSUyMiUzQSUyMiUyRiolMjBUUkVFLVNIQUtJTkclMjAqJTJGJTVDbmltcG9ydCUyMCU3QiUyMFRlc3QlMjAlN0QlMjBmcm9tJTIwJy4lMkZ0ZXN0MS5qcyclM0IlNUNuaW1wb3J0JTIwJTdCJTIwVGVzdCUyMGFzJTIwVGVzdDIlMjAlN0QlMjBmcm9tJTIwJy4lMkZ0ZXN0Mi5qcyclM0IlNUNuY29uc29sZS5sb2coVGVzdC50ZXN0JTJDJTIwVGVzdDIudGVzdCklMjIlMkMlMjJpc0VudHJ5JTIyJTNBdHJ1ZSU3RCUyQyU3QiUyMm5hbWUlMjIlM0ElMjJ0ZXN0MS5qcyUyMiUyQyUyMmNvZGUlMjIlM0ElMjIlNUNuZXhwb3J0JTIwY2xhc3MlMjBUZXN0JTIwJTdCJTVDbiUyMCUyMHN0YXRpYyUyMHRlc3QlMjAlM0QlMjAnVGVzdDEnJTNCJTVDbiUyMCUyMHN0YXRpYyUyMCU3QiUyMGNvbnNvbGUubG9nKFRlc3QudGVzdCklM0IlMjAlN0QlNUNuJTdEJTIyJTJDJTIyaXNFbnRyeSUyMiUzQWZhbHNlJTdEJTJDJTdCJTIybmFtZSUyMiUzQSUyMnRlc3QyLmpzJTIyJTJDJTIyY29kZSUyMiUzQSUyMiU1Q25leHBvcnQlMjBjbGFzcyUyMFRlc3QlMjAlN0IlNUNuJTIwJTIwc3RhdGljJTIwdGVzdCUyMCUzRCUyMCdUZXN0MiclM0IlNUNuJTIwJTIwc3RhdGljJTIwJTdCJTIwY29uc29sZS5sb2coVGVzdC50ZXN0KSUzQiUyMCU3RCU1Q24lN0QlMjIlN0QlNUQlMkMlMjJvcHRpb25zJTIyJTNBJTdCJTIyZm9ybWF0JTIyJTNBJTIyZXMlMjIlMkMlMjJuYW1lJTIyJTNBJTIybXlCdW5kbGUlMjIlMkMlMjJhbWQlMjIlM0ElN0IlMjJpZCUyMiUzQSUyMiUyMiU3RCUyQyUyMmdsb2JhbHMlMjIlM0ElN0IlN0QlN0QlMkMlMjJleGFtcGxlJTIyJTNBbnVsbCU3RA==

Expected Behaviour

When running, it should give the following in console.

Test1
Test2
Test1 Test2

Actual Behaviour

Currently it give an error:

VM93:3 Uncaught ReferenceError: Test$1 is not defined
    at <static_initializer> (<anonymous>:3:24)
    at <anonymous>:1:14

[TrickyPi]

@lukastaegert I'm wondering if this is a right way to solve this problem, using the class name instead of renderName in the static block, as shown in the following code.

let Test$1 = class Test {
  static test = 'Test1';
  static { console.log(Test.test); }
};

[lukastaegert]

Yes, I think that should work.

[nicolashenry]

I realized that keeping name on static block is maybe not enough, it should probably be kept on non-static block also. For example would give this as a result if the fix is done only on static block (so an error would be raised):

let Test$1 = class Test {
  constructor() {
    console.log(Test$1.test);
  }
  static test = 'Test1';
  static {
    console.log(Test.test);
    new Test();
  }
};

The name should probably be also be kept everywhere inside the class:

let Test$1 = class Test {
  constructor() {
    console.log(Test.test);
  }
  static test = 'Test1';
  static {
    console.log(Test.test);
    new Test();
  }
};

I am not sure that the PR is taking this into account.

[TrickyPi]

Oh, thanks! I didn't take this into account, I'll fix it later in the PR.

[rollup-bot]

This issue has been resolved via #4827 as part of rollup@3.12.1. You can test it via npm install rollup.