@jmoyers/harness 0.1.10 → 0.1.20

package/README.md

@@ -1,17 +1,22 @@
 # Harness
 
-Harness is a terminal-native workspace for running multiple coding-agent threads in parallel, without losing project context.
+Harness is minimal agent orchestration in the terminal.
 
-It is built for people who want to move faster than a single chat window: implementation, review, and follow-up work can run side by side in one keyboard-first interface.
+Programmable, agent-agnostic, threaded, ergonomic, and fast.
 
 ## What matters most
 
 - Parallel threads across `codex`, `claude`, `cursor`, `terminal`, and `critique`.
-- One command palette (`ctrl+p` / `cmd+p`) to jump threads, run actions, and control workflow quickly.
-- Long-running work survives reconnects through a detached gateway.
-- Gateway control is resilient: lifecycle operations are lock-serialized per session, and missing stale records can be recovered automatically.
-- Fast left-rail navigation with automatic, readable thread titles.
-- Built-in GitHub PR actions (`Open PR` / `Create PR`) from inside Harness.
+- One command palette (`ctrl+p`) to jump threads, run actions, and control workflow quickly.
+- Toggle the bottom debug bar with `cmd+p` when you need runtime launch/auth context.
+- Codex, Claude Code, and Cursor together in one workspace.
+- Diff with Critique, with integrated terminals (`harness diff` + critique actions).
+- Detached gateway sessions keep long-running work alive through reconnects.
+- Storage lifecycle maintenance remains paused in interactive runtime paths; use `harness gateway gc` as a manual offline truncation/compaction escape hatch.
+- Storage lifecycle policy updates from `harness.config.jsonc` still apply without restart.
+- Command palette can open a GitHub thread entry in the left rail for the active project, then show full tracked-branch PR/review details in the main panel.
+- Open the active project directly in local tools (`iTerm2`, `Ghostty`, `Zed`, `Cursor`, `VSCode`, `Warp`, `Finder`) or copy its path from the palette.
+- Command-click links inside conversation terminal output: URLs open in browser, file-like paths open in your configured editor/open-in command.
 
 ## Demo
 
@@ -44,34 +49,25 @@ Use a named session when you want isolated state:
 harness --session my-session
 ```
 
-For restart/load diagnostics, use a named session with a non-default gateway port so you do not disrupt your active workspace gateway.
+Named sessions automatically fall back to an available gateway port when the preferred port is already occupied. For deterministic restart/load diagnostics, you can still set an explicit non-default gateway port.
 
-## Typical workflow
+Standalone diff viewer (phase 1):
 
-1. Open Harness in your repository.
-2. Start separate threads for implementation and review.
-3. Use `ctrl+p` / `cmd+p` to switch context and run project actions.
-4. Open or create a PR from the same workspace.
-
-## User details
-
-- Thread-scoped command palette (`[+ thread]`) can launch/install supported agent CLIs per project.
-- Critique review actions are available from the global palette and run in a terminal thread.
-- `ctrl+g` opens the project’s critique thread (or creates one if needed).
-- Theme selection is built in (`Set a Theme`) with OpenCode-compatible presets and live preview.
-- PR actions use either `GITHUB_TOKEN` or an authenticated `gh` CLI session.
-
-## Configuration
-
-Runtime behavior is controlled by `harness.config.jsonc`.
-
-Common customizations:
-
-- Set install commands for `codex`, `claude`, `cursor`, and `critique`.
-- Configure critique launch defaults.
-- Customize keybindings.
-- Choose a theme preset or custom OpenCode-compatible theme file.
-
-## License
+```bash
+harness diff --help
+```
 
-MIT (`LICENSE`)
+## Architecture (VTE path)
+
+```mermaid
+flowchart LR
+    U[Keyboard + Command Palette] --> UI[Harness TUI]
+    UI --> CP[Control Plane Stream API]
+    CP --> TM[Thread Manager]
+    TM --> AG[Agent Threads<br/>Codex / Claude / Cursor]
+    TM --> PTY[Integrated PTY Terminals]
+    PTY --> VTE[VTE Parser + Screen Model]
+    VTE --> R[Terminal Renderer]
+    TM --> DF[harness diff + Critique]
+    DF --> R
+```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jmoyers/harness",
-  "version": "0.1.10",
+  "version": "0.1.20",
   "private": false,
   "repository": {
     "type": "git",
@@ -19,11 +19,16 @@
     "scripts/cursor-hook-relay.ts",
     "scripts/harness-animate.ts",
     "scripts/harness-bin.js",
+    "scripts/harness-commands.ts",
     "scripts/harness-core.ts",
-    "scripts/harness-inspector.ts",
+    "scripts/harness-runtime.ts",
     "scripts/harness.ts",
+    "scripts/nim-tui-smoke.ts",
     "packages/harness-ai/src",
-    "scripts/terminal-recording-gif-lib.ts",
+    "packages/harness-ui/src",
+    "packages/nim-core/src",
+    "packages/nim-ui-core/src",
+    "packages/nim-test-tui/src",
     "scripts/require-bun.js",
     "native/ptyd/Cargo.lock",
     "native/ptyd/Cargo.toml",
@@ -35,6 +40,13 @@
   "publishConfig": {
     "access": "public"
   },
+  "oclif": {
+    "bin": "harness",
+    "commands": {
+      "strategy": "explicit",
+      "target": "scripts/harness-commands.ts"
+    }
+  },
   "scripts": {
     "migrate:bun": "bash ./scripts/migrate-to-bun.sh",
     "harness": "bun scripts/harness.ts",
@@ -68,28 +80,36 @@
     "perf:codex:startup:loop": "bun scripts/perf-codex-startup-loop.ts",
     "loc": "bun scripts/loc-report.ts",
     "loc:verify": "bun scripts/check-max-loc.ts --max-loc 2000",
-    "loc:verify:enforce": "bun scripts/check-max-loc.ts --max-loc 2000 --enforce",
+    "loc:verify:enforce": "bun scripts/check-max-loc.ts --max-loc 2000 --enforce --baseline harness.max-loc-baseline.json",
     "deadcode": "bun scripts/check-dead-code.ts",
     "coverage:check": "bun scripts/check-coverage.ts --lcov .harness/coverage-bun/lcov.info --config harness.coverage.jsonc",
     "release": "bun scripts/release.ts",
-    "lint": "oxlint --deny-warnings --disable-unicorn-plugin -c .oxlintrc.json src test scripts packages/harness-ai/src",
-    "format": "oxfmt --config .oxfmtrc.json src test scripts packages/harness-ai/src",
-    "format:check": "oxfmt --check --config .oxfmtrc.json src test scripts packages/harness-ai/src",
+    "lint": "oxlint --deny-warnings --disable-unicorn-plugin -c .oxlintrc.json src test scripts packages/harness-ai/src packages/harness-ui/src packages/nim-core/src packages/nim-ui-core/src packages/nim-test-tui/src",
+    "format": "oxfmt --config .oxfmtrc.json src test scripts packages/harness-ai/src packages/harness-ui/src packages/nim-core/src packages/nim-ui-core/src packages/nim-test-tui/src",
+    "format:check": "oxfmt --check --config .oxfmtrc.json src test scripts packages/harness-ai/src packages/harness-ui/src packages/nim-core/src packages/nim-ui-core/src packages/nim-test-tui/src",
     "typecheck": "tsc --noEmit",
-    "test": "bun run build:ptyd && bun test",
+    "test": "bun run build:ptyd && bun test --concurrent",
+    "test:serial": "bun run build:ptyd && bun test",
     "test:integration:codex-status": "bun scripts/integration-codex-status-sequence.ts",
     "test:integration:codex-status:long": "bun scripts/integration-codex-status-sequence.ts --timeout-ms 180000 --prompt \"Write three short poems titled Dawn, Voltage, and Orbit. Before each poem, perform one repository inspection action and include one factual line from that action. Use at least three total tool actions and do not edit any files.\"",
     "test:integration:agent-prompts": "bun scripts/integration-agent-prompt-parity.ts",
     "test:integration:agent-prompts:long": "bun scripts/integration-agent-prompt-parity.ts --timeout-ms 180000",
+    "test:integration:nim:haiku": "bun scripts/integration-nim-haiku.ts",
+    "nim:tui": "bun scripts/nim-tui-smoke.ts",
     "smoke:harness-ai": "bun scripts/harness-ai-smoke.ts",
     "smoke:harness-ai:parity": "bun scripts/harness-ai-parity-smoke.mts",
-    "test:coverage": "bun run build:ptyd && bun test --coverage --coverage-reporter=lcov --coverage-dir .harness/coverage-bun && bun run coverage:check",
+    "test:coverage": "bun run build:ptyd && bun test --concurrent --coverage --coverage-reporter=lcov --coverage-dir .harness/coverage-bun && bun run coverage:check",
+    "test:coverage:serial": "bun run build:ptyd && bun test --coverage --coverage-reporter=lcov --coverage-dir .harness/coverage-bun && bun run coverage:check",
     "verify": "bun run format:check && bun run lint && bun run typecheck && bun run deadcode && bun run test:coverage",
-    "verify:strict": "bun run verify && bun run loc:verify:enforce"
+    "verify:coverage-gate": "bun run format:check && bun run lint && bun run typecheck && bun run deadcode && bun run test:coverage",
+    "verify:strict": "bun run verify:coverage-gate && bun run loc:verify:enforce"
   },
   "dependencies": {
+    "@linear/sdk": "^75.0.0",
     "@napi-rs/canvas": "^0.1.92",
-    "gifenc": "^1.0.3"
+    "@oclif/core": "^4.8.0",
+    "gifenc": "^1.0.3",
+    "zustand": "^5.0.11"
   },
   "devDependencies": {
     "@ai-sdk/anthropic": "^3.0.45",

package/packages/harness-ai/src/anthropic-protocol.ts

@@ -308,79 +308,79 @@ function parseContentBlock(value: unknown): AnthropicContentBlock | null {
     };
   }
 
-  if (type === 'web_fetch_tool_result') {
-    const toolUseId = asString(record['tool_use_id']);
-    const contentRecord = asRecord(record['content']);
-    if (toolUseId === null || contentRecord === null) {
+  if (type !== 'web_fetch_tool_result') {
+    return null;
+  }
+
+  const toolUseId = asString(record['tool_use_id']);
+  const contentRecord = asRecord(record['content']);
+  if (toolUseId === null || contentRecord === null) {
+    return null;
+  }
+
+  const contentType = asString(contentRecord['type']);
+  if (contentType === null) {
+    return null;
+  }
+
+  if (contentType === 'web_fetch_result') {
+    const innerContent = asRecord(contentRecord['content']);
+    const source = innerContent === null ? null : asRecord(innerContent['source']);
+    if (innerContent === null || source === null) {
       return null;
     }
 
-    const contentType = asString(contentRecord['type']);
-    if (contentType === null) {
+    const url = asString(contentRecord['url']);
+    const sourceType = asString(source['type']);
+    const sourceMediaType = asString(source['media_type']);
+    const sourceData = asString(source['data']);
+    const contentBlockType = asString(innerContent['type']);
+    if (
+      url === null ||
+      sourceType === null ||
+      sourceMediaType === null ||
+      sourceData === null ||
+      contentBlockType === null
+    ) {
       return null;
     }
 
-    if (contentType === 'web_fetch_result') {
-      const innerContent = asRecord(contentRecord['content']);
-      const source = innerContent === null ? null : asRecord(innerContent['source']);
-      if (innerContent === null || source === null) {
-        return null;
-      }
-
-      const url = asString(contentRecord['url']);
-      const sourceType = asString(source['type']);
-      const sourceMediaType = asString(source['media_type']);
-      const sourceData = asString(source['data']);
-      const contentBlockType = asString(innerContent['type']);
-      if (
-        url === null ||
-        sourceType === null ||
-        sourceMediaType === null ||
-        sourceData === null ||
-        contentBlockType === null
-      ) {
-        return null;
-      }
-
-      const retrievedAt = asString(contentRecord['retrieved_at']);
-      const title = asString(innerContent['title']);
-      const citations = Array.isArray(innerContent['citations'])
-        ? (innerContent['citations'] as unknown[])
-        : null;
-
-      return {
-        type,
-        tool_use_id: toolUseId,
-        content: {
-          type: 'web_fetch_result',
-          url,
-          ...(retrievedAt !== null ? { retrieved_at: retrievedAt } : {}),
-          content: {
-            type: contentBlockType,
-            ...(title !== null ? { title } : {}),
-            source: {
-              type: sourceType,
-              media_type: sourceMediaType,
-              data: sourceData,
-            },
-            ...(citations !== null ? { citations } : {}),
-          },
-        },
-      };
-    }
+    const retrievedAt = asString(contentRecord['retrieved_at']);
+    const title = asString(innerContent['title']);
+    const citations = Array.isArray(innerContent['citations'])
+      ? (innerContent['citations'] as unknown[])
+      : null;
 
-    const errorCode = asString(contentRecord['error_code']);
     return {
       type,
       tool_use_id: toolUseId,
       content: {
-        type: contentType,
-        ...(errorCode !== null ? { error_code: errorCode } : {}),
+        type: 'web_fetch_result',
+        url,
+        ...(retrievedAt !== null ? { retrieved_at: retrievedAt } : {}),
+        content: {
+          type: contentBlockType,
+          ...(title !== null ? { title } : {}),
+          source: {
+            type: sourceType,
+            media_type: sourceMediaType,
+            data: sourceData,
+          },
+          ...(citations !== null ? { citations } : {}),
+        },
       },
     };
   }
 
-  return null;
+  const errorCode = asString(contentRecord['error_code']);
+  return {
+    type,
+    tool_use_id: toolUseId,
+    content: {
+      type: contentType,
+      ...(errorCode !== null ? { error_code: errorCode } : {}),
+    },
+  };
 }
 
 export function parseAnthropicStreamChunk(value: unknown): AnthropicStreamChunk | null {
@@ -550,18 +550,18 @@ export function parseAnthropicStreamChunk(value: unknown): AnthropicStreamChunk
     };
   }
 
-  if (type === 'error') {
-    const errorRecord = asRecord(record['error']);
-    if (errorRecord === null) {
-      return null;
-    }
-    return {
-      type,
-      error: errorRecord,
-    };
+  if (type !== 'error') {
+    return null;
   }
 
-  return null;
+  const errorRecord = asRecord(record['error']);
+  if (errorRecord === null) {
+    return null;
+  }
+  return {
+    type,
+    error: errorRecord,
+  };
 }
 
 export function mapAnthropicStopReason(reason: string | null | undefined): FinishReason {

package/packages/harness-ai/src/stream-text.ts

@@ -37,31 +37,6 @@ import type {
   TypedToolResult,
 } from './types.ts';
 
-interface Deferred<T> {
-  readonly promise: Promise<T>;
-  resolve(value: T): void;
-  reject(error: unknown): void;
-}
-
-function createDeferred<T>(): Deferred<T> {
-  let resolveFn: ((value: T) => void) | null = null;
-  let rejectFn: ((error: unknown) => void) | null = null;
-  const promise = new Promise<T>((resolve, reject) => {
-    resolveFn = resolve;
-    rejectFn = reject;
-  });
-
-  return {
-    promise,
-    resolve(value) {
-      resolveFn?.(value);
-    },
-    reject(error) {
-      rejectFn?.(error);
-    },
-  };
-}
-
 interface ContentBlockTextState {
   readonly kind: 'text' | 'reasoning';
   readonly id: string;
@@ -144,10 +119,6 @@ function normalizeMessages(
 }
 
 function toAnthropicMessageContent(message: ModelMessage): Array<Record<string, unknown>> {
-  if (message.role === 'system') {
-    return [{ type: 'text', text: message.content }];
-  }
-
   if (typeof message.content === 'string') {
     return [{ type: 'text', text: message.content }];
   }
@@ -319,27 +290,15 @@ interface StepRunResult<TOOLS extends ToolSet> {
   readonly assistantText: string;
 }
 
-function asRecord(value: unknown): Record<string, unknown> | null {
-  if (typeof value !== 'object' || value === null || Array.isArray(value)) {
-    return null;
-  }
-  return value as Record<string, unknown>;
-}
-
-function mapWebSearchResult(result: AnthropicContentBlock):
+function mapWebSearchResult(
+  result: Extract<AnthropicContentBlock, { type: 'web_search_tool_result' }>,
+):
   | {
       readonly ok: true;
       readonly output: unknown[];
       readonly sources: Array<{ url: string; title?: string; pageAge?: string }>;
     }
   | { readonly ok: false; readonly error: unknown } {
-  if (result.type !== 'web_search_tool_result') {
-    return {
-      ok: false,
-      error: { message: 'invalid web search result payload' },
-    };
-  }
-
   if (Array.isArray(result.content)) {
     const mapped = result.content.map((entry) => ({
       type: entry.type,
@@ -371,17 +330,10 @@ function mapWebSearchResult(result: AnthropicContentBlock):
 }
 
 function mapWebFetchResult(
-  result: AnthropicContentBlock,
+  result: Extract<AnthropicContentBlock, { type: 'web_fetch_tool_result' }>,
 ):
   | { readonly ok: true; readonly output: unknown }
   | { readonly ok: false; readonly error: unknown } {
-  if (result.type !== 'web_fetch_tool_result') {
-    return {
-      ok: false,
-      error: { message: 'invalid web fetch result payload' },
-    };
-  }
-
   if ('url' in result.content && 'content' in result.content) {
     const content = result.content.content;
     return {
@@ -718,10 +670,7 @@ async function runSingleStep<TOOLS extends ToolSet>(
             providerResults.push(error);
             options.emit({ type: 'tool-error', ...error });
           }
-          continue;
         }
-
-        continue;
       }
 
       if (chunk.type === 'content_block_delta') {
@@ -798,8 +747,7 @@ async function runSingleStep<TOOLS extends ToolSet>(
       }
 
       if (chunk.type === 'error') {
-        const errorRecord = asRecord(chunk.error);
-        const message = errorRecord?.['message'];
+        const message = chunk.error['message'];
         options.emit({
           type: 'error',
           error:
@@ -809,9 +757,7 @@ async function runSingleStep<TOOLS extends ToolSet>(
         continue;
       }
 
-      if (chunk.type === 'message_stop') {
-        break;
-      }
+      if (chunk.type === 'message_stop') break;
     }
   } finally {
     reader.releaseLock();
@@ -1262,41 +1208,17 @@ export function streamText<TOOLS extends ToolSet>(
   const uiMessageStream = createUIMessageStream(uiBranchA);
   const uiResponseStream = createUIMessageStream(uiBranchB);
 
-  const textDeferred = createDeferred<string>();
-  const toolCallsDeferred = createDeferred<TypedToolCall<TOOLS>[]>();
-  const toolResultsDeferred =
-    createDeferred<Array<TypedToolResult<TOOLS> | TypedToolError<TOOLS>>>();
-  const finishReasonDeferred = createDeferred<FinishReason>();
-  const usageDeferred = createDeferred<LanguageModelUsage>();
-  const responseDeferred = createDeferred<LanguageModelResponseMetadata>();
-
-  collectResultFromStream(collectorBranch)
-    .then((collected) => {
-      textDeferred.resolve(collected.text);
-      toolCallsDeferred.resolve(collected.toolCalls);
-      toolResultsDeferred.resolve(collected.toolResults);
-      finishReasonDeferred.resolve(collected.finishReason);
-      usageDeferred.resolve(collected.usage);
-      responseDeferred.resolve(collected.response);
-    })
-    .catch((error) => {
-      textDeferred.reject(error);
-      toolCallsDeferred.reject(error);
-      toolResultsDeferred.reject(error);
-      finishReasonDeferred.reject(error);
-      usageDeferred.reject(error);
-      responseDeferred.reject(error);
-    });
+  const collectedPromise = collectResultFromStream(collectorBranch);
 
   return {
     fullStream,
     textStream,
-    text: textDeferred.promise,
-    toolCalls: toolCallsDeferred.promise,
-    toolResults: toolResultsDeferred.promise,
-    finishReason: finishReasonDeferred.promise,
-    usage: usageDeferred.promise,
-    response: responseDeferred.promise,
+    text: collectedPromise.then((collected) => collected.text),
+    toolCalls: collectedPromise.then((collected) => collected.toolCalls),
+    toolResults: collectedPromise.then((collected) => collected.toolResults),
+    finishReason: collectedPromise.then((collected) => collected.finishReason),
+    usage: collectedPromise.then((collected) => collected.usage),
+    response: collectedPromise.then((collected) => collected.response),
     toUIMessageStream() {
       return uiMessageStream;
     },

package/packages/harness-ui/src/frame-primitives.ts

@@ -0,0 +1,158 @@
+export interface RenderCursorStyle {
+  readonly shape: 'block' | 'underline' | 'bar';
+  readonly blinking: boolean;
+}
+
+interface DiffRenderedRowsResult {
+  readonly output: string;
+  readonly nextRows: readonly string[];
+  readonly changedRows: readonly number[];
+}
+
+export function diffRenderedRows(
+  currentRows: readonly string[],
+  previousRows: readonly string[],
+): DiffRenderedRowsResult {
+  const changedRows: number[] = [];
+  let output = '';
+  const rowCount = Math.max(currentRows.length, previousRows.length);
+  const nextRows: string[] = [];
+  for (let row = 0; row < rowCount; row += 1) {
+    const current = currentRows[row] ?? '';
+    const previous = previousRows[row] ?? '';
+    nextRows.push(current);
+    if (current === previous) {
+      continue;
+    }
+    changedRows.push(row);
+    output += `\u001b[${String(row + 1)};1H\u001b[2K${current}`;
+  }
+  return {
+    output,
+    nextRows,
+    changedRows,
+  };
+}
+
+export function cursorStyleToDecscusr(style: RenderCursorStyle): string {
+  if (style.shape === 'block') {
+    return style.blinking ? '\u001b[1 q' : '\u001b[2 q';
+  }
+  if (style.shape === 'underline') {
+    return style.blinking ? '\u001b[3 q' : '\u001b[4 q';
+  }
+  return style.blinking ? '\u001b[5 q' : '\u001b[6 q';
+}
+
+export function cursorStyleEqual(
+  left: RenderCursorStyle | null,
+  right: RenderCursorStyle,
+): boolean {
+  if (left === null) {
+    return false;
+  }
+  return left.shape === right.shape && left.blinking === right.blinking;
+}
+
+type ScanResult =
+  | {
+      readonly valid: true;
+    }
+  | {
+      readonly valid: false;
+      readonly reason: string;
+    };
+
+function scanAnsiText(text: string): ScanResult {
+  let index = 0;
+  while (index < text.length) {
+    const code = text.codePointAt(index)!;
+    const char = String.fromCodePoint(code);
+    const width = code > 0xffff ? 2 : 1;
+    if (char !== '\u001b') {
+      index += width;
+      continue;
+    }
+
+    const next = text[index + 1];
+    if (next === undefined) {
+      return {
+        valid: false,
+        reason: 'dangling ESC at end of row',
+      };
+    }
+
+    if (next === '[') {
+      let csiIndex = index + 2;
+      let foundFinal = false;
+      while (csiIndex < text.length) {
+        const csiCode = text.codePointAt(csiIndex)!;
+        if (csiCode >= 0x40 && csiCode <= 0x7e) {
+          foundFinal = true;
+          csiIndex += 1;
+          break;
+        }
+        if (csiCode < 0x20 || csiCode > 0x3f) {
+          return {
+            valid: false,
+            reason: `invalid CSI byte 0x${csiCode.toString(16)}`,
+          };
+        }
+        csiIndex += 1;
+      }
+      if (!foundFinal) {
+        return {
+          valid: false,
+          reason: 'unterminated CSI sequence',
+        };
+      }
+      index = csiIndex;
+      continue;
+    }
+
+    if (next === ']') {
+      let oscIndex = index + 2;
+      let terminated = false;
+      while (oscIndex < text.length) {
+        const oscCode = text.codePointAt(oscIndex)!;
+        if (oscCode === 0x07) {
+          terminated = true;
+          oscIndex += 1;
+          break;
+        }
+        if (oscCode === 0x1b && text[oscIndex + 1] === '\\') {
+          terminated = true;
+          oscIndex += 2;
+          break;
+        }
+        oscIndex += oscCode > 0xffff ? 2 : 1;
+      }
+      if (!terminated) {
+        return {
+          valid: false,
+          reason: 'unterminated OSC sequence',
+        };
+      }
+      index = oscIndex;
+      continue;
+    }
+
+    index += 2;
+  }
+
+  return {
+    valid: true,
+  };
+}
+
+export function findAnsiIntegrityIssues(rows: readonly string[]): readonly string[] {
+  const issues: string[] = [];
+  for (let rowIndex = 0; rowIndex < rows.length; rowIndex += 1) {
+    const row = rows[rowIndex] ?? '';
+    const result = scanAnsiText(row);
+    if (!result.valid) {
+      issues.push(`row ${String(rowIndex + 1)}: ${result.reason}`);
+    }
+  }
+  return issues;
+}