Skip to content

Initial commit of aeson removal. #1151

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
Closed
dmjio wants to merge 6 commits into from
Closed

Initial commit of aeson removal. #1151

dmjio wants to merge 6 commits into from

Conversation

@dmjio
Copy link
Owner

@dmjio dmjio commented Aug 29, 2025 •
edited
Loading

This PR experiments with removing aeson due to its additional overhead when dealing with marshalling. This exposes an aeson interface, defined in terms of JSM and JSVal. This also has the benefit of lowering dep. count for MicroHs, payload size, etc.

  • Use IntMap to handle multi jsaddle sessions
  • Expose an unsafePerformJSM for use in parsing (like PFromJSVal).
  • Expose additional aeson combinators
  • newtype Parser over JSM

@dmjio
Initial commit of aeson removal.
5f25b8b 
This PR experiments with removing `aeson` due to its additional overhead when
dealing with marshalling. This exposes an `aeson` interface, defined
in terms of `JSM` and `JSVal`
@dmjio
Copy link
Owner Author

dmjio commented Dec 5, 2025

Closing in favor of #1263, #1264

@dmjio dmjio closed this Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@dmjio