chore: fix lint errors

Author: puria

env_test.go

@@ -23,8 +23,14 @@ func TestEnvironmentVariables(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			// Set environment variable
-			os.Setenv(tt.envKey, tt.envValue)
-			defer os.Unsetenv(tt.envKey)
+			if err := os.Setenv(tt.envKey, tt.envValue); err != nil {
+				t.Fatalf("Failed to set env var: %v", err)
+			}
+			defer func() {
+				if err := os.Unsetenv(tt.envKey); err != nil {
+					t.Errorf("Failed to unset env var: %v", err)
+				}
+			}()
 
 			// Test string env vars
 			getEnvOrDefault := func(key, defaultValue string) string {
@@ -58,8 +64,14 @@ func TestEnvironmentVariablesInt(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			// Set environment variable
-			os.Setenv(tt.envKey, tt.envValue)
-			defer os.Unsetenv(tt.envKey)
+			if err := os.Setenv(tt.envKey, tt.envValue); err != nil {
+				t.Fatalf("Failed to set env var: %v", err)
+			}
+			defer func() {
+				if err := os.Unsetenv(tt.envKey); err != nil {
+					t.Errorf("Failed to unset env var: %v", err)
+				}
+			}()
 
 			// Test int env vars
 			getEnvOrDefaultInt := func(key string, defaultValue int) int {
@@ -95,8 +107,14 @@ func TestEnvironmentVariablesInt64(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			// Set environment variable
-			os.Setenv(tt.envKey, tt.envValue)
-			defer os.Unsetenv(tt.envKey)
+			if err := os.Setenv(tt.envKey, tt.envValue); err != nil {
+				t.Fatalf("Failed to set env var: %v", err)
+			}
+			defer func() {
+				if err := os.Unsetenv(tt.envKey); err != nil {
+					t.Errorf("Failed to unset env var: %v", err)
+				}
+			}()
 
 			// Test int64 env vars
 			getEnvOrDefaultInt64 := func(key string, defaultValue int64) int64 {

main.go

@@ -477,13 +477,14 @@ func (s *server) generateSitemap() error {
 		if !info.IsDir() {
 			entry.fileSize = info.Size()
 			category := getFileCategory(filepath.Base(path))
-			if category == FileCategoryImage {
+			switch category {
+			case FileCategoryImage:
 				entry.isImage = true
 				entry.imageURL = s.baseURL + vp
-			} else if category == FileCategoryVideo {
+			case FileCategoryVideo:
 				entry.isVideo = true
 				entry.videoURL = s.baseURL + vp
-			} else if category == FileCategoryDocument {
+			case FileCategoryDocument:
 				entry.isDocument = true
 			}
 		}
@@ -1089,14 +1090,14 @@ func (s *server) handleStaticFile(w http.ResponseWriter, r *http.Request) {
 func (s *server) handleVendoredMarked(w http.ResponseWriter, r *http.Request) {
 	w.Header().Set("Content-Type", "application/javascript; charset=utf-8")
 	w.Header().Set("Cache-Control", "public, max-age=31536000") // Cache for 1 year
-	w.Write(embeddedMarkedJS)
+	_, _ = w.Write(embeddedMarkedJS)
 }
 
 // handleVendoredDatastar serves the vendored datastar.js library
 func (s *server) handleVendoredDatastar(w http.ResponseWriter, r *http.Request) {
 	w.Header().Set("Content-Type", "application/javascript; charset=utf-8")
 	w.Header().Set("Cache-Control", "public, max-age=31536000") // Cache for 1 year
-	w.Write(embeddedDatastarJS)
+	_, _ = w.Write(embeddedDatastarJS)
 }
 
 // processHTMLTemplate replaces placeholders in HTML with dynamic content
@@ -2843,7 +2844,7 @@ func (s *server) handleComplete(w http.ResponseWriter, r *http.Request) {
 				if req.TextOnly {
 					// Use file category to check if viewable (text or image)
 					cat := getFileCategory(name)
-					if !(cat == FileCategoryText || cat == FileCategoryImage) {
+					if cat != FileCategoryText && cat != FileCategoryImage {
 						continue
 					}
 				}