Farewell jest (#4607)

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

Author: cte

.github/actions/setup-node-pnpm/action.yml

@@ -27,11 +27,21 @@ runs:
           uses: pnpm/action-setup@v4
           with:
               version: ${{ inputs.pnpm-version }}
+        - name: Get pnpm store directory
+          shell: bash
+          run: |
+              echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
+        - name: Setup pnpm cache
+          uses: actions/cache@v4
+          with:
+              path: ${{ env.STORE_PATH }}
+              key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }}
+              restore-keys: |
+                  ${{ runner.os }}-pnpm-store-
         - name: Setup Node.js
           uses: actions/setup-node@v4
           with:
               node-version: ${{ inputs.node-version }}
-              cache: "pnpm"
         - name: Install dependencies
           if: ${{ inputs.skip-install != 'true' }}
           shell: bash

.roo/rules/rules.md

@@ -4,6 +4,8 @@
 
     - Before attempting completion, always make sure that any code changes have test coverage
     - Ensure all tests pass before submitting changes
+    - The vitest framework is used for testing; the `describe`, `test`, `it`, etc functions are defined by default in `tsconfig.json` and therefore don't need to be imported
+    - Tests must be run from the same directory as the `package.json` file that specifies `vitest` in `devDependencies`
 
 2. Lint Rules:
 

apps/web-evals/package.json

@@ -54,6 +54,6 @@
 		"@types/react": "^18.3.23",
 		"@types/react-dom": "^18.3.5",
 		"tailwindcss": "^4",
-		"vitest": "^3.2.1"
+		"vitest": "^3.2.3"
 	}
 }

packages/build/package.json

@@ -19,6 +19,6 @@
 		"@roo-code/config-eslint": "workspace:^",
 		"@roo-code/config-typescript": "workspace:^",
 		"@types/node": "20.x",
-		"vitest": "^3.1.3"
+		"vitest": "^3.2.3"
 	}
 }

packages/cloud/package.json

@@ -21,6 +21,6 @@
 		"@roo-code/config-typescript": "workspace:^",
 		"@types/node": "20.x",
 		"@types/vscode": "^1.84.0",
-		"vitest": "^3.1.3"
+		"vitest": "^3.2.3"
 	}
 }

packages/cloud/src/__mocks__/vscode.ts

@@ -1,5 +1,4 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
-import { vi } from "vitest"
 
 export const window = {
 	showInformationMessage: vi.fn(),

packages/cloud/src/__tests__/RefreshTimer.test.ts

@@ -1,6 +1,6 @@
 // npx vitest run src/__tests__/RefreshTimer.test.ts
 
-import { Mock } from "vitest"
+import type { Mock } from "vitest"
 
 import { RefreshTimer } from "../RefreshTimer"
 

packages/cloud/src/__tests__/ShareService.test.ts

@@ -1,5 +1,6 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
-import { describe, it, expect, beforeEach, vi, type MockedFunction } from "vitest"
+
+import type { MockedFunction } from "vitest"
 import axios from "axios"
 import * as vscode from "vscode"
 

packages/cloud/src/__tests__/TelemetryClient.test.ts

@@ -2,8 +2,6 @@
 
 // npx vitest run src/__tests__/TelemetryClient.test.ts
 
-import { describe, it, expect, beforeEach, afterEach, vi } from "vitest"
-
 import { type TelemetryPropertiesProvider, TelemetryEventName } from "@roo-code/types"
 
 import { TelemetryClient } from "../TelemetryClient"

packages/evals/package.json

@@ -47,6 +47,6 @@
 		"@types/ps-tree": "^1.1.6",
 		"drizzle-kit": "^0.31.1",
 		"tsx": "^4.19.3",
-		"vitest": "^3.2.0"
+		"vitest": "^3.2.3"
 	}
 }