docs: keep tool directives only across the touched tree

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018VngeKZH6zBuJzNBk5YzUH

Author: claude

packages/capture-kit/src/tmp-dir.fixtures.ts

@@ -1,20 +1,12 @@
-// Package tests cannot reach root test helpers (R11); fixture copy of tmp-dir.ts.
 import fs from 'node:fs';
 import fsPromises from 'node:fs/promises';
 import os from 'node:os';
 import path from 'node:path';
 
-/**
- * Creates a fresh scratch directory for one test. Cleanup is automatic: the
- * unit suite redirects TMPDIR to a per-run directory (scripts/vitest-tmpdir-global-setup.ts)
- * that gets removed in one recursive rm after every worker finishes, so
- * individual tests never need their own afterEach/afterAll for this.
- */
 export async function mkdtempForTest(prefix: string): Promise<string> {
   return fsPromises.mkdtemp(path.join(os.tmpdir(), prefix));
 }
 
-/** Sync counterpart of {@link mkdtempForTest}, for setup code that can't await. */
 export function mkdtempForTestSync(prefix: string): string {
   return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
 }

packages/kernel/src/scoped-provider.ts

@@ -26,10 +26,6 @@ export function createScopedProvider<TProvider, TInput = TProvider>(
   };
 }
 
-/**
- * Wraps every method call on `target` in `runScope`, so leaf code below the
- * method resolves scope-injected state instead of the local default.
- */
 export function withMethodScope<T extends object>(
   target: T,
   runScope: <R>(task: () => Promise<R>) => Promise<R>,

src/__tests__/test-utils/host-process-mock.ts

@@ -2,35 +2,13 @@ type HostProcessModule = typeof import('@agent-device/host-kit/process');
 type HostProcessIdentityObservation =
   import('@agent-device/host-kit/process').HostProcessIdentityObservation;
 
-/**
- * `vi.mock` factory for the host-kit `exec` seam in tests that seed a "live"
- * owner from the test process's own identity (readCurrentOwnerIdentity and
- * friends) and later have production code re-read it during classification.
- *
- * readProcessStartTime shells out to `ps` with a 1s timeout; under full-suite
- * CPU contention a re-read can miss that deadline and return null, mismatching
- * the seeded value and flipping a genuinely-live owner to 'owner-process-dead'.
- * This factory assigns our own pid a stable synthetic start time, then serves
- * that value so every read is self-consistent without a subprocess. Other pids read as null, same
- * as a real `ps` miss. isProcessAlive stays real (a plain `kill(pid, 0)`
- * syscall, not a subprocess), so fabricated dead-pid fixtures still classify
- * as dead through that non-flaky check.
- *
- * Usage: `vi.mock('@agent-device/host-kit/process', async (importOriginal) =>
- * (await import('<path>/test-utils/host-process-mock.ts')).pinOwnProcessStartTime(importOriginal))`
- */
-// Consumed by three suites, but only through `(await import(...)).pinOwnProcessStartTime`
-// inside `vi.mock` factories — vitest hoists those above static imports, so the dynamic
-// form is required and fallow cannot trace the consumers statically.
 export async function pinOwnProcessStartTime(
   importOriginal: () => Promise<HostProcessModule>,
 ): Promise<HostProcessModule> {
   const actual = await importOriginal();
   const ownStartTime = 'test-process-start-time';
   return {
     ...actual,
-    // The seam mock cannot reach readCurrentOwnerIdentity's internal
-    // readProcessStartTime call, so the recorded identity is pinned here too.
     readCurrentOwnerIdentity: () => ({ pid: process.pid, startTime: ownStartTime }),
     readProcessStartTime: (pid: number) => {
       if (pid !== process.pid) return null;

src/platforms/web/agent-browser-lifecycle.test.ts

@@ -18,8 +18,6 @@ vi.mock('@agent-device/host-kit/process', async (importOriginal) => {
   const actual = await importOriginal<typeof import('@agent-device/host-kit/process')>();
   return {
     ...actual,
-    // The port mock cannot reach listHostProcesses' internal runCmd call, so
-    // the ps read is routed through its runCommand injection point instead.
     listHostProcesses: async (options: Parameters<typeof actual.listHostProcesses>[0]) =>
       await actual.listHostProcesses({ ...options, runCommand: runCmdMock }),
     readProcessStartTime: (pid: number) => `start-${pid}`,