feat: use locator for reporting error

textlint-ja · Jan 6, 2024 · 9d96ae5 · 9d96ae5
1 parent da658aa
commit 9d96ae5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/no-doubled-joshi.ts b/src/no-doubled-joshi.ts
@@ -194,7 +194,7 @@ const report: TextlintRuleModule<Options> = function (context, options = {}) {
     const allow = options.allow || defaultOptions.allow;
     const separatorCharacters = options.separatorCharacters || defaultOptions.separatorCharacters;
     const commaCharacters = options.commaCharacters || defaultOptions.commaCharacters;
-    const { Syntax, report, RuleError } = context;
+    const { Syntax, report, RuleError, locator } = context;
     const is読点Token = create読点Matcher(commaCharacters);
     return {
         [Syntax.Paragraph](node) {
@@ -301,6 +301,10 @@ const report: TextlintRuleModule<Options> = function (context, options = {}) {
                             });
                             // padding positionを計算する
                             const originalIndex = sentenceSource.originalIndexFromIndex(current.word_position - 1);
+                            // originalIndexがない場合は基本的にはないが、ない場合は無視する
+                            if (originalIndex === undefined) {
+                                return current;
+                            }
                             report(
                                 // @ts-expect-error: SentenceNodeは独自であるため
                                 sentence,
@@ -315,7 +319,10 @@ const report: TextlintRuleModule<Options> = function (context, options = {}) {
 同じ助詞を連続して利用しない、文の中で順番を入れ替える、文を分割するなどを検討してください。
 `,
                                     {
-                                        index: originalIndex
+                                        padding: locator.range([
+                                            originalIndex,
+                                            originalIndex + current.surface_form.length
+                                        ])
                                     }
                                 )
                             );

diff --git a/test/no-doubled-joshi-test.ts b/test/no-doubled-joshi-test.ts
@@ -338,7 +338,7 @@ tester.run("no-double-joshi", rule, {
 
 同じ助詞を連続して利用しない、文の中で順番を入れ替える、文を分割するなどを検討してください。
 `,
-                    range: [15, 16]
+                    range: [15, 17]
                 }
             ]
         },