chore: remove batch stdin source and related docs

Author: thymikee

README.md

@@ -60,15 +60,6 @@ agent-device batch \
   --json
 ```
 
-```bash
-cat /tmp/batch-steps.json | agent-device batch \
-  --session sim \
-  --platform ios \
-  --udid 00008150-001849640CF8401C \
-  --steps-stdin \
-  --json
-```
-
 Small inline payloads are also supported:
 
 ```bash
@@ -97,7 +88,7 @@ Agent usage guidelines:
 - Keep each batch to one screen-local workflow.
 - Add sync guards (`wait`, `is exists`) after mutating steps (`open`, `click`, `fill`, `swipe`).
 - Treat refs/snapshot assumptions as stale after UI mutations.
-- Prefer `--steps-file` or `--steps-stdin` over inline JSON for reliability.
+- Prefer `--steps-file` over inline JSON for reliability.
 - Keep batches moderate (about 5-20 steps) and stop on first error.
 
 ## CLI Usage
@@ -172,7 +163,6 @@ Flags:
 - `--json` for structured output
 - `--steps <json>` batch: JSON array of steps
 - `--steps-file <path>` batch: read step JSON from file
-- `--steps-stdin` batch: read step JSON from stdin
 - `--on-error stop` batch: stop when a step fails
 - `--max-steps <n>` batch: max allowed steps per request
 

skills/agent-device/SKILL.md

@@ -168,15 +168,6 @@ agent-device batch \
   --json
 ```
 
-```bash
-cat /tmp/batch-steps.json | agent-device batch \
-  --session sim \
-  --platform ios \
-  --udid 00008150-001849640CF8401C \
-  --steps-stdin \
-  --json
-```
-
 Inline JSON works for small payloads:
 
 ```bash
@@ -199,7 +190,7 @@ Batch best practices for agents:
 - Batch one screen-local flow at a time.
 - Add sync guards (`wait`, `is exists`) after mutating steps (`open`, `click`, `fill`, `swipe`).
 - Treat prior refs/snapshot assumptions as stale after UI mutations.
-- Prefer `--steps-file` or `--steps-stdin` over inline JSON.
+- Prefer `--steps-file` over inline JSON.
 - Keep batches moderate (about 5-20 steps).
 - Use failure context (`step`, `partialResults`) to replan from the failed step.
 

skills/agent-device/references/batching-for-agents.md

@@ -20,12 +20,6 @@ From file:
 agent-device batch --session sim --platform ios --steps-file /tmp/batch-steps.json --json
 ```
 
-From stdin:
-
-```bash
-cat /tmp/batch-steps.json | agent-device batch --session sim --platform ios --steps-stdin --json
-```
-
 Inline (small payloads only):
 
 ```bash

src/__tests__/cli-batch.test.ts

@@ -24,7 +24,6 @@ type RunResult = {
 
 async function runCliCapture(
   argv: string[],
-  options: { forceStdinTty?: boolean } = {},
 ): Promise<RunResult> {
   let stdout = '';
   let stderr = '';
@@ -34,7 +33,6 @@ async function runCliCapture(
   const originalExit = process.exit;
   const originalStdoutWrite = process.stdout.write.bind(process.stdout);
   const originalStderrWrite = process.stderr.write.bind(process.stderr);
-  const stdinDescriptor = Object.getOwnPropertyDescriptor(process.stdin, 'isTTY');
 
   (process as any).exit = ((nextCode?: number) => {
     throw new ExitSignal(nextCode ?? 0);
@@ -47,12 +45,6 @@ async function runCliCapture(
     stderr += String(chunk);
     return true;
   }) as typeof process.stderr.write;
-  if (options.forceStdinTty !== undefined) {
-    Object.defineProperty(process.stdin, 'isTTY', {
-      configurable: true,
-      value: options.forceStdinTty,
-    });
-  }
 
   const sendToDaemon = async (req: Omit<DaemonRequest, 'token'>): Promise<DaemonResponse> => {
     calls.push(req);
@@ -68,13 +60,6 @@ async function runCliCapture(
     process.exit = originalExit;
     process.stdout.write = originalStdoutWrite;
     process.stderr.write = originalStderrWrite;
-    if (options.forceStdinTty !== undefined) {
-      if (stdinDescriptor) {
-        Object.defineProperty(process.stdin, 'isTTY', stdinDescriptor);
-      } else {
-        delete (process.stdin as any).isTTY;
-      }
-    }
   }
 
   return { code, stdout, stderr, calls };
@@ -113,10 +98,3 @@ test('batch --steps-file parses file payload', async () => {
   assert.equal(req.command, 'batch');
   assert.equal((req.flags?.batchSteps ?? [])[0]?.command, 'wait');
 });
-
-test('batch --steps-stdin fails fast when stdin is TTY', async () => {
-  const result = await runCliCapture(['batch', '--steps-stdin'], { forceStdinTty: true });
-  assert.equal(result.code, 1);
-  assert.equal(result.calls.length, 0);
-  assert.match(result.stderr, /--steps-stdin requires piped JSON input/);
-});

src/cli.ts

@@ -68,7 +68,6 @@ export async function runCli(argv: string[], deps: CliDeps = DEFAULT_CLI_DEPS):
       const batchFlags = { ...daemonFlags, batchSteps };
       delete (batchFlags as Record<string, unknown>).steps;
       delete (batchFlags as Record<string, unknown>).stepsFile;
-      delete (batchFlags as Record<string, unknown>).stepsStdin;
 
       const response = await deps.sendToDaemon({
         session: sessionName,
@@ -296,14 +295,6 @@ async function readBatchSteps(flags: ReturnType<typeof parseArgs>['flags']): Pro
     raw = flags.steps;
   } else if (flags.stepsFile) {
     raw = fs.readFileSync(flags.stepsFile, 'utf8');
-  } else if (flags.stepsStdin) {
-    if (process.stdin.isTTY) {
-      throw new AppError(
-        'INVALID_ARGS',
-        'batch --steps-stdin requires piped JSON input on stdin.',
-      );
-    }
-    raw = await readStdin();
   }
   let parsed: unknown;
   try {
@@ -340,14 +331,6 @@ async function readBatchSteps(flags: ReturnType<typeof parseArgs>['flags']): Pro
   return steps;
 }
 
-async function readStdin(): Promise<string> {
-  const chunks: Buffer[] = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk));
-  }
-  return Buffer.concat(chunks).toString('utf8');
-}
-
 function isDaemonStartupFailure(error: AppError): boolean {
   if (error.code !== 'COMMAND_FAILED') return false;
   if (error.details?.kind === 'daemon_startup_failed') return true;

src/utils/args.ts

@@ -99,12 +99,11 @@ export function parseArgs(argv: string[], options?: ParseArgsOptions): ParsedArg
   if (command === 'batch') {
     const stepSourceCount =
       (flags.steps ? 1 : 0) +
-      (flags.stepsFile ? 1 : 0) +
-      (flags.stepsStdin ? 1 : 0);
+      (flags.stepsFile ? 1 : 0);
     if (stepSourceCount !== 1) {
       throw new AppError(
         'INVALID_ARGS',
-        'batch requires exactly one step source: --steps, --steps-file, or --steps-stdin.',
+        'batch requires exactly one step source: --steps or --steps-file.',
       );
     }
   }

src/utils/command-schema.ts

@@ -26,7 +26,6 @@ export type CliFlags = {
   replayUpdate?: boolean;
   steps?: string;
   stepsFile?: string;
-  stepsStdin?: boolean;
   batchOnError?: 'stop';
   batchMaxSteps?: number;
   batchSteps?: Array<{
@@ -246,13 +245,6 @@ export const FLAG_DEFINITIONS: readonly FlagDefinition[] = [
     usageLabel: '--steps-file <path>',
     usageDescription: 'Batch: read steps JSON from file',
   },
-  {
-    key: 'stepsStdin',
-    names: ['--steps-stdin'],
-    type: 'boolean',
-    usageLabel: '--steps-stdin',
-    usageDescription: 'Batch: read steps JSON from stdin',
-  },
   {
     key: 'batchOnError',
     names: ['--on-error'],
@@ -435,10 +427,10 @@ export const COMMAND_SCHEMAS: Record<string, CommandSchema> = {
     skipCapabilityCheck: true,
   },
   batch: {
-    usageOverride: 'batch [--steps <json> | --steps-file <path> | --steps-stdin]',
+    usageOverride: 'batch [--steps <json> | --steps-file <path>]',
     description: 'Execute multiple commands in one daemon request',
     positionalArgs: [],
-    allowedFlags: ['steps', 'stepsFile', 'stepsStdin', 'batchOnError', 'batchMaxSteps', 'out'],
+    allowedFlags: ['steps', 'stepsFile', 'batchOnError', 'batchMaxSteps', 'out'],
     skipCapabilityCheck: true,
   },
   press: {

website/docs/docs/batching.md

@@ -21,17 +21,6 @@ agent-device batch \
   --json
 ```
 
-From stdin:
-
-```bash
-cat /tmp/batch-steps.json | agent-device batch \
-  --session sim \
-  --platform ios \
-  --udid 00008150-001849640CF8401C \
-  --steps-stdin \
-  --json
-```
-
 Inline for small payloads:
 
 ```bash
@@ -107,7 +96,7 @@ Failure:
 - Batch only one related screen flow at a time.
 - After mutating steps (`open`, `click`, `fill`, `swipe`), add a sync guard (`wait`, `is exists`) before critical reads.
 - Treat prior refs/snapshots as stale after UI changes.
-- Prefer `--steps-file` or `--steps-stdin` over inline JSON.
+- Prefer `--steps-file` over inline JSON.
 - Keep batches moderate (about 5-20 steps).
 - Replan from the failing step using `details.step` and `details.partialResults`.
 

website/docs/docs/commands.md

@@ -92,7 +92,6 @@ See [Replay & E2E (Experimental)](/docs/replay-e2e) for recording and CI workflo
 
 ```bash
 agent-device batch --steps-file /tmp/batch-steps.json --json
-agent-device batch --steps-stdin --json
 agent-device batch --steps '[{"command":"open","positionals":["settings"]}]'
 ```