@tagma/sdk 0.4.12 → 0.4.13

package/src/completions/output-check.ts

@@ -1,86 +1,86 @@
-import type { CompletionPlugin, CompletionContext, TaskResult } from '../types';
-import { shellArgs, parseDuration } from '../utils';
-
-const DEFAULT_TIMEOUT_MS = 30_000;
-
-export const OutputCheckCompletion: CompletionPlugin = {
-  name: 'output_check',
-  schema: {
-    description: 'Pipe task stdout into a shell command; mark success when that command exits 0.',
-    fields: {
-      check: {
-        type: 'string',
-        required: true,
-        description: 'Shell command to run. Task stdout is piped to its stdin.',
-        placeholder: "grep -q 'PASS'",
-      },
-      timeout: {
-        type: 'duration',
-        default: '30s',
-        description: 'Maximum time to wait for the check command.',
-        placeholder: '30s',
-      },
-    },
-  },
-
-  async check(config: Record<string, unknown>, result: TaskResult, ctx: CompletionContext): Promise<boolean> {
-    const checkCmd = config.check as string;
-    if (!checkCmd) throw new Error('output_check completion: "check" is required');
-
-    const timeoutMs = config.timeout != null
-      ? parseDuration(String(config.timeout))
-      : DEFAULT_TIMEOUT_MS;
-
-    const controller = new AbortController();
-    const timer = setTimeout(() => controller.abort(), timeoutMs);
-
-    // Wire pipeline abort signal into the check process so external abort
-    // terminates the child instead of leaving it running undetected.
-    const onAbort = () => controller.abort();
-    if (ctx.signal) {
-      if (ctx.signal.aborted) {
-        controller.abort();
-      } else {
-        ctx.signal.addEventListener('abort', onAbort, { once: true });
-      }
-    }
-
-    const proc = Bun.spawn(shellArgs(checkCmd) as string[], {
-      cwd: ctx.workDir,
-      stdin: 'pipe',
-      stdout: 'pipe',
-      stderr: 'pipe',
-      signal: controller.signal,
-    });
-
-    try {
-      if (proc.stdin) {
-        try {
-          proc.stdin.write(result.stdout);
-          proc.stdin.end(); // no await — consistent with runner.ts; proc.exited handles sync
-        } catch (err: unknown) {
-          // EPIPE is expected when the check process exits before reading all of stdin
-          // (e.g. `grep -q` exits on first match). Anything else is a real failure.
-          const code = (err as NodeJS.ErrnoException)?.code;
-          if (code !== 'EPIPE') throw err;
-        }
-      }
-
-      // Consume stderr concurrently with waiting for exit to prevent pipe-buffer
-      // deadlock when check script emits more than ~64 KB of stderr output.
-      const [exitCode, stderr] = await Promise.all([
-        proc.exited,
-        new Response(proc.stderr).text(),
-      ]);
-
-      if (exitCode !== 0 && stderr.trim()) {
-        console.warn(`[output_check] "${checkCmd}" exit=${exitCode}: ${stderr.trim()}`);
-      }
-
-      return exitCode === 0;
-    } finally {
-      clearTimeout(timer);
-      if (ctx.signal) ctx.signal.removeEventListener('abort', onAbort);
-    }
-  },
-};
+import type { CompletionPlugin, CompletionContext, TaskResult } from '../types';
+import { shellArgs, parseDuration } from '../utils';
+
+const DEFAULT_TIMEOUT_MS = 30_000;
+
+export const OutputCheckCompletion: CompletionPlugin = {
+  name: 'output_check',
+  schema: {
+    description: 'Pipe task stdout into a shell command; mark success when that command exits 0.',
+    fields: {
+      check: {
+        type: 'string',
+        required: true,
+        description: 'Shell command to run. Task stdout is piped to its stdin.',
+        placeholder: "grep -q 'PASS'",
+      },
+      timeout: {
+        type: 'duration',
+        default: '30s',
+        description: 'Maximum time to wait for the check command.',
+        placeholder: '30s',
+      },
+    },
+  },
+
+  async check(
+    config: Record<string, unknown>,
+    result: TaskResult,
+    ctx: CompletionContext,
+  ): Promise<boolean> {
+    const checkCmd = config.check as string;
+    if (!checkCmd) throw new Error('output_check completion: "check" is required');
+
+    const timeoutMs =
+      config.timeout != null ? parseDuration(String(config.timeout)) : DEFAULT_TIMEOUT_MS;
+
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(), timeoutMs);
+
+    // Wire pipeline abort signal into the check process so external abort
+    // terminates the child instead of leaving it running undetected.
+    const onAbort = () => controller.abort();
+    if (ctx.signal) {
+      if (ctx.signal.aborted) {
+        controller.abort();
+      } else {
+        ctx.signal.addEventListener('abort', onAbort, { once: true });
+      }
+    }
+
+    const proc = Bun.spawn(shellArgs(checkCmd) as string[], {
+      cwd: ctx.workDir,
+      stdin: 'pipe',
+      stdout: 'pipe',
+      stderr: 'pipe',
+      signal: controller.signal,
+    });
+
+    try {
+      if (proc.stdin) {
+        try {
+          proc.stdin.write(result.stdout);
+          proc.stdin.end(); // no await — consistent with runner.ts; proc.exited handles sync
+        } catch (err: unknown) {
+          // EPIPE is expected when the check process exits before reading all of stdin
+          // (e.g. `grep -q` exits on first match). Anything else is a real failure.
+          const code = (err as NodeJS.ErrnoException)?.code;
+          if (code !== 'EPIPE') throw err;
+        }
+      }
+
+      // Consume stderr concurrently with waiting for exit to prevent pipe-buffer
+      // deadlock when check script emits more than ~64 KB of stderr output.
+      const [exitCode, stderr] = await Promise.all([proc.exited, new Response(proc.stderr).text()]);
+
+      if (exitCode !== 0 && stderr.trim()) {
+        console.warn(`[output_check] "${checkCmd}" exit=${exitCode}: ${stderr.trim()}`);
+      }
+
+      return exitCode === 0;
+    } finally {
+      clearTimeout(timer);
+      if (ctx.signal) ctx.signal.removeEventListener('abort', onAbort);
+    }
+  },
+};