@hyperspan/plugin-preact 1.0.3 → 1.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperspan/plugin-preact",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Hyperspan Plugin for Preact",
   "main": "src/index.ts",
   "types": "src/index.ts",

package/src/index.ts

@@ -1,7 +1,7 @@
 import { JS_IMPORT_MAP, JS_ISLAND_PUBLIC_PATH } from '@hyperspan/framework/client/js';
 import { assetHash } from '@hyperspan/framework/utils';
 import { IS_PROD } from '@hyperspan/framework/server';
-import { join, resolve } from 'node:path';
+import { resolve } from 'node:path';
 import type { Hyperspan as HS } from '@hyperspan/framework';
 import { html } from '@hyperspan/html';
 import { h } from 'preact';
@@ -10,6 +10,8 @@ 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;
 
@@ -71,13 +73,14 @@ const PREACT_ISLAND_CACHE = new Map<string, PreactIslandCacheEntry>();
  */
 async function copyPreactToPublicFolder(config: HS.Config) {
   const currentNodeEnv = process.env.NODE_ENV || 'production';
-  const sourceFile = resolve(__dirname, './preact-client.ts');
+  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: join('./', config.publicDir, JS_ISLAND_PUBLIC_PATH),
+    outdir,
     naming: ISLAND_JS_NAMING,
     minify: true,
     format: 'esm',
@@ -140,9 +143,10 @@ export function preactPlugin(): HS.Plugin {
             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: join('./', config.publicDir, JS_ISLAND_PUBLIC_PATH),
+              outdir: islandOutdir,
               naming: ISLAND_JS_NAMING,
               external: Array.from(JS_IMPORT_MAP.keys()),
               minify: true,

package/src/test-fixtures/minimal-island.tsx

@@ -0,0 +1,9 @@
+import { h } from 'preact';
+
+export default function MinimalIsland(props: { name?: string; count?: number }) {
+  return h(
+    'div',
+    { className: 'minimal-island-fixture' },
+    `Hello ${props.name ?? '?'} (${props.count ?? 0})`
+  );
+}

package/src/test-fixtures/uses-document-island.tsx

@@ -0,0 +1,8 @@
+import { h } from 'preact';
+
+/** Top-level DOM access (fires as soon as this module is evaluated, before any render). */
+void document.documentElement;
+
+export default function UsesDocumentIsland(_props: Record<string, never>) {
+  return h('div', { 'data-test': 'uses-document-island' }, 'fixture');
+}

package/src/test-fixtures/uses-window-vanilla.js

@@ -0,0 +1,6 @@
+'use strict';
+
+/** Top-level browser global access (fires as soon as this module is evaluated). */
+void window.__hyperspan_fixture__;
+
+export const marker = 'uses-window-vanilla';