Add setup-node step before JS syntax check

The self-hosted runner doesn't have Node.js in PATH. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add JS syntax checking to CI and merge conflict pre-commit hook
2026-07-29 00:15:42 +00:00 · 2026-02-22 14:19:13 -08:00 · 2026-02-22 14:05:35 -08:00
2 changed files with 7 additions and 0 deletions
@@ -46,6 +46,12 @@ jobs:
        lfs: false
    - name: Check BUILD.bazel dependencies
      run: ./scripts/check_build_deps.sh --strict
+    - name: Set up Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: 'lts/*'
+    - name: Check JavaScript syntax
+      run: node --check src/main/go/net/eagle0/admin_server/static/map_editor.js

  test:
    runs-on: [self-hosted, bazel]
@@ -5,6 +5,7 @@ repos:
    rev: v4.3.0
    hooks:
    -   id: check-added-large-files
+    -   id: check-merge-conflict
    -   id: no-commit-to-branch
        args: [--branch, main]
 -   repo: https://github.com/pocc/pre-commit-hooks