fix(snapshot-tests): Align MCP and CLI transcript fixtures

Route shared xcodebuild snapshot scenarios through canonical CLI fixtures and make MCP durable text rendering use the same non-interactive transcript formatter. This removes MCP-specific newline drift and keeps test discovery formatting consistent across success and failure paths.

Also surface selective test targeting in test headers and limit discovery previews to the first six tests so the canonical fixtures stay readable while preserving parity across runtimes.

Co-Authored-By: OpenAI Codex <noreply@openai.com>

Author: cameroncooke

example_projects/iOS_Calculator/.mcp.json

@@ -0,0 +1,13 @@
+{
+  "mcpServers": {
+    "XcodeBuildMCP": {
+      "type": "stdio",
+      "command": "node",
+      "args": [
+        "../../build/cli.js",
+        "mcp"
+      ],
+      "env": {}
+    }
+  }
+}

src/rendering/__tests__/text-render-parity.test.ts

@@ -0,0 +1,138 @@
+import { afterEach, describe, expect, it, vi } from 'vitest';
+import type { PipelineEvent } from '../../types/pipeline-events.ts';
+import { renderEvents } from '../render.ts';
+import { createCliTextRenderer } from '../../utils/renderers/cli-text-renderer.ts';
+
+function captureCliText(events: readonly PipelineEvent[]): string {
+  const stdoutWrite = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+  const renderer = createCliTextRenderer({ interactive: false });
+
+  for (const event of events) {
+    renderer.onEvent(event);
+  }
+  renderer.finalize();
+
+  return stdoutWrite.mock.calls.flat().join('');
+}
+
+describe('text render parity', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('matches non-interactive cli text for discovery and summary output', () => {
+    const events: PipelineEvent[] = [
+      {
+        type: 'header',
+        timestamp: '2026-04-10T22:50:00.000Z',
+        operation: 'Test',
+        params: [
+          { label: 'Scheme', value: 'CalculatorApp' },
+          { label: 'Configuration', value: 'Debug' },
+          { label: 'Platform', value: 'iOS Simulator' },
+        ],
+      },
+      {
+        type: 'test-discovery',
+        timestamp: '2026-04-10T22:50:01.000Z',
+        operation: 'TEST',
+        total: 1,
+        tests: ['CalculatorAppTests/CalculatorAppTests/testAddition'],
+        truncated: false,
+      },
+      {
+        type: 'summary',
+        timestamp: '2026-04-10T22:50:02.000Z',
+        operation: 'TEST',
+        status: 'SUCCEEDED',
+        totalTests: 1,
+        passedTests: 1,
+        skippedTests: 0,
+        durationMs: 1500,
+      },
+    ];
+
+    expect(renderEvents(events, 'text')).toBe(captureCliText(events));
+  });
+
+  it('matches non-interactive cli text for failure diagnostics and summary spacing', () => {
+    const events: PipelineEvent[] = [
+      {
+        type: 'header',
+        timestamp: '2026-04-10T22:50:00.000Z',
+        operation: 'Test',
+        params: [
+          { label: 'Scheme', value: 'MCPTest' },
+          { label: 'Configuration', value: 'Debug' },
+          { label: 'Platform', value: 'macOS' },
+        ],
+      },
+      {
+        type: 'test-discovery',
+        timestamp: '2026-04-10T22:50:01.000Z',
+        operation: 'TEST',
+        total: 2,
+        tests: [
+          'MCPTestTests/MCPTestTests/appNameIsCorrect',
+          'MCPTestTests/MCPTestsXCTests/testAppNameIsCorrect',
+        ],
+        truncated: false,
+      },
+      {
+        type: 'test-failure',
+        timestamp: '2026-04-10T22:50:02.000Z',
+        operation: 'TEST',
+        suite: 'MCPTestsXCTests',
+        test: 'testDeliberateFailure()',
+        message: 'XCTAssertTrue failed',
+        location: 'MCPTestsXCTests.swift:11',
+      },
+      {
+        type: 'summary',
+        timestamp: '2026-04-10T22:50:03.000Z',
+        operation: 'TEST',
+        status: 'FAILED',
+        totalTests: 2,
+        passedTests: 1,
+        failedTests: 1,
+        skippedTests: 0,
+        durationMs: 2200,
+      },
+    ];
+
+    expect(renderEvents(events, 'text')).toBe(captureCliText(events));
+  });
+
+  it('renders next steps in canonical cli format for text transcripts', () => {
+    const events: PipelineEvent[] = [
+      {
+        type: 'summary',
+        timestamp: '2026-04-10T22:50:05.000Z',
+        operation: 'BUILD',
+        status: 'SUCCEEDED',
+        durationMs: 7100,
+      },
+      {
+        type: 'next-steps',
+        timestamp: '2026-04-10T22:50:06.000Z',
+        runtime: 'mcp',
+        steps: [
+          {
+            label: 'Get built macOS app path',
+            tool: 'get_mac_app_path',
+            cliTool: 'get-app-path',
+            workflow: 'macos',
+            params: {
+              scheme: 'MCPTest',
+            },
+          },
+        ],
+      },
+    ];
+
+    const output = renderEvents(events, 'text');
+    expect(output).toBe(captureCliText(events));
+    expect(output).toContain('xcodebuildmcp macos get-app-path');
+    expect(output).not.toContain('get_mac_app_path({');
+  });
+});

src/rendering/render.ts

@@ -1,27 +1,9 @@
-import type {
-  CompilerErrorEvent,
-  CompilerWarningEvent,
-  PipelineEvent,
-  TestFailureEvent,
-} from '../types/pipeline-events.ts';
+import type { PipelineEvent } from '../types/pipeline-events.ts';
 import { sessionStore } from '../utils/session-store.ts';
-import { deriveDiagnosticBaseDir } from '../utils/renderers/index.ts';
 import {
-  formatBuildStageEvent,
-  formatDetailTreeEvent,
-  formatFileRefEvent,
-  formatGroupedCompilerErrors,
-  formatGroupedTestFailures,
-  formatGroupedWarnings,
-  formatHeaderEvent,
-  formatNextStepsEvent,
-  formatSectionEvent,
-  formatStatusLineEvent,
-  formatSummaryEvent,
-  formatTableEvent,
-  formatTestDiscoveryEvent,
-} from '../utils/renderers/event-formatting.ts';
-import { createCliTextRenderer } from '../utils/renderers/cli-text-renderer.ts';
+  createCliTextRenderer,
+  renderCliTextTranscript,
+} from '../utils/renderers/cli-text-renderer.ts';
 import type { RenderSession, RenderStrategy, ImageAttachment } from './types.ts';
 
 function isErrorEvent(event: PipelineEvent): boolean {
@@ -34,124 +16,13 @@ function isErrorEvent(event: PipelineEvent): boolean {
 function createTextRenderSession(): RenderSession {
   const events: PipelineEvent[] = [];
   const attachments: ImageAttachment[] = [];
-  const contentParts: string[] = [];
   const suppressWarnings = sessionStore.get('suppressWarnings');
-  const groupedCompilerErrors: CompilerErrorEvent[] = [];
-  const groupedWarnings: CompilerWarningEvent[] = [];
-  const groupedTestFailures: TestFailureEvent[] = [];
-
-  let diagnosticBaseDir: string | null = null;
   let hasError = false;
 
-  const pushText = (text: string): void => {
-    contentParts.push(text);
-  };
-
-  const pushSection = (text: string): void => {
-    pushText(`\n${text}`);
-  };
-
   return {
     emit(event: PipelineEvent): void {
       events.push(event);
       if (isErrorEvent(event)) hasError = true;
-
-      switch (event.type) {
-        case 'header': {
-          diagnosticBaseDir = deriveDiagnosticBaseDir(event);
-          pushSection(formatHeaderEvent(event));
-          break;
-        }
-
-        case 'build-stage': {
-          pushSection(formatBuildStageEvent(event));
-          break;
-        }
-
-        case 'status-line': {
-          pushSection(formatStatusLineEvent(event));
-          break;
-        }
-
-        case 'section': {
-          pushText(`\n\n${formatSectionEvent(event)}`);
-          break;
-        }
-
-        case 'detail-tree': {
-          pushSection(formatDetailTreeEvent(event));
-          break;
-        }
-
-        case 'table': {
-          pushSection(formatTableEvent(event));
-          break;
-        }
-
-        case 'file-ref': {
-          pushSection(formatFileRefEvent(event));
-          break;
-        }
-
-        case 'compiler-warning': {
-          if (!suppressWarnings) {
-            groupedWarnings.push(event);
-          }
-          break;
-        }
-
-        case 'compiler-error': {
-          groupedCompilerErrors.push(event);
-          break;
-        }
-
-        case 'test-discovery': {
-          pushText(formatTestDiscoveryEvent(event));
-          break;
-        }
-
-        case 'test-progress': {
-          break;
-        }
-
-        case 'test-failure': {
-          groupedTestFailures.push(event);
-          break;
-        }
-
-        case 'summary': {
-          const diagOpts = { baseDir: diagnosticBaseDir ?? undefined };
-          const diagnosticSections: string[] = [];
-
-          if (groupedTestFailures.length > 0) {
-            diagnosticSections.push(formatGroupedTestFailures(groupedTestFailures, diagOpts));
-            groupedTestFailures.length = 0;
-          }
-
-          if (groupedWarnings.length > 0) {
-            diagnosticSections.push(formatGroupedWarnings(groupedWarnings, diagOpts));
-            groupedWarnings.length = 0;
-          }
-
-          if (event.status === 'FAILED' && groupedCompilerErrors.length > 0) {
-            diagnosticSections.push(formatGroupedCompilerErrors(groupedCompilerErrors, diagOpts));
-            groupedCompilerErrors.length = 0;
-          }
-
-          if (diagnosticSections.length > 0) {
-            pushSection(diagnosticSections.join('\n\n'));
-          }
-
-          pushSection(formatSummaryEvent(event));
-          break;
-        }
-
-        case 'next-steps': {
-          const effectiveRuntime = event.runtime === 'cli' ? 'cli' : 'mcp';
-          pushText(`\n\n${formatNextStepsEvent(event, effectiveRuntime)}`);
-          break;
-        }
-      }
     },
 
     attach(image: ImageAttachment): void {
@@ -171,8 +42,9 @@ function createTextRenderSession(): RenderSession {
     },
 
     finalize(): string {
-      diagnosticBaseDir = null;
-      return contentParts.join('');
+      return renderCliTextTranscript(events, {
+        suppressWarnings: suppressWarnings ?? false,
+      });
     },
   };
 }