@jskit-ai/kernel 0.1.176 → 0.1.177

package/client/vite/clientBootstrapPlugin.js

@@ -52,15 +52,14 @@ function isPathInsideRoot(rootPath, candidatePath) {
 }
 
 /**
- * Generated apps intentionally preserve package symlinks, so a bare import from an editable local
- * package would otherwise keep its node_modules path. optimizeDeps.exclude only prevents prebundling;
- * Vite can still append its dependency-wide ?v hash and serve that source as one-year immutable.
- * That hash does not change when local source changes, which lets a browser reuse stale client code
- * even after Vite restarts.
+ * Vite's default real-path resolution keeps linked packages on editable application source. This
+ * table is a fallback for declared file dependencies when another resolver still returns their
+ * installed node_modules identity. optimizeDeps.exclude alone does not prevent Vite from appending
+ * a dependency version token and assigning one-year immutable browser caching.
  *
- * Build this table from every local package declared by the app, not only packages with a ./client export.
- * Resolving only the bootstrap entry is insufficient: a local client can use bare imports from another
- * local package's root, shared entry, or exported subpath and accidentally re-enter node_modules caching.
+ * Include every declared file dependency, not only packages with a client export. A local client can
+ * import another local package's root, shared entry, or exported subpath and accidentally re-enter
+ * node_modules caching.
  */
 async function resolveLocalPackageSources({ appRoot }) {
   const appPackageJson = await readJsonFile(path.resolve(appRoot, "package.json"), {});
@@ -350,6 +349,7 @@ function resolveClientRuntimeDedupeSpecifiers(userResolveConfig = {}) {
 
 function createJskitClientBootstrapPlugin({ proxyTarget = "" } = {}) {
   let appRoot = process.cwd();
+  let hasMutableLocalPackages = false;
   let localPackages = Object.freeze([]);
   let resolvePackageSpecifier = null;
 
@@ -371,6 +371,11 @@ function createJskitClientBootstrapPlugin({ proxyTarget = "" } = {}) {
       localPackages = await resolveLocalPackageSources({
         appRoot
       });
+      hasMutableLocalPackages = (
+        localPackages.length > 0 ||
+        localScopePackageIds.length > 0 ||
+        installedPackages.some((entry) => LOCAL_PACKAGE_SOURCE_TYPES.has(entry.sourceType))
+      );
       const clientExcludeSpecifiers = resolveClientOptimizeExcludeSpecifiers(clientModules);
       const localScopeExcludeSpecifiers = resolveLocalScopeOptimizeExcludeSpecifiers(localScopePackageIds);
       const userOptimizeDeps = normalizeObject(userConfig.optimizeDeps);
@@ -399,6 +404,7 @@ function createJskitClientBootstrapPlugin({ proxyTarget = "" } = {}) {
         },
         resolve: {
           ...userResolve,
+          ...(hasMutableLocalPackages ? { preserveSymlinks: false } : {}),
           dedupe
         },
         server: {
@@ -410,6 +416,11 @@ function createJskitClientBootstrapPlugin({ proxyTarget = "" } = {}) {
       };
     },
     configResolved(resolvedConfig) {
+      if (hasMutableLocalPackages && resolvedConfig.resolve?.preserveSymlinks === true) {
+        throw new Error(
+          "JSKIT mutable local packages require Vite real-path resolution. Remove resolve.preserveSymlinks: true."
+        );
+      }
       // Do not use `this.resolve()` for this lookup. That re-enters Vite's live plugin chain, where
       // the dependency optimizer can turn an unlisted local subpath into node_modules/.vite (or add
       // its dependency ?v hash) before JSKIT sees the selected file. The config resolver applies the

package/client/vite/clientBootstrapPlugin.test.js

@@ -856,7 +856,11 @@ test("createJskitClientBootstrapPlugin config excludes local package roots and c
 
     process.chdir(tempRoot);
     const plugin = createJskitClientBootstrapPlugin();
-    const result = await plugin.config({});
+    const result = await plugin.config({
+      resolve: {
+        preserveSymlinks: true
+      }
+    });
 
     assert.deepEqual(result.optimizeDeps.exclude, [
       "@example/local-client",
@@ -868,6 +872,7 @@ test("createJskitClientBootstrapPlugin config excludes local package roots and c
       "@jskit-ai/kernel/client/moduleBootstrap",
       "mime-match"
     ]);
+    assert.equal(result.resolve.preserveSymlinks, false);
     assert.deepEqual(result.resolve.dedupe, ["@tanstack/vue-query", "pinia", "vue", "vue-router", "vuetify"]);
   } finally {
     process.chdir(previousCwd);
@@ -951,6 +956,34 @@ test("createJskitClientBootstrapPlugin config excludes all @local scoped package
       "@example/remote-client/client",
       "@jskit-ai/kernel/client/moduleBootstrap"
     ]);
+    assert.equal(result.resolve.preserveSymlinks, false);
+  } finally {
+    process.chdir(previousCwd);
+  }
+});
+
+test("createJskitClientBootstrapPlugin rejects a later preserveSymlinks override for local packages", async () => {
+  const tempRoot = await mkdtemp(path.join(os.tmpdir(), "jskit-client-bootstrap-local-symlink-config-"));
+  const previousCwd = process.cwd();
+
+  try {
+    await declareInstalledPackages(tempRoot, {
+      "@local/main": { packagePath: "packages/main" }
+    });
+
+    process.chdir(tempRoot);
+    const plugin = createJskitClientBootstrapPlugin();
+    const result = await plugin.config({});
+
+    assert.equal(result.resolve.preserveSymlinks, false);
+    assert.throws(
+      () => plugin.configResolved({
+        resolve: {
+          preserveSymlinks: true
+        }
+      }),
+      /mutable local packages require Vite real-path resolution/u
+    );
   } finally {
     process.chdir(previousCwd);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/kernel",
-  "version": "0.1.176",
+  "version": "0.1.177",
   "type": "module",
   "dependencies": {
     "json-rest-schema": "^1.0.17"