Handle extglobs (parentheses) containing separators

gulpjs · Nov 14, 2016 · c694462 · c694462
1 parent e5c5313
commit c694462
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -13,7 +13,7 @@ module.exports = function globParent(str) {
 
 	// remove path parts that are globby
 	do {str = pathDirname.posix(str)}
-	while (isglob(str) || /(^|[^\\])[\{\[]/.test(str));
+	while (isglob(str) || /(^|[^\\])([\{\[]|\([^\)]+$)/.test(str));
 
 	// remove escape chars and return result
 	return str.replace(/\\([\*\?\|\[\]\(\)\{\}])/g, '$1');

diff --git a/test.js b/test.js
@@ -94,6 +94,9 @@ describe('glob-parent', function() {
     assert.equal(gp('path/foo\\[a\\\/]/'), 'path/foo[a\\\/]');
     assert.equal(gp('foo[a\\\/]'), '.');
     assert.equal(gp('foo\\[a\\\/]'), 'foo[a\\\/]');
+    assert.equal(gp('path/(foo/bar|baz)'), 'path');
+    assert.equal(gp('path/(foo/bar|baz)/'), 'path');
+    assert.equal(gp('path/\\(foo/bar|baz)/'), 'path/(foo/bar|baz)');
   });
 
   it('should handle nested braces', function() {