@pyreon/core 0.24.5 → 0.24.6

package/src/tests/lifecycle.test.ts

@@ -1,350 +0,0 @@
-import {
-  getCurrentHooks,
-  onErrorCaptured,
-  onMount,
-  onUnmount,
-  onUpdate,
-  setCurrentHooks,
-} from '../lifecycle'
-import type { LifecycleHooks } from '../types'
-
-describe('setCurrentHooks / getCurrentHooks', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('getCurrentHooks returns null by default', () => {
-    expect(getCurrentHooks()).toBeNull()
-  })
-
-  test('setCurrentHooks sets the current hooks context', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    expect(getCurrentHooks()).toBe(hooks)
-  })
-
-  test('setCurrentHooks(null) clears the context', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    expect(getCurrentHooks()).toBe(hooks)
-    setCurrentHooks(null)
-    expect(getCurrentHooks()).toBeNull()
-  })
-})
-
-describe('onMount', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('registers callback on current hooks', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    const fn = () => undefined
-    onMount(fn)
-    expect(hooks.mount!).toHaveLength(1)
-    expect(hooks.mount![0]).toBe(fn)
-  })
-
-  test('multiple onMount calls accumulate', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    onMount(() => undefined)
-    onMount(() => undefined)
-    onMount(() => undefined)
-    expect(hooks.mount!).toHaveLength(3)
-  })
-
-  test('warns when called outside component setup', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onMount(() => {})
-    expect(warnSpy).toHaveBeenCalledWith(
-      expect.stringContaining('onMount() called outside component setup'),
-    )
-    warnSpy.mockRestore()
-  })
-
-  test('is a no-op outside component setup (no crash)', () => {
-    expect(() => onMount(() => {})).not.toThrow()
-  })
-
-  test('accepts callback returning cleanup function', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    const cleanup = () => {}
-    onMount(() => cleanup)
-    expect(hooks.mount!).toHaveLength(1)
-    expect(hooks.mount![0]!()).toBe(cleanup)
-  })
-
-  test('accepts callback returning void', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    onMount(() => {})
-    expect(hooks.mount!).toHaveLength(1)
-    expect(hooks.mount![0]!()).toBeUndefined()
-  })
-})
-
-describe('onUnmount', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('registers callback on current hooks', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    const fn = () => {}
-    onUnmount(fn)
-    expect(hooks.unmount!).toHaveLength(1)
-    expect(hooks.unmount![0]).toBe(fn)
-  })
-
-  test('warns when called outside component setup', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onUnmount(() => {})
-    expect(warnSpy).toHaveBeenCalledWith(
-      expect.stringContaining('onUnmount() called outside component setup'),
-    )
-    warnSpy.mockRestore()
-  })
-})
-
-describe('onUpdate', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('registers callback on current hooks', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    const fn = () => {}
-    onUpdate(fn)
-    expect(hooks.update!).toHaveLength(1)
-    expect(hooks.update![0]).toBe(fn)
-  })
-
-  test('warns when called outside component setup', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onUpdate(() => {})
-    expect(warnSpy).toHaveBeenCalledWith(
-      expect.stringContaining('onUpdate() called outside component setup'),
-    )
-    warnSpy.mockRestore()
-  })
-})
-
-describe('onErrorCaptured', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('registers callback on current hooks', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    const fn = () => true
-    onErrorCaptured(fn)
-    expect(hooks.error!).toHaveLength(1)
-    expect(hooks.error![0]).toBe(fn)
-  })
-
-  test('warns when called outside component setup', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onErrorCaptured(() => true)
-    expect(warnSpy).toHaveBeenCalledWith(
-      expect.stringContaining('onErrorCaptured() called outside component setup'),
-    )
-    warnSpy.mockRestore()
-  })
-
-  test('registered handler receives the error', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-    let captured: unknown = null
-    onErrorCaptured((err) => {
-      captured = err
-      return true
-    })
-    // Simulate calling the handler
-    const testError = new Error('test')
-    hooks.error![0]!(testError)
-    expect(captured).toBe(testError)
-  })
-})
-
-describe('lifecycle hooks interaction', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('all hook types can be registered in same context', () => {
-    const hooks: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    setCurrentHooks(hooks)
-
-    onMount(() => undefined)
-    onUnmount(() => {})
-    onUpdate(() => {})
-    onErrorCaptured(() => true)
-
-    expect(hooks.mount!).toHaveLength(1)
-    expect(hooks.unmount!).toHaveLength(1)
-    expect(hooks.update!).toHaveLength(1)
-    expect(hooks.error!).toHaveLength(1)
-  })
-
-  test('hooks from different setCurrentHooks calls go to different stores', () => {
-    const hooks1: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-    const hooks2: LifecycleHooks = { mount: [], unmount: [], update: [], error: [] }
-
-    setCurrentHooks(hooks1)
-    onMount(() => undefined)
-    setCurrentHooks(hooks2)
-    onMount(() => undefined)
-    onMount(() => undefined)
-
-    expect(hooks1.mount).toHaveLength(1)
-    expect(hooks2.mount).toHaveLength(2)
-  })
-})
-
-// ─── Call-site warning enhancement (Phase 4) ───────────────────────────────
-// Dev warnings now include the first non-framework stack frame so the
-// developer can see WHICH of their components called the hook outside
-// setup — previously the warning just said "lifecycle.ts:17" which was
-// useless for debugging.
-
-describe('warnOutsideSetup — call-site capture', () => {
-  afterEach(() => {
-    setCurrentHooks(null)
-  })
-
-  test('warning includes a "Called from:" frame for debugging', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    // Call onUnmount outside setup — from a user-land helper so the stack
-    // has a distinct non-framework frame we can assert against.
-    const userLandHelper = () => {
-      onUnmount(() => {})
-    }
-    userLandHelper()
-    expect(warnSpy).toHaveBeenCalled()
-    const message = warnSpy.mock.calls[0]?.[0] as string
-    expect(message).toContain('onUnmount() called outside component setup')
-    expect(message).toContain('Called from:')
-    warnSpy.mockRestore()
-  })
-
-  test('onUnmount warning includes provide() hint for diagnosability', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onUnmount(() => {})
-    const message = warnSpy.mock.calls[0]?.[0] as string
-    expect(message).toContain('provide()')
-    warnSpy.mockRestore()
-  })
-
-  test('onMount warning does NOT include the onUnmount-specific hint', () => {
-    const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
-    onMount(() => {})
-    const message = warnSpy.mock.calls[0]?.[0] as string
-    // The hint is specific to onUnmount (provide() uses it), so onMount
-    // shouldn't carry it — keeps the warning message targeted.
-    expect(message).not.toContain('provide()')
-    warnSpy.mockRestore()
-  })
-})
-
-describe('captureCallSite — skip patterns cover published-bundle paths', () => {
-  // Real bug surfaced from a 0.24.1 consumer report: the pre-fix patterns
-  // only matched source-tree paths (`/lifecycle\.ts/`, `/core\/src\//`,
-  // etc.). Published packages bundle to `node_modules/@pyreon/<name>/lib/`,
-  // so the framework's own stack frames slipped through the filter — the
-  // warning's "Called from:" line pointed at the warning emitter itself
-  // (or the framework's lib code) instead of the user's call site.
-  //
-  // We can't easily synthesise a real published-bundle stack in a test,
-  // but we CAN exercise the regex set directly. The fix added matchers
-  // for `@pyreon/<name>/lib/` paths AND function-name matches
-  // (`captureCallSite`, `warnOutsideSetup`) that survive bundling.
-
-  // Mirror the patterns from the impl. (If this list drifts, the impl
-  // and this test fall out of sync — that's the regression signal.)
-  const skipPatterns = [
-    /\/lifecycle\.[tj]s/,
-    /\/context\.[tj]s/,
-    /\/component\.[tj]s/,
-    /\bcaptureCallSite\b/,
-    /\bwarnOutsideSetup\b/,
-    /\/(core|reactivity|runtime-dom|runtime-server|router|head|ui-core|styler|unistyle|rocketstyle|attrs|elements|kinetic)\/src\//,
-    /node_modules\/@pyreon\/[^/]+\/lib\//,
-    /@pyreon\/[a-z-]+\/lib\//,
-    /node:internal/,
-    /webpack-internal/,
-    /<anonymous>/,
-  ]
-
-  const isSkipped = (line: string): boolean =>
-    skipPatterns.some((p) => p.test(line))
-
-  test('skips published-bundle lib paths (`@pyreon/X/lib/`)', () => {
-    expect(
-      isSkipped(
-        'at HeadProvider (file:///app/node_modules/@pyreon/head/lib/index.js:42:7)',
-      ),
-    ).toBe(true)
-    expect(
-      isSkipped(
-        'at provide (file:///app/node_modules/@pyreon/core/lib/index.js:96:5)',
-      ),
-    ).toBe(true)
-    expect(
-      isSkipped(
-        'at ThemeProvider (file:///app/node_modules/@pyreon/styler/lib/index.js:24:3)',
-      ),
-    ).toBe(true)
-  })
-
-  test('skips workspace source paths (`bun` condition consumers)', () => {
-    expect(
-      isSkipped(
-        'at provide (/Users/me/proj/packages/core/core/src/context.ts:88:3)',
-      ),
-    ).toBe(true)
-    expect(
-      isSkipped('at HeadProvider (/Users/me/proj/packages/core/head/src/provider.ts:56:5)'),
-    ).toBe(true)
-    expect(
-      isSkipped(
-        'at RouterProvider (/Users/me/proj/packages/core/router/src/components.tsx:30:5)',
-      ),
-    ).toBe(true)
-  })
-
-  test('skips the warning infrastructure itself (function-name match)', () => {
-    // Even if the file path is mangled (minified / bundled to a single
-    // file like `lib/index.js`), the symbol names survive when bundlers
-    // preserve exports. The function-name pattern catches both.
-    expect(isSkipped('at captureCallSite (lib/index.js:22:33)')).toBe(true)
-    expect(isSkipped('at warnOutsideSetup (lib/index.js:55:21)')).toBe(true)
-    expect(isSkipped('    at captureCallSite (mangled-bundle.js:9999:11)')).toBe(true)
-  })
-
-  test('skips engine / anonymous frames', () => {
-    expect(isSkipped('at <anonymous>')).toBe(true)
-    expect(isSkipped('at runMicrotasks (node:internal/process/task_queues:96:5)')).toBe(true)
-  })
-
-  test('does NOT skip user-code paths in src/ or pages/ etc.', () => {
-    // These should ALL fall through the skip filter so captureCallSite
-    // returns them as the "Called from:" line — the user-actionable hint.
-    expect(isSkipped('at MyComponent (/Users/me/proj/src/components/Foo.tsx:42:15)')).toBe(false)
-    expect(isSkipped('at HomePage (/Users/me/proj/src/pages/Home.tsx:10:5)')).toBe(false)
-    expect(isSkipped('at NotFound (/Users/me/proj/src/routes/_not-found.tsx:5:7)')).toBe(false)
-  })
-
-  test('does NOT skip user code that happens to be in node_modules but NOT @pyreon/', () => {
-    // A user-installed third-party package's component shouldn't be
-    // skipped — only `@pyreon/*` framework bundles are silenced.
-    expect(
-      isSkipped('at SomeLib (file:///app/node_modules/some-third-party/lib/index.js:42:7)'),
-    ).toBe(false)
-  })
-})

package/src/tests/manifest-snapshot.test.ts

@@ -1,100 +0,0 @@
-import {
-  renderApiReferenceEntries,
-  renderLlmsFullSection,
-  renderLlmsTxtLine,
-} from '@pyreon/manifest'
-import coreManifest from '../manifest'
-
-describe('gen-docs — core snapshot', () => {
-  it('renders @pyreon/core to its expected llms.txt bullet', () => {
-    expect(renderLlmsTxtLine(coreManifest)).toMatchInlineSnapshot(`"- @pyreon/core — VNode, h(), Fragment, lifecycle, context, JSX runtime, Suspense, ErrorBoundary, lazy(), Dynamic, cx(), splitProps, mergeProps, createUniqueId. Pyreon components are plain functions that execute a single time. Reactivity comes from reading signals inside reactive scopes (JSX expression thunks, \`effect()\`, \`computed()\`), not from re-running the component function. \`if (!cond()) return null\` at the top level runs once and is static — use \`return (() => { if (!cond()) return null; return <div /> })\` for reactive conditional rendering."`)
-  })
-
-  it('renders @pyreon/core to its expected llms-full.txt section — full body snapshot', () => {
-    expect(renderLlmsFullSection(coreManifest)).toMatchInlineSnapshot(`
-      "## @pyreon/core — Complete API
-
-      Component model and lifecycle for Pyreon. Provides the VNode type system, \`h()\` hyperscript function, JSX automatic runtime (\`@pyreon/core/jsx-runtime\`), lifecycle hooks (\`onMount\`, \`onUnmount\`), two-tier context system (\`createContext\` for static values, \`createReactiveContext\` for signal-backed values), control-flow components (\`Show\`, \`Switch\`/\`Match\`, \`For\`, \`Suspense\`, \`ErrorBoundary\`), code-splitting via \`lazy()\`, dynamic rendering via \`Dynamic\`, and props utilities (\`splitProps\`, \`mergeProps\`, \`cx\`, \`createUniqueId\`). Components are plain functions (\`ComponentFn<P> = (props: P) => VNodeChild\`) that run ONCE — reactivity comes from reading signals inside reactive scopes, not from re-running the component.
-
-      \`\`\`typescript
-      import { h, Fragment, onMount, onUnmount, provide, createContext, createReactiveContext, useContext, Show, Switch, Match, For, Suspense, ErrorBoundary, lazy, Dynamic, cx, splitProps, mergeProps, createUniqueId, untrack } from "@pyreon/core"
-      import { signal, computed } from "@pyreon/reactivity"
-
-      // Context — static (destructure-safe) vs reactive (must call to read)
-      const ThemeCtx = createContext<"light" | "dark">("light")
-      const ModeCtx = createReactiveContext<"light" | "dark">("light")
-
-      const App = (props: { children: any }) => {
-        const mode = signal<"light" | "dark">("dark")
-        provide(ThemeCtx, "dark")                    // static — safe to destructure
-        provide(ModeCtx, () => mode())               // reactive — consumer must call
-
-        return <>{props.children}</>
-      }
-
-      // Lifecycle
-      const Timer = () => {
-        const count = signal(0)
-        onMount(() => {
-          const id = setInterval(() => count.update(n => n + 1), 1000)
-          return () => clearInterval(id)  // cleanup runs on unmount
-        })
-        return <div>{() => count()}</div>
-      }
-
-      // Control flow — reactive conditional rendering
-      const Page = (props: { items: { id: number; name: string }[]; loggedIn: () => boolean }) => (
-        <div>
-          <Show when={props.loggedIn()} fallback={<p>Please log in</p>}>
-            <For each={props.items} by={item => item.id}>
-              {item => <li>{item.name}</li>}
-            </For>
-          </Show>
-        </div>
-      )
-
-      // Props utilities — preserve reactivity
-      const Button = (props: { class?: string; size?: string; onClick: () => void; children: any }) => {
-        const [local, rest] = splitProps(props, ["class", "size"])
-        const merged = mergeProps({ size: "md" }, local)
-        const id = createUniqueId()
-        return <button id={id} {...rest} class={cx("btn", \`btn-\${merged.size}\`, local.class)} />
-      }
-
-      // Code splitting
-      const HeavyPage = lazy(() => import("./HeavyPage"))
-      const LazyApp = () => (
-        <Suspense fallback={<div>Loading...</div>}>
-          <HeavyPage />
-        </Suspense>
-      )
-      \`\`\`
-
-      > **Components run once**: Pyreon components are plain functions that execute a single time. Reactivity comes from reading signals inside reactive scopes (JSX expression thunks, \`effect()\`, \`computed()\`), not from re-running the component function. \`if (!cond()) return null\` at the top level runs once and is static — use \`return (() => { if (!cond()) return null; return <div /> })\` for reactive conditional rendering.
-      >
-      > **Destructuring props kills reactivity**: \`const { name } = props\` captures the value at setup time — it becomes static. Use \`props.name\` inside reactive scopes, or \`splitProps(props, ["name"])\` for rest patterns. The compiler handles \`const x = props.y; return <div>{x}</div>\` by inlining \`props.y\` back at the use site, but only for \`const\` (not \`let\`/\`var\`).
-      >
-      > **Two context types**: \`createContext<T>\` returns \`T\` from \`useContext()\` — safe to destructure. \`createReactiveContext<T>\` returns \`() => T\` — must call to read. Using the wrong one is a common source of stale-value bugs (static context for dynamic values) or unnecessary ceremony (reactive context for constants).
-      >
-      > **For uses by, not key**: The \`<For>\` component uses the \`by\` prop for its key function because JSX extracts \`key\` as a special VNode reconciliation prop. Writing \`<For each={items()} key={fn}>\` silently passes the key to the VNode system instead of the list reconciler.
-      >
-      > **JSX uses standard HTML attributes**: Use \`class\` not \`className\`, \`for\` not \`htmlFor\`, \`onInput\` not \`onChange\` for per-keystroke updates. Pyreon maps to native DOM events, not the React synthetic event system.
-      "
-    `)
-  })
-
-  it('renders @pyreon/core to MCP api-reference entries — one per api[] item', () => {
-    const record = renderApiReferenceEntries(coreManifest)
-    expect(Object.keys(record).length).toBe(31)
-    expect(Object.keys(record)).toContain('core/h')
-    // Compat-mode native marker — added so framework JSX components opt out
-    // of `@pyreon/{react,preact,vue,solid}-compat` wrapping.
-    expect(Object.keys(record)).toContain('core/nativeCompat')
-    expect(Object.keys(record)).toContain('core/isNativeCompat')
-    expect(Object.keys(record)).toContain('core/NATIVE_COMPAT_MARKER')
-    // Spot-check the flagship API — h() is the hyperscript function
-    const h = record['core/h']!
-    expect(h.notes).toContain('JSX')
-    expect(h.mistakes?.split('\n').length).toBeGreaterThan(2)
-  })
-})