Skip to content

feat(evaluator): implement walk builtin - #187

Open
sspaink wants to merge 1 commit into
open-policy-agent:mainfrom
sspaink:feat/walk-builtin
Open

feat(evaluator): implement walk builtin#187
sspaink wants to merge 1 commit into
open-policy-agent:mainfrom
sspaink:feat/walk-builtin

Conversation

@sspaink

@sspaink sspaink commented Aug 7, 2026

Copy link
Copy Markdown
Member

Fixes #141

Implements the walk builtin. 14 previously-skipped walk compliance fixtures now run and pass.

@sspaink
sspaink requested a review from a team as a code owner August 7, 2026 22:34
@sspaink
sspaink force-pushed the feat/walk-builtin branch from aed365b to 470d3d9 Compare August 7, 2026 22:39
@sspaink
sspaink force-pushed the feat/walk-builtin branch from 470d3d9 to 949ce46 Compare August 7, 2026 22:58
Go-OPA flags walk as its only relational builtin (Relation: true): it
yields many [path, value] tuples through an iterator instead of
returning a single value. The IR planner lowers it to a plain call plus
a scan over the result, so this returns the whole set of pairs. Sets
are keyed by their own members.

Also fixes composite object keys in opa-jackson, which used Java's
collection formatting instead of compact JSON.

Fixes open-policy-agent#141

Signed-off-by: Sebastian Spaink <[email protected]>
@sspaink
sspaink force-pushed the feat/walk-builtin branch from 949ce46 to f3716a8 Compare August 10, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement walk builtin

1 participant