fix: close store after Run and add .exe for Windows e2e tests

- Close SQLite store after Run() returns to prevent file locking issues
  on Windows (TempDir cleanup failures)
- Add .exe suffix to binary path in e2e tests for Windows

Author: thedavidweng

internal/cli/cli.go

@@ -57,6 +57,11 @@ var runPlaidLoginCLI = runPlaidLoginLive
 
 func Run(ctx context.Context, args []string, stdin io.Reader, stdout io.Writer, stderr io.Writer) int {
 	state := &runtimeState{stdin: stdin, stderr: stderr}
+	defer func() {
+		if closer, ok := state.store.(interface{ Close() error }); ok {
+			_ = closer.Close()
+		}
+	}()
 	root := newRootCommand(ctx, state, stdout, stderr)
 	root.SetArgs(args)
 	root.SetIn(stdin)

tests/e2e/binary_test.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"os"
 	"os/exec"
+	"runtime"
 	"path/filepath"
 	"regexp"
 	"sort"
@@ -35,6 +36,9 @@ func buildBinary(t *testing.T) string {
 			return
 		}
 		cachedBin = filepath.Join(dir, "money")
+		if runtime.GOOS == "windows" {
+			cachedBin += ".exe"
+		}
 		cmd := exec.Command("go", "build", "-o", cachedBin, "./cmd/money")
 		cmd.Dir = projectRoot
 		if out, err := cmd.CombinedOutput(); err != nil {