transtemporal
Explorer
Looking at the build order, if you're wanting a weapon + shield fighter, I suggest taking the Shieldmaster feat at level 4, not level 16.
Personally, I think maxing out the primary attack stat takes precedence and assuming the standard stat array, you can't do that until at least level 8. +2 attack bonus for all attacks seems to have more utility than a situational bonus to one saving throw. Although to be honest, I haven't thought it through that thoroughly so happy to take advice on it, lol.
My reasoning for taking Alert and Sentinel before Shieldmaster is that melee tanking is really the focus of the build and those two feats are key to that*. Although I might swap Lucky for Shieldmaster in the build order. Thats a little too long to be taking breath weapons to the face...
* taking Alert is really dependent on whether your DM allows the old Delay action. If thats a possibility then you could drop the feat and get party members higher in the initiative order to delay their actions until after you.