@monoes/monodesign 1.2.18 → 1.2.22

package/skill/scripts/hook-before-edit-shell.mjs

@@ -0,0 +1,184 @@
+/**
+ * Shell-command parsing for the Cursor preToolUse write gate
+ * (hook-before-edit.mjs): where a shell command writes, and what it writes.
+ */
+
+import fs from 'node:fs';
+import path from 'node:path';
+
+import { SENSITIVE_PATH, isGeneratedPath } from './hook-lib.mjs';
+
+function shellCommand(input) {
+  if (typeof input.command === 'string') return input.command;
+  if (input.args && typeof input.args.command === 'string') return input.args.command;
+  return '';
+}
+
+function shellRedirectPath(command) {
+  if (!command || typeof command !== 'string') return '';
+  const match = command.match(/(?:^|[\s;&|])(?:>>?|1>>?)\s*(?:"([^"]+)"|'([^']+)'|([^<>\s]+))/);
+  return (match?.[1] || match?.[2] || match?.[3] || '').trim();
+}
+
+function shellWriteDestination(command) {
+  return shellRedirectPath(command) || shellTeeDestination(command) || shellCopyPaths(command)?.dest || shellPythonWriteDestination(command) || '';
+}
+
+function shellPythonWriteDestination(command) {
+  if (!/\bpython(?:3)?\b/.test(command || '')) return '';
+  const directPath = firstMatch(command, /(?:^|[^\w.])(?:pathlib\.)?Path\(\s*(["'])(.*?)\1\s*\)\s*\.write_text\s*\(/);
+  if (directPath) return directPath;
+
+  const pathsByVar = new Map();
+  const assignmentRe = /\b([A-Za-z_]\w*)\s*=\s*(?:pathlib\.)?Path\(\s*(["'])(.*?)\2\s*\)/g;
+  let assignment;
+  while ((assignment = assignmentRe.exec(command))) {
+    pathsByVar.set(assignment[1], assignment[3]);
+  }
+
+  const writeVarRe = /\b([A-Za-z_]\w*)\.write_text\s*\(/g;
+  let writeVar;
+  while ((writeVar = writeVarRe.exec(command))) {
+    const candidate = pathsByVar.get(writeVar[1]);
+    if (candidate) return candidate;
+  }
+
+  return firstMatch(command, /\bopen\(\s*(["'])(.*?)\1\s*,\s*(["'])[wax](?:\+)?b?\3/);
+}
+
+function firstMatch(value, re) {
+  const match = String(value || '').match(re);
+  return (match?.[2] || '').trim();
+}
+
+function shellTeeDestination(command) {
+  const words = shellWords(command);
+  const teeIndex = words.findIndex((word) => path.basename(word) === 'tee');
+  if (teeIndex === -1) return '';
+  for (const word of words.slice(teeIndex + 1)) {
+    if (['&&', '||', ';', '|'].includes(word)) break;
+    if (word === '--') continue;
+    if (word.startsWith('-')) continue;
+    return word;
+  }
+  return '';
+}
+
+function shellCopiedFileContent(command, cwd) {
+  const source = shellCopyPaths(command)?.source;
+  if (!source) return '';
+  const sourcePath = path.isAbsolute(source) ? source : path.resolve(cwd, source);
+  if (!isInsideProject(sourcePath, cwd)) return '';
+  if (SENSITIVE_PATH.test(sourcePath) || isGeneratedPath(sourcePath, cwd)) return '';
+  try {
+    const stat = fs.statSync(sourcePath);
+    if (!stat.isFile() || stat.size > 1024 * 1024) return '';
+    return fs.readFileSync(sourcePath, 'utf-8');
+  } catch {
+    return '';
+  }
+}
+
+function shellCopyPaths(command) {
+  const words = shellWords(command);
+  if (words.length < 3 || path.basename(words[0]) !== 'cp') return null;
+  const args = [];
+  for (const word of words.slice(1)) {
+    if (['&&', '||', ';', '|'].includes(word)) break;
+    if (word === '--') continue;
+    if (word.startsWith('-')) continue;
+    args.push(word);
+  }
+  if (args.length < 2) return null;
+  return { source: args[args.length - 2], dest: args[args.length - 1] };
+}
+
+function shellWords(command) {
+  if (!command || typeof command !== 'string') return [];
+  const words = [];
+  const re = /"((?:\\"|[^"])*)"|'((?:\\'|[^'])*)'|([^\s]+)/g;
+  let match;
+  while ((match = re.exec(command))) {
+    words.push((match[1] ?? match[2] ?? match[3] ?? '').replace(/\\(["'])/g, '$1'));
+  }
+  return words;
+}
+
+function shellHereDocContent(command) {
+  if (!command || typeof command !== 'string') return '';
+  const markerMatch = command.match(/<<-?\s*['"]?([A-Za-z0-9_.-]+)['"]?[^\r\n]*\r?\n/);
+  if (!markerMatch) return '';
+  const marker = markerMatch[1];
+  const start = (markerMatch.index || 0) + markerMatch[0].length;
+  const rest = command.slice(start);
+  const endRe = new RegExp(`\\r?\\n${escapeRegExp(marker)}(?:\\r?\\n|$)`);
+  const end = rest.search(endRe);
+  return end >= 0 ? rest.slice(0, end) : '';
+}
+
+function shellPythonWriteContent(command) {
+  if (!/\bpython(?:3)?\b/.test(command || '')) return '';
+  const script = shellHereDocContent(command) || command;
+  return pythonStringArg(script, /\.write_text\s*\(\s*/g) || pythonStringArg(script, /\.write\s*\(\s*/g);
+}
+
+function pythonStringArg(script, prefixRe) {
+  let prefix;
+  while ((prefix = prefixRe.exec(script))) {
+    const start = prefixRe.lastIndex;
+    const triple = script.slice(start, start + 3);
+    if (triple === "'''" || triple === '"""') {
+      const end = script.indexOf(triple, start + 3);
+      if (end !== -1) return script.slice(start + 3, end);
+      continue;
+    }
+    const quote = script[start];
+    if (quote !== '"' && quote !== "'") continue;
+    let out = '';
+    for (let i = start + 1; i < script.length; i++) {
+      const ch = script[i];
+      if (ch === '\\') {
+        out += script[i + 1] || '';
+        i += 1;
+      } else if (ch === quote) {
+        return out;
+      } else {
+        out += ch;
+      }
+    }
+  }
+  return '';
+}
+
+function escapeRegExp(value) {
+  return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+function relativePath(filePath, cwd) {
+  try {
+    const rel = path.relative(cwd, filePath).split(path.sep).join('/');
+    if (!rel || rel.startsWith('..') || path.isAbsolute(rel)) return filePath;
+    return rel.split(path.sep).join('/');
+  } catch {
+    return filePath;
+  }
+}
+
+function isInsideProject(filePath, cwd) {
+  try {
+    const rel = path.relative(cwd, filePath).split(path.sep).join('/');
+    return rel === '' || (!rel.startsWith('..') && !path.isAbsolute(rel));
+  } catch {
+    return false;
+  }
+}
+
+export {
+  shellCommand,
+  shellWriteDestination,
+  shellCopiedFileContent,
+  shellHereDocContent,
+  shellPythonWriteContent,
+  relativePath,
+  isInsideProject,
+};

package/skill/scripts/hook-before-edit.mjs

@@ -36,6 +36,15 @@ import {
   truthy,
   writeAuditLog,
 } from './hook-lib.mjs';
+import {
+  shellCommand,
+  shellWriteDestination,
+  shellCopiedFileContent,
+  shellHereDocContent,
+  shellPythonWriteContent,
+  relativePath,
+  isInsideProject,
+} from './hook-before-edit-shell.mjs';
 
 async function readStdin() {
   if (process.stdin.isTTY) return '';
@@ -172,171 +181,6 @@ function readExistingProjectFile(filePath, cwd) {
   }
 }
 
-function shellCommand(input) {
-  if (typeof input.command === 'string') return input.command;
-  if (input.args && typeof input.args.command === 'string') return input.args.command;
-  return '';
-}
-
-function shellRedirectPath(command) {
-  if (!command || typeof command !== 'string') return '';
-  const match = command.match(/(?:^|[\s;&|])(?:>>?|1>>?)\s*(?:"([^"]+)"|'([^']+)'|([^<>\s]+))/);
-  return (match?.[1] || match?.[2] || match?.[3] || '').trim();
-}
-
-function shellWriteDestination(command) {
-  return shellRedirectPath(command) || shellTeeDestination(command) || shellCopyPaths(command)?.dest || shellPythonWriteDestination(command) || '';
-}
-
-function shellPythonWriteDestination(command) {
-  if (!/\bpython(?:3)?\b/.test(command || '')) return '';
-  const directPath = firstMatch(command, /(?:^|[^\w.])(?:pathlib\.)?Path\(\s*(["'])(.*?)\1\s*\)\s*\.write_text\s*\(/);
-  if (directPath) return directPath;
-
-  const pathsByVar = new Map();
-  const assignmentRe = /\b([A-Za-z_]\w*)\s*=\s*(?:pathlib\.)?Path\(\s*(["'])(.*?)\2\s*\)/g;
-  let assignment;
-  while ((assignment = assignmentRe.exec(command))) {
-    pathsByVar.set(assignment[1], assignment[3]);
-  }
-
-  const writeVarRe = /\b([A-Za-z_]\w*)\.write_text\s*\(/g;
-  let writeVar;
-  while ((writeVar = writeVarRe.exec(command))) {
-    const candidate = pathsByVar.get(writeVar[1]);
-    if (candidate) return candidate;
-  }
-
-  return firstMatch(command, /\bopen\(\s*(["'])(.*?)\1\s*,\s*(["'])[wax](?:\+)?b?\3/);
-}
-
-function firstMatch(value, re) {
-  const match = String(value || '').match(re);
-  return (match?.[2] || '').trim();
-}
-
-function shellTeeDestination(command) {
-  const words = shellWords(command);
-  const teeIndex = words.findIndex((word) => path.basename(word) === 'tee');
-  if (teeIndex === -1) return '';
-  for (const word of words.slice(teeIndex + 1)) {
-    if (['&&', '||', ';', '|'].includes(word)) break;
-    if (word === '--') continue;
-    if (word.startsWith('-')) continue;
-    return word;
-  }
-  return '';
-}
-
-function shellCopiedFileContent(command, cwd) {
-  const source = shellCopyPaths(command)?.source;
-  if (!source) return '';
-  const sourcePath = path.isAbsolute(source) ? source : path.resolve(cwd, source);
-  if (!isInsideProject(sourcePath, cwd)) return '';
-  if (SENSITIVE_PATH.test(sourcePath) || isGeneratedPath(sourcePath, cwd)) return '';
-  try {
-    const stat = fs.statSync(sourcePath);
-    if (!stat.isFile() || stat.size > 1024 * 1024) return '';
-    return fs.readFileSync(sourcePath, 'utf-8');
-  } catch {
-    return '';
-  }
-}
-
-function shellCopyPaths(command) {
-  const words = shellWords(command);
-  if (words.length < 3 || path.basename(words[0]) !== 'cp') return null;
-  const args = [];
-  for (const word of words.slice(1)) {
-    if (['&&', '||', ';', '|'].includes(word)) break;
-    if (word === '--') continue;
-    if (word.startsWith('-')) continue;
-    args.push(word);
-  }
-  if (args.length < 2) return null;
-  return { source: args[args.length - 2], dest: args[args.length - 1] };
-}
-
-function shellWords(command) {
-  if (!command || typeof command !== 'string') return [];
-  const words = [];
-  const re = /"((?:\\"|[^"])*)"|'((?:\\'|[^'])*)'|([^\s]+)/g;
-  let match;
-  while ((match = re.exec(command))) {
-    words.push((match[1] ?? match[2] ?? match[3] ?? '').replace(/\\(["'])/g, '$1'));
-  }
-  return words;
-}
-
-function shellHereDocContent(command) {
-  if (!command || typeof command !== 'string') return '';
-  const markerMatch = command.match(/<<-?\s*['"]?([A-Za-z0-9_.-]+)['"]?[^\r\n]*\r?\n/);
-  if (!markerMatch) return '';
-  const marker = markerMatch[1];
-  const start = (markerMatch.index || 0) + markerMatch[0].length;
-  const rest = command.slice(start);
-  const endRe = new RegExp(`\\r?\\n${escapeRegExp(marker)}(?:\\r?\\n|$)`);
-  const end = rest.search(endRe);
-  return end >= 0 ? rest.slice(0, end) : '';
-}
-
-function shellPythonWriteContent(command) {
-  if (!/\bpython(?:3)?\b/.test(command || '')) return '';
-  const script = shellHereDocContent(command) || command;
-  return pythonStringArg(script, /\.write_text\s*\(\s*/g) || pythonStringArg(script, /\.write\s*\(\s*/g);
-}
-
-function pythonStringArg(script, prefixRe) {
-  let prefix;
-  while ((prefix = prefixRe.exec(script))) {
-    const start = prefixRe.lastIndex;
-    const triple = script.slice(start, start + 3);
-    if (triple === "'''" || triple === '"""') {
-      const end = script.indexOf(triple, start + 3);
-      if (end !== -1) return script.slice(start + 3, end);
-      continue;
-    }
-    const quote = script[start];
-    if (quote !== '"' && quote !== "'") continue;
-    let out = '';
-    for (let i = start + 1; i < script.length; i++) {
-      const ch = script[i];
-      if (ch === '\\') {
-        out += script[i + 1] || '';
-        i += 1;
-      } else if (ch === quote) {
-        return out;
-      } else {
-        out += ch;
-      }
-    }
-  }
-  return '';
-}
-
-function escapeRegExp(value) {
-  return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}
-
-function relativePath(filePath, cwd) {
-  try {
-    const rel = path.relative(cwd, filePath).split(path.sep).join('/');
-    if (!rel || rel.startsWith('..') || path.isAbsolute(rel)) return filePath;
-    return rel.split(path.sep).join('/');
-  } catch {
-    return filePath;
-  }
-}
-
-function isInsideProject(filePath, cwd) {
-  try {
-    const rel = path.relative(cwd, filePath).split(path.sep).join('/');
-    return rel === '' || (!rel.startsWith('..') && !path.isAbsolute(rel));
-  } catch {
-    return false;
-  }
-}
-
 // The static HTML engine reads its input from disk, but preToolUse only has
 // the proposed content. Stage it in a temp file so html-engine targets get the
 // same DOM-structural rules pre-write that runHook applies post-edit.

package/skill/scripts/hook-lib-cache.mjs

@@ -0,0 +1,145 @@
+import fs from 'node:fs';
+import path from 'node:path';
+import { MONODESIGN_COMMAND } from './lib/provider.mjs';
+import { ENVELOPE_PREFIX, HOOK_LOCAL_IGNORE_PATTERNS, EDIT_COUNT_THRESHOLD, safeReadJson, getCachePath } from './hook-lib-config.mjs';
+
+const HOOK_IGNORE_MARKER_OPEN = '# monodesign-hook-ignore-start';
+const HOOK_IGNORE_MARKER_CLOSE = '# monodesign-hook-ignore-end';
+const CACHE_MAX_SESSIONS = 8;
+
+export function readCache(cwd) {
+  const raw = safeReadJson(getCachePath(cwd));
+  if (!raw || typeof raw !== 'object' || raw.version !== 1) {
+    return { version: 1, sessions: {} };
+  }
+  return {
+    version: 1,
+    sessions: raw.sessions && typeof raw.sessions === 'object' ? raw.sessions : {},
+  };
+}
+
+export function persistCache(cwd, cache) {
+  const sessions = cache.sessions || {};
+  const ids = Object.keys(sessions);
+  if (ids.length > CACHE_MAX_SESSIONS) {
+    // Garbage-collect oldest sessions by updatedAt.
+    const ordered = ids
+      .map((id) => [id, sessions[id]?.updatedAt || 0])
+      .sort((a, b) => b[1] - a[1])
+      .slice(0, CACHE_MAX_SESSIONS);
+    const next = {};
+    for (const [id] of ordered) next[id] = sessions[id];
+    cache = { ...cache, sessions: next };
+  }
+  const target = getCachePath(cwd);
+  try {
+    ensureHookGitExcludes(cwd);
+    fs.mkdirSync(path.dirname(target), { recursive: true });
+    fs.writeFileSync(target, JSON.stringify(cache));
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+export function ensureHookGitExcludes(cwd = process.cwd()) {
+  try {
+    const target = resolveHookGitExcludeTarget(cwd);
+    if (!target) {
+      return { mode: 'none', changed: false, patterns: [...HOOK_LOCAL_IGNORE_PATTERNS] };
+    }
+
+    const patterns = target.patternPrefix
+      ? HOOK_LOCAL_IGNORE_PATTERNS.map((pattern) => `${target.patternPrefix}/${pattern}`)
+      : [...HOOK_LOCAL_IGNORE_PATTERNS];
+    const markerSuffix = target.patternPrefix || '.';
+    const markerOpen = `${HOOK_IGNORE_MARKER_OPEN} ${markerSuffix}`;
+    const markerClose = `${HOOK_IGNORE_MARKER_CLOSE} ${markerSuffix}`;
+    const existing = fs.existsSync(target.path) ? fs.readFileSync(target.path, 'utf-8') : '';
+    const block = [markerOpen, ...patterns, markerClose].join('\n');
+    const markerRe = new RegExp(`${escapeRegExp(markerOpen)}[\\s\\S]*?${escapeRegExp(markerClose)}`);
+
+    let updated;
+    if (markerRe.test(existing)) {
+      updated = existing.replace(markerRe, block);
+    } else {
+      const prefix = existing.length === 0 ? '' : existing.endsWith('\n') ? existing : `${existing}\n`;
+      updated = `${prefix}${prefix.endsWith('\n\n') || prefix === '' ? '' : '\n'}${block}\n`;
+    }
+
+    if (updated !== existing) {
+      fs.mkdirSync(path.dirname(target.path), { recursive: true });
+      fs.writeFileSync(target.path, updated, 'utf-8');
+    }
+
+    return {
+      mode: 'git-info-exclude',
+      file: path.relative(path.resolve(cwd), target.path).split(path.sep).join('/'),
+      changed: updated !== existing,
+      patterns,
+    };
+  } catch {
+    return { mode: 'error', changed: false, patterns: [...HOOK_LOCAL_IGNORE_PATTERNS] };
+  }
+}
+
+function resolveHookGitExcludeTarget(cwd) {
+  const start = path.resolve(cwd);
+  let dir = start;
+  while (true) {
+    const dotGit = path.join(dir, '.git');
+    if (fs.existsSync(dotGit)) {
+      const gitDir = resolveGitDir(dotGit, dir);
+      if (!gitDir) return null;
+      const relPrefix = path.relative(dir, start).split(path.sep).join('/');
+      return {
+        path: path.join(gitDir, 'info', 'exclude'),
+        patternPrefix: relPrefix && relPrefix !== '.' ? relPrefix : '',
+      };
+    }
+    const parent = path.dirname(dir);
+    if (parent === dir) return null;
+    dir = parent;
+  }
+}
+
+function resolveGitDir(dotGit, worktreeDir) {
+  const stat = fs.statSync(dotGit);
+  if (stat.isDirectory()) return dotGit;
+  if (!stat.isFile()) return null;
+
+  const body = fs.readFileSync(dotGit, 'utf-8').trim();
+  const match = body.match(/^gitdir:\s*(.+)$/i);
+  if (!match) return null;
+  return path.isAbsolute(match[1]) ? match[1] : path.resolve(worktreeDir, match[1]);
+}
+
+function escapeRegExp(value) {
+  return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+export function ensureSession(cache, sessionId) {
+  if (!cache.sessions[sessionId]) {
+    cache.sessions[sessionId] = { updatedAt: Date.now(), files: {} };
+  }
+  return cache.sessions[sessionId];
+}
+
+export function ensureFile(cache, sessionId, filePath) {
+  const session = ensureSession(cache, sessionId);
+  if (!session.files[filePath]) {
+    session.files[filePath] = { editCount: 0, findings: [] };
+  }
+  return session.files[filePath];
+}
+
+export function bumpEditCount(cache, sessionId, filePath) {
+  const fileEntry = ensureFile(cache, sessionId, filePath);
+  fileEntry.editCount = (fileEntry.editCount || 0) + 1;
+  ensureSession(cache, sessionId).updatedAt = Date.now();
+  return fileEntry.editCount;
+}
+
+export function suppressionNotice(filePath) {
+  return `${ENVELOPE_PREFIX} Suppressing further design hints on ${filePath}. More than ${EDIT_COUNT_THRESHOLD} edits in this session reached. Run ${MONODESIGN_COMMAND} audit to revisit.`;
+}