reslop 0.2.3 → 0.2.4

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## [Unreleased][unreleased]
 
+## [0.2.4][] - 2026-09-28
+
+- Add AI instructions to run `npm t` or `reslop t -- node --test`
+- Cleanup old logs (older than 5 days)
+- Rename TODO list to Project Backlog and Issues
+- Word wrap in view and horizontal scrolling in edit mode
+- Return to uncommitted changes when viewing history in read-only mode
+- Improve TUI/TUX for commits and npm commands
+- Remove circular dependencies between Git modules
+- Unify list rendering and keyboard navigation
+- Simplify dependency transformations and remove redundant wrappers
+
 ## [0.2.3][] - 2026-09-27
 
 - Add brief and full commit lists, toggled with `v`
@@ -129,7 +141,8 @@
 - Feedback and todos that produce a repair plan for an agent
 - Intra-line highlighting in unified, mixed, and side-by-side layouts
 
-[unreleased]: https://github.com/tshemsedinov/reslop/compare/v0.2.3...HEAD
+[unreleased]: https://github.com/tshemsedinov/reslop/compare/v0.2.4...HEAD
+[0.2.4]: https://github.com/tshemsedinov/reslop/compare/v0.2.3...v0.2.4
 [0.2.3]: https://github.com/tshemsedinov/reslop/compare/v0.2.1...v0.2.3
 [0.2.1]: https://github.com/tshemsedinov/reslop/compare/v0.2.0...v0.2.1
 [0.2.0]: https://github.com/tshemsedinov/reslop/compare/v0.1.9...v0.2.0

package/README.md

@@ -11,21 +11,24 @@
 
 > Turn generated changes into owned changes
 
-Open a change, leave findings and todos, and prepare a repair plan
+Open a change, leave findings and a backlog, and prepare a repair plan
 for the agent.
 
 ```text
 Review → Plan → Repair → Verify
 ```
 
-- Leave feedback, issues/todos, and in-place code proposals as a repair plan
+- Leave feedback, a project backlog, and in-place code proposals as a repair plan
 - Review uncommitted git diffs, a given commit, a GitHub PR, or a GitLab MR
 - File-scope review mode: not just diffs, with editor mode
 - Stage, unstage, or revert each contiguous block of diff lines
 - Auto-reload local diffs when files change, keeping the current screen
 - Import GitHub PR and GitLab MR review comments into the local plan for AI
-- Commits list: commit, amend, fixup, reword, drop, pull, and push
+- Commits list: brief and full, view a commit's diff, commit, amend, fixup, reword, drop, pull, and push
 - Branches: checkout, create, rebase, drop, pull, and push
+- npm scripts and bins: run, edit, and reorder; double-click to run; clean logs older than 5 days
+- Long lines wrap in the diff view; the editor scrolls horizontally
+- Reduced test and lint output for agents: failures and a short summary, not passing noise
 - Review each npm dependency once across `package.json` and the lockfile
 - Propose unused removals, npm audit fixes, and outdated updates as diffs
 - Code highlighting and intra-line diffs: unified, mixed, and side-by-side
@@ -61,6 +64,8 @@ Prefix each program in an npm script with `reslop t --`. Keep `&&` between those
 
 The report is Markdown: failures, diagnostics, and a short summary. Passing results are omitted and similar problems are grouped. The same text is saved under `.log/`. Set `RESLOP_OUTPUT=raw` to pass the command through unchanged. The review screen runs scripts in raw mode and applies its own filter once.
 
+Run the full check with `npm t`. Run one or more test files with `reslop t -- node --test <files>`. New reviews include those two commands in the agent instructions.
+
 Reviews go in `.review/YYYY-MM-DD-NN.md` with frontmatter `status`:
 
 - `editing`: still writing the review in reslop

package/bin/reslop.js

@@ -1,8 +1,10 @@
 #!/usr/bin/env node
 'use strict';
 
-const { run, errorMessage } = require('../lib/cli.js');
-const { LEAVE_TERM } = require('../lib/session.js');
+const cli = require('../lib/cli.js');
+const { run, errorMessage } = cli;
+const session = require('../lib/session.js');
+const { LEAVE_TERM } = session;
 
 const fail = (reason) => {
   try {

package/lib/api.js

@@ -10,7 +10,8 @@ const detect = require('./detect.js');
 const github = require('./github.js');
 const gitlab = require('./gitlab.js');
 const source = require('./source.js');
-const { Session } = require('./session.js');
+const session = require('./session.js');
+const { Session } = session;
 const cli = require('./cli.js');
 
 module.exports = {

package/lib/cli.js

@@ -2,16 +2,25 @@
 
 const path = require('node:path');
 
-const { exists } = require('metautil');
-
-const { createGitRepo } = require('./git.js');
-const { githubToken, loadPullRequest } = require('./github.js');
-const { gitlabToken, loadMergeRequest } = require('./gitlab.js');
-const { selectChangeSource, createLoadedSource } = require('./source.js');
-const { Session } = require('./session.js');
-const { capabilitiesFor, attachCapabilities } = require('./capabilities.js');
-const { setTheme } = require('./ansi.js');
-const { runCapture } = require('./report-run.js');
+const metautil = require('metautil');
+const { exists } = metautil;
+
+const git = require('./git.js');
+const { createGitRepo } = git;
+const github = require('./github.js');
+const { githubToken, loadPullRequest } = github;
+const gitlab = require('./gitlab.js');
+const { gitlabToken, loadMergeRequest } = gitlab;
+const source = require('./source.js');
+const { selectChangeSource, createLoadedSource } = source;
+const session = require('./session.js');
+const { Session } = session;
+const capabilities = require('./capabilities.js');
+const { capabilitiesFor, attachCapabilities } = capabilities;
+const ansi = require('./ansi.js');
+const { setTheme } = ansi;
+const reportRun = require('./report-run.js');
+const { runCapture } = reportRun;
 
 const HEX_REV = /^[0-9a-f]{4,40}$/i;
 const HEAD_REV = /^HEAD([~^].*)?$/i;

package/lib/clipboard.js

@@ -1,9 +1,12 @@
 'use strict';
 
-const { spawnSync } = require('node:child_process');
+const childProcess = require('node:child_process');
+const { spawnSync } = childProcess;
 
-const { ESC } = require('./ansi.js');
-const { clipTools, spawnBase } = require('./utilities.js');
+const ansi = require('./ansi.js');
+const { ESC } = ansi;
+const utilities = require('./utilities.js');
+const { clipTools, spawnBase } = utilities;
 
 const CLIP_TIMEOUT_MS = 1000;
 

package/lib/deps-diff.js

@@ -1,13 +1,13 @@
 'use strict';
 
-const { jsonParse } = require('metautil');
-const { itemPath } = require('./files.js');
-const {
-  countSides,
-  formatHunkHeader,
-  formatPatch,
-} = require('./diff/patch.js');
-const { isCtxType } = require('./diff/diff.js');
+const metautil = require('metautil');
+const { jsonParse } = metautil;
+const files = require('./files.js');
+const { itemPath } = files;
+const patch = require('./diff/patch.js');
+const { countSides, formatHunkHeader, formatPatch } = patch;
+const diff = require('./diff/diff.js');
+const { isCtxType } = diff;
 const manifest = require('./manifest.js');
 const { MANIFEST, LOCKFILE, SECTION_KEYS, depFileMeta, relOf } = manifest;
 const { listFilePath, readSections, emptySectionMaps } = manifest;
@@ -37,14 +37,8 @@ const diffMaps = (oldMap, newMap, section) => {
   return changes;
 };
 
-const diffSections = (oldMaps, newMaps) => {
-  const changes = [];
-  for (const key of SECTION_KEYS) {
-    const found = diffMaps(oldMaps[key], newMaps[key], key);
-    for (const change of found) changes.push(change);
-  }
-  return changes;
-};
+const diffSections = (oldMaps, newMaps) =>
+  SECTION_KEYS.flatMap((key) => diffMaps(oldMaps[key], newMaps[key], key));
 
 const fieldText = (value) => {
   if (value === undefined) return '';
@@ -263,15 +257,8 @@ const compareChangeOrder = (left, right) => {
 
 const orderedChanges = (changes) => [...changes].sort(compareChangeOrder);
 
-const kindItems = (items, kind) => {
-  const matched = [];
-  for (const item of items) {
-    const rel = itemPath(item);
-    const meta = depFileMeta(rel);
-    if (meta && meta.kind === kind) matched.push(item);
-  }
-  return matched;
-};
+const kindItems = (items, kind) =>
+  items.filter((item) => depFileMeta(itemPath(item))?.kind === kind);
 
 const readGroupSnapshot = (group, readSides) => {
   const origin = group.origin;
@@ -457,26 +444,10 @@ const entryText = (change, side) => {
   return `"${change.name}": "${version}"`;
 };
 
-const ctxLine = (text) => {
-  const type = 'ctx';
-  return { type, text };
-};
-
-const warnLine = (text) => {
-  const type = 'warn';
-  return { type, text };
-};
-
-const noteLine = (text) => {
-  const type = 'note';
-  return { type, text };
-};
-
-const noteSepLine = () => {
-  const type = 'noteSep';
-  const text = '';
-  return { type, text };
-};
+const ctxLine = (text) => ({ type: 'ctx', text });
+const warnLine = (text) => ({ type: 'warn', text });
+const noteLine = (text) => ({ type: 'note', text });
+const noteSepLine = () => ({ type: 'noteSep', text: '' });
 
 const changeLine = (type, change, side) => {
   const text = entryText(change, side);
@@ -522,12 +493,7 @@ const auditNoteTexts = (audit) => {
   }
   const severity = audit.severity;
   if (!notes.length) return [`npm audit  ${severity}`];
-  const { length } = notes;
-  const texts = new Array(length);
-  for (let i = 0; i < length; i++) {
-    texts[i] = `npm audit  ${severity}  ${notes[i]}`;
-  }
-  return texts;
+  return notes.map((note) => `npm audit  ${severity}  ${note}`);
 };
 
 const pushAuditNotes = (rows, audit) => {
@@ -562,13 +528,12 @@ const buildRows = (change, oldCount, newCount) => {
   return rows;
 };
 
-const toHunkLine = (row, blockId) => {
-  const type = row.type;
-  const text = row.text;
-  const noNl = false;
-  const id = isCtxType(type) ? null : blockId;
-  return { type, text, noNl, blockId: id };
-};
+const toHunkLine = (row, blockId) => ({
+  type: row.type,
+  text: row.text,
+  noNl: false,
+  blockId: isCtxType(row.type) ? null : blockId,
+});
 
 const makeHunk = (rows, blockId) => {
   const lines = rows.map((row) => toHunkLine(row, blockId));
@@ -600,21 +565,13 @@ const filesForChange = (change, files) => {
     if (meta.kind === 'manifest') manifest.push(rel);
     else lockfile.push(rel);
   }
-  if (change.section === 'field') {
-    if (manifest.length) return manifest;
-    return files;
-  }
-  if (change.section === 'resolved') {
-    if (lockfile.length) return lockfile;
-    return files;
-  }
-  if (change.propose) {
-    if (manifest.length) return manifest;
-    return files;
+  if (change.section === 'resolved') return lockfile.length ? lockfile : files;
+  if (change.section === 'field' || change.propose) {
+    return manifest.length ? manifest : files;
   }
-  const both = [...manifest, ...lockfile];
-  if (both.length) return both;
-  return files;
+  return manifest.length || lockfile.length
+    ? [...manifest, ...lockfile]
+    : files;
 };
 
 const pickPrimary = (items) => {
@@ -630,36 +587,28 @@ const makeDepItem = (origin, fileItems, hidden, change, extra, blockId) => {
   const allFiles = uniquePaths(fileItems);
   const files = change ? filesForChange(change, allFiles) : allFiles;
   const primary = pickPrimary(fileItems);
-  const oldPath = listFilePath(itemPath(primary));
-  const newPath = oldPath;
-  const isNew = fileItems.some((item) => item.file && item.file.isNew);
-  const isDeleted = fileItems.every((item) => item.file && item.file.isDeleted);
-  let oldCount = 0;
-  let newCount = 0;
-  if (!change) {
-    oldCount = extra.oldCount ?? 0;
-    newCount = extra.newCount ?? 0;
-  }
-  const rows = buildRows(change, oldCount, newCount);
+  const rel = listFilePath(itemPath(primary));
+  const counts = change ? {} : extra;
+  const rows = buildRows(change, counts.oldCount ?? 0, counts.newCount ?? 0);
   const hunk = makeHunk(rows, blockId);
-  const isBinary = false;
-  const preamble = [];
-  const hunks = [hunk];
-  const file = {
-    oldPath,
-    newPath,
-    isNew,
-    isDeleted,
-    isBinary,
-    preamble,
-    hunks,
+  return {
+    origin,
+    file: {
+      oldPath: rel,
+      newPath: rel,
+      isNew: fileItems.some((item) => item.file?.isNew),
+      isDeleted: fileItems.every((item) => item.file?.isDeleted),
+      isBinary: false,
+      preamble: [],
+      hunks: [hunk],
+    },
+    hunk,
+    blockId,
+    patchAdd: '',
+    patchRevert: '',
+    dep: { files, items: hidden, change },
+    reload: !!change?.propose,
   };
-  const patchAdd = '';
-  const patchRevert = '';
-  const items = hidden;
-  const dep = { files, items, change };
-  const reload = !!(change && change.propose);
-  return { origin, file, hunk, blockId, patchAdd, patchRevert, dep, reload };
 };
 
 const makeDepItems = (origin, fileItems, hidden, summary) => {
@@ -670,16 +619,14 @@ const makeDepItems = (origin, fileItems, hidden, summary) => {
     if (oldCount === newCount) return [];
     return [makeDepItem(origin, fileItems, hidden, null, summary, 0)];
   }
-  const { length } = ordered;
-  const items = new Array(length);
-  for (let i = 0; i < length; i++) {
-    items[i] = makeDepItem(origin, fileItems, hidden, ordered[i], summary, i);
-  }
-  return items;
+  return ordered.map((change, index) =>
+    makeDepItem(origin, fileItems, hidden, change, summary, index),
+  );
 };
 
-const stubFileItem = (rel, origin) => {
-  const file = {
+const stubFileItem = (rel, origin) => ({
+  origin,
+  file: {
     oldPath: rel,
     newPath: rel,
     isNew: false,
@@ -687,13 +634,12 @@ const stubFileItem = (rel, origin) => {
     isBinary: false,
     preamble: [],
     hunks: [],
-  };
-  const hunk = null;
-  const blockId = 0;
-  const patchAdd = '';
-  const patchRevert = '';
-  return { origin, file, hunk, blockId, patchAdd, patchRevert };
-};
+  },
+  hunk: null,
+  blockId: 0,
+  patchAdd: '',
+  patchRevert: '',
+});
 
 module.exports = {
   DEP_CAPTION,

package/lib/deps.js

@@ -2,8 +2,10 @@
 
 const fs = require('node:fs');
 const path = require('node:path');
-const { jsonParse, isHashObject } = require('metautil');
-const { itemPath } = require('./files.js');
+const metautil = require('metautil');
+const { jsonParse, isHashObject } = metautil;
+const files = require('./files.js');
+const { itemPath } = files;
 const manifest = require('./manifest.js');
 const { MANIFEST, LOCKFILE, SECTION_KEYS, depFileMeta, relOf } = manifest;
 const { listFilePath, readSections, lockEntries, lockPackageCount } = manifest;
@@ -73,8 +75,6 @@ const pkgHasExportEntry = (pkg) => {
   return Object.keys(exported).length > 0;
 };
 
-const isTypeKey = (key) => key === 'types' || key === 'typings';
-
 const isRuntimeFile = (value) => {
   if (typeof value !== 'string' || !value) return false;
   return !value.endsWith('.d.ts');
@@ -82,16 +82,11 @@ const isRuntimeFile = (value) => {
 
 const hasRuntimeExport = (value) => {
   if (isRuntimeFile(value)) return true;
-  if (Array.isArray(value)) {
-    for (const item of value) {
-      if (hasRuntimeExport(item)) return true;
-    }
-    return false;
-  }
+  if (Array.isArray(value)) return value.some(hasRuntimeExport);
   if (!value || typeof value !== 'object') return false;
   const keys = Object.keys(value);
   for (const key of keys) {
-    if (isTypeKey(key)) continue;
+    if (key === 'types' || key === 'typings') continue;
     if (hasRuntimeExport(value[key])) return true;
   }
   return false;
@@ -220,14 +215,7 @@ const declaredDepNames = (root, pkgDir) => {
     return [];
   }
   const maps = readSections(jsonParse(text));
-  let size = 0;
-  for (const key of SECTION_KEYS) size += maps[key].size;
-  const names = new Array(size);
-  let i = 0;
-  for (const key of SECTION_KEYS) {
-    for (const name of maps[key].keys()) names[i++] = name;
-  }
-  return names;
+  return SECTION_KEYS.flatMap((key) => [...maps[key].keys()]);
 };
 
 const collectExportMeta = (root, extra = {}) => {
@@ -382,16 +370,7 @@ const markChanges = (changes, report, kind) => {
   });
 };
 
-const markAuditChanges = (changes, audit) =>
-  markChanges(changes, audit, 'audit');
-
-const markOutdatedChanges = (changes, outdated) =>
-  markChanges(changes, outdated, 'outdated');
-
-const depExports = (exportEntries, name) => {
-  if (!exportEntries || !exportEntries.has(name)) return false;
-  return exportEntries.get(name) === true;
-};
+const depExports = (exportEntries, name) => exportEntries?.get(name) === true;
 
 const isAddedDep = (change) => {
   if (change.action !== 'added') return false;
@@ -429,23 +408,12 @@ const unusedDeclaredNames = (pkg, used) => {
 };
 
 const markUnusedChanges = (changes, used, pkg, exportEntries) => {
-  if (!pkgHasExportEntry(pkg)) return changes;
-  if (!used || !used.size) return changes;
-  const { length } = changes;
-  const marked = new Array(length);
-  for (let i = 0; i < length; i++) {
-    const change = changes[i];
-    if (!isAddedDep(change) || used.has(change.name)) {
-      marked[i] = change;
-      continue;
-    }
-    if (!depExports(exportEntries, change.name)) {
-      marked[i] = change;
-      continue;
-    }
-    marked[i] = { ...change, unused: true, propose: true };
-  }
-  return marked;
+  if (!pkgHasExportEntry(pkg) || !used?.size) return changes;
+  return changes.map((change) => {
+    if (!isAddedDep(change) || used.has(change.name)) return change;
+    if (!depExports(exportEntries, change.name)) return change;
+    return { ...change, unused: true, propose: true };
+  });
 };
 
 const unusedDeclaredChanges = (pkg, used, exportEntries) => {
@@ -551,18 +519,16 @@ const loadExportEntries = (names, extra = {}, pkgDir) => {
   if (!missing.length) return supplied ?? new Map();
   const found = collectExportEntries(extra.root, dir, missing);
   if (!supplied) return found;
-  const merged = new Map(supplied);
-  for (const name of found.keys()) merged.set(name, found.get(name));
-  return merged;
+  return new Map([...supplied, ...found]);
 };
 
 const groupedSynthetics = (synthetic) => {
   const byPath = new Map();
-  for (const key of synthetic.keys()) {
+  for (const [key, items] of synthetic) {
     const dir = key.slice(key.indexOf('\0') + 1);
     const rel = relOf(dir, MANIFEST);
     const list = byPath.get(rel) ?? [];
-    for (const item of synthetic.get(key)) list.push(item);
+    for (const item of items) list.push(item);
     byPath.set(rel, list);
   }
   return byPath;
@@ -570,12 +536,10 @@ const groupedSynthetics = (synthetic) => {
 
 const spliceAtSlots = (out, synthetic, slot) => {
   const byPath = groupedSynthetics(synthetic);
-  const inserts = new Array(byPath.size);
-  let i = 0;
-  for (const rel of byPath.keys()) {
-    const at = slot.has(rel) ? slot.get(rel) : out.length;
-    inserts[i++] = { at, items: byPath.get(rel) };
-  }
+  const inserts = [...byPath].map(([rel, items]) => ({
+    at: slot.get(rel) ?? out.length,
+    items,
+  }));
   inserts.sort((left, right) => right.at - left.at);
   for (const insert of inserts) {
     out.splice(insert.at, 0, ...insert.items);
@@ -610,8 +574,7 @@ const foldDepItems = (
   const folded = new Set();
   const synthetic = new Map();
   const leftoverNames = new Map();
-  for (const key of groups.keys()) {
-    const group = groups.get(key);
+  for (const [key, group] of groups) {
     const summary = summarizeGroup(group, readSides);
     const unusedNames = unusedChangeNames(summary.changes, usedNames);
     const exportEntries = loadExportEntries(unusedNames, extra, group.dir);
@@ -621,8 +584,8 @@ const foldDepItems = (
       summary.pkg,
       exportEntries,
     );
-    const audited = markAuditChanges(unused, audit);
-    const changes = markOutdatedChanges(audited, outdated);
+    const audited = markChanges(unused, audit, 'audit');
+    const changes = markChanges(audited, outdated, 'outdated');
     const next = { ...summary, changes };
     leftoverNames.set(key, foldedDepNames(next.changes));
     const taken = takeManifestItems(next);

package/lib/detect.js

@@ -1,9 +1,11 @@
 'use strict';
 
 const path = require('node:path');
-const { fileExt } = require('metautil');
+const metautil = require('metautil');
+const { fileExt } = metautil;
 
-const { mapLang } = require('./highlight.js');
+const highlight = require('./highlight.js');
+const { mapLang } = highlight;
 
 const PROSE_BASENAMES = [
   'license',

package/lib/diff/align.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const { diffChars, tokenSimilarity } = require('./inline.js');
+const inline = require('./inline.js');
+const { diffChars, tokenSimilarity } = inline;
 
 const PAIR_MIN_SIM = 0.3;
 const PAIR_HEAD_WEIGHT = 0.4;

package/lib/diff/diff.js

@@ -1,8 +1,11 @@
 'use strict';
 
-const { diffChars } = require('./inline.js');
-const { isCtxType, pairIndices, attachInline } = require('./align.js');
-const { parseDiff } = require('./parse.js');
+const inline = require('./inline.js');
+const { diffChars } = inline;
+const align = require('./align.js');
+const { isCtxType, pairIndices, attachInline } = align;
+const parse = require('./parse.js');
+const { parseDiff } = parse;
 const patch = require('./patch.js');
 const { splitHunk, flattenBlock, countSides, formatPatch } = patch;
 const display = require('./display.js');

package/lib/diff/display.js

@@ -2,7 +2,8 @@
 
 const align = require('./align.js');
 const { isCtxType, attachInline, LAYOUT_ALIGN } = align;
-const { flattenBlock } = require('./patch.js');
+const patch = require('./patch.js');
+const { flattenBlock } = patch;
 
 const DISPLAY_CONTEXT = 3;
 

package/lib/diff/inline.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const { graphemes } = require('../ansi.js');
+const ansi = require('../ansi.js');
+const { graphemes } = ansi;
 
 const LCS_CELL_LIMIT = 250000;
 const NOISY_TINY_SPANS = 6;

package/lib/diff/patch.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const { isCtxType } = require('./align.js');
+const align = require('./align.js');
+const { isCtxType } = align;
 
 const LINE_MARK = {
   add: '+',

package/lib/diff/unit.js

@@ -1,7 +1,9 @@
 'use strict';
 
-const { attachInline } = require('./align.js');
-const { listPath } = require('../files.js');
+const align = require('./align.js');
+const { attachInline } = align;
+const files = require('../files.js');
+const { listPath } = files;
 
 const splitEditor = (text) => {
   const lines = `${text}`.split('\n');