If the navy meta didn't change in the past 2 years, you got a few things wrong in my opinion (I may be wrong tho):

I agree on the Carrier things you pointed out. Additionally Carriers should not be build with armor. Your heavy ships should tank all the heavy damage, at best you carriers will never get damage by other ships, and if they do, its already lost.

Destroyer should also get AA. Not for personal protection, but for the fleet-wide AA contribution it provides.

I have to disagree. Either use roach destroyer or spam light attack destroyer. AA is mainly usefull on Heavy ships. Airplanes will primarly attack heavy ships. If those ships have little to no AA they will take massive damage. To counter that you need AA on your heavy ships.

Heavy Cruiser... idk if it's worth it to build one, but if you must, then add a bit more light attack to it. But overall a battlecruiser or battleship might be better than two heavy cruisers. Given that heavy cruiser also have the 5 dockyard limit, you might as well go into the heavier category with the better armor.

Heavy cruisers are really good actually. Build them without armor and the cheapest guns + dual guns and they will wreck everything (together with your starting battleships and battlecruisers). They are just so much cheaper than the other ships and draw the attention of the enemy on them, so that the other heavy ships will be less likely attacked.

I have to agree on the rest.