bunch of fixes:

- fixing the walkthrough to go by highlight
- Add AGENTS.md self-documenting file
- ignore noisy 'lock' style files by default

Author: dglsparsons

AGENTS.md

@@ -14,6 +14,15 @@
 
 - `toggle_prev_code()` is a global review-mode toggle: it flips `show_old_code` and applies to all hunks in applied review buffers, not just the currently selected change block.
 
+## Learned while implementing Ask walkthrough navigation
+
+- `next_change`/`prev_change` route through `ui/walkthrough.lua` when Ask is open, so anchor-level behavior must live in `next_step`/`prev_step`; changing `ui/nav.lua` alone will not affect Ask navigation.
+
+## Learned while implementing local diff noise filtering
+
+- Filter `ReviewDiff` files before `state.set_local_review()` and before optional AI analysis so navigation, change counts, and AI walkthrough inputs stay consistent.
+- Basename-based skipping is sufficient for lockfiles: matching both exact path and `"/" .. basename` suffix excludes nested lockfiles without adding glob parsing complexity.
+
 ## Project Overview
 
 `neo-reviewer` is a Neovim plugin for reviewing GitHub PRs. Hybrid architecture: Rust CLI for GitHub API/diff parsing, Lua plugin for UI.

README.md

@@ -75,6 +75,13 @@ require("neo_reviewer").setup({
     },
     wrap_navigation = true,      -- Wrap at file boundaries
     auto_expand_deletes = false, -- Auto-expand deleted lines
+    review_diff = {
+        skip_noise_files = true,  -- Skip common lock/noise files by default
+        noise_files = {
+            "pnpm-lock.yaml",
+            "Cargo.lock",
+        },
+    },
 })
 ```
 
@@ -137,7 +144,7 @@ vim.keymap.set("n", ",dx", nr.request_changes, vim.tbl_extend("force", opts, { d
 | Command | Description |
 |---------|-------------|
 | `:ReviewPR {url}` | Open a PR for review |
-| `:ReviewDiff` | Review local git diff |
+| `:ReviewDiff` | Review local git diff (skips configured noise files by default) |
 | `:Ask` | AI-guided codebase exploration |
 | `:AddComment` | Add a review comment |
 | `:Approve` | Approve the PR |

doc/neo_reviewer.txt

@@ -88,6 +88,13 @@ Call setup() with your options: >lua
                 cancel = "<Esc>",   -- Key to cancel input
             },
         },
+        review_diff = {
+            skip_noise_files = true, -- Skip common lock/noise files by default
+            noise_files = {
+                "pnpm-lock.yaml",
+                "Cargo.lock",
+            },
+        },
         ai = {
             enabled = false,                      -- Run AI analysis by default
             model = "gpt-5.3-codex", -- Model for Codex CLI (default)
@@ -117,6 +124,14 @@ input_window.keys ~
     - `submit`: Key to submit the input (default: "<C-s>").
     - `cancel`: Key to cancel and close the input window (default: "<Esc>").
 
+                                                  *neo_reviewer-config-review_diff*
+review_diff ~
+    Configure local diff review file filtering.
+    - `skip_noise_files`: Skip common lock/noise files in |:ReviewDiff|
+      (default: true).
+    - `noise_files`: Basenames to skip when `skip_noise_files` is enabled.
+      Any file with a matching basename is excluded from local diff review.
+
                                                            *neo_reviewer-config-ai*
 ai ~
     Configure AI-powered review analysis. Requires an AI CLI (Codex CLI by
@@ -172,7 +187,10 @@ These user commands are created when you call `setup()`.
 
 :ReviewDiff [--analyze|--no-analyze]                               *:ReviewDiff*
     Review local git diff (both staged and unstaged changes). Comments are
-    written to `REVIEW_COMMENTS.md` in the project root.
+    written to `REVIEW_COMMENTS.md` in the project root. By default, common
+    lock/noise files (for example `pnpm-lock.yaml`, `Cargo.lock`) are skipped.
+    Configure this via `review_diff.skip_noise_files` and
+    `review_diff.noise_files`.
     - `--analyze`: Force AI analysis on (overrides config)
     - `--no-analyze`: Force AI analysis off (overrides config)
 
@@ -192,7 +210,8 @@ These user commands are created when you call `setup()`.
     loading message. When ready, the first step jumps to its first anchor
     (if any).
     When the exploration window is open, |neo_reviewer.next_change()| and
-    |neo_reviewer.prev_change()| navigate between exploration steps.
+    |neo_reviewer.prev_change()| move through anchors in the current step
+    first, then move between exploration steps.
 
 :AddComment                                                        *:AddComment*
     Add a review comment at cursor position. Supports visual range selection
@@ -231,7 +250,9 @@ require("neo_reviewer").review_pr({arg})                   *neo_reviewer.review_
 
 require("neo_reviewer").review_diff({opts})               *neo_reviewer.review_diff()*
     Review local git diff (both staged and unstaged changes). This allows
-    you to review code changes before committing. Accepts optional {opts}:
+    you to review code changes before committing. By default, common lock/noise
+    files are skipped using `review_diff.skip_noise_files` and
+    `review_diff.noise_files`. Accepts optional {opts}:
     - `analyze`: boolean to force AI analysis on/off (overrides config)
 
     Comments are written to `REVIEW_COMMENTS.md` in the project root with
@@ -255,10 +276,14 @@ require("neo_reviewer").ask({opts})                        *neo_reviewer.ask()*
     The file is cleared each time you start a new local diff review.
 
 require("neo_reviewer").next_change()                     *neo_reviewer.next_change()*
-    Jump to the next contiguous block of changes.
+    Jump to the next contiguous block of changes. When an Ask walkthrough is
+    open, this moves to the next anchor in the current step before advancing
+    to the next step.
 
 require("neo_reviewer").prev_change()                     *neo_reviewer.prev_change()*
-    Jump to the previous contiguous block of changes.
+    Jump to the previous contiguous block of changes. When an Ask walkthrough
+    is open, this moves to the previous anchor in the current step before
+    moving to the previous step.
 
 require("neo_reviewer").next_comment()                   *neo_reviewer.next_comment()*
     Jump to the next comment in the review. Works in both PR review mode

lua/neo_reviewer/config.lua

@@ -30,13 +30,18 @@
 ---@field height integer Minimum height of the walkthrough split (0 = auto)
 ---@field focus_on_open boolean Whether to focus the walkthrough window on open
 
+---@class NRReviewDiff
+---@field skip_noise_files boolean Whether to skip common lock/noise files in local diff reviews
+---@field noise_files string[] Basenames to skip in local diff reviews when skip_noise_files is enabled
+
 ---@class NRConfig
 ---@field cli_path string Path to the neo-reviewer CLI binary
 ---@field signs NRSigns
 ---@field wrap_navigation boolean Whether to wrap around when navigating change blocks
 ---@field auto_expand_deletes boolean Whether to auto-expand deleted lines
 ---@field thread_window NRThreadWindow
 ---@field input_window NRInputWindow
+---@field review_diff NRReviewDiff
 ---@field ai NRAI AI analysis configuration
 
 ---@class NRPartialSigns
@@ -64,6 +69,10 @@
 ---@field height? integer Minimum height of the walkthrough split (0 = auto)
 ---@field focus_on_open? boolean Whether to focus the walkthrough window on open
 
+---@class NRPartialReviewDiff
+---@field skip_noise_files? boolean Whether to skip common lock/noise files in local diff reviews
+---@field noise_files? string[] Basenames to skip in local diff reviews when skip_noise_files is enabled
+
 ---@class NRPartialAI
 ---@field enabled? boolean Whether AI analysis is enabled by default
 ---@field model? string Model for AI CLI
@@ -78,6 +87,7 @@
 ---@field auto_expand_deletes? boolean Whether to auto-expand deleted lines
 ---@field thread_window? NRPartialThreadWindow
 ---@field input_window? NRPartialInputWindow
+---@field review_diff? NRPartialReviewDiff
 ---@field ai? NRPartialAI AI analysis configuration
 
 ---@class NRConfigModule
@@ -109,6 +119,41 @@ M.values = {
             cancel = "<Esc>",
         },
     },
+    review_diff = {
+        skip_noise_files = true,
+        noise_files = {
+            "pnpm-lock.yaml",
+            "pnpm-lock.yml",
+            "package-lock.json",
+            "npm-shrinkwrap.json",
+            "yarn.lock",
+            "bun.lock",
+            "bun.lockb",
+            "Cargo.lock",
+            "Gemfile.lock",
+            "Pipfile.lock",
+            "poetry.lock",
+            "uv.lock",
+            "pdm.lock",
+            "composer.lock",
+            "go.sum",
+            "Gopkg.lock",
+            "mix.lock",
+            "pubspec.lock",
+            "Podfile.lock",
+            "Package.resolved",
+            "packages.lock.json",
+            "paket.lock",
+            "gradle.lockfile",
+            "deps.lock",
+            "Chart.lock",
+            "renv.lock",
+            "conan.lock",
+            "vcpkg-lock.json",
+            "flake.lock",
+            ".terraform.lock.hcl",
+        },
+    },
     ai = {
         enabled = false,
         model = "gpt-5.3-codex",

lua/neo_reviewer/init.lua

@@ -23,6 +23,49 @@ local function is_github_url(input)
     return input:match("github%.com") ~= nil or input:match("^https?://") ~= nil
 end
 
+---@param file_path string
+---@param noise_file string
+---@return boolean
+local function path_matches_noise_file(file_path, noise_file)
+    if file_path == noise_file then
+        return true
+    end
+
+    local suffix = "/" .. noise_file
+    return file_path:sub(-#suffix) == suffix
+end
+
+---@param files NRFile[]
+---@param review_diff_cfg NRReviewDiff
+---@return NRFile[] filtered_files
+---@return integer skipped_count
+local function filter_local_diff_files(files, review_diff_cfg)
+    if not review_diff_cfg.skip_noise_files then
+        return files, 0
+    end
+
+    local filtered_files = {}
+    local skipped_count = 0
+
+    for _, file in ipairs(files) do
+        local skip_file = false
+        for _, noise_file in ipairs(review_diff_cfg.noise_files or {}) do
+            if path_matches_noise_file(file.path, noise_file) then
+                skip_file = true
+                break
+            end
+        end
+
+        if skip_file then
+            skipped_count = skipped_count + 1
+        else
+            table.insert(filtered_files, file)
+        end
+    end
+
+    return filtered_files, skipped_count
+end
+
 ---@param review NRReview
 ---@return boolean
 local function restore_previous_branch(review)
@@ -321,8 +364,30 @@ function M.review_diff(opts)
             return
         end
 
+        local filtered_files, skipped_count = filter_local_diff_files(data.files, config.values.review_diff)
+        if #filtered_files == 0 then
+            vim.notify(
+                string.format("No reviewable changes after skipping %d noise files", skipped_count),
+                vim.log.levels.WARN
+            )
+            return
+        end
+
+        if skipped_count > 0 then
+            vim.notify(
+                string.format("[neo-reviewer] Skipped %d noise file(s) in local diff review", skipped_count),
+                vim.log.levels.INFO
+            )
+        end
+
+        ---@type NRDiffData
+        local filtered_data = {
+            git_root = data.git_root,
+            files = filtered_files,
+        }
+
         state.clear_review()
-        local review = state.set_local_review(data)
+        local review = state.set_local_review(filtered_data)
 
         local comments_file = require("neo_reviewer.ui.comments_file")
         comments_file.clear()
@@ -333,14 +398,28 @@ function M.review_diff(opts)
         end
 
         local total_changes = 0
-        for _, file in ipairs(data.files) do
+        for _, file in ipairs(filtered_files) do
             total_changes = total_changes + #(file.change_blocks or {})
         end
 
         local function finish_setup()
             M.enable_overlay()
 
-            vim.notify(string.format("Local diff review enabled (%d files changed)", #data.files), vim.log.levels.INFO)
+            if skipped_count > 0 then
+                vim.notify(
+                    string.format(
+                        "Local diff review enabled (%d files changed, %d noise files skipped)",
+                        #filtered_files,
+                        skipped_count
+                    ),
+                    vim.log.levels.INFO
+                )
+            else
+                vim.notify(
+                    string.format("Local diff review enabled (%d files changed)", #filtered_files),
+                    vim.log.levels.INFO
+                )
+            end
 
             local nav = require("neo_reviewer.ui.nav")
             nav.first_change()
@@ -350,7 +429,7 @@ function M.review_diff(opts)
             vim.notify(
                 string.format(
                     "[neo-reviewer] Local diff fetched (%d files, %d changes). Running AI analysis...",
-                    #data.files,
+                    #filtered_files,
                     total_changes
                 ),
                 vim.log.levels.INFO