Remove neo-tree experiment

Author: dglsparsons

AGENTS.md

@@ -54,11 +54,8 @@
 - Auto-sync timers are owned by `init.lua`; `state.clear_review()` should call `neo_reviewer._stop_autosync()` when available to avoid leaking periodic timers across teardown paths and tests.
 - To keep the AI walkthrough panel stable during sync, plumb `keep_ai_ui` through `state.clear_review()` and re-render with `ai_ui.open()` after rebuilding review state.
 
-## Learned while implementing the Neo-tree review source and AI pane split
+## Learned while implementing the AI pane split
 
-- `nix flake check` only sees git-tracked paths; if a Lua module exists only as a new untracked file, Nix builds/tests fail with `module not found`. For temporary or optional modules, registering them through `package.preload` inside a tracked file keeps checks honest without needing git writes.
-- Neo-tree custom sources must expose `components` on the root module or provide a tracked `mod_root .. ".components"` module; preloading only the source and commands still fails during `setup()`.
-- If a custom source should look like Neo-tree filesystem output, render `{ root }` instead of `root.children` and populate source state such as `git_status_lookup`; borrowing filesystem components alone is not enough.
 - `topleft {width}vsplit` from the AI walkthrough pane creates a full-height column across the whole tab, not a split inside the bottom walkthrough area. Use `leftabove {width}vsplit` from the detail window to keep the navigator inside the walkthrough pane; otherwise the main editor window collapses and stacked walkthrough windows fail with `E36`.
 - Transient loading panes need the same split-base exclusions as AI/Ask walkthrough panes; if `find_split_base_window()` can target `neo-reviewer-loading`, the final navigator/detail splits open relative to the loading scratch window instead of the editor layout.
 
@@ -79,8 +76,6 @@
 
 ## Learned while smoothing ReviewSync UI refresh
 
-- `neo_reviewer.neotree.is_open()` cannot rely on `manager.get_state("review").winid` alone; Neo-tree source state can point at a valid sidebar window whose visible buffer belongs to another source. Confirm `vim.b[bufnr].neo_tree_source == "review"` before deciding the review tree is already open.
-- `neo_reviewer.neotree.on_review_changed({ open = true })` should still refresh, not re-open, when the review source buffer is already visible; otherwise autosync re-runs the Neo-tree open command and disturbs the layout.
 - For AI review panes, re-rendering existing split buffers during sync should preserve current window sizes; reusing `open()` without a layout-preserving option causes background refreshes to resize walkthrough splits.
 
 ## Project Overview

doc/neo_reviewer.txt

@@ -28,7 +28,6 @@ your editor. It provides:
 - Multi-line comments with visual range selection
 - Submit reviews (approve or request changes)
 - Telescope integration for file picker
-- Optional Neo-tree changed-files review source
 - Local git diff review (review uncommitted changes before pushing)
 - AI-powered review analysis (optional, via Codex CLI by default)
 
@@ -38,7 +37,6 @@ your editor. It provides:
 - Neovim >= 0.9.0
 - plenary.nvim (required)
 - telescope.nvim (optional, for file picker)
-- neo-tree.nvim (optional, for changed-files review tree)
 - neo-reviewer CLI (Rust CLI tool)
 - GitHub authentication via `gh` CLI
 
@@ -107,10 +105,6 @@ Call setup() with your options: >lua
             periodic_interval_ms = 120000, -- 2 minutes
             cooldown_ms = 1500,          -- Min spacing between sync starts
         },
-        neo_tree = {
-            open_on_review = false,      -- Auto-open Neo-tree review source
-            position = "left",           -- Neo-tree window position
-        },
         ai = {
             enabled = false,                      -- Run AI analysis by default
             model = "gpt-5.3-codex", -- Model for Codex CLI (default)
@@ -168,28 +162,6 @@ sync ~
     Automatic save/periodic sync runs silently on success (no "Synced"
     notifications). Errors still notify.
 
-                                                    *neo_reviewer-config-neo_tree*
-neo_tree ~
-    Configure optional Neo-tree integration for changed review files.
-    - `open_on_review`: Open the Neo-tree review source when a review starts
-      (default: false).
-    - `position`: Neo-tree window position for the review source
-      (default: "left").
-
-    To enable the source in Neo-tree, add
-    `"neo_reviewer.sources.review"` to your Neo-tree `sources` list. The
-    source name inside Neo-tree is `review`, so you can open it manually with
-    `:Neotree review` or include it in the source selector. Example: >
-        require("neo-tree").setup({
-            sources = {
-                "filesystem",
-                "buffers",
-                "git_status",
-                "neo_reviewer.sources.review",
-            },
-        })
-<
-
                                                            *neo_reviewer-config-ai*
 ai ~
     Configure AI-powered review analysis. Requires an AI CLI (Codex CLI by
@@ -334,7 +306,7 @@ These user commands are created when you call `setup()`.
     By default, sync also runs automatically on save for reviewed files and as
     a periodic background refresh every 2 minutes. Save-triggered PR sync skips
     comment refresh (diff/highlights only). Automatic sync refreshes already-
-    open Neo-tree/AI walkthrough windows in place instead of reopening them.
+    open AI walkthrough windows in place instead of reopening them.
     Configure via `sync.*`.
 
 ==============================================================================
@@ -483,8 +455,8 @@ require("neo_reviewer").sync()                                   *neo_reviewer.s
     Preserves cursor position and expanded change block state when possible.
     Manual sync (`:ReviewSync`) always refreshes comments for PR reviews.
     Automatic save-triggered sync refreshes diff/highlights but skips comment
-    refresh for PR reviews, while keeping open Neo-tree/AI review panes in
-    place instead of reopening them.
+    refresh for PR reviews, while keeping open AI review panes in place instead
+    of reopening them.
     See |:ReviewSync|.
 
 require("neo_reviewer").toggle_ai_feedback()     *neo_reviewer.toggle_ai_feedback()*

lua/neo_reviewer/config.lua

@@ -33,10 +33,6 @@
 ---@field focus_on_open boolean Whether to focus the walkthrough window on open
 ---@field step_list_width integer Preferred width of the AI step list pane
 
----@class NRNeoTree
----@field open_on_review boolean Open the Neo-tree review source when a review starts
----@field position string Neo-tree window position for the review source
-
 ---@class NRReviewDiff
 ---@field skip_noise_files boolean Whether to skip common lock/noise files in PR and local diff reviews
 ---@field noise_files string[] Basenames to skip in PR and local diff reviews when skip_noise_files is enabled
@@ -57,7 +53,6 @@
 ---@field input_window NRInputWindow
 ---@field review_diff NRReviewDiff
 ---@field sync NRSync
----@field neo_tree NRNeoTree Optional Neo-tree integration for changed review files
 ---@field ai NRAI AI analysis configuration
 
 ---@class NRPartialSigns
@@ -88,10 +83,6 @@
 ---@field focus_on_open? boolean Whether to focus the walkthrough window on open
 ---@field step_list_width? integer Preferred width of the AI step list pane
 
----@class NRPartialNeoTree
----@field open_on_review? boolean Open the Neo-tree review source when a review starts
----@field position? string Neo-tree window position for the review source
-
 ---@class NRPartialReviewDiff
 ---@field skip_noise_files? boolean Whether to skip common lock/noise files in PR and local diff reviews
 ---@field noise_files? string[] Basenames to skip in PR and local diff reviews when skip_noise_files is enabled
@@ -119,7 +110,6 @@
 ---@field input_window? NRPartialInputWindow
 ---@field review_diff? NRPartialReviewDiff
 ---@field sync? NRPartialSync
----@field neo_tree? NRPartialNeoTree Optional Neo-tree integration for changed review files
 ---@field ai? NRPartialAI AI analysis configuration
 
 ---@class NRConfigModule
@@ -195,10 +185,6 @@ M.values = {
         periodic_interval_ms = 120000,
         cooldown_ms = 1500,
     },
-    neo_tree = {
-        open_on_review = false,
-        position = "left",
-    },
     ai = {
         enabled = false,
         model = "gpt-5.3-codex",

lua/neo_reviewer/init.lua

@@ -61,17 +61,11 @@ local function register_preloads_with(plugin)
 
     plugin.register_preloads()
     return package.preload["neo_reviewer.ui.loading"] ~= nil
-        and package.preload["neo_reviewer.neotree"] ~= nil
-        and package.preload["neo_reviewer.sources.review"] ~= nil
 end
 
 ---@return nil
 local function ensure_preloads()
-    if
-        package.preload["neo_reviewer.ui.loading"]
-        and package.preload["neo_reviewer.neotree"]
-        and package.preload["neo_reviewer.sources.review"]
-    then
+    if package.preload["neo_reviewer.ui.loading"] then
         return
     end
 
@@ -140,15 +134,12 @@ function M._stop_autosync()
     stop_autosync()
 end
 
----@param opts? { keep_ai_ui?: boolean, keep_neotree?: boolean }
+---@param opts? { keep_ai_ui?: boolean }
 local function clear_active_review(opts)
     local state = require("neo_reviewer.state")
     stop_autosync()
     get_loading_ui().close()
     state.clear_review(opts)
-    if not (opts and opts.keep_neotree) then
-        require("neo_reviewer.neotree").on_review_cleared()
-    end
 end
 
 ---@param bufnr integer
@@ -660,7 +651,6 @@ function M.review_diff(opts)
 
         local function finish_setup()
             M.enable_overlay()
-            require("neo_reviewer.neotree").on_review_changed()
 
             if skipped_count > 0 then
                 vim.notify(
@@ -783,7 +773,6 @@ function M.fetch_and_enable(url, on_ready, opts)
 
         local function finish_setup()
             M.enable_overlay()
-            require("neo_reviewer.neotree").on_review_changed()
 
             local nav = require("neo_reviewer.ui.nav")
             nav.first_change()
@@ -1400,9 +1389,7 @@ function M.sync(opts)
             ai_analysis = review.ai_analysis,
         }
         local ai_ui = require("neo_reviewer.ui.ai")
-        local neotree = require("neo_reviewer.neotree")
         local keep_ai_ui_open = ai_ui.is_open() and preserved.ai_analysis ~= nil
-        local keep_neotree_open = neotree.is_open()
         local cursor_pos = vim.api.nvim_win_get_cursor(0)
 
         if is_manual then
@@ -1418,12 +1405,9 @@ function M.sync(opts)
 
             local filtered_files, skipped_count = filter_review_files(data.files, config.values.review_diff)
 
-            clear_active_review({ keep_ai_ui = keep_ai_ui_open, keep_neotree = keep_neotree_open })
+            clear_active_review({ keep_ai_ui = keep_ai_ui_open })
 
             if #filtered_files == 0 then
-                if keep_neotree_open then
-                    neotree.on_review_cleared()
-                end
                 finish_sync()
                 if is_manual and #data.files == 0 then
                     vim.notify("No changes to review", vim.log.levels.WARN)
@@ -1456,7 +1440,6 @@ function M.sync(opts)
             if keep_ai_ui_open then
                 ai_ui.open({ preserve_layout = true })
             end
-            neotree.on_review_changed({ open = keep_neotree_open })
 
             pcall(vim.api.nvim_win_set_cursor, 0, cursor_pos)
 
@@ -1499,9 +1482,7 @@ function M.sync(opts)
         ai_analysis = review.ai_analysis,
     }
     local ai_ui = require("neo_reviewer.ui.ai")
-    local neotree = require("neo_reviewer.neotree")
     local keep_ai_ui_open = ai_ui.is_open() and preserved.ai_analysis ~= nil
-    local keep_neotree_open = neotree.is_open()
     local old_comment_count = #review.comments
     local cursor_pos = vim.api.nvim_win_get_cursor(0)
 
@@ -1522,11 +1503,8 @@ function M.sync(opts)
             data.comments = review.comments
         end
 
-        clear_active_review({ keep_ai_ui = keep_ai_ui_open, keep_neotree = keep_neotree_open })
+        clear_active_review({ keep_ai_ui = keep_ai_ui_open })
         if #filtered_files == 0 then
-            if keep_neotree_open then
-                neotree.on_review_cleared()
-            end
             finish_sync()
             if is_manual and #data.files == 0 then
                 vim.notify("No changes to review", vim.log.levels.WARN)
@@ -1567,7 +1545,6 @@ function M.sync(opts)
         if keep_ai_ui_open then
             ai_ui.open({ preserve_layout = true })
         end
-        neotree.on_review_changed({ open = keep_neotree_open })
 
         pcall(vim.api.nvim_win_set_cursor, 0, cursor_pos)