Skip to content

Correct when derived signal functions run - #1647

Open
Janani-bn wants to merge 3 commits into
solidjs:mainfrom
Janani-bn:docs/derived-signals-692
Open

Janani-bn wants to merge 3 commits into
solidjs:mainfrom
Janani-bn:docs/derived-signals-692

Conversation

@Janani-bn

Copy link
Copy Markdown

Closes #692

Derived signals are plain functions, so changing a dependency does not call them by itself. This updates the guide to explain that the caller's tracking scope is what reruns, adds a one-time read versus createEffect example, and clarifies component-body reads and uncached calculations versus createMemo.

Checked the MDX compiles, Prettier, and git diff --check. Runtime assertions with Solid 1.9.14 verify manual reads do not rerun when count changes, while an effect does. Full site build and rendered-page checks have not been run.

Copilot AI balanced review requested due to automatic review settings September 30, 2026 11:50
@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit c25cc1a
🔍 Latest deploy log https://app.netlify.com/projects/solid-docs/deploys/6abd4218a0005400088fe5ac
😎 Deploy Preview https://deploy-preview-1647--solid-docs.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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment on lines +39 to +42
console.log(double()); // 2: a one-time read
createEffect(() => console.log(double())); // tracks count

setCount(2); // the effect sees the new value; the one-time read does not run again

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rather than add the comments to the code snippet, can you provide explanations in text please? i think it's better to leave context in code snippets if required for understanding but explanations outside of it

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.

[Content]: misleading section about when derived-signals are called

3 participants