Fixed issue where additional package paths would add duplicate files

Author: jtpetty

app/exec/extension/_lib/vsix-manifest-builder.ts

@@ -794,22 +794,29 @@ export class VsixManifestBuilder extends ManifestBuilder {
 			});
 		}
 		return contentTypePromise.then(() => {
+			let seenPartNames = new Set();
 			Object.keys(this.files).forEach(filePath => {
 				if (this.files[filePath].contentType) {
+					let partName = "/" + toZipItemName(this.files[filePath].partName);
 					contentTypes.Types.Override.push({
 						$: {
 							ContentType: this.files[filePath].contentType,
-							PartName: "/" + toZipItemName(this.files[filePath].partName),
+							PartName: partName,
 						},
 					});
+					seenPartNames.add(partName);
 					if ((this.files[filePath] as any)._additionalPackagePaths) {
-						for (const additionalPath of (this.files[filePath] as any)._additionalPackagePaths) {
-							contentTypes.Types.Override.push({
-								$: {
-									ContentType: this.files[filePath].contentType,
-									PartName: "/" + toZipItemName(additionalPath),
-								},
-							});
+						for (const additionalPath of (this.files[filePath] as any)._additionalPackagePaths) {							
+							let additionalPartName =  "/" + toZipItemName(additionalPath);
+							if (!seenPartNames.has(additionalPartName)) {
+								contentTypes.Types.Override.push({
+									$: {
+										ContentType: this.files[filePath].contentType,
+										PartName: additionalPartName,
+									},
+								});
+								seenPartNames.add(additionalPartName);
+							}
 						}
 					}
 				}

app/exec/extension/_lib/vsix-writer.ts

@@ -150,6 +150,7 @@ export class VsixWriter {
 		let vsix = new zip();
 
 		let builderPromises: Promise<void>[] = [];
+		let seenPartNames = new Set();
 		this.manifestBuilders.forEach(builder => {
 			// Avoid the error EMFILE: too many open files
 			const addPackageFilesBatch = (
@@ -184,18 +185,28 @@ export class VsixWriter {
 						if (!builder.files[path].content) {
 							let readFilePromise = promisify(readFile)(path).then(result => {
 								vsix.file(itemName, result);
+								seenPartNames.add(itemName);
 								if ((builder.files[path] as any)._additionalPackagePaths) {
 									for (const p of (builder.files[path] as any)._additionalPackagePaths) {
-										vsix.file(p, result);
+										let additionalItemName = toZipItemName(p);
+										if (!seenPartNames.has(additionalItemName)) {
+											vsix.file(additionalItemName, result);
+											seenPartNames.add(additionalItemName);
+										}
 									}
 								}
 							});
 							readFilePromises.push(readFilePromise);
 						} else {
 							vsix.file(itemName, builder.files[path].content);
+							seenPartNames.add(itemName);
 							if ((builder.files[path] as any)._additionalPackagePaths) {
 								for (const p of (builder.files[path] as any)._additionalPackagePaths) {
-									vsix.file(p, builder.files[path].content);
+									let additionalItemName = toZipItemName(p);
+									if (!seenPartNames.has(additionalItemName)) {
+										vsix.file(additionalItemName, builder.files[path].content);
+										seenPartNames.add(additionalItemName);
+									}
 								}
 							}
 							readFilePromises.push(Promise.resolve<void>(null));