Turns out the giveTheItem flag has never been implemented. Implemented it, and NPCs no longer keep the items they buy (as was already intended).