Skip to content

Document json(field,path) function#539

Open
br41nslug wants to merge 12 commits intomainfrom
json-query-docs
Open

Document json(field,path) function#539
br41nslug wants to merge 12 commits intomainfrom
json-query-docs

Conversation

@br41nslug
Copy link
Member

@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 2, 2026 6:22pm

Request Review

@br41nslug br41nslug changed the title start on docs for json queries [WIP] json query docs Jan 19, 2026
@ComfortablyCoding
Copy link
Member

ComfortablyCoding commented Feb 27, 2026

  • object keys with special characters (e.g. $,.) while valid key names are not supported.
  • Numbers may come back as string depending on the db (e.g. mssql)
  • sqlite returns false as 0

@br41nslug br41nslug changed the title [WIP] json query docs json query docs Mar 2, 2026
@br41nslug br41nslug marked this pull request as ready for review March 2, 2026 17:17
@br41nslug br41nslug requested a review from a team as a code owner March 2, 2026 17:17
Comment on lines +168 to +176
The JSON path (second argument) also has its own configurable limit:

```
MAX_JSON_QUERY_DEPTH=10 # default
```

Depth is counted by the number of `.` and `[` characters in the normalized path. A path like `a.b.c.d.e.f.g.h.i.j` has depth 10 and is allowed by default; adding one more segment exceeds the limit.

Exceeding this limit returns an error.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep the format for this section inline with Relational Depth Limit

br41nslug and others added 2 commits March 2, 2026 19:17
Co-authored-by: judda <44623501+ComfortablyCoding@users.noreply.github.com>
Co-authored-by: judda <44623501+ComfortablyCoding@users.noreply.github.com>
@ComfortablyCoding ComfortablyCoding changed the title json query docs Document json(field,path) function Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants