Remove windows-process-tree types if they get installed

These typings cause duplicate identifier errors on windows tsc builds

build/npm/postinstall.js

@@ -60,4 +60,11 @@ runtime "${runtime}"`;
 
 yarnInstall(`build`); // node modules required for build
 yarnInstall('test/smoke'); // node modules required for smoketest
-yarnInstallBuildDependencies(); // node modules for watching, specific to host node version, not electron
+yarnInstallBuildDependencies(); // node modules for watching, specific to host node version, not electron
+
+// Remove the windows process tree typings as this causes duplicate identifier errors in tsc builds
+const processTreeDts = path.join('node_modules', 'windows-process-tree', 'typings', 'windows-process-tree.d.ts');
+if (fs.existsSync(processTreeDts)) {
+	console.log('Removing windows-process-tree.d.ts');
+	fs.unlinkSync(processTreeDts);
+}

src/tsconfig.strictNullChecks.json

@@ -701,7 +701,6 @@
 		"./vs/code/electron-browser/processExplorer/processExplorerMain.ts"
 	],
 	"exclude": [
-		"./typings/require-monaco.d.ts",
-		"./typings/windows-process-tree.d.ts"
+		"./typings/require-monaco.d.ts"
 	]
 }