fix: trailing escaped square brackets

gulpjs · Jan 27, 2021 · 99ec9fe · 99ec9fe
1 parent 8cfb0ba
commit 99ec9fe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -7,7 +7,7 @@ var pathDirname = require('path-dirname');
 module.exports = function globParent(str) {
 	// replace braces/brackets sections with *
 	str = str.replace(/(^|[^\\])(\{([^{}]*?)}|\[([^\[\]]*?)\])/g, '$1*');
-	if (/[\}\}]$/.test(str)) str += '/';
+	if (/[\}\]]$/.test(str)) str += '/';
 
 	// preserves full path in case of trailing path separator
 	str += 'a';

diff --git a/test.js b/test.js
@@ -79,6 +79,8 @@ describe('glob-parent', function() {
     assert.equal(gp('\\{,/,bar/baz,qux\\}'), '{,/,bar/baz,qux}');
     assert.equal(gp('path/foo[a\\\/]/'), 'path');
     assert.equal(gp('path/foo\\[a\\\/]/'), 'path/foo[a\\\/]');
+    assert.equal(gp('foo[a\\\/]'), '.');
+    assert.equal(gp('foo\\[a\\\/]'), 'foo[a\\\/]');
   });
 
   it('should return parent dirname from non-glob paths', function() {