Fix closing paren followed by punctuation

Closes remarkjs/remark-gfm#12.

Author: wooorm

syntax.js

@@ -9,7 +9,6 @@ var www = {tokenize: tokenizeWww, partial: true}
 var domain = {tokenize: tokenizeDomain, partial: true}
 var path = {tokenize: tokenizePath, partial: true}
 var punctuation = {tokenize: tokenizePunctuation, partial: true}
-var paren = {tokenize: tokenizeParen, partial: true}
 var namedCharacterReference = {
   tokenize: tokenizeNamedCharacterReference,
   partial: true
@@ -410,7 +409,11 @@ function tokenizePath(effects, ok) {
 
     // `)`
     if (code === 41) {
-      return effects.check(paren, parenAtPathEnd, continuedPunctuation)(code)
+      return effects.check(
+        punctuation,
+        parenAtPathEnd,
+        continuedPunctuation
+      )(code)
     }
 
     if (pathEnd(code)) {
@@ -467,26 +470,6 @@ function tokenizeNamedCharacterReference(effects, ok, nok) {
   }
 }
 
-function tokenizeParen(effects, ok, nok) {
-  return start
-
-  function start(code) {
-    // Assume a right paren.
-    effects.consume(code)
-    return after
-  }
-
-  function after(code) {
-    // If the punctuation marker is followed by the end of the path, it’s not
-    // continued punctuation.
-    return pathEnd(code) ||
-      // `)`
-      code === 41
-      ? ok(code)
-      : nok(code)
-  }
-}
-
 function tokenizePunctuation(effects, ok, nok) {
   return start
 

test/index.js

@@ -110,5 +110,62 @@ test('markdown -> html (micromark)', function (t) {
     'should support named character references in domains'
   )
 
+  t.deepEqual(
+    micromark('https://a.bc/d/e/).', {
+      extensions: [syntax],
+      htmlExtensions: [html]
+    }),
+    '<p><a href="https://a.bc/d/e/">https://a.bc/d/e/</a>).</p>',
+    'should support a closing paren and period after a path'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc/d/e/.)', {
+      extensions: [syntax],
+      htmlExtensions: [html]
+    }),
+    '<p><a href="https://a.bc/d/e/">https://a.bc/d/e/</a>.)</p>',
+    'should support a period and closing paren after a path'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc).', {extensions: [syntax], htmlExtensions: [html]}),
+    '<p><a href="https://a.bc">https://a.bc</a>).</p>',
+    'should support a closing paren and period after a domain'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc.)', {extensions: [syntax], htmlExtensions: [html]}),
+    '<p><a href="https://a.bc">https://a.bc</a>.)</p>',
+    'should support a period and closing paren after a domain'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc).d', {
+      extensions: [syntax],
+      htmlExtensions: [html]
+    }),
+    '<p><a href="https://a.bc).d">https://a.bc).d</a></p>',
+    'should support a closing paren and period in a path'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc.)d', {
+      extensions: [syntax],
+      htmlExtensions: [html]
+    }),
+    '<p><a href="https://a.bc.)d">https://a.bc.)d</a></p>',
+    'should support a period and closing paren in a path'
+  )
+
+  t.deepEqual(
+    micromark('https://a.bc/))d', {
+      extensions: [syntax],
+      htmlExtensions: [html]
+    }),
+    '<p><a href="https://a.bc/))d">https://a.bc/))d</a></p>',
+    'should support two closing parens in a path'
+  )
+
   t.end()
 })