ReviewOS

also looking at this

craft-native/craft

fix(mobile-e2e): launch iOS apps without a pty, so a cold simulator cannot eat the budget

#212
Merged glennmichael123 wants to merge fix/mobile-e2e-ios-launch into main
5 files +74 -23
scripts/mobile-e2e/support.tsmodified+3-0
Changes to scripts/mobile-e2e/support.ts
@@ -10,6 +10,8 @@ export interface CommandOptions {
1010 /** Record the failure and carry on, for teardown steps whose failure is not the story. */
1111 allowFailure?: boolean
1212 stdin?: string
13 /** Added to the inherited environment, not a replacement for it. */
14 env?: Record<string, string>
1315 /**
1416 * Write raw stdout bytes here instead of returning them as text.
1517 *
@@ -36,6 +38,7 @@ export interface CommandResult {
3638export async function command(argv: string[], options: CommandOptions = {}): Promise<CommandResult> {
3739 const spawned = Bun.spawn(argv, {
3840 cwd: options.cwd,
41 env: options.env ? { ...process.env, ...options.env } : undefined,
3942 stdin: options.stdin === undefined ? 'ignore' : new TextEncoder().encode(options.stdin),
4043 stdout: 'pipe',
4144 stderr: 'pipe',