refactor(shell-quote): one implementation, reached through the runner host port

packages/platform-android/src/perf-native-artifacts.ts

@@ -1,6 +1,7 @@
 import path from 'node:path';
 import type { DeviceInfo } from '@agent-device/kernel/device';
 import { AppError } from '@agent-device/kernel/errors';
+import { shellQuote } from '@agent-device/host-kit/command';
 import { sleep } from '@agent-device/host-kit/retry';
 import { resolveAndroidAdbExecutor, type AndroidAdbExecutor } from './adb-executor.ts';
 import { annotateAndroidNativePerfError } from './perf-native-errors.ts';
@@ -121,10 +122,6 @@ export async function readFileSize(
   }
 }
 
-export function shellQuote(value: string): string {
-  return `'${value.replaceAll("'", `'\\''`)}'`;
-}
-
 async function stopAndroidBackgroundTool(
   adb: AndroidAdbExecutor,
   session: AndroidNativePerfSession,

packages/platform-android/src/perf-native-simpleperf.ts

@@ -1,11 +1,11 @@
 import type { DeviceInfo } from '@agent-device/kernel/device';
 import { AppError } from '@agent-device/kernel/errors';
+import { shellQuote } from '@agent-device/host-kit/command';
 import { resolveAndroidAdbExecutor, type AndroidAdbExecutor } from './adb-executor.ts';
 import {
   buildAndroidNativeRemotePath,
   cleanupAndroidRemotePath,
   readFileSize,
-  shellQuote,
   stopAndroidNativePerfSession,
   writeJsonArtifact,
 } from './perf-native-artifacts.ts';

packages/platform-apple/src/core/runner-host.ts

@@ -8,6 +8,7 @@ import {
   runCmdBackground,
   runCmdStreaming,
   runCmdSync,
+  shellQuote,
 } from '@agent-device/host-kit/command';
 import { emitDiagnostic, withDiagnosticTimer } from '@agent-device/host-kit/diagnostics';
 import {
@@ -57,6 +58,7 @@ export const appleRunnerHost: AppleRunnerHost = {
   runCmdBackground,
   requireExecSuccess,
   isCommandTimeoutError,
+  shellQuote,
   emitDiagnostic,
   withDiagnosticTimer,
   retryWithPolicy,

packages/platform-apple/src/runner/host.ts

@@ -167,6 +167,8 @@ export type AppleRunnerHost = {
   ): ExecResult;
   /** True only for the error the exec layer raises when it killed a command at `timeoutMs`. */
   isCommandTimeoutError(error: unknown): boolean;
+  /** Single-quote one value for a POSIX shell fragment (hints and remote commands). */
+  shellQuote(value: string): string;
   // Diagnostics (@agent-device/host-kit/diagnostics)
   emitDiagnostic(event: DiagnosticEventInput): void;
   withDiagnosticTimer<T>(
@@ -297,6 +299,7 @@ export const requireExecSuccess: AppleRunnerHost['requireExecSuccess'] = (result
   requireHost().requireExecSuccess(result, message, extra);
 export const isCommandTimeoutError: AppleRunnerHost['isCommandTimeoutError'] = (error) =>
   requireHost().isCommandTimeoutError(error);
+export const shellQuote: AppleRunnerHost['shellQuote'] = (value) => requireHost().shellQuote(value);
 export const emitDiagnostic: AppleRunnerHost['emitDiagnostic'] = (event) =>
   requireHost().emitDiagnostic(event);
 export const withDiagnosticTimer = <T>(

packages/platform-apple/src/runner/runner-lease.ts

@@ -12,6 +12,7 @@ import {
   readProcessStartTime,
   classifyOwnerLiveness,
   leaseOwnerStateDir,
+  shellQuote,
 } from './host.ts';
 import { AppError } from '@agent-device/kernel/errors';
 import type { DeviceInfo } from '@agent-device/kernel/device';
@@ -289,10 +290,6 @@ function formatEnvAssignment(name: string, value: string): string {
   return `${name}=${shellQuote(value)}`;
 }
 
-function shellQuote(value: string): string {
-  return `'${value.replaceAll("'", String.raw`'\''`)}'`;
-}
-
 // A lease whose owner process is gone but whose runner may still be running:
 // the adoption path probes it instead of killing it. Detached leases (graceful
 // daemon shutdown rewrote the token) classify as stale too once the owner pid