Fix empty success output and TTY-aware terminal styling

Stop fabricating {"data":{"ok":true}} for empty API responses; emit
null on the JSON path and silence on the human TTY path instead.

Gate chalk styling on the relevant stream isTTY so ANSI codes do not leak
into piped stderr/stdout.

Author: cursoragent

src/run.test.ts

@@ -198,6 +198,7 @@ describe('runOperation', () => {
 
     expect(fetchMock).toHaveBeenCalledOnce();
     expect(fetchMock.mock.calls[0][1]).toMatchObject({ method: 'DELETE' });
+    expect(logSpy).toHaveBeenCalledWith('null');
   });
 
   it('passes --dry-run through to the server when supported', async () => {

src/run.ts

@@ -41,8 +41,6 @@ export function deleteGateDecision(options: {
   return options.isTTY ? 'confirm' : 'block';
 }
 
-const SYNTHETIC_OK = { data: { ok: true } };
-
 async function ensureDeleteAllowed(
   operation: CliOperation,
   flags: Record<string, FlagValue>,
@@ -105,21 +103,27 @@ export async function runOperation(
     );
   }
 
-  const payload = parsed ?? SYNTHETIC_OK;
   const isTTY = Boolean(process.stdout.isTTY);
   const useJson = shouldUseJsonOutput({
     jsonFlag: isFlagEnabled(flags.json),
     isTTY,
   });
 
+  if (parsed === null) {
+    if (useJson) {
+      console.log(isTTY ? JSON.stringify(null, null, 2) : 'null');
+    }
+    return;
+  }
+
   // When piped or non-interactive (the path agents and scripts take), emit
   // compact JSON to avoid spending tokens on indentation. Pretty-print only
   // when a human asked for JSON at a real terminal.
   let output: string;
   if (useJson) {
-    output = isTTY ? JSON.stringify(payload, null, 2) : JSON.stringify(payload);
+    output = isTTY ? JSON.stringify(parsed, null, 2) : JSON.stringify(parsed);
   } else {
-    output = formatSuccessOutput(payload, operation);
+    output = formatSuccessOutput(parsed, operation);
   }
 
   console.log(output);

src/terminal.ts

@@ -1,10 +1,17 @@
 import chalk from 'chalk';
 
+function ttyAware(
+  isEnabled: () => boolean,
+  style: (text: string) => string,
+): (text: string) => string {
+  return (text: string) => (isEnabled() ? style(text) : text);
+}
+
 export const terminal = {
-  command: chalk.cyan,
-  dim: chalk.dim,
-  error: chalk.red,
-  heading: chalk.bold,
-  success: chalk.green,
-  warning: chalk.yellow,
+  command: ttyAware(() => Boolean(process.stdout.isTTY), chalk.cyan),
+  dim: ttyAware(() => Boolean(process.stdout.isTTY), chalk.dim),
+  error: ttyAware(() => Boolean(process.stderr.isTTY), chalk.red),
+  heading: ttyAware(() => Boolean(process.stdout.isTTY), chalk.bold),
+  success: ttyAware(() => Boolean(process.stdout.isTTY), chalk.green),
+  warning: ttyAware(() => Boolean(process.stdout.isTTY), chalk.yellow),
 };