[Bug]: Empty IDs don't fail validation

[itaismith]

What happened?

Currently we allow user-provided empty IDs:

collection.add(ids=[""], documents=["hello"])

Versions

Chroma v1.0.0

Relevant log output

[hesreallyhim]

@itaismith i would guess this still has to be unique, though, right? like there can only be one id = ""? just asking to get more context

EDIT: Confirmed

[jairad26]

fixed here https://github.com/chroma-core/chroma/pull/4488