@atlaspack/cli 2.12.1-dev.3443 → 2.12.1-dev.3478

package/lib/cli.js

@@ -217,10 +217,10 @@ function applyOptions(cmd, options) {
     }
   }
 }
-let serve = program.command('serve [input...]').description('starts a development server').option('--public-url <url>', 'the path prefix for absolute urls').option('--open [browser]', 'automatically open in specified browser, defaults to default browser').option('--watch-for-stdin', 'exit when stdin closes').option('--lazy [includes]', 'Build async bundles on demand, when requested in the browser. Defaults to all async bundles, unless a comma separated list of source file globs is provided. Only async bundles whose entry points match these globs will be built lazily').option('--lazy-exclude <excludes>', 'Can only be used in combination with --lazy. Comma separated list of source file globs, async bundles whose entry points match these globs will not be built lazily').action(runCommand);
+let serve = program.command('serve [input...]').description('starts a development server').option('--public-url <url>', 'the path prefix for absolute urls').option('--open [browser]', 'automatically open in specified browser, defaults to default browser').option('--watch-for-stdin', 'exit when stdin closes').option('--lazy [includes]', 'Build async bundles on demand, when requested in the browser. Defaults to all async bundles, unless a comma separated list of source file globs is provided. Only async bundles whose entry points match these globs will be built lazily').option('--lazy-exclude <excludes>', 'Can only be used in combination with --lazy. Comma separated list of source file globs, async bundles whose entry points match these globs will not be built lazily').option('--production', 'Run with production mode defaults').action(runCommand);
 applyOptions(serve, hmrOptions);
 applyOptions(serve, commonOptions);
-let watch = program.command('watch [input...]').description('starts the bundler in watch mode').option('--public-url <url>', 'the path prefix for absolute urls').option('--no-content-hash', 'disable content hashing').option('--watch-for-stdin', 'exit when stdin closes').action(runCommand);
+let watch = program.command('watch [input...]').description('starts the bundler in watch mode').option('--public-url <url>', 'the path prefix for absolute urls').option('--no-content-hash', 'disable content hashing').option('--watch-for-stdin', 'exit when stdin closes').option('--production', 'Run with production mode defaults').action(runCommand);
 applyOptions(watch, hmrOptions);
 applyOptions(watch, commonOptions);
 let build = program.command('build [input...]').description('bundles for production').option('--no-optimize', 'disable minification').option('--no-scope-hoist', 'disable scope-hoisting').option('--public-url <url>', 'the path prefix for absolute urls').option('--no-content-hash', 'disable content hashing').action(runCommand);
@@ -415,9 +415,12 @@ function parseOptionInt(value) {
   }
   return parsedValue;
 }
+function shouldUseProductionDefaults(command) {
+  return command.name() === 'build' || command.production === true;
+}
 async function normalizeOptions(command, inputFS) {
   let nodeEnv;
-  if (command.name() === 'build') {
+  if (shouldUseProductionDefaults(command)) {
     nodeEnv = process.env.NODE_ENV || 'production';
     // Autoinstall unless explicitly disabled or we detect a CI environment.
     command.autoinstall = !(command.autoinstall === false || process.env.CI);
@@ -443,7 +446,7 @@ async function normalizeOptions(command, inputFS) {
   // Ensure port is valid and available
   let port = parsePort(command.port || '1234');
   let originalPort = port;
-  if (command.name() === 'serve' || command.hmr) {
+  if (!shouldUseProductionDefaults(command) && (command.name() === 'serve' || command.hmr)) {
     try {
       port = await (0, _getPort().default)({
         port,
@@ -481,7 +484,7 @@ async function normalizeOptions(command, inputFS) {
     };
   }
   let hmrOptions = null;
-  if (command.name() !== 'build' && command.hmr !== false) {
+  if (!shouldUseProductionDefaults(command) && command.hmr !== false) {
     let hmrport = command.hmrPort ? parsePort(command.hmrPort) : port;
     let hmrhost = command.hmrHost ? command.hmrHost : host;
     hmrOptions = {
@@ -505,7 +508,7 @@ async function normalizeOptions(command, inputFS) {
       resolveFrom: __filename
     });
   }
-  let mode = command.name() === 'build' ? 'production' : 'development';
+  let mode = shouldUseProductionDefaults(command) ? 'production' : 'development';
   const normalizeIncludeExcludeList = input => {
     if (typeof input !== 'string') return [];
     return input.split(',').map(value => value.trim());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/cli",
-  "version": "2.12.1-dev.3443+d1170cfc7",
+  "version": "2.12.1-dev.3478+5fd2da535",
   "description": "Blazing fast, zero configuration web application bundler",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
@@ -23,18 +23,18 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/config-default": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/core": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/diagnostic": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/events": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/feature-flags": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/fs": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/logger": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/package-manager": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/reporter-cli": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/reporter-dev-server": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/reporter-tracer": "2.12.1-dev.3443+d1170cfc7",
-    "@atlaspack/utils": "2.12.1-dev.3443+d1170cfc7",
+    "@atlaspack/config-default": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/core": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/diagnostic": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/events": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/feature-flags": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/fs": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/logger": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/package-manager": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/reporter-cli": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/reporter-dev-server": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/reporter-tracer": "2.12.1-dev.3478+5fd2da535",
+    "@atlaspack/utils": "2.12.1-dev.3478+5fd2da535",
     "chalk": "^4.1.0",
     "commander": "^7.0.0",
     "get-port": "^4.2.0"
@@ -44,5 +44,5 @@
     "@babel/core": "^7.22.11",
     "rimraf": "^5.0.5"
   },
-  "gitHead": "d1170cfc79beb290b2a066f472f68f71f7d7cb23"
+  "gitHead": "5fd2da535ecbe096d57e03aec15e80bb1d7601f7"
 }

package/src/cli.js

@@ -47,10 +47,10 @@ async function logUncaughtError(e: mixed) {
   }
 
   // A hack to definitely ensure we logged the uncaught exception
-  await new Promise(resolve => setTimeout(resolve, 100));
+  await new Promise((resolve) => setTimeout(resolve, 100));
 }
 
-const handleUncaughtException = async exception => {
+const handleUncaughtException = async (exception) => {
   try {
     await logUncaughtError(exception);
   } catch (err) {
@@ -100,7 +100,7 @@ const commonOptions = {
     'set the root watch directory. defaults to nearest lockfile or source control dir.',
   '--watch-ignore [path]': [
     `list of directories watcher should not be tracking for changes. defaults to ['.git', '.hg']`,
-    dirs => dirs.split(','),
+    (dirs) => dirs.split(','),
   ],
   '--watch-backend': new commander.Option(
     '--watch-backend <name>',
@@ -203,6 +203,7 @@ let serve = program
     '--lazy-exclude <excludes>',
     'Can only be used in combination with --lazy. Comma separated list of source file globs, async bundles whose entry points match these globs will not be built lazily',
   )
+  .option('--production', 'Run with production mode defaults')
   .action(runCommand);
 
 applyOptions(serve, hmrOptions);
@@ -214,6 +215,7 @@ let watch = program
   .option('--public-url <url>', 'the path prefix for absolute urls')
   .option('--no-content-hash', 'disable content hashing')
   .option('--watch-for-stdin', 'exit when stdin closes')
+  .option('--production', 'Run with production mode defaults')
   .action(runCommand);
 
 applyOptions(watch, hmrOptions);
@@ -243,7 +245,7 @@ function makeDebugCommand() {
         if (entries.length === 0) {
           entries = ['.'];
         }
-        entries = entries.map(entry => path.resolve(entry));
+        entries = entries.map((entry) => path.resolve(entry));
 
         Object.assign(command, opts);
 
@@ -281,7 +283,7 @@ program
   .command('help [command]')
   .description('display help information for a command')
   .action(function (command) {
-    let cmd = program.commands.find(c => c.name() === command) || program;
+    let cmd = program.commands.find((c) => c.name() === command) || program;
     cmd.help();
   });
 
@@ -310,7 +312,7 @@ commander.Command.prototype.optionMissingArgument = function (option) {
 var args = process.argv;
 if (args[2] === '--help' || args[2] === '-h') args[2] = 'help';
 
-if (!args[2] || !program.commands.some(c => c.name() === args[2])) {
+if (!args[2] || !program.commands.some((c) => c.name() === args[2])) {
   args.splice(2, 0, 'serve');
 }
 
@@ -329,7 +331,7 @@ async function run(
     entries = ['.'];
   }
 
-  entries = entries.map(entry => path.resolve(entry));
+  entries = entries.map((entry) => path.resolve(entry));
 
   let Atlaspack = require('@atlaspack/core').default;
   let fs = new NodeFS();
@@ -411,7 +413,7 @@ async function run(
   }
 
   if (isWatching) {
-    ({unsubscribe} = await atlaspack.watch(err => {
+    ({unsubscribe} = await atlaspack.watch((err) => {
       if (err) {
         throw err;
       }
@@ -474,12 +476,16 @@ function parseOptionInt(value) {
   return parsedValue;
 }
 
+function shouldUseProductionDefaults(command) {
+  return command.name() === 'build' || command.production === true;
+}
+
 async function normalizeOptions(
   command,
   inputFS,
 ): Promise<InitialAtlaspackOptions> {
   let nodeEnv;
-  if (command.name() === 'build') {
+  if (shouldUseProductionDefaults(command)) {
     nodeEnv = process.env.NODE_ENV || 'production';
     // Autoinstall unless explicitly disabled or we detect a CI environment.
     command.autoinstall = !(command.autoinstall === false || process.env.CI);
@@ -505,7 +511,10 @@ async function normalizeOptions(
   // Ensure port is valid and available
   let port = parsePort(command.port || '1234');
   let originalPort = port;
-  if (command.name() === 'serve' || command.hmr) {
+  if (
+    !shouldUseProductionDefaults(command) &&
+    (command.name() === 'serve' || command.hmr)
+  ) {
     try {
       port = await getPort({port, host});
     } catch (err) {
@@ -542,7 +551,7 @@ async function normalizeOptions(
   }
 
   let hmrOptions = null;
-  if (command.name() !== 'build' && command.hmr !== false) {
+  if (!shouldUseProductionDefaults(command) && command.hmr !== false) {
     let hmrport = command.hmrPort ? parsePort(command.hmrPort) : port;
     let hmrhost = command.hmrHost ? command.hmrHost : host;
 
@@ -558,7 +567,7 @@ async function normalizeOptions(
 
   let additionalReporters = [
     {packageName: '@atlaspack/reporter-cli', resolveFrom: __filename},
-    ...(command.reporter: Array<string>).map(packageName => ({
+    ...(command.reporter: Array<string>).map((packageName) => ({
       packageName,
       resolveFrom: path.join(inputFS.cwd(), 'index'),
     })),
@@ -571,11 +580,13 @@ async function normalizeOptions(
     });
   }
 
-  let mode = command.name() === 'build' ? 'production' : 'development';
+  let mode = shouldUseProductionDefaults(command)
+    ? 'production'
+    : 'development';
 
   const normalizeIncludeExcludeList = (input?: string): string[] => {
     if (typeof input !== 'string') return [];
-    return input.split(',').map(value => value.trim());
+    return input.split(',').map((value) => value.trim());
   };
 
   return {