FixUp: address golangci-lint

Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>

Author: ArangoGutierrez

cmd/gpu-mockctl/internal/errors/errors.go

@@ -69,9 +69,7 @@ func (e *Error) Error() string {
 		parts = append(parts, e.Err.Error())
 	}
 
-	for _, ctx := range e.Context {
-		parts = append(parts, ctx)
-	}
+	parts = append(parts, e.Context...)
 
 	return strings.Join(parts, ": ")
 }

cmd/gpu-mockctl/internal/logger/logger.go

@@ -301,7 +301,10 @@ func (l *logger) logText(level Level, msg string) {
 		}
 	}
 
-	fmt.Fprintln(l.config.Output, output)
+	if _, err := fmt.Fprintln(l.config.Output, output); err != nil {
+		// If we can't write to the output, try to write to stderr as a fallback
+		fmt.Fprintf(os.Stderr, "logger: failed to write log: %v\n", err)
+	}
 }
 
 // logJSON outputs structured JSON

pkg/gpu/mockdriver/nvml.go

@@ -85,7 +85,11 @@ func copyFile(src, dst string) error {
 	if err != nil {
 		return fmt.Errorf("failed to open source file: %w", err)
 	}
-	defer srcFile.Close()
+	defer func() {
+		if cerr := srcFile.Close(); cerr != nil {
+			err = fmt.Errorf("failed to close source file: %w", cerr)
+		}
+	}()
 
 	// Get source file info for permissions
 	srcInfo, err := srcFile.Stat()
@@ -98,7 +102,11 @@ func copyFile(src, dst string) error {
 	if err != nil {
 		return fmt.Errorf("failed to create destination file: %w", err)
 	}
-	defer dstFile.Close()
+	defer func() {
+		if cerr := dstFile.Close(); cerr != nil && err == nil {
+			err = fmt.Errorf("failed to close destination file: %w", cerr)
+		}
+	}()
 
 	// Copy content
 	if _, err := io.Copy(dstFile, srcFile); err != nil {
@@ -126,4 +134,3 @@ func DefaultFilesWithNVML(root string) []FileSpec {
 
 	return filtered
 }
-