test: Stabilize release filesystem lifecycle tests

Cancel pending lifecycle timers during test resets and avoid using a fake
result-bundle owner PID that can exist on CI runners. This keeps release
unit tests deterministic across macOS runner process tables.

Co-Authored-By: Codex <noreply@openai.com>

Author: cameroncooke

src/utils/__tests__/test-common.test.ts

@@ -14,6 +14,7 @@ import {
   setXcodeBuildMCPAppDirOverrideForTests,
 } from '../log-paths.ts';
 import { setRuntimeInstanceForTests } from '../runtime-instance.ts';
+import { resetWorkspaceFilesystemLifecycleStateForTests } from '../workspace-filesystem-lifecycle.ts';
 
 function createSuccessfulCommandResponse(): CommandResponse {
   return {
@@ -115,6 +116,7 @@ describe('createTestExecutor', () => {
   });
 
   afterEach(() => {
+    resetWorkspaceFilesystemLifecycleStateForTests();
     setXcodeBuildMCPAppDirOverrideForTests(null);
     setRuntimeInstanceForTests(null);
     rmSync(tempAppDir, { recursive: true, force: true });

src/utils/__tests__/workspace-filesystem-lifecycle.test.ts

@@ -25,6 +25,7 @@ import {
 import { setSimulatorLaunchOsLogRecordActiveOverrideForTests } from '../log-capture/simulator-launch-oslog-registry.ts';
 
 let appDir: string;
+const DEAD_OWNER_PID = 999_999_999;
 
 function writeFileWithMtime(filePath: string, content: string, mtimeMs: number): void {
   mkdirSync(path.dirname(filePath), { recursive: true });
@@ -37,7 +38,7 @@ function managedXcodebuildLogName(name = 'build_sim'): string {
   return `${name}_2026-05-02T12-00-00-000Z_pid123_abcdef12.log`;
 }
 
-function managedResultBundleName(name = 'test', pid = 123): string {
+function managedResultBundleName(name = 'test', pid = DEAD_OWNER_PID): string {
   return `${name}_2026-05-02T12-00-00-000Z_pid${pid}_abcdef12.xcresult`;
 }
 

src/utils/workspace-filesystem-lifecycle.ts

@@ -28,6 +28,7 @@ const FALLBACK_LOCK_DIR_NAME = '.filesystem-lifecycle.lock';
 const runningScheduledSweeps = new Set<string>();
 const lastScheduledAtByScope = new Map<string, number>();
 const lastScheduledAtByPreKey = new Map<string, number>();
+const scheduledSweepTimers = new Set<ReturnType<typeof setTimeout>>();
 
 const HELPER_PID_PATTERN = /(?:^|_)helperpid(\d+)(?:_|\.|$)/g;
 const ISO_TIMESTAMP_PATTERN = '\\d{4}-\\d{2}-\\d{2}T\\d{2}-\\d{2}-\\d{2}-\\d{3}Z';
@@ -638,9 +639,11 @@ export function scheduleWorkspaceFilesystemLifecycleSweep(
         log('warn', `[FilesystemLifecycle] Cleanup failed: ${message}`);
       })
       .finally(() => {
+        scheduledSweepTimers.delete(timer);
         runningScheduledSweeps.delete(scheduleKey);
       });
   }, WORKSPACE_FILESYSTEM_LIFECYCLE_SCHEDULE_DELAY_MS);
+  scheduledSweepTimers.add(timer);
   timer.unref?.();
 }
 
@@ -703,6 +706,10 @@ export function terminateOwnedWorkspaceFilesystemArtifactsSync(): {
 }
 
 export function resetWorkspaceFilesystemLifecycleStateForTests(): void {
+  for (const timer of scheduledSweepTimers) {
+    clearTimeout(timer);
+  }
+  scheduledSweepTimers.clear();
   runningScheduledSweeps.clear();
   lastScheduledAtByScope.clear();
   lastScheduledAtByPreKey.clear();