Merge pull request #208 from shaoohh/fix/cross-platform-stdin-input

fix: support stdin-backed file options on Windows

Author: NianJiuZst

src/commands/text/chat.ts

@@ -16,6 +16,7 @@ import type {
 } from '../../types/api';
 import { readFileSync } from 'fs';
 import { isInteractive } from '../../utils/env';
+import { readTextFromPathOrStdin } from '../../utils/fs';
 import { promptText, failIfMissing } from '../../utils/prompt';
 
 // ---------------------------------------------------------------------------
@@ -104,9 +105,7 @@ function parseMessages(flags: GlobalFlags): ParsedMessages {
 
   if (flags.messagesFile) {
     const filePath = flags.messagesFile as string;
-    const raw = filePath === '-'
-      ? readFileSync('/dev/stdin', 'utf-8')
-      : readFileSync(filePath, 'utf-8');
+    const raw = readTextFromPathOrStdin(filePath);
     const parsed = JSON.parse(raw) as Array<{ role: string; content: string | ContentBlock[] }>;
     for (const m of parsed) {
       if (m.role === 'system') {

src/utils/fs.ts

@@ -1,5 +1,11 @@
 import { readFileSync } from 'fs';
 
+export function resolveTextInput(path: string): string | number {
+  // File descriptor 0 is stdin on every platform supported by Node. `/dev/stdin`
+  // only exists on POSIX systems, so using it breaks `--*-file -` on Windows.
+  return path === '-' ? 0 : path;
+}
+
 export function readTextFromPathOrStdin(path: string): string {
-  return readFileSync(path === '-' ? '/dev/stdin' : path, 'utf-8');
+  return readFileSync(resolveTextInput(path), 'utf-8');
 }

test/utils/fs.test.ts

@@ -0,0 +1,24 @@
+import { describe, expect, it } from 'bun:test';
+import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { readTextFromPathOrStdin, resolveTextInput } from '../../src/utils/fs';
+
+describe('readTextFromPathOrStdin', () => {
+  it('uses file descriptor zero for the stdin marker on every platform', () => {
+    expect(resolveTextInput('-')).toBe(0);
+  });
+
+  it('keeps ordinary file paths unchanged and reads their text', () => {
+    const dir = mkdtempSync(join(tmpdir(), 'mmx-fs-test-'));
+    const path = join(dir, 'input.txt');
+    writeFileSync(path, 'hello from a file', 'utf-8');
+
+    try {
+      expect(resolveTextInput(path)).toBe(path);
+      expect(readTextFromPathOrStdin(path)).toBe('hello from a file');
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+});