fix(*): package code is bundled twice from src and es5 file

Remove `JavaScript` from `src` folder as this is not required by the consumers and in some cases with `Angular 4.x.x` and `@ngtools/webpack` it causes the `src` to be bundled togather with the `FESM`.

With Angular 4 and Webpack the javascript in `src` is being used rather than the FESM.

Relates: ngrx#184

build/builder.ts

@@ -9,6 +9,7 @@ export default createBuilder([
   ['Creating UMD Bundles', tasks.createUmdBundles],
   ['Renaming package entry files', tasks.renamePackageEntryFiles],
   ['Cleaning TypeScript files', tasks.cleanTypeScriptFiles],
+  ['Cleaning JavaScript files', tasks.cleanJavaScriptFiles],
   ['Removing remaining sourcemap files', tasks.removeRemainingSourceMapFiles],
   ['Copying type definition files', tasks.copyTypeDefinitionFiles],
   ['Minifying UMD bundles', tasks.minifyUmdBundles],

build/tasks.ts

@@ -122,6 +122,21 @@ export async function cleanTypeScriptFiles(config: Config) {
   await mapAsync(filesToRemove, util.remove);
 }
 
+/**
+ * Removes any leftover Javascript files from previous compilation steps,
+ * leaving the bundles and FESM in place
+ */
+export async function cleanJavaScriptFiles(config: Config) {
+  const jsFilesGlob = './dist/packages/**/*.js';
+  const jsExcludeFilesFlob = './dist/packages/(bundles|@ngrx)/**/*.js';
+  const filesToRemove = await util.getListOfFiles(
+    jsFilesGlob,
+    jsExcludeFilesFlob
+  );
+
+  await mapAsync(filesToRemove, util.remove);
+}
+
 /**
  * Renames the index files in each package to the name
  * of the package.