Fix range must count the current text length, not the replacement's

[fnando]

I'm implementing a plugin that applies textlint fixes, and noticed that the range is being incorrectly computed.

See the example below:

$ echo '2021-07-02 (Thursday)' | textlint --format json --config samples/.textlintrc.json --stdin --stdin-filename file.md | jq
[
  {
    "messages": [
      {
        "type": "lint",
        "ruleId": "date-weekday-mismatch",
        "message": "2021-07-02 (Thursday) mismatch weekday.\n2021-07-02 (Thursday) => 2021-07-02 (Friday)",
        "index": 12,
        "line": 1,
        "column": 13,
        "severity": 2,
        "fix": {
          "range": [
            12,
            18
          ],
          "text": "Friday"
        }
      }
    ],
    "filePath": "<markdown>"
  }
]

The range should be within 8 characters (Thursday), instead of 6. If I apply the suggested range, I'll end up with Fridayay.

[azu]

Thanks for report!
I've fixed in https://github.com/textlint-rule/textlint-rule-date-weekday-mismatch/releases/tag/v1.0.6

[fnando]

Thanks for acting so quickly! <3 <3 <3