build(deps): bump github.com/ccoveille/go-safecast to 1.8.1

Author: ccoVeille

go.mod

@@ -17,7 +17,7 @@ require (
 	github.com/authzed/grpcutil v0.0.0-20240123194739-2ea1e3d2d98b
 	github.com/authzed/spicedb v1.45.4
 	github.com/brianvoe/gofakeit/v6 v6.28.0
-	github.com/ccoveille/go-safecast v1.6.1
+	github.com/ccoveille/go-safecast v1.8.1
 	github.com/cenkalti/backoff/v4 v4.3.0
 	github.com/charmbracelet/lipgloss v1.1.0
 	github.com/charmbracelet/x/term v0.2.1

go.sum

@@ -810,6 +810,8 @@ github.com/ccojocar/zxcvbn-go v1.0.4 h1:FWnCIRMXPj43ukfX000kvBZvV6raSxakYr1nzyNr
 github.com/ccojocar/zxcvbn-go v1.0.4/go.mod h1:3GxGX+rHmueTUMvm5ium7irpyjmm7ikxYFOSJB21Das=
 github.com/ccoveille/go-safecast v1.6.1 h1:Nb9WMDR8PqhnKCVs2sCB+OqhohwO5qaXtCviZkIff5Q=
 github.com/ccoveille/go-safecast v1.6.1/go.mod h1:QqwNjxQ7DAqY0C721OIO9InMk9zCwcsO7tnRuHytad8=
+github.com/ccoveille/go-safecast v1.8.1 h1:RoucjfYKKcx2lFmIjRjuo8AeX9k/GaZn5SUMHlA3kMw=
+github.com/ccoveille/go-safecast v1.8.1/go.mod h1:QqwNjxQ7DAqY0C721OIO9InMk9zCwcsO7tnRuHytad8=
 github.com/cenkalti/backoff/v4 v4.3.0 h1:MyRJ/UdXutAwSAT+s3wNd7MfTIcy71VQueUuFK343L8=
 github.com/cenkalti/backoff/v4 v4.3.0/go.mod h1:Y3VNntkOUPxTVeUxJ/G5vcM//AlwfmyYozVcomhLiZE=
 github.com/census-instrumentation/opencensus-proto v0.2.1/go.mod h1:f6KPmirojxKA12rnyqOA5BBL4O983OfeGPqjHWSTneU=

internal/cmd/restorer.go

@@ -279,7 +279,7 @@ func (r *restorer) commitStream(ctx context.Context, bulkImportClient v1.Permiss
 
 	// it was a successful transaction commit without duplicates
 	if resp != nil {
-		numLoaded, err := safecast.ToUint(resp.NumLoaded)
+		numLoaded, err := safecast.Convert[uint](resp.NumLoaded)
 		if err != nil {
 			return spiceerrors.MustBugf("could not cast numLoaded to uint")
 		}
@@ -289,7 +289,7 @@ func (r *restorer) commitStream(ctx context.Context, bulkImportClient v1.Permiss
 		}
 	}
 
-	writtenAndSkipped, err := safecast.ToInt64(r.writtenRels + r.skippedRels)
+	writtenAndSkipped, err := safecast.Convert[int64](r.writtenRels + r.skippedRels)
 	if err != nil {
 		return fmt.Errorf("too many written and skipped rels for an int64")
 	}

internal/cmd/restorer_test.go

@@ -220,7 +220,11 @@ func (m *mockClientForRestore) Send(req *v1.ImportBulkRelationshipsRequest) erro
 
 	for i, rel := range req.Relationships {
 		// This is a gosec115 false positive which should be fixed in a future version.
-		uinti, _ := safecast.ToUint(i)
+		uinti, err := safecast.Convert[uint](i)
+		if err != nil {
+			// just in case to avoid accessing out of bounds in the []string
+			uinti = 0
+		}
 		require.True(m.t, proto.Equal(rel, tuple.MustParseV1Rel(m.expectedRels[((m.receivedBatches-1)*m.requestedBatchSize)+uinti])))
 	}
 

internal/cmd/schema.go

@@ -223,7 +223,7 @@ func schemaCopyCmdFunc(cmd *cobra.Command, args []string) error {
 }
 
 func schemaWriteCmdImpl(cmd *cobra.Command, args []string, client v1.SchemaServiceClient, terminalChecker termChecker) error {
-	stdInFd, err := safecast.ToInt(uint(os.Stdin.Fd()))
+	stdInFd, err := safecast.Convert[int](uint(os.Stdin.Fd()))
 	if err != nil {
 		return err
 	}
@@ -363,7 +363,7 @@ func determinePrefixForSchema(ctx context.Context, specifiedPrefix string, clien
 // Compiles an input schema written in the new composable schema syntax
 // and produces it as a fully-realized schema
 func schemaCompileCmdFunc(cmd *cobra.Command, args []string, termChecker termChecker) error {
-	stdOutFd, err := safecast.ToInt(uint(os.Stdout.Fd()))
+	stdOutFd, err := safecast.Convert[int](uint(os.Stdout.Fd()))
 	if err != nil {
 		return err
 	}

internal/cmd/validate.go

@@ -283,9 +283,16 @@ func outputErrorWithSource(sb *strings.Builder, validateContents []byte, errWith
 
 func outputForLine(sb *strings.Builder, validateContents []byte, oneIndexedLineNumber uint64, sourceCodeString string, oneIndexedColumnPosition uint64) {
 	lines := strings.Split(string(validateContents), "\n")
-	// These should be fine to be zero if the cast fails.
-	intLineNumber, _ := safecast.ToInt(oneIndexedLineNumber)
-	intColumnPosition, _ := safecast.ToInt(oneIndexedColumnPosition)
+	intLineNumber, err := safecast.Convert[int](oneIndexedLineNumber)
+	if err != nil {
+		// It's fine to be zero if the cast fails.
+		intLineNumber = 0
+	}
+	intColumnPosition, _ := safecast.Convert[int](oneIndexedColumnPosition)
+	if err != nil {
+		// It's fine to be zero if the cast fails.
+		intColumnPosition = 0
+	}
 	errorLineNumber := intLineNumber - 1
 	for i := errorLineNumber - 3; i < errorLineNumber+3; i++ {
 		if i == errorLineNumber {