WIP. Adding optional peer dep to facilitate writing the manifest

lib/loaders/manifest.js

@@ -1,27 +1,34 @@
 
+module.exports = loadManifest
 
-module.exports = async function loadManifest(
+const info = {level:"info"}
+const maybeYamlParser = maybeRequire('js-yaml')
+const fs = require('fs').promises
+const pathlib = require('path')
+
+async function loadManifest(
 	project,
 	options = {}
 ){
 	const messages = [];
 	const manifest = {};
 	if(options.manifestDefaults){
-		messages.push({level:'info', description: `Project manifest settings read from manifest defaults`});
+		const manifestPath = pathlib.resolve()
+		messages.push({...info, description: `Project manifest settings read from manifest defaults`});
 	}
 	if (project.manifest) {
-		messages.push({level: 'info', description: `Project manifest settings read from ${project.manifest.$file_path}`, location: 'project.manifest'});
+		messages.push({...info, description: `Project manifest settings read from ${project.manifest.$file_path}`, location: 'project.manifest'});
 		manifestMergeMutate()
 	} else {
 		messages.push({...projectManifestInfo, description: `No manifest.lkml file available`});
 	}
 	if(options.manifest){
-		messages.push({...projectManifestInfo, description: `Project manifest settings read from manifest argument (DEPRECATED. Prefer manifest_overrides instead.)`});
+		messages.push({...info, description: `Project manifest settings read from manifest argument (DEPRECATED. Prefer manifest_overrides instead.)`});
 		manifestMergeMutate()
 	}
 	if(options.manifestOverrides){
-		messages.push({...projectManifestInfo, description: `Project manifest settings read from manifest overrides`});
-
+		messages.push({...info, description: `Project manifest settings read from manifest overrides`});
+		manifestMergeMutate()
 	}
 
 	project.manifest = {
@@ -52,5 +59,14 @@ function manifestMergeMutate(target, toAdd){
 		};
 	Object.assign(target, toAdd);
 	Object.assign(target, ruleMerge);
-	return target
+	return targe;t
+}
+
+function maybeRequire(module){
+	try{
+		return require(module);
+	}
+	catch(e){
+		return undefined;
+	}
 }

package.json

@@ -49,5 +49,8 @@
     "eslint-plugin-jest": "^21.27.2",
     "jest": "^27.3.1",
     "jest-extended": "^1.1.0"
+  },
+  "optionalDependencies": {
+    "js-yaml": "^4.1.0"
   }
 }