Sorry, I wasn't very clear there in my last point! I meant why did you choose to change it from a highway=footway to a *=cycleway? I didn't mean to question adding either foot=* or bicycle=* (as appropriate), just the primary highway=* tag.
I'm finding a lot of preference for using highway=cycleway on shared pavements, but legally the cyclist has to give way to pedestrians so makes a lot more sense in my mind to keep them as highway=footway, footway=sidewalk,bicycle=yes/designated,segregated=no. It more accurately represents both de facto and de jure statuses, and helps to promote cycle infrastructure (no evidence here!) by not overstating the density of cycleways.
On this example specifically, there aren't any signs that this portion is a shared pavement, and indeed the open data just says it's a quiet road recommended for cyclists, so probably worth reverting it unless you know something else? I'm happy to do it bundled in with other junction modifications if you're not bothered about it?