Improved auto-completion for files and folders.

Author: h3rald

minpkg/core/shell.nim

@@ -94,24 +94,24 @@ proc getCompletions*(ed: LineEditor, i: MinInterpreter): seq[string] =
     var f = word[1..^1]
     if f == "":
       f = getCurrentDir().replace("\\", "/")
-      return toSeq(walkDir(f, true)).mapIt("\"$1" % it.path.replace("\\", "/"))
+      return toSeq(walkDir(f, true)).mapIt("\"$1" % it.path.replace("\\", "/")&"\"")
     elif f.dirExists:
       f = f.replace("\\", "/")
       if f[f.len-1] != '/':
         f = f & "/"
-      return toSeq(walkDir(f, true)).mapIt("\"$1$2" % [f, it.path.replace("\\", "/")])
+      return toSeq(walkDir(f, true)).mapIt("\"$1$2\"" % [f, it.path.replace("\\", "/")])
     else:
       var dir: string
       if f.contains("/") or dir.contains("\\"):
         dir = f.parentDir
         let file = f.extractFileName
         return toSeq(walkDir(dir, true)).filterIt(
             it.path.toLowerAscii.startsWith(file.toLowerAscii)).mapIt(
-            "\"$1/$2" % [dir, it.path.replace("\\", "/")])
+            "\"$1/$2\"" % [dir, it.path.replace("\\", "/")])
       else:
         dir = getCurrentDir()
         return toSeq(walkDir(dir, true)).filterIt(
-            it.path.toLowerAscii.startsWith(f.toLowerAscii)).mapIt("\"$1" % [
+            it.path.toLowerAscii.startsWith(f.toLowerAscii)).mapIt("\"$1\"" % [
             it.path.replace("\\", "/")])
   return symbols
 

next-release.md

@@ -1,7 +1,6 @@
 ### BREAKING CHANGES
 
 * SSL is now enabled by default when installing via nimble and when compiling by default. Use `-d:nossl` to disable.
-* The min shell no longer attempts to auto-complete executables (it never worked properly anyway).
 
 ### New Features
 
@@ -12,4 +11,6 @@
 
 ### Fixes and Improvements
 
+* Auto-completions for files and folders now automatically end with `"`.
+* The min shell no longer attempts to auto-complete executables (it never worked properly anyway).