SimpleNotificationsModule fails to load with angular-cli v1.5.0-rc8 under AOT

[asadsahi]

Bug Report or Feature Request (mark with an x)

- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Angular CLI: 1.5.0-rc.8
Node: 8.8.1
OS: win32 x64
Angular:
...

Repro steps.

with latest version of angular-cli(v1.5.0-rc.8) and angular 5.0.0-rc.9 with aot getting following error

ERROR in ./node_modules/angular2-notifications/src/simple-notifications/components/simple-notifications.component.ngfactory.ts
Module build failed: Error: C:\Users\Asad\dev\ngcli150\node_modules\angular2-notifications\src\simple-notifications\components\simple-notifications.component.ngfactory.ts is not part of the
compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile (C:\Users\Asad\dev\ngcli150\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:624:23)
    at plugin.done.then (C:\Users\Asad\dev\ngcli150\node_modules\@ngtools\webpack\src\loader.js:467:39)
    at <anonymous>
 @ ./src/app/app.component.ngfactory.js 9:0-141
 @ ./src/app/app.module.ngfactory.js
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://0.0.0.0:0 ./src/main.ts

Without AOT it works fine.

Note: I have to update typescript version to latest as well to make it work with latest angular and cli version.

Here is repo to reproduce this issue:

https://github.com/asadsahi/ngcli150

[weihe8892]

I had same issue, the older version: 1.5.0-rc.3 is fine.

The build is also fine for 1.5.0-rc.8 if following files have been removed from node_modules/angular2-notifications/src/simple-notifications/components
notification.component.ngfactory.ts
simple-notifications.component.ngfactory.ts

[clydin]

The library should not have typescript source files present in its published package. It appears that the presence of an index.ts file in the root of the package is leading the compiler to use the TS source rather than the compiled version.

[asadsahi]

@clydin do you know why it doesn't happen in non-aot mode?

Does that mean this issue needs to be raised with library owners?

[clydin]

At a minimum it should be raised with library owners. Even if there was not an underlying issue, the extra files are taking up hard drive space.

In addition there is potentially an issue with the priority of main file resolution of a module. The behavior here may be intended but further investigation is needed.

[clydin]

As an experiment, can you try manually removing the index.ts file from the package within node_modules?

[filipesilva]

I saw something similar happen in #8216 (comment). I'm going to try to get more information about this, but it looks like the AOT compiler emiting JS for libraries that have TS files.

[asadsahi]

@clydin getting more errors by removing index.ts:

c:/Users/Asad/ngnode-fullstack/src/app/core/core.module.ts
[0] src/app/core/core.module.ts(7,43): error TS7016: Could not find a declaration file for module 'angular2-notifications'. 'c:/Users/Asad/ngnode-fullstack/node_modules/angular2-notifications/dist/index.js' implicitly has an 'any' type.
[0]   Try `npm install @types/angular2-notifications` if it exists or add a new declaration (.d.ts) file containing `declare module 'angular2-notifications';`
[0] src/app/+login/reset-password/reset-password.component.ts(4,38): error TS7016: Could not find a declaration file for module 'angular2-notifications'. 'c:/Users/Asad/ngnode-fullstack/node_modules/angular2-notifications/dist/index.js' implicitly has an 'any' type.
[0]   Try `npm install @types/angular2-notifications` if it exists or add a new declaration (.d.ts) file containing `declare module 'angular2-notifications';`
[0] src/app/+profile/user-info/user-info.component.ts(6,38): error TS7016: Could not find a declaration file for module 'angular2-notifications'. 'c:/Users/Asad/ngnode-fullstack/node_modules/angular2-notifications/dist/index.js' implicitly has an 'any' type.
[0]   Try `npm install @types/angular2-notifications` if it exists or add a new declaration (.d.ts) file containing `declare module 'angular2-notifications';`
[0] src/app/+profile/user-photo/user-photo.component.ts(4,38): error TS7016: Could not find a declaration file for module 'angular2-notifications'. 'c:/Users/Asad/ngnode-fullstack/node_modules/angular2-notifications/dist/index.js' implicitly has an 'any' type.
[0]   Try `npm install @types/angular2-notifications` if it exists or add a new declaration (.d.ts) file containing `declare module 'angular2-notifications';`

[clydin]

The library's package.json is missing a typings field. It probably should be:

"typings": "./dist/index.d.ts",

[filipesilva]

To give you all an update on the issue, @hansl is trying to work with the library authors of known problematic libraries to sort these issues out.