Use rimraf directly. Add no-write option. Fixes GH-25.

README.md

@@ -1,4 +1,4 @@
-# grunt-contrib-clean [![Build Status](https://secure.travis-ci.org/gruntjs/grunt-contrib-clean.png?branch=master)](http://travis-ci.org/gruntjs/grunt-contrib-clean)
+# grunt-contrib-clean [![Build Status](https://travis-ci.org/gruntjs/grunt-contrib-clean.png?branch=master)](https://travis-ci.org/gruntjs/grunt-contrib-clean)
 
 > Clean files and folders.
 
@@ -35,7 +35,13 @@ Task targets, files and options may be specified according to the grunt [Configu
 Type: `Boolean`  
 Default: false
 
-This overrides `grunt.file.delete` from blocking deletion of folders outside current working dir (CWD). Use with caution.
+This overrides this task from blocking deletion of folders outside current working dir (CWD). Use with caution.
+
+#### no-write
+Type: `Boolean`  
+Default: false
+
+Will log messages of what would happen if the task was ran but doesn't actually delete the files.
 
 ### Usage Examples
 
@@ -80,4 +86,4 @@ clean: {
 
 Task submitted by [Tim Branyen](http://tbranyen.com/)
 
-*This file was generated on Tue Apr 16 2013 13:28:15.*
+*This file was generated on Mon Jul 15 2013 20:33:46.*

docs/clean-options.md

@@ -4,4 +4,10 @@
 Type: `Boolean`  
 Default: false
 
-This overrides `grunt.file.delete` from blocking deletion of folders outside current working dir (CWD). Use with caution.
+This overrides this task from blocking deletion of folders outside current working dir (CWD). Use with caution.
+
+## no-write
+Type: `Boolean`  
+Default: false
+
+Will log messages of what would happen if the task was ran but doesn't actually delete the files.

package.json

@@ -27,6 +27,9 @@
   "scripts": {
     "test": "grunt test"
   },
+  "dependencies": {
+    "rimraf": "~2.2.1"
+  },
   "devDependencies": {
     "grunt-contrib-jshint": "~0.2.0",
     "grunt-contrib-nodeunit": "~0.1.2",

tasks/clean.js

@@ -8,29 +8,54 @@
 
 'use strict';
 
+var rimraf = require('rimraf');
+
 module.exports = function(grunt) {
 
+  function clean(filepath, options) {
+    if (!grunt.file.exists(filepath)) {
+      return false;
+    }
+
+    grunt.log.write((options['no-write'] ? 'Not actually cleaning ' : 'Cleaning ') + filepath + '...');
+
+    // Only delete cwd or outside cwd if --force enabled. Be careful, people!
+    if (!options.force) {
+      if (grunt.file.isPathCwd(filepath)) {
+        grunt.verbose.error();
+        grunt.fail.warn('Cannot delete the current working directory.');
+        return false;
+      } else if (!grunt.file.isPathInCwd(filepath)) {
+        grunt.verbose.error();
+        grunt.fail.warn('Cannot delete files outside the current working directory.');
+        return false;
+      }
+    }
+
+    try {
+      // Actually delete. Or not.
+      if (!options['no-write']) {
+        rimraf.sync(filepath);
+      }
+      grunt.log.ok();
+    } catch (e) {
+      grunt.log.error();
+      grunt.fail.warn('Unable to delete "' + filepath + '" file (' + e.message + ').', e);
+    }
+  }
+
   grunt.registerMultiTask('clean', 'Clean files and folders.', function() {
     // Merge task-specific and/or target-specific options with these defaults.
     var options = this.options({
-      force: false
+      force: grunt.option('force') === true,
+      'no-write': grunt.option('no-write') === true,
     });
 
     grunt.verbose.writeflags(options, 'Options');
 
     // Clean specified files / dirs.
     this.filesSrc.forEach(function(filepath) {
-      if (!grunt.file.exists(filepath)) { return; }
-      grunt.log.write('Cleaning "' + filepath + '"...');
-
-      try {
-        grunt.file.delete(filepath, options);
-        grunt.log.ok();
-      } catch (e) {
-        grunt.log.error();
-        grunt.verbose.error(e);
-        grunt.fail.warn('Clean operation failed.');
-      }
+      clean(filepath, options);
     });
   });