Skip to content

Remove redundant JSON validation and reduce collection allocations - #1

Merged
kelindar merged 2 commits into
mainfrom
remove-json-check
Sep 19, 2026
Merged

kelindar merged 2 commits into
mainfrom
remove-json-check

Conversation

@kelindar

Copy link
Copy Markdown
Owner

Summary

  • Remove redundant JSON validation from boolean evaluation and defer validation for JSON programs
  • Reuse []any inputs directly and avoid duplicate intersection tracking
  • Add regression tests and benchmarks for validation, ownership, and collection operations

Testing

Not run (not requested)

@github-actions

github-actions Bot commented Sep 19, 2026 •

Copy link
Copy Markdown
100.0%

 passed

████████████████████ 100.0% on changed lines

11 lines changed   0 uncovered   2 files

 📂 2 files changed
  File Coverage Uncovered Lines
🟢 expr.go 100.0% —

| 🟢 | collection/collection.go | 100.0% | — |

 📋 Full diff-cover report

Diff Coverage

Diff: origin/main...HEAD, staged and unstaged changes

  • collection/collection.go (100%)
  • expr.go (100%)

Summary

  • Total: 11 lines
  • Missing: 0 lines
  • Coverage: 100%

🛡️ diff-cover-action

@kelindar
kelindar merged commit c0fedfd into main Sep 19, 2026
1 check passed
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.

1 participant