Skip to content

fix: handle BigInt serialization on Windows#42

Open
Jah-yee wants to merge 1 commit into
moltlaunch:mainfrom
Jah-yee:main
Open

fix: handle BigInt serialization on Windows#42
Jah-yee wants to merge 1 commit into
moltlaunch:mainfrom
Jah-yee:main

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Mar 18, 2026

Copy link
Copy Markdown

Summary

  • Add safeStringify utility function to handle BigInt values in JSON.stringify
  • Apply fix to marketplace.ts and agentcash.ts
  • Fixes Do not know how to serialize a BigInt error on Windows when running mltl register or mltl profile commands

Changes

  • Added safeStringify() function that converts BigInt to string using toString()
  • Updated 3 JSON.stringify calls to use the new utility

Testing

The fix follows the suggestion in issue #29:

JSON.stringify(data, (_, v) => typeof v === 'bigint' ? v.toString() : v)

Closes #29

- Add safeStringify utility function to handle BigInt values
- Apply to marketplace.ts and agentcash.ts JSON.stringify calls
- Fixes 'Do not know how to serialize a BigInt' error on Windows

Closes moltlaunch#29
@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏

@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👍 Looks good to merge — clean fix, thanks for contributing!

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Good BigInt serialization fix for Windows! Just a reminder for maintainer review. Thanks! 🙏

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Looks good to merge! [merge promotion]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

This PR looks good and is ready to merge! [+1]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for this contribution! Bumping for visibility.

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Would it be possible to merge this PR? Happy to make any adjustments if needed. Thank you for your time!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

➿ bump (R337)

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

👍 promote

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Looks good! Merging soon would be appreciated – thanks for maintaining this project 🙏

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

This PR looks great and is ready to merge! 🎉 Thank you for your contribution!

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.

BigInt serialization error on Windows prevents register updates and profile changes

1 participant