But is there a reason why you make %attr a package-var?

In case a sub-class wants to inspect (or extend) my fields. I'm in the opposite camp as Ovid.