reslop 0.1.6 → 0.1.7

package/lib/review.js

@@ -3,7 +3,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 
-const { itemPath, REVIEW_DIR } = require('./files.js');
+const { REVIEW_DIR, TODO_FILE } = require('./files.js');
 
 const TEMPLATES_FILE = 'templates.json';
 const TEMPLATE_SHOW = 5;
@@ -179,10 +179,12 @@ const createStore = (reviewPath, templates = []) => ({
 });
 
 const noteCounts = (store) => {
-  const counts = { feedback: 0, todo: 0, code: 0 };
+  const counts = { feedback: 0, todo: 0, todoDone: 0, code: 0 };
   if (!store) return counts;
   for (const todo of store.todos) {
-    if (todo.text.trim()) counts.todo += 1;
+    if (!todo.text.trim()) continue;
+    counts.todo += 1;
+    if (todo.done) counts.todoDone += 1;
   }
   for (const note of store.feedback.values()) {
     if (note.text.trim()) counts.feedback += 1;
@@ -420,7 +422,7 @@ const groupReviewNotes = (store) => {
   const groups = new Map();
   for (const todo of store.todos) {
     if (!todo.text.trim()) continue;
-    fileGroup(groups, todo.file).todos.push(todo);
+    fileGroup(groups, TODO_FILE).todos.push(todo);
   }
   for (const key of store.feedback.keys()) {
     const note = store.feedback.get(key);
@@ -746,64 +748,6 @@ const flushReview = (store, io = {}) => {
   return true;
 };
 
-const todosByFile = (todos) => {
-  const byFile = new Map();
-  for (const todo of todos) {
-    const list = byFile.get(todo.file);
-    if (list) list.push(todo);
-    else byFile.set(todo.file, [todo]);
-  }
-  return byFile;
-};
-
-const makeTodoItem = (file) => ({
-  origin: 'todo',
-  file: {
-    oldPath: file,
-    newPath: file,
-    isNew: false,
-    isDeleted: false,
-    isBinary: false,
-    preamble: [],
-    hunks: [],
-  },
-  hunk: null,
-  blockId: 'todo',
-  patchAdd: '',
-  patchRevert: '',
-});
-
-const mergeTodos = (gitItems, todos, extraFiles = []) => {
-  const base = gitItems.filter((item) => item.origin !== 'todo');
-  const byFile = todosByFile(todos);
-  for (const file of extraFiles) {
-    if (!file || byFile.has(file)) continue;
-    byFile.set(file, []);
-  }
-  if (!byFile.size) return base;
-  const out = [];
-  const placed = new Set();
-  const placePage = (rel) => {
-    if (placed.has(rel)) return;
-    if (!byFile.has(rel)) return;
-    placed.add(rel);
-    out.push(makeTodoItem(rel));
-  };
-  for (let i = 0; i < base.length; i++) {
-    const item = base[i];
-    const rel = itemPath(item);
-    const prev = base[i - 1];
-    const prevRel = prev ? itemPath(prev) : '';
-    if (rel !== prevRel) placePage(rel);
-    out.push(item);
-  }
-  for (const rel of byFile.keys()) {
-    if (placed.has(rel)) continue;
-    placePage(rel);
-  }
-  return out;
-};
-
 module.exports = {
   TEMPLATE_SHOW,
   AUTOSAVE_MS,
@@ -832,5 +776,4 @@ module.exports = {
   loadReview,
   loadTemplates,
   flushReview,
-  mergeTodos,
 };