@hyperspan/plugin-preact 1.0.4 → 2.0.0-alpha.1

package/package.json

@@ -1,35 +1,44 @@
 {
   "name": "@hyperspan/plugin-preact",
-  "version": "1.0.4",
+  "version": "2.0.0-alpha.1",
   "description": "Hyperspan Plugin for Preact",
+  "type": "module",
   "main": "src/index.ts",
   "types": "src/index.ts",
   "public": true,
   "publishConfig": {
-    "access": "public"
+    "access": "public",
+    "tag": "alpha"
+  },
+  "exports": {
+    ".": {
+      "types": "./src/index.ts",
+      "default": "./src/index.ts"
+    }
   },
   "author": "Vance Lucas <vance@vancelucas.com>",
   "license": "BSD-3-Clause",
-  "keywords": [],
   "homepage": "https://www.hyperspan.dev/docs/clientjs/islands",
   "bugs": {
     "url": "https://github.com/vlucas/hyperspan/issues"
   },
   "scripts": {
-    "test": "bun test"
+    "test": "vitest run"
   },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/vlucas/hyperspan.git"
   },
   "devDependencies": {
-    "@types/bun": "^1.3.8",
     "prettier": "^3.8.1",
-    "typescript": "^5.9.3"
+    "typescript": "^5.9.3",
+    "vite": "^6.3.5",
+    "vitest": "^3.2.4"
   },
   "peerDependencies": {
-    "@hyperspan/html": "^1.0.0",
-    "@hyperspan/framework": "^1.0.1"
+    "@hyperspan/html": "^2.0.0-alpha.1",
+    "@hyperspan/framework": "^2.0.0-alpha.1",
+    "vite": "^6.0.0 || ^7.0.0"
   },
   "dependencies": {
     "debug": "^4.4.3",

package/src/index.test.ts

@@ -1,4 +1,4 @@
-import { test, describe, expect } from 'bun:test';
+import { test, describe, expect } from 'vitest';
 import { h } from 'preact';
 import { buildIslandHtml, renderPreactSSR, renderPreactIsland } from './index';
 
@@ -91,9 +91,7 @@ describe('renderPreactSSR', () => {
 
 describe('renderPreactIsland', () => {
   test('throws when component has no __HS_ISLAND property', () => {
-    expect(() => renderPreactIsland(Hello, {})).toThrow(
-      'was not loaded with an island plugin'
-    );
+    expect(() => renderPreactIsland(Hello, {})).toThrow('was not loaded with an island plugin');
   });
 
   test('returns an html_safe object', () => {
@@ -179,10 +177,20 @@ describe('renderPreactIsland', () => {
     (Hello as any).__HS_ISLAND = {
       id: jsId,
       render: (props: any, options: any = {}) =>
-        buildIslandHtml(jsId, 'Hello', 'hello', 'console.log(1)', renderPreactSSR(Hello, props), options),
+        buildIslandHtml(
+          jsId,
+          'Hello',
+          'hello',
+          'console.log(1)',
+          renderPreactSSR(Hello, props),
+          options
+        ),
     };
 
-    const result = renderPreactIsland(Hello, { name: 'World' }, { ssr: true, loading: 'lazy' } as any);
+    const result = renderPreactIsland(Hello, { name: 'World' }, {
+      ssr: true,
+      loading: 'lazy',
+    } as any);
     expect(result.content).toContain('data-loading="lazy"');
     expect(result.content).toContain('<template>');
     expect(result.content).toContain('Hello World!');

package/src/index.ts

@@ -1,7 +1,8 @@
-import { JS_IMPORT_MAP, JS_ISLAND_PUBLIC_PATH } from '@hyperspan/framework/client/js';
+import type { Plugin } from 'vite';
+import { resolve, basename, dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { assetHash } from '@hyperspan/framework/utils';
-import { IS_PROD } from '@hyperspan/framework/server';
-import { resolve } from 'node:path';
+import { registerImport, JS_ISLAND_PUBLIC_PATH } from '@hyperspan/framework/client/js';
 import type { Hyperspan as HS } from '@hyperspan/framework';
 import { html } from '@hyperspan/html';
 import { h } from 'preact';
@@ -10,36 +11,6 @@ import debug from 'debug';
 
 const log = debug('hyperspan:plugin-preact');
 
-const CWD = process.cwd();
-
-/** Dev: stable `[name].js` via Bun default. Prod: hashed filenames for caching. */
-const ISLAND_JS_NAMING = IS_PROD ? '[dir]/[name]-[hash].[ext]' : undefined;
-
-function islandBundleBaseName(outputPath: string): string {
-  return String(outputPath.split('/').reverse()[0]!.replace(/\.js$/i, ''));
-}
-
-function pickEntryPointJsOutput(
-  outputs: ReadonlyArray<{ path: string; kind?: string }>,
-  entrySourcePath: string
-): { path: string } {
-  const js = outputs.filter((o) => o.path.endsWith('.js'));
-  const entry = js.find((o) => o.kind === 'entry-point');
-  if (entry) return entry;
-  const sourceBase = entrySourcePath.split('/').pop()!.replace(/\.(tsx|ts|jsx|js)$/i, '');
-  const byName = js.find((o) => {
-    const b = islandBundleBaseName(o.path);
-    return b === sourceBase || b.startsWith(`${sourceBase}-`);
-  });
-  if (byName) return byName;
-  if (js[0]) return js[0];
-  throw new Error('[Hyperspan] Preact island build produced no JS output');
-}
-
-/**
- * Build the island wrapper HTML: a div for SSR content + a module script tag for client hydration.
- * Exported so it can be imported by generated island module code and used directly in tests.
- */
 export function buildIslandHtml(
   jsId: string,
   componentName: string,
@@ -55,159 +26,94 @@ export function buildIslandHtml(
   return `<div id="${jsId}">${ssrContent}</div>\n${scriptTag}`;
 }
 
-/**
- * Render a Preact component to an HTML string (SSR).
- * Exported for direct use in tests and external tooling.
- */
-export function renderPreactSSR(Component: any, props: any = {}): string {
-  return preactRenderToString(h(Component, props));
+export function renderPreactSSR(Component: unknown, props: Record<string, unknown> = {}): string {
+  return preactRenderToString(h(Component as Parameters<typeof h>[0], props));
 }
 
-// External ESM = https://esm.sh/preact@10.26.4/compat
-type PreactIslandCacheEntry = { contents: string; esmName: string };
-
-const PREACT_ISLAND_CACHE = new Map<string, PreactIslandCacheEntry>();
-
-/**
- * Build Preact client JS and copy to public folder
- */
-async function copyPreactToPublicFolder(config: HS.Config) {
-  const currentNodeEnv = process.env.NODE_ENV || 'production';
-  const sourceFile = resolve(import.meta.dir, './preact-client.ts');
-  const outdir = resolve(CWD, config.publicDir, JS_ISLAND_PUBLIC_PATH.replace(/^\//, ''));
-
-  // Preact client JS is always production mode
-  process.env.NODE_ENV = 'production';
-  const result = await Bun.build({
-    entrypoints: [sourceFile],
-    outdir,
-    naming: ISLAND_JS_NAMING,
-    minify: true,
-    format: 'esm',
-    target: 'browser',
-  });
-  process.env.NODE_ENV = currentNodeEnv;
-
-  const preactEntry = pickEntryPointJsOutput(result.outputs, sourceFile);
-  const builtFileName = islandBundleBaseName(preactEntry.path);
-  const builtFilePath = `${JS_ISLAND_PUBLIC_PATH}/${builtFileName}.js`;
-
-  JS_IMPORT_MAP.set('preact', builtFilePath);
-  JS_IMPORT_MAP.set('preact/compat', builtFilePath);
-  JS_IMPORT_MAP.set('preact/hooks', builtFilePath);
-  JS_IMPORT_MAP.set('preact/jsx-runtime', builtFilePath);
-  JS_IMPORT_MAP.set('preact/jsx-dev-runtime', builtFilePath);
-
-  if (!JS_IMPORT_MAP.has('react')) {
-    JS_IMPORT_MAP.set('react', builtFilePath);
-    JS_IMPORT_MAP.set('react-dom', builtFilePath);
+function extractDefaultExportName(code: string): string | null {
+  const patterns = [
+    /export\{([^\s]+) as default\}/,
+    /export default function\s+([^\s(]+)/,
+    /export default function\s*\(/,
+    /export default const\s+([^\s=]+)/,
+    /export default class\s+([^\s{]+)/,
+  ];
+  for (const re of patterns) {
+    const m = code.match(re);
+    if (m?.[1]) return m[1];
   }
+  if (/export default function/.test(code)) return 'DefaultComponent';
+  const anyMatch = code.match(/export default\s+([A-Za-z_$][\w$]*)/);
+  return anyMatch?.[1] ?? null;
+}
+
+function isIslandComponent(id: string): boolean {
+  return (
+    (id.includes('/app/components/') || id.includes('\\app\\components\\')) &&
+    id.endsWith('.tsx') &&
+    !id.includes('node_modules')
+  );
 }
 
 /**
- * Hyperspan Preact Plugin
+ * Vite plugin for Preact islands.
  */
-export function preactPlugin(): HS.Plugin {
-  return async (config: HS.Config) => {
-    try {
-      log('plugin loaded');
-      // Ensure Preact can be loaded on the client
-      if (!JS_IMPORT_MAP.has('preact')) {
-        await copyPreactToPublicFolder(config);
-      }
-
-      // Define a Bun plugin to handle .tsx files
-      await Bun.plugin({
-        name: 'Hyperspan Preact Loader',
-        async setup(build) {
-          // when a .tsx file is imported...
-          build.onLoad({ filter: /\.tsx$/ }, async (args) => {
-            log('tsx file loaded', args.path);
-            const jsId = assetHash(args.path);
+export function preactVitePlugin(): Plugin {
+  const preactClientPath = resolve(dirname(fileURLToPath(import.meta.url)), './preact-client.ts');
 
-            if (!JS_IMPORT_MAP.has('preact')) {
-              await copyPreactToPublicFolder(config);
-            }
+  return {
+    name: 'hyperspan-preact',
+    enforce: 'pre',
 
-            // Cache: Avoid re-processing the same file
-            if (PREACT_ISLAND_CACHE.has(jsId)) {
-              const hit = PREACT_ISLAND_CACHE.get(jsId)!;
-              JS_IMPORT_MAP.set(hit.esmName, `${JS_ISLAND_PUBLIC_PATH}/${hit.esmName}.js`);
-              log('tsx file cached', args.path);
-              return {
-                contents: hit.contents,
-                loader: 'js',
-              };
-            }
-
-            log('tsx file not cached, building...', args.path);
-            // We need to build the file to ensure we can ship it to the client with dependencies
-            // Ironic, right? Calling Bun.build() inside of a plugin that runs on Bun.build()?
-            const islandOutdir = resolve(CWD, config.publicDir, JS_ISLAND_PUBLIC_PATH.replace(/^\//, ''));
-            const result = await Bun.build({
-              entrypoints: [args.path],
-              outdir: islandOutdir,
-              naming: ISLAND_JS_NAMING,
-              external: Array.from(JS_IMPORT_MAP.keys()),
-              minify: true,
-              format: 'esm',
-              target: 'browser',
-              env: 'APP_PUBLIC_*',
-            });
-
-            const entryOut = pickEntryPointJsOutput(result.outputs, args.path);
-            const esmName = islandBundleBaseName(entryOut.path);
-
-            // Add output file to import map
-            JS_IMPORT_MAP.set(esmName, `${JS_ISLAND_PUBLIC_PATH}/${esmName}.js`);
-            log('added to import map', esmName, `${JS_ISLAND_PUBLIC_PATH}/${esmName}.js`);
-
-            let contents = await Bun.file(entryOut.path).text();
-
-            // Look for the default export
-            const RE_EXPORT_DEFAULT = /export\{([^\s]+) as default\}/;
-            const RE_EXPORT_DEFAULT_FN = /export default function\s+([^\s]+)/;
-            const RE_EXPORT_DEFAULT_CONST = /export default const\s+([^\s]+)/;
-            const RE_EXPORT_DEFAULT_ANY = /export default\s+([^\s]+)/;
-
-            const exportedDefault = contents.match(RE_EXPORT_DEFAULT);
-            const exportedDefaultFn = contents.match(RE_EXPORT_DEFAULT_FN);
-            const exportedDefaultConst = contents.match(RE_EXPORT_DEFAULT_CONST);
-            const exportedDefaultAny = contents.match(RE_EXPORT_DEFAULT_ANY);
-
-            const componentName =
-              exportedDefault?.[1] ||
-              exportedDefaultFn?.[1] ||
-              exportedDefaultConst?.[1] ||
-              exportedDefaultAny?.[1];
-
-            if (!componentName) {
-              log('ERROR: no default export found', args.path);
-              throw new Error(
-                `No default export found in ${args.path}. Did you forget to export a component?`
-              );
-            }
+    config() {
+      return {
+        optimizeDeps: {
+          include: ['preact', 'preact/hooks', 'preact/jsx-runtime'],
+        },
+      };
+    },
+
+    configureServer() {
+      // In Vite serve, Preact is resolved from node_modules / optimizeDeps.
+      // Import-map aliases for the browser bundle are set during production build.
+      const clientUrl = '/islands/preact-client.js';
+      registerImport('preact', clientUrl);
+      registerImport('preact/hooks', clientUrl);
+      registerImport('preact/jsx-runtime', clientUrl);
+      registerImport('preact/jsx-dev-runtime', clientUrl);
+      registerImport('preact/compat', clientUrl);
+      registerImport('react', clientUrl);
+      registerImport('react-dom', clientUrl);
+    },
+
+    async transform(code, id) {
+      if (!isIslandComponent(id)) return;
+
+      log('transform island', id);
+      const jsId = assetHash(id);
+      const esmName = basename(id, '.tsx');
+      const componentName = extractDefaultExportName(code);
+
+      if (!componentName) {
+        throw new Error(
+          `No default export found in ${id}. Did you forget to export a Preact component?`
+        );
+      }
 
-            // Add to contents so this is in the client JS as well
-            contents = `import { h as __hs_h, render as __hs_render, hydrate as __hs_hydrate } from 'preact';${contents}`;
+      const publicPath = `${JS_ISLAND_PUBLIC_PATH}/${esmName}.js`;
+      registerImport(esmName, publicPath);
 
-            // Some _interesting_ work at play here...
-            // We have to modify the original file contents to add an __HS_PLUGIN export that the renderIsland() function can use to render the component.
-            // A lot of this work actaully has to be done now, ahead of time, to ensure we use the same Preact instance to hydrate and render the component so there are no errors.
-            // So... we have to import the preact-render-to-string library to render the component to a string here, with simple functions to do that work and return HTML.
-            // All imports needed for this work are prefixed with __hs_ to avoid clashing with other imports in the module, as some of them may be duplicates.
-            // Finally, we need to export all of the functions that do this work in a special way so we don't change the default export or other functions in the module, so that only the Hyperspan renderIsland() function can use them.
-            const moduleCode = `// hyperspan:processed
+      const moduleCode = `// hyperspan:processed
+import { h as __hs_h, render as __hs_render, hydrate as __hs_hydrate } from 'preact';
 import { render as __hs_renderToString } from 'preact-render-to-string';
 import { buildIslandHtml as __hs_buildIslandHtml } from '@hyperspan/plugin-preact';
 
-// Original file contents
-${contents}
+${code}
 
-// hyperspan:preact-plugin
 function __hs_renderIsland(jsContent = '', ssrContent = '', options = {}) {
   return __hs_buildIslandHtml("${jsId}", "${componentName}", "${esmName}", jsContent, ssrContent, options);
 }
+
 ${componentName}.__HS_ISLAND = {
   id: "${jsId}",
   render: (props, options = {}) => {
@@ -215,47 +121,53 @@ ${componentName}.__HS_ISLAND = {
       const jsContent = \`import { h as __hs_h, render as __hs_render } from 'preact';__hs_render(__hs_h(${componentName}, \${JSON.stringify(props)}), document.getElementById("${jsId}"));\`;
       return __hs_renderIsland(jsContent, '', options);
     }
-
     const ssrContent = __hs_renderToString(__hs_h(${componentName}, props));
     const jsContent = \`import { h as __hs_h, hydrate as __hs_hydrate } from 'preact';__hs_hydrate(__hs_h(${componentName}, \${JSON.stringify(props)}), document.getElementById("${jsId}"));\`;
     return __hs_renderIsland(jsContent, ssrContent, options);
-    
   }
-}
+};
 `;
 
-            PREACT_ISLAND_CACHE.set(jsId, { contents: moduleCode, esmName });
-
-            return {
-              contents: moduleCode,
-              loader: 'js',
-            };
-          });
-        },
-      });
-    } catch (e) {
-      log('ERROR: plugin build error', e);
-      console.error('[Hyperspan] @hyperspan/plugin-preact build error');
-      console.error(e);
-      throw e;
-    }
+      return { code: moduleCode, map: null };
+    },
+
+    buildStart() {
+      // emitFile is only available during production builds, not vite serve.
+      if (this.meta.watchMode) return;
+      try {
+        this.emitFile({
+          type: 'chunk',
+          id: preactClientPath,
+          fileName: 'islands/preact-client.js',
+        });
+      } catch {
+        // Serve mode — client is resolved via import map / Vite deps instead.
+      }
+    },
   };
 }
 
-
 /**
- * Render a Preact island component
+ * Hyperspan config plugin (no-op in v2 — use preactVitePlugin in vite.config.ts).
  */
-export function renderPreactIsland(Component: any, props: any = {}, options = {
-  ssr: true,
-  loading: undefined,
-}) {
-  // Render island with its own logic
+export function preactPlugin(): HS.Plugin {
+  return () => {
+    log('preactPlugin loaded (Vite handles bundling in v2)');
+  };
+}
+
+export function renderPreactIsland(
+  Component: {
+    __HS_ISLAND?: { render: (props: unknown, options: unknown) => string };
+    name?: string;
+  },
+  props: Record<string, unknown> = {},
+  options: { ssr?: boolean; loading?: string } = { ssr: true }
+) {
   if (Component.__HS_ISLAND?.render) {
     return html.raw(Component.__HS_ISLAND.render(props, options));
   }
-
   throw new Error(
-    `Module ${Component.name} was not loaded with an island plugin! Did you forget to install an island plugin and add it to the 'plugins' option in your hyperspan.config.ts file?`
+    `Module ${Component.name} was not loaded with an island plugin! Add preactVitePlugin() to vite.config.ts.`
   );
-}
+}