Skip to content

RFC: one store for props and loaders - #805

Closed
pleek91 wants to merge 1 commit into
mainfrom
rfc-data-store
Closed

pleek91 wants to merge 1 commit into
mainfrom
rfc-data-store

Conversation

@pleek91

@pleek91 pleek91 commented Jul 31, 2026

Copy link
Copy Markdown
Member

Description

Proposal, not an implementation — nothing here changes behaviour.

A route's values are computed in two places that cannot see each other: what a link prefetches, and what a navigation computes. Whether a child route can use something from its parent depends on which of those two places the parent's value happens to be in, and when. That is the shared cause behind #793, #794, #795, #802 and #803, and loaders will need the same problem solved again.

The RFC proposes one small store that knows nothing about props or loaders, where a value is a promise from the moment it is first asked for, and prefetched and committed values have the same shape.

Worth reading for the sections on what it deliberately does not do (deduplication, reference counting) and on sequencing — the current stack of fixes should land first, for its tests rather than its fixes.

Values a link prefetches and values a navigation computes live in separate
places that cannot see each other, which is behind a run of parent props bugs
and would need solving again for loaders.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploy Preview for kitbag-router ready!

Name Link
🔨 Latest commit d4e4d8c
🔍 Latest deploy log https://app.netlify.com/projects/kitbag-router/deploys/6a6cd0e9ec91f10008315867
😎 Deploy Preview https://deploy-preview-805--kitbag-router.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant