!(*.js|*.json) does not match a.js.gz about extglob HOT 8 OPEN

Hi, thanks for creating an issue.

Should this not match or is it a bug?

Please add a comparison of how bash handles these extglob patterns versus this library

from extglob.

Hello, according to bash 4.4.12, it is a bug.

shopt -s extglob
touch a.js a.js.gz a.json a.json.gz a.gz
ls !(*.js|*.json) # a.gz  a.js.gz  a.json.gz

extglob will only match a.gz

from extglob.

That seems like such incorrect behavior...

from extglob.

That seems like such incorrect behavior...

which result?

from extglob.

do the following instead:

console.log(extglob.makeRe('*.!(js|json)').test('a.js.gz')); // true
console.log(extglob.makeRe('*.!(js|json)').test('a.json.gz')); // true
console.log(extglob.makeRe('*.!(js|json)').test('a.gz')); // true

Even bash yields incorrect results sometimes when globs are nested inside negation globs. In javascript, we can't do negative lookbehinds in regular expressions, so it becomes very difficult or impossible to retroactively un-negate or un-match a substring.

edit: also, I forgot to mention that technically bash in this case is performing a "contains" match, rather than an exact match. This library is stricter in that regard. You can also use your original patterns with options.contains:

console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.js.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.json.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.gz')); // true

see https://github.com/micromatch/extglob#differences-from-bash

from extglob.

btw, on a totally unrelated note, if you happen to notice this table being corrupted, that's a bug in github's markdown parser.

from extglob.

Hi, thanks for the comments.

Indeed it is hard without negative lookbehinds.
The contains option seems to be an alternative but matches things I really don't want to match.

console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.js')); // true, but I want it to be false
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.js.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.json.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.gz')); // true

I also tried to use the *.!(js|json) form in the past, but *.!(js|json) did not match files without extensions.

console.log(extglob.makeRe('*.!(js|json)').test('a')); // false

I will look for a solution with 2 extglobs.

from extglob.

I'm thinking this looks like a bug then. I try to stick to bash behavior, but sometimes bash is unable to handle certain patterns. In this case, it seems like bash is doing the right thing.

edit: fwiw, here is what I think the results should be for all of the patterns you gave:

console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.js')); // false
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.js.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.json.gz')); // true
console.log(extglob.makeRe('!(*.js|*.json)', {contains: true}).test('a.gz')); // true
console.log(extglob.makeRe('*.!(js|json)').test('a')); // false

I agree with your opinion on the first one, it seems like that should be false.

The last one is correct because the pattern has *. which means that a dot is expected in the path before the negation even happens.

from extglob.