chore(tools): Improve deps, assets and system in project.config

[ludohenin]

Extensibility of the config members is too complex ...

• DEV_NPM_DEPENDENCIES / PROD_NPM_DEPENDENCIES
• SYSTEM_CONFIG_DEV & SYSTEM_BUILDER_CONFIG
• APP_ASSETS

... I'll work out something to make it more elegant than this:

export class ProjectConfig extends SeedConfig {
  //...

  DEV_NPM_DEPENDENCIES: InjectableDependency[] = [
    ...this.DEV_NPM_DEPENDENCIES,
    ...this.normalizeDependencies([
      { src: 'materialize-css/dist/css/materialize.min.css', inject: true }
    ])
  ];

  PROD_NPM_DEPENDENCIES: InjectableDependency[] = [
    ...this.PROD_NPM_DEPENDENCIES,
    ...this.normalizeDependencies([
      { src: 'materialize-css/dist/css/materialize.min.css', inject: true }
    ])
  ];

  APP_ASSETS: InjectableDependency[] = [
    { src: `${this.ASSETS_SRC}/prism.min.js`,  inject: 'libs' },
    { src: `${this.ASSETS_SRC}/prism.css`, inject: true },
    ...this.APP_ASSETS
  ];

  DEV_DEPENDENCIES = this.DEV_NPM_DEPENDENCIES.concat(this.APP_ASSETS);
  PROD_DEPENDENCIES = this.PROD_NPM_DEPENDENCIES.concat(this.APP_ASSETS);

  FONTS_SRC = [
    'node_modules/materialize-css/dist/font/**/*'
  ];

  SYSTEM_CONFIG_DEV = {
  defaultJSExtensions: true,
    paths: {
      [this.BOOTSTRAP_MODULE]: `${this.APP_BASE}${this.BOOTSTRAP_MODULE}`,
      'config': `${this.APP_ROOT}config`,
      'angular2/*': `${this.APP_BASE}angular2/*`,
      'rxjs/*': `${this.APP_BASE}rxjs/*`,
      '*': `${this.APP_BASE}node_modules/*`
    },
    packages: {
      angular2: { defaultExtension: false },
      rxjs: { defaultExtension: false },
      moment: { main: 'moment.js' },
      lodash: { main: 'lodash.js' }
    }
  };

  SYSTEM_CONFIG = this.SYSTEM_CONFIG_DEV;

  SYSTEM_BUILDER_CONFIG = {
    defaultJSExtensions: true,
    paths: {
      [`${this.TMP_DIR}/*`]: `${this.TMP_DIR}/*`,
      '*': 'node_modules/*'
    },
    packages: {
      angular2: { defaultExtension: false },
      rxjs: { defaultExtension: false },
      moment: { main: 'moment.js' },
      lodash: { main: 'lodash.js' }
    }
  };
}

[TheDonDope]

@ludohenin 598efd5 introduced an example override, what is your opinion on it? :)

[ludohenin]

What do you think adding getters for all computed config options (also related to #630) ?
This way we prevent DEV_DEST (for instance) to be set be before we redefine DIST_DIR in project conf.

get DEV_DEST() { return `${this.DIST_DIR}/dev`; }

// cc @mgechev @d3viant0ne

[joshwiens]

@ludohenin Sounds like a viable way to make the change mentions in #630 without stepping on a bunch of existing projects.

@NathanWalker - Thoughts? Advanced Seed is one of the larger extension projects.

[mgechev]

Sounds good to me.

[mgechev]

#644 fixed 1. and 3.

I think we can close this issue for now until we have further plans on it.