Extract to function can use inline imports

[mjbvz]

TypeScript Version: 3.9.0-dev.20200423

Search Terms:

• refactor / refactoring
• extract to function / method

Code
For the TypeScript code:

foo.ts

export class Foo { }
export const foo = new Foo();

index.ts

import { foo } from './foo';

{
    const x = foo;
    console.log(x);
}

1. Select console.log(x); and run the extract to function refactoring

Bug:
This adds an inline import type for Foo:

import { foo } from './foo';

{
    const x = foo;
    newFunction(x);
}

function newFunction(x: import("/Users/matb/projects/san/foo").Foo) {
    console.log(x);
}

Instead, it should add the import of Foo to the existing import at the top of the file

Related Issues:

• 'declare method' quick fix adds inline import #37781
• Add option to disable TS using inline import types in quick fixes #35078

[DanielRosenwasser]

Thanks @a-tarasyuk!