Bug in transpileModule in ts 2.9.1

[arackaf]

Please try to reproduce the issue with typescript@next. It may have already been fixed.

TypeScript Version: 3.0.0-dev.20180606 - still fails

Search terms you tried before logging this (so others can find this issue more easily)

transpileModule, 2.9.1

Code

No publicly available repro

This issue was originally raised here

TypeStrong/ts-loader#787 (comment)

until it was realized this was a bug with TS itself. It seems transpileModule is dropping valid, needed imports, which is creating runtime errors

Expected behavior:

Used imports will not be dropped

Actual behavior:

They're dropped

Playground Link:

Related Issues:

[mhegazy]

looks like a duplicate of #24551. should be fixed by #24591.

The fix should be in typescript@next today, and in typescript@2.9.2 next week.

Can you give typescript@next a try and confirm that my analysis is correct?

[arackaf]

@mhegazy - my apologies, I said I had tested with ts@next, and I did, but I think webpack's cache-loader incorrectly served me old (wrong) results. ts@next does indeed work.

Any idea when 2.9.2 will be out? You said next week - I don't suppose there's any chance it could be sooner?

[mhegazy]

thanks for confirming. typescript@2.9.2 should be out next week. there are a few other issues that we are tracking as well.

[manu-st]

I have a reproducible project where even the latest version of TypeScript available did not work. See TypeStrong/ts-loader#787 (comment)

[mhegazy]

here is the file text:

import { Options } from "./Options";
import { Parent } from "./Parent";

export class Child extends Parent
{
   constructor (name: string)
   {
      super ("1", name, new Options ());
   }
}

[mhegazy]

So super(new Options()) work, super("", new Options()) does not.

thanks for the repro.

[mhegazy]

Fixed by #24802

[arackaf]

@mhegazy you guys are awesome. Any idea when 2.9.2 will be out?

[mhegazy]

Original plan is to get it out Wednesday. we are still looking at one final issue. so it could be Thursday. but barring any unforeseen circumstances, it should be out this week.

[arackaf]

thanks a ton!