Changeset 60719 for trunk/src/wp-includes/class-wp-scripts.php

Timestamp:

09/08/2025 11:25:40 AM (2 months ago)

Author:

jonsurrell

Message:

Script Loader: Add sourceURL to inline scripts and styles.

Improve the source locations referenced by developer tooling in supporting browsers. Inline source locations are named like inline:handle-js-after and appear in the developer tools "sources" panel.

This is the second attempt to add sourceURL comments. The first attempt in [60685] was reverted due to an issue with script concatenation that has been addressed.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/9672.

Follow-up to [60685], [60690].

Props jonsurrell, westonruter, wildworks, peterwilsoncc, johnbillion, tobiasbg.
Fixes #63887.

File:

• trunk/src/wp-includes/class-wp-scripts.php (modified) (4 diffs)

trunk/src/wp-includes/class-wp-scripts.php

@@ -221 +221 @@
         if ( ! $output ) {
             return;
+        }
+
+        /*
+         * Do not print a sourceURL comment if concatenation is enabled.
+         *
+         * Extra scripts may be concatenated into a single script.
+         * The line-based sourceURL comments may a concatenated script and
+         * do not make sense when multiple are joined together.
+         */
+        if ( ! $this->do_concat ) {
+            $output .= sprintf(
+                "\n//# sourceURL=%s",
+                rawurlencode( "{$handle}-js-extra" )
+            );
         }
 
@@ -525 +539 @@
         }
 
+        /*
+         * Print sourceURL comment regardless of concatenation.
+         *
+         * Inline scripts prevent scripts from being concatenated, so
+         * sourceURL comments are safe to print for inline scripts.
+         */
+        $data[] = sprintf(
+            '//# sourceURL=%s',
+            rawurlencode( "{$handle}-js-{$position}" )
+        );
+
         return trim( implode( "\n", $data ), "\n" );
     }
@@ -697 +722 @@
         }
 
+        $source_url = rawurlencode( "{$handle}-js-translations" );
+
         $output = <<<JS
 ( function( domain, translations ) {
@@ -703 +730 @@
     wp.i18n.setLocaleData( localeData, domain );
 } )( "{$domain}", {$json_translations} );
+//# sourceURL={$source_url}
 JS;