fix: improve performance and bug fix

Author: tsangwailam

.github/workflows/release.yml

@@ -259,14 +259,22 @@ jobs:
         shell: bash
         run: |
           TARGET_REPO="${CLOUDSMITH_REPO}"
+          TARGET_DISTRIBUTION="${CLOUDSMITH_DISTRIBUTION:-}"
+          TARGET_RELEASE="${CLOUDSMITH_RELEASE:-}"
           if [[ "${CLOUDSMITH_REPO}" != */*/* ]]; then
-            if [[ -z "${CLOUDSMITH_DISTRIBUTION}" ]]; then
-              echo "CLOUDSMITH_REPO must be OWNER/REPO/DISTRO[/RELEASE] or set CLOUDSMITH_DISTRIBUTION."
-              exit 1
+            if [[ -z "${TARGET_DISTRIBUTION}" ]]; then
+              TARGET_DISTRIBUTION="ubuntu"
+              echo "CLOUDSMITH_DISTRIBUTION is not set, defaulting to '${TARGET_DISTRIBUTION}'."
             fi
-            TARGET_REPO="${CLOUDSMITH_REPO}/${CLOUDSMITH_DISTRIBUTION}"
-            if [[ -n "${CLOUDSMITH_RELEASE}" ]]; then
-              TARGET_REPO="${TARGET_REPO}/${CLOUDSMITH_RELEASE}"
+            if [[ -z "${TARGET_RELEASE}" ]]; then
+              if command -v lsb_release >/dev/null 2>&1; then
+                TARGET_RELEASE="$(lsb_release -cs)"
+              else
+                TARGET_RELEASE="stable"
+              fi
+              echo "CLOUDSMITH_RELEASE is not set, defaulting to '${TARGET_RELEASE}'."
             fi
+            TARGET_REPO="${CLOUDSMITH_REPO}/${TARGET_DISTRIBUTION}/${TARGET_RELEASE}"
           fi
+          echo "Publishing to Cloudsmith repository: ${TARGET_REPO}"
           cloudsmith push deb "${TARGET_REPO}" dist/debian-amd64/*.deb --republish

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.1.7] - 2026-03-09
+
+### Changed
+
+- Improved responsiveness by moving slow Git status and image preview work off the UI render path and into background workers.
+- Stabilized preview interaction under high-frequency input, including wheel and key reversal behavior and bounded scroll processing at pane edges.
+- Hardened large file behavior with explicit limits for text/image previews, diff generation, and preview caching.
+
 ## [0.1.6] - 2026-03-08
 
 ### Added

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "fpv"
-version = "0.1.6"
+version = "0.1.7"
 edition = "2021"
 description = "A minimal, keyboard-first TUI file previewer with syntax highlighting"
 license = "MIT"

Cargo.toml

@@ -6,7 +6,7 @@
 
 A minimal, keyboard-first TUI file previewer for browsing directories and viewing code with syntax highlighting in the terminal.
 
-Current release: `v0.1.6`
+Current release: `v0.1.7`
 
 ---
 

README.md

@@ -1,13 +1,37 @@
 use crate::app::navigation_result::NavigationActionResult;
 use crate::app::state::{NodeType, SessionState, TreeNode};
 use crate::fs::current_dir::{
-    is_filesystem_root, list_current_directory_with_visibility, parent_path,
+    directory_access_error_message, is_filesystem_root, list_current_directory_with_visibility,
+    parent_path,
 };
 use anyhow::Result;
 use std::path::{Path, PathBuf};
 
 const MAX_DIR_ENTRIES: usize = 2000;
 
+fn replace_directory_entries(
+    state: &mut SessionState,
+    nodes: &mut Vec<TreeNode>,
+    entries: Vec<TreeNode>,
+    preferred: Option<&PathBuf>,
+) {
+    state.clear_current_dir_error();
+    *nodes = entries;
+    state.restore_or_default_selection(nodes, preferred);
+    state.update_selected_path(nodes);
+}
+
+fn set_current_directory_error(
+    state: &mut SessionState,
+    nodes: &mut Vec<TreeNode>,
+    message: String,
+) {
+    state.set_current_dir_error(message);
+    nodes.clear();
+    state.selected_index = 0;
+    state.set_selected_path(state.current_path.clone());
+}
+
 fn absolute_current_path(path: &Path) -> PathBuf {
     if path.is_absolute() {
         return path.to_path_buf();
@@ -75,13 +99,19 @@ pub fn enter_selected_directory(
 
     let target = node.path.clone();
     let entries =
-        list_current_directory_with_visibility(&target, MAX_DIR_ENTRIES, state.show_hidden)?;
+        match list_current_directory_with_visibility(&target, MAX_DIR_ENTRIES, state.show_hidden) {
+            Ok(entries) => entries,
+            Err(error) => {
+                return Ok(NavigationActionResult::blocked(
+                    "enter_directory",
+                    state.current_path.clone(),
+                    directory_access_error_message(&error),
+                ));
+            }
+        };
     state.last_child_path = Some(target.clone());
     state.current_path = target.clone();
-    *nodes = entries;
-    state.selected_index = 0;
-    state.revalidate_selection(nodes);
-    state.update_selected_path(nodes);
+    replace_directory_entries(state, nodes, entries, None);
 
     Ok(NavigationActionResult::changed(
         "enter_directory",
@@ -114,11 +144,18 @@ pub fn go_to_parent_directory(
 
     let previous_child = current_path;
     let entries =
-        list_current_directory_with_visibility(&parent, MAX_DIR_ENTRIES, state.show_hidden)?;
+        match list_current_directory_with_visibility(&parent, MAX_DIR_ENTRIES, state.show_hidden) {
+            Ok(entries) => entries,
+            Err(error) => {
+                return Ok(NavigationActionResult::blocked(
+                    "go_parent",
+                    state.current_path.clone(),
+                    directory_access_error_message(&error),
+                ));
+            }
+        };
     state.current_path = parent.clone();
-    *nodes = entries;
-    state.restore_or_default_selection(nodes, Some(&previous_child));
-    state.update_selected_path(nodes);
+    replace_directory_entries(state, nodes, entries, Some(&previous_child));
 
     Ok(NavigationActionResult::changed(
         "go_parent",
@@ -132,14 +169,23 @@ pub fn refresh_current_directory(
     nodes: &mut Vec<TreeNode>,
 ) -> Result<NavigationActionResult> {
     let previous_selected_path = nodes.get(state.selected_index).map(|n| n.path.clone());
-    let entries = list_current_directory_with_visibility(
+    let entries = match list_current_directory_with_visibility(
         &state.current_path,
         MAX_DIR_ENTRIES,
         state.show_hidden,
-    )?;
-    *nodes = entries;
-    state.restore_or_default_selection(nodes, previous_selected_path.as_ref());
-    state.update_selected_path(nodes);
+    ) {
+        Ok(entries) => entries,
+        Err(error) => {
+            let message = directory_access_error_message(&error);
+            set_current_directory_error(state, nodes, message.clone());
+            return Ok(NavigationActionResult::blocked(
+                "refresh_current_dir",
+                state.current_path.clone(),
+                message,
+            ));
+        }
+    };
+    replace_directory_entries(state, nodes, entries, previous_selected_path.as_ref());
 
     Ok(NavigationActionResult::changed(
         "refresh_current_dir",