From 989ba6fcda0f986d5e7c11bea0d20e8802b02453 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Wa=C5=9B?= <jan.was@starburstdata.com>
Date: Tue, 27 Aug 2024 15:09:23 +0200
Subject: [PATCH] Make web-ui module builds reproducible

Use npm clean-install to avoid modifying the package-lock.json file and
ensure reproducible builds, especially in the CI.
---
 .../src/main/resources/webapp-preview/package.json            | 4 ++--
 core/trino-web-ui/src/main/resources/webapp/src/package.json  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/core/trino-web-ui/src/main/resources/webapp-preview/package.json b/core/trino-web-ui/src/main/resources/webapp-preview/package.json
index 41c4320ff517..d5cac2fc35dc 100644
--- a/core/trino-web-ui/src/main/resources/webapp-preview/package.json
+++ b/core/trino-web-ui/src/main/resources/webapp-preview/package.json
@@ -7,11 +7,11 @@
     "dev": "vite",
     "build": "tsc -b && vite build",
     "lint": "eslint . --ext ts,tsx --report-unused-disable-directives --max-warnings 0",
-    "package": "npm install && npm run build",
+    "package": "npm clean-install && npm run build",
     "preview": "vite preview",
     "prettier:format": "prettier --write \"./src/*.{ts,tsx}\"",
     "prettier:check": "prettier --check \"./src/*.{ts,tsx}\"",
-    "check": "npm install && npm run lint && npm run prettier:check"
+    "check": "npm clean-install && npm run lint && npm run prettier:check"
   },
   "dependencies": {
     "@emotion/react": "^11.13.0",
diff --git a/core/trino-web-ui/src/main/resources/webapp/src/package.json b/core/trino-web-ui/src/main/resources/webapp/src/package.json
index 2a78ba44d795..23f79a56a064 100644
--- a/core/trino-web-ui/src/main/resources/webapp/src/package.json
+++ b/core/trino-web-ui/src/main/resources/webapp/src/package.json
@@ -30,8 +30,8 @@
     ]
   },
   "scripts": {
-    "package": "npm install && webpack --config webpack.config.js",
-    "watch": "npm install && webpack --config webpack.config.js --watch",
+    "package": "npm clean-install && webpack --config webpack.config.js",
+    "watch": "npm clean-install && webpack --config webpack.config.js --watch",
     "flow": "flow",
     "check": "flow && prettier --check **/*.js **/*.jsx",
     "lint": "prettier --check **/*.js **/*.jsx"