Fix Vale false positives on context lines in PR diffs

[theletterf]

Summary

• Vale was flagging lines that were not modified in a PR — only present as context within a diff hunk.
• The line-range filter extracted hunk headers (@@ -x,y +a,b @@) and treated the entire hunk span as "modified." Any Vale issue on a context line within the hunk was wrongly reported.
• Confirmed in elastic/docs-content#6796 (15 MappedPages false positives) and elastic/docs-content#6664 (MenuArrowsBold on an unmodified line).

Example: a file with mapped_pages: at line 3. A description: field is added at line 2, producing a hunk @@ -1,4 +1,5 @@ (range 1–6). The old filter saw line 3 as in-range; the new one does not.

Changes

• vale/lint/action.yml — replace hunk-header extraction with a full patch walk that records only + (added) lines, one entry per line.
• vale/lint/vale_reporter.py — change the range check from start <= line_num <= end to start <= line_num < end (exclusive upper bound) so a count=1 entry for line N matches exactly N.

Test plan

• All existing unit tests pass (pytest vale/lint/test_vale_reporter.py — 17 passed)
• Re-run Vale on elastic/docs-content#6796 after merge and confirm MappedPages warnings are gone
• Re-run Vale on elastic/docs-content#6664 after merge and confirm MenuArrowsBold warning is gone

🤖 Generated with Claude Code

[github-actions]

Elastic Docs Style Checker (Vale)

Summary: 21 warnings, 19 suggestions found

⚠️ Warnings (21): Fix when the suggestion improves clarity or correctness.

File	Line	Rule	Message
vale/test-fixtures/sample-violations.md	2	Elastic.MappedPages	mapped_pages should only be added or updated in rare scenarios. Talk with your local technical writer before pushing changes to this key.
vale/test-fixtures/sample-violations.md	6	Elastic.Articles	Use 'an FAQ' instead of 'a FAQ'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	6	Elastic.Articles	Use 'an HTML' instead of 'a HTML'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	6	Elastic.Articles	Use 'an SQL' instead of 'a SQL'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	6	Elastic.Articles	Use 'an HTML' instead of 'a HTML'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	7	Elastic.BritishSpellings	Use American English spelling 'optimize' instead of British English 'optimise'.
vale/test-fixtures/sample-violations.md	7	Elastic.BritishSpellings	Use American English spelling 'customize' instead of British English 'customise'.
vale/test-fixtures/sample-violations.md	7	Elastic.BritishSpellings	Use American English spelling 'behavior' instead of British English 'behaviour'.
vale/test-fixtures/sample-violations.md	11	Elastic.DontUse	Don't use 'just'.
vale/test-fixtures/sample-violations.md	11	Elastic.DontUse	Don't use 'please'.
vale/test-fixtures/sample-violations.md	11	Elastic.DontUse	Don't use 'aka'.
vale/test-fixtures/sample-violations.md	20	Elastic.Articles	Use 'an FAQ' instead of 'a FAQ'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	20	Elastic.Articles	Use 'an HTML' instead of 'a HTML'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	20	Elastic.Articles	Use 'an SQL' instead of 'a SQL'. The article depends on pronunciation, not spelling.
vale/test-fixtures/sample-violations.md	24	Elastic.BritishSpellings	Use American English spelling 'optimize' instead of British English 'optimise'.
vale/test-fixtures/sample-violations.md	24	Elastic.BritishSpellings	Use American English spelling 'customize' instead of British English 'customise'.
vale/test-fixtures/sample-violations.md	24	Elastic.BritishSpellings	Use American English spelling 'behavior' instead of British English 'behaviour'.
vale/test-fixtures/sample-violations.md	40	Elastic.DontUse	Don't use 'just'.
vale/test-fixtures/sample-violations.md	40	Elastic.DontUse	Don't use 'please'.
vale/test-fixtures/sample-violations.md	40	Elastic.DontUse	Don't use 'aka'.
vale/test-fixtures/sample-violations.md	48	Elastic.Articles	Use 'an HTML' instead of 'a HTML'. The article depends on pronunciation, not spelling.

💡 Suggestions (19): Optional style improvements. Apply when helpful.

File	Line	Rule	Message
vale/test-fixtures/sample-violations.md	8	Elastic.WordChoice	Consider using 'efficiently' instead of 'Simply', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	8	Elastic.WordChoice	Consider using 'allowlist' instead of 'whitelist', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	8	Elastic.WordChoice	Consider using 'blocklist' instead of 'blacklist', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	9	Elastic.OxfordComma	Use the Oxford comma in 'reading, writing and executing.'.
vale/test-fixtures/sample-violations.md	10	Elastic.Wordiness	Consider using 'to' instead of 'In order to'.
vale/test-fixtures/sample-violations.md	10	Elastic.Wordiness	Consider using 'because' instead of 'due to the fact that'.
vale/test-fixtures/sample-violations.md	10	Elastic.Wordiness	Consider using 'many' instead of 'a large number of'.
vale/test-fixtures/sample-violations.md	12	Elastic.Exclamation	Use exclamation points sparingly. Consider removing the exclamation point.
vale/test-fixtures/sample-violations.md	12	Elastic.Exclamation	Use exclamation points sparingly. Consider removing the exclamation point.
vale/test-fixtures/sample-violations.md	28	Elastic.WordChoice	Consider using 'efficiently' instead of 'Simply', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	28	Elastic.WordChoice	Consider using 'allowlist' instead of 'whitelist', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	28	Elastic.WordChoice	Consider using 'blocklist' instead of 'blacklist', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	28	Elastic.WordChoice	Consider using 'efficient' instead of 'easy', unless the term is in the UI.
vale/test-fixtures/sample-violations.md	32	Elastic.OxfordComma	Use the Oxford comma in 'reading, writing and executing.'.
vale/test-fixtures/sample-violations.md	36	Elastic.Wordiness	Consider using 'to' instead of 'In order to'.
vale/test-fixtures/sample-violations.md	36	Elastic.Wordiness	Consider using 'because' instead of 'due to the fact that'.
vale/test-fixtures/sample-violations.md	36	Elastic.Wordiness	Consider using 'many' instead of 'a large number of'.
vale/test-fixtures/sample-violations.md	44	Elastic.Exclamation	Use exclamation points sparingly. Consider removing the exclamation point.
vale/test-fixtures/sample-violations.md	44	Elastic.Exclamation	Use exclamation points sparingly. Consider removing the exclamation point.

---

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.