Enable integration tests

.github/workflows/integration-tests.yml

@@ -18,9 +18,7 @@ on:
           - '24'
 
 jobs:
-  # TODO: Fix CI setup and re-enable
   generate-node-version-matrix:
-    if: false
     name: Generate Node Version Matrix
     runs-on: ubuntu-latest
     outputs:
@@ -41,26 +39,25 @@ jobs:
 
   integration-tests:
     name: Integration Tests (Node ${{ matrix.node-version }})
-    needs: generate-node-version-matrix
+    needs: [generate-node-version-matrix]
     runs-on: ubuntu-latest
 
     strategy:
       fail-fast: false
       matrix:
-        node-version: ${{ fromJson(needs.generate-node-version-matrix.outputs.node-versions) }}
+        node-version: ${{ fromJSON(needs.generate-node-version-matrix.outputs.node-versions) }}
 
     steps:
       - name: Checkout code
         uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
 
       - name: Set up Node.js
-        uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
+        uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
         with:
           node-version: ${{ matrix.node-version }}
-          package-manager-cache: false
 
       - name: Install dependencies
-        run: npm install
+        run: npm ci
 
       - name: Build
         run: npm run build || true
@@ -73,8 +70,10 @@ jobs:
 
       - name: Run integration tests
         run: npm run test:integration
+        timeout-minutes: 5
         env:
           HARPER_INTEGRATION_TEST_LOG_DIR: /tmp/harper-test-logs
+          FORCE_COLOR: '1'
 
       - name: Upload Harper logs on failure
         if: failure()

.npmrc

@@ -1,2 +1 @@
-save-exact=true
-package-lock=false
+save-exact=true

.nvmrc

@@ -0,0 +1 @@
+22.22.2

AGENTS.md

@@ -3,17 +3,20 @@
 Review the `README.md` and `CONTRIBUTING.md` for all relevant repository information.
 
 ## Development Tips
+
 - Use `npm install` to install dependencies
 - Use `npm run build` to build the project files
 - Do not edit files in `dist/`; it is compiled output and gitignored.
 - Do not run `npm version` or `npm publish`; these commands are for humans only.
 - The `.cts` extension is intentional and load-order-sensitive. Do not change file extensions in `src/`.
 
 ## Code Style
+
 - Use Prettier for formatting: `npm run format:fix`
 - `src/plugin.ts` is ESM. `src/withHarper.cts` and `src/CacheHandler.cts` are CommonJS (required by Next.js config resolution). Keep them that way.
 
 ## Testing Tips
+
 - Use `npm link` in this directory and `npm link @harperfast/nextjs` in other project directories to test out changes locally
 - Run `npm run install:fixtures` before running tests for the first time, and again after changing any fixture's `package.json`.
 - Run `npm run test:integration` to run all tests, or `npm run test:integration -- integrationTests/next-15.pw.ts` for a single file.

integrationTests/fixture.ts

@@ -4,6 +4,7 @@ import {
 	createHarperContext,
 	setupHarperWithFixture,
 	teardownHarper,
+	HarperStartupError,
 	type HarperContext,
 } from '@harperfast/integration-testing';
 import { test as base, expect } from '@playwright/test';
@@ -22,20 +23,30 @@ export function makeHarperFixture(fixtureName: string) {
 		const fixturePath = join(import.meta.dirname, '..', 'fixtures', fixtureName);
 		const ctx = createHarperContext(fixtureName);
 
-		const started = await setupHarperWithFixture(ctx, fixturePath, {
-			harperBinPath: getHarperBinPath(),
-			startupTimeoutMs: STARTUP_TIMEOUT_MS,
-			config: {
-				logging: {
-					stdStreams: true,
+		let started;
+		try {
+			started = await setupHarperWithFixture(ctx, fixturePath, {
+				harperBinPath: getHarperBinPath(),
+				startupTimeoutMs: STARTUP_TIMEOUT_MS,
+				config: {
+					logging: {
+						stdStreams: true,
+					},
+					applications: {
+						lockdown: 'none',
+						moduleLoader: 'native',
+						dependencyLoader: 'native',
+					},
 				},
-				applications: {
-					lockdown: 'none',
-					moduleLoader: 'native',
-					dependencyLoader: 'native',
-				},
-			},
-		});
+			});
+		} catch (error) {
+			if (error instanceof HarperStartupError) {
+				console.error(`[${fixtureName}] Harper failed to start`);
+				if (error.stdout) console.error(`[${fixtureName}] stdout:\n${error.stdout}`);
+				if (error.stderr) console.error(`[${fixtureName}] stderr:\n${error.stderr}`);
+			}
+			throw error;
+		}
 
 		await use(started.harper);
 

integrationTests/playwright.config.ts

@@ -13,9 +13,9 @@ export default defineConfig({
 	forbidOnly: !!process.env.CI,
 
 	// Retry on CI to smooth over flakiness in Next.js startup timing
-	retries: process.env.CI ? 1 : 0,
+	// retries: process.env.CI ? 1 : 0,
 
-	reporter: process.env.CI ? 'github' : 'list',
+	reporter: process.env.CI ? [['list'], ['github']] : 'list',
 
 	use: {
 		// No baseURL — each test uses harper.httpURL from the fixture directly,