Skip to content

sqlite: refactor error helpers and user function pointers#62794

Open
thisalihassan wants to merge 1 commit into
nodejs:mainfrom
thisalihassan:sqlite-refactor-error-and-udf-ptrs
Open

sqlite: refactor error helpers and user function pointers#62794
thisalihassan wants to merge 1 commit into
nodejs:mainfrom
thisalihassan:sqlite-refactor-error-and-udf-ptrs

Conversation

@thisalihassan

Copy link
Copy Markdown
Contributor

Two small independent refactors.

  1. Consolidate CreateSQLiteError overloads
  2. BaseObjectWeakPtr for user functions (style consistency)

Signed-off-by: Ali Hassan <ali-hassan27@outlook.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/sqlite

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. sqlite Issues and PRs related to the SQLite subsystem. labels Apr 17, 2026
@codecov

codecov Bot commented Apr 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.88889% with 4 lines in your changes missing coverage. Please review.
βœ… Project coverage is 89.68%. Comparing base (4e612c0) to head (3162244).
⚠️ Report is 224 commits behind head on main.

Files with missing lines Patch % Lines
src/node_sqlite.cc 88.88% 1 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #62794      +/-   ##
==========================================
- Coverage   89.69%   89.68%   -0.02%     
==========================================
  Files         706      706              
  Lines      218247   218252       +5     
  Branches    41777    41782       +5     
==========================================
- Hits       195760   195730      -30     
- Misses      14401    14438      +37     
+ Partials     8086     8084       -2     
Files with missing lines Coverage Ξ”
src/node_sqlite.h 80.64% <ΓΈ> (ΓΈ)
src/node_sqlite.cc 80.78% <88.88%> (+0.05%) ⬆️

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

@thisalihassan

Copy link
Copy Markdown
Contributor Author

@nodejs/sqlite @geeksilva97 can I get a review please?

@geeksilva97

Copy link
Copy Markdown
Contributor

Sure. Will take a look.

@geeksilva97

Copy link
Copy Markdown
Contributor

cc @nodejs/cpp-reviewers

@geeksilva97 geeksilva97 added the request-ci Add this label to start a Jenkins CI on a PR. label May 6, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 6, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

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

@geeksilva97 geeksilva97 requested a review from addaleax May 7, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. sqlite Issues and PRs related to the SQLite subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants