fix(cli-app-scripts): use the same build dir for modes (#302)

This removes the devOut path and usage of it, instead opting for
building the development build and production build into the same
folder: ./build.

For libraries in monorepo situations this is important as they will look
up the main or module key in the package.json file, which is usually set
to e.g. ./build/{cjs,es}/lib.js when using the cli-app-scripts to build
the lib.

cli/src/lib/compiler/compileLibrary.js

@@ -24,10 +24,7 @@ const compileLibrary = async ({ config, paths, mode, watch }) => {
     const input =
         (config.entryPoints && config.entryPoints[config.type]) ||
         'src/index.js'
-    const outDir = mode === 'production' ? paths.buildOutput : paths.devOut
-
-    fs.removeSync(outDir)
-    fs.ensureDirSync(outDir)
+    const outDir = paths.buildOutput
 
     const pkg = require(paths.package)
 
@@ -50,6 +47,9 @@ const compileLibrary = async ({ config, paths, mode, watch }) => {
     reporter.debug('Rollup config', rollupConfig)
 
     if (!watch) {
+        fs.removeSync(outDir)
+        fs.ensureDirSync(outDir)
+
         // create a bundle
         try {
             const bundle = await rollup.rollup(rollupConfig)
@@ -81,6 +81,8 @@ const compileLibrary = async ({ config, paths, mode, watch }) => {
             process.exit(1)
         }
     } else {
+        fs.ensureDirSync(outDir)
+
         return new Promise((resolve, reject) => {
             reporter.debug('watching...')
             const watcher = rollup.watch({

cli/src/lib/paths.js

@@ -35,7 +35,6 @@ module.exports = (cwd = process.cwd()) => {
         i18nLocales: path.join(base, './src/locales'),
 
         d2: path.join(base, './.d2/'),
-        devOut: path.join(base, './.d2/devOut'),
         appOutputFilename: 'App.js',
         shell: path.join(base, './.d2/shell'),
         shellAppEntrypoint: path.join(base, './.d2/shell/src/App.js'),