[RaphaelFakhri]

Motivation

With index_missing: True on a vector field, SearchIndex.create() fails. field_data put "INDEXMISSING": True inside the vector attribute block, so redis-py raises Invalid input of type: 'bool', and the server rejects the token there anyway (Bad arguments for algorithm HNSW: INDEXMISSING). FT.CREATE expects INDEXMISSING after the attribute block, which is where redis-py places it for its index_missing keyword. The review on #742 described this failure.

Fixes #770.

Changes

• redisvl/schema/fields.py: drop INDEXMISSING from BaseVectorFieldAttributes.field_data, and pass index_missing=self.attrs.index_missing to the redis-py VectorField in FlatVectorField, HNSWVectorField and SVSVectorField.as_redis_field(). This matches how the other field types handle the modifier.
• tests/unit/test_fields.py: replace the test that asserted INDEXMISSING in field_data with a parametrized test (FLAT, HNSW, SVS-VAMANA) that checks INDEXMISSING follows the attribute block in redis_args().
• tests/integration/test_field_modifier_ordering_integration.py: create FLAT and HNSW indexes with a vector index_missing field and check that ismissing(@embedding) returns only the document without the vector.

from_existing still doesn't recover index_missing on vector fields (see the comment in tests/unit/test_convert_index_info.py). That can follow separately now that the field can be recreated.

Release Notes

Vector fields declared with index_missing: true can now be created, and ismissing(@field) queries work on them.

Review record

This change was written with Claude Code (Claude Opus 5.5) and reviewed by a human before submission.

Reviewed at c5213f9 by Claude Code (Claude Opus 5.5) with pre-pr-review.
Lenses: correctness (always); tests (adds unit and integration tests); Redis behaviour (changes where FT.CREATE gets INDEXMISSING). Skipped: security (no query-building, MCP or destructive paths); design (small fix, no API change); docs (no docs or docstrings change).

• 3 findings raised, 0 kept after challenge.
• Fixed: none.
• Rejected: removing INDEXMISSING from field_data could break a caller that reads it (no caller outside fields.py reads field_data); the index_missing keyword might not exist on the redis-py floor (redis-py 6.3.0 accepts it and emits INDEXMISSING after the block, checked directly); SVS-VAMANA has no integration case (the unit test covers its argument order, and the code path is shared with FLAT and HNSW).
• Open for the reviewer: none.

Verified: pytest tests/unit/test_fields.py tests/integration/test_field_modifier_ordering_integration.py tests/unit/test_convert_index_info.py (85 passed, Redis 8.4); same two test files with fields.py from main (5 failed, the new cases); pytest tests/unit --ignore=tests/unit/test_mcp (1345 passed, 1 skipped); make check-types (no issues); black and isort --profile black on the changed files (no changes).
Docs consulted: https://redis.io/docs/latest/commands/ft.create/
Not checked: cluster mode; SVS-VAMANA against a live server; Redis versions other than 8.4.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[vishal-bala]

Hi, thanks for your contribution! Can you rebase onto the latest origin/main and follow the new contribution process?

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[RaphaelFakhri]

Rebased onto the latest main, and the description now follows the new template with a Review record for c5213f9.