Fix when lftp bookmark file is not readable (#1114)

hongquan · web-flow · commit 54f10ee7ee8a · 2025-05-03T18:25:23.000-05:00
Sorry, in the #1113 , I forgot to test the case that `lftp`'s bookmark file is not readable.
diff --git a/custom-completions/lftp/lftp-completions.nu b/custom-completions/lftp/lftp-completions.nu
@@ -73,7 +73,8 @@ module lftp-completion-utils {
 
     export def get-bookmark-sites []: nothing -> table<value: string, description: string> {
       const FILE_PATH = '~/.local/share/lftp/bookmarks' | path expand
-      let sites = open --raw $FILE_PATH | lines | split column -n 2 -r \s+ value description
+      let sites = try { open --raw $FILE_PATH | lines } catch { [] }
+        | split column -n 2 -r \s+ value description
       $sites | update value { $"bm:($in)" }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,8 @@ module lftp-completion-utils {`
`73`	`73`
`74`	`74`	`export def get-bookmark-sites []: nothing -> table<value: string, description: string> {`
`75`	`75`	`const FILE_PATH = '~/.local/share/lftp/bookmarks' \| path expand`
`76`		`- let sites = open --raw $FILE_PATH \| lines \| split column -n 2 -r \s+ value description`
	`76`	`+ let sites = try { open --raw $FILE_PATH \| lines } catch { [] }`
	`77`	`+ \| split column -n 2 -r \s+ value description`
`77`	`78`	`$sites \| update value { $"bm:($in)" }`
`78`	`79`	`}`
`79`	`80`	`}`