reslop 0.1.0 → 0.1.2

package/CHANGELOG.md

@@ -2,15 +2,31 @@
 
 ## [Unreleased][unreleased]
 
-## [0.1.0][] - 2026-09-09
+## [0.1.2][] - 2026-09-11
 
-### Added
+- 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
 
-- Interactive terminal git diff with intra-line highlighting
-- Stage, unstage, revert, and skip contiguous diff blocks
-- Feedback and todos in `.review/` markdown for an agent to execute
-- Unified, mixed, and side-by-side layouts
-- Review of a worktree, a path, or a given commit (read-only)
+## [0.1.1][] - 2026-09-10
 
-[unreleased]: https://github.com/tshemsedinov/metadiff/compare/v0.1.0...HEAD
-[0.1.0]: https://github.com/tshemsedinov/metadiff/releases/tag/v0.1.0
+- 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
+- Fix diff highlighting for snake_case identifier changes
+- Updated package metadata and README
+
+## [0.1.0][] - 2026-09-07
+
+- 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.2...HEAD
+[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

@@ -1,12 +1,23 @@
-# reslop
+# reslop — Review diff and plan changes
 
-> Review diff and plan changes
+> Turn generated changes into owned changes
 
-- Prepare specs for the agent.
-- Leave feedback, add todos.
-- Generate a plan the agent executes.
+Open a change, leave findings and todos, and prepare a repair plan
+for the agent.
 
-Walk uncommitted git changes, or a given commit. Each contiguous block of diff lines can be staged, unstaged, reverted; add review and todos. Commit review is read-only.
+```text
+Review → Plan → Repair → Verify
+```
+
+- 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.
+
+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
@@ -14,52 +25,26 @@ those characters.
 
 ## Install
 
-Global:
-
 ```bash
 npm i -g reslop
 ```
 
-Local:
-
-```bash
-npx reslop
-```
-
 ## Usage
 
 ```bash
 reslop
 reslop -n
-reslop path/to/folder
-reslop path/to/file
+reslop path/file
 reslop 7ac260c
-reslop HEAD~1 path/to/file
-reslop --help
+reslop https://github.com/metarhia/metacom/pull/555
 ```
 
-Starts on the diff. No arguments, or a folder, reviews that scope.
-A file argument opens that file's diff. A git revision that is not
-also an existing path opens that commit's patch. Esc opens the file
-list; Esc again quits.
-
-Specs go in `.review/YYYY-MM-DD-NN.md` with frontmatter `status`:
-
-| Status  | Meaning                                     |
-| ------- | ------------------------------------------- |
-| editing | Still writing the review in reslop          |
-| ready   | Ready for AI to work through the checkboxes |
-| partial | AI started; some items remain               |
-| done    | All items marked `[x]`                      |
-
-`reslop` resumes the latest file when its status is `editing`. Any other
-status starts a new file. `-n` / `--new` always starts a new file. Quit asks
-`f` (finish as `ready`, for the agent to execute) or `c` (keep `editing` and
-continue next time). The agent should execute `ready` and `partial` reviews.
-Files in `.review/` are omitted from the diff list.
+Reviews go in `.review/YYYY-MM-DD-NN.md` with frontmatter `status`:
 
-A `##` heading names the file. Todos are checkboxes under it. Feedback is a
-checkbox with ` - path:old:new:block` on the same line.
+- `editing`: still writing the review in reslop
+- `ready`: ready for AI to work through the checkboxes
+- `partial`: AI started; some items remain
+- `done`: all items marked `[x]`
 
 ## Keys and buttons
 
@@ -72,7 +57,7 @@ checkbox with ` - path:old:new:block` on the same line.
 | `←`         | Previous remaining block                |
 | `→`         | Next remaining block                    |
 | `l`         | File list (git status of this scope)    |
-| `Esc`       | File list, then quit                    |
+| `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                  |

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

@@ -7,6 +7,8 @@ const keys = require('./keys.js');
 const render = require('./render.js');
 const highlight = require('./highlight.js');
 const detect = require('./detect.js');
+const github = require('./github.js');
+const source = require('./source.js');
 const { Session } = require('./session.js');
 const cli = require('./cli.js');
 
@@ -21,6 +23,9 @@ module.exports = {
   formatPatch: diff.formatPatch,
   itemsFromFiles: diff.itemsFromFiles,
   createGitRepo: git.createGitRepo,
+  parseGithubPrUrl: github.parseGithubPrUrl,
+  loadPullRequest: github.loadPullRequest,
+  selectChangeSource: source.selectChangeSource,
   decodeChunk: keys.decodeChunk,
   hitAction: keys.hitAction,
   renderFrame: render.renderFrame,

package/lib/cli.js

@@ -4,22 +4,16 @@ const fs = require('node:fs');
 const path = require('node:path');
 
 const { createGitRepo } = require('./git.js');
+const { githubToken, loadPullRequest } = require('./github.js');
+const { selectChangeSource, createLoadedSource } = require('./source.js');
 const { Session } = require('./session.js');
-const { ACTIONS, actionHot } = require('./keys.js');
-
-const FLAGS = Object.assign(Object.create(null), {
-  '-h': 'help',
-  '--help': 'help',
-  '-v': 'version',
-  '--version': 'version',
-  '-n': 'newReview',
-  '--new': 'newReview',
-});
 
 const HEX_REV = /^[0-9a-f]{4,40}$/i;
 const HEAD_REV = /^HEAD([~^].*)?$/i;
 const RANGE_REV = /[~^]/;
 
+const USAGE = 'Usage: reslop [-n] [path | commit | pr-url]';
+
 const errorMessage = (reason) => {
   if (reason instanceof Error) return reason.message;
   if (reason === null || reason === undefined) return '';
@@ -27,37 +21,25 @@ 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;
 };
 
-const helpText = () => {
-  const lines = [
-    'Usage: reslop [-n] [commit] [path...]',
-    '',
-    'Review uncommitted git changes, or a given commit (read only).',
-    'Resumes the latest .review file when its status is editing.',
-    'Finish marks the review ready for the agent to execute.',
-    'Use -n / --new to start a new review file.',
-    '',
-    'Keys:',
-  ];
-  for (const action of ACTIONS) {
-    lines.push(`  ${actionHot(action)}    ${action.label}`);
-  }
-  lines.push('  ?    Help');
-  lines.push('');
-  return `${lines.join('\n')}\n`;
+const failUsage = (stderr, reason) => {
+  fail(stderr, reason);
+  stderr.write(`${USAGE}\n`);
+  return 1;
 };
 
 const parseArgv = (argv) => {
   const paths = [];
-  const flags = { help: false, version: false, newReview: false };
+  let newReview = false;
   for (const arg of argv) {
-    const flag = FLAGS[arg];
-    if (flag) {
-      flags[flag] = true;
+    if (arg === '-n') {
+      newReview = true;
       continue;
     }
     if (arg.startsWith('-')) {
@@ -65,10 +47,7 @@ const parseArgv = (argv) => {
     }
     paths.push(arg);
   }
-  const help = flags.help;
-  const version = flags.version;
-  const newReview = flags.newReview;
-  return { paths, help, version, newReview };
+  return { paths, newReview };
 };
 
 const looksLikeRev = (spec) => {
@@ -93,54 +72,84 @@ 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;
-  if (resolved) return { rev: resolved, paths: paths.slice(1) };
+  const resolved = repo.resolveRev(cwd, first);
+  if (resolved) {
+    if (paths.length > 1) {
+      throw new Error('use a commit or a path, not both');
+    }
+    return { rev: resolved, paths: [] };
+  }
   if (looksLikeRev(first)) {
     throw new Error(`bad revision ${first}`);
   }
   return { rev: null, paths };
 };
 
-const writeHelp = (stdout) => {
-  stdout.write(helpText());
-  return 0;
-};
+const isColorEnabled = (stdout, env) => stdout.isTTY === true && !env.NO_COLOR;
+
+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 writeVersion = (stdout, deps) => {
-  const pkgPath = path.join(__dirname, '..', 'package.json');
-  const pkg = deps.pkg ?? JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
-  stdout.write(`${pkg.version}\n`);
-  return 0;
+const startSession = (options) => {
+  const session = new Session(options);
+  session.load();
+  return session;
 };
 
-const loadSession = (proc, deps, args) => {
-  const stdout = proc.stdout;
-  const env = proc.env ?? {};
-  const repo = deps.repo ?? createGitRepo();
+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 paths = scope.paths;
-  const rev = scope.rev;
-  const stdin = proc.stdin;
-  const color = stdout.isTTY === true && !env.NO_COLOR;
-  const getSize = deps.getSize;
-  const startPane = 'diff';
-  const newReview = args.newReview;
-  const session = new Session({
-    repo,
+  const scope = resolveScope(cwd, selected.paths, repo, statSync);
+  return startSession(
+    sessionOptions(proc, deps, args, env, {
+      repo,
+      paths: scope.paths,
+      rev: scope.rev,
+    }),
+  );
+};
+
+const openPullRequestSession = async (proc, deps, args, selected, env) => {
+  const cwd = deps.cwd ?? proc.cwd();
+  const loader = deps.loadPullRequest ?? loadPullRequest;
+  const token = githubToken(env);
+  const loaded = await loader(selected.pr, {
     cwd,
-    paths,
-    rev,
-    stdin,
-    stdout,
-    color,
-    getSize,
-    startPane,
-    newReview,
+    paths: selected.paths,
+    token,
+    fetch: deps.fetch,
   });
-  session.load();
-  return session;
+  const change = loaded.change ?? null;
+  const repository = change ? change.repository : '';
+  const repo = deps.repo ?? createLoadedSource(loaded);
+  return startSession(
+    sessionOptions(proc, deps, args, env, {
+      repo,
+      paths: selected.paths,
+      repoName: repository,
+      sourceLabel: loaded.sourceLabel,
+      change,
+    }),
+  );
+};
+
+const loadSession = async (proc, deps, args) => {
+  const env = proc.env ?? {};
+  const selected = selectChangeSource(args.paths);
+  if (selected.kind === 'github-pr') {
+    return openPullRequestSession(proc, deps, args, selected, env);
+  }
+  return openLocalSession(proc, deps, args, selected, env);
 };
 
 const run = async (proc, deps = {}) => {
@@ -150,13 +159,11 @@ const run = async (proc, deps = {}) => {
   try {
     args = parseArgv(proc.argv.slice(2));
   } catch (error) {
-    return fail(stderr, error);
+    return failUsage(stderr, error);
   }
-  if (args.help) return writeHelp(stdout);
-  if (args.version) return writeVersion(stdout, deps);
   let session;
   try {
-    session = loadSession(proc, deps, args);
+    session = await loadSession(proc, deps, args);
   } catch (error) {
     return fail(stderr, error);
   }
@@ -172,10 +179,8 @@ const run = async (proc, deps = {}) => {
 };
 
 module.exports = {
-  run,
+  failLine,
   parseArgv,
-  helpText,
   resolveScope,
-  looksLikeRev,
-  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 };