@adobe/aio-cli-lib-app-config 1.0.1-pre.2023-07-14.sha-628de670 → 2.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/aio-cli-lib-app-config",
-  "version": "1.0.1-pre.2023-07-14.sha-628de670",
+  "version": "2.0.0",
   "description": "node lib to provide a consistent interface to various config files that make up Adobe Developer App Builder applications and extensions",
   "repository": "https://github.com/adobe/aio-cli-lib-app-config/",
   "license": "Apache-2.0",
@@ -19,7 +19,7 @@
   },
   "dependencies": {
     "@adobe/aio-lib-core-config": "^3.0.0",
-    "@adobe/aio-lib-core-logging": "next",
+    "@adobe/aio-lib-core-logging": "^2.0.0",
     "@adobe/aio-lib-env": "^2.0.0",
     "ajv": "^8.12.0",
     "ajv-formats": "^2.1.1",
@@ -74,6 +74,5 @@
     "setupFilesAfterEnv": [
       "./test/jest.setup.js"
     ]
-  },
-  "prereleaseSha": "628de6708a25f55af14bf772b3a6e4d16d353ac4"
-}
+  }
+}

package/src/index.js

@@ -264,28 +264,17 @@ async function loadCommonConfig (/* istanbul ignore next */options = {}) {
 // }
 
 /**
- * Resolve all includes, update relative paths and return a coalesced app
- * configuration object.
- *
- * Returns the appConfig along with an index of config keys to config file. The
- * config file paths in the index are absolute.
+ * Resolve all includes, update relative paths and return a coalesced app configuration object.
  *
  * @param {string} appConfigFile path to the app.config.yaml
  * @param {object} options options
- * @param {object} options.absolutePaths boolean, true for rewriting
- *  configuration paths to absolute, false for relative to the appConfigFile
- *  directory. Defaults to false. Note, that config values will never be
- *  rewritten as relative to the cwd. But also note that
- *  this option doesn't have any effect on the includeIndex paths which stay
- *  relative to the cwd.
- * @returns {object} { config, includeIndex }
+ * @param {object} options.absolutePaths boolean, true for absolute paths, default for relative to appConfigFile directory.
+ * @returns {object} single appConfig with resolved includes
  */
 async function coalesce (appConfigFile, options = {}) {
   // this code is traversing app.config.yaml recursively to resolve all $includes directives
 
   const absolutePaths = options.absolutePaths === undefined ? false : options.absolutePaths
-  const appRoot = path.dirname(appConfigFile)
-
   const config = yaml.safeLoad(await fs.readFile(appConfigFile, 'utf8'))
   // keep an index that will map keys like 'extensions.abc.runtimeManifest' to the config file where there are defined
   const includeIndex = {}
@@ -369,7 +358,7 @@ async function coalesce (appConfigFile, options = {}) {
   }
 
   const appConfigWithIncludeIndex = { config, includeIndex }
-  rewritePathsInPlace(appConfigWithIncludeIndex, { absolutePaths, appRoot })
+  rewritePathsInPlace(appConfigWithIncludeIndex, { absolutePaths })
 
   return appConfigWithIncludeIndex
 }
@@ -477,7 +466,7 @@ function rewritePathsInPlace (appConfigWithIncludeIndex, options) {
 
     if (typeof value === 'string' && PATH_KEYS.filter(reg => fullKey.match(reg)).length) {
       // rewrite path value to be relative to the root instead of being relative to the config file that includes it
-      parentObj[key] = resolveToRoot(value, includedFromConfigFile, options)
+      parentObj[key] = resolveToRoot(value, includedFromConfigFile, { absolutePaths: options.absolutePaths })
     }
     if (typeof value === 'object') {
       // object or Array
@@ -696,16 +685,10 @@ async function buildSingleConfig (configName, singleUserConfig, commonConfig, in
 function resolveToRoot (pathValue, includedFromConfigPath, options = {}) {
   // path.resolve => support both absolute pathValue and relative (relative joins with
   // config dir and process.cwd, absolute returns pathValue)
-  if (options.absolutePaths) {
-    return path.resolve(path.dirname(includedFromConfigPath), pathValue)
-  }
-
-  // relative paths
-  if (options.appRoot) {
-    // make sure path is relative to appRoot and not cwd
-    includedFromConfigPath = path.relative(options.appRoot, includedFromConfigPath)
-  }
-  return path.join(path.dirname(includedFromConfigPath), pathValue).split(path.sep).join(path.posix.sep)
+  return options.absolutePaths
+    ? path.resolve(path.dirname(includedFromConfigPath), pathValue)
+    // if relative keep unix paths
+    : path.join(path.dirname(includedFromConfigPath), pathValue).split(path.sep).join(path.posix.sep)
 }
 
 module.exports = {