Release: remove scripts and dev deps from dist package.json

timmywil · web-flow · commit 7eac932da717 · 2024-02-05T10:36:47.000-05:00
- this became necessary due to the addition of the prepare script - scripts aren't needed and don't work in the dist repo Close gh-5404
diff --git a/build/release/dist.js b/build/release/dist.js
@@ -15,8 +15,7 @@ module.exports = function( Release, files, complete ) {
 	const extras = [
 		"src",
 		"LICENSE.txt",
-		"AUTHORS.txt",
-		"package.json"
+		"AUTHORS.txt"
 	];
 
 	/**
@@ -101,6 +100,19 @@ module.exports = function( Release, files, complete ) {
 		shell.rm( "-f", `${ Release.dir.dist }/src/wrapper.js` );
 		shell.rm( "-f", `${ Release.dir.dist }/src/.eslintrc.json` );
 
+		// Write package.json
+		// Remove scripts and other superfluous properties,
+		// especially the prepare script, which fails on the dist repo
+		const packageJson = Object.assign( {}, pkg );
+		delete packageJson.scripts;
+		delete packageJson.devDependencies;
+		delete packageJson.dependencies;
+		delete packageJson.commitplease;
+		await fs.writeFile(
+			`${ Release.dir.dist }/package.json`,
+			JSON.stringify( packageJson, null, 2 )
+		);
+
 		// Write generated bower file
 		await fs.writeFile( `${ Release.dir.dist }/bower.json`, generateBower() );
 
