From 8c03fff967ad1b431a7f1b31e3a1caedac238e1d Mon Sep 17 00:00:00 2001 From: Honza Javorek Date: Fri, 31 Jul 2026 17:41:52 +0200 Subject: [PATCH] fix: remove Vale BlockIgnores that broke code block scoping Vale already skips fenced code blocks and HTML comments natively when parsing Markdown, so this BlockIgnores line was redundant. Worse, it was actively harmful: because BlockIgnores blanks out matched regions before the Markdown parser sees them, the naive `(?s)(```.*?```)` pattern desynchronized Vale's position mapping. The result was both phantom alerts inside code blocks and real alerts being dropped. Reproduced on sources/academy/tutorials/apify_scrapers/cheerio_scraper.md, where a probe rule reported a hit at 146:15 - inside the fenced block spanning lines 133-153 - while a genuine hit at 156:11 disappeared. Across sources/academy and sources/platform at suggestion level, removing the line drops 247 misplaced alerts and surfaces 378 previously masked ones. This affected every rule, not one in particular. At the error level that CI enforces, the result is unchanged (10 errors before and after), so this is behavior-neutral for the pipeline today and a correctness fix for local runs and any stricter rules added later. Co-Authored-By: Claude Opus 5 --- .vale.ini | 2 -- 1 file changed, 2 deletions(-) diff --git a/.vale.ini b/.vale.ini index 716ca704a1..7715281f5d 100644 --- a/.vale.ini +++ b/.vale.ini @@ -13,8 +13,6 @@ mdx = md BasedOnStyles = Vale, Apify, write-good, Microsoft # Ignore URLs, HTML/XML tags, lines with =, emails, curly braces, inline code TokenIgnores = (<\/?[A-Z][^>]*>), ([^\n]+@[^\n]+\.[^\n]), ({[^}]*}) -# Ignore HTML comments and Markdown code blocks -BlockIgnores = (?s) ()|(```.*?```) # Disabling rules (NO) Vale.Spelling = NO