database/gorm/query.go:943

• Consider adding unit tests for the new WhereJson* and OrWhereJson* methods to validate that SQL is generated as expected for various JSON path scenarios.

func (r *Query) WhereJsonContains(column string, value any) contractsorm.Query {

database/gorm/query.go:1377

• The reflect.ValueOf call requires importing the reflect package; add import "reflect" at the top of this file.

if val := reflect.ValueOf(args[0]); val.Kind() == reflect.Slice || val.Kind() == reflect.Array {

database/schema/wrap.go:95

• Default quoting in JsonPathAttributes uses r.Quote (double quotes), but the tests expect single-quoted values ('column'). Adjust the default to wrap with single quotes or update the tests to match double-quote behavior.

return r.Quote(v)

database/schema/wrap_test.go:68

• Typo in helper name: spiltPath should be splitPath. Also remove the empty if block inside this function.

spiltPath := func(v string) []string {

database/schema/wrap.go:63

• [nitpick] Regexps are being compiled on every call. Consider precompiling these patterns to package-level var so they aren’t recompiled repeatedly.

value = regexp.MustCompile(`(\\+)?'`).ReplaceAllString(value, "''")

contracts/database/driver/grammar.go:166

• You’ve extended the Grammar contract with JsonGrammar, but the default driver grammar implementations (e.g., MySQL, Postgres) must also implement these new methods (CompileJsonContains, CompileJsonContainsKey, etc.). Add their implementations or tests will fail to compile.

type JsonGrammar interface {