@pyreon/runtime-dom 0.24.5 → 0.24.6

package/src/tests/runtime-dom.browser.test.ts

@@ -1,409 +0,0 @@
-import { For, h, Portal, _rp } from '@pyreon/core'
-import { signal } from '@pyreon/reactivity'
-import { flush, mountInBrowser } from '@pyreon/test-utils/browser'
-import { afterEach, describe, expect, it, vi } from 'vitest'
-import { hydrateRoot, mount, Transition } from '../index'
-
-// Real-Chromium smoke suite for @pyreon/runtime-dom. Catches environment-
-// divergence bugs that happy-dom hides: SVG namespace property setters,
-// real PointerEvent sequencing, `import.meta.env.DEV` literal-replacement,
-// and the keyed reconciler under live signal updates.
-
-describe('runtime-dom in real browser', () => {
-  afterEach(() => {
-    vi.restoreAllMocks()
-  })
-
-  it('mounts and patches DOM when a signal updates', async () => {
-    const count = signal(0)
-    const { container, unmount } = mountInBrowser(
-      h('span', { id: 'n' }, () => String(count())),
-    )
-    expect(container.querySelector('#n')?.textContent).toBe('0')
-
-    count.set(42)
-    await flush()
-    expect(container.querySelector('#n')?.textContent).toBe('42')
-    unmount()
-  })
-
-  it('keyed <For> reconciler inserts at the right index when a list grows', async () => {
-    type Row = { id: number; label: string }
-    const rows = signal<Row[]>([
-      { id: 1, label: 'a' },
-      { id: 2, label: 'b' },
-    ])
-    const { container, unmount } = mountInBrowser(
-      h(
-        'ul',
-        { id: 'list' },
-        For({
-          each: rows,
-          by: (r: Row) => r.id,
-          children: (r: Row) => h('li', { 'data-id': String(r.id) }, r.label),
-        }),
-      ),
-    )
-
-    let items = container.querySelectorAll<HTMLLIElement>('#list li')
-    expect(items).toHaveLength(2)
-    expect(Array.from(items).map((el) => el.textContent)).toEqual(['a', 'b'])
-
-    rows.set([
-      { id: 1, label: 'a' },
-      { id: 3, label: 'c' },
-      { id: 2, label: 'b' },
-    ])
-    await flush()
-
-    items = container.querySelectorAll<HTMLLIElement>('#list li')
-    expect(items).toHaveLength(3)
-    expect(Array.from(items).map((el) => el.dataset.id)).toEqual(['1', '3', '2'])
-    expect(Array.from(items).map((el) => el.textContent)).toEqual(['a', 'c', 'b'])
-    unmount()
-  })
-
-  it('keyed <For> with _rp-wrapped each (compiled JSX shape) renders + reacts to signal updates', async () => {
-    // Regression for the `<For each={signal}>` JSX form. The compiler emits
-    // `h(For, { each: _rp(() => rows()), ... })`. `makeReactiveProps` (via
-    // mountComponent) converts the `_rp`-branded function to a getter on
-    // `props.each`. `For()` then forwards those props onto a `ForSymbol`
-    // VNode, which reaches `mountChild`'s ForSymbol branch.
-    //
-    // Before the fix, that branch destructured `{ each, by, children }`
-    // eagerly — firing the getter and binding `each` to the *resolved
-    // array*, not the function. `mountFor` then crashed on `source()`
-    // (calling an array) and the list never rendered. This test produces
-    // the exact same vnode shape the compiler emits, so it catches the
-    // regression without needing a JSX transform pipeline in the test.
-    type Row = { id: number; label: string }
-    const rows = signal<Row[]>([
-      { id: 1, label: 'a' },
-      { id: 2, label: 'b' },
-      { id: 3, label: 'c' },
-    ])
-
-    const { container, unmount } = mountInBrowser(
-      h(
-        'ul',
-        { id: 'rp-list' },
-        h(For, {
-          each: _rp(() => rows()),
-          by: (r: Row) => r.id,
-          children: (r: Row) => h('li', { 'data-id': String(r.id) }, r.label),
-        }),
-      ),
-    )
-
-    let items = container.querySelectorAll<HTMLLIElement>('#rp-list li')
-    expect(items).toHaveLength(3)
-    expect(Array.from(items).map((el) => el.dataset.id)).toEqual(['1', '2', '3'])
-
-    // Signal-driven update — replace + reorder the list.
-    rows.set([
-      { id: 2, label: 'b' },
-      { id: 4, label: 'd' },
-      { id: 1, label: 'a' },
-    ])
-    await flush()
-
-    items = container.querySelectorAll<HTMLLIElement>('#rp-list li')
-    expect(items).toHaveLength(3)
-    expect(Array.from(items).map((el) => el.dataset.id)).toEqual(['2', '4', '1'])
-    expect(Array.from(items).map((el) => el.textContent)).toEqual(['b', 'd', 'a'])
-
-    // Append — confirms reactivity persists across multiple updates.
-    rows.set([
-      { id: 2, label: 'b' },
-      { id: 4, label: 'd' },
-      { id: 1, label: 'a' },
-      { id: 5, label: 'e' },
-    ])
-    await flush()
-    items = container.querySelectorAll<HTMLLIElement>('#rp-list li')
-    expect(items).toHaveLength(4)
-    unmount()
-  })
-
-  it('creates SVG with the SVG namespace and updates reactive attributes via setAttribute', async () => {
-    const x = signal(10)
-    const { container, unmount } = mountInBrowser(
-      h(
-        'svg',
-        { id: 'svg', width: '100', height: '100' },
-        h('rect', { id: 'r', x: () => x(), y: '0', width: '20', height: '20' }),
-      ),
-    )
-
-    const svg = container.querySelector('#svg')
-    const rect = container.querySelector('#r')
-    expect(svg?.namespaceURI).toBe('http://www.w3.org/2000/svg')
-    expect(rect?.namespaceURI).toBe('http://www.w3.org/2000/svg')
-    // SVGRectElement.x is a read-only SVGAnimatedLength getter — applying
-    // via property would crash. setAttribute is the only safe path.
-    expect(rect?.getAttribute('x')).toBe('10')
-
-    x.set(55)
-    await flush()
-    expect(rect?.getAttribute('x')).toBe('55')
-    unmount()
-  })
-
-  it('delegated event handler sees `currentTarget` as the bound element, not the listener root', async () => {
-    // Regression for a real Pyreon framework bug found via PR #329's form
-    // section. Pyreon's TargetedEvent<E> type promises `currentTarget` is
-    // the per-element type (e.g. HTMLInputElement), but native event
-    // delegation leaves `currentTarget` as the container (where the
-    // listener is registered). User code that writes
-    // `(ev.currentTarget as HTMLInputElement).value` would silently read
-    // from a <div> and get undefined.
-    //
-    // The fix is in delegate.ts: per-handler Object.defineProperty
-    // override of currentTarget, matching React/Vue/Solid behavior.
-    const { container, unmount } = mountInBrowser(
-      h(
-        'div',
-        { id: 'wrap' },
-        h('input', {
-          id: 'inp',
-          type: 'text',
-          'data-marker': 'real-input',
-          onInput: (ev: Event) => {
-            const ct = ev.currentTarget as HTMLInputElement | null
-            // Capture observable signals so the test can assert on them
-            ;(globalThis as { __test_ct_tag?: string | undefined }).__test_ct_tag = ct?.tagName
-            ;(globalThis as { __test_ct_value?: string | undefined }).__test_ct_value =
-              ct?.value
-            ;(globalThis as { __test_ct_marker?: string | null | undefined }).__test_ct_marker =
-              ct?.getAttribute('data-marker') ?? null
-          },
-        }),
-      ),
-    )
-
-    const inp = container.querySelector<HTMLInputElement>('#inp')!
-    inp.value = 'hello'
-    inp.dispatchEvent(new Event('input', { bubbles: true }))
-    await flush()
-
-    // Without the fix: tagName would be 'DIV' (or whatever container is),
-    // value would be undefined, marker would be null.
-    expect((globalThis as { __test_ct_tag?: string | undefined }).__test_ct_tag).toBe('INPUT')
-    expect((globalThis as { __test_ct_value?: string | undefined }).__test_ct_value).toBe('hello')
-    expect((globalThis as { __test_ct_marker?: string | null | undefined }).__test_ct_marker).toBe(
-      'real-input',
-    )
-
-    unmount()
-    delete (globalThis as { __test_ct_tag?: string | undefined }).__test_ct_tag
-    delete (globalThis as { __test_ct_value?: string | undefined }).__test_ct_value
-    delete (globalThis as { __test_ct_marker?: string | null | undefined }).__test_ct_marker
-  })
-
-  it('dispatches a real PointerEvent and fires the onClick handler', async () => {
-    const clicks = signal(0)
-    const { container, unmount } = mountInBrowser(
-      h(
-        'button',
-        {
-          id: 'btn',
-          onClick: () => clicks.set(clicks() + 1),
-        },
-        () => `clicks: ${clicks()}`,
-      ),
-    )
-
-    const btn = container.querySelector<HTMLButtonElement>('#btn')!
-    expect(btn.textContent).toBe('clicks: 0')
-
-    btn.dispatchEvent(
-      new PointerEvent('pointerdown', { bubbles: true, pointerType: 'mouse' }),
-    )
-    btn.dispatchEvent(
-      new PointerEvent('pointerup', { bubbles: true, pointerType: 'mouse' }),
-    )
-    btn.click()
-    await flush()
-    expect(btn.textContent).toBe('clicks: 1')
-    unmount()
-  })
-
-  it('emits the duplicate-key dev warning under non-production NODE_ENV', async () => {
-    // process.env.NODE_ENV !== 'production' in this dev browser run — the
-    // bundler-agnostic gate that every modern bundler auto-replaces at
-    // consumer build time. The warning must fire here. The companion
-    // `runtime-dom.prod-bundle.test.ts` Node test proves the same code path
-    // is dead in a prod bundle (NODE_ENV='production').
-    expect(process.env.NODE_ENV).not.toBe('production')
-
-    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    const dupes = signal([
-      { id: 1, label: 'a' },
-      { id: 1, label: 'b' },
-    ])
-    const { unmount } = mountInBrowser(
-      h(
-        'div',
-        null,
-        For({
-          each: dupes,
-          by: (r: { id: number }) => r.id,
-          children: (r: { id: number; label: string }) => h('span', { class: 'dup' }, r.label),
-        }),
-      ),
-    )
-    await flush()
-
-    const calls = warn.mock.calls.flat().join('\n')
-    expect(calls).toMatch(/Duplicate key/i)
-    unmount()
-  })
-
-  it('hydrateRoot — attaches reactive listeners to existing SSR markup without rerender', async () => {
-    // Simulate SSR-rendered HTML in the container.
-    const container = document.createElement('div')
-    container.innerHTML = '<button id="ssr-btn" type="button">clicks: 0</button>'
-    document.body.appendChild(container)
-
-    const ssrButtonRef = container.querySelector<HTMLButtonElement>('#ssr-btn')!
-    const count = signal(0)
-    const cleanup = hydrateRoot(
-      container,
-      h(
-        'button',
-        {
-          id: 'ssr-btn',
-          type: 'button',
-          onClick: () => count.set(count() + 1),
-        },
-        () => `clicks: ${count()}`,
-      ),
-    )
-
-    // Same DOM node — hydrate adopts it, doesn't replace.
-    expect(container.querySelector('#ssr-btn')).toBe(ssrButtonRef)
-
-    // Click triggers the hydrated handler + reactive text update.
-    ssrButtonRef.click()
-    await flush()
-    expect(ssrButtonRef.textContent).toBe('clicks: 1')
-
-    cleanup()
-    container.remove()
-  })
-
-  it('Portal — children render in a different DOM subtree (not the wrapper)', async () => {
-    const target = document.createElement('div')
-    target.id = 'portal-target'
-    document.body.appendChild(target)
-
-    const { container, unmount } = mountInBrowser(
-      h(
-        'div',
-        { id: 'src' },
-        h(Portal, { target }, h('span', { id: 'teleported' }, 'over there')),
-      ),
-    )
-
-    // Portal child is in target, NOT in container.
-    expect(container.querySelector('#teleported')).toBeNull()
-    expect(target.querySelector('#teleported')?.textContent).toBe('over there')
-    unmount()
-    target.remove()
-  })
-
-  it('Transition — show=false applies leave classes; transitionend removes element', async () => {
-    const visible = signal(true)
-    const { container, unmount } = mountInBrowser(
-      h(
-        Transition,
-        { name: 'fade', show: () => visible() },
-        // Real CSS transition so transitionend actually fires when the
-        // class swap changes opacity (not just instantly).
-        h('div', { id: 'fading', style: 'transition: opacity 30ms; opacity: 1' }, 'hello'),
-      ),
-    )
-    await flush()
-    expect(container.querySelector('#fading')).not.toBeNull()
-
-    visible.set(false)
-    // After two rAFs the leave-active + leave-to classes are applied.
-    await new Promise<void>((r) => requestAnimationFrame(() => r()))
-    await new Promise<void>((r) => requestAnimationFrame(() => r()))
-
-    const stillRendered = container.querySelector('#fading')
-    if (stillRendered) {
-      // Expect at least one of the fade-leave classes during the
-      // active phase.
-      expect(stillRendered.className).toMatch(/fade-leave/)
-      // Manually fire transitionend to short-circuit the 5s safety
-      // timeout (we don't care about real timing here, only that the
-      // event-driven cleanup path works).
-      stillRendered.dispatchEvent(new Event('transitionend', { bubbles: true }))
-    }
-    await flush()
-    await new Promise<void>((r) => setTimeout(r, 16))
-    expect(container.querySelector('#fading')).toBeNull()
-    unmount()
-  })
-
-  it('two mount() roots stay isolated — events on one do not affect the other', async () => {
-    const c1 = signal(0)
-    const c2 = signal(0)
-    const root1 = document.createElement('div')
-    const root2 = document.createElement('div')
-    document.body.append(root1, root2)
-
-    const u1 = mount(
-      h('button', { id: 'b1', onClick: () => c1.set(c1() + 1) }, () => `c1=${c1()}`),
-      root1,
-    )
-    const u2 = mount(
-      h('button', { id: 'b2', onClick: () => c2.set(c2() + 1) }, () => `c2=${c2()}`),
-      root2,
-    )
-
-    root1.querySelector<HTMLButtonElement>('#b1')!.click()
-    root1.querySelector<HTMLButtonElement>('#b1')!.click()
-    root2.querySelector<HTMLButtonElement>('#b2')!.click()
-    await flush()
-
-    expect(c1()).toBe(2)
-    expect(c2()).toBe(1)
-
-    u1()
-    u2()
-    root1.remove()
-    root2.remove()
-  })
-
-  it('event delegation — multi-word event names like onPointerDown actually fire', async () => {
-    // Regression for the bug fixed alongside this PR:
-    // `onPointerDown` was being lowercased to `pointerDown` for the
-    // DELEGATED_EVENTS lookup, missing the all-lowercase entry, so the
-    // handler was attached via addEventListener('pointerDown', ...) which
-    // never fires. Same for mousedown, dblclick, touchstart, etc.
-    let pointerDownFired = 0
-    let dblClickFired = 0
-    const { container, unmount } = mountInBrowser(
-      h('div', {
-        id: 'evt',
-        onPointerDown: () => {
-          pointerDownFired++
-        },
-        onDblClick: () => {
-          dblClickFired++
-        },
-      }),
-    )
-    const target = container.querySelector('#evt')!
-    target.dispatchEvent(
-      new PointerEvent('pointerdown', { bubbles: true, pointerId: 1 }),
-    )
-    target.dispatchEvent(new MouseEvent('dblclick', { bubbles: true }))
-    await flush()
-    expect(pointerDownFired).toBe(1)
-    expect(dblClickFired).toBe(1)
-    unmount()
-  })
-})

package/src/tests/setup.ts

@@ -1,3 +0,0 @@
-import { GlobalRegistrator } from '@happy-dom/global-registrator'
-
-GlobalRegistrator.register()

package/src/tests/show-context.test.ts

@@ -1,270 +0,0 @@
-import { createContext, Fragment, h, provide, useContext } from '@pyreon/core'
-import { signal } from '@pyreon/reactivity'
-import { mount } from '@pyreon/runtime-dom'
-import { describe, expect, it } from 'vitest'
-
-const TestCtx = createContext('default')
-
-describe('context inheritance through reactive boundaries', () => {
-  it('child inside reactive accessor inherits parent context', async () => {
-    let childValue: string | undefined
-
-    function Child() {
-      childValue = useContext(TestCtx)
-      return h('span', null, childValue)
-    }
-
-    function Parent() {
-      provide(TestCtx, 'from-parent')
-      const show = signal(false)
-      setTimeout(() => show.set(true), 10)
-      return () => (show() ? h(Child, null) : null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(Parent, null), container)
-    await new Promise((r) => setTimeout(r, 50))
-    expect(childValue).toBe('from-parent')
-  })
-
-  it('deeply nested context survives through multiple reactive layers', async () => {
-    let innerValue: string | undefined
-
-    function Inner() {
-      innerValue = useContext(TestCtx)
-      return h('span', null, innerValue)
-    }
-
-    function Middle() {
-      const show = signal(false)
-      setTimeout(() => show.set(true), 10)
-      return () => (show() ? h(Inner, null) : null)
-    }
-
-    function Outer() {
-      provide(TestCtx, 'outer-value')
-      const show = signal(false)
-      setTimeout(() => show.set(true), 5)
-      return () => (show() ? h(Middle, null) : null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(Outer, null), container)
-    await new Promise((r) => setTimeout(r, 100))
-    expect(innerValue).toBe('outer-value')
-  })
-
-  it("sibling providers don't leak context to each other", async () => {
-    let childAValue: string | undefined
-    let childBValue: string | undefined
-
-    function ChildA() {
-      childAValue = useContext(TestCtx)
-      return h('span', null, childAValue)
-    }
-
-    function ChildB() {
-      childBValue = useContext(TestCtx)
-      return h('span', null, childBValue)
-    }
-
-    function ProviderA() {
-      provide(TestCtx, 'A')
-      return h(ChildA, null)
-    }
-
-    function ProviderB() {
-      provide(TestCtx, 'B')
-      return h(ChildB, null)
-    }
-
-    function App() {
-      return h(Fragment, null, h(ProviderA, null), h(ProviderB, null))
-    }
-
-    const container = document.createElement('div')
-    mount(h(App, null), container)
-    await new Promise((r) => setTimeout(r, 50))
-
-    expect(childAValue).toBe('A')
-    expect(childBValue).toBe('B')
-  })
-
-  it('Show toggle preserves context across hide/show cycle', async () => {
-    let childValue: string | undefined
-    let mountCount = 0
-
-    function Child() {
-      mountCount++
-      childValue = useContext(TestCtx)
-      return h('span', null, childValue)
-    }
-
-    function Parent() {
-      provide(TestCtx, 'persistent')
-      const show = signal(true)
-
-      // Hide then show again
-      setTimeout(() => show.set(false), 10)
-      setTimeout(() => show.set(true), 30)
-
-      return () => (show() ? h(Child, null) : null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(Parent, null), container)
-    await new Promise((r) => setTimeout(r, 100))
-
-    expect(childValue).toBe('persistent')
-    expect(mountCount).toBe(2) // mounted twice (initial + re-show)
-  })
-
-  it('reactive context getter updates JSX without re-running component', async () => {
-    const ModeCtx = createContext<() => string>(() => 'light')
-    let renderCount = 0
-
-    function Child() {
-      const getMode = useContext(ModeCtx)
-      renderCount++
-      // Reading the getter inside a reactive accessor — updates when mode changes
-      return h('span', null, () => getMode())
-    }
-
-    function Parent() {
-      const mode = signal<string>('light')
-      provide(ModeCtx, () => mode())
-      setTimeout(() => mode.set('dark'), 10)
-      return h(Child, null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(Parent, null), container)
-
-    expect(container.textContent).toBe('light')
-
-    await new Promise((r) => setTimeout(r, 50))
-    expect(container.textContent).toBe('dark')
-    // Component setup ran once — JSX expression re-evaluated reactively
-    expect(renderCount).toBe(1)
-  })
-
-  it('nested Show inside For with context', async () => {
-    const ItemCtx = createContext('none')
-    const collected: string[] = []
-
-    function Item() {
-      const val = useContext(ItemCtx)
-      collected.push(val)
-      return h('li', null, val)
-    }
-
-    function Parent() {
-      provide(ItemCtx, 'parent-provided')
-      const items = signal([1, 2, 3])
-      const show = signal(false)
-      setTimeout(() => show.set(true), 10)
-
-      return () => (show() ? h('ul', null, ...items().map((i) => h(Item, { key: i }))) : null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(Parent, null), container)
-    await new Promise((r) => setTimeout(r, 50))
-
-    expect(collected.length).toBe(3)
-    expect(collected.every((v) => v === 'parent-provided')).toBe(true)
-  })
-
-  // ── Lock-in for the context-truncation fix (PR #406) ──────────────────────
-  //
-  // Pre-fix `mountReactive`'s `restoreContextStack` did
-  // `stack.length = savedLength` in its finally block — which destroyed
-  // every provider frame that the synchronous mount had pushed via
-  // `provide()`. Signal-driven re-runs of `_bind` / `renderEffect` inside
-  // the mounted subtree later saw a half-empty stack and `useContext()`
-  // silently fell back to the default. The original symptom was
-  // `<PyreonUI mode={signal()}>` toggling not propagating to consumers
-  // — discovered while writing PR #406's regression e2e and traced back
-  // through the binding subscription chain to this stack truncation.
-  //
-  // The fix has two cooperating layers; each provides defense-in-depth
-  // for the other, so this assertion would still pass if you revert
-  // EITHER alone — but reverting BOTH layers together fails it. To
-  // bisect-verify cleanly, revert both:
-  //   1. `packages/core/core/src/context.ts:restoreContextStack` — change
-  //      the finally block back to `stack.length = savedLength` (truncate
-  //      everything fn() pushed).
-  //   2. `packages/core/reactivity/src/effect.ts:_bind` — remove the
-  //      `_snapshotCapture` capture/restore wiring so re-runs call fn()
-  //      against whatever the live stack happens to be at re-run time.
-  //
-  // With both reverted, this test fails with `seen[1] === 'default'`.
-  //
-  // What the test exercises: a `_bind` text binding inside a child mounted
-  // through a reactive accessor (which goes through `mountReactive`). The
-  // binding subscribes to a signal and reads `useContext(Ctx)`. After
-  // initial mount, the provider frame is at risk of being truncated by
-  // `mountReactive`'s cleanup — toggling the signal forces the binding to
-  // re-run, which re-reads context. If either fix is in place, the
-  // re-read finds the provider frame and returns the provided value.
-  it('binding re-runs preserve context lookup across mountReactive cleanup boundary (PR #406 splice + snapshot capture)', async () => {
-    const Ctx = createContext('default')
-    const trigger = signal(0)
-    let lastSeen: string | undefined
-    let runCount = 0
-
-    function Inner() {
-      // JSX text accessor compiles to a `_bind` / renderEffect text binding
-      // that subscribes to `trigger` (signal read inside the body) AND
-      // captures the external context snapshot at setup time.
-      return h('span', null, () => {
-        trigger()
-        const v = useContext(Ctx)
-        lastSeen = v
-        runCount++
-        return v
-      })
-    }
-
-    function Provider() {
-      // CRITICAL for exercising the bug: `provide()` runs INSIDE the
-      // reactive child fn (the accessor `() => h(Provider)` returned by
-      // App below). That puts Ctx on the stack DURING `mountReactive`'s
-      // restoreContextStack(snapshot, fn) execution — and pre-fix the
-      // truncating finally block (`stack.length = savedLength`) destroyed
-      // the frame the moment fn returned. If Outer pushed Ctx in its OWN
-      // body BEFORE returning the accessor, the frame would already be on
-      // the stack at snapshot-capture time and survive truncation
-      // unrelated — so the test wouldn't actually exercise the bug.
-      provide(Ctx, 'provider-value')
-      return h(Inner, null)
-    }
-
-    function App() {
-      // No provide() here — the provider frame must be pushed strictly
-      // inside the reactive accessor body (= inside `mountReactive`'s fn)
-      // so the truncation-vs-splice path is reached.
-      return () => h(Provider, null)
-    }
-
-    const container = document.createElement('div')
-    mount(h(App, null), container)
-    await new Promise((r) => setTimeout(r, 20))
-    expect(lastSeen).toBe('provider-value')
-    const initialRunCount = runCount
-
-    // Force the binding's effect to re-run AFTER the synchronous mount
-    // has fully unwound. With the broken pre-fix shape, mountReactive's
-    // `stack.length = savedLength` finally block has already destroyed
-    // the Ctx frame Provider pushed, so this re-run reads useContext
-    // against a stack that no longer contains the provider — and
-    // `lastSeen` becomes `'default'`.
-    trigger.set(1)
-    await new Promise((r) => setTimeout(r, 20))
-    expect(lastSeen).toBe('provider-value')
-    // Sanity: the re-run actually happened. Otherwise the test could
-    // pass for the wrong reason (e.g. if the trigger subscription wasn't
-    // established because the accessor didn't read `trigger()` reactively).
-    expect(runCount).toBeGreaterThan(initialRunCount)
-  })
-})