@ds-sfdc/sfparty 1.4.1 → 1.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ds-sfdc/sfparty",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "Salesforce metadata XML splitter for CI/CD",
   "type": "module",
   "repository": {

package/src/lib/packageUtil.js

@@ -68,8 +68,9 @@ export class Package {
 					packageDefinition.metadataDefinition.fallbackVersion
 			}
 			that.packageJSON = json
-			if (json.Package.types !== undefined)
+			if (json.Package.types !== undefined) {
 				transformJSON(json.Package.types)
+			}
 			cleanPackage(that)
 		}
 
@@ -80,6 +81,7 @@ export class Package {
 				)
 			if (that.packageJSON.Package == undefined)
 				throw new Error('Package initialization failed')
+
 			if (that.packageJSON.Package.types === undefined)
 				return 'No types found'
 
@@ -93,6 +95,12 @@ export class Package {
 					)
 				}
 			})
+
+			// remove all types nodes that have no members
+			that.packageJSON.Package.types =
+				that.packageJSON.Package.types.filter(
+					(subType) => subType.members.length > 0,
+				)
 		}
 	}
 

package/src/meta/PermissionSets.js

@@ -74,4 +74,5 @@ export const metadataDefinition = {
 		tabSettings: ['tab', 'visibility'],
 		userPermissions: ['name', 'enabled'],
 	},
+	delta: false,
 }

package/src/meta/Profiles.js

@@ -83,4 +83,5 @@ export const metadataDefinition = {
 		userPermissions: ['name', 'enabled'],
 	},
 	xmlFirst: 'fullName',
+	delta: false,
 }

package/src/meta/Workflows.js

@@ -124,4 +124,5 @@ export const metadataDefinition = {
 		rules: 'WorkflowRule',
 		tasks: 'WorkflowTask',
 	},
+	delta: true,
 }

package/src/party/combine.js

@@ -32,7 +32,7 @@ export class Combine {
 		mtime: undefined,
 	}
 	#json = {}
-	#diffOnly = false
+	#delta = false
 	#addedFiles = []
 	#deletedFiles = []
 	#mainDeleted = false
@@ -86,11 +86,6 @@ export class Combine {
 	combine() {
 		return new Promise((resolve, reject) => {
 			const that = this
-			that.#diffOnly =
-				global.metaTypes[that.metadataDefinition.alias].add.files
-					.length > 0 ||
-				global.metaTypes[that.metadataDefinition.alias].remove.files
-					.length > 0
 
 			if (!fileUtils.directoryExists({ dirPath: that.sourceDir, fs }))
 				reject(`Path does not exist: ${that.sourceDir}`)
@@ -148,6 +143,9 @@ export class Combine {
 					i.toLowerCase().includes(`/main.${global.format}`),
 			)
 
+			// set delta based on metadata definition
+			that.#delta = that.metadataDefinition.delta
+
 			let success = processParts(that)
 			// Ensure we only match existing metadata type directory and item
 
@@ -156,7 +154,7 @@ export class Combine {
 					!that.metadataDefinition.packageTypeIsDirectory &&
 					global.git.enabled
 				) {
-					if (!that.#diffOnly || that.#addedFiles.length > 0) {
+					if (!that.#delta || that.#addedFiles.length > 0) {
 						that.addPkg.addMember(
 							that.#root,
 							that.#fileName.shortName,
@@ -164,7 +162,7 @@ export class Combine {
 					}
 
 					// only include the workflow node if main part file is delete
-					if (that.#diffOnly && that.#mainDeleted) {
+					if (that.#delta && that.#mainDeleted) {
 						that.desPkg.addMember(
 							that.#root,
 							that.#fileName.shortName,
@@ -389,7 +387,7 @@ export class Combine {
 				)
 			}
 
-			if (that.#diffOnly) {
+			if (that.#delta) {
 				if (
 					!that.#addedFiles
 						.concat(that.#deletedFiles)

package/test/lib/package/getPackageXML.spec.js

@@ -152,7 +152,7 @@ it('should correctly process the json object returned from the XML file', async
 					name: 'PermissionSet',
 				},
 				{
-					members: ['Test', 'Test.yaml'],
+					members: ['Test.yaml'],
 					name: 'Workflow',
 				},
 			],
@@ -182,10 +182,6 @@ it('should correctly process the json object returned from the XML file', async
 					members: ['Test'],
 					name: 'PermissionSet',
 				},
-				{
-					members: ['Test'],
-					name: 'Workflow',
-				},
 			],
 			version: '56.0',
 		},