I'll have to add tests to this before it gets merged :)

Noticed the reason for the lock files being in the repo, lol
So I added them back again

Perhaps further tests should be added, but please take a look at this one first and tell me what you think, aka if it is good or not. If it is, then I'll have a look at adding some more perhaps.

There seems to be an issue with renaming the inner names of namespaces.
Aka, the actual namespace name will be changed, but if you have a function inside of that namespace with a custom name, the calling of that function will not receive that customName.

(Hence the error from tests atm)

Code for reprod:

/** @customName Test2 **/
namespace Test {
  /** @customName Func2 **/
  export function Func(): string {
    return "hi";
  }
}

export const result = Test.Func();

Seems like the .Func part (last line) isn't being passed through the identifier transformation, hm...

This code is probably hidden somewhere in the callexpression transformer. I have considered unifying that more with the transformIdentifier code but haven't looked at it into much detail.

I'll see if I can find where it is being transformed sometime later today :)
Are the tests looking fair enough though?

Tests look fine, I'm still missing one where you put the customName on a declare function func() (can't assert execution result on that one ofc).

Tests look fine, I'm still missing one where you put the customName on a declare function func() (can't assert execution result on that one ofc).

I added a test for that as well now :)

Could you please have a look at tstl-development in discord? Thank you

🎉