sqlite,src: refactor sqlite value conversion

[geeksilva97]

This PR is a minor refactor on converting SQLite values to JS. It makes the work for #56600 easier and reduces duplication.

[nodejs-github-bot]

Review requested:

• @nodejs/sqlite

[codecov]

Codecov Report

Attention: Patch coverage is 87.50000% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.23%. Comparing base (1de917b) to head (94276bc).
Report is 45 commits behind head on main.

Files with missing lines	Patch %	Lines
src/node_sqlite.cc	87.50%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #57571      +/-   ##
==========================================
- Coverage   90.23%   90.23%   -0.01%     
==========================================
  Files         629      630       +1     
  Lines      184903   185009     +106     
  Branches    36223    36221       -2     
==========================================
+ Hits       166853   166944      +91     
- Misses      11010    11038      +28     
+ Partials     7040     7027      -13     

Files with missing lines	Coverage Δ
src/node_sqlite.cc	78.20% <87.50%> (-0.29%)	⬇️

... and 66 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/65862/

[Renegade334]

Is this "returning function body macro" paradigm used elsewhere? Appreciate the rationale, but it seems like it lends potential for future mishaps?

[geeksilva97]

but it seems like it lends potential for future mishaps?

For this, specifically, I don't think so. This type conversion is unlikely to change. We need to find a way to isolate this anyways since it's being required more and more.

[cjihrig]

LGTM with one minor comment.