refactor: make go readline lib default, remove hilbiline

.github/workflows/build.yml

@@ -6,13 +6,17 @@ on:
 
 jobs:
   build:
-    name: ${{ matrix.build }}
+    name: ${{ matrix.goos }}-${{ matrix.goarch }}
-    runs-on: ${{ matrix.os }}
+    runs-on: ubuntu-latest
     strategy:
       matrix:
-        include:
+        goos: [linux, windows, darwin]
-          - build: linux-amd64
+        goarch: ["386", amd64, arm64]
-            os: ubuntu-latest
+        exclude:  
+          - goarch: "386"
+            goos: darwin 
+          - goarch: arm64
+            goos: windows
     steps:
       - name: Checkout sources
         uses: actions/checkout@v2
@@ -23,9 +27,9 @@ jobs:
         with:
           go-version: '1.16.2'
       - name: Build
-        run: make hilbiline
+        run: GOARCH=${{ matrix.goarch }} go build
       - uses: actions/upload-artifact@v2
         with:
-          name: hilbish-${{ matrix.build }}
+          name: hilbish-${{ matrix.goos }}-${{ matrix.goarch }}
           path: hilbish
 

Makefile

@@ -9,8 +9,8 @@ build:
 dev:
 	@go build -ldflags "-s -w -X main.version=$(shell git describe --tags)"
 
-hilbiline:
+gnurl:
-	@go build -ldflags "-s -w -X main.version=$(shell git describe --tags)+hilbiline" -tags hilbiline
+	@go build -ldflags "-s -w -X main.version=$(shell git describe --tags)+grnurl" -tags gnurl
 
 install:
 	@install -v -d "$(DESTDIR)$(BINDIR)/" && install -m 0755 -v hilbish "$(DESTDIR)$(BINDIR)/hilbish"

go.mod

@@ -3,16 +3,9 @@ module hilbish
 go 1.16
 
 require (
-	github.com/Rosettea/Hilbiline v0.0.0-20210624011007-8088a2d84b65
 	github.com/Rosettea/readline v0.0.0-20211207004608-4afb088da503
 	github.com/chuckpreslar/emission v0.0.0-20170206194824-a7ddd980baf9
-	github.com/creack/goselect v0.1.2 // indirect
-	github.com/creack/termios v0.0.0-20160714173321-88d0029e36a1 // indirect
-	github.com/kr/pty v1.1.8 // indirect
-	github.com/mattn/go-isatty v0.0.14 // indirect
-	github.com/mattn/go-runewidth v0.0.13 // indirect
 	github.com/maxlandon/readline v0.1.0-beta.0.20211027085530-2b76cabb8036
-	github.com/pborman/ansi v1.0.0 // indirect
 	github.com/pborman/getopt v1.1.0
 	github.com/yuin/gopher-lua v0.0.0-20210529063254-f4c35e4016d9
 	golang.org/x/sys v0.0.0-20220224120231-95c6836cb0e7 // indirect

go.sum

@@ -1,7 +1,3 @@
-github.com/Rosettea/Hilbiline v0.0.0-20210624011007-8088a2d84b65 h1:z6alP313nLg1U/V3bKSd6849L/A2LxDSwKv4CPqqzDk=
-github.com/Rosettea/Hilbiline v0.0.0-20210624011007-8088a2d84b65/go.mod h1:/FFZ4cgR6TXXYaskRUxyLIYdfG0PS4BPtWjWRQms754=
-github.com/Rosettea/Hilbiline v0.0.0-20210710124707-aa6e3ff34cb2 h1:6f1umn6mkodpGf6rK9LZjr4Gut2uS+b8QLoFBFTeOE8=
-github.com/Rosettea/Hilbiline v0.0.0-20210710124707-aa6e3ff34cb2/go.mod h1:0J2+sRC+d4a3swcH20sVlFvYUEXASvGTHJnVTTI4S9w=
 github.com/Rosettea/readline v0.0.0-20211207004608-4afb088da503 h1:hA2kXBwY8SFH3+PT67CkoUjlMuRN08RSEtsKjwQ+of4=
 github.com/Rosettea/readline v0.0.0-20211207004608-4afb088da503/go.mod h1:OH+WJSCks0t2ISvaCFUT4ZxNGr4Etq4ju9JE/UxH/5o=
 github.com/Rosettea/readline-1 v0.1.0-beta.0.20211207003625-341c7985ad7d h1:KBttN41h/tPahmpaZavviwQ8q4rCkt5CD0HdVmfgPVA=
@@ -15,14 +11,9 @@ github.com/chuckpreslar/emission v0.0.0-20170206194824-a7ddd980baf9/go.mod h1:2w
 github.com/chzyer/logex v1.1.10/go.mod h1:+Ywpsq7O8HXn0nuIou7OrIPyXbp3wmkHB+jjWRnGsAI=
 github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e/go.mod h1:nSuG5e5PlCu98SY8svDHJxuZscDgtXS6KTTbou5AhLI=
 github.com/chzyer/test v0.0.0-20180213035817-a1ea475d72b1/go.mod h1:Q3SI9o4m/ZMnBNeIyt5eFwwo7qiLfzFZmjNmxjkiQlU=
-github.com/creack/goselect v0.1.2 h1:2DNy14+JPjRBgPzAd1thbQp4BSIihxcBf0IXhQXDRa0=
-github.com/creack/goselect v0.1.2/go.mod h1:a/NhLweNvqIYMuxcMOuWY516Cimucms3DglDzQP3hKY=
-github.com/creack/pty v1.1.7/go.mod h1:lj5s0c3V2DBrqTV7llrYr5NG6My20zk30Fl46Y7DoTY=
 github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
 github.com/creack/pty v1.1.15 h1:cKRCLMj3Ddm54bKSpemfQ8AtYFBhAI2MPmdys22fBdc=
 github.com/creack/pty v1.1.15/go.mod h1:MOBLtS5ELjhRRrroQr9kyvTxUAFNvYEK993ew/Vr4O4=
-github.com/creack/termios v0.0.0-20160714173321-88d0029e36a1 h1:3ZFknr3UZk2E18CuCeA0NMRk226zM/slMEFOmJTtJjI=
-github.com/creack/termios v0.0.0-20160714173321-88d0029e36a1/go.mod h1:141QqpYDZtzU1VJMRHPU6ZWkn9K5cE6X8elajH9hJk4=
 github.com/evilsocket/islazy v1.10.6 h1:MFq000a1ByoumoJWlytqg0qon0KlBeUfPsDjY0hK0bo=
 github.com/evilsocket/islazy v1.10.6/go.mod h1:OrwQGYg3DuZvXUfmH+KIZDjwTCbrjy48T24TUpGqVVw=
 github.com/google/renameio v1.0.1/go.mod h1:t/HQoYBZSsWSNK35C6CO/TpPLDVWvxOHboWUAweKUpk=
@@ -31,27 +22,16 @@ github.com/kr/pretty v0.1.0/go.mod h1:dAy3ld7l9f0ibDNOQOHHMYYIIbhfbHSm3C4ZsoJORN
 github.com/kr/pretty v0.3.0 h1:WgNl7dwNpEZ6jJ9k1snq4pZsg7DOEN8hP9Xw0Tsjwk0=
 github.com/kr/pretty v0.3.0/go.mod h1:640gp4NfQd8pI5XOwp5fnNeVWj67G7CFk/SaSQn7NBk=
 github.com/kr/pty v1.1.1/go.mod h1:pFQYn66WHrOpPYNljwOMqo10TkYh1fy3cYio2l3bCsQ=
-github.com/kr/pty v1.1.8 h1:AkaSdXYQOWeaO3neb8EM634ahkXXe3jYbVh/F9lq+GI=
-github.com/kr/pty v1.1.8/go.mod h1:O1sed60cT9XZ5uDucP5qwvh+TE3NnUj51EiZO/lmSfw=
 github.com/kr/text v0.1.0/go.mod h1:4Jbv+DJW3UT/LiOwJeYQe1efqtUx/iVham/4vfdArNI=
 github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
 github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
 github.com/layeh/gopher-luar v1.0.10 h1:8NIv4MX1Arz96kK4buGK1D87DyDxKZyq6KKvJ2diHp0=
 github.com/layeh/gopher-luar v1.0.10/go.mod h1:TPnIVCZ2RJBndm7ohXyaqfhzjlZ+OA2SZR/YwL8tECk=
-github.com/mattn/go-isatty v0.0.13/go.mod h1:cbi8OIDigv2wuxKPP5vlRcQ1OAZbq2CE4Kysco4FUpU=
-github.com/mattn/go-isatty v0.0.14 h1:yVuAays6BHfxijgZPzw+3Zlu5yQgKGP2/hcQbHb7S9Y=
-github.com/mattn/go-isatty v0.0.14/go.mod h1:7GGIvUiUoEMVVmxf/4nioHXj79iQHKdU27kJ6hsGG94=
-github.com/mattn/go-runewidth v0.0.12/go.mod h1:RAqKPSqVFrSLVXbA8x7dzmKdmGzieGRCM46jaSJTDAk=
-github.com/mattn/go-runewidth v0.0.13 h1:lTGmDsbAYt5DmK6OnoV7EuIF1wEIFAcxld6ypU4OSgU=
-github.com/mattn/go-runewidth v0.0.13/go.mod h1:Jdepj2loyihRzMpdS35Xk/zdY8IAYHsh153qUoGf23w=
 github.com/olekukonko/ts v0.0.0-20171002115256-78ecb04241c0 h1:LiZB1h0GIcudcDci2bxbqI6DXV8bF8POAnArqvRrIyw=
 github.com/olekukonko/ts v0.0.0-20171002115256-78ecb04241c0/go.mod h1:F/7q8/HZz+TXjlsoZQQKVYvXTZaFH4QRa3y+j1p7MS0=
-github.com/pborman/ansi v1.0.0 h1:OqjHMhvlSuCCV5JT07yqPuJPQzQl+WXsiZ14gZsqOrQ=
-github.com/pborman/ansi v1.0.0/go.mod h1:SgWzwMAx1X/Ez7i90VqF8LRiQtx52pWDiQP+x3iGnzw=
 github.com/pborman/getopt v1.1.0 h1:eJ3aFZroQqq0bWmraivjQNt6Dmm5M0h2JcDW38/Azb0=
 github.com/pborman/getopt v1.1.0/go.mod h1:FxXoW1Re00sQG/+KIkuSqRL/LwQgSkv7uyac+STFsbk=
 github.com/pkg/diff v0.0.0-20210226163009-20ebb0f2a09e/go.mod h1:pJLUxLENpZxwdsKMEsNbx1VGcRFpLqf3715MtcvvzbA=
-github.com/rivo/uniseg v0.1.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
 github.com/rivo/uniseg v0.2.0 h1:S1pD9weZBuJdFmowNwbpi7BJ8TNftyUImj/0WQi72jY=
 github.com/rivo/uniseg v0.2.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
 github.com/rogpeppe/go-internal v1.6.1/go.mod h1:xXDCJY+GAPziupqXw64V24skbSoqbTEfhy4qGm1nDQc=
@@ -63,21 +43,14 @@ github.com/yuin/gopher-lua v0.0.0-20210529063254-f4c35e4016d9/go.mod h1:E1AXubJB
 golang.org/x/sync v0.0.0-20210220032951-036812b2e83c h1:5KslGYwFpkhGh+Q16bwMP3cOontH8FOep7tGV86Y7SQ=
 golang.org/x/sync v0.0.0-20210220032951-036812b2e83c/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sys v0.0.0-20190204203706-41f3e6584952/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
-golang.org/x/sys v0.0.0-20200116001909-b77594299b42/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210320140829-1e4c9ba3b0c4/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/sys v0.0.0-20210414055047-fe65e336abe0/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210615035016-665e8c7367d1/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
-golang.org/x/sys v0.0.0-20210630005230-0f9fa26af87c/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20210925032602-92d5a993a665/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220224120231-95c6836cb0e7 h1:BXxu8t6QN0G1uff4bzZzSkpsax8+ALqTGUtz08QrV00=
 golang.org/x/sys v0.0.0-20220224120231-95c6836cb0e7/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
-golang.org/x/term v0.0.0-20210406210042-72f3dc4e9b72/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
 golang.org/x/term v0.0.0-20210916214954-140adaaadfaf/go.mod h1:jbD1KX2456YbFQfuXm/mYQcufACuNUgVhRMnK/tPxf8=
 golang.org/x/term v0.0.0-20210927222741-03fcf44c2211 h1:JGgROgKl9N8DuW20oFS5gxc+lE67/N3FcwmBPMe7ArY=
 golang.org/x/term v0.0.0-20210927222741-03fcf44c2211/go.mod h1:jbD1KX2456YbFQfuXm/mYQcufACuNUgVhRMnK/tPxf8=
 gopkg.in/check.v1 v1.0.0-20180628173108-788fd7840127/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/errgo.v2 v2.1.0/go.mod h1:hNsd1EY+bozCKY1Ytp96fpM3vjJbqLJn88ws8XvfDNI=
-layeh.com/gopher-luar v1.0.10 h1:55b0mpBhN9XSshEd2Nz6WsbYXctyBT35azk4POQNSXo=
-layeh.com/gopher-luar v1.0.10/go.mod h1:TPnIVCZ2RJBndm7ohXyaqfhzjlZ+OA2SZR/YwL8tECk=
 mvdan.cc/editorconfig v0.2.0/go.mod h1:lvnnD3BNdBYkhq+B4uBuFFKatfp02eB6HixDvEz91C0=

rl.go

@@ -1,57 +1,66 @@
-//go:build !hilbiline && !goreadline
+// +build !gnurl
-// +build !hilbiline,!goreadline
 
 package main
 
 // Here we define a generic interface for readline and hilbiline,
 // making them interchangable during build time
-// this is normal readline
+// this is hilbiline's, as is obvious by the filename
 
 import (
-	"os"
+	"fmt"
+	"io"
 	"path/filepath"
 	"strings"
+	"os"
 
-	"github.com/Rosettea/readline"
+	"github.com/maxlandon/readline"
 	"github.com/yuin/gopher-lua"
 )
 
 type lineReader struct {
-	Prompt string
+	rl *readline.Instance
 }
 
+// other gophers might hate this naming but this is local, shut up
 func newLineReader(prompt string) *lineReader {
-	readline.Init()
+	rl := readline.NewInstance()
-
+	rl.Multiline = true
-	readline.Completer = func(query string, ctx string) []string {
+	rl.TabCompleter = func(line []rune, pos int, _ readline.DelayedTabContext) (string, []*readline.CompletionGroup) {
+		ctx := string(line)
 		var completions []string
-		// trim whitespace from ctx
+		
+		compGroup := []*readline.CompletionGroup{
+			&readline.CompletionGroup{
+				TrimSlash: false,
+				NoSpace: true,
+			},
+		}
+
 		ctx = strings.TrimLeft(ctx, " ")
 		if len(ctx) == 0 {
-			return []string{}
+			return "", compGroup
 		}
+
 		fields := strings.Split(ctx, " ")
 		if len(fields) == 0 {
-			return []string{}
+			return "", compGroup
 		}
+		query := fields[len(fields) - 1]
 
 		ctx = aliases.Resolve(ctx)
 		
 		if len(fields) == 1 {
-			prefixes := []string{"./", "../", "/", "~/"}
+			fileCompletions := fileComplete(query, ctx, fields)
-			for _, prefix := range prefixes {
+			if len(fileCompletions) != 0 {
-				if strings.HasPrefix(fields[0], prefix) {
+				for _, f := range fileCompletions {
-					fileCompletions := append(completions, readline.FilenameCompleter(query, ctx)...)
+					name := strings.Replace(query + f, "~", curuser.HomeDir, 1)
-					// filter out executables
+					if info, err := os.Stat(name); err == nil && info.Mode().Perm() & 0100 == 0 {
-					for _, f := range fileCompletions {
+						continue
-						name := strings.Replace(f, "~", curuser.HomeDir, 1)
-						if info, err := os.Stat(name); err == nil && info.Mode().Perm() & 0100 == 0 {
-							continue
-						}
-						completions = append(completions, f)
 					}
-					return completions
+					completions = append(completions, f)
 				}
+				compGroup[0].Suggestions = completions
+				return "", compGroup
 			}
 			
 			// filter out executables, but in path
@@ -73,12 +82,16 @@ func newLineReader(prompt string) *lineReader {
 					}
 				}
 			}
+			
 			// add lua registered commands to completions
 			for cmdName := range commands {
 				if strings.HasPrefix(cmdName, query) {
 					completions = append(completions, cmdName)
 				}
 			}
+			
+			compGroup[0].Suggestions = completions
+			return query, compGroup
 		} else {
 			if completecb, ok := luaCompletions["command." + fields[0]]; ok {
 				err := l.CallByParam(lua.P{
@@ -88,7 +101,7 @@ func newLineReader(prompt string) *lineReader {
 				})
 
 				if err != nil {
-					return []string{}
+					return "", compGroup
 				}
 
 				luacompleteTable := l.Get(-1)
@@ -123,7 +136,7 @@ func newLineReader(prompt string) *lineReader {
 											val := value.String()
 											if val == "<file>" {
 												// complete files
-												completions = append(completions, readline.FilenameCompleter(query, ctx)...)
+												completions = append(completions, fileComplete(query, ctx, fields)...)
 											} else {
 												if strings.HasPrefix(val, query) {
 													completions = append(completions, val)
@@ -171,39 +184,52 @@ func newLineReader(prompt string) *lineReader {
 			}
 
 			if len(completions) == 0 {
-				completions = readline.FilenameCompleter(query, ctx)
+				completions = fileComplete(query, ctx, fields)
 			}
 		}
-		return completions
+
+		compGroup[0].Suggestions = completions
+		return "", compGroup
 	}
-	readline.LoadHistory(defaultHistPath)
 
 	return &lineReader{
-		Prompt: prompt,
+		rl,
 	}
 }
 
 func (lr *lineReader) Read() (string, error) {
-	hooks.Em.Emit("command.precmd", nil)
+	s, err := lr.rl.Readline()
-	return readline.String(lr.Prompt)
+	// this is so dumb
+	if err == readline.EOF {
+		return "", io.EOF
+	}
+
+	return s, err // might get another error
 }
 
 func (lr *lineReader) SetPrompt(prompt string) {
-	lr.Prompt = prompt
+	halfPrompt := strings.Split(prompt, "\n")
+	if len(halfPrompt) > 1 {
+		lr.rl.SetPrompt(strings.Join(halfPrompt[:len(halfPrompt) - 1], "\n"))
+		lr.rl.MultilinePrompt = halfPrompt[len(halfPrompt) - 1:][0]
+	} else {
+		// print cursor up ansi code
+		fmt.Printf("\033[1A")
+		lr.rl.SetPrompt("")
+		lr.rl.MultilinePrompt = halfPrompt[len(halfPrompt) - 1:][0]
+	}
 }
 
 func (lr *lineReader) AddHistory(cmd string) {
-	readline.AddHistory(cmd)
+	return
-	readline.SaveHistory(defaultHistPath)
 }
 
 func (lr *lineReader) ClearInput() {
-	readline.ReplaceLine("", 0)
+	return
-	readline.RefreshLine()
 }
 
 func (lr *lineReader) Resize() {
-	readline.Resize()
+	return
 }
 
 // lua module
@@ -216,7 +242,7 @@ func (lr *lineReader) Loader(L *lua.LState) *lua.LTable {
 		"size": lr.luaSize,
 	}
 
-	mod := L.SetFuncs(L.NewTable(), lrLua)
+	mod := l.SetFuncs(l.NewTable(), lrLua)
 
 	return mod
 }
@@ -229,36 +255,17 @@ func (lr *lineReader) luaAddHistory(l *lua.LState) int {
 }
 
 func (lr *lineReader) luaSize(l *lua.LState) int {
-	l.Push(lua.LNumber(readline.HistorySize()))
+	return 0
-
-	return 1
 }
 
 func (lr *lineReader) luaGetHistory(l *lua.LState) int {
-	idx := l.CheckInt(1)
+	return 0
-	cmd := readline.GetHistory(idx)
-	l.Push(lua.LString(cmd))
-
-	return 1
 }
 
 func (lr *lineReader) luaAllHistory(l *lua.LState) int {
-	tbl := l.NewTable()
+	return 0
-	size := readline.HistorySize()
-
-	for i := 0; i < size; i++ {
-		cmd := readline.GetHistory(i)
-		tbl.Append(lua.LString(cmd))
-	}
-
-	l.Push(tbl)
-
-	return 1
 }
 
 func (lr *lineReader) luaClearHistory(l *lua.LState) int {
-	readline.ClearHistory()
-	readline.SaveHistory(defaultHistPath)
-
 	return 0
 }

rl_gnu.go

@@ -1,66 +1,46 @@
-// +build goreadline,!hilbiline
+//go:build gnurl
+// +build gnurl
 
 package main
 
 // Here we define a generic interface for readline and hilbiline,
 // making them interchangable during build time
-// this is hilbiline's, as is obvious by the filename
+// this is normal readline
 
 import (
-	"fmt"
+	"os"
-	"io"
 	"path/filepath"
 	"strings"
-	"os"
 
-	"github.com/maxlandon/readline"
+	"github.com/Rosettea/readline"
 	"github.com/yuin/gopher-lua"
 )
 
 type lineReader struct {
-	rl *readline.Instance
+	Prompt string
 }
 
-// other gophers might hate this naming but this is local, shut up
 func newLineReader(prompt string) *lineReader {
-	rl := readline.NewInstance()
+	readline.Init()
-	rl.Multiline = true
+
-	rl.TabCompleter = func(line []rune, pos int, _ readline.DelayedTabContext) (string, []*readline.CompletionGroup) {
+	readline.Completer = func(query string, ctx string) []string {
-		ctx := string(line)
 		var completions []string
-		
+		// trim whitespace from ctx
-		compGroup := []*readline.CompletionGroup{
-			&readline.CompletionGroup{
-				TrimSlash: false,
-				NoSpace: true,
-			},
-		}
-
 		ctx = strings.TrimLeft(ctx, " ")
 		if len(ctx) == 0 {
-			return "", compGroup
+			return []string{}
 		}
-
 		fields := strings.Split(ctx, " ")
 		if len(fields) == 0 {
-			return "", compGroup
+			return []string{}
 		}
-		query := fields[len(fields) - 1]
 
 		ctx = aliases.Resolve(ctx)
 
 		if len(fields) == 1 {
 			fileCompletions := fileComplete(query, ctx, fields)
 			if len(fileCompletions) != 0 {
-				for _, f := range fileCompletions {
+				return fileCompletions
-					name := strings.Replace(query + f, "~", curuser.HomeDir, 1)
-					if info, err := os.Stat(name); err == nil && info.Mode().Perm() & 0100 == 0 {
-						continue
-					}
-					completions = append(completions, f)
-				}
-				compGroup[0].Suggestions = completions
-				return "", compGroup
 			}
 
 			// filter out executables, but in path
@@ -82,16 +62,12 @@ func newLineReader(prompt string) *lineReader {
 					}
 				}
 			}
-			
 			// add lua registered commands to completions
 			for cmdName := range commands {
 				if strings.HasPrefix(cmdName, query) {
 					completions = append(completions, cmdName)
 				}
 			}
-			
-			compGroup[0].Suggestions = completions
-			return query, compGroup
 		} else {
 			if completecb, ok := luaCompletions["command." + fields[0]]; ok {
 				err := l.CallByParam(lua.P{
@@ -101,7 +77,7 @@ func newLineReader(prompt string) *lineReader {
 				})
 
 				if err != nil {
-					return "", compGroup
+					return []string{}
 				}
 
 				luacompleteTable := l.Get(-1)
@@ -136,7 +112,7 @@ func newLineReader(prompt string) *lineReader {
 											val := value.String()
 											if val == "<file>" {
 												// complete files
-												completions = append(completions, fileComplete(query, ctx, fields)...)
+												completions = append(completions, readline.FilenameCompleter(query, ctx)...)
 											} else {
 												if strings.HasPrefix(val, query) {
 													completions = append(completions, val)
@@ -184,52 +160,39 @@ func newLineReader(prompt string) *lineReader {
 			}
 
 			if len(completions) == 0 {
-				completions = fileComplete(query, ctx, fields)
+				completions = readline.FilenameCompleter(query, ctx)
 			}
 		}
-
+		return completions
-		compGroup[0].Suggestions = completions
-		return "", compGroup
 	}
+	readline.LoadHistory(defaultHistPath)
 
 	return &lineReader{
-		rl,
+		Prompt: prompt,
 	}
 }
 
 func (lr *lineReader) Read() (string, error) {
-	s, err := lr.rl.Readline()
+	hooks.Em.Emit("command.precmd", nil)
-	// this is so dumb
+	return readline.String(lr.Prompt)
-	if err == readline.EOF {
-		return "", io.EOF
-	}
-
-	return s, err // might get another error
 }
 
 func (lr *lineReader) SetPrompt(prompt string) {
-	halfPrompt := strings.Split(prompt, "\n")
+	lr.Prompt = prompt
-	if len(halfPrompt) > 1 {
-		lr.rl.SetPrompt(strings.Join(halfPrompt[:len(halfPrompt) - 1], "\n"))
-		lr.rl.MultilinePrompt = halfPrompt[len(halfPrompt) - 1:][0]
-	} else {
-		// print cursor up ansi code
-		fmt.Printf("\033[1A")
-		lr.rl.SetPrompt("")
-		lr.rl.MultilinePrompt = halfPrompt[len(halfPrompt) - 1:][0]
-	}
 }
 
 func (lr *lineReader) AddHistory(cmd string) {
-	return
+	readline.AddHistory(cmd)
+	readline.SaveHistory(defaultHistPath)
 }
 
 func (lr *lineReader) ClearInput() {
-	return
+	readline.ReplaceLine("", 0)
+	readline.RefreshLine()
 }
 
 func (lr *lineReader) Resize() {
-	return
+	readline.Resize()
 }
 
 // lua module
@@ -242,7 +205,7 @@ func (lr *lineReader) Loader(L *lua.LState) *lua.LTable {
 		"size": lr.luaSize,
 	}
 
-	mod := l.SetFuncs(l.NewTable(), lrLua)
+	mod := L.SetFuncs(L.NewTable(), lrLua)
 
 	return mod
 }
@@ -255,17 +218,36 @@ func (lr *lineReader) luaAddHistory(l *lua.LState) int {
 }
 
 func (lr *lineReader) luaSize(l *lua.LState) int {
-	return 0
+	l.Push(lua.LNumber(readline.HistorySize()))
+
+	return 1
 }
 
 func (lr *lineReader) luaGetHistory(l *lua.LState) int {
-	return 0
+	idx := l.CheckInt(1)
+	cmd := readline.GetHistory(idx)
+	l.Push(lua.LString(cmd))
+
+	return 1
 }
 
 func (lr *lineReader) luaAllHistory(l *lua.LState) int {
-	return 0
+	tbl := l.NewTable()
+	size := readline.HistorySize()
+
+	for i := 0; i < size; i++ {
+		cmd := readline.GetHistory(i)
+		tbl.Append(lua.LString(cmd))
+	}
+
+	l.Push(tbl)
+
+	return 1
 }
 
 func (lr *lineReader) luaClearHistory(l *lua.LState) int {
+	readline.ClearHistory()
+	readline.SaveHistory(defaultHistPath)
+
 	return 0
 }

rl_hilbiline.go

@@ -1,83 +0,0 @@
-// +build hilbiline,!goreadline
-
-package main
-
-// Here we define a generic interface for readline and hilbiline,
-// making them interchangable during build time
-// this is hilbiline's, as is obvious by the filename
-
-import (
-	"github.com/Rosettea/Hilbiline"
-	"github.com/yuin/gopher-lua"
-)
-
-type lineReader struct {
-	hl *hilbiline.HilbilineState
-}
-
-// other gophers might hate this naming but this is local, shut up
-func newLineReader(prompt string) *lineReader {
-	hl := hilbiline.New(prompt)
-
-	return &lineReader{
-		&hl,
-	}
-}
-
-func (lr *lineReader) Read() (string, error) {
-	return lr.hl.Read()
-}
-
-func (lr *lineReader) SetPrompt(prompt string) {
-	lr.hl.SetPrompt(prompt)
-}
-
-func (lr *lineReader) AddHistory(cmd string) {
-	return
-}
-
-func (lr *lineReader) ClearInput() {
-	return
-}
-
-func (lr *lineReader) Resize() {
-	return
-}
-
-// lua module
-func (lr *lineReader) Loader(L *lua.LState) *lua.LTable {
-	lrLua := map[string]lua.LGFunction{
-		"add": lr.luaAddHistory,
-		"all": lr.luaAllHistory,
-		"clear": lr.luaClearHistory,
-		"get": lr.luaGetHistory,
-		"size": lr.luaSize,
-	}
-
-	mod := l.SetFuncs(l.NewTable(), lrLua)
-
-	return mod
-}
-
-func (lr *lineReader) luaAddHistory(l *lua.LState) int {
-	cmd := l.CheckString(1)
-	lr.AddHistory(cmd)
-
-	return 0
-}
-
-func (lr *lineReader) luaSize(l *lua.LState) int {
-	return 0
-}
-
-func (lr *lineReader) luaGetHistory(l *lua.LState) int {
-	return 0
-}
-
-func (lr *lineReader) luaAllHistory(l *lua.LState) int {
-	return 0
-}
-
-func (lr *lineReader) luaClearHistory(l *lua.LState) int {
-	return 0
-}