Your basic errors were:

Performing an undocumented mechanical edit (the text at osm.wiki/Mechanical_Edits/Mateusz_Konieczny_-_bot_account/fixing_malformed_shop_tags does not cover this case).

Thinking that changing "rare tag I do not understand" into "common but meaningless tag" added any value.

Continuing with these ridiculous edits despite being told that they were damaging (and after you yourself had reverted previous examples after previous complaints!).

Not using any common sense when trying to work out what a sensible value might be. Did you ask anyone? No. Did you investigate? No (a search locally for shops with the name Lighthouse will find a number of "Lighthouse Charity Shop" examples).

I can guarantee that a mention on IRC that you were going to change a shop in Nottingham with the name Lighthouse into a nondescriptive value would have got an immediate response.

Did you even read https://lists.openstreetmap.org/pipermail/talk/2023-April/088189.html ?

As I have said on the mailing list already, local knowledge. Also, as I have also said on the mailing list, a quick overpass search finds lots of other similarly named shops tagged as "shop=charity" nearby.

I could forgive a completely new OSM contributor with making this kind of basic error, but am genuinely saddened that someone with your level of experience with OSM is making these kind of basic mistakes.

Note for the avoidance of doubt that the change here is NOT documented by osm_wiki_documentation_page osm.wiki/Mechanical_Edits (there is no mention of fixme on that page) despite a changeset tag suggesting that it is. This is essentially an undiscussed mechanical edit.

> , but it is also not worse than before.

untrue.

A new "shop=yes" will just get lost in the other 180k "shop=yes". We know they won't get spotted because the other 180k of them have not been.

Updating something that you do not even know exists with a "new" value means something that looks at the last modified date might think that it has been verified recently when it has not.

Your actions here are making it harder for local mappers to improve the quality of data in OSM. You should stop now.

Hello,
I can probably guess "SAU" (Saudi Arabia) and "YEM" (Yemen) here, but what are "GHSL" and "EC" here? "add villages and details" is of course perfectly descriptive.
Best Regards,
Andy

(although how "Egypt" is relevant to a change in Oman I have no idea)

Please, using meaningful changeset comments. Assuming that "#EGY" here actually means "Egypt", why not use that one extra character and have people understand what you mean, as opposed to not doing and leaving them guessing?

Are you sure that node/2361007180/history is still a shop at all? What action did you take to verify it?
If none, how does what you have done here improve the quality of OSM?

Clearly the change to e.g. way/353944525 is causing information to be lost here (not much information, but still something).
The correct tag here would be "shop=charity".

With node/6813245577/history it might make sense to talk to the original contributor. Clearly shop=yes is not a good replacement here.

You've replaced a few like node/6590059464/history , but is the replacement here perhaps less precise?

A heads-up might have been nice here. Do the subtags need on node/1178915529/history need to be changed or discussed anywhere?

See osm.org/user_blocks/7131 .

Oh dear. See See osm.org/user_blocks/7131 . .

Thanks

Alas, the "East Riding" added as a subarea to the traditional county of Yorkshire here is the wrong one, it's the modern unitary authority relation/88083 , noth the old Riding which way/56150279 would be on the boundary of.

Thanks!

Just to note another issue with this data, node/501242848/history shows "very rounded" and likely inaccurate coordinates: 7.5833330, 11.2500000 .

Also, the ways that have been added to the superrelation relation/1992599 need removing.