@coreplane/switchboard 1.228.0 → 1.229.0

package/dist/assets/config/config.example.yaml

@@ -317,19 +317,13 @@ workspaceDir: ./workspaces
 #   maxRounds: 3      # review rounds per unit
 #   maxMinutes: 120   # the ship preset's wall-clock budget in minutes (the registry's default)
 
-# Which loop drives each preset's runs (docs/reference/specs/harness-pi.md):
-# `native`, the in-process turn loop, or `pi`, the pi coding agent started
-# inside the run's own execution container and driven over its RPC protocol —
-# with the run's model-proxy bearer as its only key, every event bridged onto
-# the run's own record, our tools and tool rules as its extension. Every
-# preset declares `native` until the pi series moves it; a deployment moves a
-# preset early here. A preset with a workspace (coding, review, explore) runs
-# pi in the run's container; a preset without one (general, research,
-# conductor) runs it as a child of the bot, with none of pi's own shell or
-# file tools. A preset the registry does not know, or a value that is
-# neither harness, fails the load by name.
-# harness:
-#   coding: pi
+# `harness` is a retired key (docs/reference/specs/harness-pi.md item 1). It
+# moved the presets onto pi one at a time; every preset runs on pi now — the pi
+# coding agent started inside the run's own execution container (or as a child
+# of the bot for a preset without a workspace) with the run's model-proxy
+# bearer as its only key — and there is no other loop. A block written during
+# the series still loads when every value is `pi`; `native` names the deleted
+# loop and fails the load by name, as does an unknown preset. Remove the block.
 
 # What the harness writes into pi's per-run settings for every run on pi
 # (docs/reference/specs/harness-pi.md item 4): pi's compaction thresholds, in

package/dist/assets/deploy/cloudflare-resident/worker.ts

@@ -5417,14 +5417,13 @@ export class ResidentDO extends Sandbox<Env> {
           "deps-seed",
           REFRESH_BUILD_TIMEOUT_MS,
         );
-        const paths = mutableCachePaths(`${scratch}/node_modules`, parsed.mutableListing);
-        if (paths.length > 0) {
-          await this.runDepScript(
-            mutableCacheSwapScript(seedNm, `${scratch}/node_modules`, BUILD_USER, paths),
-            "deps-seed-swap",
-            GIT_NETWORK_TIMEOUT_MS,
-          );
-        }
+        await this.swapMutableCaches(
+          seedNm,
+          `${scratch}/node_modules`,
+          BUILD_USER,
+          parsed.mutableListing,
+          "deps-seed-swap",
+        );
         console.log(`deps: ${key.slice(0, 8)} seeded from ${seed.slice(0, 8)} (${parsed.mech})`);
       }
       // Unprivileged and token-free, in a tree only this attempt
@@ -5527,16 +5526,42 @@ export class ResidentDO extends Sandbox<Env> {
       "deps-materialize",
       REFRESH_BUILD_TIMEOUT_MS,
     );
-    const nmDst = `${tree}/node_modules`;
-    const paths = mutableCachePaths(nmDst, parsed.mutableListing);
-    if (paths.length > 0) {
-      await this.runDepScript(
-        mutableCacheSwapScript(entryNodeModules ?? `${CHECKOUT_DIR}/node_modules`, nmDst, user, paths),
-        "deps-mutable-swap",
-        GIT_NETWORK_TIMEOUT_MS,
+    await this.swapMutableCaches(
+      entryNodeModules ?? `${CHECKOUT_DIR}/node_modules`,
+      `${tree}/node_modules`,
+      user,
+      parsed.mutableListing,
+      "deps-mutable-swap",
+    );
+    return parsed.mech;
+  }
+
+  /** The tool-cache swap for a freshly hardlinked node_modules: the listing
+   *  through `mutableCachePaths`, every swap in one fork
+   *  (`mutableCacheSwapScript`). A path the source has no counterpart for is
+   *  a tree-private entry the script leaves in place and names on a
+   *  `skipped=` line; the log carries those names so a refresh that read warm
+   *  still says what it left alone (the failure path already quotes the
+   *  script's stdout). */
+  private async swapMutableCaches(
+    srcNodeModules: string,
+    dstNodeModules: string,
+    user: string,
+    mutableListing: string[],
+    step: "deps-mutable-swap" | "deps-seed-swap",
+  ): Promise<void> {
+    const paths = mutableCachePaths(dstNodeModules, mutableListing);
+    if (paths.length === 0) return;
+    const swapped = await this.runDepScript(
+      mutableCacheSwapScript(srcNodeModules, dstNodeModules, user, paths),
+      step,
+      GIT_NETWORK_TIMEOUT_MS,
+    );
+    if (swapped.skipped.length > 0) {
+      console.log(
+        `deps: ${step} left ${swapped.skipped.length} tree-private node_modules entr${swapped.skipped.length === 1 ? "y" : "ies"} with no store counterpart in place: ${swapped.skipped.join(", ")}`,
       );
     }
-    return parsed.mech;
   }
 
   /** Adopt a checkout that already holds node_modules but whose key has no