reslop 0.1.1 → 0.1.3

package/CHANGELOG.md

@@ -2,24 +2,39 @@
 
 ## [Unreleased][unreleased]
 
+## [0.1.3][] - 2026-09-12
+
+- Showed added/removed and staged/total counts in the file list
+- Highlighted JSX and TSX syntax, including
+- Added vim-style j/k navigation and Ctrl scrolling
+- Improved TUI/UX with background git and npm checks to avoid a freeze
+
+## [0.1.2][] - 2026-09-11
+
+- Review each dependency once across package.json and package-lock
+- Mark unused npm dependencies that are not imported or used
+- Propose unused dependency removals and run npm uninstall
+- Mark dependency review items and show npm audit findings
+- Show npm outdated and npm audit updates as proposed diffs
+- Propose a lockfile version bump for transitive dependencies
+
 ## [0.1.1][] - 2026-09-10
 
 - Open a GitHub pull request by URL and review its diff
 - Import GitHub pull request review comments into the local review
 - Improve TUI navigation and hotkeys
 - CLI simplification and optimisation
-- Dix diff highlighting for snake_case identifier changes
+- Fix diff highlighting for snake_case identifier changes
 - Updated package metadata and README
 
 ## [0.1.0][] - 2026-09-07
 
-### Added
-
-- Interactive terminal review of uncommitted git diffs with stage, unstage,
-  revert, and skip
+- Interactive review of uncommitted git diffs with stage, unstage, revert
 - 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.1.1...HEAD
+[unreleased]: https://github.com/tshemsedinov/reslop/compare/v0.1.3...HEAD
+[0.1.3]: https://github.com/tshemsedinov/reslop/compare/v0.1.2...v0.1.3
+[0.1.2]: https://github.com/tshemsedinov/reslop/compare/v0.1.1...v0.1.2
 [0.1.1]: https://github.com/tshemsedinov/reslop/compare/v0.1.0...v0.1.1
 [0.1.0]: https://github.com/tshemsedinov/reslop/releases/tag/v0.1.0

package/README.md

@@ -2,20 +2,22 @@
 
 > Turn generated changes into owned changes
 
+Open a change, leave findings and todos, and prepare a repair plan
+for the agent.
+
 ```text
 Review → Plan → Repair → Verify
 ```
 
-`0.1.x` implements Review: open a change, leave findings and todos, and
-prepare a repair plan for the agent. Plan, repair, and verify come later.
-
-- Prepare review findings and a repair plan for the agent.
-- Leave feedback, add todos.
-- Generate a plan the agent executes.
+- Review uncommitted git diffs, a given commit, or a GitHub pull request.
+- Stage, unstage, or revert each contiguous block of diff lines.
+- Leave feedback and todos that become a repair plan for the agent.
+- Import GitHub pull request review comments into the local plan for AI.
+- Review each npm dependency once across `package.json` and the lockfile.
+- Propose unused removals, npm audit fixes, and outdated updates as diffs.
 
-Walk uncommitted git changes, a given commit, or a GitHub pull request.
-Each contiguous block of diff lines can be staged, unstaged, reverted;
-add review and todos. Commit and pull request review is read-only.
+Commit and pull request review is read-only. Applying a dependency
+proposal stages the files and runs `npm i` or `npm uninstall`.
 
 Intra-line highlighting paints the words that actually changed in a
 stronger red/green. A close edit inside one word still marks only
@@ -46,23 +48,24 @@ Reviews go in `.review/YYYY-MM-DD-NN.md` with frontmatter `status`:
 
 ## Keys and buttons
 
-| Key         | Action                                  |
-| ----------- | --------------------------------------- |
-| `a`         | Stage this block (`git add`)            |
-| `u`         | Unstage this block (keep worktree)      |
-| `r`         | Restore this block to the last commit   |
-| `s`         | Skip (leave unstaged, drop from review) |
-| `←`         | Previous remaining block                |
-| `→`         | Next remaining block                    |
-| `l`         | File list (git status of this scope)    |
-| `Esc`       | Quit file list or a tool                |
-| `m`         | Cycle unified / mixed / side-by-side    |
-| `f`         | Feedback on this diff block             |
-| `t`         | New todo for this file                  |
-| `⌫` / `Del` | Remove the selected todo                |
-| `⏎`         | Open the selected file                  |
-| `q`         | Quit                                    |
-| `?`         | Help                                    |
+| Key                 | Action                                   |
+| ------------------- | ---------------------------------------- |
+| `a`                 | Stage this block (`git add`)             |
+| `u`                 | Unstage this block (keep worktree)       |
+| `r`                 | Restore this block to the last commit    |
+| `f`                 | Feedback on this diff block              |
+| `t`                 | New todo for this file                   |
+| `l`                 | File list (git status of this scope)     |
+| `m`                 | Cycle unified / mixed / side-by-side     |
+| `⏎`                 | Open the selected file                   |
+| `Esc`               | Quit file list or a tool                 |
+| `→` / `j`           | Next remaining block / next file         |
+| `←` / `k`           | Previous remaining block / previous file |
+| `Ctrl-e` / `Ctrl-y` | Scroll one line down / up                |
+| `Ctrl-f` / `Ctrl-b` | Scroll page down / up                    |
+| `Ctrl-d` / `Ctrl-u` | Scroll half-page down / up               |
+| `⌫` / `Del`         | Remove the selected todo                 |
+| `q`                 | Quit                                     |
 
 ## License
 

package/bin/reslop.js

@@ -1,14 +1,14 @@
 #!/usr/bin/env node
 'use strict';
 
-const { run, errorMessage } = require('../lib/cli.js');
+const { run, failLine } = require('../lib/cli.js');
 const { LEAVE_TERM } = require('../lib/session.js');
 
 const fail = (reason) => {
   try {
     process.stdout.write(LEAVE_TERM);
   } catch {}
-  process.stderr.write(`reslop: ${errorMessage(reason)}\n`);
+  process.stderr.write(failLine(reason));
   process.exit(1);
 };
 

package/lib/ansi.js

@@ -217,7 +217,6 @@ module.exports = {
   EL,
   THEME,
   CODE_FG,
-  LOG_COL_FG,
   codeFg,
   fg,
   bg,

package/lib/cli.js

@@ -21,8 +21,10 @@ const errorMessage = (reason) => {
   return `${reason}`;
 };
 
+const failLine = (reason) => `reslop: ${errorMessage(reason)}\n`;
+
 const fail = (stderr, reason) => {
-  stderr.write(`reslop: ${errorMessage(reason)}\n`);
+  stderr.write(failLine(reason));
   return 1;
 };
 
@@ -70,7 +72,7 @@ const resolveScope = (cwd, paths, repo, statSync = fs.statSync) => {
   if (pathExists(cwd, first, statSync)) {
     return { rev: null, paths };
   }
-  const resolved = repo.resolveRev ? repo.resolveRev(cwd, first) : null;
+  const resolved = repo.resolveRev(cwd, first);
   if (resolved) {
     if (paths.length > 1) {
       throw new Error('use a commit or a path, not both');
@@ -85,30 +87,41 @@ const resolveScope = (cwd, paths, repo, statSync = fs.statSync) => {
 
 const isColorEnabled = (stdout, env) => stdout.isTTY === true && !env.NO_COLOR;
 
-const openLocalSession = (proc, deps, args, env) => {
-  const stdout = proc.stdout;
-  const repo = deps.repo ?? createGitRepo();
+const sessionOptions = (proc, deps, args, env, extra) => ({
+  cwd: deps.cwd ?? proc.cwd(),
+  stdin: proc.stdin,
+  stdout: proc.stdout,
+  color: isColorEnabled(proc.stdout, env),
+  getSize: deps.getSize,
+  startPane: 'files',
+  newReview: args.newReview,
+  audit: true,
+  ...extra,
+});
+
+const startSession = (options, extra = {}) => {
+  const session = new Session(options);
+  if (extra.load) session.load();
+  return session;
+};
+
+const openLocalSession = (proc, deps, args, selected, env) => {
   const cwd = deps.cwd ?? proc.cwd();
+  const repo = deps.repo ?? createGitRepo();
   const statSync = deps.statSync ?? fs.statSync;
-  const scope = resolveScope(cwd, args.paths, repo, statSync);
-  const session = new Session({
-    repo,
-    cwd,
-    paths: scope.paths,
-    rev: scope.rev,
-    stdin: proc.stdin,
-    stdout,
-    color: isColorEnabled(stdout, env),
-    getSize: deps.getSize,
-    startPane: 'files',
-    newReview: args.newReview,
-  });
-  session.load();
+  const scope = resolveScope(cwd, selected.paths, repo, statSync);
+  const session = startSession(
+    sessionOptions(proc, deps, args, env, {
+      repo,
+      paths: scope.paths,
+      rev: scope.rev,
+    }),
+  );
+  session.ensureRepo();
   return session;
 };
 
 const openPullRequestSession = async (proc, deps, args, selected, env) => {
-  const stdout = proc.stdout;
   const cwd = deps.cwd ?? proc.cwd();
   const loader = deps.loadPullRequest ?? loadPullRequest;
   const token = githubToken(env);
@@ -117,27 +130,20 @@ const openPullRequestSession = async (proc, deps, args, selected, env) => {
     paths: selected.paths,
     token,
     fetch: deps.fetch,
-    env,
   });
   const change = loaded.change ?? null;
   const repository = change ? change.repository : '';
   const repo = deps.repo ?? createLoadedSource(loaded);
-  const session = new Session({
-    repo,
-    cwd,
-    paths: selected.paths,
-    stdin: proc.stdin,
-    stdout,
-    color: isColorEnabled(stdout, env),
-    getSize: deps.getSize,
-    startPane: 'files',
-    newReview: args.newReview,
-    repoName: repository,
-    sourceLabel: loaded.sourceLabel,
-    change,
-  });
-  session.load();
-  return session;
+  return startSession(
+    sessionOptions(proc, deps, args, env, {
+      repo,
+      paths: selected.paths,
+      repoName: repository,
+      sourceLabel: loaded.sourceLabel,
+      change,
+    }),
+    { load: true },
+  );
 };
 
 const loadSession = async (proc, deps, args) => {
@@ -146,7 +152,14 @@ const loadSession = async (proc, deps, args) => {
   if (selected.kind === 'github-pr') {
     return openPullRequestSession(proc, deps, args, selected, env);
   }
-  return openLocalSession(proc, deps, args, env);
+  return openLocalSession(proc, deps, args, selected, env);
+};
+
+const runUi = async (session, proc) => {
+  const code = await session.startUi();
+  if (session.loadError) return fail(proc.stderr, session.loadError);
+  if (session.emptyReview) proc.stdout.write('nothing to review\n');
+  return code;
 };
 
 const run = async (proc, deps = {}) => {
@@ -164,22 +177,24 @@ const run = async (proc, deps = {}) => {
   } catch (error) {
     return fail(stderr, error);
   }
+  const tty = proc.stdin.isTTY === true && stdout.isTTY === true;
+  if (tty) return runUi(session, proc);
+  try {
+    await session.loadReady();
+  } catch (error) {
+    return fail(stderr, error);
+  }
   if (!session.items.length) {
     stdout.write('nothing to review\n');
     return 0;
   }
-  if (!proc.stdin.isTTY || !stdout.isTTY) {
-    stderr.write('reslop: interactive terminal required\n');
-    return 1;
-  }
-  return session.startUi();
+  stderr.write('reslop: interactive terminal required\n');
+  return 1;
 };
 
 module.exports = {
-  run,
+  failLine,
   parseArgv,
   resolveScope,
-  looksLikeRev,
-  selectChangeSource,
-  errorMessage,
+  run,
 };

package/lib/clipboard.js

@@ -39,4 +39,4 @@ const copyText = (text, stdout) => {
   return ok;
 };
 
-module.exports = { osc52, copyText, runClip };
+module.exports = { osc52, copyText };