feat!: vite 8 / rolldown compatibility (#76)

* chore(deps): upgrade vite to v8.0.16

* chore(deps): install @eik/common

* test: update snapshots

* refactor(plugin): inline import-map logic, drop @eik/rollup-plugin

* chore(deps): uninstall @eik/rollup-plugin

* feat(plugin)!: convert external require calls to ESM facades for Rolldown

Adds handling for require() calls of mapped specifiers under Vite 8 /
Rolldown. Rolldown no longer auto-converts external require() to ESM
import (see
vite.dev/guide/migration#require-calls-for-externalized-modules), so the
plugin now routes each require() to a virtual ESM facade that imports
the URL, letting the bundler's CJS interop wrap it at the call site.

BREAKING CHANGE: targets Vite 8+ as the supported runtime. Output may
include one additional virtual chunk per require()-mapped specifier.

* test(plugin): cover require() of mapped specifier

test(fixtures): add cjs fixture for require() tests

* test(plugin): cover mixed import + require for the same specifier

test(fixtures): add mixed import + require fixture

* test(plugin): cover require() of unmapped dependency

* docs(readme): document Vite 8 / Rolldown require handling

Author: josteinkringlen

README.md

@@ -2,7 +2,7 @@
 
 [Vite](https://vite.dev/) plugin for [build-time import mapping with Eik](https://eik.dev/docs/guides/vite).
 
-This is a small wrapper around the [Rollup plugin](https://github.com/eik-lib/rollup-plugin) to make it slightly more ergonomic to use in Vite.
+Resolves bare import specifiers in your client bundle to URLs hosted on an Eik server, so shared dependencies (React, lit, etc.) load from one place across apps instead of being bundled into each one.
 
 ## Installation
 
@@ -24,7 +24,7 @@ export default defineConfig({
         // Turn off the asset hashes. Stable file names
         // make it easier to use the Eik node client:
         // https://github.com/eik-lib/node-client
-        // Publishinig a new version on Eik gives a
+        // Publishing a new version on Eik gives a
         // unique URL, no hash needed.
         assetFileNames: "[name].[ext]",
         entryFileNames: "[name].js",
@@ -37,5 +37,16 @@ export default defineConfig({
 
 ## Options
 
-The options you can give to `eik()` are the same as for the Rollup plugin.
-Refer to [the Rollup plugin documentation](https://github.com/eik-lib/rollup-plugin?tab=readme-ov-file#options) to see what you can do.
+| Option | Type                       | Default          | Description                                                                       |
+| ------ | -------------------------- | ---------------- | --------------------------------------------------------------------------------- |
+| `path` | `string`                   | `process.cwd()`  | Path to `eik.json`.                                                               |
+| `urls` | `string[]`                 | `[]`             | URLs to import maps hosted on an Eik server. Takes precedence over `eik.json`.    |
+| `maps` | `ImportMap \| ImportMap[]` | `[]`             | Inline import maps. Takes precedence over `urls` and `eik.json`.                  |
+
+An `ImportMap` is `{ imports: Record<string, string> }`.
+
+## Vite 8 / Rolldown
+
+Static `import` statements of mapped specifiers are rewritten to external URL imports. `require()` calls of mapped specifiers are routed through a virtual ESM facade so the bundler's CJS interop wraps them at the call site.
+
+Without the facade, Rolldown would leave a literal `require("<url>")` in the output bundle, which throws in the browser. See the [Vite 8 migration note](https://vite.dev/guide/migration#require-calls-for-externalized-modules) for background.

fixtures/modules/require-cjs/main.cjs

@@ -0,0 +1,7 @@
+const { LitElement } = require("lit-element");
+
+module.exports = class Inner extends LitElement {
+	render(world) {
+		return world;
+	}
+};

fixtures/modules/require-mixed/inner.cjs

@@ -0,0 +1,2 @@
+const { LitElement } = require("lit-element");
+module.exports = class Inner extends LitElement {};

fixtures/modules/require-mixed/main.js

@@ -0,0 +1,8 @@
+import { LitElement } from "lit-element";
+import inner from "./inner.cjs";
+
+export default class Outer extends LitElement {
+	render() {
+		return inner;
+	}
+}

fixtures/modules/require-unmapped/main.cjs

@@ -0,0 +1,3 @@
+const unmapped = require("some-unmapped-package");
+
+module.exports = unmapped;