Changeset 62052

Timestamp:

03/19/2026 02:48:50 AM (2 days ago)

Author:

desrosj

Message:

Build/Test Tools: Copy block editor files to src after download.

The block editor-related files were previously present in the src directory and subject to version control. However, [61439] removed them from version control and has broken some workflows and testing setups that run the PHPUnit tests from src.

There is work ongoing to restore those files to version control with their respective change history. But until that happens, this updates the download.js script and gutenberg:download Grunt task to run build:gutenberg —dev every time the build asset is downloaded to ensure the files are present in src.

Props mywp459, amykamala, jorbin.
Fixes #64716. See #64393.

Location:

trunk

Files:

• Gruntfile.js (modified) (1 diff)
• package.json (modified) (1 diff)
• tools/gutenberg/utils.js (modified) (1 diff)

trunk/Gruntfile.js

@@ -1608 +1608 @@
             opts: { stdio: 'inherit' }
         }, function( error ) {
-            done( ! error );
+            if ( error ) {
+                done( false );
+                return;
+            }
+            /*
+             * Build block editor files into the src directory every time assets
+             * are downloaded. This prevents failures when running from src
+             * without running `build:dev` after those files were removed from
+             * version control in https://core.trac.wordpress.org/changeset/61438.
+             *
+             * See https://core.trac.wordpress.org/ticket/64393.
+             */
+            grunt.util.spawn( {
+                grunt: true,
+                args: [ 'build:gutenberg', '--dev' ],
+                opts: { stdio: 'inherit' }
+            }, function( buildError ) {
+                done( ! buildError );
+            } );
         } );
     } );

trunk/package.json

@@ -142 +142 @@
         "gutenberg:copy": "node tools/gutenberg/copy.js",
         "gutenberg:verify": "node tools/gutenberg/utils.js",
-        "gutenberg:download": "node tools/gutenberg/download.js",
+        "gutenberg:download": "node tools/gutenberg/download.js && grunt build:gutenberg --dev",
         "vendor:copy": "node tools/vendors/copy-vendors.js",
         "sync-gutenberg-packages": "grunt sync-gutenberg-packages",

trunk/tools/gutenberg/utils.js

@@ -43 +43 @@
 
 /**
- * Trigger a fresh download of the Gutenberg artifact by spawning download.js.
- * Exits the process if the download fails.
+ * Trigger a fresh download of the Gutenberg artifact by spawning download.js,
+ * then run `grunt build:gutenberg --dev` to copy the build to src/.
+ * Exits the process if either step fails.
  */
 function downloadGutenberg() {
-    const result = spawnSync( 'node', [ path.join( __dirname, 'download.js' ) ], { stdio: 'inherit' } );
-    if ( result.status !== 0 ) {
-        process.exit( result.status ?? 1 );
+    const downloadResult = spawnSync( 'node', [ path.join( __dirname, 'download.js' ) ], { stdio: 'inherit' } );
+    if ( downloadResult.status !== 0 ) {
+        process.exit( downloadResult.status ?? 1 );
+    }
+
+    const buildResult = spawnSync( 'grunt', [ 'build:gutenberg', '--dev' ], { stdio: 'inherit', shell: true } );
+    if ( buildResult.status !== 0 ) {
+        process.exit( buildResult.status ?? 1 );
     }
 }