@jskit-ai/jskit-catalog 0.2.33 → 0.2.34

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/jskit-catalog",
-  "version": "0.2.33",
+  "version": "0.2.34",
   "description": "Published metadata catalog for JSKIT package metadata.",
   "type": "module",
   "files": [

package/scripts/browser-tests.mjs

@@ -117,10 +117,13 @@ function browserTestConfig({
   // process takes the environment prepare gave that server, such as its DB_NAME, before any test file loads.
   const workerIndex = managed ? 0 : Number.parseInt(String(environment.TEST_PARALLEL_INDEX || "0"), 10) || 0;
   const workerEnvironments = path.resolve(cwd, WORKER_ENVIRONMENTS, `workers-${basePort}.json`);
-  // A login state the host supplies is used as it is; the servers sign in only when there is none.
-  const signedInStates = signIn && !managed && !String(environment.PLAYWRIGHT_STORAGE_STATE || "").trim()
-    ? path.resolve(cwd, WORKER_ENVIRONMENTS, `signed-in-${basePort}`)
-    : "";
+  // A login state the host supplies is used as it is; the servers sign in only when there is none. A worker of a
+  // project that depends on another (a setup project) inherits the environment that project's workers changed,
+  // including the state this harness gave them, so one of this harness's own states is never taken for the host's.
+  const ownStates = path.resolve(cwd, WORKER_ENVIRONMENTS, `signed-in-${basePort}`);
+  const suppliedState = String(environment.PLAYWRIGHT_STORAGE_STATE || "").trim();
+  const hostState = suppliedState.startsWith(`${ownStates}-`) ? "" : suppliedState;
+  const signedInStates = signIn && !managed && !hostState ? ownStates : "";
   if (!managed && environment.TEST_PARALLEL_INDEX !== undefined) {
     Object.assign(environment, preparedWorkerEnvironment(workerEnvironments, workerIndex));
     if (signedInStates) environment.PLAYWRIGHT_STORAGE_STATE = `${signedInStates}-${workerIndex}.json`;

package/scripts/project-packages.mjs

@@ -364,9 +364,17 @@ async function preparePackageLockForInstall(projectRoot, manifests, packageLockP
   const next = structuredClone(packageLock.value);
   const localPackageNames = collectLocalPackageNames(manifests);
   if (next.packages && typeof next.packages === "object" && !Array.isArray(next.packages)) {
-    for (const [packagePath, record] of Object.entries(next.packages)) {
-      const packageId = resolveLockPackageName(packagePath, record);
-      if (packageId && record?.link !== true && !localPackageNames.has(packageId)) {
+    const removedPaths = Object.entries(next.packages)
+      .filter(([packagePath, record]) => {
+        const packageId = resolveLockPackageName(packagePath, record);
+        return packageId && record?.link !== true && !localPackageNames.has(packageId);
+      })
+      .map(([packagePath]) => packagePath);
+    // A removed package's nested dependencies go with it. Left behind without
+    // their parent, npm install drops them from the lock it writes while the
+    // new version still needs them nested, and npm ci then refuses that lock.
+    for (const packagePath of Object.keys(next.packages)) {
+      if (removedPaths.some((removed) => packagePath === removed || packagePath.startsWith(`${removed}/`))) {
         delete next.packages[packagePath];
       }
     }