CHANGELOG.md

@@ -2,8 +2,6 @@
 
 ## Unreleased
 **NOTE:** Hilbish now uses [Task] insead of Make for builds.
-Windows support is also now at a lower tier; The only thing guaranteed is
-Hilbish *compiling* on Windows.
 
 [Task]: https://taskfile.dev/#/
 

README.md

@@ -31,6 +31,7 @@ and aims to be infinitely configurable. If something isn't, open an issue!
   - [AUR](#AUR)
   - [Nixpkgs](#Nixpkgs)
   - [Manual Build](#Manual-Build)
+- [Getting Started](#Getting-Started)
 - [Contributing](#Contributing)
 
 # Screenshots
@@ -41,10 +42,6 @@ and aims to be infinitely configurable. If something isn't, open an issue!
 </div>
 
 # Installation
-**NOTE:** Hilbish is not guaranteed to work properly on Windows, starting
-from the 2.0 version. It will still be able to compile, but functionality
-may be lacking.
-
 ## Prebuilt binaries
 Go [here](https://nightly.link/Rosettea/Hilbish/workflows/build/master) for
 builds on the master branch.

nature/opts/history.lua

@@ -1,6 +1,5 @@
 local bait = require 'bait'
 
 bait.catch('command.exit', function(_, cmd, priv)
-	if not cmd then return end
 	if not priv and hilbish.opts.history then hilbish.history.add(cmd) end
 end)

readline/update.go

@@ -1,10 +1,6 @@
 package readline
 
-import (
-	"strings"
-
-	"golang.org/x/text/width"
-)
+import "golang.org/x/text/width"
 
 // updateHelpers is a key part of the whole refresh process:
 // it should coordinate reprinting the input line, any Infos and completions
@@ -56,19 +52,19 @@ func (rl *Instance) updateReferences() {
 	rl.posY = 0
 	rl.fullY = 0
 
-	var curLine []rune
+	var fullLine, cPosLine int
 	if len(rl.currentComp) > 0 {
-		curLine = rl.lineComp
+		fullLine = getWidth(rl.lineComp)
+		cPosLine = getWidth(rl.lineComp[:rl.pos])
 	} else {
-		curLine = rl.line
+		fullLine = getWidth(rl.line)
+		cPosLine = getWidth(rl.line[:rl.pos])
 	}
-	fullLine := getWidth(curLine)
-	cPosLine := getWidth(curLine[:rl.pos])
 
 	// We need the X offset of the whole line
 	toEndLine := rl.promptLen + fullLine
 	fullOffset := toEndLine / GetTermWidth()
-	rl.fullY = fullOffset + strings.Count(string(curLine), "\n")
+	rl.fullY = fullOffset
 	fullRest := toEndLine % GetTermWidth()
 	rl.fullX = fullRest
 

readline/vim.go

@@ -245,7 +245,7 @@ func (rl *Instance) vi(r rune) {
 		}
 
 		// Keep the previous cursor position
-		//prev := rl.pos
+		prev := rl.pos
 
 		new, err := rl.StartEditorWithBuffer(multiline, "")
 		if err != nil || len(new) == 0 || string(new) == string(multiline) {
@@ -257,11 +257,11 @@ func (rl *Instance) vi(r rune) {
 		// Clean the shell and put the new buffer, with adjusted pos if needed.
 		rl.clearLine()
 		rl.line = new
-		rl.pos = len(rl.line)
-		/*if prev > len(rl.line) {
+		if prev > len(rl.line) {
+			rl.pos = len(rl.line) - 1
 		} else {
 			rl.pos = prev
-		}*/
+		}
 
 	case 'w':
 		// If we were not yanking