fix: use non interactive context in sub shell (#2632)

Synced from monorepo@003eb27457e332d49dbb15612bc8ddd26741688c

Author: divyenduz

.sync-commit

@@ -1 +1 @@
-dd3da607cce506b5497c04ac07c44774e7954892
+003eb27457e332d49dbb15612bc8ddd26741688c

package.json

@@ -37,15 +37,15 @@
     "zod": "4.4.3",
     "zod-config": "1.4.0",
     "@xata.io/ai": "0.1.0",
-    "@xata.io/api": "0.1.1",
     "@xata.io/config": "0.0.1",
     "@xata.io/lang": "0.0.1",
     "@xata.io/pgroll": "0.9.0",
     "@xata.io/pgstream": "0.2.0",
     "@xata.io/sql": "0.1.4",
     "@xata.io/test-utils": "0.0.1",
     "@xata.io/tsconfig": "0.0.1",
-    "@xata.io/utils": "0.1.0"
+    "@xata.io/utils": "0.1.0",
+    "@xata.io/api": "0.1.1"
   },
   "scripts": {
     "dev": "bun src/bin/cli.ts",

src/context.ts

@@ -32,9 +32,9 @@ export interface LocalContext extends CommandContext, StricliAutoCompleteContext
   /**
    * Whether the CLI is running in an interactive context.
    *
-   * False when running in CI or when invoked from an agentic context
-   * (Cursor, Claude Code, Amp, etc.). Prompt helpers short-circuit when
-   * this is false.
+   * False when stdio is not attached to an interactive terminal, when running
+   * in CI, or when invoked from an agentic context (Cursor, Claude Code, Amp,
+   * etc.). Prompt helpers short-circuit when this is false.
    */
   readonly isInteractive: boolean;
   readonly print: typeof print;
@@ -63,6 +63,10 @@ type BuildContextOptions = {
   cliInvocationId?: string;
 };
 
+export function getIsInteractive(process: NodeJS.Process, { isCI, isAgent }: { isCI: boolean; isAgent: boolean }) {
+  return process.stdin.isTTY === true && process.stdout.isTTY === true && !(isCI || isAgent);
+}
+
 export async function buildContext(process: NodeJS.Process, options: BuildContextOptions = {}): Promise<LocalContext> {
   const debug = Boolean(Bun.env.DEBUG);
   const usingEnvApiKey = Boolean(env.XATA_API_KEY);
@@ -81,8 +85,8 @@ export async function buildContext(process: NodeJS.Process, options: BuildContex
     os,
     fs,
     path,
-    // Treat both CI and agentic invocations as non-interactive
-    isInteractive: !(ciInfo.isCI || agent.isAgent),
+    // Treat captured/piped output, CI, and agentic invocations as non-interactive.
+    isInteractive: getIsInteractive(process, { isCI: ciInfo.isCI, isAgent: agent.isAgent }),
     print,
     getActiveProfile,
     getOrganization,

src/context.unit.test.ts

@@ -0,0 +1,46 @@
+import { describe, expect, test } from 'bun:test';
+import { getIsInteractive } from './context';
+
+function buildProcess({ stdinIsTTY, stdoutIsTTY }: { stdinIsTTY: boolean; stdoutIsTTY: boolean }) {
+  return {
+    stdin: { isTTY: stdinIsTTY },
+    stdout: { isTTY: stdoutIsTTY },
+    stderr: { write: () => {} }
+  } as unknown as NodeJS.Process;
+}
+
+describe('buildContext', () => {
+  test('is interactive when stdin and stdout are TTYs', async () => {
+    const isInteractive = getIsInteractive(buildProcess({ stdinIsTTY: true, stdoutIsTTY: true }), {
+      isCI: false,
+      isAgent: false
+    });
+
+    expect(isInteractive).toBe(true);
+  });
+
+  test('is non-interactive when stdout is captured', async () => {
+    const isInteractive = getIsInteractive(buildProcess({ stdinIsTTY: true, stdoutIsTTY: false }), {
+      isCI: false,
+      isAgent: false
+    });
+
+    expect(isInteractive).toBe(false);
+  });
+
+  test('is non-interactive when stdin is not a TTY', async () => {
+    const isInteractive = getIsInteractive(buildProcess({ stdinIsTTY: false, stdoutIsTTY: true }), {
+      isCI: false,
+      isAgent: false
+    });
+
+    expect(isInteractive).toBe(false);
+  });
+
+  test('is non-interactive in CI or agentic contexts', async () => {
+    const process = buildProcess({ stdinIsTTY: true, stdoutIsTTY: true });
+
+    expect(getIsInteractive(process, { isCI: true, isAgent: false })).toBe(false);
+    expect(getIsInteractive(process, { isCI: false, isAgent: true })).toBe(false);
+  });
+});