Improve error presentation

Fixes #34

Author: sindresorhus

cli.js

@@ -2,6 +2,7 @@
 import process from 'node:process';
 import meow from 'meow';
 import {deleteAsync} from 'del';
+import {isPresentableError} from 'presentable-error';
 
 const logEvent = event => {
 	if (event.path !== undefined) {
@@ -45,15 +46,25 @@ if (cli.input.length === 0) {
 	console.error('Specify at least one path');
 	process.exitCode = 1;
 } else {
-	const {verbose, dryRun, ...flags} = cli.flags;
+	try {
+		const {verbose, dryRun, ...flags} = cli.flags;
 
-	// Only use onProgress for verbose mode when not in dry-run
-	// In dry-run mode, we print the files at the end instead
-	const onProgress = verbose && !dryRun ? logEvent : noop;
+		// Only use onProgress for verbose mode when not in dry-run
+		// In dry-run mode, we print the files at the end instead
+		const onProgress = verbose && !dryRun ? logEvent : noop;
 
-	const files = await deleteAsync(cli.input, {onProgress, dryRun, ...flags});
+		const files = await deleteAsync(cli.input, {onProgress, dryRun, ...flags});
 
-	if (dryRun && files.length > 0) {
-		console.log(files.join('\n'));
+		if (dryRun && files.length > 0) {
+			console.log(files.join('\n'));
+		}
+	} catch (error) {
+		if (isPresentableError(error)) {
+			console.error(error.message);
+		} else {
+			throw error;
+		}
+
+		process.exitCode = 1;
 	}
 }

package.json

@@ -51,8 +51,9 @@
 		"cross-platform"
 	],
 	"dependencies": {
-		"del": "^8.0.0",
-		"meow": "^13.2.0"
+		"del": "^8.0.1",
+		"meow": "^13.2.0",
+		"presentable-error": "^0.0.1"
 	},
 	"devDependencies": {
 		"ava": "^6.1.3",

test.js

@@ -50,3 +50,16 @@ test('verbose + dry-run with multiple files', async t => {
 	t.true(lines.includes(file1));
 	t.true(lines.includes(file2));
 });
+
+test('handles errors gracefully', async t => {
+	// Test with an invalid operation that should throw an error
+	const error = await t.throwsAsync(
+		execa('./cli.js', ['--force', '/']),
+		{instanceOf: Error},
+	);
+
+	// Should exit with code 1
+	t.is(error.exitCode, 1);
+	// Should not show stack trace for system errors (they're not presentable)
+	t.true(error.stderr.includes('EISDIR'));
+});