As discussed on IRC, I think we could have a comment here to explain why we use an eigenclass variable in the other barclamps. So maybe change to:
```
  # Use a class variable so that it can be dynamically extended
  @role_constraints ||= begin
    {}
  end
```