@vitest-agent/engine 0.2.11 → 0.3.0

package/index.d.ts

@@ -2427,6 +2427,21 @@ export interface SessionContext {
  * @public
  */
 export declare const parseSessionEnvExports: (content: string) => Record<string, string>;
+/**
+ * The synchronous filesystem reads {@link recoverSessionContextFromSessionEnv}
+ * performs. Each member throws on a missing or unreadable path, exactly as
+ * the `node:fs` call it stands in for; recovery treats a throw as "skip".
+ *
+ * @public
+ */
+export interface SessionEnvFileSystem {
+  /** Entry names directly under `path` (`readdirSync`). */
+  readonly readDirectory: (path: string) => ReadonlyArray<string>;
+  /** The UTF-8 text of the file at `path` (`readFileSync(path, "utf8")`). */
+  readonly readFile: (path: string) => string;
+  /** Modification time of the file at `path`, in epoch milliseconds (`statSync(path).mtimeMs`). */
+  readonly mtimeMs: (path: string) => number;
+}
 /**
  * Recover a {@link SessionContext} from the newest session-env hook file
  * whose `VITEST_AGENT_PROJECT_DIR` matches `projectDir`.
@@ -2440,12 +2455,15 @@ export declare const parseSessionEnvExports: (content: string) => Record<string,
  *
  * @param opts - `projectDir` to match against; `homeDir` is the caller's
  *   home directory (ambient input — the MCP bin passes `os.homedir()`),
- *   under which `.claude/session-env` is read
+ *   under which `.claude/session-env` is read; `fileSystem` replaces the
+ *   `node:fs` reads (tests pass an in-memory volume) and defaults to the
+ *   real disk
  * @public
  */
 export declare const recoverSessionContextFromSessionEnv: (opts: {
   readonly projectDir: string;
   readonly homeDir: string;
+  readonly fileSystem?: SessionEnvFileSystem;
 }) => SessionContext | null;
 //#endregion
 //#region src/project-dir.d.ts
@@ -2650,9 +2668,26 @@ export declare const resolveDataPath: (projectDir: string, options?: ResolveData
  * Always returns a non-empty string — falls back to the cwd basename
  * (or `"anonymous-project"`) so callers don't have to handle the
  * empty case.
+ *
+ * Synchronous and reads the real disk through `node:fs`; Effect callers
+ * that already hold a `FileSystem` use
+ * {@link resolveProjectKeyFromCwdEffect}, which applies the same rules.
  * @public
  */
 export declare const resolveProjectKeyFromCwd: (cwd: string) => string;
+/**
+ * {@link resolveProjectKeyFromCwd} read through the ambient
+ * `FileSystem` service instead of `node:fs` — same upward walk to the
+ * nearest `package.json`, same precedence, same basename fallback.
+ *
+ * Never fails: a filesystem error while probing or reading is treated like
+ * a missing or malformed `package.json`, exactly as the synchronous form
+ * does.
+ *
+ * @param cwd - Absolute directory to start the upward walk from.
+ * @public
+ */
+export declare const resolveProjectKeyFromCwdEffect: (cwd: string) => Effect.Effect<string, never, FileSystem.FileSystem>;
 //#endregion
 //#region src/utils/resolve-workspace-key.d.ts
 /**

package/index.js

@@ -55,9 +55,9 @@ import { resolveProjectDir } from "./project-dir.js";
 import { assembleHistoryRecord, assembleManifest } from "./sql/assemblers.js";
 import { _resetMigrationCacheForTesting, ensureMigrated } from "./utils/ensure-migrated.js";
 import { computeFailureSignature, normalizeAssertionShape } from "./utils/failure-signature.js";
-import { resolveProjectKeyFromCwd } from "./utils/resolve-project-key-from-cwd.js";
+import { resolveProjectKeyFromCwd, resolveProjectKeyFromCwdEffect } from "./utils/resolve-project-key-from-cwd.js";
 import { DATABASE_FILENAME, resolveDataPath } from "./utils/resolve-data-path.js";
 import { resolveWorkspaceKey } from "./utils/resolve-workspace-key.js";
 import { CURRENT_ENGINE_VERSION } from "./version.js";
 
-export { APP_NAMESPACE, AgentContext, CURRENT_ENGINE_VERSION, ConfigLive, DATABASE_FILENAME, DATA_DB_FILENAME, DATA_FALLBACK_DIR, DataReader, DataReaderLive, DataStore, DataStoreLive, DetailResolver, DetailResolverLive, DiscoveryRegistry, DiscoveryRegistryLive, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, ExecutorResolver, ExecutorResolverLive, FormatSelector, FormatSelectorLive, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, IDEMPOTENCY_ROOT_SENTINEL, INLINE_ATTACHMENT_BODY_CAP_BYTES, LoggerLive, NodePlatformLayer, OutputPipelineLive, OutputRenderer, OutputRendererLive, PROJECT_MIGRATIONS, PathResolutionLive, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PlatformLive, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityLive, ProjectIdentityTest, REGISTRY_DB_FILENAME, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, SESSIONS_DB_FILENAME, SidecarPlatformLive, VitestAgentConfigFile, _resetMigrationCacheForTesting, assembleHistoryRecord, assembleManifest, classifyEnvironment, computeFailureSignature, deriveIdempotencyKey, dispatchRecordTddArtifactEffect, endAgentEffect, ensureMigrated, formatTriageEffect, formatWrapupEffect, historyKey, makeSqliteStack, migration as migration0001, migration$1 as migration0002, normalizeAssertionShape, parseAndValidateTurnPayload, parseSessionEnvExports, recordRunWorkspaceChangesEffect, recordSessionEnd, recordSessionStart, recordTddArtifactByTaskIdEffect, recordTddArtifactEffect, recordTurnEffect, recoverSessionContextFromSessionEnv, registerAgentEffect, migration$2 as registryMigration0001, resolveDataPath, resolveHookPaths, resolveLogFile, resolveLogLevel, resolveProjectDir, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveSessionForRecording, resolveSessionMapPath, resolveWorkspaceKey, migration$3 as sessionMapMigration0001 };
+export { APP_NAMESPACE, AgentContext, CURRENT_ENGINE_VERSION, ConfigLive, DATABASE_FILENAME, DATA_DB_FILENAME, DATA_FALLBACK_DIR, DataReader, DataReaderLive, DataStore, DataStoreLive, DetailResolver, DetailResolverLive, DiscoveryRegistry, DiscoveryRegistryLive, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, ExecutorResolver, ExecutorResolverLive, FormatSelector, FormatSelectorLive, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, IDEMPOTENCY_ROOT_SENTINEL, INLINE_ATTACHMENT_BODY_CAP_BYTES, LoggerLive, NodePlatformLayer, OutputPipelineLive, OutputRenderer, OutputRendererLive, PROJECT_MIGRATIONS, PathResolutionLive, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PlatformLive, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityLive, ProjectIdentityTest, REGISTRY_DB_FILENAME, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, SESSIONS_DB_FILENAME, SidecarPlatformLive, VitestAgentConfigFile, _resetMigrationCacheForTesting, assembleHistoryRecord, assembleManifest, classifyEnvironment, computeFailureSignature, deriveIdempotencyKey, dispatchRecordTddArtifactEffect, endAgentEffect, ensureMigrated, formatTriageEffect, formatWrapupEffect, historyKey, makeSqliteStack, migration as migration0001, migration$1 as migration0002, normalizeAssertionShape, parseAndValidateTurnPayload, parseSessionEnvExports, recordRunWorkspaceChangesEffect, recordSessionEnd, recordSessionStart, recordTddArtifactByTaskIdEffect, recordTddArtifactEffect, recordTurnEffect, recoverSessionContextFromSessionEnv, registerAgentEffect, migration$2 as registryMigration0001, resolveDataPath, resolveHookPaths, resolveLogFile, resolveLogLevel, resolveProjectDir, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveProjectKeyFromCwdEffect, resolveSessionForRecording, resolveSessionMapPath, resolveWorkspaceKey, migration$3 as sessionMapMigration0001 };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/engine",
-	"version": "0.2.11",
+	"version": "0.3.0",
 	"private": false,
 	"description": "Shared services, layers, SQLite data layer, migrations and platform resolution for the vitest-agent CLI and MCP server.",
 	"keywords": [

package/programs/session-env.js

@@ -32,6 +32,12 @@ const parseSessionEnvExports = (content) => {
 	}
 	return out;
 };
+/** The default {@link SessionEnvFileSystem}: the real disk through `node:fs`. */
+const nodeSessionEnvFileSystem = {
+	readDirectory: (path) => readdirSync(path),
+	readFile: (path) => readFileSync(path, "utf8"),
+	mtimeMs: (path) => statSync(path).mtimeMs
+};
 /**
 * Recover a {@link SessionContext} from the newest session-env hook file
 * whose `VITEST_AGENT_PROJECT_DIR` matches `projectDir`.
@@ -45,15 +51,18 @@ const parseSessionEnvExports = (content) => {
 *
 * @param opts - `projectDir` to match against; `homeDir` is the caller's
 *   home directory (ambient input — the MCP bin passes `os.homedir()`),
-*   under which `.claude/session-env` is read
+*   under which `.claude/session-env` is read; `fileSystem` replaces the
+*   `node:fs` reads (tests pass an in-memory volume) and defaults to the
+*   real disk
 * @public
 */
 const recoverSessionContextFromSessionEnv = (opts) => {
+	const fs = opts.fileSystem ?? nodeSessionEnvFileSystem;
 	const root = join(opts.homeDir, ".claude", "session-env");
 	const wantDir = resolve(opts.projectDir);
 	let entries;
 	try {
-		entries = readdirSync(root);
+		entries = fs.readDirectory(root);
 	} catch {
 		return null;
 	}
@@ -61,15 +70,15 @@ const recoverSessionContextFromSessionEnv = (opts) => {
 	for (const entry of entries) {
 		const file = join(root, entry, "vitest-agent-hook.sh");
 		try {
-			const st = statSync(file);
-			const env = parseSessionEnvExports(readFileSync(file, "utf8"));
+			const mtimeMs = fs.mtimeMs(file);
+			const env = parseSessionEnvExports(fs.readFile(file));
 			const chatId = env.VITEST_AGENT_CHAT_ID;
 			const conversationId = env.VITEST_AGENT_CONVERSATION_ID;
 			const mainAgentId = env.VITEST_AGENT_MAIN_AGENT_ID ?? env.VITEST_AGENT_AGENT_ID;
 			const fileProjectDir = env.VITEST_AGENT_PROJECT_DIR;
 			if (chatId === void 0 || chatId.length === 0 || conversationId === void 0 || conversationId.length === 0 || mainAgentId === void 0 || mainAgentId.length === 0 || fileProjectDir === void 0 || resolve(fileProjectDir) !== wantDir) continue;
-			if (best === null || st.mtimeMs > best.mtimeMs) best = {
-				mtimeMs: st.mtimeMs,
+			if (best === null || mtimeMs > best.mtimeMs) best = {
+				mtimeMs,
 				ctx: {
 					chatId,
 					conversationId,

package/utils/resolve-data-path.js

@@ -1,5 +1,5 @@
 import { VitestAgentConfigFile } from "../services/Config.js";
-import { resolveProjectKeyFromCwd } from "./resolve-project-key-from-cwd.js";
+import { resolveProjectKeyFromCwdEffect } from "./resolve-project-key-from-cwd.js";
 import { VitestAgentConfig, normalizeWorkspaceKey } from "@vitest-agent/sdk";
 import { Effect, FileSystem } from "effect";
 import { AppDirs } from "@effected/xdg";
@@ -54,7 +54,7 @@ const resolveDataPath = (projectDir, options = {}) => Effect.gen(function* () {
 		return join(loaded.cacheDir, DATABASE_FILENAME);
 	}
 	const dataRoot = yield* (yield* AppDirs).ensureData;
-	const key = loaded.projectKey ? normalizeWorkspaceKey(loaded.projectKey) : resolveProjectKeyFromCwd(projectDir);
+	const key = loaded.projectKey ? normalizeWorkspaceKey(loaded.projectKey) : yield* resolveProjectKeyFromCwdEffect(projectDir);
 	const dir = join(dataRoot, key);
 	yield* fs.makeDirectory(dir, { recursive: true });
 	return join(dir, DATABASE_FILENAME);

package/utils/resolve-project-key-from-cwd.js

@@ -1,4 +1,5 @@
 import { gitUrlToProjectKey, normalizeWorkspaceKey } from "@vitest-agent/sdk";
+import { Effect, FileSystem } from "effect";
 import { existsSync, readFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 
@@ -24,6 +25,27 @@ const readRepositoryUrl = (parsed) => {
 	return null;
 };
 /**
+* The key a `package.json` body names: the canonical `host__path` form of
+* `repository.url` when present and canonicalizable, else the normalized
+* `name`, else `null` (no usable field, or malformed JSON).
+*/
+const keyFromPackageJson = (content) => {
+	try {
+		const parsed = JSON.parse(content);
+		const repoUrl = readRepositoryUrl(parsed);
+		if (repoUrl !== null) {
+			const key = gitUrlToProjectKey(repoUrl);
+			if (key !== null) return key;
+		}
+		if (typeof parsed.name === "string" && parsed.name.length > 0) return normalizeWorkspaceKey(parsed.name);
+	} catch {}
+	return null;
+};
+const basenameKey = (cwd) => {
+	const basename = cwd.split("/").filter((segment) => segment.length > 0).pop() ?? "anonymous-project";
+	return normalizeWorkspaceKey(basename);
+};
+/**
 * Compute the project key for a workspace by reading `package.json`
 * fields directly. Returns the canonical `host__path` form when a git
 * remote URL is present, otherwise the normalized package name.
@@ -31,22 +53,50 @@ const readRepositoryUrl = (parsed) => {
 * Always returns a non-empty string — falls back to the cwd basename
 * (or `"anonymous-project"`) so callers don't have to handle the
 * empty case.
+*
+* Synchronous and reads the real disk through `node:fs`; Effect callers
+* that already hold a `FileSystem` use
+* {@link resolveProjectKeyFromCwdEffect}, which applies the same rules.
 * @public
 */
 const resolveProjectKeyFromCwd = (cwd) => {
 	const pkgPath = findNearestPackageJson(cwd);
-	if (pkgPath !== null) try {
-		const parsed = JSON.parse(readFileSync(pkgPath, "utf-8"));
-		const repoUrl = readRepositoryUrl(parsed);
-		if (repoUrl !== null) {
-			const key = gitUrlToProjectKey(repoUrl);
-			if (key !== null) return key;
-		}
-		if (typeof parsed.name === "string" && parsed.name.length > 0) return normalizeWorkspaceKey(parsed.name);
-	} catch {}
-	const basename = cwd.split("/").filter((segment) => segment.length > 0).pop() ?? "anonymous-project";
-	return normalizeWorkspaceKey(basename);
+	if (pkgPath !== null) {
+		let content = null;
+		try {
+			content = readFileSync(pkgPath, "utf-8");
+		} catch {}
+		const key = content === null ? null : keyFromPackageJson(content);
+		if (key !== null) return key;
+	}
+	return basenameKey(cwd);
 };
+/**
+* {@link resolveProjectKeyFromCwd} read through the ambient
+* `FileSystem` service instead of `node:fs` — same upward walk to the
+* nearest `package.json`, same precedence, same basename fallback.
+*
+* Never fails: a filesystem error while probing or reading is treated like
+* a missing or malformed `package.json`, exactly as the synchronous form
+* does.
+*
+* @param cwd - Absolute directory to start the upward walk from.
+* @public
+*/
+const resolveProjectKeyFromCwdEffect = (cwd) => Effect.gen(function* () {
+	const fs = yield* FileSystem.FileSystem;
+	let dir = cwd;
+	while (true) {
+		const candidate = join(dir, PACKAGE_JSON);
+		if (yield* fs.exists(candidate).pipe(Effect.orElseSucceed(() => false))) {
+			const content = yield* fs.readFileString(candidate).pipe(Effect.orElseSucceed(() => null));
+			return (content === null ? null : keyFromPackageJson(content)) ?? basenameKey(cwd);
+		}
+		const parent = dirname(dir);
+		if (parent === dir) return basenameKey(cwd);
+		dir = parent;
+	}
+});
 
 //#endregion
-export { resolveProjectKeyFromCwd };
+export { resolveProjectKeyFromCwd, resolveProjectKeyFromCwdEffect };

package/version.js

@@ -1,6 +1,6 @@
 //#region src/version.ts
 /** Build-time injected package version (see architecture.md D36). @public */
-const CURRENT_ENGINE_VERSION = "0.2.11";
+const CURRENT_ENGINE_VERSION = "0.3.0";
 
 //#endregion
 export { CURRENT_ENGINE_VERSION };