It is finding an actual annotation though so I don't see why it shouldn't be listed. The more important question is should we be allowing empty annotations to be added in the first place ?

That's basically the way we delete an annotation (we save all the previous versions of an annotation, so once an entity has an annotation, the only way to get rid of it is to remove all the text).

Ok I can either remove them or leave them in. I don't thinks its practical to leave them in but make it easy to filter them out, the only way I can think to do this is is to introduce a new field, one field contains all annotationa including empty ones, the other one doesnt include empty ones but sounds like overkill.

I'm still in favour of just removing them.

FIxed with tests, empty annotations no longer indexed.