fix: resolve Windows updater file lock and release v1.1.6

Author: programmersd21

CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.6]
+
+### Fixed
+- **Windows Updater**: Resolved a critical issue where the binary update would fail due to file locking. The process now ensures a clean handover between the main application and the update helper.
+
 ## [1.1.5]
 
 ### Added

README.md

@@ -106,7 +106,7 @@ kairo update
 ```
 
 Downloads the latest GitHub Release for your OS/arch, verifies it against `checksums.txt`, and safely replaces the installed binary.
-On Windows, the update is applied after `kairo update` exits; run `kairo` again once it completes.
+On Windows, Kairo will automatically close to apply the update; simply re-run `kairo` once the terminal returns.
 
 ---
 

VERSION.txt

@@ -1 +1 @@
-1.1.5
+1.1.6

internal/lua/engine.go

@@ -55,7 +55,7 @@ func (e *Engine) SetupKairoAPI(L *lua.LState) {
 	L.SetField(kairo, "notify", L.NewFunction(e.luaNotify))
 
 	// Meta
-	L.SetField(kairo, "version", lua.LString("1.1.5"))
+	L.SetField(kairo, "version", lua.LString("1.1.6"))
 
 	// Set as global
 	L.SetGlobal("kairo", kairo)

internal/updater/updater.go

@@ -239,14 +239,14 @@ func applyWindows(target, backup, newBinPath string, stdout, stderr io.Writer) e
 	}
 
 	cmd := exec.Command(helper, args...)
-	cmd.Stdout = stdout
-	cmd.Stderr = stderr
-
+	// On Windows, we need to detach or at least not wait for it.
+	// We'll use os.StartProcess style via cmd.Start() but then we MUST exit.
 	if err := cmd.Start(); err != nil {
 		return err
 	}
 
 	_, _ = fmt.Fprintln(stdout, "Update staged. Closing to finish update...")
+	os.Exit(0)
 	return nil
 }
 

internal/updater/windows_helper.go

@@ -37,14 +37,19 @@ func MaybeRunWindowsApply(stdout, stderr io.Writer) (handled bool, err error) {
 		*backup = *target + ".old"
 	}
 
-	_, _ = fmt.Fprintln(stdout, "Finishing update...")
-	if err := applyWithRetry(*target, *backup, *source, 45*time.Second); err != nil {
-		return true, err
+	// Give parent process time to exit
+	time.Sleep(1 * time.Second)
+
+	if err := applyWithRetry(*target, *backup, *source, 30*time.Second); err != nil {
+		// Since we're in a detached-like process, we might want to log to a file
+		// but for now, we'll try to use stderr if it's still connected.
+		return true, fmt.Errorf("failed to apply update: %w", err)
 	}
 
 	_ = os.Remove(*source)
+	// We don't necessarily need to delete ourselves if we're in a temp dir,
+	// but it's cleaner.
 	scheduleSelfDelete()
-	_, _ = fmt.Fprintln(stdout, "✓ Update applied. Re-run `kairo`.")
 	return true, nil
 }