cedar-policy / cedar

Implementation of the Cedar Policy Language
https://www.cedarpolicy.com
Apache License 2.0
897 stars 80 forks source link

(Re)consider where we use `AnyId` vs `Id`, ie, where we allow reserved words vs not #635

Open cdisselkoen opened 9 months ago

cdisselkoen commented 9 months ago

Describe the improvement you'd like to request

[Comes from comments on #634]

We should reconsider if there are more places we should use AnyId where we currently use Id, ie, if there are more places we should allow Cedar reserved words to be used when an identifier is needed. #634 allows Cedar reserved words to be annotation keys, maybe there are more places we could/should allow Cedar reserved words to be used where currently they are not.

Of course, keeping them disallowed is somewhat more future-proof in case we have a need to disallow them in the future, due to new Cedar features or syntax.

Describe alternatives you've considered

Status quo is probably OK

Additional context

No response

Is this something that you'd be interested in working on?

cdisselkoen commented 5 months ago

Related: #407