fpf-cli 1.6.36 → 1.6.38

package/README.md

@@ -98,7 +98,7 @@ Installed packages are marked with `*` in the result list.
 - `FPF_DYNAMIC_RELOAD`: `always` (default), `single`, or `never`
 - Live reload uses `change:reload` by default for reliability (`ctrl-r` uses the same reload command).
 - Set `FPF_DYNAMIC_RELOAD_TRANSPORT=ipc` to opt into `--listen` + IPC query notifications on supported `fzf` builds.
-- Startup now shows a dynamic pre-search loader with per-manager progress + elapsed time.
+- Startup now shows a DynamicProgress-style pre-search bar with concurrent per-manager task text + elapsed time.
 - 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`)

package/fpf

@@ -3,7 +3,7 @@
 set -euo pipefail
 
 SCRIPT_NAME="fpf"
-SCRIPT_VERSION="1.6.36"
+SCRIPT_VERSION="1.6.38"
 TMP_ROOT="${TMPDIR:-/tmp}/fpf"
 SESSION_TMP_ROOT=""
 HELP_FILE=""
@@ -98,23 +98,49 @@ loading_progress_mark_done() {
     loading_progress_mark_state "${manager}" "done" "${detail}"
 }
 
+loading_progress_repeat_char() {
+    local count="$1"
+    local char="$2"
+    local chunk=""
+
+    if ! [[ "${count}" =~ ^[0-9]+$ ]] || [[ "${count}" -le 0 ]]; then
+        return 0
+    fi
+
+    printf -v chunk "%*s" "${count}" ""
+    printf "%s" "${chunk// /${char}}"
+}
+
 loading_progress_snapshot() {
     local status_file=""
     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_display=""
+    local active_text=""
+    local phase_text=""
     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))
@@ -126,9 +152,7 @@ loading_progress_snapshot() {
                 ;;
             running)
                 running=$((running + 1))
-                if [[ "${#active_details[@]}" -lt 2 ]]; then
-                    active_details+=("$(manager_label "${manager}"): ${detail:-working}")
-                fi
+                active_details+=("$(manager_label "${manager}"): ${detail:-working}")
                 ;;
             *)
                 queued=$((queued + 1))
@@ -138,27 +162,34 @@ loading_progress_snapshot() {
 
     [[ "${total}" -gt 0 ]] || return 0
 
-    active_display=""
-    for idx in "${!active_details[@]}"; do
-        if [[ -n "${active_display}" ]]; then
-            active_display+=", "
-        fi
-        active_display+="${active_details[$idx]}"
-    done
-    if [[ "${running}" -gt "${#active_details[@]}" ]]; then
-        if [[ -n "${active_display}" ]]; then
-            active_display+=", "
+    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
-        active_display+="+$((running - ${#active_details[@]})) more"
     fi
 
-    printf "%s/%s done" "${done}" "${total}"
-    if [[ -n "${active_display}" ]]; then
-        printf " | active: %s" "${active_display}"
-    fi
-    if [[ "${queued}" -gt 0 ]]; then
-        printf " | queued: %s" "${queued}"
+    if [[ "${#active_details[@]}" -gt 0 ]]; then
+        active_text=""
+        for idx in "${!active_details[@]}"; do
+            if [[ -n "${active_text}" ]]; then
+                active_text+="; "
+            fi
+            active_text+="${active_details[$idx]}"
+        done
+        phase_text="active: ${active_text}"
+    elif [[ "${queued}" -gt 0 ]]; then
+        phase_text="starting queued managers (${queued} remaining)"
+    else
+        phase_text="finalizing combined results"
     fi
+
+    printf "[%s] %3s%% (%s/%s) | %s" "${bar}" "${progress_percent}" "${done}" "${total}" "${phase_text}"
 }
 
 start_loading_indicator() {
@@ -172,8 +203,6 @@ start_loading_indicator() {
 
     (
         trap 'exit 0' TERM INT
-        local frames='|/-\'
-        local frame_index=0
         local started_epoch
         local elapsed_seconds=0
         local snapshot=""
@@ -191,11 +220,10 @@ start_loading_indicator() {
 
             snapshot="$(loading_progress_snapshot)"
             if [[ -n "${snapshot}" ]]; then
-                printf "\r\033[2K%s [%s] %s | elapsed: %ss" "${message}" "${frames:frame_index:1}" "${snapshot}" "${elapsed_seconds}" >&2
+                printf "\r\033[2K%s %s | elapsed: %ss" "${message}" "${snapshot}" "${elapsed_seconds}" >&2
             else
-                printf "\r\033[2K%s [%s] elapsed: %ss" "${message}" "${frames:frame_index:1}" "${elapsed_seconds}" >&2
+                printf "\r\033[2K%s [working] | elapsed: %ss" "${message}" "${elapsed_seconds}" >&2
             fi
-            frame_index=$(((frame_index + 1) % 4))
             sleep 0.1
         done
     ) &
@@ -2825,9 +2853,23 @@ manager_installed_names_cached() {
 mark_installed_packages() {
     local source_file="$1"
     local output_file="$2"
+    local unique_managers_file
+    local manager_counts_file
     local installed_file
     local installed_map_file
+    local installed_part_file
+    local manager_match_count
+    local installed_match_count
+    local installed_part_files=()
+    local gather_pids=()
     local manager
+    local gather_pid
+
+    unique_managers_file="$(mktemp "${SESSION_TMP_ROOT}/installed-managers.XXXXXX")"
+    awk -F'\t' 'NF >= 1 && $1 != "" { print $1 }' "${source_file}" | awk '!seen[$0]++' >"${unique_managers_file}"
+
+    manager_counts_file="$(mktemp "${SESSION_TMP_ROOT}/installed-counts.XXXXXX")"
+    awk -F'\t' 'NF >= 1 && $1 != "" { counts[$1]++ } END { for (mgr in counts) print mgr "\t" counts[mgr] }' "${source_file}" >"${manager_counts_file}"
 
     if [[ "${FPF_SKIP_INSTALLED_MARKERS:-0}" == "1" ]]; then
         awk -F'\t' '
@@ -2837,6 +2879,11 @@ mark_installed_packages() {
                 print $1 "\t" $2 "\t  " desc
             }
         ' "${source_file}" >"${output_file}"
+        while IFS= read -r manager; do
+            [[ -n "${manager}" ]] || continue
+            loading_progress_mark_done "${manager}" "installed marker check skipped"
+        done <"${unique_managers_file}"
+        rm -f "${unique_managers_file}" "${manager_counts_file}"
         return
     fi
 
@@ -2845,13 +2892,39 @@ mark_installed_packages() {
 
     while IFS= read -r manager; do
         [[ -n "${manager}" ]] || continue
-        installed_file="$(mktemp "${SESSION_TMP_ROOT}/installed.${manager}.XXXXXX")"
-        manager_installed_names_cached "${manager}" "${installed_file}"
-        if [[ -s "${installed_file}" ]]; then
-            awk -F'\t' -v mgr="${manager}" 'NF > 0 && $1 != "" { print mgr "\t" $1 }' "${installed_file}" >>"${installed_map_file}"
+        installed_part_file="$(mktemp "${SESSION_TMP_ROOT}/installed-map.${manager}.XXXXXX")"
+        installed_part_files+=("${installed_part_file}")
+
+        (
+            manager_match_count="$(awk -F'\t' -v mgr="${manager}" '$1 == mgr { print $2; found=1; exit } END { if (!found) print 0 }' "${manager_counts_file}")"
+            if ! [[ "${manager_match_count}" =~ ^[0-9]+$ ]]; then
+                manager_match_count=0
+            fi
+
+            loading_progress_mark_running "${manager}" "marking installed status for ${manager_match_count} matches"
+            installed_file="$(mktemp "${SESSION_TMP_ROOT}/installed.${manager}.XXXXXX")"
+            manager_installed_names_cached "${manager}" "${installed_file}" || true
+            installed_match_count=0
+            if [[ -s "${installed_file}" ]]; then
+                awk -F'\t' -v mgr="${manager}" 'NF > 0 && $1 != "" { print mgr "\t" $1 }' "${installed_file}" >"${installed_part_file}"
+                installed_match_count="$(awk 'END { print NR + 0 }' "${installed_part_file}")"
+            fi
+            rm -f "${installed_file}"
+            loading_progress_mark_done "${manager}" "${manager_match_count} matches, ${installed_match_count} installed"
+        ) &
+        gather_pids+=("$!")
+    done <"${unique_managers_file}"
+
+    for gather_pid in "${gather_pids[@]-}"; do
+        wait "${gather_pid}" || true
+    done
+
+    for installed_part_file in "${installed_part_files[@]-}"; do
+        if [[ -s "${installed_part_file}" ]]; then
+            cat "${installed_part_file}" >>"${installed_map_file}"
         fi
-        rm -f "${installed_file}"
-    done < <(awk -F'\t' 'NF >= 1 && $1 != "" { print $1 }' "${source_file}" | awk '!seen[$0]++')
+        rm -f "${installed_part_file}"
+    done
 
     awk -F'\t' '
         FILENAME == ARGV[1] {
@@ -2870,6 +2943,7 @@ mark_installed_packages() {
         }
     ' "${installed_map_file}" "${source_file}" >"${output_file}"
 
+    rm -f "${unique_managers_file}" "${manager_counts_file}"
     rm -f "${installed_map_file}"
 }
 
@@ -2921,7 +2995,7 @@ collect_search_display_rows() {
             local local_source_file
             local result_count=0
 
-            loading_progress_mark_running "${manager}" "searching package index"
+            loading_progress_mark_running "${manager}" "querying package 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
@@ -2938,7 +3012,7 @@ collect_search_display_rows() {
             if [[ -s "${part_file}" ]]; then
                 result_count="$(awk 'END { print NR + 0 }' "${part_file}")"
             fi
-            loading_progress_mark_done "${manager}" "${result_count} results indexed"
+            loading_progress_mark_running "${manager}" "${result_count} matches indexed; waiting for installed check"
         ) &
         gather_pids+=("$!")
     done
@@ -2966,6 +3040,10 @@ collect_search_display_rows() {
             awk -v limit="${query_limit}" 'NR <= limit' "${output_file}" >"${output_file}.limited"
             mv "${output_file}.limited" "${output_file}"
         fi
+    else
+        for manager in "${managers[@]-}"; do
+            loading_progress_mark_done "${manager}" "0 packages matched"
+        done
     fi
 
     rm -f "${merged_source_file}" "${merged_marked_file}"
@@ -2991,7 +3069,7 @@ collect_installed_display_rows() {
             local local_source_file
             local result_count=0
 
-            loading_progress_mark_running "${manager}" "reading installed packages"
+            loading_progress_mark_running "${manager}" "reading installed package list"
             local_source_file="$(mktemp "${SESSION_TMP_ROOT}/source.XXXXXX")"
             manager_installed_entries "${manager}" >"${local_source_file}" || true
             if [[ -s "${local_source_file}" ]]; then
@@ -3008,7 +3086,7 @@ collect_installed_display_rows() {
             if [[ -s "${part_file}" ]]; then
                 result_count="$(awk 'END { print NR + 0 }' "${part_file}")"
             fi
-            loading_progress_mark_done "${manager}" "${result_count} installed entries"
+            loading_progress_mark_done "${manager}" "${result_count} installed packages"
         ) &
         gather_pids+=("$!")
     done

package/package.json

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