Add all accepted ILIAS 12 NPM dependency PRs into package.json

[lscharmer]

Hi,
this PR merges all accepted ILIAS 12 NPM dependencies.

The following PRs are included:

For root package.json:

• Add @yaireo/tagify as npm Dependency for ILIAS 12 #10663
• Add dropzone as npm Dependency for ILIAS 12 #10664
• Add @rollup/plugin-node-resolve as npm Dependency for ILIAS 12 #10665
• Add @rollup/plugin-terser as npm Dependency for ILIAS 12 #10666
• Add eslint as npm Dependency for ILIAS 12 #10667
• Add eslint-config-airbnb-base as npm Dependency for ILIAS 12 #10668
• Add eslint-plugin-import as npm Dependency for ILIAS 12 #10669
• Add jsdom as npm Dependency for ILIAS 12 #10670
• Add rollup as npm Dependency for ILIAS 12 #10671
• Add sass as npm Dependency for ILIAS 12 #10672
• NPM: Add webui-popover for ILIAS 12 #10633
• NPM: Add mathjax for ILIAS 12 #10624
• NPM: Add linkifyjs and linkify-element for ILIAS 12 #10569
• NPM: Add jqueryfor ILIAS 12 #10567
• NPM: Add tiny as dependency for ILIAS 12 #10858

For components/ILIAS/Chatroom/chat/package.json:

• Chatroom: Add winston #10566
• Chatroom: Add uuid #10565
• Chatroom: Add socket.io and socket.io-client #10564
• Chatroom: Add mariadb #10563
• Chatroom: Add express #10559

Missing dependencies

Chart.js

Used for the Chart UI Component.

@thibsy can you look into this?

I removed the corresponding code in order for the setup process to work for now.

JSTree

Used in the ilExplorerBaseGUI (part of the UIComponent).

@thibsy can you look into this?

I removed the corresponding code in order for the setup process to work for now.

Things to note

eslint + eslint-config-airbnb-base

This is the same issue as mentioned in: #9086
The package eslint-config-airbnb-base requires an out of date version of eslint (https://eslint.org/version-support/).

I opt for dropping it for now.

@thibsy I hope this is ok for you.

rollup on Mac

As mentioned & fixed by @thibsy in #9361, some packages (e.g. for mac) where not included with the normal npm install to create the package-lock.json.

Even though I created the package-lock.json the same way as last year, this time the packages are already included AFAIK:

# This PRs and output after PR #9361, are the same
$ grep rollup-darwin package-lock.json
    "node_modules/@rollup/rollup-darwin-arm64": {
      "resolved": "https://registry.npmjs.org/@rollup/rollup-darwin-arm64/-/rollup-darwin-arm64-4.53.3.tgz",
    "node_modules/@rollup/rollup-darwin-x64": {
      "resolved": "https://registry.npmjs.org/@rollup/rollup-darwin-x64/-/rollup-darwin-x64-4.53.3.tgz",
	"@rollup/rollup-darwin-arm64": "4.53.3",
	"@rollup/rollup-darwin-x64": "4.53.3",

vs. package-lock.json before PR #9361:

$ grep rollup-darwin package-lock.json
        "@rollup/rollup-darwin-arm64": "4.38.0",
        "@rollup/rollup-darwin-x64": "4.38.0",

@thibsy it would be great if you could verify this, I will test this too as soon as I have a Mac available.

Fixing Unit Tests

The change made in: components/ILIAS/UI/tests/Client/Table/Presentation/presentation.table.test.js fixes an endless loop error. With this change all JS Unit Tests run successfully.

Summary

In comparison to ILIAS 11:

New dependencies: None

Removed dependencies:

• Chart.js
• JSTree
• node-schedule Already replaced in PR: Chatroom: Update for new dependency list #9807
• eslint-config-airbnb-base Temporally removed until issues are resolved

[thibsy]

Hi @lscharmer,

Thx a lot for providing the integration of the dependency PR's and for your detailed summary. Much appreciated.

• chart.js: @tfamula requested this last time (#8646), maybe @schmitz-ilias or @alex40724 can take over on behalf of Leifos? This would be much appreciated, I am not very familiar with the library in question, but we ultimately need it for the charts inside UI framework.
• jstree: @alex40724 requested this last time (#10215), maybe he could do so again. But sine this is legacy UI, it could also be shepherded by someone else.
• eslint-config-airbnb-base: this is my mistake, I should have requested eslint with v8. Since having no code-style is so much worse than using a deprecated development dependency, I would ask you to downgrade eslint and incorporate the eslint-config-airbnb-base package again. I also requested the same procedure last time, which got approved by JF (#9840).
• rollup: works on my machine (macOS 26.3.1) – I think we're good this time =). Thx for inspecting.

Other code changes to the UI framework LGTM. Funny that this test case worked before?

Kind regards,
@thibsy

[thibsy]

@ILIAS-eLearning/technical-board let me know if I should go to JF with this again.

[marvimarv]

@ILIAS-eLearning/technical-board let me know if I should go to JF with this again.

@thibsy: There is no need to go through the JF again, the dependency is accepted.