Delete artefacts from search index when changed to nonindexed kinds

We have had some artefacts created (incorrectly) as Answers when they should
have been CompletedTransactions. These were not removed automatically. This
change would ensure that they are in future.

I haven't written any tests for which I feel bad, though I have tested by hand. I did try moving this logic into RummageableArtefact to make it more testable, but the use of ActiveModel's dirty tracking (``kind_was``) was a bit too hidden and distant, whereas in the context of an observer it feels sane.