Johan Fylling

To reduce ambiguity, the safer approach is probably to not allow annotations to re-declare a schema with conflicting ID/URL. --- Applying `schema_defs` globally rather than according to its associated scope...

Setting the cache size to `100` was made with the assumption that a large set of regex patterns is very uncommon, and that slow performance is a better tradeoff than...

Raising the glob cache size to somewhere in the thousands should be benign. What would be a suitable limit for your use case @amirsalarsafaei? We still might want this to...

@amirsalarsafaei, for paths that don't require mid-point `**` globs, I think your solution looks very idiomatic for Regal :).

Raising the cache limit too far would bring us back to the memory issue previously solved. I think we should aim for a solution that accommodates most scenarios without being...

I think this can be worthwhile investigating. When the output from a non-deterministic built-in is reused, optimization is definitely not being optimal when it repeats the call rather than reuse...

I would need to put some time to familiarize myself a bit more with this issue to be able to give informed pointers. Let me get back to you on...

> There was a previous issue about this (suggesting //, IIRC) somewhere, but I can't find it now. https://github.com/open-policy-agent/opa/issues/2345 discusses introducing an **alternative** (`//`) operator. A possible issue with the...

> If you want a boolean, you'd do has_phone_number if input.phone_numbers.work || input.phone_numbers.work For sure, that's how you'd do it for a rule 👍. Wouldn't work inside a body, though....

> That's true, but you could argue that there's a way to make it boolean Absolutely 👍 . I'm not arguing against the usefulness of an **alternative** operator; it seems...