Philip Conrad

ND builtins now are cached entirely through logic in the evaluator. :smile: Builtins can now mark themselves as non-deterministic via a new per-builtin property (`Nondeterministic`), and that property triggers the...

I think the commit stack has grown a bit massive, so to reduce the thread clutter from future rebases, I'm going to do a squash rebase.

Once the Github Codespaces outage resolves, we'll need to kick the CI jobs for this PR again. They *should* come up green, based on local checks, but you never know....

After seeing all of the CI checks go green, I've preemptively squashed my commit chain down to make it easier to merge later.

This *might* be on the edge of "good first issue" territory. It's a well-scoped change to a single builtin. I think the only tricky part would be adding a testcase...

After taking a look at the docs text, I can see what @srenatus is getting at. `policy` isn't a particularly necessary part of that documentation addition. I think if we...

@carabasdaniel You're correct. The test failure is unrelated to these changes. Our MacOS tests have been a bit flaky recently. We'll re-run the failing CI tests, and this PR should...

I'll need to check the Golang version requirements for this, but we might be able to merge this library with minimal hassle if there's nothing breaking in the changes list.

@dependabot rebase

Thanks for taking a look at the EBNF grammar, @shaded-enmity! :smile: The parser in OPA is not generated directly from tools, so it's possible for the grammar definition docs to...