add missing ref, update to v4 (#224)

Small issue in ui-unit-test job is causing failures across all PRs since
we use pull_request_target instead of pull_request for the trigger in
development.yml which means that actions run against the workflow in
main instead of the workflow files that might have been updated in the
PR.

The last UI PR was merged and brought in this buggy ui-unit-test job.

I also updated the checkout action to v4 since the other non ui jobs are
using v4 instead of v3

Ended up updating package-lock as well since I was getting more failures
than expected, thought maybe that was the issue. Regardless, this PR
passes the checks when I temporarily update the trigger from
pull_request_target to pull_request

Signed-off-by: dalthecow <[email protected]>

Author: DaltheCow

.github/workflows/development.yml

@@ -34,11 +34,6 @@ jobs:
         with:
           node-version: '22'
 
-      - name: Set up Node.js 22
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-
       - name: Install dependencies
         run: npm ci
 
@@ -74,11 +69,6 @@ jobs:
         with:
           node-version: '22'
 
-      - name: Set up Node.js 22
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-
       - name: Install dependencies
         run: npm ci
 
@@ -114,11 +104,6 @@ jobs:
         with:
           node-version: '22'
 
-      - name: Set up Node.js 22
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-
       - name: Install dependencies
         run: npm ci
 
@@ -194,11 +179,6 @@ jobs:
         with:
           node-version: '22'
 
-      - name: Set up Node.js 22
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-
       - name: Install dependencies
         run: npm ci