Luke informed us that this used to throw an exception, though I haven’t tested to find out which versions that applies to.

Nigel and I think “name” just shouldn’t be allowed if there’s a declared namevar like “path;” it’s not introspectible, and amounts to a secret backdoor that leads to illegible Puppet code. Luke said he doesn’t have a philosophical problem with doing that, but he believes we should do a real deprecation instead of silently fixing it, since it’s a change in behavior that might cause some unknown amount of havoc out in the wild.