Skip to content

Conversation

Valchap
Copy link

@Valchap Valchap commented Sep 6, 2025

Goal

Add a new feature to the library that enables the binding of all variables to javascript integers instead of bigint

Changes

  • Added the new feature
  • Updated Readme.md to document it

Checklist

  • I have followed the steps listed in the Contributing guide.
  • If necessary, I have added documentation related to the changes made.
  • I have added or updated the tests related to the changes made.

@Valchap Valchap changed the title Add to new feature to enable the binding of all variables to integers instead of bigint Add a new feature to enable the binding of all variables to integers instead of bigint Sep 7, 2025
@Valchap
Copy link
Author

Valchap commented Sep 14, 2025

Fixed the tests that expected bigint to be generated

@eythaann
Copy link

I like this, time ago I had an issue caused by a number being used as bigint this because the deserialization is just a JSON parse no hard check of struct and transformations like could be done with libraries like zod.

By the way, currently in my project I use the attribute #[ts(type = number)] and this is enough.

@Valchap
Copy link
Author

Valchap commented Oct 18, 2025

Done in a more general way by #448

@Valchap Valchap closed this Oct 18, 2025
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.

2 participants