test: switch to glob-based vitest discovery

Author: benvinegar

package.json

@@ -4,9 +4,9 @@
   "private": true,
   "scripts": {
     "test": "vitest run --config vitest.config.mjs",
-    "test:js": "vitest run --config vitest.config.mjs pi/extensions/heartbeat.test.mjs pi/extensions/memory.test.mjs test/legacy-node-tests.test.mjs test/broker-bridge.integration.test.mjs test/integrity-status-check.test.mjs test/slack-allowed-users-check.test.mjs",
-    "test:shell": "vitest run --config vitest.config.mjs test/shell-scripts.test.mjs test/security-audit.test.mjs",
-    "test:coverage": "vitest run --config vitest.config.mjs --coverage pi/extensions/heartbeat.test.mjs pi/extensions/memory.test.mjs test/legacy-node-tests.test.mjs",
+    "test:js": "vitest run --config vitest.js.config.mjs",
+    "test:shell": "vitest run --config vitest.shell.config.mjs",
+    "test:coverage": "vitest run --config vitest.js.config.mjs --coverage",
     "lint": "npm run lint:js && npm run lint:shell",
     "lint:js": "biome lint",
     "lint:shell": "bash bin/lint-shell.sh",

test/security-audit.test.mjs test/security-audit.shell.test.mjstest/security-audit.test.mjs renamed to test/security-audit.shell.test.mjs

@@ -2,16 +2,8 @@ import { defineConfig } from "vitest/config";
 
 export default defineConfig({
   test: {
-    include: [
-      "pi/extensions/heartbeat.test.mjs",
-      "pi/extensions/memory.test.mjs",
-      "test/legacy-node-tests.test.mjs",
-      "test/broker-bridge.integration.test.mjs",
-      "test/integrity-status-check.test.mjs",
-      "test/slack-allowed-users-check.test.mjs",
-      "test/shell-scripts.test.mjs",
-      "test/security-audit.test.mjs",
-    ],
+    include: ["pi/extensions/**/*.test.mjs", "test/**/*.test.mjs"],
+    exclude: ["pi/extensions/tool-guard.test.mjs"],
     testTimeout: 120_000,
     hookTimeout: 120_000,
     fileParallelism: false,

test/shell-scripts.test.mjs test/shell-scripts.shell.test.mjstest/shell-scripts.test.mjs renamed to test/shell-scripts.shell.test.mjs

@@ -0,0 +1,11 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: {
+    include: ["pi/extensions/**/*.test.mjs", "test/**/*.test.mjs"],
+    exclude: ["test/**/*.shell.test.mjs", "pi/extensions/tool-guard.test.mjs"],
+    testTimeout: 120_000,
+    hookTimeout: 120_000,
+    fileParallelism: false,
+  },
+});

vitest.config.mjs

@@ -0,0 +1,10 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: {
+    include: ["test/**/*.shell.test.mjs"],
+    testTimeout: 120_000,
+    hookTimeout: 120_000,
+    fileParallelism: false,
+  },
+});