Context Navigation

← Previous Change
Next Change →

general-template.php

Timestamp:

02/10/2026 10:29:49 PM (5 months ago)

Author:

westonruter

Message:

Code Editor: Switch from Esprima to Espree for JavaScript linting in CodeMirror.

Esprima is no longer maintained, and it does not support the latest JavaScript features in ES11, as Espree does.

New Linter Integration: Introduces src/js/_enqueues/vendor/codemirror/javascript-lint.js using espree for parsing and error reporting, replacing the dependency on jshint and esprima scripts.
Script Modules: Registers espree as a script module and leverages the module_dependencies argument in wp_register_script() to ensure espree is available as a dynamic import.
Editor Settings: Updates wp_get_code_editor_settings() to use ES11 (ECMAScript 2020) defaults and synchronizes JSHint settings from .jshintrc for compatibility.
Editable Extensions: Adds .mjs to the list of editable file extensions for plugins and themes.
Deprecations: Marks esprima and jshint script handles as deprecated.
Build Tools: Updates Webpack configuration to bundle espree as a module and use the new local javascript-lint.js.

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

Follow-up to [61587], [61544], [61539], [42547].

Props westonruter, jonsurrell.
See #64562, #61500, #48456, #42850.
Fixes #64558.

File:

: 1 edited

trunk/src/wp-includes/general-template.php (modified) (4 diffs)

Legend:

: Unmodified
: Added
: Removed

trunk/src/wp-includes/general-template.php

-              r61497
+              r61611
                                         wp_enqueue_script( 'htmlhint' );
                                         wp_enqueue_script( 'csslint' );
-                                        wp_enqueue_script( 'jshint' );
                                         if ( ! current_user_can( 'unfiltered_html' ) ) {
                                                 wp_enqueue_script( 'htmlhint-kses' );
 …
                                 case 'text/typescript':
                                 case 'application/typescript':
-                                        wp_enqueue_script( 'jshint' );
                                         wp_enqueue_script( 'jsonlint' );
                                         break;
 …
                 ),
                 'jshint'     => array(
+                        // The following are copied from <https://github.com/WordPress/wordpress-develop/blob/4.8.1/.jshintrc>.
+                        'boss'     => true,
+                        'curly'    => true,
+                        'eqeqeq'   => true,
+                        'eqnull'   => true,
+                        'es3'      => true,
+                        'expr'     => true,
+                        'immed'    => true,
+                        'noarg'    => true,
+                        'nonbsp'   => true,
+                        'onevar'   => true,
+                        'quotmark' => 'single',
+                        'trailing' => true,
+                        'undef'    => true,
+                        'unused'   => true,
+                        'browser'  => true,
+                        'globals'  => array(
+                                '_'        => false,
+                                'Backbone' => false,
+                                'jQuery'   => false,
+                                'JSON'     => false,
+                                'wp'       => false,
+                        'esversion' => 11,
+                        'module'    => str_ends_with( $args['file'] ?? '', '.mjs' ),
+                        // The following JSHint *linting rule* options are copied from
+                        // <https://github.com/WordPress/wordpress-develop/blob/6.9.0/.jshintrc>.
+                        // Parsing-related options such as `esversion` (and, in other contexts, `es5`, `es3`, `module`, `strict`)
+                        // are honored by the Espree-based integration, but these linting-rule options are not interpreted by Espree
+                        // and are kept only for compatibility/documentation with the original JSHint configuration.
+                        'boss'      => true,
+                        'curly'     => true,
+                        'eqeqeq'    => true,
+                        'eqnull'    => true,
+                        'expr'      => true,
+                        'immed'     => true,
+                        'noarg'     => true,
+                        'nonbsp'    => true,
+                        'quotmark'  => 'single',
+                        'undef'     => true,
+                        'unused'    => true,
+                        'browser'   => true,
+                        'globals'   => array(
+                                '_'                 => false,
+                                'Backbone'          => false,
+                                'jQuery'            => false,
+                                'JSON'              => false,
+                                'wp'                => false,
+                                'export'            => false,
+                                'module'            => false,
+                                'require'           => false,
+                                'WorkerGlobalScope' => false,
+                                'self'              => false,
+                                'OffscreenCanvas'   => false,
+                                'Promise'           => false,
                         ),
                 ),
 …
                                         break;
                                 case 'js':
+                                case 'mjs':
                                         $type = 'text/javascript';
                                         break;

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 61611 for trunk/src/wp-includes/general-template.php

Legend:

trunk/src/wp-includes/general-template.php

Download in other formats: