tidy: Use cmp.Or in a few places

cmd/languageserver.go

@@ -1,6 +1,7 @@
 package cmd
 
 import (
+	"cmp"
 	"context"
 	"fmt"
 	"os"
@@ -35,12 +36,12 @@ func init() {
 				if err != nil {
 					fmt.Fprintln(os.Stderr, "error getting executable path:", err)
 				} else {
-					v := version.Version
-					if v == "" {
-						v = "Unknown"
-					}
-
-					fmt.Fprintf(os.Stderr, "Regal Language Server (path: %s, version: %s)", absPath, v)
+					fmt.Fprintf(
+						os.Stderr,
+						"Regal Language Server (path: %s, version: %s)",
+						absPath,
+						cmp.Or(version.Version, "Unknown"),
+					)
 				}
 			}
 

internal/lsp/connection.go

@@ -22,6 +22,7 @@
 package lsp
 
 import (
+	"cmp"
 	"context"
 	"fmt"
 	"io"
@@ -181,14 +182,10 @@ func buildSendHandler(
 			}
 
 		case resp != nil:
-			method := getMethod(resp.ID)
+			method := cmp.Or(getMethod(resp.ID), "(no previous request)")
 
 			deleteMethod(resp.ID)
 
-			if method == "" {
-				method = "(no previous request)"
-			}
-
 			if !cfg.ShouldLog(method) {
 				return
 			}

internal/lsp/server.go

@@ -2,6 +2,7 @@
 package lsp
 
 import (
+	"cmp"
 	"context"
 	"errors"
 	"fmt"
@@ -1220,9 +1221,7 @@ func (l *LanguageServer) templateContentsForFile(fileURI string) (string, error)
 	}
 
 	// if we are in the root, then we can use main as a default
-	if pkg == "" {
-		pkg = "main"
-	}
+	pkg = cmp.Or(pkg, "main")
 
 	if strings.HasSuffix(fileURI, "_test.rego") {
 		pkg += "_test"

pkg/fixer/fixes/directorypackagemismatch.go

@@ -1,6 +1,7 @@
 package fixes
 
 import (
+	"cmp"
 	"fmt"
 	"path/filepath"
 	"regexp"
@@ -32,12 +33,7 @@ func (d *DirectoryPackageMismatch) Fix(fc *FixCandidate, opts *RuntimeOptions) (
 		return nil, err
 	}
 
-	rootPath := opts.BaseDir
-	if rootPath == "" {
-		rootPath = filepath.Dir(fc.Filename)
-	}
-
-	rootPath = filepath.Clean(rootPath)
+	rootPath := filepath.Clean(cmp.Or(opts.BaseDir, filepath.Dir(fc.Filename)))
 
 	newPath := filepath.Join(rootPath, pkgPath, filepath.Base(fc.Filename))