diff --git a/internal/agent/tools/view.go b/internal/agent/tools/view.go
index 7b73e2e1fb0f95b3d729bc3f5da72a99d362f520..1b9e60546d10f5f287c8f6284c334843260f2051 100644
--- a/internal/agent/tools/view.go
+++ b/internal/agent/tools/view.go
@@ -98,7 +98,7 @@ func NewViewTool(
 
 			// Request permission for files outside working directory, unless it's a skill file.
 			if isOutsideWorkDir && !isSkillFile {
-				granted, err := permissions.Request(ctx,
+				granted, permReqErr := permissions.Request(ctx,
 					permission.CreatePermissionRequest{
 						SessionID:   sessionID,
 						Path:        absFilePath,
@@ -109,8 +109,8 @@ func NewViewTool(
 						Params:      ViewPermissionsParams(params),
 					},
 				)
-				if err != nil {
-					return fantasy.ToolResponse{}, err
+				if permReqErr != nil {
+					return fantasy.ToolResponse{}, permReqErr
 				}
 				if !granted {
 					return fantasy.ToolResponse{}, permission.ErrorPermissionDenied
@@ -176,9 +176,9 @@ func NewViewTool(
 					return fantasy.NewTextErrorResponse(fmt.Sprintf("This model (%s) does not support image data.", modelName)), nil
 				}
 
-				imageData, err := os.ReadFile(filePath)
-				if err != nil {
-					return fantasy.ToolResponse{}, fmt.Errorf("error reading image file: %w", err)
+				imageData, readErr := os.ReadFile(filePath)
+				if readErr != nil {
+					return fantasy.ToolResponse{}, fmt.Errorf("error reading image file: %w", readErr)
 				}
 
 				encoded := base64.StdEncoding.EncodeToString(imageData)