reslop 0.2.3 → 0.2.5

package/CHANGELOG.md

@@ -2,6 +2,30 @@
 
 ## [Unreleased][unreleased]
 
+## [0.2.5][] - 2026-09-30
+
+- Remove UI and filesystem polling timers, refresh on events
+- Refresh the commits list when HEAD changes outside reslop
+- Update stage/unstage immediately, git/npm in the background
+- Add editor hotkeys Ctrl+X, Ctrl+C, and Ctrl+V
+- Improve diff calculation
+- Improve npm command and test edit/run
+- Refactor internals: simplify and add classes
+- Hothey "i" add to ignore lists: .gitignore, .npmignore
+- Rename TODO to tasks and improve TUI/TUX
+
+## [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 +153,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.3...HEAD
+[unreleased]: https://github.com/tshemsedinov/reslop/compare/v0.2.5...HEAD
+[0.2.5]: https://github.com/tshemsedinov/reslop/compare/v0.2.4...v0.2.5
+[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,23 @@
 
 > Turn generated changes into owned changes
 
-Open a change, leave findings and todos, and prepare a repair plan
-for the agent.
+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 +63,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/capabilities.js

@@ -97,12 +97,8 @@ const sessionCapabilities = (options) => {
 };
 
 module.exports = {
-  READ_METHODS,
-  CHANGE_METHODS,
-  BRANCH_METHODS,
   SOURCE_CAPS,
   capabilitiesFor,
-  withReadOnly,
   attachCapabilities,
   sessionCapabilities,
 };

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;
 
@@ -26,8 +29,44 @@ const runClip = (cmd, args, text) => {
   return result.status === 0;
 };
 
+const UNIX_PASTE = [
+  { cmd: 'wl-paste', args: ['--no-newline'] },
+  { cmd: 'xclip', args: ['-selection', 'clipboard', '-o'] },
+  { cmd: 'xsel', args: ['--clipboard', '--output'] },
+];
+
+let copied = '';
+
+const pasteTools = (platform = process.platform) => {
+  if (platform === 'win32') {
+    const args = ['-NoProfile', '-Command', 'Get-Clipboard'];
+    return [{ cmd: 'powershell', args }];
+  }
+  if (platform === 'darwin') return [{ cmd: 'pbpaste', args: [] }];
+  return UNIX_PASTE;
+};
+
+const readPaste = (cmd, args) => {
+  const result = spawnSync(
+    cmd,
+    args,
+    spawnBase({ encoding: 'utf8', timeout: CLIP_TIMEOUT_MS }),
+  );
+  if (result.error || result.status !== 0) return '';
+  return result.stdout || '';
+};
+
+const systemPaste = () => {
+  for (const tool of pasteTools()) {
+    const text = readPaste(tool.cmd, tool.args);
+    if (text) return text;
+  }
+  return '';
+};
+
 const copyText = (text, stdout) => {
   if (!text) return false;
+  copied = text;
   let ok = false;
   if (stdout && stdout.write) {
     stdout.write(osc52(text));
@@ -39,4 +78,6 @@ const copyText = (text, stdout) => {
   return ok;
 };
 
-module.exports = { osc52, copyText };
+const pasteText = () => systemPaste() || copied;
+
+module.exports = { osc52, copyText, pasteText };

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,26 +634,22 @@ 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,
   diffSections,
   mergeResolved,
-  orderedChanges,
   summarizeGroup,
   collectGroups,
   takeManifestItems,
   foldedDepNames,
   stripFoldedDepItem,
-  buildRows,
-  makeHunk,
   makeDepItems,
   stubFileItem,
 };