Any reason not to do puppet('agent')['node_name_value'].strip?

I'd just be concerned that adding the default value to def puppet could cause side effects elsewhere...