On a somewhat different note, the way the hill giant steading is set up really rubs me the wrong way. Maybe it's because it's the way hill giants have been presented in 5e, but the whole thing just strikes me as being far too sophisticated and civilized for hill giants. The steading comes across as an oversized barbarian chieftain's hut, with the giants living inside it just oversized barbarian humans. There are also several parts that come across as far too silly for my taste (like the "typical" giant kitchen that reads like an oversized human one). The hill giant fort in Storm King's Thunder is much more believable, in my opinion.
There's also some stuff that just doesn't make any sense to me:
- Why does the head hill giant servant (area 5) have a potion of mind control that can affect hill giants? (Also, the servants' room has no door on the map.)
- Why does the visiting frost giant leave his sentient magic sword in his room, hidden by an illusion? If that were my sword, I'd want to keep it with me at all times.
- Why do the giants have so many coins?!
- What's up with the orc slave rebellion? Is it a super-recent thing and the hill giant chief hasn't been notified about it yet? Does the Keeper not know/care about it? Surely the combined might of the giants could crush the rebellion easily. Why don't the orcs just leave? They've got access to the Underdark via that hole in the floor.
- Also, it says that the bugbear on watch west of the barricade keeps an eye out for activity to the east ... can he not see the orc guards on the east side? Do they just spend the time eyeing each other up or something? Wouldn't they at least trade threats and insults and stuff?
This whole setup just strikes me as being way too ridiculous. This particular adventure hasn't aged well, in my opinion.