@the-open-engine/zeroshot 6.38.0 → 6.39.0

package/cli/event-copy.js

@@ -1,12 +1,13 @@
+"use strict";
 const EVENT_COPY = {
-  IMPLEMENTATION_READY: 'Implementation ready',
-  PR_CREATED: 'Pull request created',
+    IMPLEMENTATION_READY: 'Implementation ready',
+    PR_CREATED: 'Pull request created',
 };
-
 function formatMergeStatus(merged) {
-  if (merged === true || merged === 'true') return 'merged';
-  if (merged === false || merged === 'false') return 'auto-merge pending approval';
-  return null;
+    if (merged === true || merged === 'true')
+        return 'merged';
+    if (merged === false || merged === 'false')
+        return 'auto-merge pending approval';
+    return null;
 }
-
 module.exports = { EVENT_COPY, formatMergeStatus };

package/cli/event-copy.ts

@@ -0,0 +1,12 @@
+const EVENT_COPY = {
+  IMPLEMENTATION_READY: 'Implementation ready',
+  PR_CREATED: 'Pull request created',
+} as const;
+
+function formatMergeStatus(merged: unknown): string | null {
+  if (merged === true || merged === 'true') return 'merged';
+  if (merged === false || merged === 'false') return 'auto-merge pending approval';
+  return null;
+}
+
+export = { EVENT_COPY, formatMergeStatus };

package/cli/index.js

@@ -35,7 +35,12 @@ const {
   formatClusterFailed,
   formatGenericMessage,
 } = require('./message-formatters-normal');
-const { getColorForSender, buildMessagePrefix } = require('./message-formatter-utils');
+const {
+  getColorForSender,
+  buildMessagePrefix,
+  formatLifecycleEvent,
+  partitionValidationCriteria,
+} = require('./message-formatter-utils');
 const {
   loadSettings,
   mutateSettings,
@@ -5325,34 +5330,6 @@ function flushAllRenderBuffers(lines, buffers) {
   }
 }
 
-function formatLifecycleEvent(data) {
-  const event = data?.event;
-  let icon;
-  let eventText;
-
-  switch (event) {
-    case 'STARTED': {
-      icon = chalk.green('▶');
-      const triggers = data?.triggers?.join(', ') || 'none';
-      eventText = `started (listening for: ${chalk.dim(triggers)})`;
-      break;
-    }
-    case 'TASK_STARTED':
-      icon = chalk.yellow('⚡');
-      eventText = `${chalk.cyan(data.triggeredBy)} → task #${data.iteration} (${chalk.dim(data.model)})`;
-      break;
-    case 'TASK_COMPLETED':
-      icon = chalk.green('✓');
-      eventText = `task #${data.iteration} completed`;
-      break;
-    default:
-      icon = chalk.dim('•');
-      eventText = event || 'unknown event';
-  }
-
-  return { icon, eventText };
-}
-
 function handleLifecycleRender({ msg, prefix, lines }) {
   const data = msg.content?.data;
   const { icon, eventText } = formatLifecycleEvent(data);
@@ -5427,7 +5404,7 @@ function appendCriteriaGroup({ lines, prefix, label, items, color, reasonLabel }
 function appendCriteriaResults(lines, prefix, criteriaResults) {
   if (!Array.isArray(criteriaResults)) return;
 
-  const cannotValidateYet = criteriaResults.filter((c) => c.status === 'CANNOT_VALIDATE_YET');
+  const { cannotValidateYet, cannotValidate } = partitionValidationCriteria(criteriaResults);
   appendCriteriaGroup({
     lines,
     prefix,
@@ -5437,7 +5414,6 @@ function appendCriteriaResults(lines, prefix, criteriaResults) {
     reasonLabel: 'work incomplete',
   });
 
-  const cannotValidate = criteriaResults.filter((c) => c.status === 'CANNOT_VALIDATE');
   appendCriteriaGroup({
     lines,
     prefix,

package/cli/json-export.js

@@ -1,77 +1,74 @@
-const fs = require('fs');
-
+"use strict";
+const fs = require("fs");
+function isInteger(value) {
+    return Number.isInteger(value);
+}
 function indentJson(value, spaces) {
-  const prefix = ' '.repeat(spaces);
-  return JSON.stringify(value, null, 2)
-    .split('\n')
-    .map((line) => `${prefix}${line}`)
-    .join('\n');
+    const prefix = ' '.repeat(spaces);
+    return JSON.stringify(value, null, 2)
+        .split('\n')
+        .map((line) => `${prefix}${line}`)
+        .join('\n');
 }
-
 function writeAll(fd, value) {
-  const bytes = Buffer.from(value);
-  let offset = 0;
-  while (offset < bytes.length) {
-    const written = fs.writeSync(fd, bytes, offset, bytes.length - offset);
-    if (!Number.isInteger(written) || written <= 0) {
-      throw new Error('JSON export destination stopped accepting bytes');
+    const bytes = Buffer.from(value);
+    let offset = 0;
+    while (offset < bytes.length) {
+        const written = fs.writeSync(fd, bytes, offset, bytes.length - offset);
+        if (!Number.isInteger(written) || written <= 0) {
+            throw new Error('JSON export destination stopped accepting bytes');
+        }
+        offset += written;
     }
-    offset += written;
-  }
 }
-
 function createDestination(outputPath, stdout) {
-  if (outputPath) {
-    const fd = fs.openSync(outputPath, 'w');
-    return {
-      close: () => fs.closeSync(fd),
-      write: (value) => writeAll(fd, value),
-    };
-  }
-
-  if (Number.isInteger(stdout.fd)) {
+    if (outputPath) {
+        const fd = fs.openSync(outputPath, 'w');
+        return {
+            close: () => fs.closeSync(fd),
+            write: (value) => writeAll(fd, value),
+        };
+    }
+    if (isInteger(stdout.fd)) {
+        const fd = stdout.fd;
+        return {
+            close() { },
+            write: (value) => writeAll(fd, value),
+        };
+    }
     return {
-      close() {},
-      write: (value) => writeAll(stdout.fd, value),
+        close() { },
+        write: (value) => {
+            stdout.write(value);
+        },
     };
-  }
-
-  return {
-    close() {},
-    write: (value) => stdout.write(value),
-  };
 }
-
-function streamClusterJsonExport({
-  ledger,
-  clusterId,
-  outputPath = null,
-  stdout = process.stdout,
-}) {
-  const destination = createDestination(outputPath, stdout);
-  const iterator = ledger.iterateAll(clusterId);
-  try {
-    destination.write(`{\n  "cluster_id": ${JSON.stringify(clusterId)},\n  "messages": `);
-    let current = iterator.next();
-    if (current.done) {
-      destination.write('[]\n}\n');
-      return;
-    }
-
-    destination.write('[\n');
-    while (!current.done) {
-      destination.write(indentJson(current.value, 4));
-      current = iterator.next();
-      destination.write(current.done ? '\n' : ',\n');
-    }
-    destination.write('  ]\n}\n');
-  } finally {
+function streamClusterJsonExport({ ledger, clusterId, outputPath = null, stdout = process.stdout, }) {
+    const destination = createDestination(outputPath, stdout);
+    const iterator = ledger.iterateAll(clusterId);
     try {
-      if (typeof iterator.return === 'function') iterator.return();
-    } finally {
-      destination.close();
+        destination.write(`{\n  "cluster_id": ${JSON.stringify(clusterId)},\n  "messages": `);
+        let current = iterator.next();
+        if (current.done) {
+            destination.write('[]\n}\n');
+            return;
+        }
+        destination.write('[\n');
+        while (!current.done) {
+            destination.write(indentJson(current.value, 4));
+            current = iterator.next();
+            destination.write(current.done ? '\n' : ',\n');
+        }
+        destination.write('  ]\n}\n');
+    }
+    finally {
+        try {
+            if (typeof iterator.return === 'function')
+                iterator.return();
+        }
+        finally {
+            destination.close();
+        }
     }
-  }
 }
-
 module.exports = { streamClusterJsonExport };

package/cli/json-export.ts

@@ -0,0 +1,112 @@
+import fs = require('fs');
+
+interface LedgerMessageIterator extends Iterator<unknown> {
+  return?: () => IteratorResult<unknown>;
+}
+
+interface ClusterLedger {
+  iterateAll(clusterId: string): LedgerMessageIterator;
+}
+
+interface ExportStream {
+  readonly fd?: number;
+  write(value: string): unknown;
+}
+
+interface JsonExportOptions {
+  ledger: ClusterLedger;
+  clusterId: string;
+  outputPath?: string | null;
+  stdout?: ExportStream;
+}
+
+interface Destination {
+  close(): void;
+  write(value: string): void;
+}
+
+function isInteger(value: unknown): value is number {
+  return Number.isInteger(value);
+}
+
+function indentJson(value: unknown, spaces: number): string {
+  const prefix = ' '.repeat(spaces);
+  return JSON.stringify(value, null, 2)
+    .split('\n')
+    .map((line) => `${prefix}${line}`)
+    .join('\n');
+}
+
+function writeAll(fd: number, value: string): void {
+  const bytes = Buffer.from(value);
+  let offset = 0;
+  while (offset < bytes.length) {
+    const written = fs.writeSync(fd, bytes, offset, bytes.length - offset);
+    if (!Number.isInteger(written) || written <= 0) {
+      throw new Error('JSON export destination stopped accepting bytes');
+    }
+    offset += written;
+  }
+}
+
+function createDestination(
+  outputPath: string | null | undefined,
+  stdout: ExportStream
+): Destination {
+  if (outputPath) {
+    const fd = fs.openSync(outputPath, 'w');
+    return {
+      close: () => fs.closeSync(fd),
+      write: (value) => writeAll(fd, value),
+    };
+  }
+
+  if (isInteger(stdout.fd)) {
+    const fd = stdout.fd;
+    return {
+      close(): void {},
+      write: (value) => writeAll(fd, value),
+    };
+  }
+
+  return {
+    close(): void {},
+    write: (value): void => {
+      stdout.write(value);
+    },
+  };
+}
+
+function streamClusterJsonExport({
+  ledger,
+  clusterId,
+  outputPath = null,
+  stdout = process.stdout,
+}: JsonExportOptions): void {
+  const destination = createDestination(outputPath, stdout);
+  const iterator = ledger.iterateAll(clusterId);
+  try {
+    destination.write(`{\n  "cluster_id": ${JSON.stringify(clusterId)},\n  "messages": `);
+    let current = iterator.next();
+    if (current.done) {
+      destination.write('[]\n}\n');
+      return;
+    }
+
+    destination.write('[\n');
+    while (!current.done) {
+      destination.write(indentJson(current.value, 4));
+      current = iterator.next();
+      destination.write(current.done ? '\n' : ',\n');
+    }
+    destination.write('  ]\n}\n');
+  } finally {
+    try {
+      if (typeof iterator.return === 'function') iterator.return();
+    } finally {
+      destination.close();
+    }
+  }
+}
+
+export = { streamClusterJsonExport };

package/cli/message-formatter-utils.js

@@ -1,75 +1,87 @@
+"use strict";
+const chalk = require("chalk");
 /**
- * Message formatting utilities for CLI output
- * Extracted from index.js to reduce complexity
- */
-
-const chalk = require('chalk');
-
-/**
- * Get color for sender based on consistent hashing
- * @param {string} sender - Sender name
- * @returns {Function} Chalk color function
+ * Get color for sender based on consistent hashing.
  */
 function getColorForSender(sender) {
-  const colors = [chalk.cyan, chalk.magenta, chalk.yellow, chalk.green, chalk.blue];
-  let hash = 0;
-  for (let i = 0; i < sender.length; i++) {
-    hash = (hash << 5) - hash + sender.charCodeAt(i);
-    hash = hash & hash;
-  }
-  return colors[Math.abs(hash) % colors.length];
+    const colors = [chalk.cyan, chalk.magenta, chalk.yellow, chalk.green, chalk.blue];
+    let hash = 0;
+    for (let i = 0; i < sender.length; i++) {
+        hash = (hash << 5) - hash + sender.charCodeAt(i);
+        hash = hash & hash;
+    }
+    return colors[Math.abs(hash) % colors.length] ?? chalk.cyan;
 }
-
 /**
- * Build message prefix with timestamp, sender, and optional cluster ID
- * @param {Object} msg - Message object
- * @param {boolean} showClusterId - Whether to show cluster ID
- * @param {boolean} isActive - Whether cluster is active
- * @returns {string} Formatted prefix
+ * Build message prefix with timestamp, sender, and optional cluster ID.
  */
 function buildMessagePrefix(msg, showClusterId, isActive) {
-  const color = isActive ? getColorForSender(msg.sender) : chalk.dim;
-
-  let senderLabel = msg.sender;
-  if (showClusterId && msg.cluster_id) {
-    senderLabel = `${msg.cluster_id}/${msg.sender}`;
-  }
-
-  const modelSuffix = msg.sender_model ? chalk.dim(` [${msg.sender_model}]`) : '';
-  return color(`${senderLabel.padEnd(showClusterId ? 25 : 15)} |`) + modelSuffix;
+    const color = isActive ? getColorForSender(msg.sender) : chalk.dim;
+    let senderLabel = msg.sender;
+    if (showClusterId && msg.cluster_id) {
+        senderLabel = `${msg.cluster_id}/${msg.sender}`;
+    }
+    const modelSuffix = msg.sender_model ? chalk.dim(` [${msg.sender_model}]`) : '';
+    return color(`${senderLabel.padEnd(showClusterId ? 25 : 15)} |`) + modelSuffix;
 }
-
 /**
- * Build cluster prefix for watch mode
- * @param {string} clusterId - Cluster ID
- * @param {boolean} isActive - Whether cluster is active
- * @returns {string} Formatted prefix
+ * Build cluster prefix for watch mode.
  */
 function buildClusterPrefix(clusterId, isActive) {
-  return isActive
-    ? chalk.white(`${clusterId.padEnd(20)} |`)
-    : chalk.dim(`${clusterId.padEnd(20)} |`);
+    const color = isActive ? chalk.white : chalk.dim;
+    return color(`${clusterId.padEnd(20)} |`);
 }
-
 /**
- * Parse and normalize data fields (handles string JSON)
- * @param {string|Array} data - Data to parse
- * @returns {Array} Parsed array
+ * Parse and normalize data fields (handles string JSON).
  */
 function parseDataField(data) {
-  if (typeof data === 'string') {
-    try {
-      return JSON.parse(data);
-    } catch {
-      return [];
+    if (typeof data === 'string') {
+        try {
+            const parsed = JSON.parse(data);
+            return parsed;
+        }
+        catch {
+            return [];
+        }
     }
-  }
-  return Array.isArray(data) ? data : [];
+    return Array.isArray(data) ? data : [];
+}
+function formatLifecycleEvent(data) {
+    const event = data?.event;
+    let icon;
+    let eventText;
+    switch (event) {
+        case 'STARTED': {
+            icon = chalk.green('▶');
+            const triggers = data?.triggers?.join(', ') || 'none';
+            eventText = `started (listening for: ${chalk.dim(triggers)})`;
+            break;
+        }
+        case 'TASK_STARTED':
+            icon = chalk.yellow('⚡');
+            eventText = `${chalk.cyan(data?.triggeredBy)} → task #${data?.iteration} (${chalk.dim(data?.model)})`;
+            break;
+        case 'TASK_COMPLETED':
+            icon = chalk.green('✓');
+            eventText = `task #${data?.iteration} completed`;
+            break;
+        default:
+            icon = chalk.dim('•');
+            eventText = event || 'unknown event';
+    }
+    return { icon, eventText };
+}
+function partitionValidationCriteria(criteria) {
+    return {
+        cannotValidateYet: criteria.filter((criterion) => criterion.status === 'CANNOT_VALIDATE_YET'),
+        cannotValidate: criteria.filter((criterion) => criterion.status === 'CANNOT_VALIDATE'),
+    };
 }
-
 module.exports = {
-  getColorForSender,
-  buildMessagePrefix,
-  buildClusterPrefix,
-  parseDataField,
+    getColorForSender,
+    buildMessagePrefix,
+    buildClusterPrefix,
+    parseDataField,
+    formatLifecycleEvent,
+    partitionValidationCriteria,
 };

package/cli/message-formatter-utils.ts

@@ -0,0 +1,121 @@
+import chalk = require('chalk');
+
+interface MessagePrefixData {
+  sender: string;
+  cluster_id?: string | null;
+  sender_model?: string | null;
+}
+
+/**
+ * Get color for sender based on consistent hashing.
+ */
+function getColorForSender(sender: string): chalk.Chalk {
+  const colors = [chalk.cyan, chalk.magenta, chalk.yellow, chalk.green, chalk.blue];
+  let hash = 0;
+  for (let i = 0; i < sender.length; i++) {
+    hash = (hash << 5) - hash + sender.charCodeAt(i);
+    hash = hash & hash;
+  }
+  return colors[Math.abs(hash) % colors.length] ?? chalk.cyan;
+}
+
+/**
+ * Build message prefix with timestamp, sender, and optional cluster ID.
+ */
+function buildMessagePrefix(
+  msg: MessagePrefixData,
+  showClusterId: boolean,
+  isActive: boolean
+): string {
+  const color = isActive ? getColorForSender(msg.sender) : chalk.dim;
+
+  let senderLabel = msg.sender;
+  if (showClusterId && msg.cluster_id) {
+    senderLabel = `${msg.cluster_id}/${msg.sender}`;
+  }
+
+  const modelSuffix = msg.sender_model ? chalk.dim(` [${msg.sender_model}]`) : '';
+  return color(`${senderLabel.padEnd(showClusterId ? 25 : 15)} |`) + modelSuffix;
+}
+
+/**
+ * Build cluster prefix for watch mode.
+ */
+function buildClusterPrefix(clusterId: string, isActive: boolean): string {
+  const color = isActive ? chalk.white : chalk.dim;
+  return color(`${clusterId.padEnd(20)} |`);
+}
+
+/**
+ * Parse and normalize data fields (handles string JSON).
+ */
+function parseDataField(data: unknown): unknown {
+  if (typeof data === 'string') {
+    try {
+      const parsed: unknown = JSON.parse(data);
+      return parsed;
+    } catch {
+      return [];
+    }
+  }
+  return Array.isArray(data) ? data : [];
+}
+
+interface LifecycleEventData {
+  event?: string;
+  triggers?: readonly string[];
+  triggeredBy?: string | number;
+  iteration?: string | number;
+  model?: string;
+}
+
+interface ValidationCriterion {
+  status?: string;
+}
+
+function formatLifecycleEvent(data?: LifecycleEventData): { icon: string; eventText: string } {
+  const event = data?.event;
+  let icon: string;
+  let eventText: string;
+
+  switch (event) {
+    case 'STARTED': {
+      icon = chalk.green('▶');
+      const triggers = data?.triggers?.join(', ') || 'none';
+      eventText = `started (listening for: ${chalk.dim(triggers)})`;
+      break;
+    }
+    case 'TASK_STARTED':
+      icon = chalk.yellow('⚡');
+      eventText = `${chalk.cyan(data?.triggeredBy)} → task #${data?.iteration} (${chalk.dim(data?.model)})`;
+      break;
+    case 'TASK_COMPLETED':
+      icon = chalk.green('✓');
+      eventText = `task #${data?.iteration} completed`;
+      break;
+    default:
+      icon = chalk.dim('•');
+      eventText = event || 'unknown event';
+  }
+
+  return { icon, eventText };
+}
+
+function partitionValidationCriteria<T extends ValidationCriterion>(criteria: readonly T[]): {
+  cannotValidateYet: T[];
+  cannotValidate: T[];
+} {
+  return {
+    cannotValidateYet: criteria.filter((criterion) => criterion.status === 'CANNOT_VALIDATE_YET'),
+    cannotValidate: criteria.filter((criterion) => criterion.status === 'CANNOT_VALIDATE'),
+  };
+}
+
+export = {
+  getColorForSender,
+  buildMessagePrefix,
+  buildClusterPrefix,
+  parseDataField,
+  formatLifecycleEvent,
+  partitionValidationCriteria,
+};