docs: reacton cycle is fixed, update measured numbers to the merged state#1178
Merged
Conversation
…tate The handbook still described the reacton setter-closure cycle as unfixed and claimed disabling the gc backstop brings the sawtooth back; since widgetti/reacton#52 (context tree teardown on close) both are outdated. Re-measured everything against the merged masters: the kitchen-sink app plateaus at ~168 MB (peak 178) with gc fully disabled, so refcounting alone now frees closed kernels. Also records why weak references were not an option (reacton cf61378: a setter can be the only reference keeping its component context alive). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
maartenbreddels
temporarily deployed
to
docs/post-reacton-fix-status - solara-stable PR #1178
July 5, 2026 17:29 — with 
Render
Destroyed
Render
Destroyed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Freshness pass over the memory handbook now that widgetti/reacton#52 is merged, verified by re-running the full battery against both merged masters:
REACTON_FASTmodes)REACTON_FAST=1, repeated — all greenanonplateau ~162 MB — versus the 231 MB sawtooth before the fixesDoc changes: the "not yet fixed, lives in reacton" section now describes the merged teardown fix (including why weakrefs were not an option — reacton
cf61378, a setter can be the only reference keeping its component context alive), and the backstop section carries the re-measured numbers table plus the rendered-an-app gate rationale.🤖 Generated with Claude Code