Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

[2.0 / next] Recovering before a held error commits throws the successful value

Open
  • Is something wrong?
#4,145 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
1-2 days
Newbie friendliness
48/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
javascript, typescript
Domain
backend, testing-qa

Research direction

Start with the held-frame error handling described in the issue, especially stageError, _pendingValue, CONFIG_STAGED_ERROR, serve, and commitPendingNode. Read the related S-43 case in error-is-an-answer.test.ts and compare its recovery timing with this reproduction, which recovers before the error commits. Done means the successful value replaces the proposed error and the request can finish with the UI showing Sum: 3 without [REACTIVITY_HALTED].

Written by the indexing model from the issue text.

Description

Describe the bug

A memo throws while an async sibling keeps the rendered update held. Before the request finishes, the memo recomputes successfully. When the request then resolves, Solid throws the successful recovery value as an error, reports [REACTIVITY_HALTED], and leaves the old content on screen.

Your Example Website or App

https://s.olid.uk/id/df_wYAiPRlmgfLmezgxTag

Steps to Reproduce the Bug or Issue

Click each button once, in order; rebuild or reload to repeat.

  1. Initially the page shows Sum: 2: both memos return 1.
  2. Click 1. Start update. slow becomes pending and value throws Error("boom"). The previous sum stays visible.
  3. Click 2. Recover. value successfully returns 1 while the request is still pending.
  4. Click 3. Finish request. The request resolves to 2.
  5. The page stays at Sum: 2. The console reports [REACTIVITY_HALTED] with the successful value 1 as the error.

Control: remove if (step() === 1) throw new Error("boom"); and rebuild. The same clicks produce Sum: 3.

Changing the recovery value to 7 makes Solid throw 7 instead; the sum stays 2 instead of becoming 9.

Expected behavior

The successful recomputation should replace the proposed error. Once the request resolves, the page should show Sum: 3 without throwing the recovered value or halting reactivity.

Screenshots or Videos

No response

Platform
  • OS: Linux
  • Browser: Chromium 154.0.8037.57
  • Solid: playground pinned to next@d2ad8b288886a5c7171e6746ccc019ad1d08d8ab, also the current next head when checked on 2026-10-19. Reproduces with development and production builds.
Additional context

Follow-up to the error-outcome audit in #3886, which was closed in favor of the D-41 implementation in #4085. The related S-43 test in error-is-an-answer.test.ts covers recovery after the previous error has committed. This case replaces an error proposal with a successful value before that error commits, while the same frame is still held.

Related reports, with different triggers:

  • #4109 concerns ordinary outside reads throwing before a held frame reveals. This repro fails during the eventual reveal, after the memo has recovered.
  • #4110 concerns a direct write to a failed writable memo. This repro uses successful recomputation of an ordinary memo; there is no writable memo setter.
  • #4106 was fixed by #4129 for error/value publication through latest() derivations. This repro uses no latest() and still fails on a commit containing that fix.

Likely cause: stageError stores the proposed error in _pendingValue and sets CONFIG_STAGED_ERROR. The successful recomputation replaces _pendingValue with 1 but leaves that flag set. A subsequent tracked read in serve therefore throws the successful value. The flag is normally cleared by commitPendingNode, after the failing read in this schedule.

The component uses one signal, two memos, a native Promise and ordinary JSX, with no timers or explicit effects/flushes. Source-level controls reproduce both same-value and changed-value recovery failures with immediate and native delivery; the no-error controls succeed.

Dominant language
TypeScript
Stars
36.1k
Forks
1.1k
Avg merge
11h 37m
Merged PRs (30d)
344

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from solidjs/solid

All issues in solidjs/solid

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.