fix upgraded Vitest warnings

The upgraded test toolchain surfaced warnings for deprecated path-resolution
plugin wiring, a nested module mock, and JSDOM's missing scroll API. The path
warning came from keeping `vite-tsconfig-paths` after Vite added native
`resolve.tsconfigPaths` support, so the plugin was both noisy and redundant.

Use Vite's native `resolve.tsconfigPaths` support in both Vite and Vitest
config, and remove the now-unused `vite-tsconfig-paths` dependency. Move the
job-search API mock to module scope so Vitest's hoisting behavior is explicit,
and provide a minimal `scrollTo` test shim for code paths that expect browser
scrolling APIs.

Author: bgentry

package-lock.json

@@ -65,7 +65,6 @@
     "typescript": "^6.0.3",
     "typescript-eslint": "^8.59.0",
     "vite": "^8.0.10",
-    "vite-tsconfig-paths": "^6.1.1",
     "vitest": "^4.1.5"
   },
   "overrides": {

package.json

@@ -10,6 +10,47 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 
 import { Filter, FilterTypeId, JobSearch } from "./JobSearch";
 
+vi.mock("./api", () => ({
+  fetchSuggestions: async (
+    filterTypeId: string,
+    query: string,
+    selectedValues: string[],
+  ) => {
+    if (filterTypeId === "kind") {
+      const mockKinds = [
+        "batch",
+        "stream",
+        "scheduled",
+        "one-time",
+        "recurring",
+        "Chaos",
+        "AITrainingBatch",
+        "UtilizeNewModel",
+      ];
+      return mockKinds
+        .filter((kind) => kind.toLowerCase().includes(query.toLowerCase()))
+        .filter((kind) => !selectedValues.includes(kind));
+    } else if (filterTypeId === "queue") {
+      const mockQueues = [
+        "default",
+        "high-priority",
+        "low-priority",
+        "batch",
+        "realtime",
+      ];
+      return mockQueues
+        .filter((queue) => queue.includes(query.toLowerCase()))
+        .filter((queue) => !selectedValues.includes(queue));
+    } else if (filterTypeId === "priority") {
+      const priorities = ["1", "2", "3", "4"];
+      return priorities
+        .filter((priority) => priority.includes(query))
+        .filter((priority) => !selectedValues.includes(priority));
+    }
+    return [];
+  },
+}));
+
 // Add type declarations for test functions
 declare module "vitest" {
   interface Assertion<T> {
@@ -20,46 +61,6 @@ declare module "vitest" {
 describe("JobSearch", () => {
   beforeEach(() => {
     vi.clearAllMocks();
-    vi.mock("./api", () => ({
-      fetchSuggestions: async (
-        filterTypeId: string,
-        query: string,
-        selectedValues: string[],
-      ) => {
-        if (filterTypeId === "kind") {
-          const mockKinds = [
-            "batch",
-            "stream",
-            "scheduled",
-            "one-time",
-            "recurring",
-            "Chaos",
-            "AITrainingBatch",
-            "UtilizeNewModel",
-          ];
-          return mockKinds
-            .filter((kind) => kind.toLowerCase().includes(query.toLowerCase()))
-            .filter((kind) => !selectedValues.includes(kind));
-        } else if (filterTypeId === "queue") {
-          const mockQueues = [
-            "default",
-            "high-priority",
-            "low-priority",
-            "batch",
-            "realtime",
-          ];
-          return mockQueues
-            .filter((queue) => queue.includes(query.toLowerCase()))
-            .filter((queue) => !selectedValues.includes(queue));
-        } else if (filterTypeId === "priority") {
-          const priorities = ["1", "2", "3", "4"];
-          return priorities
-            .filter((priority) => priority.includes(query))
-            .filter((priority) => !selectedValues.includes(priority));
-        }
-        return [];
-      },
-    }));
   });
 
   describe("Basic Rendering & Props", () => {

src/components/job-search/JobSearch.test.tsx

@@ -15,6 +15,13 @@ class ResizeObserverMock {
 
 global.ResizeObserver = ResizeObserverMock;
 
+// JSDOM does not implement scrolling APIs used by router and focus helpers.
+Object.defineProperty(window, "scrollTo", {
+  configurable: true,
+  value: () => {},
+  writable: true,
+});
+
 // Headless UI relies on the Web Animations API for transitions. JSDOM doesn't
 // implement it, and Headless UI will polyfill with warnings (and may behave
 // differently across versions). Provide a minimal stable polyfill.

src/test/setup.ts

@@ -3,7 +3,6 @@ import { tanstackRouter } from "@tanstack/router-plugin/vite";
 import react from "@vitejs/plugin-react-swc";
 import path from "node:path";
 import { defineConfig } from "vite";
-import tsconfigPaths from "vite-tsconfig-paths";
 
 const dagreCjsPath = path.resolve(
   process.cwd(),
@@ -49,11 +48,11 @@ export default defineConfig({
     tanstackRouter({
       routeFileIgnorePattern: ".(const|schema|test).(ts|tsx)",
     }),
-    tsconfigPaths(),
   ],
   resolve: {
     alias: {
       "@dagrejs/dagre": dagreCjsPath,
     },
+    tsconfigPaths: true,
   },
 });

vite.config.ts

@@ -1,8 +1,9 @@
-import tsconfigPaths from "vite-tsconfig-paths";
 import { defineConfig } from "vitest/config";
 
 export default defineConfig({
-  plugins: [tsconfigPaths()],
+  resolve: {
+    tsconfigPaths: true,
+  },
   test: {
     environment: "jsdom",
     globals: true,