Non-module file does't offer import fixes

[Venthe]

• VSCode Version: 1.29.1; 1-30.0-insider (2018-11-30)
• OS Version: Windows 10 1809

Steps to Reproduce:
Use this zip
callWithTimeout.zip
or

1. Create new project through NPM
2. npm init
3. npm add --save-dev rxjs typescript
4. Create tsconfig.json

{
    "compilerOptions": {
      "rootDir": "./",
      "target": "es6",
      "module": "commonjs",
      "declaration": true,
      "outDir": "./dist",
      "strict": true
    },
    "include": ["src"],
    "exclude": ["node_modules", "**/__tests__/*"],
  }

3. create any file with .ts extension with content:
   let a: Observable<any>;
4. Press ctrl+. with cursor on observable

Does this issue occur when all extensions are disabled?: Yes

Current behaviour: No actions available
Expected behaviour: import from...

What is worth noting, if I add empty import like so import {} from "rxjs" it will suggest to add to existing import

[mjbvz]

Confirmed on typescript@3.3.0-dev.20181201

[DanielRosenwasser]

@sheetalkamat, I was pretty sure that you fixed this with #28745. Any clues?

[sheetalkamat]

@DanielRosenwasser We did that for completions.. We need to do that for suggestion diagnostics as well.

[sheetalkamat]

@DanielRosenwasser We look for code fix from program files because the program in the current scenario does not contain rxJs its not going to be able to look for the import fix that contains symbol for observable so this seems like by design?

Info 53   [10:16:15.912] Project 'c:/temp/callWithTimeout/tsconfig.json' (Configured) 0
Info 53   [10:16:15.912] 	Files (17)
	c:/typescript/built/local/lib.es6.d.ts
	c:/typescript/built/local/lib.es5.d.ts
	c:/typescript/built/local/lib.es2015.d.ts
	c:/typescript/built/local/lib.dom.d.ts
	c:/typescript/built/local/lib.dom.iterable.d.ts
	c:/typescript/built/local/lib.webworker.importscripts.d.ts
	c:/typescript/built/local/lib.scripthost.d.ts
	c:/typescript/built/local/lib.es2015.core.d.ts
	c:/typescript/built/local/lib.es2015.collection.d.ts
	c:/typescript/built/local/lib.es2015.generator.d.ts
	c:/typescript/built/local/lib.es2015.iterable.d.ts
	c:/typescript/built/local/lib.es2015.promise.d.ts
	c:/typescript/built/local/lib.es2015.proxy.d.ts
	c:/typescript/built/local/lib.es2015.reflect.d.ts
	c:/typescript/built/local/lib.es2015.symbol.d.ts
	c:/typescript/built/local/lib.es2015.symbol.wellknown.d.ts
	c:/temp/callWithTimeout/src/index.ts

[DanielRosenwasser]

Let's chat tomorrow, but I would've figured we'd search through the typeRoots.