sqlite: validate maxSize argument in createTagStore()

[Anshikakalpana]

Fixes: #63791

database.createTagStore() accepted invalid values for its maxSize argument without throwing. Negative integers caused integer overflow, NaN and floats produced garbage capacity values, and strings were silently ignored.

The maxSize parameter is documented as {integer} and represents a cache size, so negative values are meaningless.

This PR adds validation to reject:

• Non-integer values (NaN, floats, strings) with ERR_INVALID_ARG_TYPE
• Negative integers with ERR_OUT_OF_RANGE

Edit: non-integer values (NaN, floats, strings) no longer throw ERR_INVALID_ARG_TYPE.

[nodejs-github-bot]

Review requested:

• @nodejs/sqlite

[aduh95]

This should likely be ERR_OUT_OF_RANGE, like e.g. node -e 'child_process.spawn("/dev/null", { uid: 1.3 })' does

[Anshikakalpana]

Fixed — floats now throw ERR_OUT_OF_RANGE instead of ERR_INVALID_ARG_TYPE.

[trivikr]

@Anshikakalpana Can you update the PR description to include this change?
The non-integer values (NaN, floats, strings) no longer throw ERR_INVALID_ARG_TYPE

[Anshikakalpana]

Hi @aduh95! I addressed all the review comments and updated the PR. Could you please take another look when you have a chance? Thanks!

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.30%. Comparing base (8fc7341) to head (6371aa2).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #63792      +/-   ##
==========================================
- Coverage   90.31%   90.30%   -0.02%     
==========================================
  Files         759      759              
  Lines      248288   248302      +14     
  Branches    46857    46858       +1     
==========================================
- Hits       224247   224223      -24     
- Misses      15471    15514      +43     
+ Partials     8570     8565       -5     

Files with missing lines	Coverage Δ
src/node_sqlite.cc	81.13% <100.00%> (+0.08%)	⬆️

... and 34 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.