fix(component-sdk): remove raw pty diagnostics dump

packages/component-sdk/src/__tests__/runner-pty-diagnostics.test.ts

@@ -0,0 +1,73 @@
+import { EventEmitter } from 'node:events';
+import { describe, expect, it, mock, vi } from 'bun:test';
+import { createExecutionContext } from '../context';
+
+const standardSpawn = vi.fn(() => {
+  const proc = new EventEmitter() as EventEmitter & {
+    stdout: EventEmitter;
+    stderr: EventEmitter;
+    stdin: { write: ReturnType<typeof vi.fn>; end: ReturnType<typeof vi.fn> };
+    kill: ReturnType<typeof vi.fn>;
+  };
+
+  proc.stdout = new EventEmitter();
+  proc.stderr = new EventEmitter();
+  proc.stdin = {
+    write: vi.fn(),
+    end: vi.fn(),
+  };
+  proc.kill = vi.fn();
+
+  queueMicrotask(() => {
+    proc.stdout.emit('data', Buffer.from('{}'));
+    proc.emit('close', 0);
+  });
+
+  return proc;
+});
+
+mock.module('child_process', () => ({
+  spawn: standardSpawn,
+}));
+
+mock.module('node-pty', () => ({
+  spawn: () => {
+    throw new Error('pty spawn failed');
+  },
+}));
+
+const { runComponentWithRunner } = await import('../runner');
+
+describe('Docker PTY diagnostics', () => {
+  it('falls back without writing raw diagnostic objects to console.log', async () => {
+    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
+
+    try {
+      const context = createExecutionContext({
+        runId: 'pty-fallback-run',
+        componentRef: 'pty.component',
+        terminalCollector: vi.fn(),
+      });
+
+      const result = await runComponentWithRunner(
+        {
+          kind: 'docker',
+          image: 'busybox:1.36',
+          command: ['echo', '{}'],
+          timeoutSeconds: 30,
+        },
+        async () => ({}),
+        {},
+        context,
+      );
+
+      expect(result).toEqual({});
+      expect(standardSpawn).toHaveBeenCalled();
+      expect(logSpy).not.toHaveBeenCalledWith('diag', expect.any(Object));
+    } finally {
+      logSpy.mockRestore();
+      warnSpy.mockRestore();
+    }
+  });
+});

packages/component-sdk/src/runner.ts

@@ -486,7 +486,6 @@ async function runDockerWithPty<I, O>(
         } : String(error)
       };
 
-      console.log('diag', diag);
       context.logger.warn(
         `[Docker][PTY] Failed to spawn PTY: ${error instanceof Error ? error.message : String(error)}. Diagnostic: ${JSON.stringify(diag)}`,
       );