reslop 0.2.0 → 0.2.1

package/lib/session/watch.js

@@ -72,26 +72,127 @@ const closeWatcher = (watcher) => {
   }
 };
 
-const schedule = (state) => {
-  if (state.closed) return;
-  if (state.timer !== null) clearTimeout(state.timer);
-  state.timer = setTimeout(() => {
-    state.timer = null;
+const reviewMarkdown = (rel) => {
+  const norm = `${rel}`.replaceAll('\\', '/');
+  if (!norm.startsWith('.review/')) return false;
+  const rest = norm.slice('.review/'.length);
+  if (!rest || rest.includes('/')) return false;
+  return rest.endsWith('.md');
+};
+
+const isReviewDir = (root, dir) => {
+  const rel = path.relative(root, dir).replaceAll('\\', '/');
+  return rel === '.review' || rel.startsWith('.review/');
+};
+
+const hasFile = (full) => {
+  try {
+    fs.accessSync(full);
+    return true;
+  } catch {
+    return false;
+  }
+};
+
+const isBareReviewName = (root, dir, rel) => {
+  if (!rel || rel.includes('/') || !rel.endsWith('.md')) return false;
+  const base = path.basename(rel);
+  if (hasFile(path.join(dir, base))) return false;
+  return hasFile(path.join(root, '.review', base));
+};
+
+const isReviewEvent = (root, dir, name) => {
+  if (isReviewDir(root, dir)) return true;
+  if (!name) return false;
+  const rel = path.relative(root, path.join(dir, name));
+  const norm = rel.replaceAll('\\', '/');
+  if (reviewMarkdown(norm)) return true;
+  return isBareReviewName(root, dir, norm);
+};
+
+const isOwnDirEvent = (dir, name) => {
+  if (!name) return false;
+  const base = path.basename(dir);
+  const slash = `${name}`.replaceAll('\\', '/');
+  const bare = slash.startsWith('./') ? slash.slice(2) : slash;
+  const trimmed = bare.endsWith('/') ? bare.slice(0, -1) : bare;
+  if (trimmed === base) {
+    try {
+      return !fs.statSync(path.join(dir, trimmed)).isFile();
+    } catch {
+      return true;
+    }
+  }
+  const full = path.resolve(dir, name);
+  if (full === path.resolve(dir)) return true;
+  if (path.basename(full) !== base) return false;
+  try {
+    return fs.realpathSync(full) === fs.realpathSync(dir);
+  } catch {
+    return false;
+  }
+};
+
+const schedule = (state, timerKey, fire) => {
+  if (state.closed || !fire) return;
+  if (state[timerKey] !== null) clearTimeout(state[timerKey]);
+  state[timerKey] = setTimeout(() => {
+    state[timerKey] = null;
     if (state.closed) return;
-    state.onChange();
+    fire();
   }, state.debounceMs);
-  if (typeof state.timer.unref === 'function') state.timer.unref();
+  const timer = state[timerKey];
+  if (timer && typeof timer.unref === 'function') timer.unref();
+};
+
+const REVIEW_HOLD_MS = 200;
+
+const reviewBurst = (state) => {
+  const seen = state.reviewSeenAt;
+  if (!seen) return false;
+  const hold = Math.max(state.debounceMs * 2, REVIEW_HOLD_MS);
+  return Date.now() - seen < hold;
+};
+
+const scheduleChange = (state) => {
+  schedule(state, 'timer', () => {
+    if (reviewBurst(state)) return;
+    if (state.onChange) state.onChange();
+  });
+};
+
+const scheduleUnnamed = (state) => {
+  if (reviewBurst(state)) return;
+  schedule(state, 'looseTimer', () => {
+    if (reviewBurst(state)) return;
+    scheduleChange(state);
+  });
+};
+
+const dropChange = (state) => {
+  if (state.timer !== null) clearTimeout(state.timer);
+  if (state.looseTimer !== null) clearTimeout(state.looseTimer);
+  state.timer = null;
+  state.looseTimer = null;
 };
 
 const handleWatchEvent = (state, dir, eventType, filename) => {
   if (state.closed) return;
   const name = filename ? `${filename}` : '';
   const rel = name ? path.relative(state.root, path.join(dir, name)) : '';
+  if (isReviewEvent(state.root, dir, name)) {
+    state.reviewSeenAt = Date.now();
+    dropChange(state);
+    return void schedule(state, 'reviewTimer', state.onReview);
+  }
   if (rel && ignoredRel(rel)) return;
+  if (reviewBurst(state)) return;
+  if (!name) return void scheduleUnnamed(state);
+  if (isOwnDirEvent(dir, name)) return;
   if (eventType === 'rename' && name) {
     state.watchNewDir(path.join(dir, name));
   }
-  schedule(state);
+  scheduleChange(state);
 };
 
 const watchOne = (state, target, extra = {}) => {
@@ -141,6 +242,7 @@ const maybeWatchNewDir = (state, full) => {
     return;
   }
   if (!stat.isDirectory()) return;
+  if (path.basename(full) === '.review') return void watchOne(state, full);
   if (skipDirName(path.basename(full))) return;
   watchTree(state, full);
 };
@@ -166,7 +268,11 @@ const watchGit = (state) => {
 const stopState = (state) => {
   state.closed = true;
   if (state.timer !== null) clearTimeout(state.timer);
+  if (state.reviewTimer !== null) clearTimeout(state.reviewTimer);
+  if (state.looseTimer !== null) clearTimeout(state.looseTimer);
   state.timer = null;
+  state.reviewTimer = null;
+  state.looseTimer = null;
   while (state.watchers.length) closeWatcher(state.watchers.pop());
   state.watched.clear();
 };
@@ -189,13 +295,18 @@ const createDiskWatcher = (options) => {
     dirWatchOpts: recursive ? { recursive: true } : {},
     watchers: [],
     watched: new Set(),
+    onReview: options.onReview,
     timer: null,
+    reviewTimer: null,
+    looseTimer: null,
+    reviewSeenAt: 0,
     closed: false,
     watchNewDir: (full) => maybeWatchNewDir(state, full),
   };
   const scopes = options.paths && options.paths.length ? options.paths : ['.'];
   for (const spec of scopes) watchScope(state, spec);
   watchGit(state);
+  watchOne(state, path.resolve(state.root, '.review'));
   return {
     close: () => stopState(state),
     watchFiles: (rels) => watchFiles(state, rels),
@@ -207,5 +318,6 @@ module.exports = {
   POLL_MS,
   canWatchRecursive,
   ignoredRel,
+  isOwnDirEvent,
   createDiskWatcher,
 };

package/lib/session.js

@@ -394,6 +394,7 @@ class Session {
       this.status = result.error.message;
       return false;
     }
+    if (this.lifecycle) this.lifecycle.noteReviewStamp();
     return result.wrote;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reslop",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Interactive code review and repair planning for AI-assisted changes",
   "keywords": [
     "code-review",
@@ -59,7 +59,7 @@
     "eslint": "^9.39.4",
     "eslint-config-metarhia": "^9.1.9",
     "metaskills": "^1.0.5",
-    "prettier": "^3.9.8"
+    "prettier": "^3.9.9"
   },
   "dependencies": {
     "metautil": "^5.5.2"