@gjsify/rolldown-plugin-gjsify 0.4.27 → 0.4.29

package/lib/app/browser.js

@@ -29,8 +29,8 @@ export const setupForBrowser = async (input) => {
     };
     const aliasMap = {
         ...browserPolyfillAliases,
-        ...(input.pluginOptions.aliases ?? {}),
-        ...(input.userAliases ?? {}),
+        ...input.pluginOptions.aliases,
+        ...input.userAliases,
     };
     const options = {
         input: entryPoints,

package/lib/app/gjs.js

@@ -15,7 +15,7 @@ import { deepkitPlugin } from '@gjsify/rolldown-plugin-deepkit';
 import blueprintPlugin from '@gjsify/vite-plugin-blueprint';
 import { getAliasesForGjs } from '../utils/alias.js';
 import { globToEntryPoints } from '../utils/entry-points.js';
-import { nodeModulesPathRewritePlugin, getBundleDirFromOutput, } from '../plugins/rewrite-node-modules-paths.js';
+import { nodeModulesPathRewritePlugin, getBundleDirFromOutput } from '../plugins/rewrite-node-modules-paths.js';
 import { processStubPlugin } from '../plugins/process-stub.js';
 import { cssAsStringPlugin } from '../plugins/css-as-string.js';
 import { shebangPlugin, resolveShebangLine, inputShebangStripPlugin } from '../plugins/shebang.js';
@@ -77,8 +77,8 @@ export const setupForGjs = async (input) => {
     const aliasMap = {
         ...getAliasesForGjs({ external }),
         'unicorn-magic': unicornMagicShim,
-        ...(input.pluginOptions.aliases ?? {}),
-        ...(input.userAliases ?? {}),
+        ...input.pluginOptions.aliases,
+        ...input.userAliases,
     };
     // The console shim replaces all `console` references with print()/printerr()-
     // based implementations that bypass GLib.log_structured() — no prefix,
@@ -91,9 +91,7 @@ export const setupForGjs = async (input) => {
     // resolution lands at a non-existent location. Walk up via
     // createRequire's node_modules-aware resolver as a fallback.
     const consoleShimEnabled = input.pluginOptions.consoleShim !== false;
-    const consoleShimPath = consoleShimEnabled
-        ? resolveConsoleShim()
-        : null;
+    const consoleShimPath = consoleShimEnabled ? resolveConsoleShim() : null;
     // The auto-globals inject stub (when present) is side-effect-imported
     // via a virtual entry — its register modules write to globalThis, so
     // the import chain matters but no name binding does. We can't use
@@ -239,9 +237,7 @@ function wrapInputWithSideEffects(input, sideEffects) {
         }
         wrappedInput = out;
     }
-    const sideEffectImports = sideEffects
-        .map((p) => `import ${JSON.stringify(p)};`)
-        .join('\n');
+    const sideEffectImports = sideEffects.map((p) => `import ${JSON.stringify(p)};`).join('\n');
     // Resolved real-path targets from `userEntries` get their
     // moduleSideEffects forced to 'no-treeshake' so the user-entry's
     // top-level body (`run({...})`, side-effect calls) survives tree-shake
@@ -249,7 +245,7 @@ function wrapInputWithSideEffects(input, sideEffects) {
     const resolvedTargets = new Set();
     const plugin = {
         name: 'gjsify-virtual-entry',
-        async resolveId(source, importer) {
+        async resolveId(source, _importer) {
             if (source.startsWith(PREFIX))
                 return source;
             // Force-mark the resolved user-entry target as having

package/lib/app/node.js

@@ -9,7 +9,7 @@ import { deepkitPlugin } from '@gjsify/rolldown-plugin-deepkit';
 import { EXTERNALS_NODE } from '@gjsify/resolve-npm';
 import { getAliasesForNode } from '../utils/alias.js';
 import { globToEntryPoints } from '../utils/entry-points.js';
-import { nodeModulesPathRewritePlugin, getBundleDirFromOutput, } from '../plugins/rewrite-node-modules-paths.js';
+import { nodeModulesPathRewritePlugin, getBundleDirFromOutput } from '../plugins/rewrite-node-modules-paths.js';
 import { cssAsStringPlugin } from '../plugins/css-as-string.js';
 export const setupForNode = async (input) => {
     const userExternal = input.userExternal ?? [];
@@ -21,11 +21,7 @@ export const setupForNode = async (input) => {
     // way esbuild's did (`gi://*`, `@girs/*`). We use a function predicate
     // instead so the gi:// URI scheme and the @girs/ namespace are matched
     // by prefix.
-    const exactExternal = [
-        ...EXTERNALS_NODE,
-        'node-datachannel',
-        ...userExternal,
-    ];
+    const exactExternal = [...EXTERNALS_NODE, 'node-datachannel', ...userExternal];
     const external = (id) => {
         if (id.startsWith('gi://'))
             return true;
@@ -40,8 +36,8 @@ export const setupForNode = async (input) => {
     const entryPoints = await globToEntryPoints(input.input, exclude);
     const aliasMap = {
         ...getAliasesForNode({ external }),
-        ...(input.pluginOptions.aliases ?? {}),
-        ...(input.userAliases ?? {}),
+        ...input.pluginOptions.aliases,
+        ...input.userAliases,
     };
     const bundleDir = getBundleDirFromOutput(input.output);
     // Rolldown's CJS interop wraps bundled CJS via `__commonJSMin` and

package/lib/index.d.ts

@@ -3,12 +3,12 @@ export * from './utils/index.js';
 export * from './app/index.js';
 export * from './library/index.js';
 export { REWRITE_FILTER, getBundleDirFromOutput, rewriteContents, shouldRewrite, nodeModulesPathRewritePlugin, } from './plugins/rewrite-node-modules-paths.js';
-export type { NodeModulesPathRewriteOptions, RewriteResult, } from './plugins/rewrite-node-modules-paths.js';
+export type { NodeModulesPathRewriteOptions, RewriteResult } from './plugins/rewrite-node-modules-paths.js';
 export { processStubPlugin, GJS_PROCESS_STUB, composeBanner } from './plugins/process-stub.js';
 export type { ProcessStubPluginOptions } from './plugins/process-stub.js';
 export { cssAsStringPlugin } from './plugins/css-as-string.js';
 export { textLoaderPlugin } from './plugins/text-loader.js';
-export type { TextLoaderPluginOptions } from './plugins/text-loader.js';
+export type { TextLoaderPluginOptions, LoaderKind } from './plugins/text-loader.js';
 export { shebangPlugin, GJS_SHEBANG, NODE_SHEBANG, expandEnvTemplate, resolveShebangLine } from './plugins/shebang.js';
 export type { ShebangPluginOptions } from './plugins/shebang.js';
 export { gjsImportsEmptyPlugin } from './plugins/gjs-imports-empty.js';

package/lib/library/lib.js

@@ -27,8 +27,8 @@ export const setupLib = async (input) => {
     // which doesn't match the package.json `exports` map.
     const preserveModulesRoot = computeCommonRoot(entryPoints);
     const aliasMap = {
-        ...(input.pluginOptions.aliases ?? {}),
-        ...(input.userAliases ?? {}),
+        ...input.pluginOptions.aliases,
+        ...input.userAliases,
     };
     // Library mode keeps all third-party / workspace imports as-is so the
     // emitted package re-exports its dep tree by reference. Rolldown's

package/lib/plugins/alias.js

@@ -22,7 +22,6 @@
 // constructor — breaking `util.inherits(Child, Stream)` patterns.
 export function aliasPlugin(options) {
     const entries = options.entries;
-    const keys = Object.keys(entries);
     return {
         name: 'gjsify-alias',
         resolveId: {

package/lib/plugins/css-as-string.d.ts

@@ -1,4 +1,5 @@
 import type { Plugin } from 'rolldown';
+import type { Targets } from 'lightningcss';
 export interface CssAsStringOptions {
     /**
      * lightningcss browser targets passed to `bundleAsync`. When set,
@@ -7,7 +8,7 @@ export interface CssAsStringOptions {
      * match the GTK4 CSS parser. Omit or leave undefined to skip
      * lowering (output stays as-authored except for `@import` inlining).
      */
-    targets?: import('lightningcss').Targets;
+    targets?: Targets;
     /**
      * When true (default), `@import` statements are resolved by
      * lightningcss `bundleAsync`. Set false to fall back to a plain

package/lib/plugins/css-as-string.js

@@ -41,7 +41,8 @@ import { dirname, isAbsolute, resolve as resolvePath } from 'node:path';
 import { pathToFileURL } from 'node:url';
 let _bundlerPromise = null;
 async function pickBundler() {
-    const forced = globalThis.process?.env?.GJSIFY_CSS_BACKEND;
+    const forced = globalThis.process?.env
+        ?.GJSIFY_CSS_BACKEND;
     if (forced === 'npm')
         return loadNpmBundler();
     if (forced === 'native') {

package/lib/plugins/gjs-imports-empty.js

@@ -6,6 +6,14 @@
 // Marking them external would leave bare specifiers in the bundle that the
 // browser cannot resolve at runtime; instead we resolve them to a virtual
 // empty ESM module so the bundle is self-contained.
+//
+// Portability note: the `filter: { id: ... }` below is a Rolldown fast-path
+// — Rolldown pre-filters which specifiers reach `handler`. Under Vite (which
+// also runs Rolldown for `build` but does NOT honor the Rolldown-specific
+// hook-level `filter` in every code path, e.g. esbuild dep prebundle or the
+// dev server) the handler may receive ALL ids. The internal guard below makes
+// the plugin correct regardless of whether the filter pre-filtered — it is
+// the load-bearing check; the `filter` is a defense-in-depth optimization.
 const GJSIMPORTS_VIRTUAL_ID = '\0gjsify-empty-gjs-import';
 export function gjsImportsEmptyPlugin() {
     return {
@@ -13,7 +21,13 @@ export function gjsImportsEmptyPlugin() {
         resolveId: {
             order: 'pre',
             filter: { id: /^(@girs\/|gi:\/\/)/ },
-            handler(_source) {
+            handler(source) {
+                // Internal guard: do not rely solely on the Rolldown `filter`
+                // above (it may not pre-filter under Vite). Only intercept
+                // `@girs/*` and `gi://*` specifiers; let everything else fall
+                // through to the default resolver chain.
+                if (!/^(@girs\/|gi:\/\/)/.test(source))
+                    return null;
                 return { id: GJSIMPORTS_VIRTUAL_ID };
             },
         },

package/lib/plugins/text-loader.d.ts

@@ -1,10 +1,13 @@
 import type { Plugin } from 'rolldown';
+/** Loader kind values accepted under `gjsify.loaders`. */
+export type LoaderKind = 'text' | 'dataurl';
 export interface TextLoaderPluginOptions {
     /**
-     * Map of file extension (with leading `.`) → loader kind. Currently only
-     * `'text'` is implemented; the field is shaped this way to leave room
-     * for `'json'` / `'binary'` later without a config break.
+     * Map of file extension (with leading `.`) → loader kind.
+     *   `'text'`    — file contents as JS string default export.
+     *   `'dataurl'` — `data:<mime>;base64,<b64>` string default export.
+     *                 MIME inferred from extension (.png → image/png etc.).
      */
-    loaders?: Record<string, 'text'>;
+    loaders?: Record<string, LoaderKind>;
 }
 export declare function textLoaderPlugin(options?: TextLoaderPluginOptions): Plugin | null;

package/lib/plugins/text-loader.js

@@ -1,24 +1,56 @@
-// Generic "load file as JS string default export" plugin.
+// Generic "load file as JS string / data-URL default export" plugin.
 //
 // Mirrors `css-as-string` but lets the user opt-in arbitrary extensions
-// through `bundler.loaders` config, e.g.:
+// through `gjsify.loaders` config, e.g.:
 //
-//   "bundler": { "loaders": { ".ui": "text", ".asm": "text" } }
+//   "loaders": { ".ui": "text", ".asm": "text", ".glsl": "text", ".png": "dataurl" }
 //
-// — replaces the esbuild `loader: { '.ui': 'text' }` shorthand from the
-// pre-Rolldown era. Rolldown does not classify unknown extensions as text
+// — replaces the esbuild `loader: { '.ui': 'text', '.png': 'dataurl' }` shorthand
+// from the pre-Rolldown era. Rolldown does not classify unknown extensions as text
 // by default; without a hook it tries to parse them as JS and fails.
+//
+// Loader kinds:
+//   'text'    — file contents as a JS string default export
+//               (`export default "…source…"`)
+//   'dataurl' — file encoded as a data: URL string default export
+//               (`export default "data:<mime>;base64,<b64>"`)
+//               MIME is inferred from the file extension; falls back to
+//               `application/octet-stream` for unknown extensions.
+//               Useful for Excalibur's `ImageSource` constructor and similar
+//               libraries that want a data: URL rather than a separate asset.
 import { readFile } from 'node:fs/promises';
+/** MIME type lookup for common binary / shader formats. */
+const MIME_BY_EXT = {
+    '.png': 'image/png',
+    '.jpg': 'image/jpeg',
+    '.jpeg': 'image/jpeg',
+    '.gif': 'image/gif',
+    '.webp': 'image/webp',
+    '.svg': 'image/svg+xml',
+    '.ico': 'image/x-icon',
+    '.avif': 'image/avif',
+    '.bmp': 'image/bmp',
+    '.tiff': 'image/tiff',
+    '.tif': 'image/tiff',
+    '.wasm': 'application/wasm',
+};
+function mimeForExtension(ext) {
+    return MIME_BY_EXT[ext.toLowerCase()] ?? 'application/octet-stream';
+}
 export function textLoaderPlugin(options = {}) {
-    const exts = Object.entries(options.loaders ?? {})
-        .filter(([, kind]) => kind === 'text')
-        .map(([ext]) => ext);
-    if (exts.length === 0)
+    const loaders = options.loaders ?? {};
+    const entries = Object.entries(loaders);
+    const textExts = entries.filter(([, kind]) => kind === 'text').map(([ext]) => ext);
+    const dataurlExts = entries.filter(([, kind]) => kind === 'dataurl').map(([ext]) => ext);
+    const allExts = [...textExts, ...dataurlExts];
+    if (allExts.length === 0)
         return null;
     // Build a single regex matching any of the configured extensions:
-    //   ['.ui', '.asm'] → /\.(ui|asm)$/
-    const escaped = exts.map((e) => e.replace(/^\./, '').replace(/[.*+?^${}()|[\]\\]/g, '\\$&'));
+    //   ['.ui', '.asm', '.png'] → /\.(ui|asm|png)$/
+    const escaped = allExts.map((e) => e.replace(/^\./, '').replace(/[.*+?^${}()|[\]\\]/g, '\\$&'));
     const filter = new RegExp(`\\.(?:${escaped.join('|')})$`);
+    // Index extensions → kind for fast lookup in the load hook.
+    const kindMap = new Map(entries.map(([ext, kind]) => [ext.toLowerCase(), kind]));
     // Use the function-form `load(id)` (Rollup-compatible) rather than the
     // newer `load: { filter, handler }` shape. The newer shape was observed
     // not to claim unknown-extension files reliably under Rolldown rc.18 —
@@ -27,10 +59,24 @@ export function textLoaderPlugin(options = {}) {
     // as `@gjsify/vite-plugin-blueprint`'s `.blp` hook) intercepts during
     // module-load lookup and works under both Vite and Rolldown.
     return {
-        name: 'gjsify-text-loader',
+        name: 'gjsify-asset-loader',
         async load(id) {
             if (!filter.test(id))
                 return null;
+            // Determine the extension (lowercase, with leading dot).
+            const dotIdx = id.lastIndexOf('.');
+            const ext = dotIdx >= 0 ? id.slice(dotIdx).toLowerCase() : '';
+            const kind = kindMap.get(ext) ?? 'text';
+            if (kind === 'dataurl') {
+                const bytes = await readFile(id);
+                const mime = mimeForExtension(ext);
+                const b64 = bytes.toString('base64');
+                return {
+                    code: `export default ${JSON.stringify(`data:${mime};base64,${b64}`)};`,
+                    moduleType: 'js',
+                };
+            }
+            // kind === 'text'
             const code = await readFile(id, 'utf8');
             return {
                 code: `export default ${JSON.stringify(code)};`,

package/lib/shims/console-gjs.js

@@ -15,10 +15,34 @@
 // `globalThis.console` untouched and routing user `console.log(…)` calls
 // through our object.
 // @ts-ignore — resolved by Rolldown at user-build time, not by tsc here.
-import { log, info, debug, warn, error, dir, dirxml, table, time, timeEnd, timeLog, trace, assert, clear, count, countReset, group, groupCollapsed, groupEnd, profile, profileEnd, timeStamp } from '@gjsify/console';
+import * as gjsConsole from '@gjsify/console';
+// NOTE: a namespace import is used deliberately. It is a single statement that
+// no formatter (oxfmt) can wrap across lines, so the `@ts-ignore` above always
+// sits on the line immediately preceding the import and reliably suppresses the
+// TS2307 (`@gjsify/console` is resolved by Rolldown at user-build time, not by
+// tsc here). A multi-line named import would let the reformatter detach the
+// suppression from the offending `from '@gjsify/console'` line.
 export const console = {
-    log, info, debug, warn, error, dir, dirxml, table,
-    time, timeEnd, timeLog, trace, assert, clear,
-    count, countReset, group, groupCollapsed, groupEnd,
-    profile, profileEnd, timeStamp,
+    log: gjsConsole.log,
+    info: gjsConsole.info,
+    debug: gjsConsole.debug,
+    warn: gjsConsole.warn,
+    error: gjsConsole.error,
+    dir: gjsConsole.dir,
+    dirxml: gjsConsole.dirxml,
+    table: gjsConsole.table,
+    time: gjsConsole.time,
+    timeEnd: gjsConsole.timeEnd,
+    timeLog: gjsConsole.timeLog,
+    trace: gjsConsole.trace,
+    assert: gjsConsole.assert,
+    clear: gjsConsole.clear,
+    count: gjsConsole.count,
+    countReset: gjsConsole.countReset,
+    group: gjsConsole.group,
+    groupCollapsed: gjsConsole.groupCollapsed,
+    groupEnd: gjsConsole.groupEnd,
+    profile: gjsConsole.profile,
+    profileEnd: gjsConsole.profileEnd,
+    timeStamp: gjsConsole.timeStamp,
 };

package/lib/utils/alias.js

@@ -1,4 +1,4 @@
-import { EXTERNALS_NODE, EXTERNALS_NPM, ALIASES_GENERAL_FOR_GJS, ALIASES_NODE_FOR_GJS, ALIASES_WEB_FOR_GJS, ALIASES_GENERAL_FOR_NODE, ALIASES_GJS_FOR_NODE, ALIASES_WEB_FOR_NODE } from "@gjsify/resolve-npm";
+import { EXTERNALS_NODE, EXTERNALS_NPM, ALIASES_GENERAL_FOR_GJS, ALIASES_NODE_FOR_GJS, ALIASES_WEB_FOR_GJS, ALIASES_GENERAL_FOR_NODE, ALIASES_GJS_FOR_NODE, ALIASES_WEB_FOR_NODE, } from '@gjsify/resolve-npm';
 export const setNodeAliasPrefix = (ALIASES) => {
     // Also resolve alias names with `node:${ALIAS}`
     for (const ALIAS in ALIASES) {
@@ -11,12 +11,12 @@ export const setNodeAliasPrefix = (ALIASES) => {
     }
     return ALIASES;
 };
-const getAliasesGeneralForGjs = (options) => ALIASES_GENERAL_FOR_GJS;
-const getAliasesNodeForGjs = (options) => setNodeAliasPrefix(ALIASES_NODE_FOR_GJS);
-const getAliasesWebForGjs = (options) => ALIASES_WEB_FOR_GJS;
-const getAliasesGeneralForNode = (options) => ALIASES_GENERAL_FOR_NODE;
-const getAliasesGjsForNode = (options) => ALIASES_GJS_FOR_NODE;
-const getAliasesWebForNode = (options) => ALIASES_WEB_FOR_NODE;
+const getAliasesGeneralForGjs = (_options) => ALIASES_GENERAL_FOR_GJS;
+const getAliasesNodeForGjs = (_options) => setNodeAliasPrefix(ALIASES_NODE_FOR_GJS);
+const getAliasesWebForGjs = (_options) => ALIASES_WEB_FOR_GJS;
+const getAliasesGeneralForNode = (_options) => ALIASES_GENERAL_FOR_NODE;
+const getAliasesGjsForNode = (_options) => ALIASES_GJS_FOR_NODE;
+const getAliasesWebForNode = (_options) => ALIASES_WEB_FOR_NODE;
 export const getAliasesForGjs = (options) => {
     return { ...getAliasesGeneralForGjs(options), ...getAliasesNodeForGjs(options), ...getAliasesWebForGjs(options) };
 };
@@ -24,6 +24,6 @@ export const getAliasesForNode = (options) => {
     return { ...getAliasesGeneralForNode(options), ...getAliasesGjsForNode(options), ...getAliasesWebForNode(options) };
 };
 /** Array of Node.js build in module names (also with node: prefix) */
-export const externalNode = [...EXTERNALS_NODE, ...EXTERNALS_NODE.map(E => `node:${E}`)];
+export const externalNode = [...EXTERNALS_NODE, ...EXTERNALS_NODE.map((E) => `node:${E}`)];
 /** Array of NPM module names for which we have our own implementation */
 export const externalNPM = [...EXTERNALS_NPM];

package/lib/utils/auto-globals.js

@@ -58,7 +58,7 @@ async function applyExcludeGlobals(detected, currentInject, extraRegisterPaths,
     const filtered = detectedToRegisterPaths(detected);
     for (const p of extraRegisterPaths)
         filtered.add(p);
-    const injectPath = filtered.size > 0 ? (await writeRegisterInjectFile(filtered)) ?? undefined : undefined;
+    const injectPath = filtered.size > 0 ? ((await writeRegisterInjectFile(filtered)) ?? undefined) : undefined;
     return { detected, injectPath };
 }
 function detectedToRegisterPaths(detected) {
@@ -150,9 +150,7 @@ export async function detectAutoGlobals(analysisOptions, pluginOptions, gjsifyPl
         // shape: caller plugins first (PnP, user text-loaders), then the
         // gjsify chain (which may be either a single RolldownPluginOption
         // or an array of them depending on the factory shape).
-        const gjsifyPluginsArray = Array.isArray(gjsifyInstance)
-            ? gjsifyInstance
-            : [gjsifyInstance];
+        const gjsifyPluginsArray = Array.isArray(gjsifyInstance) ? gjsifyInstance : [gjsifyInstance];
         const chunkCodes = await bundler({
             rolldownInput: {
                 input: inputWithInject,
@@ -181,7 +179,8 @@ export async function detectAutoGlobals(analysisOptions, pluginOptions, gjsifyPl
                     newDetected.add(id);
             }
             catch (e) {
-                if (globalThis.process?.env?.GJSIFY_DEBUG_AUTO_GLOBALS) {
+                if (globalThis.process?.env
+                    ?.GJSIFY_DEBUG_AUTO_GLOBALS) {
                     const path = `/tmp/gjsify-auto-globals-failed-chunk-${i}.mjs`;
                     try {
                         // eslint-disable-next-line @typescript-eslint/no-require-imports
@@ -189,7 +188,9 @@ export async function detectAutoGlobals(analysisOptions, pluginOptions, gjsifyPl
                         fs.writeFileSync(path, code);
                         console.error(`[gjsify-auto-globals] parse failed on chunk #${i} — wrote ${path} for inspection`);
                     }
-                    catch { /* ignore */ }
+                    catch {
+                        /* ignore */
+                    }
                 }
                 throw e;
             }

package/lib/utils/detect-free-globals.js

@@ -113,11 +113,7 @@ function extractBindingNames(node) {
                 ? extractBindingNames(p.argument)
                 : extractBindingNames(p.value));
         case 'ArrayPattern':
-            return node.elements.flatMap((e) => e
-                ? e.type === 'RestElement'
-                    ? extractBindingNames(e.argument)
-                    : extractBindingNames(e)
-                : []);
+            return node.elements.flatMap((e) => e ? (e.type === 'RestElement' ? extractBindingNames(e.argument) : extractBindingNames(e)) : []);
         case 'AssignmentPattern':
             return extractBindingNames(node.left);
         case 'RestElement':

package/lib/utils/extension.js

@@ -1,5 +1,12 @@
 export const getJsExtensions = (allowExt) => {
-    const extensions = { '.js': '.js', '.ts': '.js', '.mts': '.js', '.cts': '.js', '.cjs': '.js', '.mjs': '.js' };
+    const extensions = {
+        '.js': '.js',
+        '.ts': '.js',
+        '.mts': '.js',
+        '.cts': '.js',
+        '.cjs': '.js',
+        '.mjs': '.js',
+    };
     if (allowExt && extensions[allowExt]) {
         delete extensions[allowExt];
     }

package/lib/utils/inline-static-reads.js

@@ -53,9 +53,7 @@ import { existsSync, readFileSync, readdirSync, statSync } from 'node:fs';
  * `readdirSync` / `existsSync` skip the AST parse entirely (cheap fast path).
  */
 export function inlineStaticReads(src, sourceFilePath) {
-    if (!src.includes('readFileSync') &&
-        !src.includes('readdirSync') &&
-        !src.includes('existsSync')) {
+    if (!src.includes('readFileSync') && !src.includes('readdirSync') && !src.includes('existsSync')) {
         return { contents: src, inlined: 0 };
     }
     let ast;
@@ -112,7 +110,7 @@ export function inlineStaticReads(src, sourceFilePath) {
  * `undefined` if the call doesn't match an inlinable pattern or the path
  * couldn't be resolved or the file doesn't exist.
  */
-function tryInlineCall(node, ctx, src) {
+function tryInlineCall(node, ctx, _src) {
     const callee = node.callee;
     // `JSON.parse(readFileSync(<path>, "utf8"))` — collapse the whole
     // composition. Recognising it specifically lets us emit a parsed-JSON
@@ -120,8 +118,10 @@ function tryInlineCall(node, ctx, src) {
     // which esbuild can dead-code-eliminate against.
     if (callee.type === 'MemberExpression' &&
         !callee.computed &&
-        callee.object.type === 'Identifier' && callee.object.name === 'JSON' &&
-        callee.property.type === 'Identifier' && callee.property.name === 'parse' &&
+        callee.object.type === 'Identifier' &&
+        callee.object.name === 'JSON' &&
+        callee.property.type === 'Identifier' &&
+        callee.property.name === 'parse' &&
         node.arguments.length >= 1 &&
         node.arguments[0].type === 'CallExpression') {
         const inner = node.arguments[0];
@@ -377,7 +377,7 @@ function evalExpr(node, ctx) {
                     }
                 }
                 const base = evalExpr(ne.arguments[1], ctx);
-                const baseStr = base instanceof URL ? base.href : (typeof base === 'string' ? base : undefined);
+                const baseStr = base instanceof URL ? base.href : typeof base === 'string' ? base : undefined;
                 if (!baseStr)
                     return undefined;
                 try {
@@ -394,7 +394,7 @@ function evalExpr(node, ctx) {
             const name = identifierName(ce.callee);
             if (name === 'fileURLToPath') {
                 const arg = evalExpr(ce.arguments[0], ctx);
-                const url = arg instanceof URL ? arg.href : (typeof arg === 'string' ? arg : undefined);
+                const url = arg instanceof URL ? arg.href : typeof arg === 'string' ? arg : undefined;
                 if (!url)
                     return undefined;
                 try {
@@ -471,7 +471,9 @@ function evalEncodingExpr(node) {
                 continue;
             const key = p.key.type === 'Identifier'
                 ? p.key.name
-                : p.key.type === 'Literal' ? String(p.key.value) : undefined;
+                : p.key.type === 'Literal'
+                    ? String(p.key.value)
+                    : undefined;
             if (key !== 'encoding')
                 continue;
             if (p.value.type === 'Literal' && typeof p.value.value === 'string') {

package/lib/utils/merge.js

@@ -19,5 +19,8 @@ export function merge(target, ...sources) {
     return target;
 }
 function isPlainObject(val) {
-    return typeof val === 'object' && val !== null && !Array.isArray(val) && Object.getPrototypeOf(val) === Object.prototype;
+    return (typeof val === 'object' &&
+        val !== null &&
+        !Array.isArray(val) &&
+        Object.getPrototypeOf(val) === Object.prototype);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gjsify/rolldown-plugin-gjsify",
-    "version": "0.4.27",
+    "version": "0.4.29",
     "description": "Rolldown / Rollup / Vite plugin orchestrator for GJS, Node, and Browser targets",
     "type": "module",
     "main": "lib/index.js",
@@ -48,11 +48,11 @@
     ],
     "license": "MIT",
     "dependencies": {
-        "@gjsify/console": "^0.4.27",
-        "@gjsify/resolve-npm": "^0.4.27",
-        "@gjsify/rolldown-plugin-deepkit": "^0.4.27",
-        "@gjsify/rolldown-plugin-pnp": "^0.4.27",
-        "@gjsify/vite-plugin-blueprint": "^0.4.27",
+        "@gjsify/console": "^0.4.29",
+        "@gjsify/resolve-npm": "^0.4.29",
+        "@gjsify/rolldown-plugin-deepkit": "^0.4.29",
+        "@gjsify/rolldown-plugin-pnp": "^0.4.29",
+        "@gjsify/vite-plugin-blueprint": "^0.4.29",
         "@rollup/pluginutils": "^5.3.0",
         "acorn": "^8.16.0",
         "acorn-walk": "^8.3.5",
@@ -60,7 +60,7 @@
         "lightningcss": "^1.32.0"
     },
     "peerDependencies": {
-        "@gjsify/lightningcss-native": "^0.4.27",
+        "@gjsify/lightningcss-native": "^0.4.29",
         "rolldown": "^1.0.0-rc.18"
     },
     "peerDependenciesMeta": {