feathersjs / feathers

The API and real-time application framework
https://feathersjs.com
MIT License
15.08k stars 750 forks source link

Optimization warnings in Angular 10 #2001

Open arkenstan opened 4 years ago

arkenstan commented 4 years ago

Steps to reproduce

On upgrading angular application to Angular 10.0.0 It gives warning for modules and packages that have dependency on commonjs or AMD for optimization bailouts

PSB Log for warnings

WARNING in src/app/core/services/feathers.service.ts depends on @feathersjs/socketio-client. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

WARNING in src/app/core/services/feathers.service.ts depends on @feathersjs/feathers. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

WARNING in src/app/core/services/feathers.service.ts depends on @feathersjs/authentication-client. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

WARNING in src/app/core/services/feathers.service.ts depends on socket.io-client. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

WARNING in src/app/core/services/feathers.service.ts depends on feathers-reactive. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

System configuration

Module versions @feathersjs/authentication-client: "^4.5.4" @feathersjs/feathers: "^4.5.3" @feathersjs/socketio-client: "^4.5.4" socket.io-client: "^2.3.0"

NodeJS version: Node 12

Operating System: Linux mint 19.3

alejandroTapiaQ commented 4 years ago

HI arkenstan well I resolve that warning adding this in angular.json file

` "architect": { "build": { "builder": "@angular-devkit/build-angular:browser", "options": { "allowedCommonJsDependencies": [ "socket.io-client", "socket.io-parser", "debug" ],

` Also you can see #18025

bertho-zero commented 4 years ago

ESM builds should be added to correctly fix these warnings.

axmad22 commented 4 years ago

Had the same problem Warning: Entry point simplebar-angular contains deep imports try adding in the angular.json in the "build"=> "options":

"allowedCommonJsDependencies": ["core-js"],

Also I kind of remember adding something in the tsconfig.json that helped here is mine case check with default settings

{
  "compileOnSave": false,
  "compilerOptions": {
    "baseUrl": "./",
    "outDir": "./public/out-tsc",
    "sourceMap": true,
    "declaration": false,
    "downlevelIteration": true,
    "experimentalDecorators": true,
    "module": "esnext",
    "moduleResolution": "node",
    "importHelpers": true,
    "target": "es2015",
    "allowSyntheticDefaultImports": true,
    "typeRoots": [
      "node_modules/@types"
    ],
    "lib": [
      "es2018",
      "dom"
    ]
  }
}