feat: add allowNumber option

azu · azu · commit 4e1026e29909 · 2020-06-21T01:34:25.000+09:00
`allowNumber` is `true` by default
diff --git a/README.md b/README.md
@@ -61,6 +61,13 @@ textlint --rule @textlint-ja/no-synonyms README.md
      * Default: true
      */
     allowAlphabet?: boolean;
+    /**
+     * 同じ語形の語の中での漢数字と数字の表記ゆれを許可するかどうか
+     * trueの場合は漢数字と数字の表記ゆれを許可します
+     * 例) 「1」と「一」
+     * Default: true
+     */
+    allowNumber?: boolean;
 }
 ```
 
@@ -71,7 +78,8 @@ textlint --rule @textlint-ja/no-synonyms README.md
     "rules": {
         "@textlint-ja/no-synonyms": {
             "allows": ["ウェブアプリ", "ウェブアプリケーション"],
-            "allowAlphabet": false
+            "allowAlphabet": false,
+            "allowNumber": false
         }
     }
 }
diff --git a/package.json b/package.json
@@ -53,16 +53,16 @@
     "tiny-segmenter": "^0.2.0"
   },
   "devDependencies": {
-    "@textlint/types": "^1.2.3",
-    "@types/node": "^12.12.12",
-    "husky": "^3.1.0",
-    "lint-staged": "^9.4.3",
-    "prettier": "^1.19.1",
-    "sudachi-synonyms-dictionary": "^4.0.1",
+    "@textlint/types": "^1.3.1",
+    "@types/node": "^14.0.13",
+    "husky": "^4.2.5",
+    "lint-staged": "^10.2.11",
+    "prettier": "^2.0.5",
+    "sudachi-synonyms-dictionary": "^5.0.1",
     "textlint-scripts": "^3.0.0",
-    "textlint-tester": "^5.1.11",
-    "ts-node": "^8.5.2",
-    "typescript": "^3.7.2"
+    "textlint-tester": "^5.1.15",
+    "ts-node": "^8.10.2",
+    "typescript": "^3.9.5"
   },
   "peerDependencies": {
     "sudachi-synonyms-dictionary": ">=4.0.1"
diff --git a/src/create-index.ts b/src/create-index.ts
@@ -1,4 +1,5 @@
 import { fetchDictionary, SudachiSynonyms } from "sudachi-synonyms-dictionary";
+import { isNumberString } from "./is-number";
 
 export type Midashi = string;
 
@@ -18,7 +19,7 @@ export class ItemGroup {
 
     }
 
-    usedItems(usedItemSet: Set<SudachiSynonyms>, { allowAlphabet, allows }: { allowAlphabet: boolean, allows: string[] }): SudachiSynonyms[] {
+    usedItems(usedItemSet: Set<SudachiSynonyms>, { allowAlphabet, allowNumber, allows }: { allowAlphabet: boolean; allowNumber: boolean; allows: string[] }): SudachiSynonyms[] {
         // sort by used
         return Array.from(usedItemSet.values()).filter(item => {
             if (allowAlphabet && (item.hyoukiYure === "アルファベット表記" || item.ryakusyou === "略語・略称/アルファベット")) {
@@ -28,6 +29,10 @@ export class ItemGroup {
                 // OS <-> オペレーションシステム
                 return false;
             }
+            // 数値の違いは無視する
+            if (allowNumber && isNumberString(item.midashi)) {
+                return false;
+            }
             if (allows.includes(item.midashi)) {
                 return false;
             }
@@ -49,7 +54,7 @@ const assertInstallationSudachiSynonymsDictionary = () => {
 $ npm install sudachi-synonyms-dictionary
 
 
-`)
+`);
     }
 };
 export type IndexType = { keyItemGroupMap: Map<Midashi, ItemGroup[]>; SudachiSynonymsItemGroup: Map<SudachiSynonyms, ItemGroup>; };
diff --git a/src/is-number.ts b/src/is-number.ts
@@ -0,0 +1,4 @@
+const kanNumbers = ["〇", "一", "二", "三", "四", "五", "六", "七", "八", "九"];
+export const isNumberString = (str: string): boolean => {
+    return /\d/.test(str) || kanNumbers.includes(str);
+};
diff --git a/src/textlint-rule-no-synonyms.ts b/src/textlint-rule-no-synonyms.ts
@@ -20,16 +20,25 @@ export interface Options {
      * Default: true
      */
     allowAlphabet?: boolean;
+    /**
+     * 同じ語形の語の中での漢数字と数字の表記ゆれを許可するかどうか
+     * trueの場合は漢数字と数字の表記ゆれを許可します
+     * 例) 「1」と「一」
+     * Default: true
+     */
+    allowNumber?: boolean;
 }
 
 
 export const DefaultOptions: Required<Options> = {
     allows: [],
-    allowAlphabet: true
+    allowAlphabet: true,
+    allowNumber: true
 };
 
 const report: TextlintRuleReporter<Options> = (context, options = {}) => {
-    const allowAlphabet = options.allowAlphabet !== undefined ? options.allowAlphabet : DefaultOptions.allowAlphabet;
+    const allowAlphabet = options.allowAlphabet ?? DefaultOptions.allowAlphabet;
+    const allowNumber = options.allowAlphabet ?? DefaultOptions.allowNumber;
     const allows = options.allows !== undefined ? options.allows : DefaultOptions.allows;
     const { Syntax, getSource, RuleError } = context;
     const usedSudachiSynonyms: Set<SudachiSynonyms> = new Set();
@@ -75,14 +84,15 @@ const report: TextlintRuleReporter<Options> = (context, options = {}) => {
                     for (const itemGroup of usedItemGroup.values()) {
                         const items = itemGroup.usedItems(usedSudachiSynonyms, {
                             allows,
-                            allowAlphabet
+                            allowAlphabet,
+                            allowNumber
                         });
                         if (items.length >= 2) {
                             const 同義の見出しList = items.map(item => item.midashi);
                             // select last used
                             const matchSegment = locationMap.get(items[items.length - 1]);
                             const index = matchSegment ? matchSegment.index : 0;
-                            const message = `同義語である「${同義の見出しList.join("」「")}」が利用されています`;
+                            const message = `同義語である「${同義の見出しList.join("」と「")}」が利用されています`;
                             report(node, new RuleError(message, {
                                 index
                             }));
diff --git a/test/textlint-rule-no-synonyms.test.ts b/test/textlint-rule-no-synonyms.test.ts
@@ -12,6 +12,8 @@ tester.run("textlint-rule-no-synonyms", rule, {
         // allowAlphabet: true
         // item.hyoukiYure === "アルファベット表記"
         "blogはブログです",
+        // allowNumber: true
+        "1は数字の一種です",
         // item.ryakusyou === "略語・略称/アルファベット"
         "「データベース」「DB」",
         // allow links by default
@@ -27,13 +29,13 @@ tester.run("textlint-rule-no-synonyms", rule, {
     invalid: [{
         text: "サーバとサーバーの表記揺れがある",
         errors: [{
-            message: "同義語である「サーバ」「サーバー」が利用されています",
+            message: "同義語である「サーバ」と「サーバー」が利用されています",
             index: 4
         }]
     }, {
         text: "この雇入と雇入れの違いは難しい問題だ",
         errors: [{
-            message: "同義語である「雇入」「雇入れ」が利用されています",
+            message: "同義語である「雇入」と「雇入れ」が利用されています",
             index: 5
         }]
     },
@@ -43,7 +45,7 @@ tester.run("textlint-rule-no-synonyms", rule, {
                 allowAlphabet: false
             },
             errors: [{
-                message: "同義語である「blog」「ブログ」が利用されています",
+                message: "同義語である「blog」と「ブログ」が利用されています",
                 index: 5
             }]
         }]
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,13 @@ textlint --rule @textlint-ja/no-synonyms README.md`
`61`	`61`	`* Default: true`
`62`	`62`	`*/`
`63`	`63`	`allowAlphabet?: boolean;`
	`64`	`+ /**`
	`65`	`+ * 同じ語形の語の中での漢数字と数字の表記ゆれを許可するかどうか`
	`66`	`+ * trueの場合は漢数字と数字の表記ゆれを許可します`
	`67`	`+ * 例) 「1」と「一」`
	`68`	`+ * Default: true`
	`69`	`+ */`
	`70`	`+ allowNumber?: boolean;`
`64`	`71`	`}`
`65`	`72`	```
`66`	`73`
`@@ -71,7 +78,8 @@ textlint --rule @textlint-ja/no-synonyms README.md`
`71`	`78`	`"rules": {`
`72`	`79`	`"@textlint-ja/no-synonyms": {`
`73`	`80`	`"allows": ["ウェブアプリ", "ウェブアプリケーション"],`
`74`		`- "allowAlphabet": false`
	`81`	`+ "allowAlphabet": false,`
	`82`	`+ "allowNumber": false`
`75`	`83`	`}`
`76`	`84`	`}`
`77`	`85`	`}`