chore: rename rule

Author: azu

packages/textlint-rule-ja-no-space-between-full-and-full-width/src/index.js

@@ -1,12 +1,12 @@
-# textlint-rule-ja-no-space-between-half-and-full-width
+# textlint-rule-ja-no-space-between-full-and-full-width
 
-半角文字と全角文字のスペースについてのtextlintルール
+全角文字同士の間のスペースについてのtextlintルール
 
 ## Install
 
 Install with [npm](https://www.npmjs.com/):
 
-    npm install textlint-rule-ja-no-space-between-half-and-full-width
+    npm install textlint-rule-ja-no-space-between-full-and-full-width
 
 ## Usage
 
@@ -15,15 +15,15 @@ Via `.textlintrc`(Recommended)
 ```json
 {
     "rules": {
-        "ja-no-space-between-half-and-full-width": true
+        "ja-no-space-between-full-and-full-width": true
     }
 }
 ```
 
 Via CLI
 
 ```
-textlint --rule ja-no-space-between-half-and-full-width README.md
+textlint --rule ja-no-space-between-full-and-full-width README.md
 ```
 
 

packages/textlint-rule-ja-no-space-between-full-and-full-width/test/index-test.js

@@ -1,7 +1,7 @@
 {
   "name": "textlint-rule-ja-no-space-between-full-width",
   "version": "1.0.0",
-  "description": "半角文字と全角文字のスペースについてのtextlintルール",
+  "description": "全角文字同士の間のスペースについてのtextlintルール",
   "main": "lib/index.js",
   "repository": {
     "type": "git",
@@ -31,6 +31,7 @@
   },
   "dependencies": {
     "match-index": "^1.0.1",
+    "regx": "^1.0.4",
     "textlint-rule-helper": "^2.0.0"
   }
 }

packages/textlint-rule-ja-no-space-between-full-width/README.md

@@ -6,7 +6,11 @@
  ただしカタカナ複合語の場合を除きます。
  */
 import {RuleHelper} from "textlint-rule-helper";
-import {matchCaptureGroupAll} from "match-index";
+import {matchAll} from "match-index";
+import regx from "regx";
+const rx = regx("g");
+const japaneseRegExp = /(?:[々〇〻\u3400-\u4DBF\u4E00-\u9FFF\uF900-\uFAFF]|[\uD840-\uD87F][\uDC00-\uDFFF]|[ぁ-んァ-ヶ])/;
+
 function reporter(context) {
     const {Syntax, RuleError, report, fixer, getSource} = context;
     const helper = new RuleHelper();
@@ -16,22 +20,27 @@ function reporter(context) {
                 return;
             }
             const text = getSource(node);
-            // アルファベットと全角の間は半角スペースではない
-            const betweenHanAndZen = matchCaptureGroupAll(text, /[A-Za-z0-9]([ 　])(?:[\u3400-\u4DBF\u4E00-\u9FFF\uF900-\uFAFF]|[\uD840-\uD87F][\uDC00-\uDFFF]|[ぁ-んァ-ヶ])/);
-            const betweenZenAndHan = matchCaptureGroupAll(text, /(?:[\u3400-\u4DBF\u4E00-\u9FFF\uF900-\uFAFF]|[\uD840-\uD87F][\uDC00-\uDFFF]|[ぁ-んァ-ヶ])([ 　])[A-Za-z0-9]/);
-            const reportMatch = (match) => {
-                const {index} = match;
-                report(node, new RuleError("原則として、全角文字と半角文字の間にスペースを入れません。", {
-                    index: match.index,
-                    fix: fixer.replaceTextRange([index, index + 1], "")
-                }));
-            };
-            betweenHanAndZen.forEach(reportMatch);
-            betweenZenAndHan.forEach(reportMatch);
+            // 全角同士の間は半角スペースを入れない
+            const matchReg = rx`${japaneseRegExp}( )${japaneseRegExp}`;
+            const katakakana = /[ァ-ヶ]( )[ァ-ヶ]/;
+            matchAll(text, matchReg).forEach(match => {
+                const {input, captureGroups} = match;
+                // ただしカタカナ複合語の場合を除きます。
+                if (katakakana.test(input)) {
+                    return;
+                }
+                captureGroups.forEach(captureGroup => {
+                    const index = captureGroup.index;
+                    report(node, new RuleError("原則として、全角文字どうしの間にスペースを入れません。", {
+                        index: index,
+                        fix: fixer.replaceTextRange([index, index + 1], "")
+                    }));
+                });
+            });
         }
-    }
+    };
 }
 module.exports = {
     linter: reporter,
     fixer: reporter
-};
+};

packages/textlint-rule-ja-no-space-between-full-width/package.json

@@ -3,37 +3,32 @@
 import TextLintTester from "textlint-tester";
 import rule from "../src/index";
 var tester = new TextLintTester();
-tester.run("全角文字と半角文字の間", rule, {
+tester.run("全角文字どうし", rule, {
     valid: [
-        "JTF標準",
+        "これは正解",
+        "This is 大丈夫",
         "This is a pen.",
-        "1. `./*.*`にマッチするファイルを取得 = Readable Stream",
-        `[CONTRIBUTING.md](./CONTRIBUTING.md)に、書籍で扱うべきプラグインアーキテクチャのProposalの書き方や
-Pull Request、コミットのやりかたなどが書かれています。`
+        "ユーザー インターフェース"//カタカナは例外
     ],
     invalid: [
         {
-            text: "JTF 標準",
-            output: "JTF標準",
+            text: "これは ダメ",
+            output: "これはダメ",
             errors: [
                 {
-                    message: "原則として、全角文字と半角文字の間にスペースを入れません。",
+                    message: "原則として、全角文字どうしの間にスペースを入れません。",
                     column: 4
                 }
             ]
         },
         {
-            text: "これは Unicode",
-            output: "これはUnicode",
+            text: "これは どういうこと？",
+            output: "これはどういうこと？",
             errors: [
-                {message: "原則として、全角文字と半角文字の間にスペースを入れません。"}
-            ]
-        },
-        {
-            text: "これは　Unicode",
-            output: "これはUnicode",
-            errors: [
-                {message: "原則として、全角文字と半角文字の間にスペースを入れません。"}
+                {
+                    message: "原則として、全角文字どうしの間にスペースを入れません。",
+                    column: 4
+                }
             ]
         }
     ]

packages/textlint-rule-ja-no-space-between-full-width/src/index.js

@@ -1,12 +1,12 @@
-# textlint-rule-ja-no-space-between-full-and-full-width
+# textlint-rule-ja-no-space-between-half-and-full-width
 
-全角文字同士の間のスペースについてのtextlintルール
+半角文字と全角文字のスペースについてのtextlintルール
 
 ## Install
 
 Install with [npm](https://www.npmjs.com/):
 
-    npm install textlint-rule-ja-no-space-between-full-and-full-width
+    npm install textlint-rule-ja-no-space-between-half-and-full-width
 
 ## Usage
 
@@ -15,15 +15,15 @@ Via `.textlintrc`(Recommended)
 ```json
 {
     "rules": {
-        "ja-no-space-between-full-and-full-width": true
+        "ja-no-space-between-half-and-full-width": true
     }
 }
 ```
 
 Via CLI
 
 ```
-textlint --rule ja-no-space-between-full-and-full-width README.md
+textlint --rule ja-no-space-between-half-and-full-width README.md
 ```
 
 

packages/textlint-rule-ja-no-space-between-full-width/test/index-test.js

@@ -1,7 +1,7 @@
 {
-  "name": "textlint-rule-ja-no-space-between-full-and-full-width",
+  "name": "textlint-rule-ja-no-space-between-half-and-full-width",
   "version": "1.0.0",
-  "description": "全角文字同士の間のスペースについてのtextlintルール",
+  "description": "半角文字と全角文字のスペースについてのtextlintルール",
   "main": "lib/index.js",
   "repository": {
     "type": "git",
@@ -31,7 +31,6 @@
   },
   "dependencies": {
     "match-index": "^1.0.1",
-    "regx": "^1.0.4",
     "textlint-rule-helper": "^2.0.0"
   }
 }

packages/textlint-rule-ja-no-space-between-full-and-full-width/.babelrc renamed to packages/textlint-rule-ja-no-space-between-half-and-full-width/.babelrc

@@ -0,0 +1,37 @@
+// LICENSE : MIT
+"use strict";
+/*
+ 全角文字と半角文字の間
+ 原則として、全角文字と半角文字の間にスペースを入れません。
+ ただしカタカナ複合語の場合を除きます。
+ */
+import {RuleHelper} from "textlint-rule-helper";
+import {matchCaptureGroupAll} from "match-index";
+function reporter(context) {
+    const {Syntax, RuleError, report, fixer, getSource} = context;
+    const helper = new RuleHelper();
+    return {
+        [Syntax.Str](node){
+            if (helper.isChildNode(node, [Syntax.Link, Syntax.Image, Syntax.BlockQuote, Syntax.Emphasis])) {
+                return;
+            }
+            const text = getSource(node);
+            // アルファベットと全角の間は半角スペースではない
+            const betweenHanAndZen = matchCaptureGroupAll(text, /[A-Za-z0-9]([ 　])(?:[\u3400-\u4DBF\u4E00-\u9FFF\uF900-\uFAFF]|[\uD840-\uD87F][\uDC00-\uDFFF]|[ぁ-んァ-ヶ])/);
+            const betweenZenAndHan = matchCaptureGroupAll(text, /(?:[\u3400-\u4DBF\u4E00-\u9FFF\uF900-\uFAFF]|[\uD840-\uD87F][\uDC00-\uDFFF]|[ぁ-んァ-ヶ])([ 　])[A-Za-z0-9]/);
+            const reportMatch = (match) => {
+                const {index} = match;
+                report(node, new RuleError("原則として、全角文字と半角文字の間にスペースを入れません。", {
+                    index: match.index,
+                    fix: fixer.replaceTextRange([index, index + 1], "")
+                }));
+            };
+            betweenHanAndZen.forEach(reportMatch);
+            betweenZenAndHan.forEach(reportMatch);
+        }
+    }
+}
+module.exports = {
+    linter: reporter,
+    fixer: reporter
+};