@hitachivantara/app-shell-vite-plugin 2.3.0 → 2.5.0

package/dist/locales-utils.d.ts

@@ -1,8 +1,4 @@
 export declare const SUPPORTED_LOCALES_FILE = "supported-locales.json";
-/**
- * Deep merge two objects. `override` keys take priority over `base`.
- */
-export declare function deepMerge(base: Record<string, unknown>, override: Record<string, unknown>): Record<string, unknown>;
 /**
  * Reads a `supported-locales.json` file and returns its contents as an array
  * of locale strings. Returns an empty array if the file doesn't exist or
@@ -31,11 +27,6 @@ export declare function discoverLanguageDirs(localesDir: string): string[];
  * @param appLocalesDir   - The app's locales directory (downstream / local).
  */
 export declare function computeSupportedLocales(shellLocalesDir: string | undefined, appLocalesDir: string | undefined): string[];
-/**
- * Parses a JSON file, wrapping parse errors with the file path for
- * actionable diagnostics.
- */
-export declare function readJsonFile(filePath: string): unknown;
 /**
  * Recursively merges source directory into destination.
  * JSON files are deep-merged with destination keys taking priority.

package/dist/locales-utils.js

@@ -1,26 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
+import { deepMerge, readJsonFile } from "./utils.js";
 export const SUPPORTED_LOCALES_FILE = "supported-locales.json";
-/**
- * Deep merge two objects. `override` keys take priority over `base`.
- */
-export function deepMerge(base, override) {
-    const result = { ...base };
-    for (const [key, value] of Object.entries(override)) {
-        if (value != null &&
-            typeof value === "object" &&
-            !Array.isArray(value) &&
-            result[key] != null &&
-            typeof result[key] === "object" &&
-            !Array.isArray(result[key])) {
-            result[key] = deepMerge(result[key], value);
-        }
-        else {
-            result[key] = value;
-        }
-    }
-    return result;
-}
 /**
  * Reads a `supported-locales.json` file and returns its contents as an array
  * of locale strings. Returns an empty array if the file doesn't exist or
@@ -30,7 +11,7 @@ export function readSupportedLocales(filePath) {
     if (!fs.existsSync(filePath))
         return [];
     try {
-        const data = JSON.parse(fs.readFileSync(filePath, "utf-8"));
+        const data = readJsonFile(filePath);
         if (Array.isArray(data) && data.every((item) => typeof item === "string")) {
             return data;
         }
@@ -107,19 +88,6 @@ export function computeSupportedLocales(shellLocalesDir, appLocalesDir) {
     }
     return result.toSorted();
 }
-/**
- * Parses a JSON file, wrapping parse errors with the file path for
- * actionable diagnostics.
- */
-export function readJsonFile(filePath) {
-    const raw = fs.readFileSync(filePath, "utf-8");
-    try {
-        return JSON.parse(raw);
-    }
-    catch (cause) {
-        throw new Error(`[app-shell-locales] Failed to parse ${filePath}: ${cause instanceof Error ? cause.message : cause}`, { cause });
-    }
-}
 /**
  * Recursively merges source directory into destination.
  * JSON files are deep-merged with destination keys taking priority.

package/dist/nodeModule.d.ts

@@ -6,4 +6,4 @@ export declare const require: NodeJS.Require;
  * @param suffix to be added after the module path
  * @returns The module path normalized
  */
-export declare function resolveModule(moduleName: string, suffix?: string): string;
+export declare function resolveModule(moduleName: string, suffix?: string): any;

package/dist/shared-dependencies.js

@@ -87,7 +87,8 @@ export default [
         moduleId: "@hitachivantara/app-shell-services",
         bundle: "app-shell-services.esm.js",
         bundleSrc: "@hitachivantara/app-shell-services/bundles/app-shell-services.esm.js",
-        virtualSrc: `export * from "@hitachivantara/app-shell-services";`,
+        virtualSrc: `export { default } from "@hitachivantara/app-shell-services";
+    export * from "@hitachivantara/app-shell-services";`,
     },
     {
         moduleId: "@hitachivantara/uikit-react-shared",

package/dist/utils.d.ts

@@ -0,0 +1,9 @@
+/**
+ * Deep merge two objects. `override` keys take priority over `base`.
+ */
+export declare function deepMerge(base: Record<string, unknown>, override: Record<string, unknown>): Record<string, unknown>;
+/**
+ * Parses a JSON file, wrapping parse errors with the file path for
+ * actionable diagnostics.
+ */
+export declare function readJsonFile(filePath: string): any;

package/dist/utils.js

@@ -0,0 +1,34 @@
+import { readFileSync } from "node:fs";
+/**
+ * Deep merge two objects. `override` keys take priority over `base`.
+ */
+export function deepMerge(base, override) {
+    const result = { ...base };
+    for (const [key, value] of Object.entries(override)) {
+        if (value != null &&
+            typeof value === "object" &&
+            !Array.isArray(value) &&
+            result[key] != null &&
+            typeof result[key] === "object" &&
+            !Array.isArray(result[key])) {
+            result[key] = deepMerge(result[key], value);
+        }
+        else {
+            result[key] = value;
+        }
+    }
+    return result;
+}
+/**
+ * Parses a JSON file, wrapping parse errors with the file path for
+ * actionable diagnostics.
+ */
+export function readJsonFile(filePath) {
+    const raw = readFileSync(filePath, "utf-8");
+    try {
+        return JSON.parse(raw);
+    }
+    catch (cause) {
+        throw new Error(`Failed to parse ${filePath}: ${cause instanceof Error ? cause.message : cause}`, { cause });
+    }
+}

package/dist/vite-plugin-configuration-processor.d.ts

@@ -0,0 +1,30 @@
+import type { PluginOption } from "vite";
+import type { HvAppShellConfig } from "@hitachivantara/app-shell-shared";
+/**
+ * Options for the configuration processor plugin.
+ */
+export interface ProcessConfigurationOptions {
+    /** Project root directory. */
+    root: string;
+    /** The original App Shell configuration json. */
+    appShellConfig: HvAppShellConfig;
+    /** The name of the application bundle being built. */
+    selfAppName: string;
+    /** The set of modules to be created by the rollup. */
+    modules: string[];
+    /** If true, the index.html entry point will be added to the bundle. */
+    buildEntryPoint: boolean;
+    /** Flag to control if config is included at index.html. */
+    inlineConfig: boolean;
+    /** Flag to control if we are creating an empty AppShell instance. */
+    generateEmptyShell: boolean;
+    /** If true, always writes app-shell.config.json to dist for dual-use packages. */
+    experimentalNewPackageLayout: boolean;
+}
+/**
+ * Process configuration, executing several tasks:
+ *  - Create rollup configuration to support module creation
+ *  - Generates final transformed configuration json
+ *  - "base" value is always "./" for build, and main app baseUrl for preview or dev
+ */
+export default function processConfiguration(options: ProcessConfigurationOptions): PluginOption;

package/dist/{vite-configuration-processor-plugin.js → vite-plugin-configuration-processor.js}

@@ -7,19 +7,13 @@ import sharedDependencies from "./shared-dependencies.js";
  *  - Create rollup configuration to support module creation
  *  - Generates final transformed configuration json
  *  - "base" value is always "./" for build, and main app baseUrl for preview or dev
- * @param root Project root directory.
- * @param appShellConfig The original App Shell configuration json.
- * @param selfAppName The name of the application bundle being built.
- * @param buildEntryPoint If true, the index.html entry point will be added to the bundle.
- * @param inlineConfig flag to control if config is included at index.html
- * @param generateEmptyShell flag to control if we are creating an empty AppShell instance
- * @param modules the set of modules to be created by the rollup
  */
-export default function processConfiguration(root, appShellConfig, selfAppName, buildEntryPoint, inlineConfig, generateEmptyShell, modules = []) {
+export default function processConfiguration(options) {
+    const { root, appShellConfig, selfAppName, modules, buildEntryPoint, inlineConfig, generateEmptyShell, experimentalNewPackageLayout, } = options;
     let finalAppShellConfig;
     let basePath;
     return {
-        name: "vite-plugin-appShell-configuration-processor",
+        name: "vite-plugin-appShell-config-processor",
         config(config, { command }) {
             const projectRoot = root ?? config.root;
             let appModules = {};
@@ -58,7 +52,7 @@ export default function processConfiguration(root, appShellConfig, selfAppName,
          * @param options build options
          */
         async generateBundle(options) {
-            if (generateEmptyShell || !buildEntryPoint) {
+            if (generateEmptyShell) {
                 return;
             }
             // obtain the directory (dist) where the new config file will be placed
@@ -82,11 +76,16 @@ export default function processConfiguration(root, appShellConfig, selfAppName,
                 finalAppShellConfig.baseUrl = basePath;
             }
             finalAppShellConfig.apps = undefined;
-            // Replace all @self references using simple string replacement
+            // Replace all $app and @self (deprecated) references using simple string replacement.
             let configString = JSON.stringify(finalAppShellConfig);
+            configString = configString.replaceAll(`"$app/`, `"${selfAppName}/`);
+            // TODO(major): remove @self/ support in favour of $app/
             configString = configString.replaceAll(`"@self/`, `"${selfAppName}/`);
             finalAppShellConfig = JSON.parse(configString);
-            if (!inlineConfig) {
+            // Write app-shell.config.json to dist when:
+            // - inlineConfig is false (standard flow), OR
+            // - experimentalNewPackageLayout is true (ensures dual-use: app shell or app bundle)
+            if (!inlineConfig || experimentalNewPackageLayout) {
                 fs.writeFileSync(path.resolve(targetDir, "app-shell.config.json"), JSON.stringify(finalAppShellConfig));
             }
         },

package/dist/{vite-crossorigin-fix-plugin.d.ts → vite-plugin-crossorigin-fix.d.ts}

@@ -5,5 +5,4 @@ export declare const isExternalUrl: (url: string) => boolean;
 export declare const dataUrlRE: RegExp;
 export declare const isDataUrl: (url: string) => boolean;
 export declare function checkCrossOrigin(html: string): any[];
-declare const fixCrossOrigin: () => PluginOption;
-export default fixCrossOrigin;
+export default function fixCrossOrigin(): PluginOption;

package/dist/{vite-crossorigin-fix-plugin.js → vite-plugin-crossorigin-fix.js}

@@ -48,11 +48,11 @@ export function checkCrossOrigin(html) {
     }
     return withCredentials;
 }
-const fixCrossOrigin = () => {
+export default function fixCrossOrigin() {
     const withCredentials = {};
     return [
         {
-            name: "vite-crossorigin-fix-collect-info-plugin",
+            name: "vite-plugin-crossorigin-fix-collect-info",
             transformIndexHtml: {
                 order: "pre",
                 handler(html, info) {
@@ -61,7 +61,7 @@ const fixCrossOrigin = () => {
             },
         },
         {
-            name: "vite-crossorigin-fix-replace-plugin",
+            name: "vite-plugin-crossorigin-fix-replace",
             transformIndexHtml: {
                 order: "post",
                 handler(html, context) {
@@ -79,5 +79,4 @@ const fixCrossOrigin = () => {
             },
         },
     ];
-};
-export default fixCrossOrigin;
+}

package/dist/vite-plugin-dist-package-json.d.ts

@@ -0,0 +1,5 @@
+import type { Plugin } from "vite";
+/**
+ * Generates a cleaned dist/package.json during Vite builds.
+ */
+export default function distPackageJsonPlugin(root?: string, sourceCondition?: string): Plugin;

package/dist/vite-plugin-dist-package-json.js

@@ -0,0 +1,271 @@
+import * as fs from "node:fs";
+import * as path from "node:path";
+// ─── Constants ───────────────────────────────────────────────────────────────
+/**
+ * Default custom export condition used internally by workspace packages
+ * to resolve TypeScript source during development.
+ *
+ * This condition must never leak into published artifacts because external
+ * consumers would not be able to resolve it.
+ */
+const DEFAULT_SOURCE_CONDITION = "@pentaho-apps:source";
+/**
+ * Runtime-relevant package.json fields copied into dist output.
+ *
+ * All other fields are intentionally excluded.
+ */
+const INCLUDED_FIELDS = [
+    "name",
+    "version",
+    "type",
+    "license",
+    "description",
+    "dependencies",
+    "peerDependencies",
+    "peerDependenciesMeta",
+    "optionalDependencies",
+];
+// ─── Helpers ─────────────────────────────────────────────────────────────────
+/**
+ * Reads and parses a JSON file.
+ *
+ * Returns `undefined` if the file does not exist or cannot be parsed.
+ */
+function readJsonFile(filePath) {
+    try {
+        return JSON.parse(fs.readFileSync(filePath, "utf8"));
+    }
+    catch {
+        return undefined;
+    }
+}
+/**
+ * Extracts the package name from a module specifier.
+ *
+ * Examples:
+ *   react-dom/client       → react-dom
+ *   @emotion/react/jsx-dev → @emotion/react
+ */
+function toPackageName(specifier) {
+    if (specifier.startsWith("@")) {
+        const segments = specifier.split("/");
+        return segments.length >= 2 ? `${segments[0]}/${segments[1]}` : segments[0];
+    }
+    return specifier.split("/")[0];
+}
+/**
+ * Removes the build output prefix from an export path.
+ *
+ * Example:
+ *   ./dist/index.js → ./index.js
+ */
+function stripDistPrefix(prefix, value) {
+    return value.startsWith(prefix) ? `./${value.slice(prefix.length)}` : value;
+}
+/**
+ * Transforms a package export target for dist/package.json.
+ *
+ * Processing rules:
+ *
+ * Strings:
+ * - Must resolve inside build.outDir
+ * - build.outDir prefix is stripped
+ *
+ * Objects:
+ * - Removes dev-only source condition
+ * - Recursively transforms nested targets
+ * - Removed if empty after transformation
+ *
+ * Arrays:
+ * - Node fallback arrays are eagerly collapsed to the first
+ *   valid non-null target because import maps do not support
+ *   Node conditional fallback resolution semantics.
+ *
+ * null:
+ * - Preserved as explicit export exclusion
+ *
+ * undefined:
+ * - Internal sentinel meaning "omit this entry"
+ */
+function transformExportTargetForDist(target, distPrefix, sourceCondition) {
+    // Explicit export exclusion must be preserved
+    if (target === null) {
+        return null;
+    }
+    // Direct export path
+    if (typeof target === "string") {
+        const stripped = stripDistPrefix(distPrefix, target);
+        // Ignore paths outside dist output
+        return stripped === target ? undefined : stripped;
+    }
+    // Fallback array
+    if (Array.isArray(target)) {
+        for (const entry of target) {
+            const resolved = transformExportTargetForDist(entry, distPrefix, sourceCondition);
+            if (resolved !== undefined && resolved !== null) {
+                return resolved;
+            }
+        }
+        return undefined;
+    }
+    // Conditional exports object
+    const result = {};
+    for (const [condition, value] of Object.entries(target)) {
+        // Remove workspace-only source condition
+        if (condition === sourceCondition) {
+            continue;
+        }
+        const transformed = transformExportTargetForDist(value, distPrefix, sourceCondition);
+        // Omitted targets disappear entirely
+        if (transformed === undefined) {
+            continue;
+        }
+        result[condition] = transformed;
+    }
+    return Object.keys(result).length === 0 ? undefined : result;
+}
+/**
+ * Normalizes package exports into canonical subpath map form.
+ *
+ * Converts:
+ *
+ *   "exports": "./index.js"
+ *   → { ".": "./index.js" }
+ *
+ *   "exports": { "import": "./index.js" }
+ *   → { ".": { "import": "./index.js" } }
+ *
+ * Leaves already-normalized subpath maps untouched.
+ */
+function normalizeExportsToSubpathMap(exportsField) {
+    if (exportsField == null) {
+        throw new Error(`[App Shell]: package.json is missing "exports".`);
+    }
+    // Sugar form:
+    // "exports": "./index.js"
+    // "exports": ["./a.js", "./b.js"]
+    if (typeof exportsField === "string" || Array.isArray(exportsField)) {
+        return { ".": exportsField };
+    }
+    const keys = Object.keys(exportsField);
+    // Already a subpath map
+    if (keys.every((key) => key.startsWith("."))) {
+        return exportsField;
+    }
+    // Root conditional exports sugar
+    return { ".": exportsField };
+}
+/**
+ * Appends standard metadata exports required at runtime.
+ */
+function appendStandardExports(exportsMap, outDir) {
+    exportsMap["./package.json"] = "./package.json";
+    exportsMap["./app-shell.config.json"] = "./app-shell.config.json";
+    if (fs.existsSync(path.join(outDir, "locales"))) {
+        exportsMap["./locales/*"] = "./locales/*";
+    }
+}
+// ─── Dist package.json generation ────────────────────────────────────────────
+/**
+ * Generates dist/package.json from the source package.
+ *
+ * The generated package:
+ * - strips dev-only export conditions
+ * - rewrites export paths relative to dist/
+ * - preserves explicit null exclusions
+ * - supports nested conditional exports
+ */
+function generateDistPackageJson(root, outDir, buildOutDir, sourceCondition, externalizedPackages) {
+    const pkgPath = path.join(root, "package.json");
+    const pkg = readJsonFile(pkgPath);
+    if (!pkg || !fs.existsSync(outDir)) {
+        return false;
+    }
+    const rawExports = pkg.exports;
+    const distPrefix = `./${buildOutDir}/`;
+    const transformedExports = transformExportTargetForDist(normalizeExportsToSubpathMap(rawExports), distPrefix, sourceCondition);
+    /**
+     * The entire exports tree may collapse after removing dev-only
+     * conditions and invalid dist targets.
+     */
+    if (transformedExports === undefined) {
+        throw new Error(`[App Shell] ${pkg.name}: exports resolved to empty after transformation.`);
+    }
+    appendStandardExports(transformedExports, outDir);
+    const distPkg = {};
+    // Copy runtime-relevant fields. The "dependencies" and "optionalDependencies"
+    // fields are narrowed to only the modules that Rollup actually externalized,
+    // so the published package declares exactly the runtime modules it expects the
+    // host to provide.
+    for (const field of INCLUDED_FIELDS) {
+        if (pkg[field] == null) {
+            continue;
+        }
+        if (field === "dependencies" || field === "optionalDependencies") {
+            const filtered = filterExternalizedDependencies(pkg[field], externalizedPackages);
+            if (Object.keys(filtered).length > 0) {
+                distPkg[field] = filtered;
+            }
+            continue;
+        }
+        distPkg[field] = pkg[field];
+    }
+    distPkg.exports = transformedExports;
+    fs.writeFileSync(path.join(outDir, "package.json"), JSON.stringify(distPkg, null, 2) + "\n");
+    console.info(`[App Shell] Generated ${buildOutDir}/package.json for ${pkg.name}`);
+    return true;
+}
+/**
+ * Keeps only the declared dependencies whose package was externalized by the
+ * bundle (i.e. expected to be resolved by the host at runtime).
+ */
+function filterExternalizedDependencies(dependencies, externalizedPackages) {
+    const filtered = {};
+    for (const [name, range] of Object.entries(dependencies)) {
+        if (externalizedPackages.has(name)) {
+            filtered[name] = range;
+        }
+    }
+    return filtered;
+}
+// ─── Plugin ──────────────────────────────────────────────────────────────────
+/**
+ * Generates a cleaned dist/package.json during Vite builds.
+ */
+export default function distPackageJsonPlugin(root, sourceCondition = DEFAULT_SOURCE_CONDITION) {
+    let config;
+    const externalizedPackages = new Set();
+    return {
+        name: "vite-plugin-dist-package-json",
+        apply: "build",
+        configResolved(resolved) {
+            config = resolved;
+        },
+        buildStart() {
+            // Reset per build so stale externalized packages from a previous build
+            // (e.g. in watch mode, where the plugin instance is reused) don't leak
+            // into the next dist/package.json.
+            externalizedPackages.clear();
+        },
+        generateBundle(_options, bundle) {
+            const emittedFiles = new Set(Object.keys(bundle));
+            for (const file of Object.values(bundle)) {
+                if (file.type !== "chunk") {
+                    continue;
+                }
+                // Imports that do not point to an emitted file are external modules
+                // that Rollup left for the host runtime to resolve.
+                for (const imported of [...file.imports, ...file.dynamicImports]) {
+                    if (!emittedFiles.has(imported)) {
+                        externalizedPackages.add(toPackageName(imported));
+                    }
+                }
+            }
+        },
+        closeBundle() {
+            const packageRoot = root ?? config.root;
+            const outDir = path.resolve(packageRoot, config.build.outDir);
+            generateDistPackageJson(packageRoot, outDir, config.build.outDir, sourceCondition, externalizedPackages);
+        },
+    };
+}

package/dist/{vite-generate-base-plugin.d.ts → vite-plugin-generate-base.d.ts}

@@ -8,7 +8,7 @@ import type { HvAppShellConfig } from "@hitachivantara/app-shell-shared";
  * @param appShellConfig App shell config values
  * @returns The app title
  */
-export declare const getAppTitle: (generateEmptyShell: boolean, appShellConfig: HvAppShellConfig) => any;
+export declare const getAppTitle: (generateEmptyShell: boolean, appShellConfig: HvAppShellConfig) => string;
 /**
  * Add the <BASE> tag and icons preload in the index.html file.
  * @param appShellConfig The app shell configuration.

package/dist/{vite-generate-bash-script-plugin.js → vite-plugin-generate-bash-script.js}

@@ -1,7 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
 import SHARED_DEPENDENCIES from "./shared-dependencies.js";
-import { getExtraDependenciesString } from "./vite-importmap-plugin.js";
+import { getExtraDependenciesString } from "./vite-plugin-importmap.js";
 export default function generateBashScript(externalImportMap, inlineConfig) {
     let config;
     let targetDir;

package/dist/{vite-locales-plugin.js → vite-plugin-locales.js}

@@ -1,7 +1,8 @@
 import fs from "node:fs";
 import { createRequire } from "node:module";
 import path from "node:path";
-import { computeSupportedLocales, deepMerge, mergeDirs, readJsonFile, SUPPORTED_LOCALES_FILE, } from "./locales-utils.js";
+import { computeSupportedLocales, mergeDirs, SUPPORTED_LOCALES_FILE, } from "./locales-utils.js";
+import { deepMerge, readJsonFile } from "./utils.js";
 /**
  * Resolves the app-shell-ui's locales directory path via its `./locales/*` export.
  */
@@ -63,7 +64,7 @@ export default function copyAppShellLocales(mergeUpstream = true) {
     let resolvedOutDir;
     let isBuild = false;
     return {
-        name: "vite-plugin-appshell-copy-locales",
+        name: "vite-plugin-copy-appShell-locales",
         // Capture the resolved output directory from Vite's config
         configResolved(config) {
             resolvedOutDir = path.resolve(config.root, config.build.outDir);

package/dist/{vite-metadata-plugin.d.ts → vite-plugin-metadata.d.ts}

@@ -4,5 +4,4 @@ import type { PluginOption } from "vite";
  * The metadata is used to help any troubleshoot activity by referencing
  * the version of the app-shell-vite-plugin and app-shell-ui packages used by the app.
  */
-declare const injectMetadata: () => PluginOption;
-export default injectMetadata;
+export default function injectMetadata(): PluginOption;

package/dist/{vite-metadata-plugin.js → vite-plugin-metadata.js}

@@ -1,20 +1,18 @@
-import fs from "node:fs";
 import { resolveModule } from "./nodeModule.js";
+import { readJsonFile } from "./utils.js";
 const extractVersion = (packageJsonFile) => {
-    const packageJson = fs.readFileSync(packageJsonFile, "utf8");
-    const packageObject = JSON.parse(packageJson);
-    return packageObject.version;
+    return readJsonFile(packageJsonFile).version;
 };
 /**
  * This plugin injects metadata into the index.html file.
  * The metadata is used to help any troubleshoot activity by referencing
  * the version of the app-shell-vite-plugin and app-shell-ui packages used by the app.
  */
-const injectMetadata = () => {
+export default function injectMetadata() {
     const appShellUIVersion = extractVersion(resolveModule("@hitachivantara/app-shell-ui/package.json"));
     const appShellVitePluginVersion = extractVersion(resolveModule("@hitachivantara/app-shell-vite-plugin/package.json"));
     return {
-        name: "vite-metadata-plugin",
+        name: "vite-plugin-metadata",
         transformIndexHtml() {
             return [
                 {
@@ -34,5 +32,4 @@ const injectMetadata = () => {
             ];
         },
     };
-};
-export default injectMetadata;
+}

package/dist/{vite-watch-config-plugin.js → vite-plugin-watch-config.js}

@@ -1,12 +1,14 @@
 import path from "node:path";
 const prepareConfigForDevMode = (config, selfAppName) => {
     let configString = JSON.stringify(config);
+    configString = configString.replaceAll(`"$app/`, `"${selfAppName}/`);
+    // TODO(major): remove @self/ support in favour of $app/
     configString = configString.replaceAll(`"@self/`, `"${selfAppName}/`);
     return JSON.parse(configString);
 };
 export default function serveAppShellConfig(appShellConfig, root, selfAppName, appShellConfigFile, automaticViewsFolder) {
     return {
-        name: "vite-plugin-watch-app-shell-config",
+        name: "vite-plugin-watch-appShell-config",
         apply: "serve",
         configureServer(server) {
             const restartServer = (file) => {