Add test cases for JSON null values and fix validator/dialect handling

[Copilot]

The test suite didn't cover JSON objects or arrays containing null values as elements (e.g., { key: null } or [1, null, 3]). Adding these tests revealed bugs in both the validator and SQLite dialect.

Changes

• Test cases: Added works with JSON objects containing null values and works with JSON arrays containing null values tests covering creation, equality filters, and not filters

• Validator fix (validator/index.ts): makeJsonValueSchema now allows null elements in JSON arrays, matching the JsonArray type definition

  // Before: didn't allow null in array elements
  z.lazy(() => this.makeJsonValueSchema(false, false).array())
  
  // After: allows null elements like object values do
  z.lazy(() => z.union([this.makeJsonValueSchema(false, false), z.null()]).array())

• SQLite dialect fix (dialects/sqlite.ts): transformPrimitive now handles JSON type before the array check, ensuring JSON values are serialized as a whole rather than element-by-element (which would cause SQLite binding errors for arrays with null elements)

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ymc9]

@copilot where're your changes?

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• checkpoint.prisma.io
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/zenstack-v3/zenstack-v3/node_modules/.pnpm/prisma@6.19.0_magicast@0.3.5_typescript@5.9.3/node_modules/prisma/build/child {"product":"prisma","version":"6.19.0","cli_install_type":"local","information":"","local_timestamp":"2025-12-04T10:54:30Z","project_hash":"302ae6b3","cli_path":"/tmp/tmp-5741-NwbV6Sd0WbHr/node_modules/prisma/build/index.js","cli_path_hash":"410c208a","en sh (dns block)
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/zenstack-v3/zenstack-v3/node_modules/.pnpm/prisma@6.19.0_magicast@0.3.5_typescript@5.9.3/node_modules/prisma/build/child {"product":"prisma","version":"6.19.0","cli_install_type":"local","information":"","local_timestamp":"2025-12-04T10:54:33Z","project_hash":"09e8a64b","cli_path":"/tmp/tmp-5741-uLAvJyFLUg61/node_modules/prisma/build/index.js","cli_path_hash":"4404d1c1","en node k/ze�� ts son-filter.test.ts h --verify feat/json-nulls--c tools/pnpm/10.23tsc uname px ilter.test.ts git node tsup-node origin tnet/tools/sh node (dns block)
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/zenstack-v3/zenstack-v3/node_modules/.pnpm/prisma@6.19.0_magicast@0.3.5_typescript@5.9.3/node_modules/prisma/build/child {"product":"prisma","version":"6.19.0","cli_install_type":"local","information":"","local_timestamp":"2025-12-04T10:54:36Z","project_hash":"f23ce6a7","cli_path":"/tmp/tmp-5741-1TK3jRpo6hlA/node_modules/prisma/build/index.js","cli_path_hash":"dbcc54c7","en node ites�� lient-api/json-filter.test.ts ts de --noprofile (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[Copilot]

@copilot where're your changes?

The changes are in commit ba45db2. Here's what was changed:

1. Test cases (tests/e2e/orm/client-api/json-filter.test.ts): Added two new tests:

   • works with JSON objects containing null values
   • works with JSON arrays containing null values

2. Validator fix (packages/orm/src/client/crud/validator/index.ts): Updated makeJsonValueSchema to allow null elements in JSON arrays

3. SQLite dialect fix (packages/orm/src/client/crud/dialects/sqlite.ts): Moved JSON type handling before the array check to ensure proper serialization