feat(comparison): Add contains / containedBy methods to Comparison expression

[ElwynVdb]

Q	A
Type	improvement
BC Break	no
Fixed issues

Summary

• Added a static contains method to Comparison that generates an expression using the @> operator for containment checks.
• • Added a static containedBy method to Comparison that generates an expression using the <@ operator for containment checks.

[Copilot]

Pull request overview

Adds JSONB containment support to the expression layer by introducing a JsonbComparison::contains() helper that renders the PostgreSQL @> operator, with an accompanying integration test to validate expected query results.

Changes:

• Add JsonbComparison::contains(Expression $jsonField, Expression $contained) producing jsonField @> contained.
• Add integration coverage validating JSONB containment against a VALUES-backed dataset.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/Expression/JsonbComparison.php	Introduces contains() for JSONB containment using the @> operator.
tests/Integration/Expression/JsonbComparisonTest.php	Adds an integration test asserting correct behavior of contains().

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[veewee]

Thanks for the PR! The @> operator is useful, but I think JsonbComparison isn't the right place for it.

The @> (contains) operator in PostgreSQL isn't JSONB-specific — it works on arrays, ranges, hstore, and geometric types too. Compare this to ?| (has any keys), which is genuinely JSONB-only and belongs here.

I think Comparison is a better fit. It's already the home for binary comparison operators (=, !=, >, >=, <, <=), and @> / <@ follow exactly the same left operator right pattern. This way the methods work naturally regardless of the underlying type:

// jsonb
Comparison::contains(new Column('roles'), new SqlExpression("'[\"ROLE_USER\"]'::jsonb"));

// arrays
Comparison::contains(new Column('tags'), new SqlExpression("ARRAY['php']"));

Could you move contains to Comparison and also add containedBy for the <@ operator while you're at it?

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ElwynVdb]

The changes have been made, please let me know if anything else must change 😄