@ds-sfdc/sfparty 1.5.1 → 1.5.3

package/.github/workflows/cicd.yaml

@@ -9,8 +9,12 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node: [18, 20]
     env:
       working-directory: ./
+      HUSKY: 0  # Disable Husky during the CI/CD process
     steps:
       - run: echo "🎉 The job was automatically triggered by a ${{ github.event_name }} event."
       - run: echo "🐧 This job is now running on a ${{ runner.os }} server hosted by GitHub!"
@@ -20,7 +24,7 @@ jobs:
       - name: Setup node ${{ matrix.node }}
         uses: actions/setup-node@v3
         with:
-          node-version: 19
+          node-version: ${{ matrix.node }}
           cache: 'npm'
           cache-dependency-path: '**/package-lock.json'
       - run: npm ci
@@ -35,4 +39,4 @@ jobs:
       - run: echo "🧪 Running tests ${{ job.status }}."
       - run: npm test
         working-directory: ${{env.working-directory}}
-      - run: echo "🍏 This job's status is ${{ job.status }}."
+      - run: echo "🍏 This job's status is ${{ job.status }}."

package/.vscode/settings.json

@@ -0,0 +1,5 @@
+{
+	"editor.formatOnPaste": true,
+	"editor.formatOnSave": true,
+	"editor.formatOnType": true
+}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ds-sfdc/sfparty",
-	"version": "1.5.1",
+	"version": "1.5.3",
 	"description": "Salesforce metadata XML splitter for CI/CD",
 	"type": "module",
 	"repository": {
@@ -12,9 +12,8 @@
 	},
 	"main": "src/index.js",
 	"scripts": {
-		"test": "jest",
-		"prepack": "pinst --disable",
-		"postpack": "pinst --enable"
+		"test": "jest --no-cache",
+		"prepare": "husky || true"
 	},
 	"keywords": [
 		"salesforce,metadata,xml,split,yaml,json"
@@ -22,31 +21,33 @@
 	"author": "Tim Paulaskas",
 	"license": "BSD-3-Clause",
 	"dependencies": {
-		"axios": "^1.2.2",
-		"cli-color": "^2.0.3",
-		"cli-spinners": "^2.7.0",
+		"axios": "^1.6.8",
+		"cli-color": "^2.0.4",
+		"cli-spinners": "^2.9.2",
 		"convert-hrtime": "^5.0.0",
 		"js-yaml": "^4.1.0",
-		"log-update": "^5.0.1",
-		"marked": "^4.2.12",
-		"marked-terminal": "^5.1.1",
-		"pinst": "^3.0.0",
-		"pretty-quick": "^3.1.3",
-		"semver": "^7.3.8",
-		"util": "^0.10.3",
-		"winston": "^3.8.2",
+		"log-update": "^6.0.0",
+		"marked": "^12.0.1",
+		"marked-terminal": "^7.0.0",
+		"pretty-quick": "^4.0.0",
+		"semver": "^7.6.0",
+		"util": "^0.12.5",
+		"winston": "^3.13.0",
 		"xml2js": "^0.6.2",
-		"yargs": "^17.6.2"
+		"yargs": "^17.7.2"
 	},
 	"devDependencies": {
-		"@babel/core": "^7.20.12",
-		"@babel/preset-env": "^7.20.2",
-		"@commitlint/cli": "^17.4.0",
-		"@commitlint/config-conventional": "^17.4.0",
-		"babel-jest": "^29.3.1",
-		"jest": "^29.3.1",
-		"nodemon": "^3.0.3",
-		"prettier": "^2.8.3"
+		"@babel/core": "^7.24.4",
+		"@babel/preset-env": "^7.24.4",
+		"@commitlint/cli": "^19.2.1",
+		"@commitlint/config-conventional": "^19.1.0",
+		"babel-jest": "^29.7.0",
+		"eslint": "^9.0.0",
+		"husky": "^9.0.11",
+		"jest": "^29.7.0",
+		"lint-staged": "^15.2.2",
+		"nodemon": "^3.1.0",
+		"prettier": "^3.2.5"
 	},
 	"engines": {
 		"node": ">=0.11"
@@ -58,5 +59,11 @@
 	"directories": {
 		"lib": "lib",
 		"test": "tests"
+	},
+	"lint-staged": {
+		"*.js": [
+			"eslint --fix",
+			"prettier --write"
+		]
 	}
 }

package/src/lib/fileUtils.js

@@ -126,11 +126,11 @@ export function saveFile(
 		fileName = fileName.replaceSpecialChars()
 		switch (format) {
 			case 'json':
-				let jsonString = JSON.stringify(json, null, '\t')
+				const jsonString = JSON.stringify(json, null, '\t')
 				fsTmp.writeFileSync(fileName, jsonString)
 				break
 			case 'yaml':
-				let doc = yaml.dump(json)
+				const doc = yaml.dump(json)
 				fsTmp.writeFileSync(fileName, doc)
 				break
 		}
@@ -151,7 +151,11 @@ export function readFile(filePath, convert = true, fsTmp = fs) {
 				flag: 'r',
 			})
 			if (convert && filePath.indexOf('.yaml') != -1) {
-				result = yaml.load(data)
+				result = yaml.load(data, {
+					onWarning: (warning) => {
+						throw new Error(`YAML parsing warning: ${warning}`)
+					},
+				})
 			} else if (convert && filePath.indexOf('.json') != -1) {
 				result = JSON.parse(data)
 			} else if (convert && filePath.indexOf('.xml') != -1) {
@@ -173,7 +177,7 @@ export function readFile(filePath, convert = true, fsTmp = fs) {
 async function convertXML(data) {
 	return new Promise((resolve, reject) => {
 		try {
-			let parser = new Parser()
+			const parser = new Parser()
 			parser.parseString(data, function (err, result) {
 				if (err) throw err
 				resolve(result)

package/src/party/combine.js

@@ -89,7 +89,7 @@ export class Combine {
 
 			if (!fileUtils.directoryExists({ dirPath: that.sourceDir, fs }))
 				reject(`Path does not exist: ${that.sourceDir}`)
-			let types = ['directories', 'singleFiles', 'main']
+			const types = ['directories', 'singleFiles', 'main']
 			types.forEach((type) => {
 				if (that.metadataDefinition[type] !== undefined) {
 					that.#types = that.#types.concat(
@@ -120,9 +120,7 @@ export class Combine {
 			that.#delta = that.metadataDefinition.delta && global.git.delta
 
 			if (that.#delta) {
-				const pathMatch = `/${that.metadataDefinition.directory}/${
-					that.#fileName.shortName
-				}/`
+				const pathMatch = `/${that.metadataDefinition.directory}/${that.#fileName.shortName}/`
 
 				// get a list of all the added files
 				that.#addedFiles = global.metaTypes[
@@ -148,7 +146,7 @@ export class Combine {
 				)
 			}
 
-			let success = processParts(that)
+			const success = processParts(that)
 			// Ensure we only match existing metadata type directory and item
 
 			if (success) {
@@ -253,7 +251,7 @@ export class Combine {
 								`main.${global.format}`,
 							),
 						}
-						let success = processFile(that, key, fileObj, 'main')
+						const success = processFile(that, key, fileObj, 'main')
 						if (!success) {
 							throw new Error('delete XML')
 						}
@@ -559,7 +557,7 @@ export class Combine {
 
 		function hydrateObject(that, json, key, fileObj) {
 			const sortKey = that.metadataDefinition.sortKeys[key]
-			let object = json.object
+			const object = json.object
 
 			try {
 				json[key].forEach((arrItem) => {
@@ -631,9 +629,9 @@ export class Combine {
 		}
 
 		function finishMessage(that) {
-			let executionTime = getTimeDiff(BigInt(that.#startTime))
-			let durationMessage = `${executionTime.seconds}.${executionTime.milliseconds}s`
-			let stateIcon =
+			const executionTime = getTimeDiff(BigInt(that.#startTime))
+			const durationMessage = `${executionTime.seconds}.${executionTime.milliseconds}s`
+			const stateIcon =
 				that.#errorMessage == ''
 					? global.icons.success
 					: global.icons.fail
@@ -746,7 +744,7 @@ function arrangeKeys(that, json, key = undefined) {
 
 function getTimeDiff(startTime, endTime = process.hrtime.bigint()) {
 	const diff = BigInt(endTime) - BigInt(startTime)
-	let executionTime = convertHrtime(diff)
+	const executionTime = convertHrtime(diff)
 	executionTime.seconds = Math.round(executionTime.seconds)
 	executionTime.milliseconds = Math.round(executionTime.milliseconds / 1000)
 	if (executionTime.milliseconds == 0 && executionTime.nanoseconds > 0)