No, we really shouldn't. Enterprise generally does hardware refreshes every 3 years. The average consumer can take one of those machines and extend at least another 2 years out of it. Being able to securely wipe or remove storage modules out of the machines for repurposing should be a pretty easy target to hit.
Additionally, some industries like K12 or non-profits a lot of times have to try to squeeze as much life out of their systems as possible. These organizations would benefit insanely from better repairability and cheaper access to parts.
Of course motherboards are a bit harder to repair and usually means replacing the whole board, but it doesn't have to if we had a skilled workforce that could easily refurbish boards. Apple has taken great measures to make sure this never happens.
There's also the argument to be made for historic preservation. The retro computer community has been doing nothing but growing and I can already see in 20+ years time that these machines will be absolutely unusable without some sort of software hack. Even then, your onboard storage is shot and you have to boot externally.
You can make repairable, cost effective, well built, and top of the line machines. Apple just doesn't do it.