fix(2.2.2): 11種類」が「1一種類」と誤検知される問題を修正 (#66)

fix #65

Author: azu

src/1.1.1.js

@@ -28,9 +28,13 @@ module.exports = function(context) {
             matches.forEach(match => {
                 let message;
                 if (desumasu) {
-                    message = `本文を常体(である調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"${match.value}"が敬体(ですます調)です。`;
+                    message = `本文を常体(である調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"${
+                        match.value
+                    }"が敬体(ですます調)です。`;
                 } else if (dearu) {
-                    message = `本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"${match.value}"が常体(である調)です。`;
+                    message = `本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"${
+                        match.value
+                    }"が常体(である調)です。`;
                 }
                 report(
                     node,

src/1.1.3.js

@@ -43,9 +43,13 @@ module.exports = function(context) {
             matches.forEach(match => {
                 let message;
                 if (desumasu) {
-                    message = `箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"${match.value}"が常体(である調)です。`;
+                    message = `箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"${
+                        match.value
+                    }"が常体(である調)です。`;
                 } else if (dearu) {
-                    message = `箇条書きを常体(である調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"${match.value}"が敬体(ですます調)です。`;
+                    message = `箇条書きを常体(である調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"${
+                        match.value
+                    }"が敬体(ですます調)です。`;
                 }
                 report(
                     node,

src/1.1.5.js

@@ -25,9 +25,13 @@ module.exports = function(context) {
             matches.forEach(match => {
                 let message;
                 if (desumasu) {
-                    message = `図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"${match.value}"が常体(である調)です。`;
+                    message = `図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"${
+                        match.value
+                    }"が常体(である調)です。`;
                 } else if (dearu) {
-                    message = `図表のキャプションを常体(である調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"${match.value}"が敬体(ですます調)です。`;
+                    message = `図表のキャプションを常体(である調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"${
+                        match.value
+                    }"が敬体(ですます調)です。`;
                 }
                 report(
                     node,

src/1.2.1.js

@@ -44,10 +44,13 @@ const reporter = context => {
                 const indexOfSymbol = match.index;
                 report(
                     node,
-                    new RuleError("句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。", {
-                        index: indexOfSymbol,
-                        fix: fixer.replaceTextRange([indexOfSymbol, indexOfSymbol + 1], symbol)
-                    })
+                    new RuleError(
+                        "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
+                        {
+                            index: indexOfSymbol,
+                            fix: fixer.replaceTextRange([indexOfSymbol, indexOfSymbol + 1], symbol)
+                        }
+                    )
                 );
             });
         }

src/2.2.2.js

@@ -2,6 +2,7 @@
 "use strict";
 import { isUserWrittenNode } from "./util/node-util";
 import ja2num from "japanese-numerals-to-number";
+
 function matchToReplace(text, pattern, matchFn) {
     var match = pattern.exec(text);
     if (match) {
@@ -129,8 +130,12 @@ function reporter(context) {
                 report(node, ruleError);
             };
 
-            // 算用数字 -> 漢数字
-
+            /**
+             * 算用数字 -> 漢数字
+             * @param {string} text
+             * @param {RegExp} pattern
+             * @param {*[]} match
+             */
             const toKanNumber = (text, pattern, match) => {
                 const matchedString = match[0];
                 const expected = matchedString.replace(pattern, function(all, match) {
@@ -183,7 +188,8 @@ function reporter(context) {
             matchToReplace(text, /(1)時的/g, toKanNumber);
             matchToReplace(text, /(1)部分/g, toKanNumber);
             matchToReplace(text, /第(3)者/g, toKanNumber);
-            matchToReplace(text, /(1)種/g, toKanNumber);
+            // 1種 -> 一種: 11種類などにはマッチしない
+            matchToReplace(text, /[^\d](1)種(?!類)/g, toKanNumber);
             matchToReplace(text, /(1)部の/g, toKanNumber);
             matchToReplace(text, /(1)番に/g, toKanNumber);
             matchToReplace(text, /数([0-9]+)倍/g, toKanNumber);

src/4.2.4.js

@@ -27,10 +27,13 @@ function reporter(context) {
                 const { index } = match;
                 report(
                     node,
-                    new RuleError("カタカナ複合語を区切る場合または同格の語句を並列する場合には全角の中黒（・）を使用します。", {
-                        index: index,
-                        fix: fixer.replaceTextRange([index, index + 1], "・")
-                    })
+                    new RuleError(
+                        "カタカナ複合語を区切る場合または同格の語句を並列する場合には全角の中黒（・）を使用します。",
+                        {
+                            index: index,
+                            fix: fixer.replaceTextRange([index, index + 1], "・")
+                        }
+                    )
                 );
             });
         }

test/1.1.1-test.js

@@ -25,7 +25,8 @@ tester.run("1.1.1.本文", rule, {
 それはペンです。`,
             errors: [
                 {
-                    message: '本文を常体(である調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"です。"が敬体(ですます調)です。',
+                    message:
+                        '本文を常体(である調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"です。"が敬体(ですます調)です。',
                     line: 5,
                     column: 6
                 }
@@ -38,7 +39,8 @@ tester.run("1.1.1.本文", rule, {
 `,
             errors: [
                 {
-                    message: '本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"のだが"が常体(である調)です。',
+                    message:
+                        '本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"のだが"が常体(である調)です。',
                     line: 1,
                     column: 8
                 }
@@ -49,7 +51,8 @@ tester.run("1.1.1.本文", rule, {
             text: `今日はいい天気ですね。今日はいい天気である。`,
             errors: [
                 {
-                    message: '本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"である。"が常体(である調)です。',
+                    message:
+                        '本文を敬体(ですます調)に統一して下さい。\n本文の文体は、敬体(ですます調)あるいは常体(である調)のどちらかで統一します。\n"である。"が常体(である調)です。',
                     line: 1,
                     column: 19
                 }

test/1.1.3-test.js

@@ -27,7 +27,8 @@ tester.run("1.1.3.箇条書き", rule, {
             `.trim(),
             errors: [
                 {
-                    message: '箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"である。"が常体(である調)です。',
+                    message:
+                        '箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"である。"が常体(である調)です。',
                     line: 3,
                     column: 14
                 }
@@ -41,7 +42,8 @@ tester.run("1.1.3.箇条書き", rule, {
             `.trim(),
             errors: [
                 {
-                    message: '箇条書きを常体(である調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"です。"が敬体(ですます調)です。',
+                    message:
+                        '箇条書きを常体(である調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"です。"が敬体(ですます調)です。',
                     line: 1,
                     column: 15
                 }
@@ -55,7 +57,8 @@ tester.run("1.1.3.箇条書き", rule, {
             `.trim(),
             errors: [
                 {
-                    message: '箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"である。"が常体(である調)です。',
+                    message:
+                        '箇条書きを敬体(ですます調)に統一して下さい。\nひとまとまりの箇条書きでは、敬体と常体を混在させません。\n"である。"が常体(である調)です。',
                     line: 2,
                     column: 14
                 }
@@ -68,7 +71,8 @@ tester.run("1.1.3.箇条書き", rule, {
             `.trim(),
             errors: [
                 {
-                    message: "箇条書きの文末に句点(。)を付けて下さい。\n箇条書きの文末に句点(。)を付けるかを統一します。",
+                    message:
+                        "箇条書きの文末に句点(。)を付けて下さい。\n箇条書きの文末に句点(。)を付けるかを統一します。",
                     line: 2,
                     column: 1 // 無指定なので - PR Welcome
                 }
@@ -82,7 +86,8 @@ tester.run("1.1.3.箇条書き", rule, {
             `.trim(),
             errors: [
                 {
-                    message: "箇条書きの文末から句点(。)を外して下さい。\n箇条書きの文末に句点(。)を付けるかを統一します。",
+                    message:
+                        "箇条書きの文末から句点(。)を外して下さい。\n箇条書きの文末に句点(。)を付けるかを統一します。",
                     line: 1,
                     column: 1 // 無指定なので - PR Welcome
                 }

test/1.1.5-test.js

@@ -15,7 +15,8 @@ tester.run("1.1.5.図表のキャプション", rule, {
                 "![これは図です。](http://example.com/img) と ![これは図です。](http://example.com/img)  と ![これは図である。](http://example.com/img).",
             errors: [
                 {
-                    message: '図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"である。"が常体(である調)です。',
+                    message:
+                        '図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"である。"が常体(である調)です。',
                     line: 1,
                     column: 82
                 }
@@ -26,7 +27,8 @@ tester.run("1.1.5.図表のキャプション", rule, {
                 "![これは図です。](http://example.com/img) と ![これは図である。](http://example.com/img)  と ![これは図である。](http://example.com/img).",
             errors: [
                 {
-                    message: '図表のキャプションを常体(である調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"です。"が敬体(ですます調)です。',
+                    message:
+                        '図表のキャプションを常体(である調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"です。"が敬体(ですます調)です。',
                     line: 1,
                     column: 7 // imgのposition
                 }
@@ -37,7 +39,8 @@ tester.run("1.1.5.図表のキャプション", rule, {
             text: "![これは図です。](http://example.com/img) と ![これは図である。](http://example.com/img).",
             errors: [
                 {
-                    message: '図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"である。"が常体(である調)です。',
+                    message:
+                        '図表のキャプションを敬体(ですます調)に統一して下さい。\n図表のキャプション内で敬体、常体を混在させないことが重要です。\n"である。"が常体(である調)です。',
                     line: 1,
                     column: 44
                 }

test/1.2.1-test.js

@@ -15,11 +15,13 @@ tester.run("1.2.1.句点(。)と読点(、)", rule, {
             output: "これは、見本となる例です。",
             errors: [
                 {
-                    message: "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
+                    message:
+                        "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
                     column: 4
                 },
                 {
-                    message: "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
+                    message:
+                        "句読点には全角の「、」と「。」を使います。和文の句読点としてピリオド(.)とカンマ(,)を使用しません。",
                     column: 13
                 }
             ]