Simplify build

Author: youknowriad

package.json

@@ -7,7 +7,7 @@
 		"url": "https://develop.svn.wordpress.org/trunk"
 	},
 	"gutenberg": {
-		"ref": "dbb0e0f6bd414445ad52d3aec967ee0900e9167c"
+		"ref": "c5a9e02338f37d8d7c80570aaaaff261e3f9d198"
 	},
 	"engines": {
 		"node": ">=20.10.0",

tools/gutenberg/build-gutenberg.js

@@ -13,9 +13,6 @@ const { spawn } = require( 'child_process' );
 const fs = require( 'fs' );
 const path = require( 'path' );
 
-// Constants
-const GUTENBERG_BUILD_COMMAND = 'npm run build';
-
 // Paths
 const rootDir = path.resolve( __dirname, '../..' );
 const gutenbergDir = path.join( rootDir, 'gutenberg' );
@@ -97,7 +94,6 @@ async function main() {
 	}
 
 	console.log( '✅ Gutenberg directory found' );
-	console.log( `   Build command: ${ GUTENBERG_BUILD_COMMAND }` );
 
 	// Modify Gutenberg's package.json for Core build
 	console.log( '\n⚙️  Configuring build for WordPress Core...' );
@@ -124,17 +120,20 @@ async function main() {
 		process.exit( 1 );
 	}
 
-	// Parse build command
-	const [ cmd, ...args ] = GUTENBERG_BUILD_COMMAND.split( ' ' );
-
 	// Build Gutenberg
-	console.log( '\n🔨 Building Gutenberg...' );
+	console.log( '\n🔨 Building Gutenberg for WordPress Core...' );
 	console.log( '   (This may take a few minutes)' );
 
 	const startTime = Date.now();
 
 	try {
-		await exec( cmd, args, { cwd: gutenbergDir } );
+		// On Windows, shell mode is used and needs the argument wrapped in quotes
+		// On Unix, arguments are passed directly without shell parsing
+		const baseUrlArg = process.platform === 'win32'
+			? '--base-url="includes_url( \'build\' )"'
+			: '--base-url=includes_url( \'build\' )';
+
+		await exec( 'npm', [ 'run', 'build', '--', baseUrlArg ], { cwd: gutenbergDir } );
 
 		const duration = Math.round( ( Date.now() - startTime ) / 1000 );
 		console.log( `✅ Build completed in ${ duration }s` );

tools/gutenberg/copy-gutenberg-build.js

@@ -253,8 +253,6 @@ function generateScriptModulesPackages() {
 					if ( match ) {
 						// Parse PHP array to JavaScript object
 						const assetData = parsePHPArray( match[1] );
-
-						// Create entries for both minified and non-minified versions
 						assetsMin[ jsPathMin ] = assetData;
 						assetsRegular[ jsPathRegular ] = assetData;
 					}
@@ -680,52 +678,18 @@ function parsePHPArray( phpArrayContent ) {
 function transformPHPContent( content, srcPath, destPath ) {
 	let transformed = content;
 
-	// Replace plugins_url() with includes_url()
-	// Handles patterns like: plugins_url( 'build/...' . $var, dirname( __FILE__ ) )
-	transformed = transformed.replace(
-		/plugins_url\(\s*([^,]+),\s*(?:dirname\(\s*__FILE__\s*\)|__FILE__)\s*\)/g,
-		( match, firstArg ) => {
-			return `includes_url( ${ firstArg.trim() } )`;
-		}
-	);
-
-	// Replace plugin_dir_path( __FILE__ ) with ABSPATH . WPINC . '/build/'
-	transformed = transformed.replace(
-		/plugin_dir_path\(\s*__FILE__\s*\)/g,
-		"ABSPATH . WPINC . '/build/'"
-	);
-
-	// Replace dirname( __FILE__ ) patterns in path construction
+	// Fix boot module asset file path for Core's different directory structure
+	// FROM: __DIR__ . '/../../modules/boot/index.min.asset.php'
+	// TO:   ABSPATH . WPINC . '/js/dist/script-modules/boot/index.min.asset.php'
+	// This is needed because Core copies modules to a different location than the plugin structure
 	transformed = transformed.replace(
-		/dirname\(\s*__FILE__\s*\)/g,
-		"ABSPATH . WPINC . '/build'"
-	);
-
-	// Replace __DIR__ with ABSPATH . WPINC . '/build'
-	transformed = transformed.replace(
-		/__DIR__\s*\.\s*['"]\/\.\.\/(.*?)['"]/g,
-		( match, relativePath ) => {
-			return `ABSPATH . WPINC . '/build/${ relativePath }'`;
-		}
+		/__DIR__\s*\.\s*['"]\/\.\.\/\.\.\/modules\/boot\/index\.min\.asset\.php['"]/g,
+		"ABSPATH . WPINC . '/js/dist/script-modules/boot/index.min.asset.php'"
 	);
 
 	// Special transformations for page-wp-admin.php files
 	if ( destPath.includes( 'page-wp-admin.php' ) ) {
-		// Fix boot module asset file path
-		// FROM: ABSPATH . WPINC . '/build/' . '../../modules/boot/index.min.asset.php'
-		// TO:   ABSPATH . WPINC . '/js/dist/script-modules/boot/index.min.asset.php'
-		transformed = transformed.replace(
-			/ABSPATH\s*\.\s*WPINC\s*\.\s*['"]\/build\/['"]\s*\.\s*['"]\.\.\/\.\.\/modules\/boot\/index\.min\.asset\.php['"]/g,
-			"ABSPATH . WPINC . '/js/dist/script-modules/boot/index.min.asset.php'"
-		);
 
-		// Fix loader.js path - replace plugin_dir_url with data URI for empty module
-		// FROM: plugin_dir_url( __FILE__ ) . 'loader.js'
-		// TO:   'data:text/javascript,' (empty module for dependency registration only)
-		transformed = transformed.replace(
-			/plugin_dir_url\(\s*__FILE__\s*\)\s*\.\s*['"]loader\.js['"]/g,
-			"'data:text/javascript,'"
-		);
 
 		// Fix enqueue condition to also work for direct page files (e.g., fonts.php)
 		// This allows the page to work both via menu (admin.php?page=X) and direct file (X.php)