fpf-cli 1.6.38 → 1.6.40

package/README.md

@@ -102,5 +102,9 @@ Installed packages are marked with `*` in the result list.
 - Set `FPF_LOADING_INDICATOR=0` to disable the pre-search loading indicator.
 - `FPF_RELOAD_MIN_CHARS`: minimum query length before live reload (default `2`)
 - `FPF_RELOAD_DEBOUNCE`: reload debounce seconds (default `0.12`)
+- `FPF_ENABLE_QUERY_CACHE`: `auto` (default), `1`, or `0` (`auto` enables query cache for `apt`, `brew`, `pacman`, and `bun`)
+- `FPF_QUERY_CACHE_TTL`: default query-cache TTL seconds for heavy manager caches (default `300`)
+- `FPF_APT_QUERY_CACHE_TTL`, `FPF_BREW_QUERY_CACHE_TTL`, `FPF_PACMAN_QUERY_CACHE_TTL`: per-manager query-cache TTL overrides
+- `FPF_BUN_QUERY_CACHE_TTL`: Bun query-cache TTL (default `900`)
 - `FPF_DISABLE_INSTALLED_CACHE=1` disables installed-package marker cache
 - `FPF_INSTALLED_CACHE_TTL`: installed-package marker cache freshness window in seconds (default `300`, set `0` to always refresh)

package/fpf

@@ -3,7 +3,7 @@
 set -euo pipefail
 
 SCRIPT_NAME="fpf"
-SCRIPT_VERSION="1.6.38"
+SCRIPT_VERSION="1.6.40"
 TMP_ROOT="${TMPDIR:-/tmp}/fpf"
 SESSION_TMP_ROOT=""
 HELP_FILE=""
@@ -25,6 +25,61 @@ query_cache_flags() {
     printf "%s" "query_limit=${FPF_QUERY_RESULT_LIMIT:-0};per_manager_limit=${FPF_QUERY_PER_MANAGER_LIMIT:-40};no_query_limit=${FPF_NO_QUERY_RESULT_LIMIT:-120};no_query_npm_limit=${FPF_NO_QUERY_NPM_LIMIT:-120}"
 }
 
+query_cache_default_enabled_for_manager() {
+    local manager="$1"
+
+    case "${manager}" in
+        apt|brew|pacman|bun)
+            return 0
+            ;;
+        *)
+            return 1
+            ;;
+    esac
+}
+
+query_cache_ttl_seconds_for_manager() {
+    local manager="$1"
+    local default_ttl="${FPF_QUERY_CACHE_TTL:-300}"
+    local manager_ttl="0"
+
+    if ! [[ "${default_ttl}" =~ ^[0-9]+$ ]]; then
+        default_ttl=300
+    fi
+
+    case "${manager}" in
+        bun)
+            manager_ttl="${FPF_BUN_QUERY_CACHE_TTL:-900}"
+            if ! [[ "${manager_ttl}" =~ ^[0-9]+$ ]]; then
+                manager_ttl=900
+            fi
+            ;;
+        apt)
+            manager_ttl="${FPF_APT_QUERY_CACHE_TTL:-${default_ttl}}"
+            if ! [[ "${manager_ttl}" =~ ^[0-9]+$ ]]; then
+                manager_ttl="${default_ttl}"
+            fi
+            ;;
+        brew)
+            manager_ttl="${FPF_BREW_QUERY_CACHE_TTL:-${default_ttl}}"
+            if ! [[ "${manager_ttl}" =~ ^[0-9]+$ ]]; then
+                manager_ttl="${default_ttl}"
+            fi
+            ;;
+        pacman)
+            manager_ttl="${FPF_PACMAN_QUERY_CACHE_TTL:-${default_ttl}}"
+            if ! [[ "${manager_ttl}" =~ ^[0-9]+$ ]]; then
+                manager_ttl="${default_ttl}"
+            fi
+            ;;
+        *)
+            manager_ttl=0
+            ;;
+    esac
+
+    printf "%s" "${manager_ttl}"
+}
+
 log() {
     printf "%s\n" "$*" >&2
 }
@@ -98,17 +153,39 @@ loading_progress_mark_done() {
     loading_progress_mark_state "${manager}" "done" "${detail}"
 }
 
-loading_progress_repeat_char() {
-    local count="$1"
-    local char="$2"
-    local chunk=""
+loading_indicator_terminal_cols() {
+    local cols="${COLUMNS:-}"
 
-    if ! [[ "${count}" =~ ^[0-9]+$ ]] || [[ "${count}" -le 0 ]]; then
-        return 0
+    if ! [[ "${cols}" =~ ^[0-9]+$ ]] || [[ "${cols}" -lt 40 ]]; then
+        if command_exists tput; then
+            cols="$(tput cols 2>/dev/null || true)"
+        fi
+    fi
+
+    if ! [[ "${cols}" =~ ^[0-9]+$ ]] || [[ "${cols}" -lt 40 ]]; then
+        cols=120
+    fi
+
+    printf "%s" "${cols}"
+}
+
+loading_indicator_fit_line() {
+    local line="$1"
+    local cols
+    local max_len
+
+    cols="$(loading_indicator_terminal_cols)"
+    max_len=$((cols - 1))
+    if [[ "${max_len}" -lt 20 ]]; then
+        max_len=20
     fi
 
-    printf -v chunk "%*s" "${count}" ""
-    printf "%s" "${chunk// /${char}}"
+    if [[ "${#line}" -gt "${max_len}" ]]; then
+        printf "%s..." "${line:0:$((max_len - 3))}"
+        return
+    fi
+
+    printf "%s" "${line}"
 }
 
 loading_progress_snapshot() {
@@ -116,31 +193,22 @@ loading_progress_snapshot() {
     local manager=""
     local status=""
     local detail=""
-    local bar_width="${FPF_LOADING_BAR_WIDTH:-26}"
     local total=0
     local done=0
     local running=0
     local queued=0
     local progress_percent=0
-    local filled_width=0
-    local remaining_width=0
-    local bar=""
     local -a active_details=()
     local active_text=""
     local phase_text=""
+    local active_total=0
+    local max_active_items=3
     local idx
 
     if [[ -z "${LOADING_PROGRESS_DIR}" || ! -d "${LOADING_PROGRESS_DIR}" ]]; then
         return 0
     fi
 
-    if ! [[ "${bar_width}" =~ ^[0-9]+$ ]] || [[ "${bar_width}" -lt 10 ]]; then
-        bar_width=26
-    fi
-    if [[ "${bar_width}" -gt 60 ]]; then
-        bar_width=60
-    fi
-
     for status_file in "${LOADING_PROGRESS_DIR}"/*.status; do
         [[ -f "${status_file}" ]] || continue
         total=$((total + 1))
@@ -152,7 +220,10 @@ loading_progress_snapshot() {
                 ;;
             running)
                 running=$((running + 1))
-                active_details+=("$(manager_label "${manager}"): ${detail:-working}")
+                active_total=$((active_total + 1))
+                if [[ "${#active_details[@]}" -lt "${max_active_items}" ]]; then
+                    active_details+=("${manager}: ${detail:-work}")
+                fi
                 ;;
             *)
                 queued=$((queued + 1))
@@ -163,25 +234,18 @@ loading_progress_snapshot() {
     [[ "${total}" -gt 0 ]] || return 0
 
     progress_percent=$((done * 100 / total))
-    filled_width=$((done * bar_width / total))
-    if [[ "${done}" -ge "${total}" ]]; then
-        bar="$(loading_progress_repeat_char "${bar_width}" "=")"
-    else
-        remaining_width=$((bar_width - filled_width - 1))
-        bar="$(loading_progress_repeat_char "${filled_width}" "=")>"
-        if [[ "${remaining_width}" -gt 0 ]]; then
-            bar+=$(printf "%${remaining_width}s" "")
-        fi
-    fi
 
     if [[ "${#active_details[@]}" -gt 0 ]]; then
         active_text=""
         for idx in "${!active_details[@]}"; do
             if [[ -n "${active_text}" ]]; then
-                active_text+="; "
+                active_text+=", "
             fi
             active_text+="${active_details[$idx]}"
         done
+        if [[ "${active_total}" -gt "${#active_details[@]}" ]]; then
+            active_text+=", +$((active_total - ${#active_details[@]}))"
+        fi
         phase_text="active: ${active_text}"
     elif [[ "${queued}" -gt 0 ]]; then
         phase_text="starting queued managers (${queued} remaining)"
@@ -189,7 +253,7 @@ loading_progress_snapshot() {
         phase_text="finalizing combined results"
     fi
 
-    printf "[%s] %3s%% (%s/%s) | %s" "${bar}" "${progress_percent}" "${done}" "${total}" "${phase_text}"
+    printf "%3s%% (%s/%s) | %s" "${progress_percent}" "${done}" "${total}" "${phase_text}"
 }
 
 start_loading_indicator() {
@@ -206,6 +270,7 @@ start_loading_indicator() {
         local started_epoch
         local elapsed_seconds=0
         local snapshot=""
+        local line_output=""
 
         started_epoch="$(date +%s)"
         if ! [[ "${started_epoch}" =~ ^[0-9]+$ ]]; then
@@ -220,11 +285,13 @@ start_loading_indicator() {
 
             snapshot="$(loading_progress_snapshot)"
             if [[ -n "${snapshot}" ]]; then
-                printf "\r\033[2K%s %s | elapsed: %ss" "${message}" "${snapshot}" "${elapsed_seconds}" >&2
+                line_output="${message} ${snapshot} | ${elapsed_seconds}s"
             else
-                printf "\r\033[2K%s [working] | elapsed: %ss" "${message}" "${elapsed_seconds}" >&2
+                line_output="${message} working | ${elapsed_seconds}s"
             fi
-            sleep 0.1
+            line_output="$(loading_indicator_fit_line "${line_output}")"
+            printf "\r\033[2K%s" "${line_output}" >&2
+            sleep 0.15
         done
     ) &
     LOADING_INDICATOR_PID="$!"
@@ -2481,8 +2548,10 @@ manager_search_entries_uncached() {
 manager_search_entries() {
     local manager="$1"
     local query="$2"
-    local query_cache_enabled="${FPF_ENABLE_QUERY_CACHE:-0}"
-    local bun_cache_ttl="${FPF_BUN_QUERY_CACHE_TTL:-900}"
+    local query_cache_enabled="0"
+    local query_cache_setting="${FPF_ENABLE_QUERY_CACHE:-auto}"
+    local query_cache_ttl="0"
+    local bun_cache_ttl="900"
     local bypass_query_cache="${FPF_BYPASS_QUERY_CACHE:-0}"
     local flags
     local key
@@ -2495,19 +2564,28 @@ manager_search_entries() {
     initialize_cache_root
 
     flags="$(query_cache_flags)"
+    query_cache_setting="$(trim_whitespace "${query_cache_setting}")"
+    query_cache_setting="$(printf "%s" "${query_cache_setting}" | tr '[:upper:]' '[:lower:]')"
+    query_cache_ttl="$(query_cache_ttl_seconds_for_manager "${manager}")"
 
-    if ! [[ "${bun_cache_ttl}" =~ ^[0-9]+$ ]]; then
-        bun_cache_ttl=900
+    if [[ "${manager}" == "bun" ]]; then
+        bun_cache_ttl="${query_cache_ttl}"
     fi
 
     case "${bypass_query_cache}" in
         1|true|yes|on)
-            query_cache_enabled="0"
             ;;
         *)
-            if [[ "${manager}" == "bun" ]]; then
-                query_cache_enabled="1"
-            fi
+            case "${query_cache_setting}" in
+                1|true|yes|on)
+                    query_cache_enabled="1"
+                    ;;
+                auto|"")
+                    if query_cache_default_enabled_for_manager "${manager}"; then
+                        query_cache_enabled="1"
+                    fi
+                    ;;
+            esac
             ;;
     esac
 
@@ -2546,7 +2624,7 @@ manager_search_entries() {
     fi
 
     if [[ "${query_cache_enabled}" == "1" && -s "${cache_file}" ]]; then
-        if [[ "${manager}" != "bun" ]] || cache_is_fresh_with_ttl "${key}" "${bun_cache_ttl}"; then
+        if [[ "${query_cache_ttl}" -eq 0 ]] || cache_is_fresh_with_ttl "${key}" "${query_cache_ttl}"; then
             cat "${cache_file}"
             return
         fi
@@ -2901,7 +2979,7 @@ mark_installed_packages() {
                 manager_match_count=0
             fi
 
-            loading_progress_mark_running "${manager}" "marking installed status for ${manager_match_count} matches"
+            loading_progress_mark_running "${manager}" "mark installed (${manager_match_count})"
             installed_file="$(mktemp "${SESSION_TMP_ROOT}/installed.${manager}.XXXXXX")"
             manager_installed_names_cached "${manager}" "${installed_file}" || true
             installed_match_count=0
@@ -2995,7 +3073,7 @@ collect_search_display_rows() {
             local local_source_file
             local result_count=0
 
-            loading_progress_mark_running "${manager}" "querying package index"
+            loading_progress_mark_running "${manager}" "query index"
             local_source_file="$(mktemp "${SESSION_TMP_ROOT}/source.XXXXXX")"
             manager_search_entries "${manager}" "${query}" >"${local_source_file}" || true
             if [[ -s "${local_source_file}" ]]; then
@@ -3012,7 +3090,7 @@ collect_search_display_rows() {
             if [[ -s "${part_file}" ]]; then
                 result_count="$(awk 'END { print NR + 0 }' "${part_file}")"
             fi
-            loading_progress_mark_running "${manager}" "${result_count} matches indexed; waiting for installed check"
+            loading_progress_mark_running "${manager}" "${result_count} indexed; mark installed"
         ) &
         gather_pids+=("$!")
     done
@@ -3069,7 +3147,7 @@ collect_installed_display_rows() {
             local local_source_file
             local result_count=0
 
-            loading_progress_mark_running "${manager}" "reading installed package list"
+            loading_progress_mark_running "${manager}" "read installed"
             local_source_file="$(mktemp "${SESSION_TMP_ROOT}/source.XXXXXX")"
             manager_installed_entries "${manager}" >"${local_source_file}" || true
             if [[ -s "${local_source_file}" ]]; then

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fpf-cli",
-  "version": "1.6.38",
+  "version": "1.6.40",
   "description": "Cross-platform fuzzy package finder powered by fzf",
   "bin": {
     "fpf": "fpf"