Drupal 10 JavaScript dependency plan [#3238507]

Problem/Motivation

Drupal 10 is scheduled to ship sometime in 2022. We should plan what to do with each of our JavaScript dependencies.
CKEditor is not part of this list, it will be handled as part of #3231364: Add CKEditor 5 module to Drupal core

Package	Description	Action	Issue
@babel/core	ECMAScript 5 build	Remove	#3238497: What to do with assets build step?
@babel/preset-env	ECMAScript 5 build	Remove	#3238497: What to do with assets build step?
@babel/register	ECMAScript 5 build	Remove	#3238497: What to do with assets build step?
@drupal/once	Drupal Once	Keep
@popperjs/core	Library for positioning elements	Remove	#3301545: Remove popperjs from Drupal 10
babel-plugin-add-header-comment	ECMAScript 5 build	Remove	#3238497: What to do with assets build step?
Backbone	JavaScript framework used by CKEditor 4, QuickEdit, Contextual Links, Tour, and Toolbar.	Remove	#3145958: [META] Re-evaluate use of Backbone.js in core
chalk	Babel and jQuery UI build tools	Remove	#3266274: Remove chalk as a dependency
chokidar	Babel and PostCSS build tools	Keep
chromedriver	Nightwatch testing	Update	#3306446: Update Nightwatch to 2.4.1/#3306450: Update Chrome container to use newer version
cross-env	Babel and jQuery UI build tools	Remove	#3278415: Remove usages of the JavaScript ES6 build step, the build step itself, and associated dev dependencies
cspell	Spell check	Update	#3306441: Update to the latest cspell version (6.8.1)
css.escape	Internet Explorer 11 Polyfill	Remove	#3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0
dotenv-safe	Nightwatch testing	Keep
es6-promise	Internet Explorer 11 Polyfill	Remove	#3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0
eslint	JavaScript linting	Update	#3265618: Update to eslint 8
eslint-config-airbnb	JavaScript linting	~~Keep~~ (possibly replace)	#3239838: Update core eslint configuration to remove unused React and JSX rules
eslint-config-prettier	JavaScript linting	Update
eslint-plugin-import	JavaScript linting	Update
eslint-plugin-jquery	JavaScript linting	Keep
eslint-plugin-jsx-a11y	JavaScript linting	~~Keep~~ remove	#3239838: Update core eslint configuration to remove unused React and JSX rules
eslint-plugin-prettier	JavaScript linting	Update
eslint-plugin-react	JavaScript linting	~~Update~~ remove	#3239838: Update core eslint configuration to remove unused React and JSX rules
eslint-plugin-react-hooks	JavaScript linting	~~Keep~~ remove	#3239838: Update core eslint configuration to remove unused React and JSX rules
eslint-plugin-yml	YAML linting	Keep
farbtastic	Color picker	Remove	#3306210: Remove Farbtastic library from core
glob	Babel, PostCSS and jQuery UI build tools	Keep
joyride	Required for BC in Drupal 9	Remove from Drupal 10	#3268228: Remove Jquery joyride
jquery		Remove	#3052002: [meta] Replace JQuery with vanilla Javascript in core
jquery-form		Remove	#3189416: Remove jQuery Form dependency from misc/ajax.js
jquery-once	Required for BC in Drupal 9	Remove from Drupal 10	#3268550: Remove deprecated jquery-once
js-cookie	Cookie library	Keep	#3267791: Remove deprecated jquery.cookie shim
loadjs	used for the add_js Ajax command	Keep
minimist	Babel, PostCSS and jQuery UI build tools	Keep
mkdirp	Nightwatch testing	Keep
nightwatch	Nightwatch testing	Update	#3306446: Update Nightwatch to 2.4.1
normalize.css	CSS	Keep
picturefill	Internet Explorer 11 Polyfill	Remove	#3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0
postcss	CSS build tool	Update
postcss-calc	Internet Explorer 11 Polyfill	Remove	#3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0
postcss-header	CSS build tool	Update
postcss-import	CSS build tool	Update
postcss-preset-env	CSS build tool	Keep
postcss-pxtorem	CSS build tool	Update
postcss-url	CSS build tool	Update
prettier	JavaScript linting	Update
raw-loader	CKE5 plugins build dependency	remove	#3319917: Remove raw-loader dependency
shepherd.js	Tour	Update	#3265619: Update Shepherd.js to 9.x
sortablejs	Sorting library	Update
stylelint	CSS linting	Keep
stylelint-checkstyle-formatter	CSS linting	Keep
stylelint-config-standard	CSS linting	Keep
stylelint-order	CSS linting	Update
tabbable	Library for checking tabbable elements	Keep
terser	jQuery UI build tools, needs issue for updating to latest minor release	Update	#3296481: Update terser and terser-webpack-plugin to the latest versions
underscore	This is a dependency of Backbone. Usage outside of that is light.	Remove	#3239796: [meta] Remove dependency on Underscore.js outside of Backbone and mark it internal
Modernizr	Feature detection for JS behaviors	Remove	#3101922: Find replacement for Modernizr touchevent test and deprecate , #3239980: Deprecate Modernizr

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments

Comment #1

22 September 2021 at 11:12

lauriii created an issue. See original summary.

Comment #2

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 28 September 2021 at 13:28

Issue summary:

View changes

Comment #3

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 29 September 2021 at 06:51

Issue summary:

View changes

Added link to Underscore.js issue.

Comment #4

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 29 September 2021 at 12:00

Issue summary:

View changes

Comment #5

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 30 September 2021 at 07:57

Issue summary:

View changes

Comment #6

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 30 September 2021 at 07:59

Issue summary:

View changes

Comment #7

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 17 December 2021 at 21:15

Issue tags:		+Drupal 10, +JavaScript
Parent issue:		» #3118154: [meta] Deprecate dependencies, libraries, modules, and themes that will be removed from Drupal 10 core by 9.4.0-beta1

Comment #8

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 17 December 2021 at 21:29

Thanks, I incorporated the issues from this plan into #3118154: [meta] Deprecate dependencies, libraries, modules, and themes that will be removed from Drupal 10 core by 9.4.0-beta1.

The issues in the list are a mix of the deprecation/decoupling issues for D9 and the actual removals for D10. E.g., are the IE11 polyfills and Modernizr already deprecated, or do we still need to add deprecations for those before we remove them from 10.0.x?

Comment #9

mherchel

he/him

English

Gainesville, FL, US

CreditAttribution: mherchel commented 12 January 2022 at 00:58

Related #3258114: Upgrade Chromedriver.

Comment #10

alexpott

he/they

English

🇪🇺🌍

CreditAttribution: alexpott at Acro Commerce, Thunder commented 25 January 2022 at 18:40

In order to update chalk to the latest version if looks like we have to change quite a bit. If you run yarn upgrade --latest and then the build command you'll get

Error [ERR_REQUIRE_ESM]: require() of ES Module core/node_modules/chalk/source/index.js from core/scripts/css/compile.js not supported.
Instead change the require of index.js in core/scripts/css/compile.js to a dynamic import() which is available in all CommonJS modules.

I'm not sure the extent of the changes necessary but https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c looks quite involved.

Comment #11

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 19 February 2022 at 20:49

Issue summary:

View changes

Comment #12

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 19 February 2022 at 21:06

Issue summary:

View changes

Comment #13

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 22 February 2022 at 17:21

Issue summary:

View changes

Comment #14

effulgentsia CreditAttribution: effulgentsia at Acquia commented 22 February 2022 at 21:18

For jquery-form, I wrote in #3189416-9: Remove jQuery Form dependency from misc/ajax.js that we might want to keep it but as internal only, just like the Backbone decision in #3145958: [META] Re-evaluate use of Backbone.js in core.

Perhaps there are other ones in this issue's table we should do that with as well?

Comment #15

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 4 March 2022 at 06:18

Issue summary:

View changes

Updating to clarify that Joyride and jQuery Once are required for BC in D9 and that we need issues to remove them from 10.0.x HEAD.

Comment #16

mradcliffe

he/him

English

CreditAttribution: mradcliffe at Kosada commented 4 March 2022 at 13:19

Issue summary:

View changes

I created a child issue for #3267791: Remove deprecated jquery.cookie shim.

Comment #17

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 8 March 2022 at 15:44

Issue summary:

View changes

Comment #18

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 8 March 2022 at 15:49

Issue summary:

View changes

Comment #19

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 8 March 2022 at 15:56

Issue summary:

View changes

Comment #20

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 8 March 2022 at 16:01

Issue summary:

View changes

Comment #21

effulgentsia CreditAttribution: effulgentsia at Acquia commented 8 March 2022 at 18:58

The only "Remove" left in the table without an issue link is jquery-once.

Comment #22

effulgentsia CreditAttribution: effulgentsia at Acquia commented 8 March 2022 at 19:00

Also, I think that at this point, "Remove" is the incorrect action for jquery, at least for D10.

Comment #23

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 9 March 2022 at 10:15

Issue summary:

View changes

Opened issue for removing jquery-once.

Comment #24

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and commented 9 March 2022 at 10:16

Issue summary:

View changes

Comment #25

lauriii

he/him

Finnish

Finland

CreditAttribution: lauriii as a volunteer and at Acquia commented 11 March 2022 at 10:34

Should we remove js-cookie as well given that its only usage in core was removed in #3267791: Remove deprecated jquery.cookie shim?

Comment #26

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 11 March 2022 at 10:36

I would say yes, the work to get rid of the need for cookies on the frontend bears fruit :)

Comment #27

xjm

she/her

English

CreditAttribution: xjm as a volunteer commented 14 March 2022 at 19:21

Issue summary:

View changes

Comment #28

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 13 July 2022 at 07:49

Issue summary:

View changes

Comment #29

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 13 July 2022 at 07:49

Issue summary:

View changes

Comment #30

catch

he/him

English

CreditAttribution: catch at Third and Grove commented 15 July 2022 at 01:53

Opened #3296086: Deprecate/remove js-cookie dependency per #25.

Comment #31

kostyashupenko

Russian

Omsk

CreditAttribution: kostyashupenko at Skilld commented 18 July 2022 at 07:24

Issue summary:

View changes

Opened #3296481: Update terser and terser-webpack-plugin to the latest versions

Comment #32

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 26 August 2022 at 18:13

Issue summary:

View changes

Comment #33

quietone CreditAttribution: quietone at PreviousNext commented 31 August 2022 at 08:39

Issue tags:

+Drupal 10 beta should-have

Comment #34

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 7 September 2022 at 05:36

Issue summary:

View changes

Comment #35

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 7 September 2022 at 05:38

Issue summary:

View changes

Comment #36

Spokje

Dutch

CreditAttribution: Spokje as a volunteer commented 7 September 2022 at 05:39

Issue summary:

View changes

Comment #37

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 9 September 2022 at 13:01

Issue summary:

View changes

Comment #38

nod_

French

Lille

CreditAttribution: nod_ at Acquia commented 9 September 2022 at 13:06

Issue summary:

View changes

Comment #39

effulgentsia CreditAttribution: effulgentsia at Acquia commented 11 September 2022 at 16:04

Issue tags:

+Needs issue summary update

We're coming up on the one year anniversary of this issue. So much great progress since then! The title of this issue is "Drupal 10 JavaScript dependency plan", but some of the items seem clearly to be Drupal 11 material at this point, such as #3238501: Remove empty and deprecated IE11 polyfill library stub entries in 11.0.0 which is now explicitly Drupal 11 and #3052002: [meta] Replace JQuery with vanilla Javascript in core which perhaps some of the items can still happen in Drupal 10 but the thing in its entirety is impractical for Drupal 10 at this point.

I think it would be helpful at this point to clone this issue into a "Drupal 11 JavaScript dependency plan", and update the summary of each to be appropriate for each respective version. I think that would help clarify which of these items, if any, are still needed before we tag a Drupal 10 beta, or can still happen during the beta.