@plaited/acp-harness 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plaited/acp-harness",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "CLI tool for capturing agent trajectories from ACP-compatible agents",
   "license": "ISC",
   "engines": {

package/src/headless-cli.ts

@@ -235,9 +235,10 @@ const mapToACPUpdate = (update: { type: string; content?: string; title?: string
  * Runs the headless adapter main loop.
  *
  * @param schema - Headless adapter configuration
+ * @param verbose - Whether to show debug output
  */
-const runAdapter = async (schema: HeadlessAdapterConfig): Promise<void> => {
-  const sessions = createSessionManager({ schema })
+const runAdapter = async (schema: HeadlessAdapterConfig, verbose = false): Promise<void> => {
+  const sessions = createSessionManager({ schema, verbose })
   const handlers = createHandlers(schema, sessions)
 
   // Method handlers (requests expect responses)
@@ -349,6 +350,7 @@ export const headless = async (args: string[]): Promise<void> => {
     args,
     options: {
       schema: { type: 'string', short: 's' },
+      verbose: { type: 'boolean', short: 'v' },
       help: { type: 'boolean', short: 'h' },
     },
     allowPositionals: false,
@@ -357,10 +359,11 @@ export const headless = async (args: string[]): Promise<void> => {
   if (values.help) {
     // biome-ignore lint/suspicious/noConsole: CLI help output
     console.log(`
-Usage: acp-harness headless --schema <path>
+Usage: acp-harness headless --schema <path> [--verbose]
 
 Arguments:
   -s, --schema    Path to headless adapter schema (JSON)
+  -v, --verbose   Show constructed commands (for debugging)
   -h, --help      Show this help message
 
 Description:
@@ -421,7 +424,7 @@ Examples:
   }
 
   // Run the adapter
-  await runAdapter(schema)
+  await runAdapter(schema, values.verbose ?? false)
 }
 
 // Allow direct execution

package/src/headless-session-manager.ts

@@ -57,6 +57,8 @@ export type SessionManagerConfig = {
   schema: HeadlessAdapterConfig
   /** Default timeout for operations in ms */
   timeout?: number
+  /** Whether to show debug output (constructed commands) */
+  verbose?: boolean
 }
 
 // ============================================================================
@@ -84,7 +86,7 @@ export type SessionManagerConfig = {
  * @returns Session manager with create, prompt, and cancel methods
  */
 export const createSessionManager = (config: SessionManagerConfig) => {
-  const { schema, timeout = 60000 } = config
+  const { schema, timeout = 60000, verbose = false } = config
   const sessions = new Map<string, Session>()
   const outputParser = createOutputParser(schema)
 
@@ -164,9 +166,10 @@ export const createSessionManager = (config: SessionManagerConfig) => {
         stderr: 'inherit',
       })
 
-      // If using stdin, write the prompt
+      // If using stdin, write the prompt and close stdin
+      // (stream mode spawns new process per turn, so stdin should close after writing)
       if (schema.prompt.stdin && session.process) {
-        writePromptToStdin(session.process, promptText)
+        writePromptToStdin(session.process, promptText, true)
       }
     } else {
       // Subsequent turns: spawn new process with resume flag
@@ -180,9 +183,10 @@ export const createSessionManager = (config: SessionManagerConfig) => {
         stderr: 'inherit',
       })
 
-      // If using stdin, write the prompt
+      // If using stdin, write the prompt and close stdin
+      // (stream mode spawns new process per turn, so stdin should close after writing)
       if (schema.prompt.stdin && session.process) {
-        writePromptToStdin(session.process, promptText)
+        writePromptToStdin(session.process, promptText, true)
       }
     }
 
@@ -211,9 +215,10 @@ export const createSessionManager = (config: SessionManagerConfig) => {
       stderr: 'inherit',
     })
 
-    // If using stdin, write the prompt
+    // If using stdin, write the prompt and close stdin
+    // (iterative mode spawns new process per turn, so stdin should close after writing)
     if (schema.prompt.stdin && session.process) {
-      writePromptToStdin(session.process, fullPrompt)
+      writePromptToStdin(session.process, fullPrompt, true)
     }
 
     const result = await collectOutput(session, outputParser, onUpdate, timeout)
@@ -233,8 +238,10 @@ export const createSessionManager = (config: SessionManagerConfig) => {
   const buildCommand = (session: Session, promptText: string): string[] => {
     const args = [...schema.command]
 
-    // Add output format flags
-    args.push(schema.output.flag, schema.output.value)
+    // Add output format flags (only if non-empty)
+    if (schema.output.flag) {
+      args.push(schema.output.flag, schema.output.value)
+    }
 
     // Add auto-approve flags
     if (schema.autoApprove) {
@@ -261,6 +268,12 @@ export const createSessionManager = (config: SessionManagerConfig) => {
       }
     }
 
+    // Debug output: show constructed command
+    if (verbose) {
+      const stdinNote = schema.prompt.stdin ? ' (+ stdin)' : ''
+      console.error(`[headless] Command: ${args.join(' ')}${stdinNote}`)
+    }
+
     return args
   }
 
@@ -334,15 +347,24 @@ const generateSessionId = (): string => {
  * - `'ignore'` → null (not writable)
  * - number → file descriptor (not a FileSink)
  *
+ * **Closing stdin:** When `closeAfterWrite` is true, the stdin stream is
+ * closed after writing. This is required for CLIs that read from stdin
+ * with `-` and wait for EOF before processing (e.g., Codex). For stream
+ * mode sessions where stdin stays open for subsequent prompts, pass false.
+ *
  * @param process - Subprocess with stdin stream
  * @param prompt - Prompt text to write
+ * @param closeAfterWrite - Whether to close stdin after writing (default: false)
  *
  * @internal
  */
-const writePromptToStdin = (process: Subprocess, prompt: string): void => {
+const writePromptToStdin = (process: Subprocess, prompt: string, closeAfterWrite = false): void => {
   if (process.stdin && typeof process.stdin !== 'number') {
     process.stdin.write(`${prompt}\n`)
     process.stdin.flush()
+    if (closeAfterWrite) {
+      process.stdin.end()
+    }
   }
 }