reslop 0.1.0 → 0.1.2

package/lib/session.js

@@ -16,43 +16,51 @@ const { refreshIndexPatches } = require('./diff.js');
 const { renderFrame, presentRows, presentCursor } = render;
 const { noteInnerWidth, todoInnerWidth } = render;
 const { QUIT_FINISH, QUIT_CONTINUE } = render;
-const { actionFromKey, hitAction, decodeChunk, mouseKind } = keys;
+const { actionFromKey, hitAction, decodeChunk } = keys;
+const { FILES_DISABLED } = keys;
 const { cursorInWrap, wrapDoc, wrapMove } = wrap;
 const { AUTOSAVE_MS, listReviewNames, loadTemplates } = review;
 const { resolveReviewPath, loadReview, createStore } = review;
 const { noteCounts, hasNotes, setFeedback, addTodo } = review;
+const { applyImportedNotes } = review;
 const { setTodoText, removeTodo, flushReview, mergeTodos } = review;
 const { feedbackKey, checkLabel, rankedTemplates } = review;
 const { prefixTemplates, TEMPLATE_SHOW, rememberTemplate } = review;
 
 const LAYOUTS = ['unified', 'mixed', 'side'];
-const FILES_IDLE = ['add', 'unstage', 'revert', 'skip'];
-const FILES_MOVE = Object.assign(Object.create(null), {
-  next: 1,
-  prev: -1,
-});
-const ADD_STATUS = Object.assign(Object.create(null), {
-  staged: 'already staged',
-  commit: 'read only',
-});
-const UNSTAGE_STATUS = Object.assign(Object.create(null), {
-  commit: 'read only',
-});
-const REVERT_STATUS = Object.assign(Object.create(null), {
-  commit: 'read only',
-});
+const READ_ONLY_ORIGINS = ['commit', 'pr'];
 const ENTER_TERM = '\x1b[?1049h\x1b[?25l\x1b[?7l\x1b[?1002h\x1b[?1006h';
 const LEAVE_TERM =
   '\x1b[?1006l\x1b[?1002l\x1b[?7h\x1b[0 q\x1b[?12l\x1b[?25h\x1b[?1049l';
 const CURSOR_BLINK_MS = 500;
 
-const layoutStatus = (id) => (id === 'side' ? 'side-by-side' : id);
+const systemNow = () => new Date();
+
+const isReadOnlyOrigin = (origin) => READ_ONLY_ORIGINS.includes(origin);
+
+const relabelStatus = (entries, from, to) => {
+  if (!to) return entries;
+  return entries.map((entry) => {
+    if (entry.status !== from) return entry;
+    return { ...entry, status: to };
+  });
+};
+
+const depChangeKey = (item) => {
+  const change = item.dep && item.dep.change;
+  if (!change) return '';
+  return `${change.section}/${change.name}`;
+};
+
+const needsReload = (item) => item.reload === true;
 
 const itemKey = (item) => {
   if (!item || !item.file) return '';
   const rel = itemPath(item);
   if (item.origin === 'todo') return `todo:${rel}`;
   const origin = item.origin ?? '';
+  const depId = depChangeKey(item);
+  if (depId) return `${origin}:${rel}:dep:${depId}`;
   if (!item.hunk) {
     const block = item.blockId ?? '-';
     return `${origin}:${rel}:file:${block}`;
@@ -66,12 +74,12 @@ const itemKey = (item) => {
 const itemFeedKey = (item) => {
   if (!item || !item.file || item.origin === 'todo') return '';
   const hunk = item.hunk;
-  return feedbackKey({
-    file: itemPath(item),
-    oldStart: hunk ? hunk.oldStart : 0,
-    newStart: hunk ? hunk.newStart : 0,
-    blockId: item.blockId ?? 0,
-  });
+  const depId = depChangeKey(item);
+  const file = depId ? `${itemPath(item)}#${depId}` : itemPath(item);
+  const oldStart = hunk ? hunk.oldStart : 0;
+  const newStart = hunk ? hunk.newStart : 0;
+  const blockId = item.blockId ?? 0;
+  return feedbackKey({ file, oldStart, newStart, blockId });
 };
 
 const sameCursor = (a, b) => {
@@ -80,16 +88,16 @@ const sameCursor = (a, b) => {
   return a.x === b.x && a.y === b.y;
 };
 
-const QUIT_CONFIRM = Object.assign(Object.create(null), {
+const QUIT_CONFIRM = {
   [QUIT_FINISH]: (session) => session.finishQuit('ready'),
   [QUIT_CONTINUE]: (session) => session.finishQuit('editing'),
   'ctrl-c': (session) => session.finishQuit('ready'),
   escape: (session) => session.cancelQuit(),
-});
+};
 
 const COMPOSE_EDITS = ['backspace', 'delete', 'ctrl-z', 'ctrl-y'];
 
-const COMPOSE_SPECIAL = Object.assign(Object.create(null), {
+const COMPOSE_SPECIAL = {
   backspace: (session) => session.editor.backspace(),
   delete: (session) => session.editor.delete(),
   left: (session) => session.editor.move(-1),
@@ -103,13 +111,12 @@ const COMPOSE_SPECIAL = Object.assign(Object.create(null), {
   'ctrl-z': (session) => session.editor.undoEdit(),
   'ctrl-y': (session) => session.editor.redoEdit(),
   tab: (session) => session.applyTemplate(),
-});
+};
 
 const HANDLERS = {
   add: (session) => session.onAdd(),
   unstage: (session) => session.onUnstage(),
   revert: (session) => session.onRevert(),
-  skip: (session) => session.onSkip(),
   next: (session) => session.onNext(),
   prev: (session) => session.onPrev(),
   layout: (session) => session.onLayout(),
@@ -121,21 +128,20 @@ const HANDLERS = {
   quit: (session) => session.onQuit(),
   scrollUp: (session) => session.onScroll(-1),
   scrollDown: (session) => session.onScroll(1),
-  help: (session) => session.onHelp(),
 };
 
-const KEY_MODE = Object.assign(Object.create(null), {
+const KEY_MODE = {
   confirmQuit: (session, key) => session.onConfirmQuit(key),
   compose: (session, key) => session.handleComposeKey(key),
-});
+};
 
-const MOUSE_KIND = Object.assign(Object.create(null), {
+const MOUSE_KIND = {
   wheelUp: (session) => session.onScroll(-1),
   wheelDown: (session) => session.onScroll(1),
   press: (session, event) => session.onMousePress(event),
   drag: (session, event) => session.onMouseDrag(event),
   release: (session, event) => session.onMouseRelease(event),
-});
+};
 
 const RELEASE_HITS = [
   {
@@ -167,35 +173,40 @@ class Session {
     this.stdout = options.stdout;
     this.color = options.color ?? true;
     this.copyText = options.copyText ?? copyText;
-    this.getSize =
-      options.getSize ??
-      (() => ({
-        width: this.stdout.columns ?? 80,
-        height: this.stdout.rows ?? 24,
-      }));
-    this.pane = options.startPane ?? 'diff';
+    const termSize = () => {
+      const width = this.stdout.columns ?? 80;
+      const height = this.stdout.rows ?? 24;
+      return { width, height };
+    };
+    this.getSize = options.getSize ?? termSize;
+    this.pane = options.startPane ?? 'files';
     this.rev = options.rev ?? null;
-    this.revShort = options.revShort ?? '';
+    this.revShort = '';
+    this.sourceLabel = options.sourceLabel ?? '';
+    this.repoName = options.repoName ?? '';
+    this.change = options.change ?? null;
     this.readdirSync = options.readdirSync ?? fs.readdirSync;
     this.readFileSync = options.readFileSync ?? fs.readFileSync;
     this.writeFileSync = options.writeFileSync ?? fs.writeFileSync;
     this.mkdirSync = options.mkdirSync ?? fs.mkdirSync;
-    this.now = options.now ?? (() => new Date());
+    this.now = options.now ?? systemNow;
     this.forceNewReview = options.newReview === true;
+    this.audit = options.audit === true;
+    this.outdatedMap = options.outdatedMap;
+    this.auditMap = options.auditMap;
     this.setInterval = options.setInterval ?? setInterval;
     this.clearInterval = options.clearInterval ?? clearInterval;
   }
 
   resetState() {
     this.items = [];
+    this.dismissed = new Set();
     this.top = this.cwd;
     this.index = 0;
-    this.skipped = new Set();
     this.fileCursor = 0;
     this.reviewPath = null;
     this.scroll = 0;
     this.status = '';
-    this.help = false;
     this.layout = 'unified';
     this.done = false;
     this.exitCode = 0;
@@ -205,10 +216,11 @@ class Session {
     this.lastSize = null;
     this.selection = null;
     this.mouseAnchor = null;
-    this.dragging = false;
     this.pendingClick = null;
     this.abortController = null;
     this.notes = null;
+    this.didResumeReview = false;
+    this.importedRemoteNotes = false;
     this.mode = 'review';
     this.editor = null;
     this.composeKind = null;
@@ -230,11 +242,15 @@ class Session {
 
   load() {
     const here = itemKey(this.items[this.index]);
-    const loaded = this.repo.load(this.cwd, this.paths, {
-      commit: this.rev,
-    });
+    const commit = this.rev;
+    const audit = this.audit;
+    const outdatedMap = this.outdatedMap;
+    const auditMap = this.auditMap;
+    const options = { commit, audit, outdatedMap, auditMap };
+    const loaded = this.repo.load(this.cwd, this.paths, options);
     this.top = loaded.top ?? this.cwd;
-    this.items = loaded.items;
+    const items = loaded.items;
+    this.items = items.filter((item) => !this.dismissed.has(itemKey(item)));
     if (loaded.rev) {
       this.rev = loaded.rev;
     }
@@ -243,19 +259,22 @@ class Session {
     } else if (this.rev && !this.revShort) {
       this.revShort = this.rev.slice(0, 7);
     }
+    if (loaded.sourceLabel) this.sourceLabel = loaded.sourceLabel;
+    if (loaded.change) this.change = loaded.change;
+    const repository = this.change ? this.change.repository : '';
+    if (!this.repoName && repository) this.repoName = repository;
     if (!this.notes) this.initReview();
+    const imported = loaded.imported;
+    if (imported && !this.didResumeReview && !this.importedRemoteNotes) {
+      applyImportedNotes(this.notes, imported);
+      this.importedRemoteNotes = true;
+    }
     this.injectTodos();
     if (here) {
       const idx = this.items.findIndex((item) => itemKey(item) === here);
       if (idx >= 0) this.index = idx;
     }
-    this.pruneSkipped();
-    if (this.index >= this.items.length) {
-      this.index = Math.max(0, this.items.length - 1);
-    }
-    if (!this.isRemaining(this.items[this.index])) {
-      this.focusRemaining(this.index, 1);
-    }
+    this.clampIndex();
     this.syncReviewPath();
     this.syncFileCursor();
   }
@@ -273,6 +292,7 @@ class Session {
         templates,
         this.readFileSync,
       );
+      this.didResumeReview = true;
       return;
     }
     this.notes = createStore(resolved.reviewPath, templates);
@@ -284,62 +304,23 @@ class Session {
     this.items = mergeTodos(this.items, todos, extra);
   }
 
-  pruneSkipped() {
-    const live = new Set();
-    for (const item of this.items) live.add(itemKey(item));
-    for (const key of [...this.skipped]) {
-      if (!live.has(key)) this.skipped.delete(key);
-    }
-  }
-
-  isRemaining(item) {
-    if (!item) return false;
-    return !this.skipped.has(itemKey(item));
-  }
-
-  remainingIndices() {
-    const indices = [];
-    for (let i = 0; i < this.items.length; i++) {
-      if (this.isRemaining(this.items[i])) indices.push(i);
-    }
-    return indices;
-  }
-
-  focusRemaining(from, direction) {
-    const remaining = this.remainingIndices();
-    if (!remaining.length) {
+  clampIndex() {
+    if (!this.items.length) {
       this.index = 0;
-      return false;
-    }
-    if (direction >= 0) {
-      for (const i of remaining) {
-        if (i >= from) {
-          this.index = i;
-          return true;
-        }
-      }
-      this.index = remaining[remaining.length - 1];
-      return true;
+      return;
     }
-    for (let k = remaining.length - 1; k >= 0; k--) {
-      const i = remaining[k];
-      if (i <= from) {
-        this.index = i;
-        return true;
-      }
+    if (this.index < 0) this.index = 0;
+    if (this.index >= this.items.length) {
+      this.index = this.items.length - 1;
     }
-    this.index = remaining[0];
-    return true;
   }
 
   fileList() {
-    const remaining = new Set(this.remainingIndices());
-    const entries = fileEntries(this.items, remaining);
-    if (!this.revShort) return entries;
-    return entries.map((entry) => {
-      if (entry.status !== 'commit') return entry;
-      return { ...entry, status: this.revShort };
-    });
+    const entries = fileEntries(this.items);
+    if (this.sourceLabel) {
+      return relabelStatus(entries, 'pr', this.sourceLabel);
+    }
+    return relabelStatus(entries, 'commit', this.revShort);
   }
 
   syncReviewPath() {
@@ -348,7 +329,18 @@ class Session {
     this.reviewPath = item ? itemPath(item) : null;
   }
 
-  syncFileCursor() {
+  clampFileCursor() {
+    const files = this.fileList();
+    if (!files.length) {
+      this.fileCursor = 0;
+      return;
+    }
+    if (this.fileCursor >= files.length) {
+      this.fileCursor = files.length - 1;
+    }
+  }
+
+  followReviewPath() {
     const files = this.fileList();
     if (!files.length) {
       this.fileCursor = 0;
@@ -358,9 +350,17 @@ class Session {
     const idx = files.findIndex((entry) => entry.path === here);
     if (idx >= 0) {
       this.fileCursor = idx;
-    } else if (this.fileCursor >= files.length) {
-      this.fileCursor = files.length - 1;
+      return;
+    }
+    this.clampFileCursor();
+  }
+
+  syncFileCursor() {
+    if (this.pane === 'files') {
+      this.clampFileCursor();
+      return;
     }
+    this.followReviewPath();
   }
 
   counts() {
@@ -369,15 +369,11 @@ class Session {
       unstaged: 0,
       untracked: 0,
       commit: 0,
-      skipped: 0,
+      pr: 0,
       todo: 0,
       feedback: 0,
     };
     for (const item of this.items) {
-      if (!this.isRemaining(item)) {
-        counts.skipped += 1;
-        continue;
-      }
       if (!Object.hasOwn(counts, item.origin)) continue;
       counts[item.origin] += 1;
     }
@@ -388,16 +384,16 @@ class Session {
   }
 
   view() {
-    const remaining = this.remainingIndices();
-    const pos = remaining.indexOf(this.index);
     const templates = this.shownTemplates();
+    const change = this.change;
+    const isGithubPr = change && change.source === 'github-pr';
+    const repoName = this.repoName || path.basename(this.top || this.cwd || '');
     return {
       item: this.current(),
-      index: pos < 0 ? 0 : pos,
-      total: remaining.length,
+      index: this.index,
+      total: this.items.length,
       scroll: this.scroll,
       status: this.status,
-      help: this.help,
       layout: this.layout,
       counts: this.counts(),
       selection: this.selection,
@@ -405,12 +401,13 @@ class Session {
       files: this.fileList(),
       fileCursor: this.fileCursor,
       reviewPath: this.reviewPath,
-      repoName: path.basename(this.top || this.cwd || ''),
+      repoName,
       revShort: this.revShort || '',
+      sourceLabel: this.sourceLabel || '',
+      sourceKind: isGithubPr ? 'pr' : '',
       mode: this.mode,
       compose: this.composeView(),
       noteText: this.idleNoteText(),
-      noteKind: this.idleNoteKind(),
       todos: this.todoTexts(),
       todoFocus: this.clampedTodoFocus(),
       todoEdit: this.todoEditView(),
@@ -454,12 +451,9 @@ class Session {
       this.closeCompose();
     }
     if (this.pane === 'files') {
-      const delta = FILES_MOVE[action];
-      if (delta) {
-        this.onFileMove(delta);
-        return;
-      }
-      if (FILES_IDLE.includes(action)) return;
+      if (action === 'next') return void this.onFileMove(1);
+      if (action === 'prev') return void this.onFileMove(-1);
+      if (FILES_DISABLED.includes(action)) return;
     }
     const handler = HANDLERS[action];
     if (!handler) return;
@@ -467,9 +461,7 @@ class Session {
   }
 
   current() {
-    const item = this.items[this.index] ?? null;
-    if (!this.isRemaining(item)) return null;
-    return item;
+    return this.items[this.index] ?? null;
   }
 
   reloadAfterChange() {
@@ -482,7 +474,8 @@ class Session {
       this.status = 'nothing to review';
       return;
     }
-    this.focusRemaining(here, 1);
+    this.index = here;
+    this.clampIndex();
     this.syncReviewPath();
     this.syncFileCursor();
     this.scroll = 0;
@@ -508,7 +501,7 @@ class Session {
 
   keepOrigin(item, origin) {
     item.origin = origin;
-    refreshIndexPatches(this.items, item);
+    if (!item.dep) refreshIndexPatches(this.items, item);
     this.clearSelection();
     this.syncReviewPath();
     this.syncFileCursor();
@@ -524,27 +517,160 @@ class Session {
     return item;
   }
 
+  fileCursorEntry() {
+    const files = this.fileList();
+    return files[this.fileCursor] ?? null;
+  }
+
+  fileActionItems() {
+    const entry = this.fileCursorEntry();
+    if (!entry) return [];
+    const rel = entry.path;
+    const items = [];
+    for (const item of this.items) {
+      if (item.origin === 'todo') continue;
+      if (itemPath(item) !== rel) continue;
+      items.push(item);
+    }
+    return items;
+  }
+
+  guardFileGit(items) {
+    if (!items.length) return false;
+    const first = items[0];
+    if (isReadOnlyOrigin(first.origin)) {
+      this.status = 'read only';
+      return false;
+    }
+    return true;
+  }
+
+  applyFileGit(items, label, write, afterItem) {
+    try {
+      for (const item of items) {
+        write(this.top, item);
+        afterItem(item);
+      }
+      this.status = label;
+      return true;
+    } catch (error) {
+      this.status = error.message;
+      return false;
+    }
+  }
+
+  selectNextFileAfter(rel) {
+    const files = this.fileList();
+    if (!files.length) {
+      this.fileCursor = 0;
+      this.reviewPath = null;
+      return;
+    }
+    const idx = files.findIndex((entry) => entry.path === rel);
+    let at = idx < 0 ? this.fileCursor : idx + 1;
+    if (at >= files.length) at = files.length - 1;
+    const entry = files[at];
+    this.fileCursor = at;
+    this.reviewPath = entry.path;
+    this.index = entry.firstIndex;
+  }
+
+  onFileAdd() {
+    const items = this.fileActionItems();
+    if (!this.guardFileGit(items)) return;
+    const targets = items.filter((item) => item.origin !== 'staged');
+    if (!targets.length) {
+      this.status = 'already staged';
+      return;
+    }
+    const rel = itemPath(targets[0]);
+    const proposed = targets.some((item) => needsReload(item));
+    const ok = this.applyFileGit(
+      targets,
+      'staged',
+      (top, current) => this.repo.add(top, current),
+      (item) => this.keepOrigin(item, 'staged'),
+    );
+    if (ok && proposed) this.reloadAfterChange();
+    if (ok) this.selectNextFileAfter(rel);
+  }
+
+  onFileUnstage() {
+    const items = this.fileActionItems();
+    if (!this.guardFileGit(items)) return;
+    const targets = items.filter((item) => item.origin === 'staged');
+    if (!targets.length) {
+      this.status = 'not staged';
+      return;
+    }
+    const rel = itemPath(targets[0]);
+    const ok = this.applyFileGit(
+      targets,
+      'unstaged',
+      (top, current) => this.repo.unstage(top, current),
+      (item) => {
+        const origin = item.file.isNew === true ? 'untracked' : 'unstaged';
+        this.keepOrigin(item, origin);
+      },
+    );
+    if (ok) this.selectNextFileAfter(rel);
+  }
+
+  onFileRevert() {
+    const items = this.fileActionItems();
+    if (!this.guardFileGit(items)) return;
+    const rel = itemPath(items[0]);
+    for (const item of items) {
+      if (needsReload(item)) this.dismissed.add(itemKey(item));
+    }
+    let ok = false;
+    try {
+      this.repo.revertFile(this.top, rel, items);
+      this.status = 'reverted';
+      ok = true;
+    } catch (error) {
+      this.status = error.message;
+    }
+    this.reloadAfterChange();
+    if (ok) this.selectNextFileAfter(rel);
+  }
+
   onAdd() {
+    if (this.pane === 'files') return void this.onFileAdd();
     const item = this.activeDiffItem();
     if (!item) return;
-    const blocked = ADD_STATUS[item.origin];
-    if (blocked) {
-      this.status = blocked;
+    if (isReadOnlyOrigin(item.origin)) {
+      this.status = 'read only';
+      return;
+    }
+    if (item.origin === 'staged') {
+      this.status = 'already staged';
       return;
     }
     this.applyGit(
       item,
       'staged',
       (top, current) => this.repo.add(top, current),
-      () => this.keepOrigin(item, 'staged'),
+      () => {
+        if (needsReload(item)) {
+          this.reloadAfterChange();
+          return;
+        }
+        this.keepOrigin(item, 'staged');
+      },
     );
   }
 
   onUnstage() {
+    if (this.pane === 'files') return void this.onFileUnstage();
     const item = this.activeDiffItem();
     if (!item) return;
+    if (isReadOnlyOrigin(item.origin)) {
+      this.status = 'read only';
+      return;
+    }
     if (item.origin !== 'staged') {
-      this.status = UNSTAGE_STATUS[item.origin] ?? 'not staged';
+      this.status = 'not staged';
       return;
     }
     const origin = item.file.isNew === true ? 'untracked' : 'unstaged';
@@ -557,62 +683,39 @@ class Session {
   }
 
   onRevert() {
+    if (this.pane === 'files') return void this.onFileRevert();
     const item = this.activeDiffItem();
     if (!item) return;
-    const blocked = REVERT_STATUS[item.origin];
-    if (blocked) {
-      this.status = blocked;
+    if (isReadOnlyOrigin(item.origin)) {
+      this.status = 'read only';
       return;
     }
     this.applyGit(
       item,
       'reverted',
       (top, current) => this.repo.revert(top, current),
-      () => this.reloadAfterChange(),
+      () => {
+        if (needsReload(item)) this.dismissed.add(itemKey(item));
+        this.reloadAfterChange();
+      },
     );
   }
 
-  onSkip() {
-    const item = this.items[this.index];
-    if (!this.isRemaining(item)) return;
-    this.skipped.add(itemKey(item));
-    this.scroll = 0;
-    this.clearSelection();
-    const remaining = this.remainingIndices();
-    if (!remaining.length) {
-      this.status = 'all skipped';
-      return;
-    }
-    this.focusRemaining(this.index + 1, 1);
-    this.syncReviewPath();
-    this.status = 'skipped';
-  }
-
   moveRemaining(step) {
-    const remaining = this.remainingIndices();
-    if (!remaining.length) {
-      this.status = 'all skipped';
-      return;
-    }
-    const pos = remaining.indexOf(this.index);
-    if (pos < 0) {
-      this.focusRemaining(this.index, step);
-      this.scroll = 0;
-      this.status = '';
-      this.clearSelection();
-      this.syncReviewPath();
+    if (!this.items.length) {
+      this.status = 'nothing to review';
       return;
     }
-    const next = pos + step;
+    const next = this.index + step;
     if (next < 0) {
       this.status = 'first block';
       return;
     }
-    if (next >= remaining.length) {
+    if (next >= this.items.length) {
       this.status = 'last block';
       return;
     }
-    this.index = remaining[next];
+    this.index = next;
     this.scroll = 0;
     this.status = '';
     this.clearSelection();
@@ -620,18 +723,10 @@ class Session {
   }
 
   onNext() {
-    if (this.mode === 'compose') {
-      this.commitCompose();
-      this.closeCompose();
-    }
     this.moveRemaining(1);
   }
 
   onPrev() {
-    if (this.mode === 'compose') {
-      this.commitCompose();
-      this.closeCompose();
-    }
     this.moveRemaining(-1);
   }
 
@@ -680,7 +775,6 @@ class Session {
       this.status = 'not a diff block';
       return;
     }
-    this.help = false;
     const prev = this.notes.feedback.get(itemFeedKey(item));
     const text = prev ? prev.text : '';
     this.openCompose('feedback', text);
@@ -688,8 +782,7 @@ class Session {
 
   todoFile() {
     if (this.pane === 'files') {
-      const files = this.fileList();
-      const entry = files[this.fileCursor];
+      const entry = this.fileCursorEntry();
       return entry ? entry.path : '';
     }
     const item = this.current();
@@ -768,9 +861,7 @@ class Session {
     this.injectTodos();
     const left = this.fileTodos(file);
     if (!left.length) {
-      if (!this.isRemaining(this.items[this.index])) {
-        this.focusRemaining(this.index, 1);
-      }
+      this.clampIndex();
     } else if (this.todoFocus >= left.length) {
       this.todoFocus = left.length - 1;
     }
@@ -799,7 +890,6 @@ class Session {
   createTodo(file) {
     if (!file) return;
     this.pane = 'diff';
-    this.help = false;
     this.scroll = 0;
     this.clearSelection();
     this.startDraftCompose(file);
@@ -841,8 +931,8 @@ class Session {
 
   commitTodo(text) {
     const id = this.composeTodoId;
-    const prev =
-      id === null ? null : this.notes.todos.find((entry) => entry.id === id);
+    const todos = this.notes.todos;
+    const prev = id === null ? null : todos.find((entry) => entry.id === id);
     const current = this.current();
     const file = prev ? prev.file : itemPath(current) || this.todoDraftFile;
     if (prev) {
@@ -873,9 +963,7 @@ class Session {
     this.templateFocus = false;
     this.todoDraftFile = null;
     this.injectTodos();
-    if (!this.isRemaining(this.items[this.index])) {
-      this.focusRemaining(this.index, 1);
-    }
+    this.clampIndex();
     this.syncReviewPath();
     this.syncFileCursor();
   }
@@ -930,11 +1018,10 @@ class Session {
   composeView() {
     if (this.mode !== 'compose' || !this.editor) return null;
     if (this.composeKind !== 'feedback') return null;
-    return {
-      kind: 'feedback',
-      text: this.editor.text,
-      cursor: this.editor.cursor,
-    };
+    const kind = this.composeKind;
+    const text = this.editor.text;
+    const cursor = this.editor.cursor;
+    return { kind, text, cursor };
   }
 
   todoEditView() {
@@ -945,8 +1032,6 @@ class Session {
 
   fileTodos(rel) {
     if (!this.notes || !rel) return [];
-    const page = { origin: 'todo', file: { newPath: rel, oldPath: rel } };
-    if (!this.isRemaining(page)) return [];
     const todos = [];
     for (const todo of this.notes.todos) {
       if (todo.file !== rel) continue;
@@ -959,10 +1044,8 @@ class Session {
     const item = this.current();
     if (!item || item.origin !== 'todo') return [];
     const texts = [];
-    const editingId =
-      this.mode === 'compose' && this.composeKind === 'todo'
-        ? this.composeTodoId
-        : null;
+    const editingTodo = this.mode === 'compose' && this.composeKind === 'todo';
+    const editingId = editingTodo ? this.composeTodoId : null;
     for (const todo of this.fileTodos(itemPath(item))) {
       const live = editingId === todo.id && this.editor;
       const text = live ? this.editor.text : todo.text;
@@ -981,7 +1064,7 @@ class Session {
   }
 
   clampedTodoFocus() {
-    const last = Math.max(0, this.todoTexts().length - 1);
+    const last = Math.max(0, this.todoRowCount() - 1);
     if (this.todoFocus < 0) return 0;
     if (this.todoFocus > last) return last;
     return this.todoFocus;
@@ -1004,12 +1087,6 @@ class Session {
     else this.todoFocus = next;
   }
 
-  idleNoteKind() {
-    const item = this.current();
-    if (!item || item.origin === 'todo') return '';
-    return 'feedback';
-  }
-
   idleNoteText() {
     if (!this.notes) return '';
     if (this.mode === 'compose') return '';
@@ -1023,11 +1100,10 @@ class Session {
   flushReview(force) {
     if (!this.notes) return false;
     try {
-      return flushReview(this.notes, {
-        writeFileSync: this.writeFileSync,
-        mkdirSync: this.mkdirSync,
-        force: force === true,
-      });
+      const writeFileSync = this.writeFileSync;
+      const mkdirSync = this.mkdirSync;
+      const io = { writeFileSync, mkdirSync, force: force === true };
+      return flushReview(this.notes, io);
     } catch (error) {
       this.status = error.message;
       return false;
@@ -1146,9 +1222,8 @@ class Session {
 
   showFiles() {
     this.pane = 'files';
-    this.help = false;
     this.clearSelection();
-    this.syncFileCursor();
+    this.followReviewPath();
   }
 
   onEscape() {
@@ -1163,7 +1238,6 @@ class Session {
     if (this.pane === 'files') {
       if (!this.reviewPath) return;
       this.pane = 'diff';
-      this.help = false;
       this.clearSelection();
       return;
     }
@@ -1183,7 +1257,6 @@ class Session {
     this.pane = 'diff';
     this.scroll = 0;
     this.status = '';
-    this.help = false;
     this.clearSelection();
     this.syncReviewPath();
   }
@@ -1191,7 +1264,7 @@ class Session {
   onFileMove(delta) {
     const files = this.fileList();
     if (!files.length) {
-      this.status = 'all skipped';
+      this.status = 'nothing to review';
       return;
     }
     const next = this.fileCursor + delta;
@@ -1204,6 +1277,9 @@ class Session {
       return;
     }
     this.fileCursor = next;
+    const entry = files[next];
+    this.reviewPath = entry.path;
+    this.index = entry.firstIndex;
     this.status = '';
   }
 
@@ -1221,16 +1297,11 @@ class Session {
     this.clearSelection();
   }
 
-  onHelp() {
-    this.help = !this.help;
-    this.clearSelection();
-  }
-
   onLayout() {
     const i = LAYOUTS.indexOf(this.layout);
     const next = (i < 0 ? 0 : i + 1) % LAYOUTS.length;
     this.layout = LAYOUTS[next];
-    this.status = layoutStatus(this.layout);
+    this.status = this.layout === 'side' ? 'side-by-side' : this.layout;
     this.scroll = 0;
     this.clearSelection();
   }
@@ -1245,7 +1316,6 @@ class Session {
   onMousePress(event) {
     const cell = this.mouseCell(event);
     this.mouseAnchor = cell;
-    this.dragging = false;
     this.pendingClick = null;
     this.selection = null;
     if (this.lastFrame && cell.y === this.lastFrame.height) {
@@ -1255,7 +1325,6 @@ class Session {
 
   onMouseDrag(event) {
     if (!this.mouseAnchor) return;
-    this.dragging = true;
     this.pendingClick = null;
     this.selection = {
       start: this.mouseAnchor,
@@ -1298,7 +1367,6 @@ class Session {
     const clickId = this.pendingClick;
     this.pendingClick = null;
     this.mouseAnchor = null;
-    this.dragging = false;
     if (this.clickButton(cell, clickId)) return;
     const frame = this.lastFrame;
     for (const target of RELEASE_HITS) {
@@ -1328,7 +1396,7 @@ class Session {
       return;
     }
     if (event.type !== 'mouse') return;
-    const kind = mouseKind(event);
+    const kind = event.kind;
     const wheel = kind === 'wheelUp' || kind === 'wheelDown';
     if (!wheel && event.btn !== 0 && event.btn !== undefined) return;
     const handler = MOUSE_KIND[kind];
@@ -1406,11 +1474,6 @@ class Session {
 }
 
 module.exports = {
-  LAYOUTS,
-  ENTER_TERM,
   LEAVE_TERM,
-  CURSOR_BLINK_MS,
-  HANDLERS,
-  itemKey,
   Session,
 };