Hello,
thanks for the feedback.
isn't the check already done by the previous contributor  ? I have changed the value "unattended" to the more common "vaccant" because unattended have 2 meanings (the other meaning is a shop open but without staff/control), see previous and only contributor for shop=unattended  https://www.openstreetmap.org/changeset/130756523
it is indeed possible to change shop=vaccant to disused:shop=*, however this is another problem that is not specific to this contributor.
let's me known if you want I revert it, I find it a step in the right direction though (as well as the correction of buildings inside a building)
Regards,
Marc