@monoes/monodesign 1.2.20 → 1.2.23

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

@@ -0,0 +1,137 @@
+// Compacting a staged copy-edit batch down to what's worth sending an AI
+// runner, normalizing its reply shape, and the mock provider used in tests.
+// Split out of live-copy-edit-agent.mjs (file-size sweep — pure move, no
+// behaviour change).
+
+import fs from 'node:fs';
+import path from 'node:path';
+import { truncate, isPathInsideOrEqual, tryParseJson } from './live-copy-edit-runners.mjs';
+import { parseCopyEditBatchResult } from './live-copy-edit-agent.mjs';
+
+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 {
+  compactBatchForPrompt,
+  compactBatchOp,
+  compactContextForBatch,
+  stripLiveRuntimeHtml,
+  normalizeBatchResult,
+  mockBatchResult,
+  applyMockWrites,
+};

package/skill/scripts/live-copy-edit-post-apply.mjs

@@ -0,0 +1,175 @@
+// Post-apply verification: JSON/JS syntax checks, JSX/TSX parse checks,
+// leftover monodesign marker detection, and the project's own
+// manual-edit-validate script. Split out of live-copy-edit-agent.mjs
+// (file-size sweep — pure move, no behaviour change).
+
+import { spawnSync } from 'node:child_process';
+import fs from 'node:fs';
+import path from 'node:path';
+import { createRequire } from 'node:module';
+import { isPathInsideOrEqual } from './live-copy-edit-runners.mjs';
+
+const require = createRequire(import.meta.url);
+
+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;
+  }
+}
+
+export {
+  checkFrameworkSourceSyntax,
+  findLeftoverMonodesignMarker,
+  isInsideQuotedLiteral,
+  runManualEditValidationScript,
+  readManualEditValidationScript,
+};

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

@@ -0,0 +1,297 @@
+// Running the Codex/Claude CLI as a child process to apply a copy-edit
+// batch, plus the provider-selection and error-message helpers around it.
+// Split out of live-copy-edit-agent.mjs (file-size sweep — pure move, no
+// behaviour change).
+
+import { spawn, spawnSync } from 'node:child_process';
+import fs from 'node:fs';
+import path from 'node:path';
+
+const DEFAULT_TIMEOUT_MS = 60_000;
+
+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 {
+  runCodex,
+  runClaude,
+  runAgentProcess,
+  isPathInsideOrEqual,
+  tryParseJson,
+  truncate,
+  commandExists,
+  commandAuthed,
+  computeCommandAuthed,
+};