@monoes/monodesign 1.2.18 → 1.2.22

package/skill/scripts/live-copy-edit-agent.mjs

@@ -7,14 +7,18 @@
  * helper to ask Codex/Claude to edit true source files.
  */
 
-import { spawn, spawnSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
-import { createRequire } from 'node:module';
-
-const DEFAULT_TIMEOUT_MS = 60_000;
-const require = createRequire(import.meta.url);
+import { compactBatchForPrompt, normalizeBatchResult, mockBatchResult } from './live-copy-edit-compact.mjs';
+import {
+  parseCopyEditAgentResult,
+  chooseCopyEditAgent,
+  describeNoProviderError,
+  runCodex,
+  runClaude,
+} from './live-copy-edit-runners.mjs';
+import { runCopyEditPostApplyChecks } from './live-copy-edit-post-apply.mjs';
 
 export function buildCopyEditBatchPrompt(batch, { cwd = process.cwd() } = {}) {
   const repairLines = batch?.repair ? [
@@ -134,550 +138,11 @@ export async function runCopyEditBatchAgent(batch, opts = {}) {
   throw new Error(`AI copy-edit batch did not return a valid completion payload. ${tail.trim()}`);
 }
 
-export function runCopyEditPostApplyChecks({ cwd = process.cwd(), files = [] } = {}) {
-  const failures = [];
-  const warnings = [];
-  const uniqueFiles = [...new Set((files || []).filter((file) => typeof file === 'string' && file.trim()))];
-  for (const relativeFile of uniqueFiles) {
-    const file = path.resolve(cwd, relativeFile);
-    if (!isPathInsideOrEqual(cwd, file) || !fs.existsSync(file)) {
-      warnings.push({ file: relativeFile, reason: 'file_missing_or_outside_cwd' });
-      continue;
-    }
-    let content = '';
-    try { content = fs.readFileSync(file, 'utf-8'); } catch (err) {
-      failures.push({ file: relativeFile, reason: 'read_failed', message: err.message });
-      continue;
-    }
-    const markerMatch = findLeftoverMonodesignMarker(content);
-    if (markerMatch) failures.push({ file: relativeFile, reason: 'leftover_monodesign_marker', marker: markerMatch });
-    if (/\.json$/.test(relativeFile)) {
-      try {
-        JSON.parse(content);
-      } catch (err) {
-        failures.push({
-          file: relativeFile,
-          reason: 'invalid_json',
-          message: err.message || String(err),
-        });
-      }
-    }
-    const syntaxCheck = checkFrameworkSourceSyntax(relativeFile, content);
-    if (syntaxCheck?.failure) failures.push(syntaxCheck.failure);
-    if (syntaxCheck?.warning) warnings.push(syntaxCheck.warning);
-    if (/\.(mjs|cjs|js)$/.test(relativeFile)) {
-      const check = spawnSync(process.execPath, ['--check', file], { cwd, encoding: 'utf-8' });
-      if (check.status !== 0) {
-        failures.push({
-          file: relativeFile,
-          reason: 'invalid_js',
-          message: (check.stderr || check.stdout || '').trim(),
-        });
-      }
-    }
-  }
-  const validation = runManualEditValidationScript(cwd);
-  if (validation?.failure) failures.push(validation.failure);
-  if (validation?.warning) warnings.push(validation.warning);
-  return { ok: failures.length === 0, failures, warnings };
-}
-
-function checkFrameworkSourceSyntax(relativeFile, content) {
-  if (!/\.(jsx|tsx|ts)$/.test(relativeFile)) return null;
-  let parser;
-  try {
-    parser = require('@babel/parser');
-  } catch {
-    return { warning: { file: relativeFile, reason: 'syntax_parser_unavailable' } };
-  }
-  const plugins = ['jsx'];
-  if (/\.(ts|tsx)$/.test(relativeFile)) plugins.push('typescript');
-  try {
-    parser.parse(content, {
-      sourceType: 'module',
-      plugins,
-      errorRecovery: false,
-    });
-    return null;
-  } catch (err) {
-    return {
-      failure: {
-        file: relativeFile,
-        reason: 'invalid_source_syntax',
-        message: err.message || String(err),
-      },
-    };
-  }
-}
-
-function findLeftoverMonodesignMarker(content) {
-  const commentMarker = content.match(/^\s*(?:<!--|\{\/\*)\s*monodesign-carbonize-(?:start|end)\b|^\s*(?:<!--|\{\/\*)\s*monodesign-variants-(?:start|end)\b/m);
-  if (commentMarker) return commentMarker[0];
-
-  const attrPattern = /\bdata-monodesign-(?:variants?|original-text|editable|text-wrap)\s*=/g;
-  for (const line of content.split(/\r?\n/)) {
-    attrPattern.lastIndex = 0;
-    let match;
-    while ((match = attrPattern.exec(line))) {
-      if (!isInsideQuotedLiteral(line, match.index)) return match[0];
-    }
-  }
-  return null;
-}
-
-function isInsideQuotedLiteral(line, index) {
-  let quote = null;
-  let escaped = false;
-  for (let i = 0; i < index; i++) {
-    const ch = line[i];
-    if (escaped) {
-      escaped = false;
-      continue;
-    }
-    if (ch === '\\') {
-      escaped = true;
-      continue;
-    }
-    if (quote) {
-      if (ch === quote) quote = null;
-      continue;
-    }
-    if (ch === '"' || ch === "'" || ch === '`') quote = ch;
-  }
-  return quote !== null;
-}
-
-function runManualEditValidationScript(cwd) {
-  const script = readManualEditValidationScript(cwd);
-  if (!script) return null;
-  const validation = spawnSync(script, {
-    cwd,
-    encoding: 'utf-8',
-    shell: true,
-    timeout: 30_000,
-  });
-  if (validation.error) {
-    return {
-      failure: {
-        file: 'package.json',
-        reason: 'manual_edit_validation_failed',
-        message: validation.error.message || String(validation.error),
-      },
-    };
-  }
-  if (validation.status !== 0) {
-    return {
-      failure: {
-        file: 'package.json',
-        reason: 'manual_edit_validation_failed',
-        message: [validation.stderr, validation.stdout].filter(Boolean).join('\n').trim(),
-      },
-    };
-  }
-  return null;
-}
-
-function readManualEditValidationScript(cwd) {
-  const pkgPath = path.join(cwd, 'package.json');
-  if (!fs.existsSync(pkgPath)) return null;
-  try {
-    const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
-    const script = pkg?.scripts?.['monodesign:manual-edit-validate'];
-    return typeof script === 'string' && script.trim() ? script : null;
-  } catch {
-    return null;
-  }
-}
-
-function compactBatchForPrompt(batch) {
-  return {
-    pageUrl: batch?.pageUrl || null,
-    repair: batch?.repair || undefined,
-    entries: (batch?.entries || []).map((entry) => ({
-      id: entry.id,
-      pageUrl: entry.pageUrl,
-      stagedAt: entry.stagedAt || null,
-      element: compactContextForBatch(entry.element),
-      ops: (entry.ops || []).map(compactBatchOp),
-    })),
-    candidates: batch?.candidates || [],
-  };
-}
-
-function compactBatchOp(op) {
-  return {
-    entryId: op.entryId,
-    ref: op.ref,
-    contextRef: op.contextRef,
-    tag: op.tag,
-    elementId: op.elementId,
-    classes: op.classes,
-    originalText: op.originalText,
-    newText: op.newText,
-    deleted: op.deleted === true || undefined,
-    sourceHint: op.sourceHint,
-    leaf: compactContextForBatch(op.leaf),
-    nearbyEditableTexts: Array.isArray(op.nearbyEditableTexts) ? op.nearbyEditableTexts.slice(0, 8) : [],
-    container: compactContextForBatch(op.container),
-    contextHints: Array.isArray(op.contextHints) ? op.contextHints.slice(0, 12) : [],
-  };
-}
-
-function compactContextForBatch(value) {
-  if (!value || typeof value !== 'object') return value || null;
-  return {
-    ref: value.ref,
-    tagName: value.tagName,
-    id: value.id,
-    classes: value.classes,
-    textContent: truncate(value.textContent, 900),
-    outerHTML: truncate(stripLiveRuntimeHtml(value.outerHTML), 1800),
-  };
-}
-
-function stripLiveRuntimeHtml(html) {
-  if (typeof html !== 'string') return html || null;
-  return html
-    .replace(/\sdata-monodesign-(?:original-text|editable|text-wrap)(?:=(?:"[^"]*"|'[^']*'|[^\s>]+))?/g, '')
-    .replace(/\scontenteditable(?:=(?:"[^"]*"|'[^']*'|[^\s>]+))?/g, '')
-    .replace(/\sstyle=(["'])(?:(?!\1)[\s\S])*(?:-webkit-user-modify|user-select:\s*text|cursor:\s*text)(?:(?!\1)[\s\S])*\1/g, '');
-}
-
-function normalizeBatchResult(result) {
-  const status = result.status === 'partial' ? 'partial' : result.status === 'error' ? 'error' : 'done';
-  const appliedEntryIds = Array.isArray(result.appliedEntryIds)
-    ? result.appliedEntryIds.filter((id) => typeof id === 'string')
-    : [];
-  const failed = Array.isArray(result.failed)
-    ? result.failed.filter(Boolean).map((item) => ({
-        entryId: item.entryId || item.id || null,
-        reason: item.reason || item.message || 'failed',
-        candidates: Array.isArray(item.candidates) ? item.candidates : [],
-      }))
-    : [];
-  const files = Array.isArray(result.files) ? result.files.filter((file) => typeof file === 'string') : [];
-  const notes = Array.isArray(result.notes) ? result.notes.filter((note) => typeof note === 'string') : [];
-  const warnings = Array.isArray(result.warnings)
-    ? result.warnings
-        .filter(Boolean)
-        .map((warning) => typeof warning === 'string' ? { message: warning } : warning)
-        .filter((warning) => warning && typeof warning === 'object')
-    : [];
-  return {
-    status,
-    message: result.message || null,
-    appliedEntryIds,
-    failed,
-    files,
-    notes,
-    warnings,
-  };
-}
-
-function mockBatchResult(batch, env, cwd = process.cwd()) {
-  applyMockWrites(env, cwd);
-  const raw = env.MONODESIGN_LIVE_COPY_AGENT_MOCK_RESULT;
-  if (raw) {
-    const parsed = parseCopyEditBatchResult(raw);
-    if (parsed) return parsed;
-    throw new Error('Invalid MONODESIGN_LIVE_COPY_AGENT_MOCK_RESULT JSON');
-  }
-  return {
-    status: 'done',
-    appliedEntryIds: (batch?.entries || []).map((entry) => entry.id).filter(Boolean),
-    failed: [],
-    files: [],
-    notes: ['mock copy-edit batch result'],
-  };
-}
-
-function applyMockWrites(env, cwd) {
-  const raw = env.MONODESIGN_LIVE_COPY_AGENT_MOCK_WRITES;
-  if (!raw) return;
-  const writes = tryParseJson(raw);
-  if (!writes || typeof writes !== 'object' || Array.isArray(writes)) {
-    throw new Error('Invalid MONODESIGN_LIVE_COPY_AGENT_MOCK_WRITES JSON');
-  }
-  for (const [relativeFile, content] of Object.entries(writes)) {
-    if (typeof relativeFile !== 'string' || typeof content !== 'string') continue;
-    const absolute = path.resolve(cwd, relativeFile);
-    if (!isPathInsideOrEqual(cwd, absolute)) continue;
-    fs.mkdirSync(path.dirname(absolute), { recursive: true });
-    fs.writeFileSync(absolute, content, 'utf-8');
-  }
-}
-
-export function parseCopyEditAgentResult(text) {
-  const trimmed = String(text || '').trim();
-  if (!trimmed) return null;
-
-  const parsedOuter = tryParseJson(trimmed);
-  if (parsedOuter) {
-    if (typeof parsedOuter.result === 'string') {
-      const nested = parseCopyEditAgentResult(parsedOuter.result);
-      if (nested) return nested;
-    }
-    if (parsedOuter.status === 'done' || parsedOuter.status === 'partial' || parsedOuter.status === 'error') return parsedOuter;
-  }
-
-  const jsonMatch = trimmed.match(/\{[\s\S]*\}/);
-  if (!jsonMatch) return null;
-  const parsed = tryParseJson(jsonMatch[0]);
-  if (parsed?.status === 'done' || parsed?.status === 'partial' || parsed?.status === 'error') return parsed;
-  return null;
-}
-
-export function chooseCopyEditAgent({
-  env = process.env,
-  authCheck = commandAuthed,
-  chatAvailable = () => false,
-} = {}) {
-  const mode = (env.MONODESIGN_LIVE_COPY_AGENT || 'auto').trim().toLowerCase();
-  if (mode === '0' || mode === 'false' || mode === 'off' || mode === 'none') return null;
-  if (mode === 'mock') return 'mock';
-  if (mode === 'chat') return chatAvailable() ? 'chat' : null;
-  if (mode === 'codex') return commandExists('codex') ? 'codex' : null;
-  if (mode === 'claude') return commandExists('claude') ? 'claude' : null;
-  if (mode !== 'auto') return null;
-  if (authCheck('codex')) return 'codex';
-  if (authCheck('claude')) return 'claude';
-  if (chatAvailable()) return 'chat';
-  return null;
-}
-
-function runCodex(prompt, { cwd, env, resultPath, logPath, timeoutMs = DEFAULT_TIMEOUT_MS }) {
-  const args = [
-    'exec',
-    '--cd', cwd,
-    '--dangerously-bypass-approvals-and-sandbox',
-    '--ephemeral',
-    '--output-last-message', resultPath,
-    '-c', `model_reasoning_effort="${env.MONODESIGN_LIVE_COPY_AGENT_EFFORT || 'low'}"`,
-  ];
-  if (env.MONODESIGN_LIVE_COPY_AGENT_MODEL) {
-    args.push('--model', env.MONODESIGN_LIVE_COPY_AGENT_MODEL);
-  }
-  args.push('-');
-  return runAgentProcess('codex', args, prompt, { cwd, env, logPath, timeoutMs });
-}
-
-function runClaude(prompt, { cwd, env, resultPath, logPath, timeoutMs = DEFAULT_TIMEOUT_MS }) {
-  const args = [
-    '--print',
-    '--permission-mode', 'bypassPermissions',
-    '--output-format', 'json',
-  ];
-  if (env.MONODESIGN_LIVE_COPY_AGENT_MODEL) {
-    args.push('--model', env.MONODESIGN_LIVE_COPY_AGENT_MODEL);
-  }
-  args.push(prompt);
-  // Forward env as-is so CLAUDE_CODE_OAUTH_TOKEN and ANTHROPIC_API_KEY flow
-  // through. On macOS, `claude /login` stores creds in the Keychain, which a
-  // non-TTY subprocess cannot read; setting CLAUDE_CODE_OAUTH_TOKEN (via
-  // `claude setup-token`) is the supported headless auth path.
-  return runAgentProcess('claude', args, '', { cwd, env, logPath, timeoutMs, mirrorOutputPath: resultPath });
-}
-
-function runAgentProcess(command, args, stdin, { cwd, env, logPath, timeoutMs, mirrorOutputPath }) {
-  return new Promise((resolve, reject) => {
-    const log = fs.createWriteStream(logPath, { flags: 'a' });
-    const child = spawn(command, args, {
-      cwd,
-      env,
-      stdio: ['pipe', 'pipe', 'pipe'],
-    });
-    let output = '';
-    let settled = false;
-    const timer = setTimeout(() => {
-      child.kill('SIGTERM');
-      rejectOnce(new Error(`AI copy-edit worker timed out after ${timeoutMs}ms`));
-    }, timeoutMs);
-
-    const rejectOnce = (err) => {
-      if (settled) return;
-      settled = true;
-      clearTimeout(timer);
-      log.end();
-      reject(err);
-    };
-    const resolveOnce = () => {
-      if (settled) return;
-      settled = true;
-      clearTimeout(timer);
-      if (mirrorOutputPath) fs.writeFileSync(mirrorOutputPath, output);
-      log.end();
-      resolve();
-    };
-
-    process.once('SIGTERM', () => {
-      try { child.kill('SIGTERM'); } catch {}
-    });
-    child.stdout.on('data', (chunk) => {
-      output += chunk.toString();
-      log.write(chunk);
-    });
-    child.stderr.on('data', (chunk) => {
-      log.write(chunk);
-    });
-    child.on('error', rejectOnce);
-    child.on('exit', (code, signal) => {
-      if (code === 0) {
-        resolveOnce();
-      } else {
-        const hint = extractRunnerErrorMessage(output, command);
-        rejectOnce(new Error(hint || `${command} exited with ${signal || code}`));
-      }
-    });
-    if (stdin) child.stdin.end(stdin);
-    else child.stdin.end();
-  });
-}
-
-function isPathInsideOrEqual(cwd, file) {
-  const relative = path.relative(path.resolve(cwd), path.resolve(file)).split(path.sep).join('/');
-  return relative === '' || (!relative.startsWith('..') && !path.isAbsolute(relative));
-}
-
-function tryParseJson(text) {
-  try { return JSON.parse(text); } catch { return null; }
-}
-
-function truncate(value, max) {
-  if (typeof value !== 'string') return value;
-  if (value.length <= max) return value;
-  return `${value.slice(0, max)}... [truncated ${value.length - max} chars]`;
-}
-
-function commandExists(command) {
-  const result = spawnSync(command, ['--version'], { stdio: 'ignore' });
-  return !result.error && result.status === 0;
-}
-
-/**
- * Build a diagnostic error message explaining why no AI runner is usable.
- * Splits the previous "Install/authenticate Codex or Claude" lump into a
- * per-provider summary so the user knows exactly which step unblocks them.
- */
-export function describeNoProviderError({
-  exists = commandExists,
-  chatAvailable = () => false,
-  env = process.env,
-} = {}) {
-  const lines = ['No live copy-edit AI runner is available.'];
-  if (exists('claude')) {
-    if (env.CLAUDE_CODE_OAUTH_TOKEN) {
-      lines.push('  • Claude CLI: installed; CLAUDE_CODE_OAUTH_TOKEN is set but the CLI still rejected it. The token may be expired or invalid.');
-    } else {
-      lines.push('  • Claude CLI: installed but not selected. If Apply still fails, the subprocess may be unable to read your `claude /login` credentials (on macOS, the Keychain can be unreachable from a no-TTY child).');
-      lines.push('      Headless fix: run `claude setup-token` once, then `export CLAUDE_CODE_OAUTH_TOKEN=<the printed sk-ant-oat01-… token>` before starting `live-server.mjs`.');
-      lines.push('      Alternative: `export ANTHROPIC_API_KEY=<key>` if you have console.anthropic.com credits.');
-    }
-  } else {
-    lines.push('  • Claude CLI: not installed.');
-  }
-  if (exists('codex')) {
-    lines.push('  • Codex CLI: installed. If Apply still fails, run `codex login` to authenticate.');
-  } else {
-    lines.push('  • Codex CLI: not installed.');
-  }
-  if (chatAvailable()) {
-    lines.push('  • Chat: an Monodesign live session is polling but selection chose another provider — unexpected; please report.');
-  } else {
-    lines.push('  • Chat: no Monodesign live session is currently polling on this server. Start Monodesign live in your chat to route Apply through the chat agent.');
-  }
-  lines.push('Fix one of the above, or set MONODESIGN_LIVE_COPY_AGENT=mock for tests.');
-  return lines.join('\n');
-}
-
-/**
- * Pull a human-readable failure reason out of a subprocess's stdout when the
- * process exited non-zero. Recognizes:
- *   - Claude CLI `--output-format json` errors:
- *     {"is_error": true, "result": "Not logged in · Please run /login", ...}
- *   - Generic JSON payloads with `message` or `error` strings.
- *   - The last non-empty line of unstructured output.
- * Returns null when nothing meaningful surfaces, so the caller can fall back
- * to its existing "X exited with N" message.
- */
-export function extractRunnerErrorMessage(output, command) {
-  const text = String(output || '').trim();
-  if (!text) return null;
-  const candidates = [];
-  const direct = tryParseJson(text);
-  if (direct) candidates.push(direct);
-  const trailingMatch = text.match(/\{[\s\S]*\}\s*$/);
-  if (trailingMatch) {
-    const tail = tryParseJson(trailingMatch[0]);
-    if (tail && tail !== direct) candidates.push(tail);
-  }
-  for (const parsed of candidates) {
-    if (!parsed || typeof parsed !== 'object') continue;
-    if (parsed.is_error === true && typeof parsed.result === 'string' && parsed.result.trim()) {
-      return `${command} CLI: ${parsed.result.trim()}`;
-    }
-    if (typeof parsed.message === 'string' && parsed.message.trim()) {
-      return `${command} CLI: ${parsed.message.trim()}`;
-    }
-    if (typeof parsed.error === 'string' && parsed.error.trim()) {
-      return `${command} CLI: ${parsed.error.trim()}`;
-    }
-  }
-  const lines = text.split(/\r?\n/).map((line) => line.trim()).filter(Boolean);
-  if (lines.length > 0) {
-    const last = lines[lines.length - 1];
-    if (last.length > 0 && last.length < 400) return `${command}: ${last}`;
-  }
-  return null;
-}
 
-/**
- * Pre-flight a CLI provider with a trivial prompt and report whether it can
- * actually do work. Cached per process so the `auto` branch of
- * chooseCopyEditAgent only pays the cost once per server boot.
- *
- * For claude we run the same `--print --output-format json` invocation we use
- * for real batches; an unauthenticated CLI fails in ~36 ms with
- * { is_error: true, result: "Not logged in · ..." }.
- * For codex we only confirm the binary exists — `codex exec` always burns a
- * real LLM call, so checking auth without spending tokens is not possible
- * here; if the user has codex installed but unauthed, the runtime error from
- * runCodex (now improved by extractRunnerErrorMessage) will surface clearly.
- */
-const COMMAND_AUTH_CACHE = new Map();
-
-function commandAuthed(command) {
-  if (COMMAND_AUTH_CACHE.has(command)) return COMMAND_AUTH_CACHE.get(command);
-  const ok = computeCommandAuthed(command);
-  COMMAND_AUTH_CACHE.set(command, ok);
-  return ok;
-}
-
-function computeCommandAuthed(command) {
-  if (!commandExists(command)) return false;
-  if (command === 'codex') return true;
-  if (command !== 'claude') return false;
-  let result;
-  try {
-    result = spawnSync('claude', [
-      '--print',
-      '--output-format', 'json',
-      'ping',
-    ], {
-      encoding: 'utf-8',
-      timeout: 10000,
-      env: process.env,
-    });
-  } catch {
-    return false;
-  }
-  if (result.error || result.signal) return false;
-  const stdout = String(result.stdout || '').trim();
-  if (result.status !== 0) {
-    // Non-zero exit: probably an auth or config error. Definitely not usable.
-    return false;
-  }
-  if (!stdout) return true;
-  const parsed = tryParseJson(stdout) || tryParseJson(stdout.match(/\{[\s\S]*\}\s*$/)?.[0] || '');
-  if (parsed && parsed.is_error === true) return false;
-  return true;
-}
+export {
+  runCopyEditPostApplyChecks,
+  parseCopyEditAgentResult,
+  chooseCopyEditAgent,
+  describeNoProviderError,
+};
+export { extractRunnerErrorMessage } from './live-copy-edit-runners.mjs';