reslop 0.2.1 → 0.2.4

package/CHANGELOG.md

@@ -2,6 +2,30 @@
 
 ## [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`
+- Select a commit from the list to view its diff
+- Edit a multi-line commit message
+- Propose unused removals only for libraries that export code
+- Improve TUI/TUX, multiple small fixes
+- Implement npm command runner with termination and re-run
+- Filter tests output to reduce AI costs: `npm t` and other scripts
+- Save script logs under `.log/`
+- Group similar problems in test and other output
+
 ## [0.2.1][] - 2026-09-23
 
 - Type a new commit message at the top of the list
@@ -117,7 +141,9 @@
 - 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.1...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
 [0.1.9]: https://github.com/tshemsedinov/reslop/compare/v0.1.8...v0.1.9

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
@@ -39,6 +42,7 @@ npm i -g reslop
 ```
 
 Works on Linux, macOS, and Windows. On Windows use Windows Terminal.
+Requires Node.js `>=18.15.0`.
 
 ## Usage
 
@@ -49,6 +53,18 @@ Works on Linux, macOS, and Windows. On Windows use Windows Terminal.
 - `reslop -n` start a new review even if the latest is still editing
 - `reslop -r` read-only mode
 - `reslop -light` light color theme (default dark)
+- `reslop t -- <program> [args]` run one program and print a reduced report
+
+Prefix each program in an npm script with `reslop t --`. Keep `&&` between those wraps, so each program is captured on its own and a failing step does not start the next one:
+
+```json
+"lint": "reslop t -- eslint . && reslop t -- prettier -c \"**/*.js\"",
+"test": "reslop t -- npm run -s lint && reslop t -- node --test"
+```
+
+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`:
 
@@ -65,7 +81,6 @@ Reviews go in `.review/YYYY-MM-DD-NN.md` with frontmatter `status`:
 - Send anonymized code blocks for expert review
 - Send questions and the repair plan to experts for approval
 - Ask experts
-- Verify the codebase
 - Apply refactoring skills
 - Call agents, harnesses, and IDEs to execute prepared plans
 - Squash, cherry-pick, and interactive rebase

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/ansi.js

@@ -60,7 +60,6 @@ const DARK_UI = {
   headerDirFg: [0x6e, 0x6e, 0x6e],
   headerSlashFg: [0x8a, 0x8a, 0x8a],
   headerFileFg: [0x4e, 0x4e, 0x4e],
-  headerMetaFg: [0x5a, 0x5a, 0x5a],
   chromeFg: [0xd0, 0xd0, 0xe0],
   chromeBg: [0x18, 0x18, 0x24],
   noteBg: [0x24, 0x24, 0x36],
@@ -153,7 +152,6 @@ const LIGHT_UI = {
   headerDirFg: [0x57, 0x60, 0x6a],
   headerSlashFg: [0x8c, 0x95, 0x9f],
   headerFileFg: [0x1f, 0x23, 0x28],
-  headerMetaFg: [0x42, 0x4a, 0x53],
   chromeFg: [0x24, 0x29, 0x2f],
   chromeBg: [0xf0, 0xf2, 0xf5],
   noteBg: [0xe8, 0xf0, 0xfe],
@@ -280,9 +278,14 @@ const codeFg = (style) => {
   return CODE_FG.plain;
 };
 
-const fg = (rgb) => `${ESC}[38;2;${rgb[0]};${rgb[1]};${rgb[2]}m`;
+const channels = (rgb) => {
+  if (typeof rgb === 'number') return `${rgb};${rgb};${rgb}`;
+  return `${rgb[0]};${rgb[1]};${rgb[2]}`;
+};
+
+const fg = (rgb) => `${ESC}[38;2;${channels(rgb)}m`;
 
-const bg = (rgb) => `${ESC}[48;2;${rgb[0]};${rgb[1]};${rgb[2]}m`;
+const bg = (rgb) => `${ESC}[48;2;${channels(rgb)}m`;
 
 const seq = (fgRgb, bgRgb) => `${fg(fgRgb)}${bg(bgRgb)}`;
 
@@ -352,6 +355,117 @@ const truncateVisible = (text, width) => {
   return out;
 };
 
+const csiLength = (text, index) => {
+  if (text[index] !== ESC || text[index + 1] !== '[') return 0;
+  let at = index + 2;
+  while (at < text.length) {
+    const code = text.charCodeAt(at);
+    at += 1;
+    if (code >= 64 && code <= 126) return at - index;
+  }
+  return 0;
+};
+
+const firstGrapheme = (text) => {
+  const next = SEGMENTER.segment(text)[Symbol.iterator]().next();
+  if (next.done || !next.value) return '';
+  return next.value.segment;
+};
+
+const clipAnsi = (text, width) => {
+  const raw = `${text ?? ''}`;
+  if (width <= 0) return '';
+  if (visibleWidth(raw) <= width) return raw;
+  let used = 0;
+  let out = '';
+  let index = 0;
+  while (index < raw.length && used < width) {
+    const csi = csiLength(raw, index);
+    if (csi) {
+      out += raw.slice(index, index + csi);
+      index += csi;
+      continue;
+    }
+    const ch = firstGrapheme(raw.slice(index));
+    if (!ch) break;
+    const w = graphemeWidth(ch, used);
+    if (used + w > width) break;
+    out += ch;
+    used += w;
+    index += ch.length;
+  }
+  if (stripAnsi(out) !== stripAnsi(raw)) out += RESET;
+  return out;
+};
+
+const sgrParams = (seq) => {
+  const inner = seq.slice(2, -1);
+  if (!inner) return [0];
+  return inner.split(';').map((part) => (part === '' ? 0 : Number(part)));
+};
+
+const colorSpan = (params, index) => {
+  const mode = params[index + 1];
+  if (mode === 5) return 2;
+  if (mode === 2) return 4;
+  return 0;
+};
+
+const keepFgParams = (params) => {
+  const kept = [];
+  for (let i = 0; i < params.length; i++) {
+    const code = params[i];
+    if (code === 38 || code === 48) {
+      const extra = colorSpan(params, i);
+      if (code === 38 && extra) {
+        kept.push(code, ...params.slice(i + 1, i + 1 + extra));
+      }
+      i += extra;
+      continue;
+    }
+    if (code >= 40 && code <= 49) continue;
+    if (code >= 100 && code <= 107) continue;
+    if (code === 7 || code === 27) continue;
+    kept.push(code);
+  }
+  return kept;
+};
+
+const foregroundOn = (text, bgRgb) => {
+  const raw = `${text ?? ''}`;
+  const ground = bg(bgRgb);
+  let out = '';
+  let index = 0;
+  while (index < raw.length) {
+    if (raw[index] !== ESC) {
+      const next = raw.indexOf(ESC, index);
+      const end = next < 0 ? raw.length : next;
+      out += raw.slice(index, end);
+      index = end;
+      continue;
+    }
+    const csi = csiLength(raw, index);
+    if (!csi) {
+      out += raw[index];
+      index += 1;
+      continue;
+    }
+    const seq = raw.slice(index, index + csi);
+    index += csi;
+    if (!seq.endsWith('m')) {
+      out += seq;
+      continue;
+    }
+    const kept = keepFgParams(sgrParams(seq));
+    const reset = kept.includes(0);
+    const rest = kept.filter((code) => code !== 0);
+    if (reset) out += `${ESC}[0m`;
+    out += ground;
+    if (rest.length) out += `${ESC}[${rest.join(';')}m`;
+  }
+  return out;
+};
+
 module.exports = {
   ESC,
   RESET,
@@ -373,4 +487,6 @@ module.exports = {
   graphemeWidth,
   visibleWidth,
   truncateVisible,
+  clipAnsi,
+  foregroundOn,
 };

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,15 +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 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;
@@ -180,6 +190,8 @@ const runUi = async (session, proc) => {
 };
 
 const run = async (proc, deps = {}) => {
+  const argv = proc.argv.slice(2);
+  if (argv[0] === 't') return runCapture(proc, deps);
   const stdout = proc.stdout;
   const stderr = proc.stderr;
   let args;

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,