fix(r3): use errors.Is for sentinel error comparisons

Replace direct == checks with errors.Is so wrapped errors are handled
correctly (authkeys Verify, daemon server/EOF, Magefile uninstall).

Refs: ask r3, 100 Go Mistakes #51
Made-with: Cursor

1 files changed, 5 insertions, 4 deletions

diff --git a/internal/daemon/daemon.go b/internal/daemon/daemon.go
index b913c43..cad2812 100644
--- a/internal/daemon/daemon.go
+++ b/internal/daemon/daemon.go
@@ -3,6 +3,7 @@ package daemon
 import (
 	"bytes"
 	"context"
+	"errors"
 	"fmt"
 	"io"
 	"log"
@@ -131,12 +132,12 @@ func Run(ctx context.Context, cfg Config) error {
 		defer cancel()
 		_ = srv.Shutdown(shutCtx)
 		err := <-errCh
-		if err != nil && err != http.ErrServerClosed {
+		if err != nil && !errors.Is(err, http.ErrServerClosed) {
 			return fmt.Errorf("shutdown: %w", err)
 		}
 		return ctx.Err()
 	case err := <-errCh:
-		if err == http.ErrServerClosed {
+		if errors.Is(err, http.ErrServerClosed) {
 			return nil
 		}
 		if err != nil {
@@ -200,7 +201,7 @@ func checkReadinessDirs(statsDir, authDB string) error {
 func checkDirReadableWritable(dir string) error {
 	fi, err := os.Stat(dir)
 	if err != nil {
-		if os.IsNotExist(err) {
+		if errors.Is(err, os.ErrNotExist) {
 			return fmt.Errorf("missing")
 		}
 		return err
@@ -214,7 +215,7 @@ func checkDirReadableWritable(dir string) error {
 	}
 	_, err = f.Readdirnames(1)
 	_ = f.Close()
-	if err != nil && err != io.EOF {
+	if err != nil && !errors.Is(err, io.EOF) {
 		return fmt.Errorf("not readable: %w", err)
 	}
 	tmp, err := os.CreateTemp(dir, ".goprecords-ready-*")