@pyreon/vite-plugin 0.24.5 → 0.25.0

package/src/rocketstyle-collapse.ts

@@ -1,199 +0,0 @@
-/**
- * P0 — build-time rocketstyle-collapse resolver.
- *
- * For a collapsible call site (`<Button state="primary" size="md">Save</Button>`
- * — every dimension prop a string literal, children static text) this
- * resolves the FULL rocketstyle/styler pipeline ONCE by SSR-rendering the
- * REAL component, light AND dark, and returns: the resolved styler class
- * per mode, the styler rule text, and a class-stripped `_tpl` template.
- *
- * The render runs through a programmatic Vite SSR server bound to the
- * CONSUMER's own `vite.config` — so module resolution is identical to
- * the app's real build (workspace `bun` condition, app aliases,
- * app-local relative imports, whatever). Parity with the runtime-mounted
- * class is then guaranteed BY CONSTRUCTION: it is literally the same
- * `renderToString` + `@pyreon/styler` code path the client uses, and
- * styler's FNV-1a class hashing is identical in SSR and DOM (styler's
- * hydration contract). No reimplementation, no closure re-execution, no
- * drift (RFC decision 2).
- *
- * Every failure returns `null` (graceful bail → the call site keeps its
- * normal rocketstyle mount). Correct-but-slow is acceptable; wrong
- * output is not.
- */
-import type { InlineConfig, ViteDevServer } from 'vite'
-
-// Inline FNV-1a (same algorithm as @pyreon/styler/hash) — avoids pulling
-// the styler module graph into the vite-plugin's cheap entry path.
-function fnv1a(str: string): string {
-  let h = 2166136261
-  for (let i = 0; i < str.length; i++) {
-    h ^= str.charCodeAt(i)
-    h = Math.imul(h, 16777619)
-  }
-  return (h >>> 0).toString(36)
-}
-
-export interface CollapseImportSpec {
-  /** Imported binding name, e.g. `PyreonUI` / `theme` / `useMode`. */
-  name: string
-  /** Module specifier, e.g. `@pyreon/ui-core` / `@pyreon/ui-theme`. */
-  source: string
-}
-
-export interface CollapseConfig {
-  /** Theme/mode provider component. Default: PyreonUI from @pyreon/ui-core. */
-  provider: CollapseImportSpec
-  /** Theme object. Default: theme from @pyreon/ui-theme. */
-  theme: CollapseImportSpec
-  /** Live mode accessor — emitted into the collapsed site for dual-emit. */
-  mode: CollapseImportSpec
-}
-
-export const DEFAULT_COLLAPSE_CONFIG: CollapseConfig = {
-  provider: { name: 'PyreonUI', source: '@pyreon/ui-core' },
-  theme: { name: 'theme', source: '@pyreon/ui-theme' },
-  mode: { name: 'useMode', source: '@pyreon/ui-core' },
-}
-
-export interface ResolveInput {
-  /** The collapsible component's import. */
-  component: CollapseImportSpec
-  /** Literal dimension/HTML props, e.g. `{ state: 'primary', size: 'md' }`. */
-  props: Record<string, string>
-  /** Static text children (empty ⇒ no children). */
-  childrenText: string
-  config: CollapseConfig
-}
-
-export interface ResolvedCollapse {
-  /** Element HTML with the root `class="..."` removed (the `_tpl` template). */
-  templateHtml: string
-  lightClass: string
-  darkClass: string
-  /** Pre-resolved styler rule text (full snapshot) for `injectRules`. */
-  rules: string[]
-  /** FNV over the rule set — `injectRules` idempotency + cross-site dedupe. */
-  key: string
-}
-
-const FIRST_CLASS_RE = /^(\s*<[a-zA-Z][\w-]*)([^>]*?)\sclass="([^"]*)"([^>]*>)/
-
-/** Strip the FIRST element's `class="..."`, returning [stripped, class]. */
-export function stripRootClass(html: string): { stripped: string; cls: string } | null {
-  const m = FIRST_CLASS_RE.exec(html)
-  if (!m) return null
-  const stripped = html.replace(FIRST_CLASS_RE, '$1$2$4')
-  return { stripped, cls: m[3] ?? '' }
-}
-
-/**
- * Pure extraction half — given the two rendered HTML strings and the
- * styler rule snapshot, derive the ResolvedCollapse (or null on a shape
- * the slice doesn't collapse). Separated for direct unit-testing without
- * spinning Vite.
- */
-export function deriveCollapse(
-  lightHtml: string,
-  darkHtml: string,
-  rules: string[],
-): ResolvedCollapse | null {
-  const light = stripRootClass(lightHtml)
-  const dark = stripRootClass(darkHtml)
-  if (!light || !dark || !light.cls || !dark.cls) return null
-  // The structural template must be identical between modes (only the
-  // class differs). Divergent markup ⇒ not a simple single-root
-  // collapsible — bail.
-  if (light.stripped !== dark.stripped) return null
-  return {
-    templateHtml: light.stripped.trim(),
-    lightClass: light.cls,
-    darkClass: dark.cls,
-    rules,
-    key: fnv1a(rules.join('\u0000')),
-  }
-}
-
-export interface CollapseResolver {
-  resolve(input: ResolveInput): Promise<ResolvedCollapse | null>
-  dispose(): Promise<void>
-}
-
-/**
- * Create a resolver backed by ONE programmatic Vite SSR server bound to
- * `projectRoot`'s vite config. Reused across every call site in a build;
- * `dispose()` at buildEnd. Module loads are cached by Vite's own SSR
- * module graph (provider/theme/component import once).
- */
-export async function createCollapseResolver(projectRoot: string): Promise<CollapseResolver> {
-  const { createServer } = (await import('vite')) as typeof import('vite')
-  const inline: InlineConfig = {
-    // No `configFile` override — Vite auto-loads the project's own
-    // vite.config from `root`, so module resolution (workspace `bun`
-    // condition, app aliases) matches the real build exactly.
-    root: projectRoot,
-    server: { middlewareMode: true },
-    appType: 'custom',
-    logLevel: 'silent',
-    optimizeDeps: { noDiscovery: true, include: [] },
-  }
-  let server: ViteDevServer | null = await createServer(inline)
-
-  // Resolved-bundle cache — identical input must hit the same result
-  // without a second double-render (deterministic by construction).
-  const cache = new Map<string, ResolvedCollapse | null>()
-
-  async function load(spec: string): Promise<Record<string, unknown>> {
-    return (await server!.ssrLoadModule(spec)) as Record<string, unknown>
-  }
-
-  return {
-    async resolve(input) {
-      const ck = JSON.stringify([
-        input.component,
-        input.props,
-        input.childrenText,
-        input.config.provider,
-        input.config.theme,
-      ])
-      if (cache.has(ck)) return cache.get(ck) ?? null
-      try {
-        if (!server) return null
-        const rs = await load('@pyreon/runtime-server')
-        const core = await load('@pyreon/core')
-        const styler = await load('@pyreon/styler')
-        const prov = await load(input.config.provider.source)
-        const thm = await load(input.config.theme.source)
-        const comp = await load(input.component.source)
-        const renderToString = rs.renderToString as (n: unknown) => Promise<string>
-        const h = core.h as (t: unknown, p: unknown, ...c: unknown[]) => unknown
-        const Provider = prov[input.config.provider.name]
-        const themeVal = thm[input.config.theme.name]
-        const Component = comp[input.component.name]
-        const sheet = styler.sheet as { getStyleRules(): readonly string[] }
-        if (typeof Component !== 'function' || Provider == null || themeVal == null) {
-          cache.set(ck, null)
-          return null
-        }
-        const childArgs = input.childrenText ? [input.childrenText] : []
-        const node = (mode: string) =>
-          h(Provider, { theme: themeVal, mode }, h(Component, input.props, ...childArgs))
-        const lightHtml = await renderToString(node('light'))
-        const darkHtml = await renderToString(node('dark'))
-        const rules = sheet.getStyleRules().slice()
-        const result = deriveCollapse(lightHtml, darkHtml, rules)
-        cache.set(ck, result)
-        return result
-      } catch {
-        cache.set(ck, null)
-        return null
-      }
-    },
-    async dispose() {
-      const s = server
-      server = null
-      cache.clear()
-      if (s) await s.close()
-    },
-  }
-}

package/src/tests/cache-eviction-on-delete.test.ts

@@ -1,187 +0,0 @@
-/**
- * REPRODUCTION + REGRESSION — `signalExportRegistry`, `resolveCache`,
- * and `islandRegistry` accumulated stale entries for the lifetime of
- * a `vite dev` session. Vite's `watchChange` hook fires on filesystem
- * `'create' | 'update' | 'delete'` events; pre-fix none of the four
- * per-instance caches subscribed, so deleting / renaming a source
- * file left orphaned entries forever.
- *
- * Bounded by total source-tree size in practice, but a real Class C
- * leak over hours of editing on a large project — every source file
- * the developer touches that later gets deleted leaves one entry per
- * cache stuck until process exit.
- */
-import { describe, expect, it } from 'vitest'
-import type { PyreonPluginOptions } from '../index'
-import pyreonPlugin from '../index'
-
-type ConfigHook = (
-  userConfig: Record<string, unknown>,
-  env: { command: string; isSsrBuild?: boolean },
-) => Record<string, unknown>
-
-function createServePlugin(opts?: PyreonPluginOptions) {
-  const plugin = pyreonPlugin(opts)
-  ;(plugin.config as unknown as ConfigHook)({}, { command: 'serve' })
-  return plugin
-}
-
-interface PluginInternalShape {
-  buildStart: () => Promise<void> | void
-  transform: (
-    this: {
-      warn: (msg: string) => void
-      resolve: (
-        id: string,
-        importer?: string,
-        opts?: { skipSelf: boolean },
-      ) => Promise<{ id: string } | null>
-    },
-    code: string,
-    id: string,
-  ) => Promise<{ code: string; map: null } | undefined>
-  watchChange: (id: string, change: { event: 'create' | 'update' | 'delete' }) => void
-}
-
-interface PluginCaches {
-  signalExportRegistry: Map<string, Set<string>>
-  resolveCache: Map<string, string | null>
-  pyreonWorkspaceDirCache: Map<string, boolean>
-  islandRegistry: Map<string, unknown[]>
-}
-
-const CACHES_SYMBOL = Symbol.for('pyreon/vite-plugin:caches')
-
-function getCaches(plugin: ReturnType<typeof pyreonPlugin>): PluginCaches {
-  const caches = (plugin as unknown as Record<symbol, PluginCaches | undefined>)[CACHES_SYMBOL]
-  if (!caches) throw new Error('plugin should expose CACHES_SYMBOL')
-  return caches
-}
-
-async function transform(
-  plugin: ReturnType<typeof pyreonPlugin>,
-  code: string,
-  id: string,
-): Promise<void> {
-  const p = plugin as unknown as PluginInternalShape
-  await p.transform.call(
-    {
-      warn: () => {},
-      resolve: async (specifier: string, importer?: string) => {
-        // Simulate resolution — bare relative imports map to virtual
-        // paths so resolveCache gets real entries.
-        if (specifier.startsWith('./') && importer) {
-          return { id: `/test-project/${specifier.slice(2)}.ts` }
-        }
-        return null
-      },
-    },
-    code,
-    id,
-  )
-}
-
-describe('@pyreon/vite-plugin — file-delete cache eviction (watchChange)', () => {
-  it('REGRESSION: signalExportRegistry entry is evicted on file delete', async () => {
-    const plugin = createServePlugin()
-    const p = plugin as unknown as PluginInternalShape
-    const caches = getCaches(plugin)
-
-    // Transform a source file that exports a top-level signal. The
-    // plugin's incremental scanner populates the registry.
-    await transform(plugin, `export const count = signal(0)`, '/test-project/store.tsx')
-    expect(caches.signalExportRegistry.has('/test-project/store.tsx')).toBe(true)
-
-    // Fire the delete event. The critical assertion: the registry
-    // entry is GONE post-delete. Pre-fix (no watchChange hook), the
-    // entry would persist forever.
-    p.watchChange('/test-project/store.tsx', { event: 'delete' })
-    expect(caches.signalExportRegistry.has('/test-project/store.tsx')).toBe(false)
-  })
-
-  it('REGRESSION: resolveCache entries pointing at the deleted file are evicted', async () => {
-    const plugin = createServePlugin()
-    const p = plugin as unknown as PluginInternalShape
-    const caches = getCaches(plugin)
-
-    // Populate signalExportRegistry first.
-    await transform(plugin, `export const a = signal(0)`, '/test-project/a.tsx')
-    // Consumer file imports a.ts — populates resolveCache.
-    await transform(
-      plugin,
-      `import { a } from './a'\nexport default () => a`,
-      '/test-project/consumer.tsx',
-    )
-
-    const beforeSize = caches.resolveCache.size
-    expect(beforeSize).toBeGreaterThan(0)
-
-    // Delete `a.ts`. Both the importer-keyed entry AND any entry
-    // whose VALUE is `/test-project/a.tsx` should evict.
-    p.watchChange('/test-project/a.tsx', { event: 'delete' })
-
-    // Critical: no entry in resolveCache references the deleted file.
-    for (const [key, value] of caches.resolveCache) {
-      expect(key.startsWith('/test-project/a.tsx::')).toBe(false)
-      expect(value).not.toBe('/test-project/a.tsx')
-    }
-  })
-
-  it('REGRESSION: islandRegistry entry is evicted on file delete', async () => {
-    const plugin = createServePlugin({ islands: true })
-    const p = plugin as unknown as PluginInternalShape
-    const caches = getCaches(plugin)
-
-    // Populate the island registry. Use a minimal island declaration.
-    await transform(
-      plugin,
-      `import { island } from '@pyreon/server'\nexport const C = island(() => import('./c'), { name: 'C' })`,
-      '/test-project/c-island.tsx',
-    )
-
-    // Either the absolute id or its normalized form may have landed
-    // in the registry — assert at least one is there.
-    const hasEntry
-      = caches.islandRegistry.has('/test-project/c-island.tsx')
-      || [...caches.islandRegistry.keys()].some((k) => k.includes('c-island'))
-
-    if (hasEntry) {
-      p.watchChange('/test-project/c-island.tsx', { event: 'delete' })
-      // Post-delete the registry should NOT have the entry.
-      expect(caches.islandRegistry.has('/test-project/c-island.tsx')).toBe(false)
-    } else {
-      // If the scanner didn't pick up the island (test fixture too
-      // minimal), the watchChange call must still be a no-op without
-      // throwing — verifies the defensive path.
-      expect(() =>
-        p.watchChange('/test-project/c-island.tsx', { event: 'delete' }),
-      ).not.toThrow()
-    }
-  })
-
-  it('REGRESSION: watchChange ignores create/update events (handled by transform)', async () => {
-    const plugin = createServePlugin()
-    const p = plugin as unknown as PluginInternalShape
-    const caches = getCaches(plugin)
-
-    // Populate then update — update should NOT evict.
-    await transform(plugin, `export const v = signal(0)`, '/test-project/v.tsx')
-    expect(caches.signalExportRegistry.has('/test-project/v.tsx')).toBe(true)
-    p.watchChange('/test-project/v.tsx', { event: 'create' })
-    expect(caches.signalExportRegistry.has('/test-project/v.tsx')).toBe(true)
-    p.watchChange('/test-project/v.tsx', { event: 'update' })
-    expect(caches.signalExportRegistry.has('/test-project/v.tsx')).toBe(true)
-
-    // Only delete evicts.
-    p.watchChange('/test-project/v.tsx', { event: 'delete' })
-    expect(caches.signalExportRegistry.has('/test-project/v.tsx')).toBe(false)
-  })
-
-  it('REGRESSION: deleting an untracked file is a safe no-op', () => {
-    const plugin = createServePlugin()
-    const p = plugin as unknown as PluginInternalShape
-    expect(() =>
-      p.watchChange('/test-project/never-tracked.tsx', { event: 'delete' }),
-    ).not.toThrow()
-  })
-})

package/src/tests/compat-resolve.test.ts

@@ -1,260 +0,0 @@
-/**
- * Compat-mode `resolveId` and `getCompatTarget` coverage for
- * @pyreon/vite-plugin (PR #323). The existing test file covers
- * compat-mode `transform` short-circuiting; this covers the
- * resolveId hook + the JSX-runtime aliasing branch.
- */
-
-import { resolve } from 'node:path'
-import { describe, expect, it } from 'vitest'
-import pyreonPlugin, { type PyreonPluginOptions } from '../index'
-
-type ConfigHook = (
-  userConfig: Record<string, unknown>,
-  env: { command: string; isSsrBuild?: boolean },
-) => Record<string, unknown>
-
-type ResolveIdCtx = {
-  resolve: (
-    id: string,
-    importer?: string,
-    options?: { skipSelf: boolean },
-  ) => Promise<{ id: string } | null>
-}
-type ResolveIdHook = (
-  this: ResolveIdCtx,
-  id: string,
-  importer?: string,
-) => Promise<string | undefined>
-
-function bootstrap(opts?: PyreonPluginOptions) {
-  const plugin = pyreonPlugin(opts)
-  ;(plugin.config as unknown as ConfigHook)({}, { command: 'serve' })
-  return plugin
-}
-
-async function callResolveId(
-  plugin: ReturnType<typeof pyreonPlugin>,
-  id: string,
-  resolveMap: Record<string, string> = {},
-  importer?: string,
-): Promise<string | undefined> {
-  const hook = plugin.resolveId as ResolveIdHook
-  return hook.call(
-    {
-      resolve: async (specifier: string) => {
-        const resolved = resolveMap[specifier]
-        return resolved ? { id: resolved } : null
-      },
-    },
-    id,
-    importer,
-  )
-}
-
-describe('compat-mode resolveId — react', () => {
-  it('redirects "react" → @pyreon/react-compat', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(plugin, 'react', {
-      '@pyreon/react-compat': '/abs/react-compat/index.ts',
-    })
-    expect(resolved).toBe('/abs/react-compat/index.ts')
-  })
-
-  it('redirects "react/jsx-runtime" → @pyreon/react-compat/jsx-runtime', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(plugin, 'react/jsx-runtime', {
-      '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-
-  it('redirects @pyreon/core/jsx-runtime → @pyreon/react-compat/jsx-runtime in react compat', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(plugin, '@pyreon/core/jsx-runtime', {
-      '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-
-  it('redirects @pyreon/core/jsx-dev-runtime in react compat', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(plugin, '@pyreon/core/jsx-dev-runtime', {
-      '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-
-  it('returns undefined for non-aliased imports', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(plugin, 'lodash', {})
-    expect(resolved).toBeUndefined()
-  })
-})
-
-describe('compat-mode resolveId — preact', () => {
-  it('redirects "preact" → @pyreon/preact-compat', async () => {
-    const plugin = bootstrap({ compat: 'preact' })
-    const resolved = await callResolveId(plugin, 'preact', {
-      '@pyreon/preact-compat': '/abs/preact-compat/index.ts',
-    })
-    expect(resolved).toBe('/abs/preact-compat/index.ts')
-  })
-
-  it('redirects "preact/hooks" → @pyreon/preact-compat/hooks', async () => {
-    const plugin = bootstrap({ compat: 'preact' })
-    const resolved = await callResolveId(plugin, 'preact/hooks', {
-      '@pyreon/preact-compat/hooks': '/abs/preact-compat/hooks.ts',
-    })
-    expect(resolved).toBe('/abs/preact-compat/hooks.ts')
-  })
-
-  it('redirects @pyreon/core/jsx-runtime in preact compat', async () => {
-    const plugin = bootstrap({ compat: 'preact' })
-    const resolved = await callResolveId(plugin, '@pyreon/core/jsx-runtime', {
-      '@pyreon/preact-compat/jsx-runtime': '/abs/preact-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/preact-compat/jsx-runtime.ts')
-  })
-
-  it('redirects @preact/signals → @pyreon/preact-compat/signals', async () => {
-    const plugin = bootstrap({ compat: 'preact' })
-    const resolved = await callResolveId(plugin, '@preact/signals', {
-      '@pyreon/preact-compat/signals': '/abs/preact-compat/signals.ts',
-    })
-    expect(resolved).toBe('/abs/preact-compat/signals.ts')
-  })
-})
-
-describe('compat-mode resolveId — vue', () => {
-  it('redirects "vue" → @pyreon/vue-compat', async () => {
-    const plugin = bootstrap({ compat: 'vue' })
-    const resolved = await callResolveId(plugin, 'vue', {
-      '@pyreon/vue-compat': '/abs/vue-compat/index.ts',
-    })
-    expect(resolved).toBe('/abs/vue-compat/index.ts')
-  })
-
-  it('redirects @pyreon/core/jsx-runtime in vue compat', async () => {
-    const plugin = bootstrap({ compat: 'vue' })
-    const resolved = await callResolveId(plugin, '@pyreon/core/jsx-runtime', {
-      '@pyreon/vue-compat/jsx-runtime': '/abs/vue-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/vue-compat/jsx-runtime.ts')
-  })
-})
-
-describe('compat-mode resolveId — solid', () => {
-  it('redirects "solid-js" → @pyreon/solid-compat', async () => {
-    const plugin = bootstrap({ compat: 'solid' })
-    const resolved = await callResolveId(plugin, 'solid-js', {
-      '@pyreon/solid-compat': '/abs/solid-compat/index.ts',
-    })
-    expect(resolved).toBe('/abs/solid-compat/index.ts')
-  })
-
-  it('redirects @pyreon/core/jsx-runtime in solid compat', async () => {
-    const plugin = bootstrap({ compat: 'solid' })
-    const resolved = await callResolveId(plugin, '@pyreon/core/jsx-runtime', {
-      '@pyreon/solid-compat/jsx-runtime': '/abs/solid-compat/jsx-runtime.ts',
-    })
-    expect(resolved).toBe('/abs/solid-compat/jsx-runtime.ts')
-  })
-})
-
-describe('compat-mode resolveId — framework-importer carve-out', () => {
-  // Regression: in compat mode, `@pyreon/core/jsx-runtime` must NOT be
-  // redirected to the compat package when the importer is itself a
-  // `@pyreon/*` workspace-package source file (zero, router, runtime-dom,
-  // etc.). Pre-fix, OXC's project-wide importSource was set to the compat
-  // package, so framework-internal JSX got rewritten to import a runtime
-  // shape it doesn't speak. The fix sets OXC to `@pyreon/core` always and
-  // redirects in `resolveId` only for non-framework importers. Caught by
-  // `cpa-smoke-app-*-compat` cells in `scripts/scaffold-smoke.ts`.
-  // Bisect-verified: dropping the `isPyreonWorkspaceFile(importer)` guard
-  // makes these tests fail with the redirected jsx-runtime path.
-
-  const repoRoot = resolve(import.meta.dirname, '../../../../..')
-  const frameworkImporter = `${repoRoot}/packages/zero/zero/src/link.tsx`
-  const userImporter = `${repoRoot}/examples/some-user-app/src/foo.tsx`
-  // The 4 existing compat-layer example apps under `examples/` have
-  // package.json names like `@pyreon/example-react-compat`. The carve-out
-  // helper must NOT treat their source files as framework files — doing so
-  // skips the JSX-runtime redirect and breaks the compat layer end-to-end.
-  // Bisect-verified: when the helper checked `name.startsWith('@pyreon/')`
-  // alone (without the `/examples/` exclusion), all 4 compat-layer e2e
-  // suites failed in CI with `section.demo` never rendering.
-  const exampleAppImporter = `${repoRoot}/examples/react-compat/src/Foo.tsx`
-
-  it('does NOT redirect @pyreon/core/jsx-runtime when imported FROM @pyreon/zero workspace source (react)', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(
-      plugin,
-      '@pyreon/core/jsx-runtime',
-      { '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts' },
-      frameworkImporter,
-    )
-    expect(resolved).toBeUndefined() // pass through to Vite's resolver
-  })
-
-  it('does NOT redirect @pyreon/core/jsx-dev-runtime when imported FROM framework source (preact)', async () => {
-    const plugin = bootstrap({ compat: 'preact' })
-    const resolved = await callResolveId(
-      plugin,
-      '@pyreon/core/jsx-dev-runtime',
-      { '@pyreon/preact-compat/jsx-runtime': '/abs/preact-compat/jsx-runtime.ts' },
-      frameworkImporter,
-    )
-    expect(resolved).toBeUndefined()
-  })
-
-  it('STILL redirects @pyreon/core/jsx-runtime when imported FROM user code (react)', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(
-      plugin,
-      '@pyreon/core/jsx-runtime',
-      { '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts' },
-      userImporter,
-    )
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-
-  it('STILL redirects @pyreon/core/jsx-runtime when no importer (entry point)', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(
-      plugin,
-      '@pyreon/core/jsx-runtime',
-      { '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts' },
-    )
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-
-  it('STILL redirects @pyreon/core/jsx-runtime when imported FROM an example app under examples/ (e.g. @pyreon/example-react-compat)', async () => {
-    const plugin = bootstrap({ compat: 'react' })
-    const resolved = await callResolveId(
-      plugin,
-      '@pyreon/core/jsx-runtime',
-      { '@pyreon/react-compat/jsx-runtime': '/abs/react-compat/jsx-runtime.ts' },
-      exampleAppImporter,
-    )
-    expect(resolved).toBe('/abs/react-compat/jsx-runtime.ts')
-  })
-})
-
-describe('compat-mode resolveId — no compat', () => {
-  it('returns undefined for any framework alias when compat is unset', async () => {
-    const plugin = bootstrap()
-    expect(await callResolveId(plugin, 'react', {})).toBeUndefined()
-    expect(await callResolveId(plugin, 'vue', {})).toBeUndefined()
-    expect(await callResolveId(plugin, 'preact', {})).toBeUndefined()
-    expect(await callResolveId(plugin, 'solid-js', {})).toBeUndefined()
-  })
-
-  it('still resolves the HMR runtime virtual id (independent of compat)', async () => {
-    const plugin = bootstrap()
-    const resolved = await callResolveId(plugin, 'virtual:pyreon/hmr-runtime', {})
-    // Internal ID — has the leading '\0' marker convention or similar
-    expect(resolved).toBeDefined()
-    expect(typeof resolved).toBe('string')
-  })
-})