Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add NodeJS support #118

Merged
merged 81 commits into from
Jan 19, 2025
Merged

Add NodeJS support #118

merged 81 commits into from
Jan 19, 2025

Conversation

kevinheavey
Copy link
Collaborator

@kevinheavey kevinheavey commented Jan 13, 2025 •
edited
Loading

The only change to existing crates is that the LiteSVM struct now has internal methods like set_compute_budget. This is because napi-rs couldn't call the with_* methods, so I needed a way to mutate the struct without returning it.

New stuff:

  • Add litesvm-node crate and litesvm TypeScript package
  • Add docs site for this package
  • Add CI for this package

@kevinheavey
init litesvm-node
a6979c7
@kevinheavey
napi rough work
af83dc8
@kevinheavey
more napi rough work
9acae7e
@kevinheavey
more rough work
645fbbd
@kevinheavey
get the setup methods compiling
421e91e
@kevinheavey
typescript setup
cfc93c4
@kevinheavey
add most LiteSVM methods
78185c6
@kevinheavey
add compute budget support
31384af
@kevinheavey
add missing methods
afb0969
@kevinheavey
move profile to fix warning
dbed94a
@kevinheavey
add lint scripts
b1c2677
@kevinheavey
add structured transaction error support
4f2e463
@kevinheavey
delete accidental file
11bea51
@kevinheavey
add tests, add toString methods, add get/set_clock, add friendly Simu…
e13dc8d 
…latedTransactionInfo wrapper
@kevinheavey
fix withComputeBudget
b46e13e
@kevinheavey
fix lint
cf13844
@kevinheavey
use swc-jest
d9cad33
@kevinheavey
fix FeatureSet usage
43a67e7
@kevinheavey
update some i64 types
04d753a
@kevinheavey
add rent sysvar
4088840
@kevinheavey
add more tests
417cc66
@kevinheavey
update lock file
4337161
@kevinheavey
add more toString methods
b9c8929
@kevinheavey
move files and add some setters
dbda652
@kevinheavey
add epoch_rewards
b891cde
@kevinheavey
add epoch_schedule
358f309
@kevinheavey
add last_restart_slot
05e85f8
@kevinheavey
add slot_hashes
b90ef8f
@kevinheavey
add slot_history
c6316b8
@kevinheavey
add stake_history
e80e844
@kevinheavey kevinheavey marked this pull request as ready for review January 13, 2025 15:58
@kevinheavey kevinheavey requested a review from Aursen January 13, 2025 15:59
Aursen
Aursen reviewed Jan 19, 2025
View reviewed changes
Copy link
Collaborator

@Aursen Aursen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the exception of the previous comment, everything looks good with a good doc.

Thanks a lot

crates/node-litesvm/.gitignore
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need to put multiple git ignore in the repo?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This came from a preset, but it does have some stuff we need in there, and there's little benefit to merging them so I suggest we just leave it as is

@Aursen Aursen self-requested a review January 19, 2025 19:38
@kevinheavey kevinheavey merged commit 4f00b0d into master Jan 19, 2025
11 checks passed
@Aursen Aursen deleted the napi branch January 20, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aursen Aursen Aursen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kevinheavey @Aursen