From 20acac3aaccfbd7262c07835c73de265d0598437 Mon Sep 17 00:00:00 2001
From: "Steven R. Loomis" <srl295@gmail.com>
Date: Thu, 30 Mar 2023 13:13:20 -0500
Subject: [PATCH] CLDR-16522 deps add .node-version file @ v18.15.0 (#2823)

- add a .node-version file, defacto format defined at https://github.com/shadowspawn/node-version-usage
- mvn/ant code to use the .node-version file within frontend-maven-plugin, workaround for https://github.com/eirslett/frontend-maven-plugin/issues/1047
- use the nodenv/actions/node-version@v3 action which reads .node-version in relevant places
- turn on webpack's persistent cache, and cache it across builds
- also, move some maven plugin dependencies up to tools/pom.xml
I'm keeping package dependencies (such as mysql) in cldr-apps/pom.xml
---
 .github/workflows/gh-pages.yml       |  4 +--
 .github/workflows/js.yml             | 13 +++++++-
 .github/workflows/maven.yml          |  8 +++++
 .node-version                        |  1 +
 tools/cldr-apps/js/webpack.config.js |  4 +++
 tools/cldr-apps/pom.xml              | 46 +++++++++++++++++++++++-----
 tools/pom.xml                        | 29 ++++++++++++++++++
 7 files changed, 94 insertions(+), 11 deletions(-)
 create mode 100644 .node-version

diff --git a/.github/workflows/gh-pages.yml b/.github/workflows/gh-pages.yml
index 1c625913c0f..cf1f1e6bc97 100644
--- a/.github/workflows/gh-pages.yml
+++ b/.github/workflows/gh-pages.yml
@@ -28,9 +28,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nodetr-
             nodetr-
-      - uses: actions/setup-node@v3
-        with:
-          node-version: '16'
+      - uses: nodenv/actions/node-version@v3 # setup using .node-version
       - name: Run TR archiver
         # Note: will update ToC if out of date
         run: 'cd tools/scripts/tr-archive/ && npm ci && npm run build'
diff --git a/.github/workflows/js.yml b/.github/workflows/js.yml
index 7bf9b33232b..75f936a8bb8 100644
--- a/.github/workflows/js.yml
+++ b/.github/workflows/js.yml
@@ -3,10 +3,12 @@ name: JavaScript Tests
 on:
   push:
     paths:
+      - '.node-version'
       - 'tools/cldr-apps/js/**'
       - '.github/workflows/js.yml'
   pull_request:
     paths:
+      - '.node-version'
       - 'tools/cldr-apps/js/**'
       - '.github/workflows/js.yml'
 
@@ -17,6 +19,15 @@ jobs:
     - uses: actions/checkout@v3
       with:
         lfs: false # not needed for this job, as we don’t currently do a Java build
+    - uses: nodenv/actions/node-version@v3 # setup using .node-version
+    - name: Cache webpack
+      uses: actions/cache@v3
+      with:
+        path: tools/cldr-apps/target/webpack_cache
+        key: ${{ runner.os }}-webpack-${{ hashFiles('tools/cldr-apps/package*') }}-${{ hashFiles('tools/cldr-apps/src/**') }}
+        restore-keys: |
+          ${{ runner.os }}-webpack-
+          webpack-
     - name: Cache npm repository
       uses: actions/cache@v3
       with:
@@ -31,5 +42,5 @@ jobs:
       # stopgap: fail this if it takes too long
       timeout-minutes: 10
       run: (cd tools/cldr-apps/js && npm t)
-    - name: Run Webpack build
+    - name: Run Webpack production build
       run: (cd tools/cldr-apps/js && npm run build)
diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
index 6952305d0c7..d5f2e804444 100644
--- a/.github/workflows/maven.yml
+++ b/.github/workflows/maven.yml
@@ -46,6 +46,14 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
             node-
+      - name: Cache webpack
+        uses: actions/cache@v3
+        with:
+          path: tools/cldr-apps/target/webpack_cache
+          key: ${{ runner.os }}-webpack-${{ hashFiles('tools/cldr-apps/package*') }}-${{ hashFiles('tools/cldr-apps/src/**') }}
+          restore-keys: |
+            ${{ runner.os }}-webpack-
+            webpack-
       - name: Lint GitHub Actions
         run: npx yaml-lint .github/workflows/*.yml
       - name: Build with Maven
diff --git a/.node-version b/.node-version
new file mode 100644
index 00000000000..e048c8ca198
--- /dev/null
+++ b/.node-version
@@ -0,0 +1 @@
+v18.15.0
diff --git a/tools/cldr-apps/js/webpack.config.js b/tools/cldr-apps/js/webpack.config.js
index e2a20fd21fa..7707bdb673e 100644
--- a/tools/cldr-apps/js/webpack.config.js
+++ b/tools/cldr-apps/js/webpack.config.js
@@ -7,6 +7,10 @@ module.exports = (env, argv) => {
   const DEV = (mode === 'development');
   return {
     entry: "./src/index.js",
+    cache: {
+      type: 'filesystem',
+      cacheDirectory: path.resolve(__dirname, '../target/webpack_cache'),
+    },
     output: {
       filename: "bundle.js",
       path: path.resolve(__dirname, "..", "src", "main", "webapp", "dist"),
diff --git a/tools/cldr-apps/pom.xml b/tools/cldr-apps/pom.xml
index 8236e1cb7b0..01eef91b818 100644
--- a/tools/cldr-apps/pom.xml
+++ b/tools/cldr-apps/pom.xml
@@ -152,6 +152,44 @@
 	<build>
 		<finalName>${project.artifactId}</finalName>
 		<plugins>
+			<plugin>
+				<artifactId>maven-antrun-plugin</artifactId>
+				<executions>
+					<execution>
+						<phase>initialize</phase>
+						<configuration>
+							<target>
+								<concat append="no" destfile="${project.basedir}/target/node-version.properties">node.version=</concat>
+								<concat append="yes" destfile="${project.basedir}/target/node-version.properties">
+									<fileset dir="${project.basedir}/../..">
+										<include name=".node-version" />
+									</fileset>
+								</concat>
+							</target>
+						</configuration>
+						<goals>
+							<goal>run</goal>
+						</goals>
+					</execution>
+				</executions>
+			</plugin>
+			<plugin>
+				<groupId>org.codehaus.mojo</groupId>
+				<artifactId>properties-maven-plugin</artifactId>
+				<executions>
+					<execution>
+						<phase>initialize</phase>
+						<goals>
+							<goal>read-project-properties</goal>
+						</goals>
+						<configuration>
+							<files>
+								<file>${project.basedir}/target/node-version.properties</file>
+							</files>
+						</configuration>
+					</execution>
+				</executions>
+			</plugin>
 			<plugin>
 				<artifactId>maven-clean-plugin</artifactId>
 			</plugin>
@@ -166,7 +204,6 @@
 			<plugin>
 				<artifactId>maven-war-plugin</artifactId>
 				<groupId>org.apache.maven.plugins</groupId>
-				<version>3.2.3</version>
 				<configuration>
 					<archive>
 						<manifestEntries>
@@ -181,7 +218,6 @@
 			<plugin>
 				<groupId>io.openliberty.tools</groupId>
 				<artifactId>liberty-maven-plugin</artifactId>
-				<version>3.5.1</version>
 				<configuration>
 					<serverName>cldr</serverName>
 					<copyDependencies>
@@ -196,7 +232,6 @@
 			</plugin>
 			<plugin>
 				<artifactId>maven-resources-plugin</artifactId>
-				<version>2.5</version>
 				<executions>
 				<execution>
 					<id>copy-resources</id>
@@ -219,7 +254,6 @@
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-failsafe-plugin</artifactId>
-				<version>2.22.2</version>
 				<configuration>
 					<systemPropertyVariables>
 						<http.port>${default.http.port}</http.port>
@@ -248,7 +282,6 @@
 			<plugin>
 				<groupId>com.github.eirslett</groupId>
 				<artifactId>frontend-maven-plugin</artifactId>
-				<version>1.12.1</version>
 				<executions>
 					<execution>
 						<id>install node and npm</id>
@@ -284,8 +317,7 @@
 					</execution>
 				</executions>
 				<configuration>
-					<nodeVersion>v16.17.1</nodeVersion>
-					<npmVersion>8.15.0</npmVersion>
+					<nodeVersion>${node.version}</nodeVersion>
 					<!-- run in project sourcedir -->
 					<workingDirectory>${project.basedir}/js</workingDirectory>
 				</configuration>
diff --git a/tools/pom.xml b/tools/pom.xml
index 51baa2dbc4d..6f5d10b1601 100644
--- a/tools/pom.xml
+++ b/tools/pom.xml
@@ -196,6 +196,26 @@
 						<argLine>-Xmx6g -enableassertions</argLine>
 					</configuration>
 				</plugin>
+				<plugin>
+					<groupId>io.openliberty.tools</groupId>
+					<artifactId>liberty-maven-plugin</artifactId>
+					<version>3.5.1</version>
+				</plugin>
+				<plugin>
+					<groupId>com.github.eirslett</groupId>
+					<artifactId>frontend-maven-plugin</artifactId>
+					<version>1.12.1</version>
+				</plugin>
+				<plugin>
+					<groupId>org.apache.maven.plugins</groupId>
+					<artifactId>maven-failsafe-plugin</artifactId>
+					<version>2.22.2</version>
+				</plugin>
+				<plugin>
+					<groupId>org.codehaus.mojo</groupId>
+					<artifactId>properties-maven-plugin</artifactId>
+					<version>1.1.0</version>
+				</plugin>
 				<plugin>
 					<artifactId>maven-jar-plugin</artifactId>
 					<version>3.0.2</version>
@@ -204,6 +224,15 @@
 					<artifactId>maven-install-plugin</artifactId>
 					<version>3.0.1</version>
 				</plugin>
+				<plugin>
+					<artifactId>maven-antrun-plugin</artifactId>
+					<version>3.1.0</version>
+				</plugin>
+				<plugin>
+					<artifactId>maven-war-plugin</artifactId>
+					<groupId>org.apache.maven.plugins</groupId>
+					<version>3.2.3</version>
+				</plugin>
 				<plugin>
 					<artifactId>maven-deploy-plugin</artifactId>
 					<version>2.8.2</version>