From bd3db196a56a545155c6c09ab535e5808e9b5ae2 Mon Sep 17 00:00:00 2001
From: Martin Ball <martin.ball@ucscaregroup.co.uk>
Date: Mon, 2 Nov 2026 21:56:44 +0000
Subject: [PATCH] =?UTF-8?q?craft-proof=20es=20=E2=80=94=20a=20run=20of=20s?=
 =?UTF-8?q?paces=20is=20one=20Espacio=20doble=20issue=20over=20the=20whole?=
 =?UTF-8?q?=20run=20(#496=20review)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 crates/proof/src/es/grammar.rs | 26 +++++++++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/crates/proof/src/es/grammar.rs b/crates/proof/src/es/grammar.rs
index 42eede50..687ecac3 100644
--- a/crates/proof/src/es/grammar.rs
+++ b/crates/proof/src/es/grammar.rs
@@ -75,11 +75,21 @@ fn spaces(text: &str, lang: ProofLang, v: &mut Vec<Issue>) {
             v.push(issue(i, i + 1 + p.len_utf8(), "Espacio antes de un signo de puntuación".into(), vec![p.to_string()], lang));
         }
     }
-    for (i, _) in text.match_indices("  ") {
-        if i > 0 && text.get(..i).is_some_and(|t| t.ends_with(' ')) {
+    // Each run of two or more spaces once, over the whole run.
+    let bytes = text.as_bytes();
+    let mut i = 0;
+    while i < bytes.len() {
+        if bytes.get(i) != Some(&b' ') {
+            i += 1;
             continue;
         }
-        v.push(issue(i, i + 2, "Espacio doble".into(), vec![" ".into()], lang));
+        let start = i;
+        while bytes.get(i) == Some(&b' ') {
+            i += 1;
+        }
+        if i - start >= 2 {
+            v.push(issue(start, i, "Espacio doble".into(), vec![" ".into()], lang));
+        }
     }
 }
 
@@ -194,6 +204,16 @@ mod tests {
             .collect()
     }
 
+    /// craft-libs#496 review: a run of spaces is one issue over the whole run.
+    #[test]
+    fn a_run_of_spaces_is_reported_whole() {
+        for (text, run) in [("casa  roja", "  "), ("casa   roja", "   "), ("casa    roja", "    "), ("casa     roja", "     ")] {
+            let m: Vec<_> = messages(text).into_iter().filter(|(_, msg)| msg == "Espacio doble").collect();
+            assert_eq!(m, [(run.to_string(), "Espacio doble".to_string())], "{text}");
+        }
+        assert!(messages("casa roja").is_empty());
+    }
+
     #[test]
     fn repeated_words_and_spaces_have_spanish_messages() {
         let text = "Es es la casa ,  dijo.";