From 84990f977700e99dc48a70006873a1671038b0e8 Mon Sep 17 00:00:00 2001 From: gensokyo-dev Date: Sat, 14 Nov 2026 03:01:32 +0000 Subject: [PATCH] test: remove fixed child-process deadlines from tests 239 literal timeoutMs values on cli.run and processes.run only restated a guess at host speed. On hosted runners with four workers, cliOutput's 3-second value failed while the command still produced correct output. Deadlines that are the behavior under test, derived from an operation budget, or that bound a regression's wait remain. Co-Authored-By: gensokyo-dev <342580529+gensokyo-dev@users.noreply.github.com> Co-Authored-By: Claude Opus 5.5 --- .../McpRegistrationCatalog.test.ts | 1 - ...upClientConfiguration.nativeSyntax.test.ts | 1 - src/browser/history/CaptureJson.test.ts | 22 +++++------- tests/acceptance/appleIpsCli.test.ts | 1 - .../applications/electronBlankPathCli.test.ts | 1 - .../javascriptCallLiteralTransfer.test.ts | 1 - .../javascriptCancellationCli.test.ts | 16 ++++----- .../javascriptCheckpointReserveCli.test.ts | 2 -- ...javascriptChildProcessArgumentsCli.test.ts | 9 +++-- .../javascriptConfigurationReadsCli.test.ts | 2 +- .../javascriptEndpointPayload.test.ts | 1 - .../javascriptEventPayload.test.ts | 3 -- .../javascriptPartialEndpointsCli.test.ts | 1 - .../javascriptWorkerUrlCli.test.ts | 2 +- .../typescriptThisParametersCli.test.ts | 8 ++--- .../artifactComparisonPlist.test.ts | 1 - ...ifactComparisonUnavailableMetadata.test.ts | 1 - .../browserScenarioLaunchFailure.test.ts | 1 - .../browser/webBundleContainmentCli.test.ts | 2 -- ...rowserScenarioReferenceDiagnostics.test.ts | 1 - .../javascriptAsyncChunkBudget.test.ts | 1 - .../javascriptRouteBudgetRecovery.test.ts | 2 -- tests/acceptance/nonBinaryComposition.test.ts | 6 ---- .../pcapCredentialErrorsCliMcp.test.ts | 2 -- .../providers/hopperStartupRecovery.test.ts | 2 -- tests/acceptance/pyinstallerArchive.test.ts | 11 ------ tests/acceptance/setup/dshClientSetup.test.ts | 1 - .../setup/hermesAliasedTableSetup.test.ts | 6 ---- .../setup/hermesAnchoredEntrySetup.test.ts | 1 - .../setup/hermesClientSetup.test.ts | 9 ----- .../setup/launcherAliasesCli.test.ts | 2 -- tests/acceptance/setup/ompClientSetup.test.ts | 2 -- tests/acceptance/setup/piClientSetup.test.ts | 5 --- tests/acceptance/setup/updateAliasCli.test.ts | 1 - .../webArtifactAcquisitionCli.test.ts | 1 - tests/acceptance/webModuleCli.test.ts | 1 - tests/acceptance/webSourceLocationCli.test.ts | 11 ------ .../browser/browserFixtureHttpTargets.test.ts | 20 +++++------ .../browserStorageListenerScope.test.ts | 22 +++++------- .../browser/domVerifierCleanup.test.ts | 1 - .../browser/electronActiveHook.test.ts | 2 +- .../browser/largeBase64Capture.test.ts | 1 - .../cli/ampRegistrationPreservation.test.ts | 2 -- tests/boundary/cli/cliOutput.test.ts | 4 --- tests/boundary/cli/cliSetup.test.ts | 1 - .../cli/clientProfileLifecycle.test.ts | 3 -- tests/boundary/cli/clientProfiles.test.ts | 3 -- .../cli/defaultAmpRegistration.test.ts | 6 ---- tests/boundary/cli/desktopBuildHost.test.ts | 4 --- .../cli/doctorHopperDiscovery.test.ts | 1 - .../cli/doctorRegistrationEnvironment.test.ts | 1 - .../cli/electronFuseCleanupCli.test.ts | 1 - .../boundary/cli/environmentBuildCli.test.ts | 6 ---- .../cli/environmentCompareCli.test.ts | 2 -- .../boundary/cli/environmentExportCli.test.ts | 8 ----- .../cli/environmentExportInputsCli.test.ts | 3 -- .../boundary/cli/environmentServeCli.test.ts | 5 --- .../cli/environmentServeRemediation.test.ts | 1 - .../cli/foreignUniversalMachoCli.test.ts | 32 ++++++++--------- .../cli/ghidraInstallationProperties.test.ts | 1 - tests/boundary/cli/ghidraJavaDoctor.test.ts | 1 - .../boundary/cli/githubActionsExample.test.ts | 1 - .../cli/harBrowserSerializedUrlsCli.test.ts | 1 - .../cli/hopperLaunchFailureCli.test.ts | 1 - .../cli/idaDoctorRegistration.test.ts | 4 --- .../cli/idaRetainedCleanupCli.test.ts | 1 - .../cli/idaStartupAttributionCli.test.ts | 1 - .../javascriptRuntimeWideLoadStates.test.ts | 2 +- tests/boundary/cli/jebCli.test.ts | 1 - tests/boundary/cli/kunaDoctorIdentity.test.ts | 1 - .../boundary/cli/markdownLiteralCells.test.ts | 1 - tests/boundary/cli/mcpAddDiscovery.test.ts | 2 -- .../cli/mcpAddOptionValidation.test.ts | 4 --- tests/boundary/cli/nativeDemangleCli.test.ts | 2 -- .../cli/nativeMachoArm64_32Cli.test.ts | 14 +++++--- .../cli/nativeMachoDylibIdentityCli.test.ts | 31 ++++++++--------- .../cli/nativeMachoEchoedPathCli.test.ts | 24 ++++++------- .../cli/nativeMachoEncryptionCli.test.ts | 4 +-- .../cli/nativeMachoLinkerOptionsCli.test.ts | 12 ++++--- .../cli/nativeMachoMissingUuidCli.test.ts | 4 +-- .../cli/nativeMachoObjectSegmentCli.test.ts | 12 ++++--- .../packageRunnerBoundInstallation.test.ts | 6 ++-- .../cli/packageRunnerRegistration.test.ts | 1 - .../cli/setupProviderReadiness.test.ts | 3 -- .../cli/webScriptSelectionCli.test.ts | 5 +-- .../browserFixtureRequestFailures.test.ts | 1 - .../clientNativeConnectionAdmission.test.ts | 1 - .../filesystem/lintRuleConfiguration.test.ts | 3 +- .../referenceSourceFifoRace.test.ts | 10 +++--- .../filesystem/typedLintRules.test.ts | 1 - .../filesystem/verifierFileSnapshots.test.ts | 10 +++--- .../mcp/javascriptLiteralBudget.test.ts | 1 - .../mcp/javascriptLiteralPayload.test.ts | 1 - .../mcp/wideOccurrenceCompletion.test.ts | 26 ++++++-------- .../process/agentEvaluationCommands.test.ts | 2 -- .../process/androidFixtureBootstrap.test.ts | 10 +++--- .../browserRequestVerifierOwnership.test.ts | 1 - .../process/domCallbackFailures.test.ts | 34 +++++++------------ .../process/electronLaunchRejections.test.ts | 16 ++++----- .../process/fixtureAsyncFailures.test.ts | 13 +++---- .../ghidraDecompilerVerifierCleanup.test.ts | 1 - .../process/ghidraVerifierAssertions.test.ts | 20 +++++------ .../process/goVerifierLifetime.test.ts | 1 - .../process/javascriptSourceModes.test.ts | 4 +-- .../numericLimitGuardPermission.test.ts | 2 +- .../process/packageNpmInvocation.test.ts | 4 +-- .../packageSetupTerminalOwnership.test.ts | 1 - .../packageVerifierCommandOwnership.test.ts | 3 +- .../process/providerVerifierCleanup.test.ts | 1 - .../publishedPackageNpmSelection.test.ts | 2 +- .../process/releaseArtifactCanary.test.ts | 2 +- .../process/sourceMapUncHosts.test.ts | 16 ++++----- .../process/spawnArgumentLimit.test.ts | 4 +-- .../process/verifierSourceLoading.test.ts | 16 ++++----- .../process/websiteInitialization.test.ts | 10 +++--- .../windowsExecutableSelection.test.ts | 2 -- .../windowsNativeArtifactVerifier.test.ts | 2 +- .../windowsNativeInspectionWorkspace.test.ts | 2 +- 118 files changed, 204 insertions(+), 429 deletions(-) diff --git a/src/application/McpRegistrationCatalog.test.ts b/src/application/McpRegistrationCatalog.test.ts index 40d0812769..8f1d2f000c 100644 --- a/src/application/McpRegistrationCatalog.test.ts +++ b/src/application/McpRegistrationCatalog.test.ts @@ -296,7 +296,6 @@ it( process.execPath, ["--import", "tsx", "--input-type=module", "--eval", CATALOG_PROGRAM], { - timeoutMs: 50_000, env: { PATH: process.env.PATH, SystemRoot: process.env.SystemRoot, diff --git a/src/application/SetupClientConfiguration.nativeSyntax.test.ts b/src/application/SetupClientConfiguration.nativeSyntax.test.ts index ed4b7aeee1..3043a1955a 100644 --- a/src/application/SetupClientConfiguration.nativeSyntax.test.ts +++ b/src/application/SetupClientConfiguration.nativeSyntax.test.ts @@ -102,7 +102,6 @@ it("repairs equal-launcher Copilot JSONC syntax while strict JSON and standard J process.execPath, ["--import", "tsx", entrypoint], { - timeoutMs: 20_000, env: { PATH: process.env.PATH, SystemRoot: process.env.SystemRoot, diff --git a/src/browser/history/CaptureJson.test.ts b/src/browser/history/CaptureJson.test.ts index 239ba86fc9..a42e494dbb 100644 --- a/src/browser/history/CaptureJson.test.ts +++ b/src/browser/history/CaptureJson.test.ts @@ -21,19 +21,15 @@ it("retains escape-heavy strings within a heap near their decoded size", async ( if (!Array.isArray(parsed) || parsed.length !== 463 || parsed.some((value) => value !== body)) process.exit(3); `; - const child = await processes.run( - process.execPath, - [ - "--max-old-space-size=48", - "--max-semi-space-size=8", - "--import", - "tsx", - "--input-type=module", - "--eval", - program, - ], - { timeoutMs: 30_000 }, - ); + const child = await processes.run(process.execPath, [ + "--max-old-space-size=48", + "--max-semi-space-size=8", + "--import", + "tsx", + "--input-type=module", + "--eval", + program, + ]); expect(child.exitCode, child.stderr).toBe(0); }); diff --git a/tests/acceptance/appleIpsCli.test.ts b/tests/acceptance/appleIpsCli.test.ts index 6d6f44bac2..648ba7041f 100644 --- a/tests/acceptance/appleIpsCli.test.ts +++ b/tests/acceptance/appleIpsCli.test.ts @@ -29,7 +29,6 @@ cliTest( "--json", ], environment: { NODE_OPTIONS: "--max-old-space-size=256" }, - timeoutMs: 30000, }); expect(result.exitCode, result.stderr).toBe(0); const evidence = toolContract("inspect_apple_ips").outputSchema.parse( diff --git a/tests/acceptance/applications/electronBlankPathCli.test.ts b/tests/acceptance/applications/electronBlankPathCli.test.ts index a485add76c..fc4ceeb235 100644 --- a/tests/acceptance/applications/electronBlankPathCli.test.ts +++ b/tests/acceptance/applications/electronBlankPathCli.test.ts @@ -16,7 +16,6 @@ cliTest.for(["", " \t "])( "--json", ], environment: { REA_LOG_LEVEL: "silent" }, - timeoutMs: 15000, }); expect(response.exitCode).toBe(1); expect(response.json).toMatchObject({ diff --git a/tests/acceptance/applications/javascriptCallLiteralTransfer.test.ts b/tests/acceptance/applications/javascriptCallLiteralTransfer.test.ts index 7af4447cba..3b553f0ae3 100644 --- a/tests/acceptance/applications/javascriptCallLiteralTransfer.test.ts +++ b/tests/acceptance/applications/javascriptCallLiteralTransfer.test.ts @@ -30,7 +30,6 @@ cliTest( bundlePath, "--json", ], - timeoutMs: 60_000, }); expect(response.exitCode).toBe(0); const bundle = evidenceBundleSchema.parse( diff --git a/tests/acceptance/applications/javascriptCancellationCli.test.ts b/tests/acceptance/applications/javascriptCancellationCli.test.ts index 0e7ce3c5ab..4b416ca9f7 100644 --- a/tests/acceptance/applications/javascriptCancellationCli.test.ts +++ b/tests/acceptance/applications/javascriptCancellationCli.test.ts @@ -9,16 +9,12 @@ processTest( "returns a nonzero exit for a streamed JSON analysis failure", async ({ processes }) => { const root = await createTestTempDirectory("rea-cli-js-missing-"); - const result = await processes.run( - process.execPath, - [ - "scripts/rea.mjs", - "analyze-javascript-application", - join(root, "missing"), - "--json", - ], - { timeoutMs: 20_000 }, - ); + const result = await processes.run(process.execPath, [ + "scripts/rea.mjs", + "analyze-javascript-application", + join(root, "missing"), + "--json", + ]); expect(result.exitCode).toBe(1); expect(JSON.parse(result.stdout)).toMatchObject({ code: "invalid_request", diff --git a/tests/acceptance/applications/javascriptCheckpointReserveCli.test.ts b/tests/acceptance/applications/javascriptCheckpointReserveCli.test.ts index eb85110b6e..77d86d8d9f 100644 --- a/tests/acceptance/applications/javascriptCheckpointReserveCli.test.ts +++ b/tests/acceptance/applications/javascriptCheckpointReserveCli.test.ts @@ -84,7 +84,6 @@ cliTest.skipIf(process.versions.bun !== undefined).for([ environment: { NODE_OPTIONS: `--max-old-space-size=${String(heapMb)} --max-semi-space-size=8`, }, - timeoutMs: 60_000, }); expect(response.exitCode).toBe(0); const result = javascriptApplicationAnalysisResultSchema.parse( @@ -201,7 +200,6 @@ cliTest.skipIf(process.versions.bun !== undefined)( environment: { NODE_OPTIONS: "--max-old-space-size=640 --max-semi-space-size=8", }, - timeoutMs: 75_000, }); expect(response.stderr).not.toMatch(/FATAL ERROR:.*heap/iu); expect( diff --git a/tests/acceptance/applications/javascriptChildProcessArgumentsCli.test.ts b/tests/acceptance/applications/javascriptChildProcessArgumentsCli.test.ts index 84bfca0304..5714fbe564 100644 --- a/tests/acceptance/applications/javascriptChildProcessArgumentsCli.test.ts +++ b/tests/acceptance/applications/javascriptChildProcessArgumentsCli.test.ts @@ -32,11 +32,10 @@ cliTest.for(["CLI", "stdio MCP"] as const)( ["a", "b"], [], ]) { - const control = await processes.run( - process.execPath, - [join(root, "child.cjs"), ...values], - { timeoutMs: 5_000 }, - ); + const control = await processes.run(process.execPath, [ + join(root, "child.cjs"), + ...values, + ]); expect(control.exitCode, control.stderr).toBe(0); counts.push(Number(control.stdout)); } diff --git a/tests/acceptance/applications/javascriptConfigurationReadsCli.test.ts b/tests/acceptance/applications/javascriptConfigurationReadsCli.test.ts index db7af4df5c..021964670d 100644 --- a/tests/acceptance/applications/javascriptConfigurationReadsCli.test.ts +++ b/tests/acceptance/applications/javascriptConfigurationReadsCli.test.ts @@ -27,7 +27,7 @@ cliTest.for(["CLI", "stdio MCP"] as const)( const control = await processes.run( process.execPath, [join(root, "entry.mjs"), "abc", " mode "], - { env: { REA_ENDPOINT: "endpoint" }, timeoutMs: 15_000 }, + { env: { REA_ENDPOINT: "endpoint" } }, ); expect(control.exitCode, control.stderr).toBe(0); expect(JSON.parse(control.stdout)).toEqual([ diff --git a/tests/acceptance/applications/javascriptEndpointPayload.test.ts b/tests/acceptance/applications/javascriptEndpointPayload.test.ts index 9ade67a728..447ccc48a9 100644 --- a/tests/acceptance/applications/javascriptEndpointPayload.test.ts +++ b/tests/acceptance/applications/javascriptEndpointPayload.test.ts @@ -46,7 +46,6 @@ cliTest( "directory", "--json", ], - timeoutMs: 60_000, }); expect(cliResponse.exitCode).toBe(0); const cliEvidence = toolContract( diff --git a/tests/acceptance/applications/javascriptEventPayload.test.ts b/tests/acceptance/applications/javascriptEventPayload.test.ts index 10fef5c363..2390d826b6 100644 --- a/tests/acceptance/applications/javascriptEventPayload.test.ts +++ b/tests/acceptance/applications/javascriptEventPayload.test.ts @@ -35,7 +35,6 @@ cliTest( "directory", "--json", ], - timeoutMs: 60_000, }); expect(cliResponse.exitCode).toBe(0); const cliEvidence = toolContract( @@ -160,7 +159,6 @@ cliTest( "directory", "--json", ], - timeoutMs: 60_000, }); expect(analyzed.exitCode, analyzed.stderr).toBe(0); const application = toolContract( @@ -193,7 +191,6 @@ cliTest( ); const traced = await cli.run({ arguments: ["trace-javascript-semantics", request, "--json"], - timeoutMs: 60_000, }); expect(traced.exitCode, traced.stdout).toBe(0); const trace = javaScriptSemanticTraceResultSchema.parse( diff --git a/tests/acceptance/applications/javascriptPartialEndpointsCli.test.ts b/tests/acceptance/applications/javascriptPartialEndpointsCli.test.ts index 9513e1fd98..51f74cb6d5 100644 --- a/tests/acceptance/applications/javascriptPartialEndpointsCli.test.ts +++ b/tests/acceptance/applications/javascriptPartialEndpointsCli.test.ts @@ -19,7 +19,6 @@ cliTest( const cliResult = await driver.run({ arguments: ["analyze-javascript-application", root, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(cliResult.exitCode, cliResult.stderr).toBe(0); const result = javascriptApplicationAnalysisResultSchema.parse( diff --git a/tests/acceptance/applications/javascriptWorkerUrlCli.test.ts b/tests/acceptance/applications/javascriptWorkerUrlCli.test.ts index a029f003e2..3da8f80d9e 100644 --- a/tests/acceptance/applications/javascriptWorkerUrlCli.test.ts +++ b/tests/acceptance/applications/javascriptWorkerUrlCli.test.ts @@ -591,7 +591,7 @@ cliTest( const result = await analyzeJavaScriptDirectory( "CLI", - { run: (invocation) => cli.run({ ...invocation, timeoutMs: 20_000 }) }, + { run: (invocation) => cli.run({ ...invocation }) }, root, ); expect(result.statistics.parse_failures).toBe(0); diff --git a/tests/acceptance/applications/typescriptThisParametersCli.test.ts b/tests/acceptance/applications/typescriptThisParametersCli.test.ts index a90dddec5c..43cd08c52c 100644 --- a/tests/acceptance/applications/typescriptThisParametersCli.test.ts +++ b/tests/acceptance/applications/typescriptThisParametersCli.test.ts @@ -39,11 +39,9 @@ cliTest.for(["CLI", "stdio MCP"] as const)( "console.log(JSON.stringify([a, b, c, d, first.length, rest.length, mixed.length]));", ].join("\n"), }); - const control = await processes.run( - process.execPath, - [join(root, "entry.mts")], - { timeoutMs: 15_000 }, - ); + const control = await processes.run(process.execPath, [ + join(root, "entry.mts"), + ]); expect(control.exitCode, control.stderr).toBe(0); expect(JSON.parse(control.stdout)).toEqual([ 7, diff --git a/tests/acceptance/artifactComparisonPlist.test.ts b/tests/acceptance/artifactComparisonPlist.test.ts index b21de2aa13..236c9406e6 100644 --- a/tests/acceptance/artifactComparisonPlist.test.ts +++ b/tests/acceptance/artifactComparisonPlist.test.ts @@ -25,7 +25,6 @@ cliTest( const cliResult = await driver.run({ arguments: ["inspect-artifact", leftPath, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(cliResult.exitCode, cliResult.stderr).toBe(0); const cliEvidence = parseEvidence(JSON.parse(cliResult.stdout)); diff --git a/tests/acceptance/artifactComparisonUnavailableMetadata.test.ts b/tests/acceptance/artifactComparisonUnavailableMetadata.test.ts index 1172074866..268df1965d 100644 --- a/tests/acceptance/artifactComparisonUnavailableMetadata.test.ts +++ b/tests/acceptance/artifactComparisonUnavailableMetadata.test.ts @@ -23,7 +23,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--detail", "complete", "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( diff --git a/tests/acceptance/browser/browserScenarioLaunchFailure.test.ts b/tests/acceptance/browser/browserScenarioLaunchFailure.test.ts index 3e220768f7..39f4486931 100644 --- a/tests/acceptance/browser/browserScenarioLaunchFailure.test.ts +++ b/tests/acceptance/browser/browserScenarioLaunchFailure.test.ts @@ -39,7 +39,6 @@ cliTest.skipIf(process.platform === "win32")( "--json", ], environment: { BROWSER_FIXTURE_SECRET: secret }, - timeoutMs: 20_000, }); expect(response.exitCode, response.stdout + response.stderr).toBe(1); expect(response.stdout).not.toContain(secret); diff --git a/tests/acceptance/browser/webBundleContainmentCli.test.ts b/tests/acceptance/browser/webBundleContainmentCli.test.ts index 59d6159072..d09e694f23 100644 --- a/tests/acceptance/browser/webBundleContainmentCli.test.ts +++ b/tests/acceptance/browser/webBundleContainmentCli.test.ts @@ -47,7 +47,6 @@ cliTest( "--total-analysis-timeout-ms", "1", ], - timeoutMs: 30_000, }); expect( interrupted.exitCode, @@ -74,7 +73,6 @@ cliTest( const recovered = await cli.run({ arguments: baseArguments, - timeoutMs: 30_000, }); expect(recovered.exitCode, recovered.stdout + recovered.stderr).toBe(0); expect(recovered.json).toMatchObject({ diff --git a/tests/acceptance/browserScenarioReferenceDiagnostics.test.ts b/tests/acceptance/browserScenarioReferenceDiagnostics.test.ts index ecb2595c84..8b97ba0cbf 100644 --- a/tests/acceptance/browserScenarioReferenceDiagnostics.test.ts +++ b/tests/acceptance/browserScenarioReferenceDiagnostics.test.ts @@ -46,7 +46,6 @@ cliTest.for([ const result = await cli.run({ arguments: ["capture-browser-scenario", path, "--json"], environment: { HOME: home, REA_LOG_LEVEL: "silent" }, - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject(expected); diff --git a/tests/acceptance/javascriptAsyncChunkBudget.test.ts b/tests/acceptance/javascriptAsyncChunkBudget.test.ts index 59b807b156..4e095d3a5d 100644 --- a/tests/acceptance/javascriptAsyncChunkBudget.test.ts +++ b/tests/acceptance/javascriptAsyncChunkBudget.test.ts @@ -30,7 +30,6 @@ cliTest( "30000", "--json", ], - timeoutMs: 55_000, }); // Analysis retains the interrupted file rather than failing the command. expect(response.exitCode, `${response.stdout}\n${response.stderr}`).toBe(0); diff --git a/tests/acceptance/javascriptRouteBudgetRecovery.test.ts b/tests/acceptance/javascriptRouteBudgetRecovery.test.ts index 6d7fe16845..17a67c1960 100644 --- a/tests/acceptance/javascriptRouteBudgetRecovery.test.ts +++ b/tests/acceptance/javascriptRouteBudgetRecovery.test.ts @@ -29,7 +29,6 @@ cliTest( }), "--json", ], - timeoutMs: 60_000, }); expect(unpublished.exitCode, unpublished.stderr).toBe(1); expect(unpublished.json).toMatchObject({ @@ -60,7 +59,6 @@ cliTest( bundle, "--json", ], - timeoutMs: 60_000, }); expect(analyzed.exitCode, analyzed.stderr).toBe(0); const request = (fields?: readonly string[]) => diff --git a/tests/acceptance/nonBinaryComposition.test.ts b/tests/acceptance/nonBinaryComposition.test.ts index 6b9ff1f8de..dbbbd22746 100644 --- a/tests/acceptance/nonBinaryComposition.test.ts +++ b/tests/acceptance/nonBinaryComposition.test.ts @@ -66,7 +66,6 @@ for (const { jarConfigured, setting } of [ const cliResult = await cli.run({ arguments: ["inspect-android-package", apk, "--json"], environment, - timeoutMs: 10_000, }); expect(cliResult.exitCode).toBe(1); expect(cliResult.json).toMatchObject({ code: "capability_unavailable" }); @@ -91,7 +90,6 @@ cliTest.skipIf(process.platform === "win32")( const cliResult = await cli.run({ arguments: ["inspect-android-package", apk, "--json"], environment: { REA_LOG_LEVEL: "silent", ...environment }, - timeoutMs: 10_000, }); expect(cliResult.exitCode).toBe(1); const client = await connect({ REA_LOG_LEVEL: "silent", ...environment }); @@ -146,7 +144,6 @@ cliTest.skipIf(process.platform !== "linux")( const cliResult = await cli.run({ arguments: ["inspect-firmware-regions", path, "--json"], environment, - timeoutMs: 10_000, }); expect(cliResult.exitCode).toBe(1); expect(cliResult.json).toMatchObject({ code: "capability_unavailable" }); @@ -172,7 +169,6 @@ cliTest.skipIf(process.platform !== "linux")( arguments: ["inspect-firmware-regions", "input.bin", "--json"], cwd: root, environment: { REA_LOG_LEVEL: "silent" }, - timeoutMs: 10_000, }); // A relative path must pass input validation and reach the provider, // which reports the missing firmware engine — not a validation error. @@ -199,7 +195,6 @@ cliTest( "--json", ], environment, - timeoutMs: 10000, }); expect(cliResult.exitCode).toBe(1); expect(cliResult.json).toMatchObject({ code: "capability_unavailable" }); @@ -239,7 +234,6 @@ cliTest( "--json", ], environment, - timeoutMs: 10_000, }); const response = await client.callTool({ name: "recover_javascript_sources", diff --git a/tests/acceptance/pcapCredentialErrorsCliMcp.test.ts b/tests/acceptance/pcapCredentialErrorsCliMcp.test.ts index 1d3ea05bc1..7f8980d64a 100644 --- a/tests/acceptance/pcapCredentialErrorsCliMcp.test.ts +++ b/tests/acceptance/pcapCredentialErrorsCliMcp.test.ts @@ -60,7 +60,6 @@ cliTest.skipIf(missingNative)( const success = await cli.run({ arguments: ["inspect-pcap-capture", single, "--json"], environment: environmentFor(root), - timeoutMs: 60_000, }); expect(success.exitCode, success.stderr).toBe(0); expect(success.stdout.includes(credential)).toBe(false); @@ -78,7 +77,6 @@ cliTest.skipIf(missingNative)( const failed = await cli.run({ arguments: ["inspect-pcap-capture", many, "--json"], environment: environmentFor(root), - timeoutMs: 60_000, }); expect(failed.exitCode, failed.stderr).toBe(1); expect(failed.stdout.includes(credential)).toBe(false); diff --git a/tests/acceptance/providers/hopperStartupRecovery.test.ts b/tests/acceptance/providers/hopperStartupRecovery.test.ts index 12414d930e..0aae52cec5 100644 --- a/tests/acceptance/providers/hopperStartupRecovery.test.ts +++ b/tests/acceptance/providers/hopperStartupRecovery.test.ts @@ -132,7 +132,6 @@ test.for(["target", "loader"] as const)( const result = await cli.run({ arguments: ["analyze", target, "--provider", "hopper", "--json"], environment, - timeoutMs: 15_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -223,7 +222,6 @@ test("recovers a denied Hopper launch through the same stdio MCP session and CLI const deniedCli = await cli.run({ arguments: ["analyze", target, "--provider", "hopper", "--json"], environment, - timeoutMs: 15_000, }); expect(deniedCli.exitCode).toBe(1); expect(deniedCli.json).toMatchObject({ diff --git a/tests/acceptance/pyinstallerArchive.test.ts b/tests/acceptance/pyinstallerArchive.test.ts index 7f503df9dc..cd3b69ce39 100644 --- a/tests/acceptance/pyinstallerArchive.test.ts +++ b/tests/acceptance/pyinstallerArchive.test.ts @@ -160,7 +160,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -328,7 +327,6 @@ cliTest( const result = await driver.run({ arguments: ["extract-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const extraction = artifactExtractionResultSchema.parse( @@ -392,7 +390,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -445,7 +442,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -473,7 +469,6 @@ cliTest( const result = await driver.run({ arguments: ["extract-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const extracted = artifactExtractionResultSchema.parse( @@ -536,7 +531,6 @@ cliTest( const inspectResult = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(inspectResult.exitCode, inspectResult.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -610,7 +604,6 @@ cliTest( const extractResult = await driver.run({ arguments: ["extract-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(extractResult.exitCode, extractResult.stderr).toBe(0); const extracted = artifactExtractionResultSchema.parse( @@ -645,7 +638,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -688,7 +680,6 @@ cliTest.for([ const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -752,7 +743,6 @@ cliTest.for([ const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode, result.stderr).toBe(0); const inspection = artifactInspectionResultSchema.parse( @@ -800,7 +790,6 @@ cliTest( const result = await driver.run({ arguments: ["inspect-artifact", path, "--json"], environment: process.env, - timeoutMs: 15_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/acceptance/setup/dshClientSetup.test.ts b/tests/acceptance/setup/dshClientSetup.test.ts index c59b94d711..3a900b5b1f 100644 --- a/tests/acceptance/setup/dshClientSetup.test.ts +++ b/tests/acceptance/setup/dshClientSetup.test.ts @@ -200,7 +200,6 @@ cliTest( arguments: [...args, "--json"], cwd: home, environment, - timeoutMs: 20_000, }); const plan = await run(["setup", "--client", "dsh", "--dry-run"]); expect(plan.exitCode).toBe(0); diff --git a/tests/acceptance/setup/hermesAliasedTableSetup.test.ts b/tests/acceptance/setup/hermesAliasedTableSetup.test.ts index 6b8a913be2..031917a870 100644 --- a/tests/acceptance/setup/hermesAliasedTableSetup.test.ts +++ b/tests/acceptance/setup/hermesAliasedTableSetup.test.ts @@ -50,7 +50,6 @@ for (const version of ["1.1", "1.2"] as const) { HERMES_HOME: profile, HOPPER_LAUNCHER_PATH: process.execPath, }, - timeoutMs: 20_000, }); expect(removed.exitCode).toBe(0); expect(await readFile(config, "utf8")).toBe(original); @@ -84,7 +83,6 @@ for (const version of ["1.1", "1.2"] as const) { GHIDRA_INSTALL_DIR: ghidra, JAVA_HOME: java, }, - timeoutMs: 20_000, }); const setup = [ "setup", @@ -166,7 +164,6 @@ cliTest("Hermes still registers into an explicit table", async ({ cli }) => { HERMES_HOME: profile, HOPPER_LAUNCHER_PATH: process.execPath, }, - timeoutMs: 20_000, }); expect(applied.exitCode).toBe(0); expect(applied.json).toMatchObject({ status: "ready" }); @@ -209,7 +206,6 @@ for (const version of ["1.1", "1.2"] as const) { HERMES_HOME: profile, HOPPER_LAUNCHER_PATH: process.execPath, }, - timeoutMs: 20_000, }); expect(applied.exitCode).toBe(0); const configured = parse(await readFile(config, "utf8"), { version }); @@ -237,7 +233,6 @@ cliTest( HERMES_HOME: profile, HOPPER_LAUNCHER_PATH: process.execPath, }, - timeoutMs: 20_000, }); const [command, ...args] = currentClientRegistrationCommand({ entrypointPath: resolve("scripts/rea.mjs"), @@ -316,7 +311,6 @@ for (const original of [ HERMES_HOME: profile, HOPPER_LAUNCHER_PATH: process.execPath, }, - timeoutMs: 20_000, }); expect(rejected.exitCode).not.toBe(0); expect(rejected.json).toMatchObject({ diff --git a/tests/acceptance/setup/hermesAnchoredEntrySetup.test.ts b/tests/acceptance/setup/hermesAnchoredEntrySetup.test.ts index 92c65b2276..121cebf93b 100644 --- a/tests/acceptance/setup/hermesAnchoredEntrySetup.test.ts +++ b/tests/acceptance/setup/hermesAnchoredEntrySetup.test.ts @@ -23,7 +23,6 @@ const cliLifecycle = (cli: TestCli, home: string, profile: string) => { arguments: arguments_, cwd: home, environment: { HOME: home, USERPROFILE: home, HERMES_HOME: profile }, - timeoutMs: 20_000, }); const setup = [ "setup", diff --git a/tests/acceptance/setup/hermesClientSetup.test.ts b/tests/acceptance/setup/hermesClientSetup.test.ts index 00c45c74d0..c140a32298 100644 --- a/tests/acceptance/setup/hermesClientSetup.test.ts +++ b/tests/acceptance/setup/hermesClientSetup.test.ts @@ -61,7 +61,6 @@ for (const [version, keyStyle] of [ arguments: arguments_, cwd: home, environment: { USERPROFILE: home, HERMES_HOME: profile }, - timeoutMs: 20_000, }); const setup = [ "setup", @@ -141,7 +140,6 @@ cliTest( arguments: arguments_, cwd: home, environment: { USERPROFILE: home, HERMES_HOME: profile }, - timeoutMs: 20_000, }); const setup = ["setup", "--client", "hermes", "--skill=false", "--json"]; const preview = await run([...setup, "--dry-run"]); @@ -223,7 +221,6 @@ cliTest( REA_CLIENT_PROFILE_HOME: home, HERMES_HOME: " ${REA_CLIENT_PROFILE_HOME}/profile ", }, - timeoutMs: 20_000, }); const planned = await run([ "setup", @@ -333,7 +330,6 @@ for (const selection of ["default", "custom", "suffix", "explicit"] as const) { ? root : undefined, }, - timeoutMs: 20_000, }); const plan = await run([ "setup", @@ -468,7 +464,6 @@ for (const selection of [ LOCALAPPDATA: join(home, "AppData", "Local"), HERMES_HOME: root, }, - timeoutMs: 20_000, }); if (selection === "missing") { expect(plan.exitCode).toBe(1); @@ -573,7 +568,6 @@ for (const layout of ["symlinked-default", "escaping-prefix"] as const) { LOCALAPPDATA: join(home, "AppData", "Local"), HERMES_HOME: root, }, - timeoutMs: 20_000, }); expect(plan.exitCode, plan.stdout).toBe(0); expect(plan.json).toMatchObject({ @@ -650,7 +644,6 @@ for (const name of ["missing", "deleted", "ghost", "../escape", "root"]) { arguments: args, cwd: home, environment: { USERPROFILE: home, HERMES_HOME: root }, - timeoutMs: 20_000, }); const rejected = await run([ "setup", @@ -720,7 +713,6 @@ cliTest.skipIf(process.platform === "win32")( HERMES_HOME: "$REA_HERMES_PROFILE", REA_HERMES_PROFILE: override, }, - timeoutMs: 20_000, }); const planned = await run([ "setup", @@ -783,7 +775,6 @@ cliTest( arguments: arguments_, cwd: home, environment: { USERPROFILE: home, HERMES_HOME: override }, - timeoutMs: 20_000, }); const setup = await run(["setup", "--client", "hermes", "--yes", "--json"]); expect(setup.exitCode).not.toBe(0); diff --git a/tests/acceptance/setup/launcherAliasesCli.test.ts b/tests/acceptance/setup/launcherAliasesCli.test.ts index 8270a3b622..8864b3923f 100644 --- a/tests/acceptance/setup/launcherAliasesCli.test.ts +++ b/tests/acceptance/setup/launcherAliasesCli.test.ts @@ -161,7 +161,6 @@ cliTest.skipIf(process.platform === "win32").for([ XDG_CONFIG_HOME: join(home, ".config"), PATH: process.env.PATH ?? "", }, - timeoutMs: 30_000, }); const setup = await run(registered, [ @@ -296,7 +295,6 @@ cliTest.skipIf(process.platform === "win32")( XDG_CONFIG_HOME: join(home, ".config"), PATH: process.env.PATH ?? "", }, - timeoutMs: 30_000, }); const uninstall = await run(["uninstall", "--json"]); expect(uninstall.exitCode, uninstall.stderr).toBe(0); diff --git a/tests/acceptance/setup/ompClientSetup.test.ts b/tests/acceptance/setup/ompClientSetup.test.ts index 1a1ddfa9c2..0cda7950a7 100644 --- a/tests/acceptance/setup/ompClientSetup.test.ts +++ b/tests/acceptance/setup/ompClientSetup.test.ts @@ -34,7 +34,6 @@ for (const [variable, profile] of [ PI_CODING_AGENT_DIR: agent, [variable]: profile, }, - timeoutMs: 20_000, }); for (const option of ["--dry-run", "--yes"]) { const rejected = await run( @@ -101,7 +100,6 @@ cliTest( OMP_PROFILE: " ", PI_PROFILE: "Bad Name", }, - timeoutMs: 20_000, }); expect(result.exitCode).toBe(0); expect(result.json).toMatchObject({ status: "ready" }); diff --git a/tests/acceptance/setup/piClientSetup.test.ts b/tests/acceptance/setup/piClientSetup.test.ts index 5939b4443a..2837db434f 100644 --- a/tests/acceptance/setup/piClientSetup.test.ts +++ b/tests/acceptance/setup/piClientSetup.test.ts @@ -138,7 +138,6 @@ cliTest( arguments: arguments_, cwd: home, environment, - timeoutMs: 20_000, }); expect(applied.exitCode).toBe(0); expect(applied.json).toMatchObject({ @@ -193,7 +192,6 @@ cliTest( arguments: arguments_, cwd: home, environment, - timeoutMs: 20_000, }); expect(repeated.exitCode).toBe(0); expect(repeated.json).toMatchObject({ @@ -232,7 +230,6 @@ cliTest( PI_PROFILE: "legacy", PI_CONFIG_DIR: ".omp-other", }, - timeoutMs: 20_000, }); const planned = await run([ @@ -345,7 +342,6 @@ cliTest( HOPPER_LAUNCHER_PATH: join(home, "unconfigured-hopper"), PI_CODING_AGENT_DIR: "custom-agent", }, - timeoutMs: 20_000, }); const run = runIn(created); const setup = await run(["setup", "--client", "pi", "--yes", "--json"]); @@ -456,7 +452,6 @@ cliTest( PI_CODING_AGENT_DIR: "file:///agent%2fdir", REA_DARWIN_MODULE_CACHE_PATH: moduleCache, }, - timeoutMs: 20_000, }); const plan = await run([ "setup", diff --git a/tests/acceptance/setup/updateAliasCli.test.ts b/tests/acceptance/setup/updateAliasCli.test.ts index 7587c4638b..f4ba6bf367 100644 --- a/tests/acceptance/setup/updateAliasCli.test.ts +++ b/tests/acceptance/setup/updateAliasCli.test.ts @@ -46,7 +46,6 @@ cliTest( cli.run({ arguments: arguments_, cwd: home, - timeoutMs: 20_000, environment: { USERPROFILE: home, PATH: [ diff --git a/tests/acceptance/webArtifactAcquisitionCli.test.ts b/tests/acceptance/webArtifactAcquisitionCli.test.ts index b161dd71d7..44b47eb7f5 100644 --- a/tests/acceptance/webArtifactAcquisitionCli.test.ts +++ b/tests/acceptance/webArtifactAcquisitionCli.test.ts @@ -36,7 +36,6 @@ cliTest( "--json", ], environment, - timeoutMs: 30_000, }); expect(result.exitCode, result.stdout + result.stderr).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/acceptance/webModuleCli.test.ts b/tests/acceptance/webModuleCli.test.ts index 468d1551b9..e49f0e0259 100644 --- a/tests/acceptance/webModuleCli.test.ts +++ b/tests/acceptance/webModuleCli.test.ts @@ -43,7 +43,6 @@ cliTest( // This single CLI invocation cannot reuse a newly written cache. NODE_DISABLE_COMPILE_CACHE: "1", }, - timeoutMs: 15000, }); expect(cliResponse.exitCode, cliResponse.stdout + cliResponse.stderr).toBe( 0, diff --git a/tests/acceptance/webSourceLocationCli.test.ts b/tests/acceptance/webSourceLocationCli.test.ts index fe31f7bb94..dfcf13cd5d 100644 --- a/tests/acceptance/webSourceLocationCli.test.ts +++ b/tests/acceptance/webSourceLocationCli.test.ts @@ -72,7 +72,6 @@ const expectCliSourceLocationSummary = async ( const { bundlePath, evidenceId, sha256 } = expected; const missingSink = await cli.run({ arguments: [...arguments_, "--detail", "summary"], - timeoutMs: 45000, }); expect(missingSink.exitCode).toBe(1); expect(missingSink.json).toMatchObject({ @@ -87,7 +86,6 @@ const expectCliSourceLocationSummary = async ( "--evidence_output", bundlePath, ], - timeoutMs: 45000, }); expect(summary.exitCode, summary.stderr).toBe(0); expect(summary.stdout.length).toBeLessThan(30000); @@ -217,7 +215,6 @@ cliTest( REA_BROWSER_EXECUTABLE: "relative-unconfigured-browser", CLANG_MODULE_CACHE_PATH: map, }, - timeoutMs: 45000, }); expect(response.exitCode, response.stdout + response.stderr).toBe(0); expect(response.json).toMatchObject({ @@ -315,7 +312,6 @@ cliTest( "--json", ], environment: { REA_LOG_LEVEL: "silent" }, - timeoutMs: 45000, }); expect(invalid.exitCode).toBe(1); expectInvalidSourceLocation( @@ -372,7 +368,6 @@ cliTest( REA_LOG_LEVEL: "silent", REA_BROWSER_EXECUTABLE: "relative-unconfigured-browser", }, - timeoutMs: 45_000, }); const response = await call("trace_web_source_location", { manifest_path: manifest, @@ -491,7 +486,6 @@ cliTest( "--json", ], environment: { REA_LOG_LEVEL: "silent" }, - timeoutMs: 45_000, }); const response = await call("trace_web_source_location", { manifest_path: manifest, @@ -578,7 +572,6 @@ cliTest( REA_LOG_LEVEL: "silent", REA_BROWSER_EXECUTABLE: "relative-unconfigured-browser", }, - timeoutMs: 45_000, }); expect(result.exitCode, result.stdout + result.stderr).toBe(0); expect(result.json).toMatchObject({ @@ -649,7 +642,6 @@ cliTest( ]; const refused = await cli.run({ arguments: [...args, "--source-map-bytes", "128"], - timeoutMs: 45000, }); expect(refused.exitCode).toBe(1); expect((refused.json as { code?: unknown }).code).toBe("truncated"); @@ -665,7 +657,6 @@ cliTest( "--source-map-bytes", String(Buffer.byteLength(fixture.sourceMap.text)), ], - timeoutMs: 45000, }); expect(recovered.exitCode, recovered.stderr).toBe(0); expect(recovered.json).toMatchObject({ @@ -721,7 +712,6 @@ cliTest( ]; const refused = await cli.run({ arguments: [...arguments_, "--source-map-bytes", "4194304"], - timeoutMs: 45000, }); expect(refused.exitCode).toBe(1); expect(JSON.stringify(refused.json)).toContain( @@ -729,7 +719,6 @@ cliTest( ); const completed = await cli.run({ arguments: arguments_, - timeoutMs: 45000, }); expect(completed.exitCode, completed.stderr).toBe(0); const evidence = completed.json as { diff --git a/tests/boundary/browser/browserFixtureHttpTargets.test.ts b/tests/boundary/browser/browserFixtureHttpTargets.test.ts index 7619fb9484..4b4c09f1df 100644 --- a/tests/boundary/browser/browserFixtureHttpTargets.test.ts +++ b/tests/boundary/browser/browserFixtureHttpTargets.test.ts @@ -7,19 +7,15 @@ for (const kind of ["proxy", "workers", "storage"] as const) { processTest( `${kind} HTTP fixture preserves targets and survives invalid requests`, async ({ processes }) => { - const result = await processes.run( - process.execPath, - [ - fileURLToPath( - new URL( - "../../fixtures/browserFixtureHttpTargets.mjs", - import.meta.url, - ), + const result = await processes.run(process.execPath, [ + fileURLToPath( + new URL( + "../../fixtures/browserFixtureHttpTargets.mjs", + import.meta.url, ), - kind, - ], - { timeoutMs: 15_000 }, - ); + ), + kind, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); expect(result.stderr).toBe(""); diff --git a/tests/boundary/browser/browserStorageListenerScope.test.ts b/tests/boundary/browser/browserStorageListenerScope.test.ts index 8149ac3cf0..9ea17251a1 100644 --- a/tests/boundary/browser/browserStorageListenerScope.test.ts +++ b/tests/boundary/browser/browserStorageListenerScope.test.ts @@ -11,20 +11,16 @@ const hasNonLoopbackIpv4 = Object.values(networkInterfaces()) processTest.skipIf(!hasNonLoopbackIpv4)( "storage fixture serves localhost and refuses this host's non-loopback interface", async ({ processes }) => { - const result = await processes.run( - process.execPath, - [ - fileURLToPath( - new URL( - "../../fixtures/browserFixtureHttpTargets.mjs", - import.meta.url, - ), + const result = await processes.run(process.execPath, [ + fileURLToPath( + new URL( + "../../fixtures/browserFixtureHttpTargets.mjs", + import.meta.url, ), - "storage", - "listener-scope", - ], - { timeoutMs: 15_000 }, - ); + ), + "storage", + "listener-scope", + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); expect(result.stderr).toBe(""); diff --git a/tests/boundary/browser/domVerifierCleanup.test.ts b/tests/boundary/browser/domVerifierCleanup.test.ts index f3d5375da2..eb99985091 100644 --- a/tests/boundary/browser/domVerifierCleanup.test.ts +++ b/tests/boundary/browser/domVerifierCleanup.test.ts @@ -73,7 +73,6 @@ processTest( ["--input-type=module", "--eval", source], { env: { ...process.env, REA_BROWSER_EXECUTABLE: "fixture-chrome" }, - timeoutMs: 15_000, }, ); expect(result.exitCode, result.stderr).toBe(0); diff --git a/tests/boundary/browser/electronActiveHook.test.ts b/tests/boundary/browser/electronActiveHook.test.ts index fd2660702d..360126a76d 100644 --- a/tests/boundary/browser/electronActiveHook.test.ts +++ b/tests/boundary/browser/electronActiveHook.test.ts @@ -3774,7 +3774,7 @@ const runNode = async ( const result = await processes.run( process.execPath, [...nodeArguments, ...(preloadHook ? ["-r", hook] : []), script], - { cwd: root, env: process.env, timeoutMs: 10_000 }, + { cwd: root, env: process.env }, ); return { code: result.exitCode, diff --git a/tests/boundary/browser/largeBase64Capture.test.ts b/tests/boundary/browser/largeBase64Capture.test.ts index 06ebc313d5..8a7abb9ee7 100644 --- a/tests/boundary/browser/largeBase64Capture.test.ts +++ b/tests/boundary/browser/largeBase64Capture.test.ts @@ -90,7 +90,6 @@ cliTest( const fromCli = await cli.run({ arguments: ["inspect-web-network-capture", path, "har", "--json"], - timeoutMs: 120_000, }); expect(fromCli.exitCode).toBe(0); const cliSummary = summarize(fromCli.json); diff --git a/tests/boundary/cli/ampRegistrationPreservation.test.ts b/tests/boundary/cli/ampRegistrationPreservation.test.ts index 1f9faf34b4..960e98cf6f 100644 --- a/tests/boundary/cli/ampRegistrationPreservation.test.ts +++ b/tests/boundary/cli/ampRegistrationPreservation.test.ts @@ -29,7 +29,6 @@ cliTest( arguments: ["mcp", "add", "--agent=amp", ...options, "--json"], cwd: root, environment, - timeoutMs: 20_000, }); const settingsPath = ampSettings(home); const original = JSON.stringify( @@ -133,7 +132,6 @@ cliTest( arguments: ["mcp", "add", "--agent", "amp", "--json"], cwd: root, environment, - timeoutMs: 20_000, }); expect(rejected.exitCode, rejected.stderr).toBe(1); expect(mcpRegistrationReportSchema.parse(rejected.json)).toMatchObject({ diff --git a/tests/boundary/cli/cliOutput.test.ts b/tests/boundary/cli/cliOutput.test.ts index e755711b1e..85cc02eb1d 100644 --- a/tests/boundary/cli/cliOutput.test.ts +++ b/tests/boundary/cli/cliOutput.test.ts @@ -293,7 +293,6 @@ describe("CLI output flag value forms", () => { ]) { const result = await cli.run({ arguments: [...filter, "providers", "--json"], - timeoutMs: 10_000, }); expect(result.exitCode).toBe(0); expect(JSON.parse(result.stdout)).toEqual({}); @@ -393,7 +392,6 @@ describe("unclosed CLI output filter slices", () => { "--filter-output", "result[0,1].name[", ], - timeoutMs: 3000, }); expect(result.exitCode).toBe(1); expect(result.stdout).toContain("INVALID_OUTPUT_FILTER"); @@ -402,7 +400,6 @@ describe("unclosed CLI output filter slices", () => { } const splitPath = await cli.run({ arguments: ["providers", "--json", "--filter-output", "][,]"], - timeoutMs: 3000, }); expect(splitPath.exitCode).toBe(1); expect(splitPath.json).toMatchObject({ @@ -422,7 +419,6 @@ describe("unclosed CLI output filter slices", () => { "--filter-output", "capabilities[0,1].provider.id,open", ], - timeoutMs: 3000, }); expect(result.exitCode).toBe(0); expect(result.json).toEqual({ diff --git a/tests/boundary/cli/cliSetup.test.ts b/tests/boundary/cli/cliSetup.test.ts index 93fe9ba1d4..98fe26ee3f 100644 --- a/tests/boundary/cli/cliSetup.test.ts +++ b/tests/boundary/cli/cliSetup.test.ts @@ -365,7 +365,6 @@ cliTest.skipIf(process.platform === "win32")( XDG_DATA_HOME: join(home, ".local", "share"), XDG_CACHE_HOME: join(home, ".cache"), }, - timeoutMs: 20_000, }); const planned = await setup("--dry-run"); diff --git a/tests/boundary/cli/clientProfileLifecycle.test.ts b/tests/boundary/cli/clientProfileLifecycle.test.ts index 6667e6b186..fd9da15c50 100644 --- a/tests/boundary/cli/clientProfileLifecycle.test.ts +++ b/tests/boundary/cli/clientProfileLifecycle.test.ts @@ -306,7 +306,6 @@ cliTest( XDG_CONFIG_HOME: join(root, "xdg-config"), XDG_CACHE_HOME: join(root, "xdg-cache"), }, - timeoutMs: 20_000, }); expect(result.exitCode, result.stderr).toBe(0); @@ -352,7 +351,6 @@ for (const group of groups) { arguments: arguments_, cwd: home, environment, - timeoutMs: 20_000, }); const planned = await run([ "setup", @@ -470,7 +468,6 @@ cliTest("uninstall names each retained skill directory", async ({ cli }) => { USERPROFILE: home, CLAUDE_CONFIG_DIR: join(home, ".claude"), }, - timeoutMs: 20_000, }); expect(result.exitCode, result.stderr).toBe(0); diff --git a/tests/boundary/cli/clientProfiles.test.ts b/tests/boundary/cli/clientProfiles.test.ts index d8ff64e722..261b7de194 100644 --- a/tests/boundary/cli/clientProfiles.test.ts +++ b/tests/boundary/cli/clientProfiles.test.ts @@ -33,7 +33,6 @@ cliTest( USERPROFILE: home, CLAUDE_CONFIG_DIR: "", }, - timeoutMs: 20_000, }); const setup = (flags: readonly string[] = []) => run(["setup", "--client", "claude_code", ...flags, "--json"]); @@ -151,7 +150,6 @@ for (const policy of [{ excluded: ["rea"] }, { allowed: ["other"] }]) { arguments: arguments_, cwd: home, environment: { HOME: home, USERPROFILE: home }, - timeoutMs: 20_000, }); const setup = await run([ "setup", @@ -221,7 +219,6 @@ cliTest( arguments: arguments_, cwd: home, environment: { HOME: home, USERPROFILE: home }, - timeoutMs: 20_000, }); const before = await run(["doctor", "--client", "opencode", "--json"]); expect(before.exitCode).toBe(1); diff --git a/tests/boundary/cli/defaultAmpRegistration.test.ts b/tests/boundary/cli/defaultAmpRegistration.test.ts index b735f5581c..c0041d3d53 100644 --- a/tests/boundary/cli/defaultAmpRegistration.test.ts +++ b/tests/boundary/cli/defaultAmpRegistration.test.ts @@ -49,7 +49,6 @@ cliTest( ], cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); const first = await run(); expect(first.exitCode, first.stderr).toBe(0); @@ -107,7 +106,6 @@ cliTest( arguments: ["mcp", "add", ...(json ? ["--json"] : [])], cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); expect(result.exitCode).toBe(1); const document = report(result.stdout, json); @@ -142,7 +140,6 @@ cliTest( arguments: arguments_, cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); expect(result.exitCode, result.stdout).toBe(0); expect(await readFile(f.settings, "utf8")).toBe("broken JSON\n"); @@ -167,7 +164,6 @@ cliTest( arguments: arguments_, cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); const registered = await run(); expect(registered.exitCode, registered.stdout).toBe(0); @@ -235,7 +231,6 @@ cliTest( ], cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); expect(result.exitCode, result.stdout).toBe(0); expect(report(result.stdout, true)).toMatchObject({ @@ -265,7 +260,6 @@ cliTest( arguments: ["mcp", "add", ...options, "--json"], cwd: f.home, environment: f.environment, - timeoutMs: 20_000, }); expect(result.exitCode, result.stderr).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/desktopBuildHost.test.ts b/tests/boundary/cli/desktopBuildHost.test.ts index 8a6539295c..6bbc48e7d9 100644 --- a/tests/boundary/cli/desktopBuildHost.test.ts +++ b/tests/boundary/cli/desktopBuildHost.test.ts @@ -91,7 +91,6 @@ cliTest.skipIf(!["darwin", "linux"].includes(process.platform))( "--json", ], environment, - timeoutMs: 45_000, }); const report = z .object({ scope_checks: z.array(checkSchema) }) @@ -152,7 +151,6 @@ cliTest.skipIf(!["darwin", "linux"].includes(process.platform))( "--json", ], environment, - timeoutMs: 45_000, }); const report = z .object({ scope_checks: z.array(checkSchema) }) @@ -183,7 +181,6 @@ cliTest.skipIf(!["darwin", "linux"].includes(process.platform))( ...environment, REA_DOCKER_COMMAND: join(environment.HOME, "..", "bin", "docker"), }, - timeoutMs: 45_000, }); expect(result.exitCode, result.stderr).toBe(1); @@ -242,7 +239,6 @@ cliTest.skipIf(!["darwin", "linux"].includes(process.platform))( "--json", ], environment, - timeoutMs: 45_000, }); const plan = z diff --git a/tests/boundary/cli/doctorHopperDiscovery.test.ts b/tests/boundary/cli/doctorHopperDiscovery.test.ts index 7e9c692bed..d099756d4b 100644 --- a/tests/boundary/cli/doctorHopperDiscovery.test.ts +++ b/tests/boundary/cli/doctorHopperDiscovery.test.ts @@ -178,7 +178,6 @@ cliTest.skipIf(!["darwin", "linux"].includes(process.platform))( REA_HOPPER_TEST_MARKER: launchMarker, GHIDRA_INSTALL_DIR: join(home, "missing-ghidra"), }, - timeoutMs: 20_000, }); const hopperReady = process.platform === "darwin"; diff --git a/tests/boundary/cli/doctorRegistrationEnvironment.test.ts b/tests/boundary/cli/doctorRegistrationEnvironment.test.ts index 7775a84dcc..d337548b4e 100644 --- a/tests/boundary/cli/doctorRegistrationEnvironment.test.ts +++ b/tests/boundary/cli/doctorRegistrationEnvironment.test.ts @@ -75,7 +75,6 @@ const run = async ( const result = await cli.run({ arguments: [...arguments_, "--json"], environment, - timeoutMs: 30_000, }); return { result, report: reportSchema.safeParse(result.json) }; }; diff --git a/tests/boundary/cli/electronFuseCleanupCli.test.ts b/tests/boundary/cli/electronFuseCleanupCli.test.ts index e03a72fe48..51373bcf1a 100644 --- a/tests/boundary/cli/electronFuseCleanupCli.test.ts +++ b/tests/boundary/cli/electronFuseCleanupCli.test.ts @@ -44,7 +44,6 @@ cliTest.for([ REA_FUSE_PROBE_TRACE: tracePath, REA_FUSE_PROBE_ALLOW: join(root, "allow"), }, - timeoutMs: 30_000, }, ); const trace = (await readFile(tracePath, "utf8")) diff --git a/tests/boundary/cli/environmentBuildCli.test.ts b/tests/boundary/cli/environmentBuildCli.test.ts index 3d3500938b..0746086b41 100644 --- a/tests/boundary/cli/environmentBuildCli.test.ts +++ b/tests/boundary/cli/environmentBuildCli.test.ts @@ -59,7 +59,6 @@ cliTest( "--json", ], cwd: root, - timeoutMs: 60_000, }); expect(built.exitCode, built.stderr).toBe(0); expect(built.json).toMatchObject({ @@ -111,7 +110,6 @@ cliTest( join(root, "three-step"), "--json", ], - timeoutMs: 60_000, }); expect(exported.exitCode, exported.stderr).toBe(0); @@ -178,7 +176,6 @@ cliTest( "--capture-timeout-ms=030000", "--json", ], - timeoutMs: 60_000, }); expect(planned.exitCode, planned.stderr).toBe(0); expect(planned.json).toMatchObject({ @@ -231,7 +228,6 @@ cliTest( const noDirectory = await cli.run({ arguments: ["environment-build", har, "--json"], - timeoutMs: 60_000, }); expect(noDirectory.exitCode).toBe(1); expect(noDirectory.json).toMatchObject({ @@ -265,7 +261,6 @@ cliTest.for(["--dry-run=abc", "--dry-run="] as const)( await writeFile(har, JSON.stringify(environmentManifestHar())); const result = await cli.run({ arguments: ["environment-build", har, output, dryRun, "--json"], - timeoutMs: 60_000, }); expect(result.exitCode).not.toBe(0); expect(`${result.stdout}\n${result.stderr}`).toContain("--dry-run"); @@ -284,7 +279,6 @@ cliTest.for(["--dry-run=true", "--dry-run=false"] as const)( await writeFile(har, JSON.stringify(environmentManifestHar())); const result = await cli.run({ arguments: ["environment-build", har, output, dryRun, "--json"], - timeoutMs: 60_000, }); expect(result.exitCode, result.stderr).toBe(0); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/environmentCompareCli.test.ts b/tests/boundary/cli/environmentCompareCli.test.ts index 0113e4b1a0..f12dbc0532 100644 --- a/tests/boundary/cli/environmentCompareCli.test.ts +++ b/tests/boundary/cli/environmentCompareCli.test.ts @@ -709,7 +709,6 @@ describe("rea environment-compare replay and chronology", () => { files.probe, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(0); expect(result.json).toMatchObject({ @@ -737,7 +736,6 @@ describe("rea environment-compare replay and chronology", () => { files.manifest, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/environmentExportCli.test.ts b/tests/boundary/cli/environmentExportCli.test.ts index bf71062df2..bde25d3117 100644 --- a/tests/boundary/cli/environmentExportCli.test.ts +++ b/tests/boundary/cli/environmentExportCli.test.ts @@ -46,7 +46,6 @@ cliTest( NODE_OPTIONS: `--import=${preload} --max-old-space-size=768`, NODE_DISABLE_COMPILE_CACHE: "1", }, - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -93,7 +92,6 @@ cliTest( name, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -123,7 +121,6 @@ cliTest( directory, "--json", ], - timeoutMs: 30_000, }); expect(result.stderr).toBe(""); expect(result.exitCode).toBe(0); @@ -211,7 +208,6 @@ cliTest( directory, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -253,7 +249,6 @@ cliTest( directory, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -290,7 +285,6 @@ cliTest.skipIf(process.platform === "win32")( NODE_OPTIONS: `--import=${preload} --max-old-space-size=768`, NODE_DISABLE_COMPILE_CACHE: "1", }, - timeoutMs: 30_000, }); expect(result.exitCode, result.stderr).toBe(0); for (const relative of BUNDLE_FILES) { @@ -332,7 +326,6 @@ cliTest.skipIf(process.platform === "win32")( packagePath, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode, result.stderr).toBe(0); expect(result.json).toMatchObject({ @@ -387,7 +380,6 @@ cliTest.skipIf(HOST_PATH_MAX[process.platform] === undefined)( directory, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/environmentExportInputsCli.test.ts b/tests/boundary/cli/environmentExportInputsCli.test.ts index cf1e986ac9..c0b6ee4ceb 100644 --- a/tests/boundary/cli/environmentExportInputsCli.test.ts +++ b/tests/boundary/cli/environmentExportInputsCli.test.ts @@ -46,7 +46,6 @@ for (const role of ["manifest", "evidence"] as const) { REA_TEST_EXPORT_READ_MARKER: marker, REA_TEST_EXPORT_READ_ACTION: "append", }, - timeoutMs: 30_000, }); expect(await readFile(marker, "utf8")).toBe("read"); expect( @@ -122,7 +121,6 @@ cliTest( REA_TEST_EXPORT_READ_MARKER: marker, REA_TEST_EXPORT_READ_ACTION: "record", }, - timeoutMs: 30_000, }); expect(result.exitCode, result.stderr).toBe(1); const limits = heapRefusalSchema.parse(result.json).details @@ -178,7 +176,6 @@ cliTest( REA_TEST_EXPORT_READ_MARKER: marker, REA_TEST_EXPORT_READ_ACTION: "fail-copy", }, - timeoutMs: 30_000, }); expect(await readFile(marker, "utf8")).toBe("copy"); expect(await readFile(tarball, "utf8")).toBe("admitted package"); diff --git a/tests/boundary/cli/environmentServeCli.test.ts b/tests/boundary/cli/environmentServeCli.test.ts index b552aa333b..bf7da50111 100644 --- a/tests/boundary/cli/environmentServeCli.test.ts +++ b/tests/boundary/cli/environmentServeCli.test.ts @@ -41,7 +41,6 @@ describe("rea environment-serve refusals", () => { "0", "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -66,7 +65,6 @@ describe("rea environment-serve refusals", () => { const files = await writeEnvironmentServeFiles(root); const result = await cli.run({ arguments: ["environment-serve", files.manifestPath, "--json"], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -97,7 +95,6 @@ describe("rea environment-serve refusals", () => { flag, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -121,7 +118,6 @@ describe("rea environment-serve refusals", () => { files.capturePath, "--json", ], - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ @@ -195,7 +191,6 @@ cliTest( "0", "--json", ], - timeoutMs: 30000, }); expect(result.exitCode).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/environmentServeRemediation.test.ts b/tests/boundary/cli/environmentServeRemediation.test.ts index 77bbf7fd84..e219f747d8 100644 --- a/tests/boundary/cli/environmentServeRemediation.test.ts +++ b/tests/boundary/cli/environmentServeRemediation.test.ts @@ -37,7 +37,6 @@ const serve = async ( String(port), "--json", ], - timeoutMs: 30_000, }); return { exitCode: result.exitCode, diff --git a/tests/boundary/cli/foreignUniversalMachoCli.test.ts b/tests/boundary/cli/foreignUniversalMachoCli.test.ts index c1d4f3042c..116f4aa7d5 100644 --- a/tests/boundary/cli/foreignUniversalMachoCli.test.ts +++ b/tests/boundary/cli/foreignUniversalMachoCli.test.ts @@ -23,24 +23,22 @@ cliTest.skipIf(process.platform !== "darwin")( const directory = await createTestTempDirectory("rea-foreign-fat-cli-"); const thin = join(directory, "thin"); const universal = join(directory, "universal"); - const compiled = await processes.run( - "/usr/bin/xcrun", - [ - "clang", - "-arch", - foreign.name, - resolve("tests/conformance/c/fixture.c"), - "-o", - thin, - ], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-arch", + foreign.name, + resolve("tests/conformance/c/fixture.c"), + "-o", + thin, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); - const created = await processes.run( - "/usr/bin/xcrun", - ["lipo", "-create", thin, "-output", universal], - { timeoutMs: 30_000 }, - ); + const created = await processes.run("/usr/bin/xcrun", [ + "lipo", + "-create", + thin, + "-output", + universal, + ]); expect(created.exitCode, created.stderr).toBe(0); const result = await cli.run({ diff --git a/tests/boundary/cli/ghidraInstallationProperties.test.ts b/tests/boundary/cli/ghidraInstallationProperties.test.ts index d490c6b562..0edba30c03 100644 --- a/tests/boundary/cli/ghidraInstallationProperties.test.ts +++ b/tests/boundary/cli/ghidraInstallationProperties.test.ts @@ -31,7 +31,6 @@ cliTest.for(["pipe", "capacity"] as const)( JAVA_HOME: join(root, "missing-jdk"), REA_LOG_LEVEL: "silent", }, - timeoutMs: 5000, }); expect(result.exitCode, `${result.stdout}\n${result.stderr}`).toBe(1); expect(result.signal).toBeNull(); diff --git a/tests/boundary/cli/ghidraJavaDoctor.test.ts b/tests/boundary/cli/ghidraJavaDoctor.test.ts index 1377170ea3..07e1dc5e9a 100644 --- a/tests/boundary/cli/ghidraJavaDoctor.test.ts +++ b/tests/boundary/cli/ghidraJavaDoctor.test.ts @@ -156,7 +156,6 @@ setTimeout(() => process.exit(92), 12000); GHIDRA_INSTALL_DIR: install, REA_LOG_LEVEL: "silent", }, - timeoutMs: 15000, }); // Capture the actual group while the selected process is alive; the PID // alone is never used as cleanup authority. Release is a private protocol. diff --git a/tests/boundary/cli/githubActionsExample.test.ts b/tests/boundary/cli/githubActionsExample.test.ts index a8e7df0a38..03bf5e4cd7 100644 --- a/tests/boundary/cli/githubActionsExample.test.ts +++ b/tests/boundary/cli/githubActionsExample.test.ts @@ -147,7 +147,6 @@ describe.skipIf(process.platform === "win32")( { cwd: workspace, env, - timeoutMs: 30_000, }, ); const checkoutReports = join(workspace, "reports", "rea"); diff --git a/tests/boundary/cli/harBrowserSerializedUrlsCli.test.ts b/tests/boundary/cli/harBrowserSerializedUrlsCli.test.ts index b50fb8e87b..d559d513ab 100644 --- a/tests/boundary/cli/harBrowserSerializedUrlsCli.test.ts +++ b/tests/boundary/cli/harBrowserSerializedUrlsCli.test.ts @@ -79,7 +79,6 @@ cliTest( }); const built = await cli.run({ arguments: ["environment-build", har, "--dry-run", "--json"], - timeoutMs: 60_000, }); expect(built.exitCode, built.stdout + built.stderr).toBe(0); expect(built.json).toMatchObject({ diff --git a/tests/boundary/cli/hopperLaunchFailureCli.test.ts b/tests/boundary/cli/hopperLaunchFailureCli.test.ts index 14ccd8437c..133d3be61d 100644 --- a/tests/boundary/cli/hopperLaunchFailureCli.test.ts +++ b/tests/boundary/cli/hopperLaunchFailureCli.test.ts @@ -54,7 +54,6 @@ describe("Hopper launcher failure through the one-shot CLI", () => { HOPPER_LOADER_ARGS_JSON: JSON.stringify(loaderArgs), REA_LOG_LEVEL: "silent", }, - timeoutMs: 45_000, }); const retained = RetainedRuntime.safeParse(result.json); if (retained.success) diff --git a/tests/boundary/cli/idaDoctorRegistration.test.ts b/tests/boundary/cli/idaDoctorRegistration.test.ts index 46c005e2a6..856449d2a4 100644 --- a/tests/boundary/cli/idaDoctorRegistration.test.ts +++ b/tests/boundary/cli/idaDoctorRegistration.test.ts @@ -15,7 +15,6 @@ cliTest.skipIf(process.platform === "win32")( const result = await cli.run({ arguments: ["doctor", "--provider", "ida", "--json"], environment: { HOME: root, REA_IDA_MCP_CONFIG: registration }, - timeoutMs: 10_000, }); expect(result.exitCode, result.stderr).toBe(1); expect(result.json).toMatchObject({ @@ -64,7 +63,6 @@ cliTest.skipIf(process.platform !== "darwin").for([true, false])( const result = await cli.run({ arguments: ["doctor", "--provider", "ida", "--json"], environment: { HOME: root, REA_IDA_MCP_CONFIG: registration }, - timeoutMs: 10_000, }); expect(result.exitCode, result.stderr).toBe(usable ? 0 : 1); expect(result.json).toMatchObject({ @@ -144,7 +142,6 @@ cliTest.for([ const result = await cli.run({ arguments: ["doctor", "--provider", "ida", "--json"], environment: { HOME: root, REA_IDA_MCP_CONFIG: path }, - timeoutMs: 10_000, }); expect(result.exitCode, result.stderr).toBe( constraint === undefined ? 0 : 1, @@ -171,7 +168,6 @@ cliTest.for([ const providers = await cli.run({ arguments: ["providers", "--json"], environment: { HOME: root, REA_IDA_MCP_CONFIG: path }, - timeoutMs: 10_000, }); expect(providers.exitCode, providers.stderr).toBe(0); expect(providers.json).toMatchObject({ diff --git a/tests/boundary/cli/idaRetainedCleanupCli.test.ts b/tests/boundary/cli/idaRetainedCleanupCli.test.ts index e21054020f..eb439e007a 100644 --- a/tests/boundary/cli/idaRetainedCleanupCli.test.ts +++ b/tests/boundary/cli/idaRetainedCleanupCli.test.ts @@ -58,7 +58,6 @@ describe("headless IDA timeout through the one-shot CLI", () => { REA_IDA_MCP_CONFIG: registration, REA_LOG_LEVEL: "silent", }, - timeoutMs: 30_000, }); expect(result.exitCode).toBe(1); diff --git a/tests/boundary/cli/idaStartupAttributionCli.test.ts b/tests/boundary/cli/idaStartupAttributionCli.test.ts index 68a233dcb4..07395ed961 100644 --- a/tests/boundary/cli/idaStartupAttributionCli.test.ts +++ b/tests/boundary/cli/idaStartupAttributionCli.test.ts @@ -227,7 +227,6 @@ workspaceCliTest.for([ REA_IDA_MCP_CONFIG: registration, REA_LOG_LEVEL: "silent", }, - timeoutMs: 15000, }); expect(result.stderr).toBe(""); expect(supervisor.database).toBeUndefined(); diff --git a/tests/boundary/cli/javascriptRuntimeWideLoadStates.test.ts b/tests/boundary/cli/javascriptRuntimeWideLoadStates.test.ts index 87fbb7f8e3..0c566a4aaa 100644 --- a/tests/boundary/cli/javascriptRuntimeWideLoadStates.test.ts +++ b/tests/boundary/cli/javascriptRuntimeWideLoadStates.test.ts @@ -112,7 +112,7 @@ processTest.for([1_000, 16_000])( path, "--json", ], - { cwd: root, timeoutMs: 30_000 }, + { cwd: root }, ); expect( response.exitCode, diff --git a/tests/boundary/cli/jebCli.test.ts b/tests/boundary/cli/jebCli.test.ts index 2ec76b0fa8..ef3cf8bdf2 100644 --- a/tests/boundary/cli/jebCli.test.ts +++ b/tests/boundary/cli/jebCli.test.ts @@ -124,7 +124,6 @@ it.each( const result = await cli.run({ arguments: argv, environment: { HOME: home, REA_JEB_MCP_URL: fixture.endpoint }, - timeoutMs: 10_000, }); const output = JSON.parse(result.stdout); if (stage === "success") { diff --git a/tests/boundary/cli/kunaDoctorIdentity.test.ts b/tests/boundary/cli/kunaDoctorIdentity.test.ts index 26b3b9a8bc..9de389ca59 100644 --- a/tests/boundary/cli/kunaDoctorIdentity.test.ts +++ b/tests/boundary/cli/kunaDoctorIdentity.test.ts @@ -41,7 +41,6 @@ ${mutation} REA_KUNA_CACHE_DIR: join(root, "cache"), REA_LOG_LEVEL: "silent", }, - timeoutMs: 20_000, }); const after = await readFile(executable); const afterHash = createHash("sha256").update(after).digest("hex"); diff --git a/tests/boundary/cli/markdownLiteralCells.test.ts b/tests/boundary/cli/markdownLiteralCells.test.ts index 9fb142afee..c17f04f691 100644 --- a/tests/boundary/cli/markdownLiteralCells.test.ts +++ b/tests/boundary/cli/markdownLiteralCells.test.ts @@ -212,7 +212,6 @@ cliTest( "md", "--full-output", ], - timeoutMs: 20_000, }); expect(markdown.exitCode, markdown.stderr).toBe(0); const renderer = await createMarkdownRenderer(root, { linkify: false }); diff --git a/tests/boundary/cli/mcpAddDiscovery.test.ts b/tests/boundary/cli/mcpAddDiscovery.test.ts index 6d20faece2..4c2e31811b 100644 --- a/tests/boundary/cli/mcpAddDiscovery.test.ts +++ b/tests/boundary/cli/mcpAddDiscovery.test.ts @@ -19,7 +19,6 @@ cliTest( arguments: arguments_, cwd: home, environment, - timeoutMs: 30_000, }); const schema = await run([ "mcp", @@ -110,7 +109,6 @@ cliTest( arguments: arguments_, cwd: home, environment, - timeoutMs: 30_000, }); const filtered = await run([ "mcp", diff --git a/tests/boundary/cli/mcpAddOptionValidation.test.ts b/tests/boundary/cli/mcpAddOptionValidation.test.ts index ea9b6563cc..9fdff480f5 100644 --- a/tests/boundary/cli/mcpAddOptionValidation.test.ts +++ b/tests/boundary/cli/mcpAddOptionValidation.test.ts @@ -23,7 +23,6 @@ cliTest("mcp add rejects valued options without a value", async ({ cli }) => { arguments: [...arguments_, "--json"], cwd: root, environment: await registrationEnvironment(selectedHome), - timeoutMs: 20_000, }); for (const [flag, field] of [ @@ -106,7 +105,6 @@ cliTest( arguments: [...arguments_, "--json"], cwd: root, environment, - timeoutMs: 20_000, }); // Incur skips a stray word, so the registration used to succeed for a request @@ -178,7 +176,6 @@ cliTest( arguments: [...arguments_, "--json"], cwd: root, environment, - timeoutMs: 20_000, }); // A stray word alongside a missing value: the message describes the missing @@ -223,7 +220,6 @@ cliTest( ], cwd: project, environment, - timeoutMs: 20_000, }); expect(result.exitCode, result.stderr).toBe(1); expect(result.json).toMatchObject({ diff --git a/tests/boundary/cli/nativeDemangleCli.test.ts b/tests/boundary/cli/nativeDemangleCli.test.ts index c18b6f5422..361f216836 100644 --- a/tests/boundary/cli/nativeDemangleCli.test.ts +++ b/tests/boundary/cli/nativeDemangleCli.test.ts @@ -58,7 +58,6 @@ describe("Swift demangling CLI with line-terminator characters", () => { "--json", ], environment, - timeoutMs: 20_000, }); expect(result.signal).toBeNull(); expect(result.exitCode).toBe(1); @@ -76,7 +75,6 @@ describe("Swift demangling CLI with line-terminator characters", () => { const result = await cli.run({ arguments: ["demangle-swift", "--json"], environment, - timeoutMs: 20_000, }); expect(result.signal).toBeNull(); expect(result.exitCode).toBe(1); diff --git a/tests/boundary/cli/nativeMachoArm64_32Cli.test.ts b/tests/boundary/cli/nativeMachoArm64_32Cli.test.ts index 81445ab546..71a03e2de3 100644 --- a/tests/boundary/cli/nativeMachoArm64_32Cli.test.ts +++ b/tests/boundary/cli/nativeMachoArm64_32Cli.test.ts @@ -31,11 +31,15 @@ describe.skipIf(process.platform !== "darwin")( const source = join(directory, "watch.c"); await writeFile(source, "int rea_watch_value(void) { return 42; }\n"); const object = join(directory, "watch.o"); - const compiled = await processes.run( - "/usr/bin/xcrun", - ["clang", "-arch", "arm64_32", "-c", source, "-o", object], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-arch", + "arm64_32", + "-c", + source, + "-o", + object, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); for (const command of [ "list-architectures", diff --git a/tests/boundary/cli/nativeMachoDylibIdentityCli.test.ts b/tests/boundary/cli/nativeMachoDylibIdentityCli.test.ts index 091897b891..8c55433dd6 100644 --- a/tests/boundary/cli/nativeMachoDylibIdentityCli.test.ts +++ b/tests/boundary/cli/nativeMachoDylibIdentityCli.test.ts @@ -20,18 +20,14 @@ describe.skipIf(process.platform !== "darwin")( const source = join(directory, "library.c"); const library = join(directory, "libprobe.dylib"); await writeFile(source, "int exported_value(void) { return 42; }\n"); - const compiled = await processes.run( - "/usr/bin/xcrun", - [ - "clang", - "-dynamiclib", - source, - "-Wl,-install_name,@rpath/libprobe.dylib", - "-o", - library, - ], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-dynamiclib", + source, + "-Wl,-install_name,@rpath/libprobe.dylib", + "-o", + library, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); const result = await cli.run({ arguments: ["inspect-macho", library, "--json"], @@ -54,11 +50,12 @@ describe.skipIf(process.platform !== "darwin")( // One architecture can still have a FAT container header. const universal = join(directory, "libprobe-universal.dylib"); - const combined = await processes.run( - "/usr/bin/lipo", - ["-create", library, "-output", universal], - { timeoutMs: 30_000 }, - ); + const combined = await processes.run("/usr/bin/lipo", [ + "-create", + library, + "-output", + universal, + ]); expect(combined.exitCode, combined.stderr).toBe(0); const fatResult = await cli.run({ arguments: ["inspect-macho", universal, "--json"], diff --git a/tests/boundary/cli/nativeMachoEchoedPathCli.test.ts b/tests/boundary/cli/nativeMachoEchoedPathCli.test.ts index 91e32ad697..76f9465aa2 100644 --- a/tests/boundary/cli/nativeMachoEchoedPathCli.test.ts +++ b/tests/boundary/cli/nativeMachoEchoedPathCli.test.ts @@ -52,20 +52,16 @@ describe.skipIf(process.platform !== "darwin")( "rea-native-cli-echo-", ); const fixture = join(directory, "fat-fixture"); - const compiled = await processes.run( - "/usr/bin/xcrun", - [ - "clang", - "-arch", - "arm64", - "-arch", - "x86_64", - resolve("tests/conformance/c/fixture.c"), - "-o", - fixture, - ], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-arch", + "arm64", + "-arch", + "x86_64", + resolve("tests/conformance/c/fixture.c"), + "-o", + fixture, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); const inspect = async (name: string) => { const path = join(directory, name); diff --git a/tests/boundary/cli/nativeMachoEncryptionCli.test.ts b/tests/boundary/cli/nativeMachoEncryptionCli.test.ts index 2a60d39b8f..841ff86fb6 100644 --- a/tests/boundary/cli/nativeMachoEncryptionCli.test.ts +++ b/tests/boundary/cli/nativeMachoEncryptionCli.test.ts @@ -37,9 +37,7 @@ describe.skipIf(process.platform !== "darwin")( const fat = join(directory, "fat_enc"); await writeFile(source, "int main(void) { return 0; }\n"); const run = async (command: string, arguments_: string[]) => { - const result = await processes.run(command, arguments_, { - timeoutMs: 30_000, - }); + const result = await processes.run(command, arguments_); expect(result.exitCode, result.stderr).toBe(0); return result.stdout; }; diff --git a/tests/boundary/cli/nativeMachoLinkerOptionsCli.test.ts b/tests/boundary/cli/nativeMachoLinkerOptionsCli.test.ts index 706863201e..3a76a3aa7d 100644 --- a/tests/boundary/cli/nativeMachoLinkerOptionsCli.test.ts +++ b/tests/boundary/cli/nativeMachoLinkerOptionsCli.test.ts @@ -28,11 +28,13 @@ describe.skipIf(process.platform !== "darwin")( "", ].join("\n"), ); - const compiled = await processes.run( - "/usr/bin/xcrun", - ["clang", "-c", source, "-o", object], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-c", + source, + "-o", + object, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); const result = await cli.run({ arguments: ["inspect-macho", object, "--json"], diff --git a/tests/boundary/cli/nativeMachoMissingUuidCli.test.ts b/tests/boundary/cli/nativeMachoMissingUuidCli.test.ts index a054c0fefe..c20ddcc713 100644 --- a/tests/boundary/cli/nativeMachoMissingUuidCli.test.ts +++ b/tests/boundary/cli/nativeMachoMissingUuidCli.test.ts @@ -58,9 +58,7 @@ describe.skipIf(process.platform !== "darwin")( ["clang", "-o", control, source], ["clang", "-c", source, "-o", object], ]) { - const compiled = await processes.run("/usr/bin/xcrun", arguments_, { - timeoutMs: 30_000, - }); + const compiled = await processes.run("/usr/bin/xcrun", arguments_); expect(compiled.exitCode, compiled.stderr).toBe(0); } diff --git a/tests/boundary/cli/nativeMachoObjectSegmentCli.test.ts b/tests/boundary/cli/nativeMachoObjectSegmentCli.test.ts index 83eb8fd40f..3c6852ece7 100644 --- a/tests/boundary/cli/nativeMachoObjectSegmentCli.test.ts +++ b/tests/boundary/cli/nativeMachoObjectSegmentCli.test.ts @@ -20,11 +20,13 @@ describe.skipIf(process.platform !== "darwin")( const source = join(directory, "qa.c"); const object = join(directory, "qa.o"); await writeFile(source, "int qa(void) { return 7; }\n"); - const compiled = await processes.run( - "/usr/bin/xcrun", - ["clang", "-c", source, "-o", object], - { timeoutMs: 30_000 }, - ); + const compiled = await processes.run("/usr/bin/xcrun", [ + "clang", + "-c", + source, + "-o", + object, + ]); expect(compiled.exitCode, compiled.stderr).toBe(0); const result = await cli.run({ arguments: ["inspect-macho", object, "--json"], diff --git a/tests/boundary/cli/packageRunnerBoundInstallation.test.ts b/tests/boundary/cli/packageRunnerBoundInstallation.test.ts index 274fefec40..28ec890050 100644 --- a/tests/boundary/cli/packageRunnerBoundInstallation.test.ts +++ b/tests/boundary/cli/packageRunnerBoundInstallation.test.ts @@ -70,7 +70,6 @@ cliTest.skipIf(process.platform === "win32")( const setup = await processes.run(npm.command, npm.arguments, { cwd: trusted, env: environment, - timeoutMs: 30_000, }); expect(setup.exitCode, setup.stderr).toBe(0); const configPath = join(home, ".omp", "agent", "mcp.json"); @@ -160,7 +159,7 @@ cliTest.skipIf(process.platform === "win32")( const missing = await processes.run( registration.command, registration.args, - { cwd: project, env: clientEnvironment, timeoutMs: 10_000 }, + { cwd: project, env: clientEnvironment }, ); expect(missing.exitCode).not.toBe(0); await expect(access(marker)).rejects.toMatchObject({ code: "ENOENT" }); @@ -192,7 +191,6 @@ cliTest.skipIf(process.platform === "win32")( USERPROFILE: home, PATH: [bin, process.env.PATH ?? ""].join(delimiter), }, - timeoutMs: 20_000, }, ); expect(result.exitCode, result.stderr).toBe(0); @@ -211,7 +209,7 @@ cliTest.skipIf(process.platform === "win32")( const start = await processes.run( settings["amp.mcpServers"].rea.command, [alias, "--version"], - { cwd: home, timeoutMs: 10_000 }, + { cwd: home }, ); expect(start.exitCode, start.stderr).toBe(0); expect(start.stdout).toContain(PRODUCT_IDENTITY.packageVersion); diff --git a/tests/boundary/cli/packageRunnerRegistration.test.ts b/tests/boundary/cli/packageRunnerRegistration.test.ts index 4329be78b4..f883bce32d 100644 --- a/tests/boundary/cli/packageRunnerRegistration.test.ts +++ b/tests/boundary/cli/packageRunnerRegistration.test.ts @@ -75,7 +75,6 @@ for (const { runner, userAgent } of [ arguments: arguments_, cwd: home, environment, - timeoutMs: 20_000, }); const setup = await run([ "setup", diff --git a/tests/boundary/cli/setupProviderReadiness.test.ts b/tests/boundary/cli/setupProviderReadiness.test.ts index 3c29d86a01..d4a065bc5a 100644 --- a/tests/boundary/cli/setupProviderReadiness.test.ts +++ b/tests/boundary/cli/setupProviderReadiness.test.ts @@ -28,7 +28,6 @@ cliTest.skipIf(process.platform === "win32").for(["blocked", "ready"] as const)( REA_KUNA_COMMAND: command, REA_KUNA_CACHE_DIR: cachePath, }, - timeoutMs: 20_000, }); expect(result.exitCode, result.stderr).toBe(0); expect(result.json).toMatchObject({ @@ -88,7 +87,6 @@ cliTest.skipIf(process.platform === "win32")( const setup = await cli.run({ arguments: ["setup", "--client", "claude_code", "--dry-run", "--json"], environment, - timeoutMs: 20_000, }); expect(setup.exitCode, setup.stdout).toBe(0); expect(setup.json).toMatchObject({ @@ -111,7 +109,6 @@ cliTest.skipIf(process.platform === "win32")( const doctor = await cli.run({ arguments: ["doctor", "--json"], environment, - timeoutMs: 20_000, }); expect(doctor.json).toMatchObject({ providerInspections: expect.arrayContaining([ diff --git a/tests/boundary/cli/webScriptSelectionCli.test.ts b/tests/boundary/cli/webScriptSelectionCli.test.ts index 8c6bfc8f40..725576ac5d 100644 --- a/tests/boundary/cli/webScriptSelectionCli.test.ts +++ b/tests/boundary/cli/webScriptSelectionCli.test.ts @@ -130,15 +130,12 @@ cliTest.skipIf(process.platform === "win32")( const root = await createTestTempDirectory("rea-script-selection-cli-"); const capturePath = join(root, "capture.pipe"); const outputDirectory = join(root, "export"); - const created = await processes.run("mkfifo", [capturePath], { - timeoutMs: 5_000, - }); + const created = await processes.run("mkfifo", [capturePath]); expect(created.exitCode).toBe(0); const response = await cli.run({ arguments: ["export-web-scripts", capturePath, outputDirectory, "--json"], environment: { REA_LOG_LEVEL: "silent" }, - timeoutMs: 5_000, }); expect(response.exitCode).toBe(1); expect(response.signal).toBeNull(); diff --git a/tests/boundary/filesystem/browserFixtureRequestFailures.test.ts b/tests/boundary/filesystem/browserFixtureRequestFailures.test.ts index bfe54017ce..ff2e8ea91e 100644 --- a/tests/boundary/filesystem/browserFixtureRequestFailures.test.ts +++ b/tests/boundary/filesystem/browserFixtureRequestFailures.test.ts @@ -53,7 +53,6 @@ for (const [fixture, factory, ordinaryPath, malformedPath] of [ ["--input-type=module", "-e", program], { cwd: fileURLToPath(new URL("../../../", import.meta.url)), - timeoutMs: 10_000, }, ); expect(result.signal, result.stderr).toBeNull(); diff --git a/tests/boundary/filesystem/clientNativeConnectionAdmission.test.ts b/tests/boundary/filesystem/clientNativeConnectionAdmission.test.ts index bc13078a2f..40a4f8672d 100644 --- a/tests/boundary/filesystem/clientNativeConnectionAdmission.test.ts +++ b/tests/boundary/filesystem/clientNativeConnectionAdmission.test.ts @@ -116,7 +116,6 @@ it("keeps owned Gemini launcher facts separate from native connection admission process.execPath, ["--import", "tsx", entrypoint], { - timeoutMs: 20_000, env: { PATH: process.env.PATH, SystemRoot: process.env.SystemRoot, diff --git a/tests/boundary/filesystem/lintRuleConfiguration.test.ts b/tests/boundary/filesystem/lintRuleConfiguration.test.ts index e4f2fcafeb..5187faac5c 100644 --- a/tests/boundary/filesystem/lintRuleConfiguration.test.ts +++ b/tests/boundary/filesystem/lintRuleConfiguration.test.ts @@ -249,7 +249,6 @@ for (const scenario of cases) { ], { cwd: project, - timeoutMs: 15_000, }, ); expect(result.signal, result.stderr).toBeNull(); @@ -294,7 +293,7 @@ export const unnecessary = /\!/; "--fix", applicationSource, ], - { cwd: project, timeoutMs: 15_000 }, + { cwd: project }, ); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr || result.stdout).toBe(0); diff --git a/tests/boundary/filesystem/referenceSourceFifoRace.test.ts b/tests/boundary/filesystem/referenceSourceFifoRace.test.ts index eb6d1be1ef..5f771c7084 100644 --- a/tests/boundary/filesystem/referenceSourceFifoRace.test.ts +++ b/tests/boundary/filesystem/referenceSourceFifoRace.test.ts @@ -18,11 +18,11 @@ processTest.skipIf(process.platform === "win32")( import.meta.url, ), ); - const result = await processes.run( - process.execPath, - [probe, readerModule, root], - { timeoutMs: 3_000 }, - ); + const result = await processes.run(process.execPath, [ + probe, + readerModule, + root, + ]); expect(result.exitCode, result.stderr).toBe(0); expect(result.stdout.trim()).toBe("changed"); }, diff --git a/tests/boundary/filesystem/typedLintRules.test.ts b/tests/boundary/filesystem/typedLintRules.test.ts index eaf3ddf178..5e18a87ec0 100644 --- a/tests/boundary/filesystem/typedLintRules.test.ts +++ b/tests/boundary/filesystem/typedLintRules.test.ts @@ -154,7 +154,6 @@ processTest( [reporter, "typed-lint", ...Object.keys(sources)], { cwd: root, - timeoutMs: 20_000, env: { ...process.env, GOMAXPROCS: "2", GOMEMLIMIT: "1GiB" }, }, ); diff --git a/tests/boundary/filesystem/verifierFileSnapshots.test.ts b/tests/boundary/filesystem/verifierFileSnapshots.test.ts index 65b8ed6ebb..4781886f24 100644 --- a/tests/boundary/filesystem/verifierFileSnapshots.test.ts +++ b/tests/boundary/filesystem/verifierFileSnapshots.test.ts @@ -263,11 +263,11 @@ processTest( else await removeTestWorkspace(workspace.root); } `; - const result = await processes.run( - process.execPath, - ["--input-type=module", "--eval", program], - { timeoutMs: 20_000 }, - ); + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + program, + ]); expect(result.exitCode, result.stderr).toBe(0); expect(result.signal, result.stderr).toBeNull(); diff --git a/tests/boundary/mcp/javascriptLiteralBudget.test.ts b/tests/boundary/mcp/javascriptLiteralBudget.test.ts index 9656214634..6222051493 100644 --- a/tests/boundary/mcp/javascriptLiteralBudget.test.ts +++ b/tests/boundary/mcp/javascriptLiteralBudget.test.ts @@ -46,7 +46,6 @@ for (const scenario of [ const digest = createHash("sha256").update(source).digest("hex"); const cliResponse = await cli.run({ arguments: ["analyze-javascript-application", root, "--json"], - timeoutMs: 60_000, }); expect(cliResponse.exitCode).toBe(0); const cliEvidence = toolContract( diff --git a/tests/boundary/mcp/javascriptLiteralPayload.test.ts b/tests/boundary/mcp/javascriptLiteralPayload.test.ts index f9df9bfdff..3eb9649d4a 100644 --- a/tests/boundary/mcp/javascriptLiteralPayload.test.ts +++ b/tests/boundary/mcp/javascriptLiteralPayload.test.ts @@ -48,7 +48,6 @@ cliTest( "directory", "--json", ], - timeoutMs: 60_000, }); expect(cliResponse.exitCode).toBe(0); const cliEvidence = toolContract( diff --git a/tests/boundary/mcp/wideOccurrenceCompletion.test.ts b/tests/boundary/mcp/wideOccurrenceCompletion.test.ts index 6419034bfc..a1fe147d14 100644 --- a/tests/boundary/mcp/wideOccurrenceCompletion.test.ts +++ b/tests/boundary/mcp/wideOccurrenceCompletion.test.ts @@ -7,22 +7,18 @@ processTest.for([3, 130_000])( async (count, { processes }) => { // Thread-pool tests have a larger stack than the stdio server. Use a fresh // main thread and enough heap for the ledger's conservative admission. - const result = await processes.run( - process.execPath, - [ - "--max-old-space-size=4096", - "--import", - "tsx", - fileURLToPath( - new URL( - "../../fixtures/promptOccurrenceCompletion.ts", - import.meta.url, - ), + const result = await processes.run(process.execPath, [ + "--max-old-space-size=4096", + "--import", + "tsx", + fileURLToPath( + new URL( + "../../fixtures/promptOccurrenceCompletion.ts", + import.meta.url, ), - String(count), - ], - { timeoutMs: 30_000 }, - ); + ), + String(count), + ]); expect(result.exitCode, result.stderr).toBe(0); expect(JSON.parse(result.stdout)).toMatchObject({ values: Math.min(count + 1, 100), diff --git a/tests/boundary/process/agentEvaluationCommands.test.ts b/tests/boundary/process/agentEvaluationCommands.test.ts index c40fe01de9..07227a3311 100644 --- a/tests/boundary/process/agentEvaluationCommands.test.ts +++ b/tests/boundary/process/agentEvaluationCommands.test.ts @@ -180,7 +180,6 @@ registerHooks({ load(url, context, nextLoad) { REA_AGENT_EVAL_ROOT: parent, REA_AGENT_EVAL_KEEP_FIXTURES: "false", }, - timeoutMs: 10_000, }, ); expect(result.exitCode).toBe(1); @@ -282,7 +281,6 @@ registerHooks({ load(url, context, nextLoad) { REA_AGENT_EVAL_ROOT: parent, REA_AGENT_EVAL_KEEP_FIXTURES: "false", }, - timeoutMs: 10_000, }, ); expect(result.exitCode).toBe(1); diff --git a/tests/boundary/process/androidFixtureBootstrap.test.ts b/tests/boundary/process/androidFixtureBootstrap.test.ts index 215dadf52c..40bb7a1228 100644 --- a/tests/boundary/process/androidFixtureBootstrap.test.ts +++ b/tests/boundary/process/androidFixtureBootstrap.test.ts @@ -26,11 +26,11 @@ processTest( const entry = join(root, "scripts/fixtures/android/fetch.mjs"); // Invalid arguments stop before downloads while exercising Node's real // source-module loading, without Vitest's .js-to-.ts resolution. - const result = await processes.run( - process.execPath, - [entry, "extra", "argument"], - { timeoutMs: 10000 }, - ); + const result = await processes.run(process.execPath, [ + entry, + "extra", + "argument", + ]); expect(result.exitCode).toBe(1); expect(result.stderr).toContain( "Usage: npm run fixtures:android -- [DIRECTORY]", diff --git a/tests/boundary/process/browserRequestVerifierOwnership.test.ts b/tests/boundary/process/browserRequestVerifierOwnership.test.ts index 57f477c8af..7797e1e34e 100644 --- a/tests/boundary/process/browserRequestVerifierOwnership.test.ts +++ b/tests/boundary/process/browserRequestVerifierOwnership.test.ts @@ -139,7 +139,6 @@ process.stdout.write(JSON.stringify({ pids, live, roots, failure })); ); try { const result = await processes.run(process.execPath, [driver], { - timeoutMs: 30_000, env: { ...process.env, TMPDIR: tmp, diff --git a/tests/boundary/process/domCallbackFailures.test.ts b/tests/boundary/process/domCallbackFailures.test.ts index b069057287..8c13c53ad7 100644 --- a/tests/boundary/process/domCallbackFailures.test.ts +++ b/tests/boundary/process/domCallbackFailures.test.ts @@ -88,16 +88,12 @@ for (const scenario of ${JSON.stringify(cases)}) { } } `; - const result = await processes.run( - process.execPath, - [ - "--unhandled-rejections=strict", - "--input-type=module", - "--eval", - source, - ], - { timeoutMs: 10_000 }, - ); + const result = await processes.run(process.execPath, [ + "--unhandled-rejections=strict", + "--input-type=module", + "--eval", + source, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); }, @@ -193,17 +189,13 @@ for (const [directory, fails] of ${JSON.stringify([ } } `; - const result = await processes.run( - process.execPath, - [ - "--experimental-vm-modules", - "--unhandled-rejections=strict", - "--input-type=module", - "--eval", - source, - ], - { timeoutMs: 10_000 }, - ); + const result = await processes.run(process.execPath, [ + "--experimental-vm-modules", + "--unhandled-rejections=strict", + "--input-type=module", + "--eval", + source, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); }, diff --git a/tests/boundary/process/electronLaunchRejections.test.ts b/tests/boundary/process/electronLaunchRejections.test.ts index 0c098b518d..9e9c8981f4 100644 --- a/tests/boundary/process/electronLaunchRejections.test.ts +++ b/tests/boundary/process/electronLaunchRejections.test.ts @@ -140,16 +140,12 @@ const run = async ( executable = "/not/launched", root = "/", ) => - processes.run( - process.execPath, - [ - "--input-type=module", - "--eval", - script, - JSON.stringify([scenario, executable, root]), - ], - { timeoutMs: 150_000 }, - ); + processes.run(process.execPath, [ + "--input-type=module", + "--eval", + script, + JSON.stringify([scenario, executable, root]), + ]); const outcome = (result: { readonly stdout: string }): Outcome => JSON.parse(result.stdout.trim().split("\n").at(-1) ?? "") as Outcome; diff --git a/tests/boundary/process/fixtureAsyncFailures.test.ts b/tests/boundary/process/fixtureAsyncFailures.test.ts index 0ffff41141..11a601c810 100644 --- a/tests/boundary/process/fixtureAsyncFailures.test.ts +++ b/tests/boundary/process/fixtureAsyncFailures.test.ts @@ -25,11 +25,13 @@ for (const blocked of [false, true]) { const handoff = `${receipt}.document-created`; await writeFile(mode, "denied-after-handoff"); if (blocked) await mkdir(handoff); - const result = await processes.run( - process.execPath, - [hopper, mode, receipt, "-Y", "controlled-bootstrap"], - { timeoutMs: 5_000 }, - ); + const result = await processes.run(process.execPath, [ + hopper, + mode, + receipt, + "-Y", + "controlled-bootstrap", + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(blocked ? 1 : 2); expect(result.stderr).toContain("Not authorized to send Apple events"); @@ -54,7 +56,6 @@ processTest( const pending = `${endpointPath}.pending`; await writeFile(pending, "previous selected evidence"); const result = await processes.run(process.execPath, [ghidra], { - timeoutMs: 5_000, env: { ...process.env, REA_GHIDRA_FIXTURE_CONFIG: JSON.stringify({ diff --git a/tests/boundary/process/ghidraDecompilerVerifierCleanup.test.ts b/tests/boundary/process/ghidraDecompilerVerifierCleanup.test.ts index 7e181f9880..fa7774d7c7 100644 --- a/tests/boundary/process/ghidraDecompilerVerifierCleanup.test.ts +++ b/tests/boundary/process/ghidraDecompilerVerifierCleanup.test.ts @@ -82,7 +82,6 @@ processTest TMP: temporary, TEMP: temporary, }, - timeoutMs: 15_000, }, ); expect(result.exitCode, result.stderr).toBe(1); diff --git a/tests/boundary/process/ghidraVerifierAssertions.test.ts b/tests/boundary/process/ghidraVerifierAssertions.test.ts index 6f8aebe956..6de924acce 100644 --- a/tests/boundary/process/ghidraVerifierAssertions.test.ts +++ b/tests/boundary/process/ghidraVerifierAssertions.test.ts @@ -57,11 +57,11 @@ processTest( /Ghidra session commitment drifted/, ); `; - const result = await processes.run( - process.execPath, - ["--input-type=module", "--eval", source], - { timeoutMs: 10_000 }, - ); + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + source, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); }, @@ -102,11 +102,11 @@ processTest( result = "EACCES"; await assert.rejects(assertCleanup(coordinates), { code: "EACCES" }); `; - const result = await processes.run( - process.execPath, - ["--input-type=module", "--eval", source], - { timeoutMs: 10_000 }, - ); + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + source, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); }, diff --git a/tests/boundary/process/goVerifierLifetime.test.ts b/tests/boundary/process/goVerifierLifetime.test.ts index de89672cab..b8e1bd6db8 100644 --- a/tests/boundary/process/goVerifierLifetime.test.ts +++ b/tests/boundary/process/goVerifierLifetime.test.ts @@ -48,7 +48,6 @@ processTest TMP: temporary, TEMP: temporary, }, - timeoutMs: 40_000, }, ); expect(result.exitCode, result.stderr + result.stdout).toBe(0); diff --git a/tests/boundary/process/javascriptSourceModes.test.ts b/tests/boundary/process/javascriptSourceModes.test.ts index 64c15a2d84..d3b9df2caa 100644 --- a/tests/boundary/process/javascriptSourceModes.test.ts +++ b/tests/boundary/process/javascriptSourceModes.test.ts @@ -15,13 +15,13 @@ for (const source of [ const commonjs = await processes.run( process.execPath, ["--check", "--input-type=commonjs"], - { input: source, timeoutMs: 5_000 }, + { input: source }, ); expect(commonjs.exitCode).toBe(0); const module = await processes.run( process.execPath, ["--check", "--input-type=module"], - { input: source, timeoutMs: 5_000 }, + { input: source }, ); expect(module.exitCode).not.toBe(0); diff --git a/tests/boundary/process/numericLimitGuardPermission.test.ts b/tests/boundary/process/numericLimitGuardPermission.test.ts index f3a9fbe09c..825b039c3b 100644 --- a/tests/boundary/process/numericLimitGuardPermission.test.ts +++ b/tests/boundary/process/numericLimitGuardPermission.test.ts @@ -32,7 +32,7 @@ processTest.skipIf( processes.run( process.execPath, ["scripts/verify/rea-numeric-limits.mjs"], - { cwd: root, timeoutMs: 20_000 }, + { cwd: root }, ); const healthy = await run(); expect(healthy.exitCode, healthy.stderr).toBe(0); diff --git a/tests/boundary/process/packageNpmInvocation.test.ts b/tests/boundary/process/packageNpmInvocation.test.ts index f5150975e5..ec1e5b37bf 100644 --- a/tests/boundary/process/packageNpmInvocation.test.ts +++ b/tests/boundary/process/packageNpmInvocation.test.ts @@ -39,7 +39,7 @@ processTest( const result = await processes.run( process.execPath, ["--input-type=module", "--eval", program], - { windowsHide: true, timeoutMs: 20_000 }, + { windowsHide: true }, ); expect(result.exitCode, result.stderr).toBe(0); expect(result.signal, result.stderr).toBeNull(); @@ -95,7 +95,7 @@ windowsIt( const result = await processes.run( node, ["--input-type=module", "--eval", program], - { env: environment, windowsHide: true, timeoutMs: 20_000 }, + { env: environment, windowsHide: true }, ); expect(result.exitCode, result.stderr).toBe(0); expect(result.signal, result.stderr).toBeNull(); diff --git a/tests/boundary/process/packageSetupTerminalOwnership.test.ts b/tests/boundary/process/packageSetupTerminalOwnership.test.ts index 2398f37d38..781f088da6 100644 --- a/tests/boundary/process/packageSetupTerminalOwnership.test.ts +++ b/tests/boundary/process/packageSetupTerminalOwnership.test.ts @@ -174,7 +174,6 @@ process.stdout.write(JSON.stringify({ ready, liveAtRemoval, configurationAtRemov ); try { const result = await processes.run(process.execPath, [driver], { - timeoutMs: 25_000, cwd: process.cwd(), env: { ...process.env, diff --git a/tests/boundary/process/packageVerifierCommandOwnership.test.ts b/tests/boundary/process/packageVerifierCommandOwnership.test.ts index c0d5ab8d3d..a435e50443 100644 --- a/tests/boundary/process/packageVerifierCommandOwnership.test.ts +++ b/tests/boundary/process/packageVerifierCommandOwnership.test.ts @@ -41,7 +41,7 @@ processTest.skipIf(process.platform === "win32")( console.log(JSON.stringify(await completeVerifierRun(run))); `, ], - { env: { ...process.env, TSX_DISABLE_CACHE: "1" }, timeoutMs: 20_000 }, + { env: { ...process.env, TSX_DISABLE_CACHE: "1" } }, ); expect(result.exitCode, result.stderr).toBe(0); expect(JSON.parse(result.stdout)).toMatchObject({ @@ -220,7 +220,6 @@ process.stdout.write(JSON.stringify({ pids, liveAtRemoval, rootRemoved: !existsS ); try { const result = await processes.run(process.execPath, [driver], { - timeoutMs: 20_000, env: { ...process.env, npm_execpath: entry, diff --git a/tests/boundary/process/providerVerifierCleanup.test.ts b/tests/boundary/process/providerVerifierCleanup.test.ts index c96b3cccb2..b543d38db2 100644 --- a/tests/boundary/process/providerVerifierCleanup.test.ts +++ b/tests/boundary/process/providerVerifierCleanup.test.ts @@ -149,7 +149,6 @@ processTest.for(cases)( REA_TEST_VERIFIER_CLOSE_FAIL: closeFails ? "1" : "0", REA_TEST_VERIFIER_CLOSE_LOG: fixture.closeLog, }, - timeoutMs: 10_000, }, ); diff --git a/tests/boundary/process/publishedPackageNpmSelection.test.ts b/tests/boundary/process/publishedPackageNpmSelection.test.ts index e5881b33ea..34b32212b2 100644 --- a/tests/boundary/process/publishedPackageNpmSelection.test.ts +++ b/tests/boundary/process/publishedPackageNpmSelection.test.ts @@ -120,7 +120,7 @@ processTest.skipIf(process.platform === "win32").for([true, false])( const result = await processes.run( process.execPath, ["--import", hook, "scripts/verify-published-package.mjs", "6.9.0"], - { cwd: controller, env, timeoutMs: 20_000 }, + { cwd: controller, env }, ); expect(result.exitCode, result.stderr).toBe(0); const calls = z diff --git a/tests/boundary/process/releaseArtifactCanary.test.ts b/tests/boundary/process/releaseArtifactCanary.test.ts index 0850fe06e4..2257ce8508 100644 --- a/tests/boundary/process/releaseArtifactCanary.test.ts +++ b/tests/boundary/process/releaseArtifactCanary.test.ts @@ -113,7 +113,7 @@ if (process.argv[2] === "mcp") { const result = await processes.run( "npm", ["pack", "--ignore-scripts", "--json"], - { cwd: candidate, timeoutMs: 30_000 }, + { cwd: candidate }, ); expect(result.exitCode, result.stderr).toBe(0); return result; diff --git a/tests/boundary/process/sourceMapUncHosts.test.ts b/tests/boundary/process/sourceMapUncHosts.test.ts index 99953509f8..e09bfec02a 100644 --- a/tests/boundary/process/sourceMapUncHosts.test.ts +++ b/tests/boundary/process/sourceMapUncHosts.test.ts @@ -68,18 +68,14 @@ processTest.for(cases)( const sourceRoot = "root" in scenario ? scenario.root : null; // Import the built production helper in a child. A native assertion must // become a failed signal assertion here, never terminate the Vitest worker. - const result = await processes.run( - process.execPath, - [ - "--input-type=module", - "--eval", - `import { resolveJavaScriptSourceMapReferences } from "./dist/domain/javascript/javascriptSourceMapPaths.js"; + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + `import { resolveJavaScriptSourceMapReferences } from "./dist/domain/javascript/javascriptSourceMapPaths.js"; const [source, root] = JSON.parse(process.argv[1]); console.log(JSON.stringify(resolveJavaScriptSourceMapReferences([source], root, "maps/main.js.map")[0]));`, - JSON.stringify([scenario.source, sourceRoot]), - ], - { timeoutMs: 10_000 }, - ); + JSON.stringify([scenario.source, sourceRoot]), + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); expect(JSON.parse(result.stdout)).toMatchObject({ diff --git a/tests/boundary/process/spawnArgumentLimit.test.ts b/tests/boundary/process/spawnArgumentLimit.test.ts index 78e64518ac..e7bd6472d3 100644 --- a/tests/boundary/process/spawnArgumentLimit.test.ts +++ b/tests/boundary/process/spawnArgumentLimit.test.ts @@ -66,7 +66,7 @@ processTest( const result = await processes.run( process.execPath, ["-e", "process.exit(0)", file], - { env: minimalSpawnEnvironment(), timeoutMs: 15_000 }, + { env: minimalSpawnEnvironment() }, ); expect(result.exitCode).toBe(0); }, @@ -161,7 +161,7 @@ processTest.skipIf(process.platform !== "linux")( processes.run( process.execPath, ["-e", script, "x".repeat(payloadBytes)], - { env: environment, timeoutMs: 15_000 }, + { env: environment }, ); if (accepted !== undefined) { const result = await run(accepted); diff --git a/tests/boundary/process/verifierSourceLoading.test.ts b/tests/boundary/process/verifierSourceLoading.test.ts index 3533e96214..f77e224a71 100644 --- a/tests/boundary/process/verifierSourceLoading.test.ts +++ b/tests/boundary/process/verifierSourceLoading.test.ts @@ -25,11 +25,11 @@ module.syncBuiltinESMExports(); const { importVerifierSource } = await import(${scriptUrl("verifier-source-loader.mjs")}); await importVerifierSource(new URL("./src/domain/skillManifest.ts", import.meta.url)); `; - const result = await processes.run( - process.execPath, - ["--input-type=module", "--eval", source], - { timeoutMs: 20_000 }, - ); + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + source, + ]); expect(result.exitCode).toBe(1); expect(result.stderr).toContain("Node.js version pinned in .nvmrc"); expect(result.stderr).toContain("source build and verifier scripts"); @@ -86,7 +86,7 @@ console.log(JSON.stringify({ const result = await processes.run( process.execPath, ["--input-type=module", "--eval", source], - { env: { ...process.env, TSX_DISABLE_CACHE: "1" }, timeoutMs: 20_000 }, + { env: { ...process.env, TSX_DISABLE_CACHE: "1" } }, ); expect(result.exitCode, result.stderr).toBe(0); const observation = z @@ -151,7 +151,7 @@ await withVerifierCleanup(async () => { const result = await processes.run( process.execPath, ["--input-type=module", "--eval", source], - { timeoutMs: 20_000, env: { ...process.env, TSX_DISABLE_CACHE: "1" } }, + { env: { ...process.env, TSX_DISABLE_CACHE: "1" } }, ); expect(result.exitCode, result.stderr).toBe(0); expect(JSON.parse(result.stdout)).toEqual({ @@ -233,7 +233,7 @@ console.log(JSON.stringify({ const result = await processes.run( process.execPath, [workspace.path("probe.mjs")], - { cwd: workspace.root, timeoutMs: 20_000 }, + { cwd: workspace.root }, ); expect(result.exitCode, result.stderr).toBe(0); expect(JSON.parse(result.stdout)).toEqual({ diff --git a/tests/boundary/process/websiteInitialization.test.ts b/tests/boundary/process/websiteInitialization.test.ts index 73b5fa1bf4..683b472198 100644 --- a/tests/boundary/process/websiteInitialization.test.ts +++ b/tests/boundary/process/websiteInitialization.test.ts @@ -36,11 +36,11 @@ const [message, cause] = await reported; assert.equal(message, "Website initialization failed"); assert.equal(cause, globalThis.translationFailure); `; - const result = await processes.run( - process.execPath, - ["--input-type=module", "--eval", source], - { timeoutMs: 10_000 }, - ); + const result = await processes.run(process.execPath, [ + "--input-type=module", + "--eval", + source, + ]); expect(result.signal, result.stderr).toBeNull(); expect(result.exitCode, result.stderr).toBe(0); }, diff --git a/tests/boundary/process/windowsExecutableSelection.test.ts b/tests/boundary/process/windowsExecutableSelection.test.ts index bc508b223e..dd057304bb 100644 --- a/tests/boundary/process/windowsExecutableSelection.test.ts +++ b/tests/boundary/process/windowsExecutableSelection.test.ts @@ -74,7 +74,6 @@ processTest.skipIf(process.platform !== "win32")( ["--input-type=module", "--eval", program], { cwd: working, - timeoutMs: 30_000, windowsHide: true, }, ); @@ -133,7 +132,6 @@ processTest.skipIf(process.platform !== "win32")( ["--input-type=module", "--eval", program], { cwd: root, - timeoutMs: 40_000, windowsHide: true, }, ); diff --git a/tests/boundary/process/windowsNativeArtifactVerifier.test.ts b/tests/boundary/process/windowsNativeArtifactVerifier.test.ts index 9bd0ce1ac3..1289dbca4b 100644 --- a/tests/boundary/process/windowsNativeArtifactVerifier.test.ts +++ b/tests/boundary/process/windowsNativeArtifactVerifier.test.ts @@ -97,7 +97,7 @@ for (const mode of ["preinstall source", "packed source-free"]) { processes.run( process.execPath, [join(root, "scripts/verify-windows-native-artifact.mjs")], - { cwd: root, timeoutMs: 10_000 }, + { cwd: root }, ); const valid = await run(); expect(valid.exitCode, valid.stderr).toBe(0); diff --git a/tests/boundary/process/windowsNativeInspectionWorkspace.test.ts b/tests/boundary/process/windowsNativeInspectionWorkspace.test.ts index 8d70ecf2cd..e9a69ae856 100644 --- a/tests/boundary/process/windowsNativeInspectionWorkspace.test.ts +++ b/tests/boundary/process/windowsNativeInspectionWorkspace.test.ts @@ -39,7 +39,7 @@ processTest( 'assert.equal(typeof loader.systemWindowsNativeAuthority, "function");', ].join("\n"), ], - { cwd: workspace, timeoutMs: 10_000 }, + { cwd: workspace }, ); expect(imported.exitCode, imported.stderr).toBe(0); expect(await realpath(join(workspace, "node_modules"))).toBe(