From 89a3abb3e4effb94808a388e6551892c5a7f0f2c Mon Sep 17 00:00:00 2001
From: "devsy-app[bot]" <277138668+devsy-app[bot]@users.noreply.github.com>
Date: Sun, 9 Aug 2026 13:05:47 +0000
Subject: [PATCH] fix: drop redundant else in feature cache check

Drops the trailing else block in checkFeatureCache where the if branch already returns, outdenting the cleanup statements. Resolves Codefactor Maintainability issue 6a77b13d5b1ec0b4ee20c456 in pkg/devcontainer/feature/features.go.
---
 pkg/devcontainer/feature/features.go | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/pkg/devcontainer/feature/features.go b/pkg/devcontainer/feature/features.go
index abbaedef65..6b4c6b2266 100644
--- a/pkg/devcontainer/feature/features.go
+++ b/pkg/devcontainer/feature/features.go
@@ -159,10 +159,9 @@ func checkFeatureCache(id string) (string, bool) {
 		if err == nil {
 			log.Debugf("feature already cached: folder=%s", featureExtractedFolder)
 			return featureExtractedFolder, true
-		} else {
-			log.Debugf("feature folder exists but seems empty: folder=%s", featureExtractedFolder)
-			_ = os.RemoveAll(featureFolder)
 		}
+		log.Debugf("feature folder exists but seems empty: folder=%s", featureExtractedFolder)
+		_ = os.RemoveAll(featureFolder)
 	}
 	return "", false
 }