[Bug 🐞] Pydantic `EmailStr` Fields fail to work with redis-om find queries

[XChikuX]

from redis_om import HashModel, Field
from pydantic import EmailStr, ConfigDict

class Email(HashModel):
    email: EmailStr = Field(
        index=True, title="Email", description="Enter your email address."
    )

    class Meta:
        database = redis_conn

    model_config = ConfigDict(validate_assignment=True, str_strip_whitespace=True)

    @classmethod
    async def migrate(cls):
        await Migrator().run()

The above definition fails when used for a query:

await Email.find(Email.email == email).first()

Gives:

 File "/root/code/backend/app/query.py", line 174, in login                                                                                                                        
   user = await RedisUser.find(RedisUser.email == email).first()                                                                                                                   
                                       ^^^^^^^^^^^^^^^                                                                                                                                     
    File "/root/code/backend/.env/lib/python3.12/site-packages/pydantic/_internal/_model_construction.py", line 271, in __getattr__                                                   
   raise AttributeError(item)                                                                                                                                                      
    AttributeError: email     

redis-om version: v1.0.2-beta
pydantic: v2

[XChikuX]

This used to work with pydantic v1 with redis-om v0.3.3
@abrookins #691

[kaidotio]

I had the same problem, but in my environment I was able to avoid the error by adding index=True as follows.

class Email(HashModel, index=True):

There seem to be some breaking changes since 1.0.0-beta.

[XChikuX]

@kaidotio , thanks for the workaround! :)

I wouldn't call a bug a breaking change though. It's just a bug.

[abrookins]

Cannot reproduce with current main branch.

Tested both HashModel and JsonModel with EmailStr fields:

class EmailHashModel(HashModel, index=True):
    email: EmailStr = Field(index=True)

class EmailJsonModel(JsonModel, index=True):
    email: Optional[EmailStr] = Field(index=True, default=None)

# Both work correctly:
await EmailHashModel.find(EmailHashModel.email == "test@example.com").all()
# Returns: [EmailHashModel(pk='...', email='test@example.com')]

await EmailJsonModel.find(EmailJsonModel.email == "json@example.com").all()
# Returns: [EmailJsonModel(pk='...', email='json@example.com')]

The issue was reported on v1.0.2-beta and has been fixed in current main. Closing as resolved.