refactor(deps): update to sentence-spilitter@5 (#28)

* refactor(deps): update to sentence-spilitter@5

**BREAKING CHANGES**

- Require Node.js 18+

**Enhancement**

- improve error location using `locator.range()`

**Dependencies**

- update to sentence-spilitter@5
  - https://github.com/textlint-rule/sentence-splitter/releases/tag/v5.0.0

* CI: update node versions

* Update package.json

Author: azu

.github/workflows/test.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node-version: [10, 12, 14]
+        node-version: [ 18, 20 ]
     steps:
       - name: checkout
         uses: actions/checkout@v2

package.json

@@ -31,19 +31,20 @@
     "watch": "textlint-scripts build --watch",
     "prepublish": "npm run --if-present build",
     "test": "textlint-scripts test",
-    "prettier": "prettier --write \"**/*.{js,jsx,ts,tsx,css}\"",
-    "prepare": "git config --local core.hooksPath .githooks"
+    "prepare": "git config --local core.hooksPath .githooks",
+    "format": "prettier --write \"**/*.{js,jsx,ts,tsx,css}\""
   },
   "devDependencies": {
-    "lint-staged": "^10.5.4",
-    "prettier": "^2.2.1",
-    "textlint-scripts": "^3.0.0"
+    "@textlint/types": "^13.4.1",
+    "lint-staged": "^15.1.0",
+    "prettier": "^3.1.0",
+    "textlint-scripts": "^13.4.1"
   },
   "dependencies": {
     "kuromojin": "^3.0.0",
-    "sentence-splitter": "^3.2.0",
-    "textlint-rule-helper": "^2.3.0",
-    "textlint-util-to-string": "^3.3.0"
+    "sentence-splitter": "^5.0.0",
+    "textlint-rule-helper": "^2.3.1",
+    "textlint-util-to-string": "^3.3.4"
   },
   "prettier": {
     "singleQuote": false,

src/max-ten.js

@@ -2,7 +2,7 @@
 "use strict";
 import { RuleHelper } from "textlint-rule-helper";
 import { tokenize } from "kuromojin";
-import { splitAST, Syntax as SentenceSyntax } from "sentence-splitter";
+import { splitAST, SentenceSplitterSyntax as SentenceSyntax } from "sentence-splitter";
 import { StringSource } from "textlint-util-to-string";
 
 const defaultOptions = {
@@ -66,16 +66,17 @@ function findSiblingMeaningToken({ tokens, currentIndex, direction }) {
 }
 
 /**
- * @param {RuleContext} context
+ * @param {import("@textlint/types").TextlintRuleContext} context
  * @param {typeof defaultOptions} [options]
+ * @return {import("@textlint/types").TextlintFilterRuleReportHandler}}
  */
 module.exports = function (context, options = {}) {
     const maxLen = options.max ?? defaultOptions.max;
     const isStrict = options.strict ?? defaultOptions.strict;
     const touten = options.touten ?? defaultOptions.touten;
     const kuten = options.kuten ?? defaultOptions.kuten;
     const helper = new RuleHelper(context);
-    const { Syntax, RuleError, report, getSource } = context;
+    const { Syntax, RuleError, report, locator } = context;
     const separatorCharacters = [
         "?", // question mark
         "!", //  exclamation mark
@@ -154,7 +155,7 @@ module.exports = function (context, options = {}) {
                         const ruleError = new context.RuleError(
                             `一つの文で"${touten}"を${maxLen + 1}つ以上使用しています`,
                             {
-                                index
+                                padding: locator.range([index, index + touten.length])
                             }
                         );
                         report(node, ruleError);

test/max-ten-test.js

@@ -134,6 +134,19 @@ tester.run("max-ten", rule, {
                 }
             ]
         },
+        {
+            text: `複数のセンテンスがある場合。これでも、columnが、ちゃんと計算、されているはず、そのためのテキストです。`,
+            //                                                                        ^ 42
+            options: {
+                max: 3
+            },
+            errors: [
+                {
+                    message: `一つの文で"、"を4つ以上使用しています`,
+                    range: [41, 42]
+                }
+            ]
+        },
         {
             text: `複数のセンテンスがあって、改行されている場合でも\n大丈夫です。これでも、lineとcolumnが、ちゃんと計算、されているはず、そのためのテキストです。`,
             options: {