poveste 0.16.0 → 0.16.1

package/dist/node/collect/runner.d.ts

@@ -1,4 +1,34 @@
+import type { EvaluatedModuleNode } from 'vite/module-runner';
 import { EvaluatedModules, ModuleRunner } from 'vite/module-runner';
 export type Invoke = (name: string, data: unknown[]) => Promise<unknown>;
-/** A runner over a module server, reached through `invoke`. */
-export declare function createRunner(invoke: Invoke, evaluatedModules?: EvaluatedModules): ModuleRunner;
+/**
+ * The file a CJS module should believe it is, for `require`, `__filename` and
+ * `__dirname`.
+ *
+ * A Vite id is root-relative, so `/node_modules/vue/index.js` is an id and not a
+ * path that exists. `isAbsolute()` answers "does this start with a separator",
+ * which that satisfies, so an id used to become a `filename` and the module's own
+ * `require('./dist/…')` resolved against a directory under the filesystem root.
+ * Quasar's config pulls `vue` through here, and a Quasar book could not collect a
+ * story at all (#1048).
+ *
+ * `file` is no better a source: measured over two books, it was `cleanUrl(id)` on
+ * every one of 1360 calls and a resolved path on none, so it differs from the id
+ * only in having the query stripped. It is read first because when the two ever do
+ * disagree Vite means `file`, not because being set makes it a path.
+ *
+ * So an absolute-looking candidate is accepted only if it names something — one
+ * `existsSync` against evaluating a module, and it is the question the old
+ * predicate meant to ask. `root` is the Vite root and not `process.cwd()`: they
+ * coincide only when the consumer runs poveste from the directory Vite is rooted
+ * at, and this branch is taken on most modules in a book.
+ *
+ * Vite's `/@fs/` ids — its escape hatch for a file outside the root — never reach
+ * here: `normalizeModuleId` strips the prefix server-side, so such a module arrives
+ * as a plain absolute path and is found by the first `existsSync`. Measured, not
+ * assumed: 81 of the Quasar book's modules resolve outside its root, all through
+ * that branch, and no id or file in either book carried the prefix.
+ */
+export declare function filenameOf(module: Readonly<EvaluatedModuleNode>, root: string): string;
+/** A runner over a module server, reached through `invoke`. `root` is the Vite root. */
+export declare function createRunner(invoke: Invoke, root: string, evaluatedModules?: EvaluatedModules): ModuleRunner;

package/dist/node/collect/runner.js

@@ -1,3 +1,4 @@
+import { existsSync } from 'node:fs';
 import { createRequire } from 'node:module';
 import { dirname, isAbsolute, join } from 'node:path';
 import process from 'node:process';
@@ -19,18 +20,70 @@ function exportAll(exports, source) {
         }
     }
 }
+/**
+ * The file a CJS module should believe it is, for `require`, `__filename` and
+ * `__dirname`.
+ *
+ * A Vite id is root-relative, so `/node_modules/vue/index.js` is an id and not a
+ * path that exists. `isAbsolute()` answers "does this start with a separator",
+ * which that satisfies, so an id used to become a `filename` and the module's own
+ * `require('./dist/…')` resolved against a directory under the filesystem root.
+ * Quasar's config pulls `vue` through here, and a Quasar book could not collect a
+ * story at all (#1048).
+ *
+ * `file` is no better a source: measured over two books, it was `cleanUrl(id)` on
+ * every one of 1360 calls and a resolved path on none, so it differs from the id
+ * only in having the query stripped. It is read first because when the two ever do
+ * disagree Vite means `file`, not because being set makes it a path.
+ *
+ * So an absolute-looking candidate is accepted only if it names something — one
+ * `existsSync` against evaluating a module, and it is the question the old
+ * predicate meant to ask. `root` is the Vite root and not `process.cwd()`: they
+ * coincide only when the consumer runs poveste from the directory Vite is rooted
+ * at, and this branch is taken on most modules in a book.
+ *
+ * Vite's `/@fs/` ids — its escape hatch for a file outside the root — never reach
+ * here: `normalizeModuleId` strips the prefix server-side, so such a module arrives
+ * as a plain absolute path and is found by the first `existsSync`. Measured, not
+ * assumed: 81 of the Quasar book's modules resolve outside its root, all through
+ * that branch, and no id or file in either book carried the prefix.
+ */
+export function filenameOf(module, root) {
+    for (const candidate of [module.file, module.id]) {
+        if (candidate === undefined || candidate === null) {
+            continue;
+        }
+        const path = candidate.replace(/[?#].*$/, '');
+        if (!isAbsolute(path)) {
+            continue;
+        }
+        if (existsSync(path)) {
+            return path;
+        }
+        const rooted = join(root, path);
+        if (existsSync(rooted)) {
+            return rooted;
+        }
+    }
+    return join(root, 'virtual-module');
+}
 /** As vite-node did: a CJS package's named exports are read through its default. */
 class InteropEvaluator {
+    root;
     // The runner offsets source maps by the lines a wrapper adds, which match Vite's own.
     startOffset = new ESModulesEvaluator().startOffset;
+    constructor(root) {
+        this.root = root;
+    }
     /**
      * Vite's own evaluator passes only import bindings, so a CJS dependency inlined
      * through `viteNodeInlineDeps` threw `exports is not defined` (#167).
      */
     async runInlinedModule(context, code, module) {
         const exports = context[ssrModuleExportsKey];
+        // Identity, not a location: only `.mjs` is read off it, below.
         const path = (module.file || module.id).replace(/[?#].*$/, '');
-        const filename = isAbsolute(path) ? path : join(process.cwd(), 'virtual-module');
+        const filename = filenameOf(module, this.root);
         let assignedExports;
         let assigned = false;
         const cjsExports = new Proxy(exports, {
@@ -99,8 +152,8 @@ const env = new Proxy(process.env, {
         ? undefined
         : ['DEV', 'PROD', 'SSR'].includes(key) ? !!target[key] : target[key],
 });
-/** A runner over a module server, reached through `invoke`. */
-export function createRunner(invoke, evaluatedModules = new EvaluatedModules()) {
+/** A runner over a module server, reached through `invoke`. `root` is the Vite root. */
+export function createRunner(invoke, root, evaluatedModules = new EvaluatedModules()) {
     return new ModuleRunner({
         transport: {
             async invoke(message) {
@@ -116,5 +169,5 @@ export function createRunner(invoke, evaluatedModules = new EvaluatedModules())
         sourcemapInterceptor: false,
         evaluatedModules,
         createImportMeta: modulePath => ({ ...createDefaultImportMeta(modulePath), env }),
-    }, new InteropEvaluator());
+    }, new InteropEvaluator(root));
 }

package/dist/node/collect/worker.js

@@ -59,7 +59,9 @@ async function collect(payload) {
         ;
         globalThis[key] = value;
     }
-    const runner = _runner ?? (_runner = createRunner((name, data) => _invoke(name, data), _evaluatedModules));
+    // Memoised for the worker's life though `root` arrives per task: a worker pool
+    // belongs to one server, so every task it is sent carries that server's root.
+    const runner = _runner ?? (_runner = createRunner((name, data) => _invoke(name, data), payload.root, _evaluatedModules));
     if (_domEnv) {
         resetDomEnv(_domEnv);
     }

package/dist/node/dom/env.js

@@ -62,6 +62,30 @@ export function createDomEnv() {
         unobserve(_target) { }
         takeRecords() { return []; }
     };
+    /*
+     * jsdom implements `Screen` — seven properties, no `orientation` — and has no
+     * `ScreenOrientation` constructor at all, so the property is absent rather than
+     * present and inert. There is nothing to feature-detect but the absence, and
+     * nothing to assign through: the property has no setter, so this defines it
+     * rather than using the `||` the three above can.
+     *
+     * Quasar's Screen plugin reads it unguarded while installing — `const { type,
+     * angle } = window.screen.orientation` — so a Quasar book cannot collect a
+     * story without it (#1053). Quasar 2.34 added that read; 2.33 had none.
+     */
+    if (window.screen !== undefined && window.screen.orientation === undefined) {
+        Object.defineProperty(window.screen, 'orientation', {
+            configurable: true,
+            value: {
+                type: 'landscape-primary',
+                angle: 0,
+                onchange: null,
+                addEventListener: () => { },
+                removeEventListener: () => { },
+                dispatchEvent: () => false,
+            },
+        });
+    }
     window.matchMedia = window.matchMedia || ((query) => ({
         matches: false,
         media: query,

package/dist/node/load.js

@@ -6,7 +6,7 @@ let _load;
 export function useModuleLoader(options) {
     const { server } = options;
     const node = createModuleServer(server, { inline: [] });
-    const runner = createRunner((name, data) => node.invoke(name, data));
+    const runner = createRunner((name, data) => node.invoke(name, data), server.config.root);
     function clearCache() {
         server.moduleGraph.invalidateAll();
         node.clearCache();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "poveste",
   "type": "module",
-  "version": "0.16.0",
+  "version": "0.16.1",
   "description": "Interactive component playgrounds for Vue, Nuxt, Svelte, SvelteKit and Quasar — a drop-in fork of histoire, with Quasar support histoire does not ship.",
   "author": {
     "name": "Sorin Gitlan",
@@ -74,10 +74,10 @@
   },
   "dependencies": {
     "@11ty/gray-matter": "^3.0.0",
-    "@poveste/app": "^0.16.0",
-    "@poveste/controls": "^0.16.0",
-    "@poveste/shared": "^0.16.0",
-    "@poveste/vendors": "^0.16.0",
+    "@poveste/app": "^0.16.1",
+    "@poveste/controls": "^0.16.1",
+    "@poveste/shared": "^0.16.1",
+    "@poveste/vendors": "^0.16.1",
     "change-case": "^5.4.4",
     "chokidar": "^5.0.0",
     "defu": "^6.1.4",