@vitest-agent/engine 0.2.10 → 0.2.11

package/index.d.ts

@@ -1370,6 +1370,15 @@ export declare const OutputRendererLive: Layer.Layer<OutputRenderer, never, neve
  * @public
  */
 export declare const APP_NAMESPACE = "vitest-agent";
+/**
+ * Where the XDG data root lands when `XDG_DATA_HOME` is unset:
+ * `$HOME/.local/share/vitest-agent` — the XDG spec default. `AppDirs` alone
+ * would fall back to `$HOME/.vitest-agent`. Shared by `PathResolutionLive`
+ * (reporter/MCP route) and `resolveHookPaths` (hook/sidecar route) so both
+ * routes land in the same directory.
+ * @public
+ */
+export declare const DATA_FALLBACK_DIR = ".local/share/vitest-agent";
 /**
  * Composite layer providing every service `resolveDataPath` requires:
  * `AppDirs` (XDG path resolution), `VitestAgentConfigFile` (TOML

package/index.js

@@ -22,7 +22,7 @@ import { LoggerLive, resolveLogFile, resolveLogLevel } from "./layers/LoggerLive
 import { OutputRenderer } from "./services/OutputRenderer.js";
 import { OutputRendererLive } from "./layers/OutputRendererLive.js";
 import { OutputPipelineLive } from "./layers/OutputPipelineLive.js";
-import { APP_NAMESPACE, PathResolutionLive } from "./layers/PathResolutionLive.js";
+import { APP_NAMESPACE, DATA_FALLBACK_DIR, PathResolutionLive } from "./layers/PathResolutionLive.js";
 import { PerClientSessionMapReader, PerClientSessionMapWriter } from "./services/PerClientSessionMap.js";
 import { PerClientSessionMapReaderLive, PerClientSessionMapWriterLive } from "./layers/PerClientSessionMapLive.js";
 import { ProjectDiscovery } from "./services/ProjectDiscovery.js";
@@ -60,4 +60,4 @@ 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, 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, resolveSessionForRecording, resolveSessionMapPath, resolveWorkspaceKey, migration$3 as sessionMapMigration0001 };

package/layers/DataReaderLive.js

@@ -606,14 +606,28 @@ const DataReaderLive = Layer.effect(DataReader, Effect.gen(function* () {
 		const runs = yield* sql`SELECT id FROM test_runs WHERE project = ${project} ORDER BY timestamp DESC LIMIT 1`;
 		return runs.length === 0 ? null : runs[0].id;
 	});
-	const attachmentsFor = (column, ownerId) => Effect.gen(function* () {
-		return (column === "annotation_id" ? yield* sql`SELECT content_type, path, body, body_encoding, byte_size FROM attachments WHERE annotation_id = ${ownerId} ORDER BY id` : yield* sql`SELECT content_type, path, body, body_encoding, byte_size FROM attachments WHERE artifact_id = ${ownerId} ORDER BY id`).map((r) => ({
-			...r.content_type !== null && { contentType: r.content_type },
-			...r.path !== null && { path: r.path },
-			...r.body !== null && { body: r.body },
-			...r.body_encoding !== null && { bodyEncoding: r.body_encoding },
-			byteSize: r.byte_size
-		}));
+	/**
+	* Load the attachments of every owner in `ownerIds` with a single query,
+	* grouped by owner id. Per-owner order follows `attachments.id`. An empty
+	* owner list issues no query.
+	*/
+	const attachmentsForMany = (column, ownerIds) => Effect.gen(function* () {
+		const grouped = /* @__PURE__ */ new Map();
+		if (ownerIds.length === 0) return grouped;
+		const rows = column === "annotation_id" ? yield* sql`SELECT annotation_id AS owner_id, content_type, path, body, body_encoding, byte_size FROM attachments WHERE annotation_id IN ${sql.in(ownerIds)} ORDER BY id` : yield* sql`SELECT artifact_id AS owner_id, content_type, path, body, body_encoding, byte_size FROM attachments WHERE artifact_id IN ${sql.in(ownerIds)} ORDER BY id`;
+		for (const r of rows) {
+			const attachment = {
+				...r.content_type !== null && { contentType: r.content_type },
+				...r.path !== null && { path: r.path },
+				...r.body !== null && { body: r.body },
+				...r.body_encoding !== null && { bodyEncoding: r.body_encoding },
+				byteSize: r.byte_size
+			};
+			const list = grouped.get(r.owner_id);
+			if (list === void 0) grouped.set(r.owner_id, [attachment]);
+			else list.push(attachment);
+		}
+		return grouped;
 	});
 	const getAnnotationsForTest = (project, fullName, options) => Effect.gen(function* () {
 		yield* Effect.logDebug("getAnnotationsForTest").pipe(Effect.annotateLogs({
@@ -631,9 +645,10 @@ const DataReaderLive = Layer.effect(DataReader, Effect.gen(function* () {
 					WHERE tm.run_id = ${runId} AND tc.full_name = ${fullName}
 						AND (${modulePath} IS NULL OR tm.relative_module_id = ${modulePath})
 					ORDER BY ta.id`;
+		const attachmentsByOwner = yield* attachmentsForMany("annotation_id", rows.map((r) => r.id));
 		const out = [];
 		for (const r of rows) {
-			const attachments = yield* attachmentsFor("annotation_id", r.id);
+			const attachments = attachmentsByOwner.get(r.id) ?? [];
 			out.push({
 				id: r.id,
 				type: r.type,
@@ -668,9 +683,10 @@ const DataReaderLive = Layer.effect(DataReader, Effect.gen(function* () {
 					WHERE tm.run_id = ${runId} AND tc.full_name = ${fullName}
 						AND (${modulePath} IS NULL OR tm.relative_module_id = ${modulePath})
 					ORDER BY ta.id`;
+		const attachmentsByOwner = yield* attachmentsForMany("artifact_id", rows.map((r) => r.id));
 		const out = [];
 		for (const r of rows) {
-			const attachments = yield* attachmentsFor("artifact_id", r.id);
+			const attachments = attachmentsByOwner.get(r.id) ?? [];
 			out.push({
 				id: r.id,
 				type: r.type,

package/layers/PathResolutionLive.js

@@ -11,12 +11,24 @@ import { AppDirs, Xdg } from "@effected/xdg";
 */
 const APP_NAMESPACE = "vitest-agent";
 /**
+* Where the XDG data root lands when `XDG_DATA_HOME` is unset:
+* `$HOME/.local/share/vitest-agent` — the XDG spec default. `AppDirs` alone
+* would fall back to `$HOME/.vitest-agent`. Shared by `PathResolutionLive`
+* (reporter/MCP route) and `resolveHookPaths` (hook/sidecar route) so both
+* routes land in the same directory.
+* @public
+*/
+const DATA_FALLBACK_DIR = `.local/share/${APP_NAMESPACE}`;
+/**
 * `AppDirs` over the ambient `Xdg` environment, bound to a `const` so the
 * layer memoizes by reference (calling `AppDirs.layer(...)` inline at two
 * provide sites would resolve two independent services). Requires
 * `FileSystem` + `Path` at the edge.
 */
-const AppDirsLive = AppDirs.layer({ namespace: APP_NAMESPACE }).pipe(Layer.provide(Xdg.layer));
+const AppDirsLive = AppDirs.layer({
+	namespace: APP_NAMESPACE,
+	fallbackDir: DATA_FALLBACK_DIR
+}).pipe(Layer.provide(Xdg.layer));
 /**
 * Composite layer providing every service `resolveDataPath` requires:
 * `AppDirs` (XDG path resolution), `VitestAgentConfigFile` (TOML
@@ -43,4 +55,4 @@ const PathResolutionLive = (projectDir) => {
 };
 
 //#endregion
-export { APP_NAMESPACE, PathResolutionLive };
+export { APP_NAMESPACE, DATA_FALLBACK_DIR, PathResolutionLive };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/engine",
-	"version": "0.2.10",
+	"version": "0.2.11",
 	"private": false,
 	"description": "Shared services, layers, SQLite data layer, migrations and platform resolution for the vitest-agent CLI and MCP server.",
 	"keywords": [
@@ -42,14 +42,14 @@
 	"dependencies": {
 		"@effect/platform-node": "4.0.0-rc.118",
 		"@effect/sql-sqlite-node": "4.0.0-rc.118",
-		"@effected/config-file": "^0.13.0",
+		"@effected/config-file": "^0.13.1",
 		"@effected/engine": "^0.2.0",
 		"@effected/glob": "^0.9.0",
 		"@effected/jsonc": "^0.14.0",
 		"@effected/toml": "^0.10.0",
-		"@effected/walker": "^0.13.0",
-		"@effected/workspaces": "^0.30.1",
-		"@effected/xdg": "^0.8.0",
+		"@effected/walker": "^0.14.0",
+		"@effected/workspaces": "^0.30.2",
+		"@effected/xdg": "^0.8.1",
 		"@effected/yaml": "^0.18.0",
 		"@vitest-agent/sdk": "5.1.3",
 		"effect": "4.0.0-rc.118",

package/programs/hook-paths.js

@@ -1,4 +1,4 @@
-import { APP_NAMESPACE } from "../layers/PathResolutionLive.js";
+import { APP_NAMESPACE, DATA_FALLBACK_DIR } from "../layers/PathResolutionLive.js";
 import { ProjectIdentityNotResolvableError } from "@vitest-agent/sdk";
 import { ConfigProvider, Effect, FileSystem, Layer, Path } from "effect";
 import { AppDirs, Xdg } from "@effected/xdg";
@@ -22,12 +22,6 @@ const SESSIONS_DB_FILENAME = "sessions.db";
 * @public
 */
 const REGISTRY_DB_FILENAME = "registry.db";
-/**
-* Where the XDG data root lands when `XDG_DATA_HOME` is unset:
-* `$HOME/.local/share/vitest-agent` — the XDG spec default the sidecar has
-* always used. (`AppDirs` alone would fall back to `$HOME/.vitest-agent`.)
-*/
-const HOOK_DATA_FALLBACK_DIR = `.local/share/${APP_NAMESPACE}`;
 /** Directory under the home dir that holds the per-client `sessions.db` fallback. */
 const SESSION_MAP_HOME_DIR = `.${APP_NAMESPACE}`;
 const nonEmpty = (value) => value !== void 0 && value.length > 0 ? value : void 0;
@@ -48,7 +42,7 @@ const hookAppDirs = (env) => {
 	const XdgLive = Xdg.layer.pipe(Layer.provide(ConfigProvider.layer(provider)));
 	return AppDirs.layer({
 		namespace: APP_NAMESPACE,
-		fallbackDir: HOOK_DATA_FALLBACK_DIR
+		fallbackDir: DATA_FALLBACK_DIR
 	}).pipe(Layer.provide(XdgLive));
 };
 /**

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.10";
+const CURRENT_ENGINE_VERSION = "0.2.11";
 
 //#endregion
 export { CURRENT_ENGINE_VERSION };