Exclude pattern hack

I’ve wrote a function to simplify writing exclude pattern, due to the fact that ** matches all children and the parent.

del.sync(extendNegGlobs(['public/assets/**', '!public/assets/goat.png']));
// is equivalent with 
del.sync(['public/assets/**', '!public/assets/goat.png', '!public/assets', '!public']);

Here is my function:

function extendNegGlobs(arr) {
    var ext = [];
    var negs = {};
    arr.forEach((g, i) => {
        if ( g.slice(0,1) == '!' ) {
            negs[g] = i;
        }
    });
    Object.keys(negs).forEach((g) => {
        var d;
        if ( g.slice(-1) == '*' ) do {
            g = path.dirname(d = g);
            if ( g == '!' || g == '.' || g == d ) break;
            if ( !(g in negs) ) {
                negs[g] = ext.push(g);
            }
        }
        while ( true );
    });
    return arr.concat(ext);
}

I don’t pretend this is a complete/nice solution, but it might be a starting point to finding one!

Comments ?

[duzun]

I think a call like del.sync(['public/assets/**', '!public/assets/goat.png']); should remove all files inside public/assets/ except public/assets/goat.png file. The intention is clear in the pattern, but the behavior is not doing what “is obvious”.

The intention of my hack was to make the “obvious” thing happen, but I agree it is not universal and only works with simple patterns.

The universal solution would be to make del not remove folders that contain at least one file that matches an exclude pattern.

The issue lays in the implementation: del gets a files & folders list using globby and then removes these files and folders. But at this point exclude patterns are already applied, and the list might contains folders of excluded files, thus folders get removed together with their files.

I think the algo should be changed if we want to solve this issue. If my understanding of glob is right, there is only one pattern that excludes files (and folders) from previous patterns, and it starts with “!”. So del should not remove any folder that contains at least one file that matches any exclude pattern (in any subfolder).

[tantv]

Hi @duzun,

I have the problems with the nested folder and how can apply your solution for this problem or we have another solution for this.

Structure:

Test folder ---- A folder ---- B folder ---- Resource folder ---- ---- assets folder ---- ---- ---- css folder ---- ---- ---- tools folder ---- ---- ---- ---- node_modules ---- ---- ---- ---- bower_components

How can I delete the Test folder without delete the node & bower folder.

My setting doesn’t work

gulp.task('clean', function () {
        return del([
            'Test/**/*',
            'Test/'
            '!Test/Resource/',
            '!Test/Resource/**',
            '!Test/Resource/assets/',
            '!Test/Resource/assets/**',
            '!Test/Resource/assets/tools/',
            '!Test/Resource/assets/tools/**'
        ], {force: true});
});

Thanks,