@agentv/core 2.17.2 → 2.18.1

package/dist/index.d.cts

@@ -548,11 +548,9 @@ type WorkspaceConfig = {
     /** Workspace lifecycle hooks */
     readonly hooks?: WorkspaceHooksConfig;
     /** Workspace materialization mode */
-    readonly mode?: 'pooled' | 'ephemeral' | 'static';
+    readonly mode?: 'pooled' | 'temp' | 'static';
     /** Required when mode=static: use this existing directory directly */
-    readonly static_path?: string;
-    /** @deprecated Use mode=pooled|ephemeral|static */
-    readonly pool?: boolean;
+    readonly path?: string;
 };
 type CodeEvaluatorConfig = {
     readonly name: string;
@@ -2479,7 +2477,7 @@ interface RunEvaluationOptions {
     /** Pre-existing workspace directory to use directly (skips clone/copy/pool) */
     readonly workspace?: string;
     /** Workspace materialization mode override */
-    readonly workspaceMode?: 'pooled' | 'ephemeral' | 'static';
+    readonly workspaceMode?: 'pooled' | 'temp' | 'static';
     /** Static workspace path override (used when workspaceMode=static) */
     readonly workspacePath?: string;
     /** Workspace clean policy override for pooled reset */
@@ -3011,9 +3009,11 @@ interface PoolSlot {
 }
 /**
  * Compute a deterministic SHA-256 fingerprint for a workspace configuration.
- * The fingerprint captures template path and all repo configs in a canonical order.
+ * The fingerprint captures only repo materialization inputs (source, checkout, clone options)
+ * in a canonical order. Template path is excluded because template files are re-copied on
+ * every pool reuse and don't affect the cloned checkout state.
  */
-declare function computeWorkspaceFingerprint(templatePath: string | undefined | null, repos: readonly RepoConfig[]): string;
+declare function computeWorkspaceFingerprint(repos: readonly RepoConfig[]): string;
 /**
  * Pools entire workspaces (template files + git repos) for reuse across eval runs.
  *

package/dist/index.d.ts

@@ -548,11 +548,9 @@ type WorkspaceConfig = {
     /** Workspace lifecycle hooks */
     readonly hooks?: WorkspaceHooksConfig;
     /** Workspace materialization mode */
-    readonly mode?: 'pooled' | 'ephemeral' | 'static';
+    readonly mode?: 'pooled' | 'temp' | 'static';
     /** Required when mode=static: use this existing directory directly */
-    readonly static_path?: string;
-    /** @deprecated Use mode=pooled|ephemeral|static */
-    readonly pool?: boolean;
+    readonly path?: string;
 };
 type CodeEvaluatorConfig = {
     readonly name: string;
@@ -2479,7 +2477,7 @@ interface RunEvaluationOptions {
     /** Pre-existing workspace directory to use directly (skips clone/copy/pool) */
     readonly workspace?: string;
     /** Workspace materialization mode override */
-    readonly workspaceMode?: 'pooled' | 'ephemeral' | 'static';
+    readonly workspaceMode?: 'pooled' | 'temp' | 'static';
     /** Static workspace path override (used when workspaceMode=static) */
     readonly workspacePath?: string;
     /** Workspace clean policy override for pooled reset */
@@ -3011,9 +3009,11 @@ interface PoolSlot {
 }
 /**
  * Compute a deterministic SHA-256 fingerprint for a workspace configuration.
- * The fingerprint captures template path and all repo configs in a canonical order.
+ * The fingerprint captures only repo materialization inputs (source, checkout, clone options)
+ * in a canonical order. Template path is excluded because template files are re-copied on
+ * every pool reuse and don't affect the cloned checkout state.
  */
-declare function computeWorkspaceFingerprint(templatePath: string | undefined | null, repos: readonly RepoConfig[]): string;
+declare function computeWorkspaceFingerprint(repos: readonly RepoConfig[]): string;
 /**
  * Pools entire workspaces (template files + git repos) for reuse across eval runs.
  *

package/dist/index.js

@@ -17,7 +17,7 @@ import {
   readTextFile,
   resolveFileReference,
   resolveTargetDefinition
-} from "./chunk-PSYFRPNT.js";
+} from "./chunk-I4VQY3XJ.js";
 import {
   OtlpJsonFileExporter
 } from "./chunk-HFSYZHGF.js";
@@ -3187,6 +3187,17 @@ async function resolveWorkspaceConfig(raw, evalFileDir) {
 function parseWorkspaceConfig(raw, evalFileDir) {
   if (!isJsonObject(raw)) return void 0;
   const obj = raw;
+  if ("static_path" in obj) {
+    throw new Error(
+      "workspace.static_path has been removed. Use workspace.path with workspace.mode=static."
+    );
+  }
+  if ("pool" in obj) {
+    throw new Error("workspace.pool has been removed. Use workspace.mode='pooled' or 'temp'.");
+  }
+  if ("static" in obj) {
+    throw new Error("workspace.static has been removed. Use workspace.mode='static'.");
+  }
   let template = typeof obj.template === "string" ? obj.template : void 0;
   if (template && !path8.isAbsolute(template)) {
     template = path8.resolve(evalFileDir, template);
@@ -3194,19 +3205,17 @@ function parseWorkspaceConfig(raw, evalFileDir) {
   const isolation = obj.isolation === "shared" || obj.isolation === "per_test" ? obj.isolation : void 0;
   const repos = Array.isArray(obj.repos) ? obj.repos.map(parseRepoConfig).filter(Boolean) : void 0;
   const hooks = parseWorkspaceHooksConfig(obj.hooks, evalFileDir);
-  const mode = obj.mode === "pooled" || obj.mode === "ephemeral" || obj.mode === "static" ? obj.mode : void 0;
-  const staticPath = typeof obj.static_path === "string" ? obj.static_path : void 0;
-  const pool = typeof obj.pool === "boolean" ? obj.pool : void 0;
-  if (!template && !isolation && !repos && !hooks && !mode && !staticPath && pool === void 0)
-    return void 0;
+  const explicitMode = obj.mode === "pooled" || obj.mode === "temp" || obj.mode === "static" ? obj.mode : void 0;
+  const workspacePath = typeof obj.path === "string" ? obj.path : void 0;
+  const mode = explicitMode ?? (workspacePath ? "static" : void 0);
+  if (!template && !isolation && !repos && !hooks && !mode && !workspacePath) return void 0;
   return {
     ...template !== void 0 && { template },
     ...isolation !== void 0 && { isolation },
     ...repos !== void 0 && { repos },
     ...hooks !== void 0 && { hooks },
     ...mode !== void 0 && { mode },
-    ...staticPath !== void 0 && { static_path: staticPath },
-    ...pool !== void 0 && { pool }
+    ...workspacePath !== void 0 && { path: workspacePath }
   };
 }
 function mergeWorkspaceConfigs(suiteLevel, caseLevel) {
@@ -3233,8 +3242,7 @@ function mergeWorkspaceConfigs(suiteLevel, caseLevel) {
     repos: caseLevel.repos ?? suiteLevel.repos,
     ...hasHooks && { hooks: mergedHooks },
     mode: caseLevel.mode ?? suiteLevel.mode,
-    static_path: caseLevel.static_path ?? suiteLevel.static_path,
-    pool: caseLevel.pool ?? suiteLevel.pool
+    path: caseLevel.path ?? suiteLevel.path
   };
 }
 function asString6(value) {
@@ -12940,9 +12948,8 @@ function normalizeRepoForFingerprint(repo) {
   }
   return result;
 }
-function computeWorkspaceFingerprint(templatePath, repos) {
+function computeWorkspaceFingerprint(repos) {
   const canonical = {
-    templatePath: templatePath ?? null,
     repos: [...repos].sort((a, b) => a.path.localeCompare(b.path)).map(normalizeRepoForFingerprint)
   };
   return createHash("sha256").update(JSON.stringify(canonical)).digest("hex");
@@ -12984,7 +12991,7 @@ var WorkspacePoolManager = class {
    */
   async acquireWorkspace(options) {
     const { templatePath, repos, maxSlots, repoManager, poolReset } = options;
-    const fingerprint = computeWorkspaceFingerprint(templatePath, repos);
+    const fingerprint = computeWorkspaceFingerprint(repos);
     const poolDir = path36.join(this.poolRoot, fingerprint);
     await mkdir11(poolDir, { recursive: true });
     const drifted = await this.checkDrift(poolDir, fingerprint);
@@ -13595,19 +13602,27 @@ async function runEvaluation(options) {
     }
   };
   const isPerTestIsolation = suiteWorkspace?.isolation === "per_test";
-  const configuredMode = suiteWorkspace?.mode ?? workspaceMode;
-  const configuredStaticPath = suiteWorkspace?.static_path ?? workspacePath ?? legacyWorkspacePath;
-  const useStaticWorkspace = configuredMode === "static" || !!configuredStaticPath && !configuredMode;
+  const cliWorkspacePath = workspacePath ?? legacyWorkspacePath;
+  const yamlWorkspacePath = suiteWorkspace?.path;
+  if (cliWorkspacePath && workspaceMode && workspaceMode !== "static") {
+    throw new Error("--workspace-path requires --workspace-mode static when both are provided");
+  }
+  const configuredMode = cliWorkspacePath ? "static" : workspaceMode ?? suiteWorkspace?.mode ?? (yamlWorkspacePath ? "static" : "pooled");
+  const configuredStaticPath = cliWorkspacePath ?? yamlWorkspacePath;
+  const useStaticWorkspace = configuredMode === "static";
   if (useStaticWorkspace && isPerTestIsolation) {
     throw new Error(
       "static workspace mode is incompatible with isolation: per_test. Use isolation: shared (default)."
     );
   }
   if (configuredMode === "static" && !configuredStaticPath) {
-    throw new Error("workspace.mode=static requires workspace.static_path or --workspace-path");
+    throw new Error("workspace.mode=static requires workspace.path or --workspace-path");
+  }
+  if (configuredMode !== "static" && configuredStaticPath) {
+    throw new Error("workspace.path requires workspace.mode=static");
   }
   const hasSharedWorkspace = !!(useStaticWorkspace || workspaceTemplate || suiteWorkspace?.hooks || suiteWorkspace?.repos?.length && !isPerTestIsolation);
-  const poolEnabled = configuredMode === "pooled" ? true : configuredMode === "ephemeral" || useStaticWorkspace ? false : suiteWorkspace?.pool ?? poolWorkspaces ?? true;
+  const poolEnabled = configuredMode === "pooled";
   const usePool = poolEnabled !== false && !!suiteWorkspace?.repos?.length && !isPerTestIsolation && !useStaticWorkspace;
   const resolvedRetainOnSuccess = retainOnSuccess ?? (keepWorkspaces ? "keep" : "cleanup");
   const resolvedRetainOnFailure = retainOnFailure ?? (cleanupWorkspaces ? "cleanup" : "keep");