Skip to content

Replace deprecated Uglify with Terser for modern JS compatibility#143

Open
nandini888-hash wants to merge 5 commits intoRocketChat:masterfrom
nandini888-hash:fix/webpack-uglify-conflict
Open

Replace deprecated Uglify with Terser for modern JS compatibility#143
nandini888-hash wants to merge 5 commits intoRocketChat:masterfrom
nandini888-hash:fix/webpack-uglify-conflict

Conversation

@nandini888-hash
Copy link
Copy Markdown

This PR replaces the deprecated Uglify-based minification setup with Terser to ensure compatibility with modern JavaScript (ES6+).

Key updates include:

  1. Removed uglifyjs-webpack-plugin
  2. Added and configured terser-webpack-plugin
  3. Updated Webpack configuration to use Terser for minification

Motivation :
uglifyjs-webpack-plugin is deprecated and does not fully support modern JavaScript features. This was causing build failures and compatibility issues.
Changes Made:

  1. Removed deprecated uglifyjs-webpack-plugin
  2. Installed terser-webpack-plugin
  3. Updated Webpack config:
  4. Replaced Uglify with Terser in optimization.minimizer

Testing

  • Installed dependencies (npm install)
  • Tests pass (npm test)
  • Applied NODE_OPTIONS=--openssl-legacy-provider to fix build issue
  • npm run serve

Disclosure :
Mostly AI-generated

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ nandini888-hash
❌ Nandini Vijay


Nandini Vijay seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants