feat: support OCLIF_STDIN_TIMEOUT_MS env var for stdin read timeout @W-24151668@

The hard-coded 10ms timeout in readStdin() causes intermittent failures
when piping input on loaded systems or CI runners. Allow users to override
it via the OCLIF_STDIN_TIMEOUT_MS environment variable, defaulting to
the current 10ms when not set.

Author: WillieRuemmele

src/parser/parse.ts

@@ -73,7 +73,7 @@ export const readStdin = async (): Promise<null | string> => {
     const lines: string[] = []
     const ac = new AbortController()
     const {signal} = ac
-    const timeout = setTimeout(() => ac.abort(), 10)
+    const timeout = setTimeout(() => ac.abort(), Number(process.env.OCLIF_STDIN_TIMEOUT_MS) || 10)
 
     const rl = createInterface({
       input: stdin,
@@ -749,3 +749,4 @@ export class Parser<
     return flagTokenMap
   }
 }
+// test

test/parser/parse.test.ts

@@ -2185,6 +2185,79 @@ See more help with --help`)
   })
 })
 
+describe('readStdin', () => {
+  let originalEnv: string | undefined
+
+  beforeEach(() => {
+    originalEnv = process.env.OCLIF_STDIN_TIMEOUT_MS
+    delete globalThis.oclif?.stdinCache
+  })
+
+  afterEach(() => {
+    if (originalEnv === undefined) {
+      delete process.env.OCLIF_STDIN_TIMEOUT_MS
+    } else {
+      process.env.OCLIF_STDIN_TIMEOUT_MS = originalEnv
+    }
+
+    sinon.restore()
+  })
+
+  it('should use OCLIF_STDIN_TIMEOUT_MS env var for timeout', async () => {
+    process.env.OCLIF_STDIN_TIMEOUT_MS = '5000'
+    const spy = sinon.spy(globalThis, 'setTimeout')
+
+    Object.defineProperty(process.stdin, 'isTTY', {configurable: true, value: undefined})
+    try {
+      await parser.readStdin()
+    } catch {
+      // may abort, that's fine
+    }
+
+    const stdinTimeoutCall = spy.getCalls().find((call) => {
+      const delay = call.args[1]
+      return delay === 5000
+    })
+    expect(stdinTimeoutCall).to.not.be.undefined
+  })
+
+  it('should default to 10ms when OCLIF_STDIN_TIMEOUT_MS is not set', async () => {
+    delete process.env.OCLIF_STDIN_TIMEOUT_MS
+    const spy = sinon.spy(globalThis, 'setTimeout')
+
+    Object.defineProperty(process.stdin, 'isTTY', {configurable: true, value: undefined})
+    try {
+      await parser.readStdin()
+    } catch {
+      // may abort, that's fine
+    }
+
+    const stdinTimeoutCall = spy.getCalls().find((call) => {
+      const delay = call.args[1]
+      return delay === 10
+    })
+    expect(stdinTimeoutCall).to.not.be.undefined
+  })
+
+  it('should default to 10ms when OCLIF_STDIN_TIMEOUT_MS is invalid', async () => {
+    process.env.OCLIF_STDIN_TIMEOUT_MS = 'not-a-number'
+    const spy = sinon.spy(globalThis, 'setTimeout')
+
+    Object.defineProperty(process.stdin, 'isTTY', {configurable: true, value: undefined})
+    try {
+      await parser.readStdin()
+    } catch {
+      // may abort, that's fine
+    }
+
+    const stdinTimeoutCall = spy.getCalls().find((call) => {
+      const delay = call.args[1]
+      return delay === 10
+    })
+    expect(stdinTimeoutCall).to.not.be.undefined
+  })
+})
+
 describe('allowStdin', () => {
   const stdinValue = 'x'
   const stdinPromise = new Promise<null | string>((resolve) => {