Skip to content

fix: delete toJson parser print#1293

Merged
grdsdev merged 1 commit into
supabase:mainfrom
NazarenoCavazzon:fix/toJson-print
Dec 29, 2025
Merged

fix: delete toJson parser print#1293
grdsdev merged 1 commit into
supabase:mainfrom
NazarenoCavazzon:fix/toJson-print

Conversation

@NazarenoCavazzon
Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Bug fix.

What is the current behavior?

The toJson prints "errors" while decoding json, even though those errors doesn't exist.

Example:

image image

What is the new behavior?

Now the toJson handles the errors as expected, with the intentional decode-and-fallback logic.

Additional context

The toJson function correctly attempts to decode JSON strings and falls back to the raw value on failure. However, it logs every parse failure, which pollutes console output for valid use cases (e.g., jsonb columns containing unquoted strings from Postgres realtime).

This removes the unnecessary print statement while preserving the intended try-decode-or-fallback behavior.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 20565296177

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 79.952%

Totals Coverage Status
Change from base Build 20319197891: 0.02%
Covered Lines: 3362
Relevant Lines: 4205

💛 - Coveralls

@grdsdev grdsdev merged commit 0d68e50 into supabase:main Dec 29, 2025
15 checks 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.

3 participants