FIX-33/34/35: Fix infinite loop — grammar corruption + spelling conflict

FIX-33: Grammar rules fix_prepositions_advanced was corrupting
الامتحان→الامتحين. The regex blindly replaced ان→ين after prepositions,
but الامتحان is a root noun not a dual. Added blocklist of ~60 common
root-form nouns and skip all ال-prefixed words ending in ان.

FIX-34: Remove auto-re-analysis from Apply All in editor.js.
When spelling and grammar disagree (spelling strips ذهبن→ذهب,
grammar re-adds ذهبوا→ذهبن), the auto-re-analysis creates an
infinite loop of contradictory suggestions.

FIX-35: Spelling no longer strips valid conjugation suffixes.
The insertion fix was accepting ذهبن→ذهب (removing final ن) as
'extra char removed'. But ن is a feminine plural verb suffix.
Now blocks removal of suffix chars (ن,ت,ا,ي,ة,و,ه) from word
endings, preventing spelling from undoing grammar corrections.

src/app.py

@@ -1062,6 +1062,26 @@ def _is_small_spelling_change(orig_word, corr_word, vocab_manager=None):
                 _orig_oov = not vocab_manager.is_iv(orig_word)
                 _corr_iv = vocab_manager.is_iv(corr_word)
                 if _orig_oov and _corr_iv:
+                    # FIX-35: Don't strip valid conjugation suffixes.
+                    # If the "extra char" is at the END and is a common
+                    # Arabic verb/noun suffix, this is likely a grammar
+                    # conjugation, not a typo. Skip it.
+                    _CONJUGATION_SUFFIXES = {'ن', 'ت', 'ا', 'ي', 'ة', 'و', 'ه'}
+                    _removed_char = None
+                    for _di2 in range(len(orig_word)):
+                        if orig_word[:_di2] + orig_word[_di2 + 1:] == corr_word:
+                            _removed_char = orig_word[_di2]
+                            _removed_pos = _di2
+                            break
+                    if (_removed_char in _CONJUGATION_SUFFIXES
+                            and _removed_pos == len(orig_word) - 1
+                            and len(corr_word) >= 3):
+                        logger.info(
+                            f"[SPELLING] Rejected suffix strip: "
+                            f"'{orig_word}'→'{corr_word}' "
+                            f"(removing suffix '{_removed_char}' likely strips conjugation)"
+                        )
+                        return 0.0
                     logger.info(
                         f"[SPELLING] Insertion fix accepted (OOV→IV): "
                         f"'{orig_word}'→'{corr_word}' (extra char removed)"

src/js/editor.js

@@ -719,12 +719,12 @@ function applyAllSuggestions() {
     updateSuggestionsList([]);
     if (typeof showToast === 'function') showToast('✓ تم تطبيق ' + suggestions.length + ' تصحيح');
   } finally {
-    // FIX-32: Delay guard reset until AFTER re-analysis fires,
-    // preventing input events from triggering double analysis.
+    // FIX-32: Delay guard reset
     setTimeout(() => { _isApplyingSuggestion = false; }, 400);
   }
-  // P2/User Request: Auto re-analyze after applying all suggestions
-  setTimeout(() => { analyzeText(); }, 300);
+  // FIX-34: Do NOT auto-re-analyze after Apply All.
+  // This caused an infinite loop when spelling/grammar disagree.
+  // User can click "Analyze" manually if they want to re-check.
 }
 
 function clearEditor() {

src/nlp/grammar/grammar_rules.py

@@ -201,11 +201,41 @@ class ArabicGrammarGuard:
 
         return text
 
+    # FIX-33: Words ending in ان that are root-form nouns, NOT duals/plurals.
+    # These must never have ان→ين replacement.
+    _PREP_BLOCKLIST = {
+        'الامتحان', 'امتحان', 'الإنسان', 'إنسان', 'انسان', 'الانسان',
+        'الميدان', 'ميدان', 'البرلمان', 'برلمان', 'السلطان', 'سلطان',
+        'العنوان', 'عنوان', 'الديوان', 'ديوان', 'البستان', 'بستان',
+        'البنيان', 'بنيان', 'الإيمان', 'إيمان', 'ايمان', 'الايمان',
+        'الأمان', 'أمان', 'امان', 'الامان', 'العدوان', 'عدوان',
+        'البيان', 'بيان', 'البرهان', 'برهان', 'الشيطان', 'شيطان',
+        'الأذان', 'أذان', 'السودان', 'لبنان', 'عمان', 'الأردن',
+        'الحيوان', 'حيوان', 'القرآن', 'قرآن', 'الدخان', 'دخان',
+        'المكان', 'مكان', 'الزمان', 'زمان', 'الجدران', 'جدران',
+        'النيران', 'نيران', 'الألوان', 'ألوان', 'البلدان', 'بلدان',
+        'الأوطان', 'أوطان', 'الأبدان', 'أبدان', 'الأركان', 'أركان',
+        'الفرسان', 'فرسان', 'الغزلان', 'غزلان', 'القضبان', 'قضبان',
+    }
+
     def fix_prepositions_advanced(self, text):
         # Allow conjunctions (و، ف) before prepositions
         # (في المهندسون) -> (في المهندسين)
-        # Require stem >= 4 chars to avoid matching root-level ان endings (الامتحان, الإنسان, etc.)
-        text = re.sub(r'\b([وف]?(?:في|من|إلى|على|عن|حتى))\s+([أ-ي]{4,})(ون|ان)\b', r'\1 \2ين', text)
+        # FIX-33: Use callback to skip root-form nouns ending in ان
+        def _prep_replace(m):
+            prep = m.group(1)
+            stem = m.group(2)
+            suffix = m.group(3)
+            full_word = stem + suffix
+            # Skip words in blocklist (root nouns, not duals)
+            if full_word in self._PREP_BLOCKLIST:
+                return m.group(0)  # return unchanged
+            # Skip ال-prefixed words ending in ان — almost always root nouns
+            if stem.startswith('ال') and suffix == 'ان':
+                return m.group(0)  # return unchanged
+            return f'{prep} {stem}ين'
+
+        text = re.sub(r'\b([وف]?(?:في|من|إلى|على|عن|حتى))\s+([أ-ي]{4,})(ون|ان)\b', _prep_replace, text)
 
         # (وبالمبرمجون) -> (وبالمبرمجين)
         text = re.sub(r'\b([وف]?[بلكف])ال([أ-ي]{4,})(ون|ان)\b', r'\1ال\2ين', text)