https://www.openstreetmap.org/way/416176771/history has shop = cigars

Why not shop=tobacco (with tobacco=cigars), see https://wiki.openstreetmap.org/wiki/Tag:shop=tobacco ?

Do we really need both shop=tobacco and shop=cigars as top level values? Having separate top level values for minor variations makes using OSM data harder.

If shop=cigars should exist, documenting it at https://wiki.openstreetmap.org/wiki/Tag:shop=cigarettes would be useful (see https://wiki.openstreetmap.org/wiki/Creating_a_page_describing_key_or_value )