Skip to content

feat: add napi bindings#173

Merged
wemeetagain merged 162 commits intomainfrom
napi
Feb 12, 2026
Merged

feat: add napi bindings#173
wemeetagain merged 162 commits intomainfrom
napi

Conversation

@wemeetagain
Copy link
Copy Markdown
Member

Add scaffolding for napi bindings with some example code

wemeetagain and others added 30 commits January 8, 2026 11:01
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
Signed-off-by: Chen Kai <281165273grape@gmail.com>
@spiral-ladder spiral-ladder force-pushed the napi branch 8 times, most recently from a184e7b to 8143492 Compare February 11, 2026 05:24
@spiral-ladder
Copy link
Copy Markdown
Collaborator

spiral-ladder commented Feb 11, 2026

I think this should be ready for a proper look. I opened #213 to fix some unrelated issues to bindings that caused some spec tests failures because of the way effective balance increments were being initialized and used.

The main issue with the bindings test failing was most likely due to being OOM-killed, which we now manually sidestep by first deserializing and destructuring the treeview state before manually garbage collecting the unused state.

Comment thread biome.jsonc Outdated
@wemeetagain
Copy link
Copy Markdown
Member Author

I re-reviewed this and fixed up a few things.
This is a solid foundation to build on, lets take additional comments into new issues and PRs so this crazy PR can finally die.

@wemeetagain wemeetagain merged commit b7d499e into main Feb 12, 2026
10 checks passed
@wemeetagain wemeetagain deleted the napi branch February 12, 2026 22:40
wemeetagain pushed a commit that referenced this pull request Feb 23, 2026
- pluralize the term
- fix init issues found in
#173 leading to spec tests
failure
twoeths pushed a commit that referenced this pull request Mar 3, 2026
Add scaffolding for napi bindings with some example code

---------

Signed-off-by: Chen Kai <281165273grape@gmail.com>
Co-authored-by: Chen Kai <281165273grape@gmail.com>
Co-authored-by: bing <spiralladder@fastmail.com>
Co-authored-by: guha-rahul <52607971+guha-rahul@users.noreply.github.com>
Co-authored-by: Jeff Chung <78428519+jeffoodchain@users.noreply.github.com>
Co-authored-by: Jeff Chung <sh1001309@gmail.com>
twoeths pushed a commit that referenced this pull request Mar 3, 2026
- pluralize the term
- fix init issues found in
#173 leading to spec tests
failure
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.

5 participants