@stencil/core 2.14.1 → 2.15.1

package/bin/stencil

@@ -2,6 +2,7 @@
 'use strict';
 
 var minimumVersion = '12.10';
+var futureDeprecationMinVersion = '14.0';
 var recommendedVersion = '14.5';
 var currentVersion = process.versions.node;
 
@@ -17,18 +18,22 @@ if (isNodeLT(minimumVersion)) {
       currentVersion +
       ', however Stencil requires v' +
       minimumVersion +
-      '.0 or greater. It is recommended to use an Active LTS version of Node (https://nodejs.org/en/about/releases/).\n',
+      '.0 or greater. It is recommended to use an Active LTS version of Node (https://nodejs.org/en/about/releases/).\n'
   );
   process.exit(1);
 }
 
-if (isNodeLT(recommendedVersion)) {
+if (isNodeLT(futureDeprecationMinVersion)) {
+  console.warn(
+    '\nIn an upcoming major release of Stencil, Node v' + recommendedVersion + '.0 or higher will be required.\n'
+  );
+} else if (isNodeLT(recommendedVersion)) {
   console.warn(
     '\nYour current version of Node is v' +
       currentVersion +
       ", however Stencil's recommendation is v" +
       recommendedVersion +
-      '.0 or greater. Note that future versions of Stencil will eventually remove support for non-LTS Node versions and an Active LTS version is recommended (https://nodejs.org/en/about/releases/).\n',
+      '.0 or greater. Note that future versions of Stencil will eventually remove support for non-LTS Node versions and an Active LTS version is recommended (https://nodejs.org/en/about/releases/).\n'
   );
 }
 
@@ -39,13 +44,14 @@ var nodeSys = nodeApi.createNodeSys({ process: process, logger: nodeLogger });
 
 nodeApi.setupNodeProcess({ process: process, logger: nodeLogger });
 
-cli.run({
-  args: process.argv.slice(2),
-  logger: nodeLogger,
-  sys: nodeSys,
-  checkVersion: nodeApi.checkVersion
-})
-.catch(function (err) {
-  console.error('uncaught error', err);
-  process.exit(1);
-});
+cli
+  .run({
+    args: process.argv.slice(2),
+    logger: nodeLogger,
+    sys: nodeSys,
+    checkVersion: nodeApi.checkVersion,
+  })
+  .catch(function (err) {
+    console.error('uncaught error', err);
+    process.exit(1);
+  });

package/cli/index.cjs

@@ -1,5 +1,5 @@
 /*!
- Stencil CLI (CommonJS) v2.14.1 | MIT Licensed | https://stenciljs.com
+ Stencil CLI (CommonJS) v2.15.1 | MIT Licensed | https://stenciljs.com
  */
 'use strict';
 
@@ -259,15 +259,16 @@ const pathComponents = (path, rootLength) => {
  * @returns an error message if the tag has an invalid name, undefined if the tag name passes all checks
  */
 const validateComponentTag = (tag) => {
+    // we want to check this first since we call some String.prototype methods below
+    if (typeof tag !== 'string') {
+        return `Tag "${tag}" must be a string type`;
+    }
     if (tag !== tag.trim()) {
         return `Tag can not contain white spaces`;
     }
     if (tag !== tag.toLowerCase()) {
         return `Tag can not contain upper case characters`;
     }
-    if (typeof tag !== 'string') {
-        return `Tag "${tag}" must be a string type`;
-    }
     if (tag.length === 0) {
         return `Received empty tag value`;
     }
@@ -329,9 +330,19 @@ const parseFlags = (args, sys) => {
     });
     return flags;
 };
+/**
+ * Parse command line arguments that are whitelisted via the BOOLEAN_ARG_OPTS,
+ * STRING_ARG_OPTS, and NUMBER_ARG_OPTS arrays in this file. Handles leading
+ * dashes on arguments, aliases that are defined for a small number of argument
+ * types, and parsing values for non-boolean arguments (e.g. port number).
+ *
+ * @param flags     a ConfigFlags object
+ * @param args      an array of command-line arguments to parse
+ * @param knownArgs an array to which all recognized, legal arguments are added
+ */
 const parseArgs = (flags, args, knownArgs) => {
-    ARG_OPTS.boolean.forEach((booleanName) => {
-        const alias = ARG_OPTS.alias[booleanName];
+    BOOLEAN_ARG_OPTS.forEach((booleanName) => {
+        const alias = ARG_OPTS_ALIASES[booleanName];
         const flagKey = configCase(booleanName);
         if (typeof flags[flagKey] !== 'boolean') {
             flags[flagKey] = null;
@@ -359,8 +370,8 @@ const parseArgs = (flags, args, knownArgs) => {
             }
         });
     });
-    ARG_OPTS.string.forEach((stringName) => {
-        const alias = ARG_OPTS.alias[stringName];
+    STRING_ARG_OPTS.forEach((stringName) => {
+        const alias = ARG_OPTS_ALIASES[stringName];
         const flagKey = configCase(stringName);
         if (typeof flags[flagKey] !== 'string') {
             flags[flagKey] = null;
@@ -403,8 +414,8 @@ const parseArgs = (flags, args, knownArgs) => {
             }
         }
     });
-    ARG_OPTS.number.forEach((numberName) => {
-        const alias = ARG_OPTS.alias[numberName];
+    NUMBER_ARG_OPTS.forEach((numberName) => {
+        const alias = ARG_OPTS_ALIASES[numberName];
         const flagKey = configCase(numberName);
         if (typeof flags[flagKey] !== 'number') {
             flags[flagKey] = null;
@@ -448,50 +459,56 @@ const parseArgs = (flags, args, knownArgs) => {
 };
 const configCase = (prop) => {
     prop = dashToPascalCase(prop);
-    return prop.charAt(0).toLowerCase() + prop.substr(1);
+    return prop.charAt(0).toLowerCase() + prop.slice(1);
 };
-const ARG_OPTS = {
-    boolean: [
-        'build',
-        'cache',
-        'check-version',
-        'ci',
-        'compare',
-        'debug',
-        'dev',
-        'devtools',
-        'docs',
-        'e2e',
-        'es5',
-        'esm',
-        'headless',
-        'help',
-        'log',
-        'open',
-        'prerender',
-        'prerender-external',
-        'prod',
-        'profile',
-        'service-worker',
-        'screenshot',
-        'serve',
-        'skip-node-check',
-        'spec',
-        'ssr',
-        'stats',
-        'update-screenshot',
-        'verbose',
-        'version',
-        'watch',
-    ],
-    number: ['max-workers', 'port'],
-    string: ['address', 'config', 'docs-json', 'emulate', 'log-level', 'root', 'screenshot-connector'],
-    alias: {
-        config: 'c',
-        help: 'h',
-        port: 'p',
-        version: 'v',
-    },
+const BOOLEAN_ARG_OPTS = [
+    'build',
+    'cache',
+    'check-version',
+    'ci',
+    'compare',
+    'debug',
+    'dev',
+    'devtools',
+    'docs',
+    'e2e',
+    'es5',
+    'esm',
+    'headless',
+    'help',
+    'log',
+    'open',
+    'prerender',
+    'prerender-external',
+    'prod',
+    'profile',
+    'service-worker',
+    'screenshot',
+    'serve',
+    'skip-node-check',
+    'spec',
+    'ssr',
+    'stats',
+    'update-screenshot',
+    'verbose',
+    'version',
+    'watch',
+];
+const NUMBER_ARG_OPTS = ['max-workers', 'port'];
+const STRING_ARG_OPTS = [
+    'address',
+    'config',
+    'docs-json',
+    'emulate',
+    'log-level',
+    'root',
+    'screenshot-connector',
+];
+const ARG_OPTS_ALIASES = {
+    config: 'c',
+    help: 'h',
+    port: 'p',
+    version: 'v',
 };
 const getNpmConfigEnvArgs = (sys) => {
     // process.env.npm_config_argv
@@ -513,7 +530,7 @@ const getNpmConfigEnvArgs = (sys) => {
 const dependencies = [
 	{
 		name: "@stencil/core",
-		version: "2.14.1",
+		version: "2.15.1",
 		main: "compiler/stencil.js",
 		resources: [
 			"package.json",
@@ -1453,7 +1470,7 @@ describe('${name}', () => {
 /**
  * Convert a dash case string to pascal case.
  */
-const toPascalCase = (str) => str.split('-').reduce((res, part) => res + part[0].toUpperCase() + part.substr(1), '');
+const toPascalCase = (str) => str.split('-').reduce((res, part) => res + part[0].toUpperCase() + part.slice(1), '');
 
 const taskTelemetry = async (config, sys, logger) => {
     const prompt = logger.dim(sys.details.platform === 'windows' ? '>' : '$');

package/cli/index.js

@@ -1,5 +1,5 @@
 /*!
- Stencil CLI v2.14.1 | MIT Licensed | https://stenciljs.com
+ Stencil CLI v2.15.1 | MIT Licensed | https://stenciljs.com
  */
 const toLowerCase = (str) => str.toLowerCase();
 const dashToPascalCase = (str) => toLowerCase(str)
@@ -235,15 +235,16 @@ const pathComponents = (path, rootLength) => {
  * @returns an error message if the tag has an invalid name, undefined if the tag name passes all checks
  */
 const validateComponentTag = (tag) => {
+    // we want to check this first since we call some String.prototype methods below
+    if (typeof tag !== 'string') {
+        return `Tag "${tag}" must be a string type`;
+    }
     if (tag !== tag.trim()) {
         return `Tag can not contain white spaces`;
     }
     if (tag !== tag.toLowerCase()) {
         return `Tag can not contain upper case characters`;
     }
-    if (typeof tag !== 'string') {
-        return `Tag "${tag}" must be a string type`;
-    }
     if (tag.length === 0) {
         return `Received empty tag value`;
     }
@@ -305,9 +306,19 @@ const parseFlags = (args, sys) => {
     });
     return flags;
 };
+/**
+ * Parse command line arguments that are whitelisted via the BOOLEAN_ARG_OPTS,
+ * STRING_ARG_OPTS, and NUMBER_ARG_OPTS arrays in this file. Handles leading
+ * dashes on arguments, aliases that are defined for a small number of argument
+ * types, and parsing values for non-boolean arguments (e.g. port number).
+ *
+ * @param flags     a ConfigFlags object
+ * @param args      an array of command-line arguments to parse
+ * @param knownArgs an array to which all recognized, legal arguments are added
+ */
 const parseArgs = (flags, args, knownArgs) => {
-    ARG_OPTS.boolean.forEach((booleanName) => {
-        const alias = ARG_OPTS.alias[booleanName];
+    BOOLEAN_ARG_OPTS.forEach((booleanName) => {
+        const alias = ARG_OPTS_ALIASES[booleanName];
         const flagKey = configCase(booleanName);
         if (typeof flags[flagKey] !== 'boolean') {
             flags[flagKey] = null;
@@ -335,8 +346,8 @@ const parseArgs = (flags, args, knownArgs) => {
             }
         });
     });
-    ARG_OPTS.string.forEach((stringName) => {
-        const alias = ARG_OPTS.alias[stringName];
+    STRING_ARG_OPTS.forEach((stringName) => {
+        const alias = ARG_OPTS_ALIASES[stringName];
         const flagKey = configCase(stringName);
         if (typeof flags[flagKey] !== 'string') {
             flags[flagKey] = null;
@@ -379,8 +390,8 @@ const parseArgs = (flags, args, knownArgs) => {
             }
         }
     });
-    ARG_OPTS.number.forEach((numberName) => {
-        const alias = ARG_OPTS.alias[numberName];
+    NUMBER_ARG_OPTS.forEach((numberName) => {
+        const alias = ARG_OPTS_ALIASES[numberName];
         const flagKey = configCase(numberName);
         if (typeof flags[flagKey] !== 'number') {
             flags[flagKey] = null;
@@ -424,50 +435,56 @@ const parseArgs = (flags, args, knownArgs) => {
 };
 const configCase = (prop) => {
     prop = dashToPascalCase(prop);
-    return prop.charAt(0).toLowerCase() + prop.substr(1);
+    return prop.charAt(0).toLowerCase() + prop.slice(1);
 };
-const ARG_OPTS = {
-    boolean: [
-        'build',
-        'cache',
-        'check-version',
-        'ci',
-        'compare',
-        'debug',
-        'dev',
-        'devtools',
-        'docs',
-        'e2e',
-        'es5',
-        'esm',
-        'headless',
-        'help',
-        'log',
-        'open',
-        'prerender',
-        'prerender-external',
-        'prod',
-        'profile',
-        'service-worker',
-        'screenshot',
-        'serve',
-        'skip-node-check',
-        'spec',
-        'ssr',
-        'stats',
-        'update-screenshot',
-        'verbose',
-        'version',
-        'watch',
-    ],
-    number: ['max-workers', 'port'],
-    string: ['address', 'config', 'docs-json', 'emulate', 'log-level', 'root', 'screenshot-connector'],
-    alias: {
-        config: 'c',
-        help: 'h',
-        port: 'p',
-        version: 'v',
-    },
+const BOOLEAN_ARG_OPTS = [
+    'build',
+    'cache',
+    'check-version',
+    'ci',
+    'compare',
+    'debug',
+    'dev',
+    'devtools',
+    'docs',
+    'e2e',
+    'es5',
+    'esm',
+    'headless',
+    'help',
+    'log',
+    'open',
+    'prerender',
+    'prerender-external',
+    'prod',
+    'profile',
+    'service-worker',
+    'screenshot',
+    'serve',
+    'skip-node-check',
+    'spec',
+    'ssr',
+    'stats',
+    'update-screenshot',
+    'verbose',
+    'version',
+    'watch',
+];
+const NUMBER_ARG_OPTS = ['max-workers', 'port'];
+const STRING_ARG_OPTS = [
+    'address',
+    'config',
+    'docs-json',
+    'emulate',
+    'log-level',
+    'root',
+    'screenshot-connector',
+];
+const ARG_OPTS_ALIASES = {
+    config: 'c',
+    help: 'h',
+    port: 'p',
+    version: 'v',
 };
 const getNpmConfigEnvArgs = (sys) => {
     // process.env.npm_config_argv
@@ -489,7 +506,7 @@ const getNpmConfigEnvArgs = (sys) => {
 const dependencies = [
 	{
 		name: "@stencil/core",
-		version: "2.14.1",
+		version: "2.15.1",
 		main: "compiler/stencil.js",
 		resources: [
 			"package.json",
@@ -1429,7 +1446,7 @@ describe('${name}', () => {
 /**
  * Convert a dash case string to pascal case.
  */
-const toPascalCase = (str) => str.split('-').reduce((res, part) => res + part[0].toUpperCase() + part.substr(1), '');
+const toPascalCase = (str) => str.split('-').reduce((res, part) => res + part[0].toUpperCase() + part.slice(1), '');
 
 const taskTelemetry = async (config, sys, logger) => {
     const prompt = logger.dim(sys.details.platform === 'windows' ? '>' : '$');

package/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stencil/core/cli",
-  "version": "2.14.1",
+  "version": "2.15.1",
   "description": "Stencil CLI.",
   "main": "./index.cjs",
   "module": "./index.js",

package/compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stencil/core/compiler",
-  "version": "2.14.1",
+  "version": "2.15.1",
   "description": "Stencil Compiler.",
   "main": "./stencil.js",
   "types": "./stencil.d.ts",