@pipelab/core-node 1.0.0-beta.17 → 1.0.0-beta.19

package/src/runner.ts

@@ -1,11 +1,9 @@
 import { executeGraphWithHistory } from "./utils";
 import { setupConfigFile } from "./config";
-import { isDev, PipelabContext } from "./context";
+import { isDev, PipelabContext, CacheFolder } from "./context";
 import { readFile, access, writeFile, mkdir } from "node:fs/promises";
 import { resolve, isAbsolute, join, dirname } from "node:path";
 import { savedFileMigrator } from "@pipelab/shared";
-import type { AppConfig } from "@pipelab/shared";
-import { registerMigrationHandlers } from "./migrations";
 import { registerAllHandlers } from "./handlers/index";
 
 export interface RunOptions {
@@ -65,11 +63,8 @@ export async function runPipelineCommand(file: string, options: RunOptions, vers
   });
 
   await registerAllHandlers({ version, context });
-  registerMigrationHandlers(context);
 
-  const settings = await setupConfigFile<AppConfig>("settings", { context });
-  const config = await settings.getConfig();
-  const cachePath = join(context.userDataPath, "cache");
+  const cachePath = context.getCachePath(CacheFolder.Pipelines, effectivePipelineId);
   await mkdir(cachePath, { recursive: true });
 
   const abortController = new AbortController();

package/src/server.ts

@@ -9,11 +9,9 @@ import {
 import { getUiDevServerMissingWarning, uiDevPort } from "@pipelab/constants";
 import { existsSync } from "node:fs";
 import { readFile } from "node:fs/promises";
-import { join } from "node:path";
 import http from "http";
 // @ts-expect-error serve-handler has no type definitions
 import handler from "serve-handler";
-import { registerMigrationHandlers } from "./migrations";
 
 export interface ServeOptions {
   port: string | number;
@@ -116,6 +114,7 @@ export async function serveCommand(options: ServeOptions, version: string, _dirn
 
     return handler(request, response, {
       public: rawAssetFolder,
+      rewrites: [{ source: "/**", destination: "/index.html" }],
     });
   });
 
@@ -134,7 +133,6 @@ export async function serveCommand(options: ServeOptions, version: string, _dirn
     version,
     context,
   });
-  registerMigrationHandlers(context);
 
   sendStartupReady();
 

package/src/utils/github.test.ts

@@ -29,7 +29,7 @@ describe("GitHub Release Updates API", () => {
     expect(releases).toEqual([]);
     expect(fetchSpy).toHaveBeenCalledWith(
       expect.stringContaining("matching-refs/tags/%40pipelab%2Fapp"),
-      expect.any(Object)
+      expect.any(Object),
     );
   });
 
@@ -153,7 +153,7 @@ describe("GitHub Release Updates API", () => {
     expect(releases[0].tag_name).toBe("@pipelab/app@1.2.3");
     expect(fetchSpy).toHaveBeenCalledWith(
       expect.stringContaining("releases/tags/%40pipelab%2Fapp%401.2.3"),
-      expect.any(Object)
+      expect.any(Object),
     );
   });
 
@@ -189,9 +189,7 @@ describe("GitHub Release Updates API", () => {
     // Mock matching refs
     fetchSpy.mockResolvedValueOnce({
       ok: true,
-      json: async () => [
-        { ref: "refs/tags/@pipelab/app@3.0.0" },
-      ],
+      json: async () => [{ ref: "refs/tags/@pipelab/app@3.0.0" }],
     } as any);
 
     // Mock release lookup

package/src/utils/remote.test.ts

@@ -93,13 +93,16 @@ describe("remote utilities & offline mode", () => {
 
   test("fetchPipelabPlugin maps latest to releaseTag for official plugins", async () => {
     vi.advanceTimersByTime(30000); // Bypass 10s caching
-    const context = new PipelabContext({ userDataPath: "/tmp/pipelab-test-remote", releaseTag: "beta" });
+    const context = new PipelabContext({
+      userDataPath: "/tmp/pipelab-test-remote",
+      releaseTag: "beta",
+    });
     const pluginName = "@pipelab/plugin-poki";
     const packageBaseDir = context.getPackagesPath(pluginName);
     const cachedVersionDir = path.join(packageBaseDir, "1.0.0-beta.15");
 
     vi.spyOn(dns, "lookup").mockResolvedValue({ address: "1.2.3.4", family: 4 } as any);
-    
+
     vi.mocked(pacote.packument).mockResolvedValue({
       name: pluginName,
       versions: {
@@ -128,13 +131,16 @@ describe("remote utilities & offline mode", () => {
 
   test("fetchPipelabPlugin maps latest to releaseTag for custom plugins and falls back to latest if tag is missing", async () => {
     vi.advanceTimersByTime(40000); // Bypass 10s caching
-    const context = new PipelabContext({ userDataPath: "/tmp/pipelab-test-remote-custom", releaseTag: "beta" });
+    const context = new PipelabContext({
+      userDataPath: "/tmp/pipelab-test-remote-custom",
+      releaseTag: "beta",
+    });
     const pluginName = "custom-cool-plugin";
     const packageBaseDir = context.getPackagesPath(pluginName);
     const cachedVersionDir = path.join(packageBaseDir, "2.0.0");
 
     vi.spyOn(dns, "lookup").mockResolvedValue({ address: "1.2.3.4", family: 4 } as any);
-    
+
     vi.mocked(pacote.packument).mockResolvedValue({
       name: pluginName,
       versions: {
@@ -163,13 +169,16 @@ describe("remote utilities & offline mode", () => {
 
   test("fetchPipelabPlugin maps latest to releaseTag for custom plugins and falls back to latest if beta is stale", async () => {
     vi.advanceTimersByTime(50000); // Bypass 10s caching
-    const context = new PipelabContext({ userDataPath: "/tmp/pipelab-test-remote-stale", releaseTag: "beta" });
+    const context = new PipelabContext({
+      userDataPath: "/tmp/pipelab-test-remote-stale",
+      releaseTag: "beta",
+    });
     const pluginName = "stale-beta-plugin";
     const packageBaseDir = context.getPackagesPath(pluginName);
     const cachedVersionDir = path.join(packageBaseDir, "2.0.0");
 
     vi.spyOn(dns, "lookup").mockResolvedValue({ address: "1.2.3.4", family: 4 } as any);
-    
+
     vi.mocked(pacote.packument).mockResolvedValue({
       name: pluginName,
       versions: {

package/src/utils/remote.ts

@@ -14,7 +14,7 @@ import { existsSync, constants, statSync, readdirSync } from "node:fs";
 import dns from "node:dns/promises";
 import pacote from "pacote";
 import semver from "semver";
-import { isDev, projectRoot, PipelabContext } from "../context";
+import { isDev, projectRoot, PipelabContext, CacheFolder } from "../context";
 import { execa } from "execa";
 import { sendStartupProgress } from "../server";
 import { downloadFile, extractZip, extractTarGz } from "./fs-extras";
@@ -163,7 +163,7 @@ export async function fetchPackage(
   } else {
     try {
       // 1. Resolve version/range using npm with session-wide memoization and disk cache
-      const cachePath = join(ctx.userDataPath, "cache", "pacote");
+      const cachePath = ctx.getCachePath(CacheFolder.Pacote);
       let packumentPromise = packumentRequests.get(packageName);
       if (!packumentPromise) {
         packumentPromise = pacote.packument(packageName, { cache: cachePath });
@@ -183,7 +183,10 @@ export async function fetchPackage(
       if (range === "latest" && ctx.releaseTag && ctx.releaseTag !== "latest") {
         const releaseTagVersion = packument["dist-tags"]?.[ctx.releaseTag];
         if (releaseTagVersion && semver.valid(releaseTagVersion)) {
-          if (!foundVersion || (semver.valid(foundVersion) && semver.gte(releaseTagVersion, foundVersion))) {
+          if (
+            !foundVersion ||
+            (semver.valid(foundVersion) && semver.gte(releaseTagVersion, foundVersion))
+          ) {
             console.log(
               `[Fetcher] Using release tag "${ctx.releaseTag}" (${releaseTagVersion}) instead of "latest" (${foundVersion || "none"}) for ${packageName}`,
             );
@@ -227,7 +230,7 @@ export async function fetchPackage(
     }
   }
 
-  const cachePath = join(ctx.userDataPath, "cache", "pacote");
+  const cachePath = ctx.getCachePath(CacheFolder.Pacote);
   const packageDir = join(baseDir, resolvedVersion);
 
   const checkStart = Date.now();
@@ -351,7 +354,7 @@ export async function runPnpm(
       ...process.env,
       NODE_ENV: "production",
       PATH: nodePath ? `${dirname(nodePath)}${delimiter}${process.env.PATH}` : process.env.PATH,
-      PNPM_HOME: join(ctx.userDataPath, "pnpm"),
+      PNPM_HOME: ctx.getPnpmPath(),
       PNPM_ONLY_ALLOW_TRUSTED_DEPENDENCIES: "false",
       ...extraEnv,
     },

package/src/utils/storage.ts

@@ -12,7 +12,8 @@ export class JsonFileStorage {
   private logger = useLogger().logger;
 
   constructor(fileName: string, context: PipelabContext) {
-    this.filePath = path.join(context.userDataPath, fileName);
+    this.filePath = context.getConfigPath(fileName);
+
     const dir = path.dirname(this.filePath);
     if (!fs.existsSync(dir)) {
       try {

package/src/utils.ts

@@ -21,7 +21,7 @@ import { BuildHistoryStorage } from "./handlers/build-history";
 import { ensure, extractTarGz, extractZip, zipFolder } from "./utils/fs-extras";
 import { fetchPipelabAsset } from "./utils/remote";
 import { loadPipelabPlugin } from "./plugins-registry";
-import { setupConfigFile } from "./config";
+import { setupSettingsConfigFile } from "./config";
 
 export const getFinalPlugins = () => {
   const { plugins } = usePlugins();
@@ -102,6 +102,7 @@ export const executeGraphWithHistory = async ({
     projectName,
     projectPath,
     pipelineId,
+    cachePath,
     startTime,
     status: "running",
     logs: [],
@@ -148,7 +149,7 @@ export const executeGraphWithHistory = async ({
   }
 
   logger().info(`[Sandbox] Execution sandbox created at: ${sandboxPath}`);
-  const settingsFile = await setupConfigFile<AppConfig>("settings", { context: ctx });
+  const settingsFile = await setupSettingsConfigFile(ctx);
   const config = await settingsFile.getConfig();
   const shouldCleanup = true;
 
@@ -247,18 +248,17 @@ export const executeGraphWithHistory = async ({
 
     throw error;
   } finally {
-    // Apply retention policy regardless of outcome
-    if (!shouldDisableHistory) {
-      // Don't await, let it run in the background
-      buildHistoryStorage.applyRetentionPolicy();
-    }
-
     if (shouldCleanup) {
       try {
         await rm(sandboxPath, { recursive: true, force: true });
       } catch (e) {
         console.warn(`Failed to cleanup sandbox at ${sandboxPath}:`, e);
       }
+      try {
+        await rm(cachePath, { recursive: true, force: true });
+      } catch (e) {
+        console.warn(`Failed to cleanup cache at ${cachePath}:`, e);
+      }
     }
   }
 };

package/dist/config-CEkOf95p.mjs

@@ -1,2 +0,0 @@
-import { setupConfigFile } from "./index.mjs";
-export { setupConfigFile };

package/src/migrations.ts

@@ -1,60 +0,0 @@
-import { useAPI } from "./ipc-core";
-import { PipelabContext } from "./context";
-import { setupConfigFile, getMigrator } from "./config";
-import { useLogger } from "@pipelab/shared";
-
-/**
- * Registers migration handlers for the CLI/Standalone server.
- * This overrides the default handlers from @pipelab/core-node to include
- * pipeline and file repo migrations directly in the backend.
- */
-export function registerMigrationHandlers(context: PipelabContext) {
-  const { handle } = useAPI();
-  const { logger } = useLogger();
-
-  handle("config:load", async (_, { send, value }) => {
-    const { config: name } = value;
-    logger().info("[CLI Migration] config:load", name);
-
-    try {
-      // Determine which migrator to use
-      const migrator = getMigrator(name);
-
-      if (!migrator) {
-        throw new Error(
-          `No migrator found for configuration: ${name}. All files loaded via config:load MUST have a migration schema.`,
-        );
-      }
-
-      // We use a custom loading logic that applies the migrator
-      // Since setupConfigFile in core-node expects the migrator to be in the configRegistry,
-      // and we want to keep migrations in the CLI package, we'll manually apply them here.
-
-      const manager = await setupConfigFile(name, { context, migrator });
-      const json = await manager.getConfig();
-
-      send({
-        type: "end",
-        data: {
-          type: "success",
-          result: {
-            result: json,
-          },
-        },
-      });
-    } catch (e) {
-      logger().error(`[CLI Migration] config:load error for ${name}:`, e);
-      send({
-        type: "end",
-        data: {
-          type: "error",
-          ipcError: e instanceof Error ? e.message : `Unable to load config ${name}`,
-        },
-      });
-    }
-  });
-
-  // We also need to override config:save if we want to ensure consistency,
-  // although mostly we just care about migrations on load.
-  // The default config:save from core-node will work fine as it uses the same setupConfigFile logic.
-}