Changeset 61794

Timestamp:

03/03/2026 01:45:29 PM (7 months ago)

Author:

adamsilverstein

Message:

General: VIPS library omit un-minimized and .map files.

Omit these files which offer little value since the VIPS library is primarily inlined WASM. Saves ~30MB total from the build output.

Props swissspidy, westonruter, adamsilverstein, berislav.grgicak, knutsp.
Fixes: #64734.

Location:

trunk

Files:

• src/wp-includes/script-modules.php (modified) (1 diff)
• tests/phpunit/tests/script-modules/wpScriptModules.php (modified) (1 diff)
• tools/gutenberg/copy-gutenberg-build.js (modified) (1 diff)

trunk/src/wp-includes/script-modules.php

@@ -191 +191 @@
                 }
 
+                // VIPS files are always minified — the non-minified versions are not
+                // shipped because they are ~10MB of inlined WASM with no debugging value.
+                if ( str_starts_with( $file_name, 'vips/' ) && ! str_contains( $file_name, '.min.' ) ) {
+                        $file_name = str_replace( '.js', '.min.js', $file_name );
+                }
+
                 $path        = includes_url( "js/dist/script-modules/{$file_name}" );
                 $module_deps = $script_module_data['module_dependencies'] ?? array();

trunk/tests/phpunit/tests/script-modules/wpScriptModules.php

@@ -1905 +1905 @@
 
         /**
+         * Tests that VIPS script modules always use minified file paths.
+         *
+         * Non-minified VIPS files are not shipped because they are ~10MB of
+         * inlined WASM with no debugging value, so the registration should
+         * always point to the .min.js variants.
+         *
+         * @ticket 64734
+         *
+         * @covers ::wp_default_script_modules
+         */
+        public function test_vips_script_modules_always_use_minified_paths() {
+                wp_default_script_modules();
+                wp_enqueue_script_module( '@wordpress/vips/loader' );
+
+                $actual = get_echo( array( wp_script_modules(), 'print_enqueued_script_modules' ) );
+
+                $this->assertStringContainsString( 'vips/loader.min.js', $actual );
+                $this->assertStringNotContainsString( 'vips/loader.js"', $actual );
+        }
+
+        /**
          * Normalizes markup for snapshot.
          *

trunk/tools/gutenberg/copy-gutenberg-build.js

@@ -168 +168 @@
                         copyDirectory( srcPath, destPath, transform, options );
                 } else {
+                        // Skip source map files (.map) — these are not useful in Core
+                        // and the sourceMappingURL references are already stripped from JS files.
+                        if ( /\.map$/.test( entry.name ) ) {
+                                continue;
+                        }
+
+                        // Skip non-minified VIPS files — they are ~10MB of inlined WASM
+                        // with no debugging value over the minified versions.
+                        if (
+                                srcPath.includes( '/vips/' ) &&
+                                /(?<!\.min)\.js$/.test( entry.name )
+                        ) {
+                                continue;
+                        }
+
                         // Skip PHP files if excludePHP is true
                         if ( options.excludePHP && /\.php$/.test( entry.name ) ) {