ExploderWizard
Hero
I completely agree.
See below.
I understand needing/wanting some information and detail about the village and villagers. But really:
Bold emphasis in the original text.Bold emphasis in the original text.
Is the combat information necessary? Is the bold emphasis on the people's hidden valuables necessary? (The emphasis on their treasure always struck me as odd.) I think EGG did a better job with "town" NPCs in Keep on the Borderland.
Bullgrit
Combat info - yup. What if the village is attacked and the PC's must organize the militia to fight? What if the PC's are evil and want to kill them?
Valuables info-yup. If the PC's do kill them they are gonna want to take thier stuff.

Aside from that knowing the resources available in an area can become important in a lot of ways.
What if a bad guy kidnapped the leatherworker's wife or child, knowing about the necklace and demanding it as ransom? He could come to the PC's for help to get his wife back. The existence of that necklace could be important to how things play out. Maybe the leatherworker is a coldhearted bastard and pretends he doesn't have the necklace? Searching his place and finding it would matter.
The presentation is just solid dungeon stocking technique.
1) Define monsters, traps/tricks.
2) Place treasure.
The DM can make those details as meaningful as desired for his/her campaign.