Skip to content

✨ the one where I implement memo ✨#405

Merged
hayleigh-dot-dev merged 16 commits intomainfrom
memo
Nov 27, 2025
Merged

✨ the one where I implement memo ✨#405
hayleigh-dot-dev merged 16 commits intomainfrom
memo

Conversation

@yoshi-monster
Copy link
Contributor

Finally!

Lots of the diff is just shuffling code around, please don't be scared 💜

yoshie added 16 commits November 23, 2025 17:16
this improves performance by 3-10%.
all the places.

This never materialises or diffs or stores lazy nodes, so it doesn't do
anything useful yet.
this simplifies diffing a bunch since we don't have to track the mapper
anymore. It's a _required_ change for lazy to not have mappers inside a
single events tree.

It's also faster!
these are technically not necessary since it can ✨ never happen
✨ but we should not write code that can crash, even in theory.
@hayleigh-dot-dev
Copy link
Collaborator

Super exciting stuff!

@hayleigh-dot-dev hayleigh-dot-dev merged commit c53e53c into main Nov 27, 2025
1 check passed
@yoshi-monster yoshi-monster deleted the memo branch November 27, 2025 16:02
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