refactor(rule): move all rule to index-based reporter

azu · azu · commit d81b138d3dce · 2016-03-07T00:40:35.000+09:00
diff --git a/src/1.1.2.js b/src/1.1.2.js
@@ -31,7 +31,7 @@ function mixer(context) {
             if (index !== -1) {
                 report(node, {
                     message: "見出しの文末には、句点(。)を付けません。",
-                    column: index,
+                    index: index,
                     fix: fixer.removeRange([index, index + 1])
                 });
             }
diff --git a/src/1.2.1.js b/src/1.2.1.js
@@ -44,7 +44,7 @@ const reporter = (context) => {
                 const indexOfSymbol = match.index;
                 report(node, {
                     message: "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
-                    column: indexOfSymbol,
+                    index: indexOfSymbol,
                     fix: fixer.replaceTextRange([indexOfSymbol, indexOfSymbol + 1], symbol)
                 });
             })
diff --git a/src/1.2.2.js b/src/1.2.2.js
@@ -26,7 +26,7 @@ function report(context) {
                 const symbol = replaceSymbol[text[index]];
                 report(node, {
                     message: "全角のピリオドとカンマは使用しません。",
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], symbol)
                 });
             }
diff --git a/src/2.1.10.js b/src/2.1.10.js
@@ -27,7 +27,7 @@ function reporter(context) {
                 matchCaptureGroupAll(matchedString, strictMatchReg).forEach(subMatch => {
                     const {index} = subMatch;
                     report(node, new RuleError("小数点には「ピリオド」を使います。", {
-                        column: match.index + index,
+                        index: match.index + index,
                         fix: fixer.replaceTextRange([match.index + index, match.index + index + 1], ".")
                     }));
                 });
diff --git a/src/2.1.2.js b/src/2.1.2.js
@@ -25,7 +25,9 @@ export default function (context) {
             for (let index = 0; index < strArray.length; index++) {
                 const item = strArray[index];
                 if (kanjiRegExp.test(item) && !isJoyo(item)) {
-                    report(node, new RuleError("「" + item + "」は「常用漢字表」外の漢字です。", index));
+                    report(node, new RuleError("「" + item + "」は「常用漢字表」外の漢字です。", {
+                        index
+                    }));
                 }
             }
         }
diff --git a/src/2.1.5.js b/src/2.1.5.js
@@ -44,7 +44,7 @@ function reporter(context) {
             const {index, text} = match;
             report(node, {
                 message: "カタカナは「全角」で表記します。",
-                column: index,
+                index: index,
                 fix: fixer.replaceTextRange([index, index + text.length], toZenkaku(text))
             });
 
diff --git a/src/2.1.8.js b/src/2.1.8.js
@@ -25,7 +25,7 @@ function reporter(context) {
                 const {index, text} = match;
                 report(node, {
                     message: "算用数字は「半角」で表記します。",
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + text.length], toHankaku(text))
                 })
             });
diff --git a/src/2.1.9.js b/src/2.1.9.js
@@ -25,7 +25,7 @@ function reporter(context) {
                 const {index, text} = match;
                 report(node, {
                     message: "アルファベットは「半角」で表記します。",
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + text.length], toHankaku(text))
                 })
             });
diff --git a/src/2.2.2.js b/src/2.2.2.js
@@ -125,7 +125,7 @@ function reporter(context) {
                 });
                 const ruleError = new RuleError(`${matchedString} => ${expected}
 数量を表現し、数を数えられるものは算用数字を使用します。任意の数に置き換えても通用する語句がこれに該当します。`, {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + matchedString.length], expected)
                 });
                 report(node, ruleError);
@@ -142,7 +142,7 @@ function reporter(context) {
                 const index = match.index;
                 report(node, new RuleError(`${matchedString} => ${expected}
 慣用的表現、熟語、概数、固有名詞、副詞など、漢数字を使用することが一般的な語句では漢数字を使います。`, {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + matchedString.length], expected)
                 }));
             };
diff --git a/src/3.1.1.js b/src/3.1.1.js
@@ -22,7 +22,7 @@ function reporter(context) {
             const reportMatch = (match) => {
                 const {index} = match;
                 report(node, new RuleError("原則として、全角文字と半角文字の間にスペースを入れません。", {
-                    column: match.index,
+                    index: match.index,
                     fix: fixer.replaceTextRange([index, index + 1], "")
                 }));
             };
diff --git a/src/3.1.2.js b/src/3.1.2.js
@@ -31,7 +31,7 @@ function reporter(context) {
                 captureGroups.forEach(captureGroup => {
                     const index = captureGroup.index;
                     report(node, new RuleError("原則として、全角文字どうしの間にスペースを入れません。", {
-                        column: index,
+                        index: index,
                         fix: fixer.replaceTextRange([index, index + 1], "")
                     }));
                 });
diff --git a/src/3.2.js b/src/3.2.js
@@ -22,7 +22,7 @@ export default function (context) {
                 const {text} = match;
                 if (text === "　") {
                     report(node, new RuleError("カタカナ語間は中黒（・）または半角スペースを用いてカタカナ語を区切ります", {
-                        column: match.index
+                        index: match.index
                     }));
                 }
             });
diff --git a/src/3.3.js b/src/3.3.js
@@ -12,10 +12,10 @@ const brackets = [
 ];
 
 const leftBrackets = brackets.map(bracket => {
-    return new RegExp("\(\\s\)" + bracket, "g");
+    return new RegExp("\([ 　]\)" + bracket, "g");
 });
 const rightBrackets = brackets.map(bracket => {
-    return new RegExp(bracket + "\(\\s\)", "g");
+    return new RegExp(bracket + "\([ 　])", "g");
 });
 function reporter(context) {
     let {Syntax, RuleError, report, fixer, getSource} = context;
@@ -30,21 +30,21 @@ function reporter(context) {
                 matchCaptureGroupAll(text, pattern).forEach(match => {
                     const {index} = match;
                     report(node, new RuleError("かっこの外側、内側ともにスペースを入れません。", {
-                        column: index,
+                        index: index,
                         fix: fixer.replaceTextRange([index, index + 1], "")
                     }));
-                })
+                });
             });
             // 右にスペース
             rightBrackets.forEach(pattern => {
                 matchCaptureGroupAll(text, pattern).forEach(match => {
-                    const {index} = match;
+                    const {index, text} = match;
                     report(node, new RuleError("かっこの外側、内側ともにスペースを入れません。", {
-                        column: index,
+                        index: index,
                         fix: fixer.replaceTextRange([index, index + 1], "")
                     }));
-                })
-            })
+                });
+            });
         }
     }
 }
diff --git a/src/4.1.1.js b/src/4.1.1.js
@@ -27,7 +27,7 @@ var reporter = function reporter(context) {
                 matchCaptureGroupAll(text, pattern).forEach(match => {
                     const {index} = match;
                     report(node, new RuleError("文中にかぎかっこが入る場合は、閉じかっこの前に句点を打ちません。", {
-                        column: index,
+                        index: index,
                         fix: fixer.replaceTextRange([index, index + 1], "")
                     }));
                 });
diff --git a/src/4.1.3.js b/src/4.1.3.js
@@ -23,7 +23,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchReg).forEach(match => {
                 const index = match.index;
                 report(node, new RuleError("和文の句読点としてはピリオドを使用しません。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "。")
                 }));
             });
diff --git a/src/4.2.1.js b/src/4.2.1.js
@@ -27,7 +27,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 return report(node, new RuleError("感嘆符(！)を使用する場合は「全角」で表記します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "！")
                 }));
             });
@@ -37,7 +37,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchAfter).forEach(match => {
                 const {index} = match;
                 return report(node, new RuleError("文末に感嘆符を使用し、後に別の文が続く場合は、直後に全角スペースを挿入します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "　")
                 }));
             });
diff --git a/src/4.2.2.js b/src/4.2.2.js
@@ -28,7 +28,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 return report(node, new RuleError("疑問符(？)を使用する場合は「全角」で表記します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "？")
                 }));
             });
@@ -38,7 +38,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchAfter).forEach(match => {
                 const {index} = match;
                 return report(node, new RuleError("文末に感嘆符を使用し、後に別の文が続く場合は、直後に全角スペースを挿入します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "　")
                 }));
             });
diff --git a/src/4.2.4.js b/src/4.2.4.js
@@ -26,7 +26,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchHanNakaguro).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("カタカナ複合語を区切る場合または同格の語句を並列する場合には全角の中黒（・）を使用します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "・")
                 }));
             })
diff --git a/src/4.2.5.js b/src/4.2.5.js
@@ -20,7 +20,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchHanQuestion).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("数値の範囲を示す場合には全角の〜を使用します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "〜")
                 }));
             });
diff --git a/src/4.2.6.js b/src/4.2.6.js
@@ -30,7 +30,7 @@ export default function (context) {
                 const {index} = match;
                 report(node, new RuleError(`原則として和文ではハイフン(-)を使用しません。
 例外は、住所や電話番号の区切りに使う場合です。`, {
-                    column: index
+                    index: index
                 }))
             });
         }
diff --git a/src/4.2.7.js b/src/4.2.7.js
@@ -25,7 +25,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchHanQuestion).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("コロン(：)を使用する場合は「全角」で表記します。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], "：")
                 }))
             })
diff --git a/src/4.2.8.js b/src/4.2.8.js
@@ -23,7 +23,7 @@ export default function (context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("原則として和文ではセミコロン(;)を使用しません。", {
-                    column: index
+                    index: index
                 }));
             })
         }
diff --git a/src/4.2.9.js b/src/4.2.9.js
@@ -27,7 +27,7 @@ export default function (context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("原則として和文ではダッシュ(―)を使用しません。", {
-                    column: index
+                    index
                 }));
             })
         }
diff --git a/src/4.3.1.js b/src/4.3.1.js
@@ -34,7 +34,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("半角のかっこ()が使用されています。全角のかっこ（）を使用してください。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], replaceSymbol(match.text))
                 }));
             });
diff --git a/src/4.3.2.js b/src/4.3.2.js
@@ -34,7 +34,7 @@ function reporter(context) {
             matchCaptureGroupAll(text, matchRegExp).forEach(match => {
                 const {index} = match;
                 report(node, new RuleError("半角の大かっこ[]が使用されています。全角のかっこ［］を使用してください。", {
-                    column: index,
+                    index: index,
                     fix: fixer.replaceTextRange([index, index + 1], replaceSymbol(match.text))
                 }));
             });
diff --git a/src/util/pair-checker.js b/src/util/pair-checker.js
@@ -9,7 +9,7 @@
  */
 import assert from "assert";
 import {RuleHelper} from "textlint-rule-helper";
-export function checkPair(context, { left, right }) {
+export function checkPair(context, {left, right}) {
     assert(left);
     assert(right);
     let {Syntax, RuleError, report, getSource} = context;
@@ -50,7 +50,9 @@ export function checkPair(context, { left, right }) {
             }
 
             matchParentheses.forEach(({node, index}) => {
-                report(node, new RuleError(`${left}の対となる${right}が見つかりません。${left}${right}`, index));
+                report(node, new RuleError(`${left}の対となる${right}が見つかりません。${left}${right}`, {
+                    index
+                }));
             });
             // clear state
             matchParentheses = [];
diff --git a/test/3.3-test.js b/test/3.3-test.js
@@ -6,7 +6,13 @@ var tester = new TextLintTester();
 tester.run("3.3.かっこ類と隣接する文字の間のスペースの有無", rule, {
     valid: [
         "「良い」",
-        "テスト［文章］です"
+        "テスト［文章］です",
+        `
+実装をみてもらうと分かりますが、JavaScriptの\`prototype\`の仕組みをそのまま利用しています。
+そのため、特別な実装は必要なく
+「拡張する時は\`calculator.prototype\`の代わりに\`calculator.fn\`を拡張してください」
+というルールがあるだけとも言えます。
+`
     ],
     invalid: [
         {
@@ -26,32 +32,6 @@ tester.run("3.3.かっこ類と隣接する文字の間のスペースの有無"
                     column: 4
                 }
             ]
-        },
-        {
-            text:  // 改行はスペースではないとする
-                `
-実装をみてもらうと分かりますが、JavaScriptの\`prototype\`の仕組みをそのまま利用しています。
-そのため、特別な実装は必要なく
-「拡張する時は\`calculator.prototype\`の代わりに\`calculator.fn\`を拡張してください」
-というルールがあるだけとも言えます。
-`,
-            output: `
-実装をみてもらうと分かりますが、JavaScriptの\`prototype\`の仕組みをそのまま利用しています。
-そのため、特別な実装は必要なく「拡張する時は\`calculator.prototype\`の代わりに\`calculator.fn\`を拡張してください」というルールがあるだけとも言えます。
-`,
-            errors: [
-                {
-                    message: "かっこの外側、内側ともにスペースを入れません。",
-                    line: 2,
-                    column: 15
-                },
-                {
-                    message: "かっこの外側、内側ともにスペースを入れません。",
-                    line: 4,
-                    column: 15
-                }
-
-            ]
         }
     ]
 });

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ function mixer(context) {`
`31`	`31`	`if (index !== -1) {`
`32`	`32`	`report(node, {`
`33`	`33`	`message: "見出しの文末には、句点(。)を付けません。",`
`34`		`- column: index,`
	`34`	`+ index: index,`
`35`	`35`	`fix: fixer.removeRange([index, index + 1])`
`36`	`36`	`});`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ function report(context) {`
`26`	`26`	`const symbol = replaceSymbol[text[index]];`
`27`	`27`	`report(node, {`
`28`	`28`	`message: "全角のピリオドとカンマは使用しません。",`
`29`		`- column: index,`
	`29`	`+ index: index,`
`30`	`30`	`fix: fixer.replaceTextRange([index, index + 1], symbol)`
`31`	`31`	`});`
`32`	`32`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,9 @@ export default function (context) {`
`25`	`25`	`for (let index = 0; index < strArray.length; index++) {`
`26`	`26`	`const item = strArray[index];`
`27`	`27`	`if (kanjiRegExp.test(item) && !isJoyo(item)) {`
`28`		`- report(node, new RuleError("「" + item + "」は「常用漢字表」外の漢字です。", index));`
	`28`	`+ report(node, new RuleError("「" + item + "」は「常用漢字表」外の漢字です。", {`
	`29`	`+ index`
	`30`	`+ }));`
`29`	`31`	`}`
`30`	`32`	`}`
`31`	`33`	`}`