Update Node.js GitHub action to use npm lockfile, switch to latest Ja…

…eger UI (#6074)

.github/actions/setup-node.js/action.yml

@@ -11,5 +11,5 @@ runs:
     - uses: actions/setup-node@1e60f620b9541d16bece96c5465dc8ee9832be0b # v4.0.3
       with:
         node-version: ${{ env.JAEGER_UI_NODE_JS_VERSION }}
-        cache: 'yarn'
-        cache-dependency-path: jaeger-ui/yarn.lock
+        cache: 'npm'
+        cache-dependency-path: jaeger-ui/package-lock.json

.github/workflows/ci-docker-hotrod.yml

@@ -35,6 +35,8 @@ jobs:
       with:
         go-version: 1.23.x
 
+    - uses: ./.github/actions/setup-node.js
+
     - uses: ./.github/actions/setup-branch
 
     - uses: docker/setup-qemu-action@49b3bc8e6bdd4a60e6116a5414239cba5943d3cf # v3.2.0

CONTRIBUTING.md

@@ -15,7 +15,6 @@ We gratefully welcome improvements to documentation as well as to code.
 
 ### Pre-requisites
 * Install [Go](https://golang.org/doc/install) and setup GOPATH and add $GOPATH/bin in PATH
-* Install [Yarn](https://yarnpkg.com/) for running local build with the UI
 
 This library uses Go modules to manage dependencies.
 

scripts/rebuild-ui.sh

@@ -7,7 +7,7 @@ set -euxf -o pipefail
 
 cd jaeger-ui
 
-git fetch --all --tags
+git fetch --all --unshallow --tags
 git log --oneline --decorate=full -n 10 | cat
 
 last_tag=$(git describe --tags --dirty 2>/dev/null)
@@ -32,4 +32,4 @@ if [[ "$last_tag" =~ ^v[0-9]+\.[0-9]+\.[0-9]+$ ]];  then
 fi
 
 # do a regular full build
-yarn install --frozen-lockfile && cd packages/jaeger-ui && yarn build
+npm ci && cd packages/jaeger-ui && npm run build