refactor: Add skip confirmation option

Author: d3xter666

packages/cli/lib/cli/commands/cache.js

@@ -20,15 +20,21 @@ cacheCommand.builder = function(cli) {
 		.demandCommand(1, "Command required. Available command is 'clean'")
 		.command("clean", "Remove all cached UI5 data", {
 			handler: handleCache,
-			builder: noop,
+			builder: function(yargs) {
+				return yargs.option("interactive", {
+					describe: "Show confirmation prompt before cleaning. Use --no-interactive to skip (e.g. for CI)",
+					default: true,
+					type: "boolean",
+				});
+			},
 			middlewares: [baseMiddleware],
 		})
 		.example("$0 cache clean",
-			"Remove all cached UI5 data");
+			"Remove all cached UI5 data")
+		.example("$0 cache clean --no-interactive",
+			"Remove all cached UI5 data without confirmation (CI mode)");
 };
 
-function noop() {}
-
 /**
  * Format a byte size as a human-readable string.
  *
@@ -66,7 +72,9 @@ async function confirm(question) {
 	});
 }
 
-async function handleCache() {
+async function handleCache(argv) {
+	const interactive = argv?.interactive !== false;
+
 	// Resolve UI5 data directory
 	let ui5DataDir = process.env.UI5_DATA_DIR;
 	if (!ui5DataDir) {
@@ -105,11 +113,13 @@ async function handleCache() {
 	}
 	process.stderr.write(chalk.bold(`\nTotal: ${formatSize(totalSize)}\n\n`));
 
-	// Ask for confirmation
-	const confirmed = await confirm("Do you want to continue? (y/N) ");
-	if (!confirmed) {
-		process.stderr.write("Cancelled\n");
-		return;
+	// Ask for confirmation (skip in non-interactive mode)
+	if (interactive) {
+		const confirmed = await confirm("Do you want to continue? (y/N) ");
+		if (!confirmed) {
+			process.stderr.write("Cancelled\n");
+			return;
+		}
 	}
 
 	// Perform the actual cleanup (orchestrate both domains)

packages/cli/test/lib/cli/commands/cache.js

@@ -71,7 +71,7 @@ test("Command builder", async (t) => {
 	t.is(result, cliStub, "Builder returns cli instance");
 	t.is(cliStub.demandCommand.callCount, 1, "demandCommand called once");
 	t.is(cliStub.command.callCount, 1, "command called once");
-	t.is(cliStub.example.callCount, 1, "example called once");
+	t.is(cliStub.example.callCount, 2, "example called twice");
 });
 
 test.serial("ui5 cache clean: nothing to clean", async (t) => {
@@ -253,3 +253,32 @@ test.serial("ui5 cache clean: uses config.getUi5DataDir when no env var", async
 		}
 	}
 });
+
+test.serial("ui5 cache clean --no-interactive: skips confirmation prompt", async (t) => {
+	const {cache, argv, stderrWriteStub, frameworkCacheCleanCache, frameworkCacheGetCacheInfo,
+		buildCacheCleanCache, buildCacheGetCacheInfo, mockRLInterface} = t.context;
+
+	frameworkCacheGetCacheInfo.resolves({path: "framework/", size: 10 * 1024 * 1024, type: "directory"});
+	buildCacheGetCacheInfo.resolves({
+		path: "buildCache/v0_7 (database records)", size: 5 * 1024 * 1024, type: "database"
+	});
+
+	frameworkCacheCleanCache.resolves({path: "framework", type: "framework", size: 10 * 1024 * 1024});
+	buildCacheCleanCache.resolves({path: "buildCache/v0_7", type: "buildCache", size: 5 * 1024 * 1024});
+
+	argv["_"] = ["cache", "clean"];
+	argv["interactive"] = false;
+	await cache.handler(argv);
+
+	// Confirmation should NOT be asked
+	t.is(mockRLInterface.question.callCount, 0, "Should not ask for confirmation in non-interactive mode");
+
+	// Cleanup should still proceed
+	t.is(frameworkCacheCleanCache.callCount, 1, "frameworkCache.cleanCache should be called");
+	t.is(buildCacheCleanCache.callCount, 1, "buildCache.cleanCache should be called");
+
+	// Check output
+	const allOutput = stderrWriteStub.args.map((a) => a[0]).join("");
+	t.true(allOutput.includes("following items from cache will be removed"), "Shows items to be removed");
+	t.true(allOutput.includes("Success"), "Shows success message");
+});