ci: align CI npm with the local lock-file generator

Two CI breakages surfaced after the first PR run:

1. Lock-file mismatch. package-lock.json is generated locally by npm
   11 (Node 24); the GitHub runner ships npm 10 with Node 22, which
   refuses to npm ci because cosmiconfig and ts-node are absent in
   shapes the older format expects. Bump npm on the runner before
   install in both ci.yml and publish.yaml, instead of regenerating
   the lock file in an older format and fighting drift on every
   developer install.

2. Lint failures in the new test files. Add an eslint override for
   src/__tests__/** that turns off a small set of rules that conflict
   with vitest patterns (vi.mock between imports, non-null assertions
   after expect-truthy, no-op stub methods), and run prettier over
   ConeRendering.test.tsx for the remaining formatting nits.

Author: sankhesh

.eslintrc

@@ -35,5 +35,16 @@
     "import/export": 0,
     "n/no-callback-literal": 0,
     "@typescript-eslint/no-explicit-any": 0
-  }
+  },
+  "overrides": [
+    {
+      "files": ["src/__tests__/**"],
+      "rules": {
+        "import/first": 0,
+        "no-useless-return": 0,
+        "@typescript-eslint/no-non-null-assertion": 0,
+        "@typescript-eslint/no-non-null-asserted-optional-chain": 0
+      }
+    }
+  ]
 }

.github/workflows/ci.yml

@@ -22,6 +22,10 @@ jobs:
         with:
           node-version: 22
           cache: 'npm'
+      - name: Upgrade npm
+        # The lock file is generated by npm >= 11 locally; bump the runner's
+        # npm so its lock-file expectations match before running npm ci.
+        run: npm install -g npm@latest
       - name: Install dependencies
         run: npm ci
       - name: Test

.github/workflows/publish.yaml

@@ -18,6 +18,8 @@ jobs:
         uses: actions/setup-node@v1
         with:
           node-version: 22
+      - name: Upgrade npm
+        run: npm install -g npm@latest
       - name: Install dependencies
         run: npm ci
       - name: Build

src/__tests__/ConeRendering.test.tsx

@@ -30,8 +30,8 @@
  *   latter proves the former already happened.
  */
 
-import { createRef } from 'react';
 import { cleanup, render } from '@testing-library/react';
+import { createRef } from 'react';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 
 // ---------------------------------------------------------------------------
@@ -119,8 +119,8 @@ import vtkConeSource from '@kitware/vtk.js/Filters/Sources/ConeSource';
 import Algorithm from '../core/Algorithm';
 import GeometryRepresentation from '../core/GeometryRepresentation';
 import View from '../core/View';
-import deletionRegistry from '../utils/DeletionRegistry';
 import { IRepresentation, IView } from '../types';
+import deletionRegistry from '../utils/DeletionRegistry';
 
 // ---------------------------------------------------------------------------
 // Helpers
@@ -186,7 +186,10 @@ describe('Cone rendering pipeline', () => {
     const { repRef } = renderConeScene();
 
     const mapper = repRef.current?.getMapper();
-    expect(mapper, 'GeometryRepresentation should expose a mapper').toBeTruthy();
+    expect(
+      mapper,
+      'GeometryRepresentation should expose a mapper'
+    ).toBeTruthy();
 
     const polyData = mapper!.getInputData();
     expect(