Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

`createRoot`'s dispose function keeps `fn` alive, so `<For>`/`<Index>` rows retain removed list items

Đang mở
  • Có gì không ổn?
#4,095 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@boombang đang làm issue này rồi.

Từ ngày 15/10/2026.

  • #4096 của @boombang — đang mở

Đánh giá

Độ khó
4/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
45/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
javascript, typescript
Lĩnh vực
frontend, performance

Hướng nghiên cứu

Start with createRoot and the updateFn line that builds the dispose closure, then trace how mapArray and indexArray store each row's disposer in disposers. Run the repro script with node --expose-gc --conditions=browser repro.mjs. Done means both lines print false, and the existing tests for createRoot and the array helpers still pass.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

1.x
Describe the bug

createRoot creates the dispose function it passes to fn inside its own scope:

updateFn = unowned ? ... : () => fn(() => untrack(() => cleanNode(root)));

That scope also holds fn, because updateFn closes over it. So anything that keeps the dispose function also keeps fn, and everything fn closes over.

mapArray and indexArray keep every row's dispose function in disposers. Every row's fn is mapper, and mapper closes over newItems, the list that was current when the row was created. So a mounted row keeps that whole list reachable, including items that were removed from the list later.

We found this in production. A long-lived <For> row kept a list it was created from, and with it a replaced document of a few hundred MB.

Your Example Website or App

Standalone script below (Node 22, solid-js 1.9.17, client build).

Steps to Reproduce the Bug or Issue
// npm i [email protected]
// node --expose-gc --conditions=browser repro.mjs
import { createRoot, createSignal, createMemo, mapArray, indexArray } from "solid-js";

const collect = async () => {
  // WeakRef targets stay alive until the current job ends.
  await new Promise(r => setTimeout(r, 0));
  globalThis.gc();
};

// A row created from [kept, removed] stays mounted after the list becomes [kept].
async function check(name, map) {
  const kept = { name: "kept" };
  let removed, setList;
  const dispose = createRoot(dispose => {
    const [list, set] = createSignal(
      (() => {
        const item = { name: "removed" };
        removed = new WeakRef(item);
        return [kept, item];
      })()
    );
    setList = set;
    createMemo(map(list))();
    return dispose;
  });
  setList([kept]);
  await collect();
  console.log(`${name}: removed item still reachable:`, removed.deref() !== undefined);
  dispose();
}

await check("mapArray (<For>)", list => mapArray(list, item => item.name));
await check("indexArray (<Index>)", list => indexArray(list, item => item().name));

Output:

mapArray (<For>): removed item still reachable: true
indexArray (<Index>): removed item still reachable: true

The dev build (dist/dev.js) gives the same output, and so does Bun (JavaScriptCore) with Bun.gc(true).

Expected behavior

Once the row for removed is disposed, nothing should keep removed reachable, so both lines should print false. They do once the dispose function is created outside createRoot's scope, in both V8 and JavaScriptCore.

Platform
  • solid-js: 1.9.17 (main)
  • Node 22.23 (V8), Bun 1.4.2 (JavaScriptCore)
Additional context

I haven't checked next (2.0) yet. The fix for 1.x is in the PR linked below.

Ngôn ngữ chính
TypeScript
Star
36.1k
Fork
1.1k
Merge trung bình
11 giờ 17 phút
Pull request đã merge (30 ngày)
371

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của solidjs/solid

Tất cả issue của solidjs/solid

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.