@stencil/core 2.17.1 → 2.17.3

package/compiler/stencil.js

@@ -1,5 +1,5 @@
 /*!
- Stencil Compiler v2.17.1 | MIT Licensed | https://stenciljs.com
+ Stencil Compiler v2.17.3 | MIT Licensed | https://stenciljs.com
  */
 (function(exports) {
 'use strict';
@@ -898,24 +898,37 @@ const buildWarn = (diagnostics) => {
   diagnostics.push(diagnostic);
   return diagnostic;
 };
-const buildJsonFileError = (compilerCtx, diagnostics, jsonFilePath, msg, pkgKey) => {
+/**
+ * Create a diagnostic message suited for representing an error in a JSON
+ * file. This includes information about the exact lines in the JSON file which
+ * caused the error and the path to the file.
+ *
+ * @param compilerCtx the current compiler context
+ * @param diagnostics a list of diagnostics used as a return param
+ * @param jsonFilePath the path to the JSON file where the error occurred
+ * @param msg the error message
+ * @param jsonField the key for the field which caused the error, used for finding
+ * the error line in the original JSON file
+ * @returns a reference to the newly-created diagnostic
+ */
+const buildJsonFileError = (compilerCtx, diagnostics, jsonFilePath, msg, jsonField) => {
   const err = buildError(diagnostics);
   err.messageText = msg;
   err.absFilePath = jsonFilePath;
-  if (typeof pkgKey === 'string') {
+  if (typeof jsonField === 'string') {
     try {
       const jsonStr = compilerCtx.fs.readFileSync(jsonFilePath);
       const lines = jsonStr.replace(/\r/g, '\n').split('\n');
       for (let i = 0; i < lines.length; i++) {
         const txtLine = lines[i];
-        const txtIndex = txtLine.indexOf(pkgKey);
+        const txtIndex = txtLine.indexOf(jsonField);
         if (txtIndex > -1) {
           const warnLine = {
             lineIndex: i,
             lineNumber: i + 1,
             text: txtLine,
             errorCharStart: txtIndex,
-            errorLength: pkgKey.length,
+            errorLength: jsonField.length,
           };
           err.lineNumber = warnLine.lineNumber;
           err.columnNumber = txtIndex + 1;
@@ -4052,7 +4065,7 @@ const createCustomResolverAsync = (sys, inMemoryFs, exts) => {
   };
 };
 
-const buildId = '20220711170106';
+const buildId = '20220802224219';
 const minfyJsId = 'terser5.6.1_7';
 const optimizeCssId = 'autoprefixer10.2.5_postcss8.2.13_7';
 const parse5Version = '6.0.1';
@@ -4060,8 +4073,8 @@ const rollupVersion = '2.42.3';
 const sizzleVersion = '2.42.3';
 const terserVersion = '5.6.1';
 const typescriptVersion = '4.5.4';
-const vermoji = '😊';
-const version$3 = '2.17.1';
+const vermoji = '🌭';
+const version$3 = '2.17.3';
 const versions = {
   stencil: version$3,
   parse5: parse5Version,
@@ -14939,6 +14952,25 @@ class MockMouseEvent extends MockEvent {
     }
   }
 }
+class MockUIEvent extends MockEvent {
+  constructor(type, uiEventInitDic) {
+    super(type);
+    this.detail = null;
+    this.view = null;
+    if (uiEventInitDic != null) {
+      Object.assign(this, uiEventInitDic);
+    }
+  }
+}
+class MockFocusEvent extends MockUIEvent {
+  constructor(type, focusEventInitDic) {
+    super(type);
+    this.relatedTarget = null;
+    if (focusEventInitDic != null) {
+      Object.assign(this, focusEventInitDic);
+    }
+  }
+}
 class MockEventListener {
   constructor(type, handler) {
     this.type = type;
@@ -15852,7 +15884,7 @@ class MockElement extends MockNode {
     return shadowRoot;
   }
   blur() {
-    /**/
+    dispatchEvent(this, new MockFocusEvent('blur', { relatedTarget: null, bubbles: true, cancelable: true, composed: true }));
   }
   get shadowRoot() {
     return this.__shadowRoot || null;
@@ -15922,7 +15954,9 @@ class MockElement extends MockNode {
   get firstElementChild() {
     return this.children[0] || null;
   }
-  focus(_options) { }
+  focus(_options) {
+    dispatchEvent(this, new MockFocusEvent('focus', { relatedTarget: null, bubbles: true, cancelable: true, composed: true }));
+  }
   getAttribute(attrName) {
     if (attrName === 'style') {
       if (this.__style != null && this.__style.length > 0) {
@@ -16267,6 +16301,9 @@ class MockElement extends MockNode {
   set title(value) {
     this.setAttributeNS(null, 'title', value);
   }
+  animate() {
+    /**/
+  }
   onanimationstart() {
     /**/
   }
@@ -16531,6 +16568,18 @@ class MockElement extends MockNode {
   onwheel() {
     /**/
   }
+  requestFullscreen() {
+    /**/
+  }
+  scrollBy() {
+    /**/
+  }
+  scrollTo() {
+    /**/
+  }
+  scrollIntoView() {
+    /**/
+  }
   toString(opts) {
     return serializeNodeToHtml(this, opts);
   }
@@ -17572,6 +17621,7 @@ const GLOBAL_CONSTRUCTORS = [
   ['CustomEvent', MockCustomEvent],
   ['Event', MockEvent],
   ['Headers', MockHeaders],
+  ['FocusEvent', MockFocusEvent],
   ['KeyboardEvent', MockKeyboardEvent],
   ['MouseEvent', MockMouseEvent],
   ['Request', MockRequest],
@@ -54587,6 +54637,23 @@ const isMemberPrivate = (member) => {
   }
   return false;
 };
+/**
+ * Convert a JavaScript value to the TypeScript Intermediate Representation
+ * (IR) for a literal Abstract Syntax Tree (AST) node with that same value. The
+ * value to convert may be a primitive type like `string`, `boolean`, etc or
+ * may be an `Object`, `Array`, etc.
+ *
+ * Note that this function takes a param (`refs`) with a default value,
+ * normally a value should _not_ be passed for this parameter since it is
+ * intended to be used for recursive calls.
+ *
+ * @param val the value to convert
+ * @param refs a set of references, used in recursive calls to avoid
+ * circular references when creating object literal IR instances. **note that
+ * you shouldn't pass this parameter unless you know what you're doing!**
+ * @returns TypeScript IR for a literal corresponding to the JavaScript value
+ * with which the function was called
+ */
 const convertValueToLiteral = (val, refs = null) => {
   if (refs == null) {
     refs = new WeakSet();
@@ -54617,12 +54684,39 @@ const convertValueToLiteral = (val, refs = null) => {
   }
   return t.createLiteral(val);
 };
+/**
+ * Convert a JavaScript Array instance to TypeScript's Intermediate
+ * Representation (IR) for an array literal. This is done by recursively using
+ * {@link convertValueToLiteral} to create a new array consisting of the
+ * TypeScript IR of each element in the array to be converted, and then creating
+ * the TypeScript IR for _that_ array.
+ *
+ * @param list the array instance to convert
+ * @param refs a set of references to objects, used when converting objects to
+ * avoid circular references
+ * @returns TypeScript IR for the array we want to convert
+ */
 const arrayToArrayLiteral = (list, refs) => {
   const newList = list.map((l) => {
     return convertValueToLiteral(l, refs);
   });
   return t.createArrayLiteral(newList);
 };
+/**
+ * Convert a JavaScript object (i.e. an object existing at runtime) to the
+ * corresponding TypeScript Intermediate Representation (IR)
+ * ({@see ts.ObjectLiteralExpression}) for an object literal. This function
+ * takes an argument holding a `WeakSet` of references to objects which is
+ * used to avoid circular references. Objects that are converted in this
+ * function are added to the set, and if an object is already present then an
+ * `undefined` literal (in TypeScript IR) is returned instead of another
+ * object literal, as continuing to convert a circular reference would, well,
+ * never end!
+ *
+ * @param obj the JavaScript object to convert to TypeScript IR
+ * @param refs a set of references to objects, used to avoid circular references
+ * @returns a TypeScript object literal expression
+ */
 const objectToObjectLiteral = (obj, refs) => {
   if (refs.has(obj)) {
     return t.createIdentifier('undefined');
@@ -55990,17 +56084,16 @@ const KEEP_IMPORTS = new Set([
  * which do actual work of generating the rollup configuration, creating an
  * entry chunk, running, the build, etc.
  *
- * @param config the user-supplied compiler configuration we're using
+ * @param config the validated compiler configuration we're using
  * @param compilerCtx the current compiler context
  * @param buildCtx the current build context
  * @returns an empty Promise which won't resolve until the work is done!
  */
 const outputCustomElements = async (config, compilerCtx, buildCtx) => {
-  var _a;
   if (!config.buildDist) {
     return;
   }
-  const outputTargets = ((_a = config.outputTargets) !== null && _a !== void 0 ? _a : []).filter(isOutputTargetDistCustomElements);
+  const outputTargets = config.outputTargets.filter(isOutputTargetDistCustomElements);
   if (outputTargets.length === 0) {
     return;
   }
@@ -56013,7 +56106,7 @@ const outputCustomElements = async (config, compilerCtx, buildCtx) => {
  * Get bundle options for our current build and compiler context which we'll use
  * to generate a Rollup build and so on.
  *
- * @param config user-supplied Stencil configuration
+ * @param config a validated Stencil configuration object
  * @param buildCtx the current build context
  * @param compilerCtx the current compiler context
  * @param outputTarget the outputTarget we're currently dealing with
@@ -56044,9 +56137,10 @@ const getBundleOptions = (config, buildCtx, compilerCtx, outputTarget) => ({
 /**
  * Get bundle options for rollup, run the rollup build, optionally minify the
  * output, and write files to disk.
- * @param config user-supplied Stencil configuration
- * @param buildCtx the current build context
+ *
+ * @param config the validated Stencil configuration we're using
  * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
  * @param outputTarget the outputTarget we're currently dealing with
  * @returns an empty promise
  */
@@ -56135,7 +56229,7 @@ const addCustomElementInputs = (buildCtx, bundleOpts) => {
       exp.push(`import { ${importName} as ${importAs}, defineCustomElement as cmpDefCustomEle } from '${cmp.sourceFilePath}';`);
       exp.push(`export const ${exportName} = ${importAs};`);
       exp.push(`export const defineCustomElement = cmpDefCustomEle;`);
-      // Here we push an export (with a rename for `defineCustomElement` for
+      // Here we push an export (with a rename for `defineCustomElement`) for
       // this component onto our array which references the `coreKey` (prefixed
       // with `\0`). We have to do this so that our import is referencing the
       // correct virtual module, if we instead referenced, for instance,
@@ -60170,8 +60264,7 @@ const relDts$1 = (fromPath, dtsPath) => {
  * @param typesDir the path to the directory where type declarations are saved
  */
 const generateCustomElementsTypes = async (config, compilerCtx, buildCtx, typesDir) => {
-  var _a;
-  const outputTargets = ((_a = config.outputTargets) !== null && _a !== void 0 ? _a : []).filter(isOutputTargetDistCustomElements);
+  const outputTargets = config.outputTargets.filter(isOutputTargetDistCustomElements);
   await Promise.all(outputTargets.map((outputTarget) => generateCustomElementsTypesOutput(config, compilerCtx, buildCtx, typesDir, outputTarget)));
 };
 /**
@@ -60187,7 +60280,7 @@ const generateCustomElementsTypesOutput = async (config, compilerCtx, buildCtx,
   // the path where we're going to write the typedef for the whole dist-custom-elements output
   const customElementsDtsPath = join(outputTarget.dir, 'index.d.ts');
   // the directory where types for the individual components are written
-  const componentsTypeDirectoryPath = relative$1(outputTarget.dir, join(typesDir, 'components'));
+  const componentsTypeDirectoryRelPath = relative$1(outputTarget.dir, typesDir);
   const components = buildCtx.components.filter((m) => !m.isCollectionDependency);
   const code = [
     `/* ${config.namespace} custom elements */`,
@@ -60196,7 +60289,14 @@ const generateCustomElementsTypesOutput = async (config, compilerCtx, buildCtx,
       const importName = component.componentClassName;
       // typedefs for individual components can be found under paths like
       // $TYPES_DIR/components/my-component/my-component.d.ts
-      const componentDTSPath = join(componentsTypeDirectoryPath, component.tagName, component.tagName);
+      //
+      // To construct this path we:
+      //
+      // - get the relative path to the component's source file from the source directory
+      // - join that relative path to the relative path from the `index.d.ts` file to the
+      //   directory where typedefs are saved
+      const componentSourceRelPath = relative$1(config.srcDir, component.sourceFilePath).replace('.tsx', '');
+      const componentDTSPath = join(componentsTypeDirectoryRelPath, componentSourceRelPath);
       return `export { ${importName} as ${exportName} } from '${componentDTSPath}';`;
     }),
     ``,
@@ -62099,6 +62199,16 @@ const parseModuleImport = (config, compilerCtx, buildCtx, moduleFile, dirPath, i
   }
 };
 
+/**
+ * Update an instance of TypeScript's Intermediate Representation (IR) for a
+ * class declaration ({@link ts.ClassDeclaration}) with a static getter for the
+ * compiler metadata that we produce as part of the compilation process.
+ *
+ * @param cmpNode an instance of the TypeScript IR for a class declaration (i.e.
+ * a stencil component) to be updated
+ * @param cmpMeta the component metadata corresponding to that component
+ * @returns the updated typescript class declaration
+ */
 const addComponentMetaStatic = (cmpNode, cmpMeta) => {
   const publicCompilerMeta = getPublicCompilerMeta(cmpMeta);
   const cmpMetaStaticProp = createStaticGetter('COMPILER_META', convertValueToLiteral(publicCompilerMeta));
@@ -62230,6 +62340,14 @@ const parseStaticEvents = (staticMembers) => {
   });
 };
 
+/**
+ * Parse a list of {@link ts.ClassElement} objects representing static props
+ * into a list of our own Intermediate Representation (IR) of properties on
+ * components.
+ *
+ * @param staticMembers TypeScript IR for the properties on our component
+ * @returns a manifest of compiler properties in our own Stencil IR
+ */
 const parseStaticProps = (staticMembers) => {
   const parsedProps = getStaticValue(staticMembers, 'properties');
   if (!parsedProps) {
@@ -62466,7 +62584,22 @@ const setComponentBuildConditionals = (cmpMeta) => {
     !cmpMeta.hasMember && !cmpMeta.hasStyle && !cmpMeta.hasLifecycle && !cmpMeta.hasListener && !cmpMeta.hasVdomRender;
 };
 
-const parseStaticComponentMeta = (compilerCtx, typeChecker, cmpNode, moduleFile, nodeMap, transformOpts) => {
+/**
+ * Given an instance of TypeScript's Intermediate Representation (IR) for a
+ * class declaration ({@see ts.ClassDeclaration}) which represents a Stencil
+ * component class declaration, parse and format various pieces of data about
+ * static class members which we use in the compilation process
+ *
+ * @param compilerCtx the current compiler context
+ * @param typeChecker a TypeScript type checker instance
+ * @param cmpNode the TypeScript class declaration for the component
+ * @param moduleFile Stencil's IR for a module, used here as an out param
+ * @param transformOpts options which control various aspects of the
+ * transformation
+ * @returns the TypeScript class declaration IR instance with which the
+ * function was called
+ */
+const parseStaticComponentMeta = (compilerCtx, typeChecker, cmpNode, moduleFile, transformOpts) => {
   if (cmpNode.members == null) {
     return cmpNode;
   }
@@ -62583,7 +62716,7 @@ const parseStaticComponentMeta = (compilerCtx, typeChecker, cmpNode, moduleFile,
   // add to module map
   moduleFile.cmps.push(cmp);
   // add to node map
-  nodeMap.set(cmpNode, cmp);
+  compilerCtx.nodeMap.set(cmpNode, cmp);
   return cmpNode;
 };
 const parseVirtualProps = (docs) => {
@@ -62651,7 +62784,7 @@ const updateModule = (config, compilerCtx, buildCtx, tsSourceFile, sourceFileTex
   compilerCtx.changedModules.add(moduleFile.sourceFilePath);
   const visitNode = (node) => {
     if (t.isClassDeclaration(node)) {
-      parseStaticComponentMeta(compilerCtx, typeChecker, node, moduleFile, compilerCtx.nodeMap);
+      parseStaticComponentMeta(compilerCtx, typeChecker, node, moduleFile);
       return;
     }
     else if (t.isImportDeclaration(node)) {
@@ -62812,6 +62945,15 @@ const outputServiceWorkers = async (config, buildCtx) => {
   }
 };
 
+/**
+ * Validate the package.json file for a project, checking that various fields
+ * are set correctly for the currently-configured output targets.
+ *
+ * @param config the user-supplied Stencil config
+ * @param compilerCtx the compiler context
+ * @param buildCtx the build context
+ * @returns an empty Promise
+ */
 const validateBuildPackageJson = async (config, compilerCtx, buildCtx) => {
   if (config.watch) {
     return;
@@ -62819,26 +62961,41 @@ const validateBuildPackageJson = async (config, compilerCtx, buildCtx) => {
   if (buildCtx.packageJson == null) {
     return;
   }
-  const outputTargets = config.outputTargets.filter(isOutputTargetDistCollection);
+  const distCollectionOutputTargets = config.outputTargets.filter(isOutputTargetDistCollection);
   const typesOutputTargets = config.outputTargets.filter(isOutputTargetDistTypes);
   await Promise.all([
-    ...outputTargets.map((outputsTarget) => {
-      return validatePackageJsonOutput(config, compilerCtx, buildCtx, outputsTarget);
-    }),
-    ...typesOutputTargets.map((outputTarget) => {
-      return validateTypes(config, compilerCtx, buildCtx, outputTarget);
-    }),
+    ...distCollectionOutputTargets.map((distCollectionOT) => validateDistCollectionPkgJson(config, compilerCtx, buildCtx, distCollectionOT)),
+    ...typesOutputTargets.map((typesOT) => validateTypes(config, compilerCtx, buildCtx, typesOT)),
+    validateModule(config, compilerCtx, buildCtx),
   ]);
 };
-const validatePackageJsonOutput = async (config, compilerCtx, buildCtx, outputTarget) => {
+/**
+ * Validate package.json contents for the `DIST_COLLECTION` output target,
+ * checking that various fields like `files`, `main`, and so on are set
+ * correctly.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param outputTarget a DIST_COLLECTION output target
+ */
+const validateDistCollectionPkgJson = async (config, compilerCtx, buildCtx, outputTarget) => {
   await Promise.all([
     validatePackageFiles(config, compilerCtx, buildCtx, outputTarget),
     validateMain(config, compilerCtx, buildCtx, outputTarget),
-    validateModule(config, compilerCtx, buildCtx, outputTarget),
     validateCollection$1(config, compilerCtx, buildCtx, outputTarget),
     validateBrowser(config, compilerCtx, buildCtx),
   ]);
 };
+/**
+ * Validate that the `files` field in `package.json` contains directories and
+ * files that are necessary for the `DIST_COLLECTION` output target.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param outputTarget a DIST_COLLECTION output target
+ */
 const validatePackageFiles = async (config, compilerCtx, buildCtx, outputTarget) => {
   if (!config.devMode && Array.isArray(buildCtx.packageJson.files)) {
     const actualDistDir = normalizePath$1(relative$1(config.rootDir, outputTarget.dir));
@@ -62862,6 +63019,15 @@ const validatePackageFiles = async (config, compilerCtx, buildCtx, outputTarget)
     }));
   }
 };
+/**
+ * Check that the `main` field is set correctly in `package.json` for the
+ * `DIST_COLLECTION` output target.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param outputTarget a DIST_COLLECTION output target
+ */
 const validateMain = (config, compilerCtx, buildCtx, outputTarget) => {
   const mainAbs = join(outputTarget.dir, 'index.cjs.js');
   const mainRel = relative$1(config.rootDir, mainAbs);
@@ -62874,26 +63040,71 @@ const validateMain = (config, compilerCtx, buildCtx, outputTarget) => {
     packageJsonWarn(config, compilerCtx, buildCtx, msg, `"main"`);
   }
 };
-const validateModule = (config, compilerCtx, buildCtx, outputTarget) => {
-  const customElementsOutput = config.outputTargets.find(isOutputTargetDistCustomElementsBundle);
+/**
+ * Validate the package.json 'module' field, taking into account output targets
+ * and other configuration details. This will look for a value for the `module`
+ * field. If not present it will set a relevant warning message with an
+ * output-target specific recommended value. If it is present and is not equal
+ * to that recommended value it will set a different warning message.
+ *
+ * @param config the current user-supplied configuration
+ * @param compilerCtx the compiler context
+ * @param buildCtx the build context
+ * @returns an empty Promise
+ */
+const validateModule = async (config, compilerCtx, buildCtx) => {
   const currentModule = buildCtx.packageJson.module;
-  const distAbs = join(outputTarget.dir, 'index.js');
-  const distRel = relative$1(config.rootDir, distAbs);
-  let recommendedRelPath = distRel;
-  if (customElementsOutput) {
-    const customElementsAbs = join(customElementsOutput.dir, 'index.js');
-    recommendedRelPath = relative$1(config.rootDir, customElementsAbs);
-  }
+  const recommendedRelPath = recommendedModulePath(config);
   if (!isString$1(currentModule)) {
-    const msg = `package.json "module" property is required when generating a distribution. It's recommended to set the "module" property to: ${recommendedRelPath}`;
+    let msg = 'package.json "module" property is required when generating a distribution.';
+    if (recommendedRelPath !== null) {
+      msg += ` It's recommended to set the "module" property to: ${normalizePath$1(recommendedRelPath)}`;
+    }
     packageJsonWarn(config, compilerCtx, buildCtx, msg, `"module"`);
+    return;
   }
-  else if (normalizePath$1(currentModule) !== normalizePath$1(recommendedRelPath) &&
-    normalizePath$1(currentModule) !== normalizePath$1(distRel)) {
-    const msg = `package.json "module" property is set to "${currentModule}". It's recommended to set the "module" property to: ${recommendedRelPath}`;
+  if (recommendedRelPath !== null && normalizePath$1(recommendedRelPath) !== normalizePath$1(currentModule)) {
+    const msg = `package.json "module" property is set to "${currentModule}". It's recommended to set the "module" property to: ${normalizePath$1(recommendedRelPath)}`;
     packageJsonWarn(config, compilerCtx, buildCtx, msg, `"module"`);
   }
 };
+// TODO(STENCIL-516): Investigate the hierarchy of these output targets
+/**
+ * Get the recommended `"module"` path for `package.json` given the output
+ * targets that a user has set on their config.
+ *
+ * @param config the user-supplied Stencil configuration
+ * @returns a recommended module path or a null value to indicate no default
+ * value is supplied
+ */
+function recommendedModulePath(config) {
+  const customElementsBundleOT = config.outputTargets.find(isOutputTargetDistCustomElementsBundle);
+  const customElementsOT = config.outputTargets.find(isOutputTargetDistCustomElements);
+  const distCollectionOT = config.outputTargets.find(isOutputTargetDistCollection);
+  if (distCollectionOT) {
+    return relative$1(config.rootDir, join(distCollectionOT.dir, 'index.js'));
+  }
+  if (customElementsOT) {
+    const componentsIndexAbs = join(customElementsOT.dir, 'index.js');
+    return relative$1(config.rootDir, componentsIndexAbs);
+  }
+  if (customElementsBundleOT) {
+    const customElementsAbs = join(customElementsBundleOT.dir, 'index.js');
+    return relative$1(config.rootDir, customElementsAbs);
+  }
+  // if no output target for which we define a recommended output target is set
+  // we return `null`
+  return null;
+}
+/**
+ * Check that the `types` field is set correctly in `package.json` for the
+ * `DIST_COLLECTION` output target.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param outputTarget a DIST_COLLECTION output target
+ */
 const validateTypes = async (config, compilerCtx, buildCtx, outputTarget) => {
   const typesAbs = getComponentsDtsTypesFilePath(outputTarget);
   const recommendedPath = relative$1(config.rootDir, typesAbs);
@@ -62917,6 +63128,15 @@ const validateTypes = async (config, compilerCtx, buildCtx, outputTarget) => {
     }
   }
 };
+/**
+ * Check that the `collection` field is set correctly in `package.json` for the
+ * `DIST_COLLECTION` output target.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param outputTarget a DIST_COLLECTION output target
+ */
 const validateCollection$1 = (config, compilerCtx, buildCtx, outputTarget) => {
   if (outputTarget.collectionDir) {
     const collectionRel = join(relative$1(config.rootDir, outputTarget.collectionDir), COLLECTION_MANIFEST_FILE_NAME);
@@ -62926,19 +63146,51 @@ const validateCollection$1 = (config, compilerCtx, buildCtx, outputTarget) => {
     }
   }
 };
+/**
+ * Check that the `browser` field is set correctly in `package.json` for the
+ * `DIST_COLLECTION` output target.
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ */
 const validateBrowser = (config, compilerCtx, buildCtx) => {
   if (isString$1(buildCtx.packageJson.browser)) {
     const msg = `package.json "browser" property is set to "${buildCtx.packageJson.browser}". However, for maximum compatibility with all bundlers it's recommended to not set the "browser" property and instead ensure both "module" and "main" properties are set.`;
     packageJsonWarn(config, compilerCtx, buildCtx, msg, `"browser"`);
   }
 };
-const packageJsonError = (config, compilerCtx, buildCtx, msg, warnKey) => {
-  const err = buildJsonFileError(compilerCtx, buildCtx.diagnostics, config.packageJsonFilePath, msg, warnKey);
+/**
+ * Build a diagnostic for an error resulting from a particular field in a
+ * package.json file
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param msg an error string
+ * @param jsonField the key for the field which caused the error, used for
+ * finding the error line in the original JSON file
+ * @returns a diagnostic object
+ */
+const packageJsonError = (config, compilerCtx, buildCtx, msg, jsonField) => {
+  const err = buildJsonFileError(compilerCtx, buildCtx.diagnostics, config.packageJsonFilePath, msg, jsonField);
   err.header = `Package Json`;
   return err;
 };
-const packageJsonWarn = (config, compilerCtx, buildCtx, msg, warnKey) => {
-  const warn = buildJsonFileError(compilerCtx, buildCtx.diagnostics, config.packageJsonFilePath, msg, warnKey);
+/**
+ * Build a diagnostic for a warning resulting from a particular field in a
+ * package.json file
+ *
+ * @param config the stencil config
+ * @param compilerCtx the current compiler context
+ * @param buildCtx the current build context
+ * @param msg an error string
+ * @param jsonField the key for the field which caused the error, used for
+ * finding the error line in the original JSON file
+ * @returns a diagnostic object
+ */
+const packageJsonWarn = (config, compilerCtx, buildCtx, msg, jsonField) => {
+  const warn = buildJsonFileError(compilerCtx, buildCtx.diagnostics, config.packageJsonFilePath, msg, jsonField);
   warn.header = `Package Json`;
   warn.level = 'warn';
   return warn;
@@ -64397,14 +64649,37 @@ const updateCompilerCtxCache = (config, compilerCtx, path, kind) => {
   }
 };
 
+/**
+ * All the Boolean options supported by the Stencil CLI
+ */
+/**
+ * Helper function for initializing a `ConfigFlags` object. Provide any overrides
+ * for default values and off you go!
+ *
+ * @param init an object with any overrides for default values
+ * @returns a complete CLI flag object
+ */
+const createConfigFlags = (init = {}) => {
+  const flags = {
+    task: null,
+    args: [],
+    knownArgs: [],
+    unknownArgs: [],
+    ...init,
+  };
+  return flags;
+};
+
 const getConfig = (userConfig) => {
-  var _a, _b;
-  const flags = (_a = userConfig.flags) !== null && _a !== void 0 ? _a : {};
-  const logger = (_b = userConfig.logger) !== null && _b !== void 0 ? _b : createLogger();
-  const config = { ...userConfig, flags, logger };
-  if (!config.sys) {
-    config.sys = createSystem({ logger: config.logger });
-  }
+  var _a, _b, _c, _d;
+  const logger = (_a = userConfig.logger) !== null && _a !== void 0 ? _a : createLogger();
+  const config = {
+    ...userConfig,
+    flags: createConfigFlags((_b = userConfig.flags) !== null && _b !== void 0 ? _b : {}),
+    logger,
+    outputTargets: (_c = userConfig.outputTargets) !== null && _c !== void 0 ? _c : [],
+    sys: (_d = userConfig.sys) !== null && _d !== void 0 ? _d : createSystem({ logger }),
+  };
   setPlatformPath(config.sys.platformPath);
   if (config.flags.debug || config.flags.verbose) {
     config.logLevel = 'debug';
@@ -65110,7 +65385,7 @@ const getComponentPathContent = (componentGraph, outputTarget) => {
 const dependencies = [
 	{
 		name: "@stencil/core",
-		version: "2.17.1",
+		version: "2.17.3",
 		main: "compiler/stencil.js",
 		resources: [
 			"package.json",
@@ -66463,7 +66738,8 @@ const validateWorkers = (config) => {
  * @returns an object with config and diagnostics props
  */
 const validateConfig = (userConfig = {}, bootstrapConfig) => {
-  const config = Object.assign({}, userConfig || {}); // not positive it's json safe
+  var _a, _b, _c;
+  const config = Object.assign({}, userConfig); // not positive it's json safe
   const diagnostics = [];
   const logger = bootstrapConfig.logger || config.logger || createLogger();
   const validatedConfig = {
@@ -66471,6 +66747,8 @@ const validateConfig = (userConfig = {}, bootstrapConfig) => {
     // flags _should_ be JSON safe
     flags: JSON.parse(JSON.stringify(config.flags || {})),
     logger,
+    outputTargets: (_a = config.outputTargets) !== null && _a !== void 0 ? _a : [],
+    sys: (_c = (_b = config.sys) !== null && _b !== void 0 ? _b : bootstrapConfig.sys) !== null && _c !== void 0 ? _c : createSystem({ logger }),
   };
   // default devMode false
   if (validatedConfig.flags.prod) {
@@ -66737,6 +67015,7 @@ const hasStencilConfigInclude = (includeProp) => Array.isArray(includeProp) && i
  * @public
  */
 const loadConfig = async (init = {}) => {
+  var _a;
   const results = {
     config: null,
     diagnostics: [],
@@ -66751,9 +67030,12 @@ const loadConfig = async (init = {}) => {
   };
   const unknownConfig = {};
   try {
-    const sys = init.sys || createSystem();
     const config = init.config || {};
     let configPath = init.configPath || config.configPath;
+    // Pull the {@link CompilerSystem} out of the initialization object, or create one if it does not exist.
+    // This entity is needed to load the project's configuration (and therefore needs to be created before it can be
+    // attached to a configuration entity, validated or otherwise)
+    const sys = (_a = init.sys) !== null && _a !== void 0 ? _a : createSystem();
     const loadedConfigFile = await loadConfigFile(sys, results.diagnostics, configPath);
     if (hasError(results.diagnostics)) {
       return results;
@@ -67045,7 +67327,7 @@ const convertStaticToMeta = (config, compilerCtx, buildCtx, typeChecker, collect
     let moduleFile;
     const visitNode = (node) => {
       if (t.isClassDeclaration(node)) {
-        return parseStaticComponentMeta(compilerCtx, typeChecker, node, moduleFile, compilerCtx.nodeMap, transformOpts);
+        return parseStaticComponentMeta(compilerCtx, typeChecker, node, moduleFile, transformOpts);
       }
       else if (t.isImportDeclaration(node)) {
         parseModuleImport(config, compilerCtx, buildCtx, moduleFile, dirPath, node, !transformOpts.isolatedModules);