Simplify text editor fallback

src/cli/init.go

@@ -1,6 +1,7 @@
 package cli
 
 import (
+	"cmp"
 	"fmt"
 	"os"
 	"strconv"
@@ -27,10 +28,7 @@ func TempInitCli() {
 	}
 
 	// textEditor
-	textEditor := input("Text editor (leave blank for $EDITOR, falls back to \"" + fallbackEditor + "\"):")
+	textEditor := cmp.Or(input("Text editor (leave blank for $EDITOR, falls back to \""+fallbackEditor+"\"):"), os.Getenv("EDITOR"), fallbackEditor)
-	if textEditor == "" {
-		textEditor = textEditorFallback()
-	}
 
 	// SSH info
 	configSSH := inputBinary("Configure SSH settings (for synchronization)?")

src/cli/initUNIX.go

@@ -2,17 +2,4 @@
 
 package cli
 
-import (
+const fallbackEditor = "vi" // vi is pre-installed on most UNIX-like systems
-	"os"
-)
-
-const fallbackEditor = "vi" // vi is pre-installed on most UNIX systems
-
-// textEditorFallback returns the value of the $EDITOR environment variable, or FallbackEditor if it is not set
-func textEditorFallback() string {
-	textEditor := os.Getenv("EDITOR")
-	if textEditor == "" {
-		textEditor = fallbackEditor
-	}
-	return textEditor
-}

src/cli/initWIN.go

@@ -3,8 +3,3 @@
 package cli
 
 const fallbackEditor = "nvim" // since there is no pre-installed CLI editor on Windows, default to the most popular one
-
-// textEditorFallback returns FallbackEditor
-func textEditorFallback() string {
-	return fallbackEditor
-}