chore: Faster npm pre-commit (#3206)

# What does this PR do?
Adds npm to pre-commit.yml installation and caches ui
Removes node installation during pre-commit.

<!-- If resolving an issue, uncomment and update the line below -->
<!-- Closes #[issue-number] -->

## Test Plan
<!-- Describe the tests you ran to verify your changes with result
summaries. *Provide clear instructions so the plan can be easily
re-executed.* -->

Signed-off-by: Francisco Javier Arceo <[email protected]>

Author: franciscojavierarceo

.github/workflows/pre-commit.yml

@@ -36,6 +36,17 @@ jobs:
             **/requirements*.txt
             .pre-commit-config.yaml
 
+      - name: Set up Node.js
+        uses: actions/setup-node@39370e3970a6d050c480ffad4ff0ed4d3fdee5af # v4.1.0
+        with:
+          node-version: '20'
+          cache: 'npm'
+          cache-dependency-path: 'llama_stack/ui/'
+
+      - name: Install npm dependencies
+        run: npm ci
+        working-directory: llama_stack/ui
+
       - uses: pre-commit/action@2c7b3805fd2a0fd8c1884dcaebf91fc102a13ecd # v3.0.1
         continue-on-error: true
         env:

.pre-commit-config.yaml

@@ -155,7 +155,7 @@ repos:
         require_serial: true
       - id: ui-eslint
         name: Lint UI code with ESLint
-        entry: bash -c 'cd llama_stack/ui && npm ci && npm run lint -- --fix --quiet'
+        entry: bash -c 'cd llama_stack/ui && npm run lint -- --fix --quiet'
         language: system
         files: ^llama_stack/ui/.*\.(ts|tsx)$
         pass_filenames: false