fix: support LinkReference types (#17)

* fix: support LinkReference types [title][id] [title] [title]: https://example.com [id]: https://example.com 形式を判定しないように修正 * Apply suggestions from code review
textlint-ja · Sep 9, 2020 · 55ac641 · 55ac641
1 parent 1c04a8c
commit 55ac641
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 7 deletions.
diff --git a/packages/textlint-rule-ja-no-space-around-parentheses/src/index.js b/packages/textlint-rule-ja-no-space-around-parentheses/src/index.js
@@ -22,7 +22,7 @@ function reporter(context) {
     const helper = new RuleHelper();
     return {
         [Syntax.Str](node){
-            if (helper.isChildNode(node, [Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
+            if (helper.isChildNode(node, [Syntax.Link, "LinkReference", Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
                 return;
             }
             const text = getSource(node);

diff --git a/packages/textlint-rule-ja-no-space-around-parentheses/test/index-test.js b/packages/textlint-rule-ja-no-space-around-parentheses/test/index-test.js
@@ -12,7 +12,11 @@ tester.run("かっこ類と隣接する文字の間のスペースの有無", ru
 そのため、特別な実装は必要なく
 「拡張する時は\`calculator.prototype\`の代わりに\`calculator.fn\`を拡張してください」
 というルールがあるだけとも言えます。
-`
+`,
+        "[テスト 「文章」 です](https://example)", // ignore Link
+        "[テスト 「文章」 です][]" +
+        "\n\n" +
+        "[テスト 「文章」 です]: https://example.com" // ignore ReferenceDef
     ],
     invalid: [
         {

diff --git a/packages/textlint-rule-ja-no-space-between-full-width/src/index.js b/packages/textlint-rule-ja-no-space-between-full-width/src/index.js
@@ -16,7 +16,7 @@ function reporter(context) {
     const helper = new RuleHelper();
     return {
         [Syntax.Str](node){
-            if (helper.isChildNode(node, [Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
+            if (helper.isChildNode(node, [Syntax.Link, "LinkReference", Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
                 return;
             }
             const text = getSource(node);

diff --git a/packages/textlint-rule-ja-space-after-exclamation/src/index.js b/packages/textlint-rule-ja-space-after-exclamation/src/index.js
@@ -12,7 +12,7 @@ function reporter(context) {
     const helper = new RuleHelper();
     return {
         [Syntax.Str](node){
-            if (helper.isChildNode(node, [Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
+            if (helper.isChildNode(node, [Syntax.Link, "LinkReference", Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
                 return;
             }
             let text = getSource(node);

diff --git a/packages/textlint-rule-ja-space-after-question/src/index.js b/packages/textlint-rule-ja-space-after-question/src/index.js
@@ -12,7 +12,7 @@ function reporter(context) {
     const helper = new RuleHelper();
     return {
         [Syntax.Str](node){
-            if (helper.isChildNode(node, [Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
+            if (helper.isChildNode(node, [Syntax.Link, "LinkReference", Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
                 return;
             }
             let text = getSource(node);

diff --git a/packages/textlint-rule-ja-space-between-half-and-full-width/src/index.js b/packages/textlint-rule-ja-space-between-half-and-full-width/src/index.js
@@ -77,7 +77,7 @@ function reporter(context, options = {}) {
     return {
         [Syntax.Str](node){
             const isIgnoredParentNode = helper.isChildNode(node, [
-                Syntax.Header, Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis
+                Syntax.Header, Syntax.Link, "LinkReference", Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis
             ]);
             if (isIgnoredParentNode) {
                 return;
@@ -96,4 +96,4 @@ function reporter(context, options = {}) {
 module.exports = {
     linter: reporter,
     fixer: reporter
-};
+};