Remove support for unknown attributes.

Author: nkanu17

redisvl/schema/fields.py

@@ -107,16 +107,15 @@ def _normalize_field_modifiers(
 
     RediSearch has a parser limitation (redis/redis#5177) where INDEXEMPTY and
     INDEXMISSING must appear BEFORE SORTABLE in field definitions. This function
-    reorders field.args_suffix to match the canonical order while preserving
-    unknown modifiers at the start.
+    reorders field.args_suffix to match the canonical order.
 
     Args:
         field: Redis field object whose args_suffix will be normalized
         canonical_order: List of modifiers in desired canonical order
         want_unf: Whether UNF should be added after SORTABLE (default: False)
 
     Time Complexity: O(n + m) where n = len(field.args_suffix), m = len(canonical_order)
-    Space Complexity: O(n + m)
+    Space Complexity: O(n)
 
     Example:
         >>> field = RedisTextField("title")
@@ -126,12 +125,9 @@ def _normalize_field_modifiers(
         ['INDEXMISSING', 'SORTABLE']
     """
     suffix_set = set(field.args_suffix)
-    known_set = set(canonical_order)
 
-    # Preserve unknown modifiers in original order
-    new_suffix = [t for t in field.args_suffix if t not in known_set]
-
-    # Add known modifiers in canonical order
+    # Build new suffix with only known modifiers in canonical order
+    new_suffix = []
     for modifier in canonical_order:
         if modifier in suffix_set:
             new_suffix.append(modifier)

tests/unit/test_field_modifier_ordering.py

@@ -252,15 +252,7 @@ def test_basic_reordering(self):
 
         assert field.args_suffix == ["INDEXMISSING", "SORTABLE"]
 
-    def test_preserves_unknown_modifiers(self):
-        """Test that unknown modifiers are preserved at the start."""
-        field = RedisTextField("test")
-        field.args_suffix = ["CUSTOM", "SORTABLE", "INDEXMISSING"]
-        canonical_order = ["INDEXMISSING", "SORTABLE"]
-
-        _normalize_field_modifiers(field, canonical_order)
 
-        assert field.args_suffix == ["CUSTOM", "INDEXMISSING", "SORTABLE"]
 
     def test_unf_added_with_sortable(self):
         """Test that UNF is added when want_unf=True and SORTABLE is present."""
@@ -318,25 +310,7 @@ def test_empty_suffix(self):
 
         assert field.args_suffix == []
 
-    def test_only_unknown_modifiers(self):
-        """Test with only unknown modifiers."""
-        field = RedisTextField("test")
-        field.args_suffix = ["CUSTOM1", "CUSTOM2"]
-        canonical_order = ["INDEXMISSING", "SORTABLE"]
-
-        _normalize_field_modifiers(field, canonical_order)
-
-        assert field.args_suffix == ["CUSTOM1", "CUSTOM2"]
-
-    def test_multiple_unknown_modifiers_preserved_order(self):
-        """Test that multiple unknown modifiers preserve their original order."""
-        field = RedisTextField("test")
-        field.args_suffix = ["CUSTOM1", "SORTABLE", "CUSTOM2", "INDEXMISSING"]
-        canonical_order = ["INDEXMISSING", "SORTABLE"]
-
-        _normalize_field_modifiers(field, canonical_order)
 
-        assert field.args_suffix == ["CUSTOM1", "CUSTOM2", "INDEXMISSING", "SORTABLE"]
 
 
 class TestMLPCommandsScenario: