🤖 ci: speed up integration suite

- Run heavy runtime-only suites only for local runtime to keep remote fixtures lightweight.
- Ensure CI uses at least 2 Jest workers so the integration suite stays under the 10-minute job timeout.

Author: ammar-agent

jest.config.js

@@ -21,7 +21,9 @@ module.exports = {
   // Transform ESM modules to CommonJS for Jest
   transformIgnorePatterns: ["node_modules/(?!(@orpc|shiki|json-schema-typed|rou3)/)"],
   // Run tests in parallel (use 50% of available cores, or 4 minimum)
-  maxWorkers: "50%",
+  // CI runners often have a low core count; "50%" can result in a single Jest worker,
+  // which can push the integration job over its 10-minute timeout.
+  maxWorkers: process.env.CI ? 2 : "50%", 
   // Force exit after tests complete to avoid hanging on lingering handles
   forceExit: true,
   // 10 minute timeout for integration tests, 10s for unit tests

tests/runtime/runtime.test.ts

@@ -76,13 +76,18 @@ describeIntegration("Runtime integration tests", () => {
 
       // DockerRuntime is slower than local/ssh, and the integration job has a hard
       // time budget. Keep the Docker coverage focused on the core Runtime contract.
+      const describeLocalOnly = type === "local" ? describe : describe.skip;
+
       const describeNonDocker = type === "docker" ? describe.skip : describe;
 
       // Running these runtime contract tests with test.concurrent can easily overwhelm
       // the docker/ssh fixtures in CI and cause the overall integration job to hit its
       // 10-minute timeout. Keep runtime tests deterministic by running them sequentially
       // for remote runtimes.
       const testForRuntime = type === "local" ? test.concurrent : test;
+      const isRemote = type !== "local";
+      const testLocalOnly = isRemote ? test.skip : testForRuntime;
+      const testDockerOnly = type === "docker" ? testForRuntime : test.skip;
       const createRuntime = (): Runtime =>
         createTestRuntime(
           type,
@@ -121,7 +126,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(result.exitCode).toBe(42);
         });
 
-        testForRuntime("handles stdin input", async () => {
+        testLocalOnly("handles stdin input", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -174,7 +179,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(result.exitCode).toBe(0);
         });
 
-        testForRuntime("handles commands with quotes and special characters", async () => {
+        testLocalOnly("handles commands with quotes and special characters", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -194,7 +199,7 @@ describeIntegration("Runtime integration tests", () => {
 
           expect(result.stdout.trim()).toContain(workspace.path);
         });
-        testForRuntime(
+        testLocalOnly(
           "handles timeout correctly",
           async () => {
             const runtime = createRuntime();
@@ -272,7 +277,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(content).toBe("");
         });
 
-        testForRuntime("reads binary data correctly", async () => {
+        testLocalOnly("reads binary data correctly", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -368,7 +373,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(content).toBe("");
         });
 
-        testForRuntime("writes binary data", async () => {
+        testLocalOnly("writes binary data", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -407,7 +412,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(content).toBe(specialContent);
         });
 
-        testForRuntime("preserves symlinks when editing target file", async () => {
+        testDockerOnly("preserves symlinks when editing target file", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -450,7 +455,7 @@ describeIntegration("Runtime integration tests", () => {
           expect(targetContent).toBe("new content");
         });
 
-        testForRuntime("preserves file permissions when editing through symlink", async () => {
+        testDockerOnly("preserves file permissions when editing through symlink", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -547,7 +552,7 @@ describeIntegration("Runtime integration tests", () => {
         });
       });
 
-      describeNonDocker("Edge cases", () => {
+      describeLocalOnly("Edge cases", () => {
         testForRuntime(
           "handles large files efficiently",
           async () => {
@@ -566,7 +571,7 @@ describeIntegration("Runtime integration tests", () => {
           30000
         );
 
-        testForRuntime("handles concurrent operations", async () => {
+        testLocalOnly("handles concurrent operations", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);
 
@@ -784,7 +789,7 @@ describeIntegration("Runtime integration tests", () => {
         });
       });
 
-      describeNonDocker("Error handling", () => {
+      describeLocalOnly("Error handling", () => {
         testForRuntime("handles command not found", async () => {
           const runtime = createRuntime();
           await using workspace = await TestWorkspace.create(runtime, type);