@vitest-agent/sdk 2.0.16 → 2.2.0

package/errors/DataStoreError.js

@@ -35,18 +35,35 @@ const extractSqlReason = (e) => {
 	while (node && typeof node === "object" && !seen.has(node)) {
 		seen.add(node);
 		const n = node;
-		if (typeof n.message === "string" && n.message.length > 0) best = n.message;
-		if (typeof n.cause === "string") {
-			if (n.cause.length > 0) best = n.cause;
+		let msg;
+		let cause;
+		try {
+			msg = n.message;
+		} catch {
+			msg = void 0;
+		}
+		try {
+			cause = n.cause;
+		} catch {
+			cause = void 0;
+		}
+		if (typeof msg === "string" && msg.length > 0) best = msg;
+		if (typeof cause === "string") {
+			if (cause.length > 0) best = cause;
 			break;
 		}
-		node = n.cause;
+		node = cause;
 	}
 	if (best !== void 0) return best;
 	if (e && typeof e === "object") try {
-		return JSON.stringify(e);
+		const json = JSON.stringify(e);
+		if (json !== void 0) return json;
 	} catch {}
-	return String(e);
+	try {
+		return String(e);
+	} catch {
+		return "<unserializable error>";
+	}
 };
 
 //#endregion

package/index.d.ts

@@ -375,6 +375,16 @@ declare const RenderState: Schema.Struct<{
     readonly detail: Schema.String;
     readonly targetTool: Schema.optional<Schema.String>;
   }>>;
+  /**
+   * Count of every collected module, passing modules included. Folded
+   * from `RunFinished.collectedModules`. Renderers prefer this over
+   * `moduleOrder.length` for "N modules all-passed" copy — a report
+   * replay only queues failing modules, so `moduleOrder` alone
+   * undercounts a green run to zero (issue #204). Optional — a run
+   * that never carried the field (older replay data, hand-built
+   * fixtures) falls back to `moduleOrder.length`.
+   */
+  readonly collectedModules: Schema.optional<Schema.Number>;
 }>;
 /** @public */
 type RenderState = typeof RenderState.Type;
@@ -519,6 +529,14 @@ declare const ReportSummary: Schema.Struct<{
   readonly failed: Schema.Number;
   readonly skipped: Schema.Number;
   readonly duration: Schema.Number;
+  /**
+   * Count of every collected module, passing modules included.
+   * `AgentReport.failed` only carries failing modules, so a green run
+   * has no other way to know how many files actually ran — without
+   * this a fully-passing replay reads as "0 modules all-passed"
+   * (issue #204). Optional to keep existing fixtures/reports valid.
+   */
+  readonly modules: Schema.optional<Schema.Number>;
 }>;
 /** @public */
 type ReportSummary = typeof ReportSummary.Type;
@@ -603,6 +621,14 @@ declare const AgentReport: Schema.Struct<{
     readonly failed: Schema.Number;
     readonly skipped: Schema.Number;
     readonly duration: Schema.Number;
+    /**
+     * Count of every collected module, passing modules included.
+     * `AgentReport.failed` only carries failing modules, so a green run
+     * has no other way to know how many files actually ran — without
+     * this a fully-passing replay reads as "0 modules all-passed"
+     * (issue #204). Optional to keep existing fixtures/reports valid.
+     */
+    readonly modules: Schema.optional<Schema.Number>;
   }>;
   readonly failed: Schema.$Array<Schema.Struct<{
     readonly file: Schema.String;
@@ -1090,6 +1116,14 @@ declare const RunEvent: Schema.Union<readonly [Schema.TaggedStruct<"RunStarted",
   readonly skipCount: Schema.Number;
   readonly durationMs: Schema.Number;
   readonly timeoutCount: Schema.optional<Schema.Number>;
+  /**
+   * Count of every collected module, passing modules included.
+   * Carries `AgentReport.summary.modules` (or the live module
+   * count) into the event stream so the reducer can fold the
+   * true count into `RenderState` even when `moduleOrder` only
+   * tracks failing modules (report replay). See issue #204.
+   */
+  readonly collectedModules: Schema.optional<Schema.Number>;
 }>]>;
 /** @public */
 type RunEvent = typeof RunEvent.Type;
@@ -4457,6 +4491,12 @@ interface VitestTestSuite {
     line: number;
     column: number;
   };
+  /**
+   * Errors attached to the suite entity itself — a `beforeAll`/`afterAll`
+   * hook throw lands here, not on any individual `TestCase`. Optional
+   * because older duck-typed callers may not supply it.
+   */
+  errors?(): Array<VitestModuleError>;
 }
 /**
  * Duck-typed Vitest TestModule from the Reporter v2 API.
@@ -4564,6 +4604,42 @@ declare function classifyTest(current: "passed" | "failed", priorRuns: ReadonlyA
   state: string;
 }>): TestClassification;
 //#endregion
+//#region src/utils/coerce-error-text.d.ts
+/**
+ * Coerce an unknown error-field value (message, name, diff, stack, ...) into a
+ * string that is safe to bind to a SQLite TEXT column or render to a terminal.
+ *
+ * Vitest failure values are typed as strings but are whatever the test threw:
+ * `Effect.flip` on an unexpectedly-succeeding effect puts arbitrary success
+ * values (routinely plain objects) into the error channel, and Effect's
+ * `ConfigError.message` is a getter that throws when its cause lacks
+ * `toString`. Every access here is exception-safe.
+ *
+ * - `undefined` / `null` → `undefined` (caller decides null vs sentinel)
+ * - string → unchanged
+ * - other primitives → `String(value)`
+ * - objects → `JSON.stringify`, falling back to `String(value)`, falling back
+ *   to `"<unserializable>"` when both throw
+ *
+ * @public
+ */
+declare const coerceErrorText: (value: unknown) => string | undefined;
+/**
+ * Read a named field off an unknown error-like value and coerce it with
+ * {@link coerceErrorText}, guarding the property access itself. A live getter
+ * that throws (Effect's `ConfigError.message` when its cause lacks
+ * `toString`) throws at the read site — `coerceErrorText(e.message)` never
+ * reaches the helper's own exception handling. Route every read of an
+ * untrusted error object's field through this helper instead.
+ *
+ * - `null` / non-object sources → `undefined`
+ * - a getter that throws → `"<unreadable field>"`
+ * - otherwise → `coerceErrorText(value)`
+ *
+ * @public
+ */
+declare const coerceErrorField: (source: unknown, key: string) => string | undefined;
+//#endregion
 //#region src/utils/compress-lines.d.ts
 /**
  * Compress an array of line numbers into a compact range string.
@@ -5152,6 +5228,88 @@ declare const resolveWorkspaceKey: (projectDir: string) => Effect.Effect<string,
  */
 declare function safeFilename(name: string): string;
 //#endregion
+//#region src/utils/test-location.d.ts
+/** Directory holding source files whose tests may be co-located. @public */
+declare const SRC_DIR = "src";
+/** Canonical test directory name. @public */
+declare const TEST_DIR = "__test__";
+/** Support directories under `__test__/` that are never collected. @public */
+declare const TEST_HELPER_DIRS: readonly ["fixtures", "snapshots", "utils"];
+/** Glob suffix matching every discoverable test file. @public */
+declare const TEST_FILE_GLOB_SUFFIX = "*.{test,spec}.{ts,tsx,js,jsx}";
+/**
+ * Directory names discovery never walks into, and therefore never collects
+ * tests from. Keep in step with {@link TEST_FILE_GLOB_SUFFIX}'s extension set:
+ * this is the directory half of the same "what can Vitest reach" question.
+ *
+ * `@vitest-agent/plugin`'s test-file walker and its cache-signature walk both
+ * prune these before recursing (pruning first, rather than filtering results
+ * after, is what keeps a symlinked `node_modules` from dragging in the whole
+ * pnpm store), and {@link classifyTestPath} declines to render a verdict for
+ * any path that crosses one.
+ * @public
+ */
+declare const NON_DISCOVERABLE_DIRS: ReadonlySet<string>;
+/**
+ * True when `filePathOrName` ends in a discoverable test-file suffix — the
+ * extension half of the layout rule, matching {@link TEST_FILE_GLOB_SUFFIX}.
+ *
+ * Says nothing about location: a `node_modules` copy of `foo.test.ts` passes
+ * this and is still not discoverable. Pair it with {@link classifyTestPath}
+ * when the answer has to account for where the file sits.
+ * @public
+ */
+declare function isTestFileName(filePathOrName: string): boolean;
+/**
+ * Outcome of classifying a path against the test-layout rule.
+ *
+ * - `valid` — discoverable by Vitest.
+ * - `excluded` — deliberately not discovered, and correct where it is.
+ * - `invalid` — not a legal test location.
+ * @public
+ */
+type TestPathVerdict = "valid" | "excluded" | "invalid";
+/** Result of {@link classifyTestPath}. @public */
+interface TestPathClassification {
+  readonly verdict: TestPathVerdict;
+  readonly workspace: string | null;
+  readonly suggestedPath: string | null;
+}
+/** Minimal workspace shape {@link classifyTestPath} needs to attribute a path. @public */
+interface WorkspaceLike {
+  readonly name: string;
+  readonly path: string;
+}
+/**
+ * Returns the deepest supplied workspace containing `filePath`, or `null` when
+ * none does. Attribution is by longest matching path, so a nested package wins
+ * over the repository root that contains it.
+ *
+ * Exported because callers that need to reason about a path relative to its
+ * owning package — walking down from the package root, for instance — must
+ * agree with {@link classifyTestPath} about which package owns it.
+ * @public
+ */
+declare function findOwningWorkspace(workspaces: ReadonlyArray<WorkspaceLike>, filePath: string): WorkspaceLike | null;
+/**
+ * Classifies `filePath` against the test-layout rule: a test file is
+ * discoverable only at `<workspace>/src/**` or `<workspace>/__test__/**`,
+ * excluding the helper directories under `__test__/`.
+ *
+ * Returns `null` when no supplied workspace contains the path, and when the
+ * path crosses one of the {@link NON_DISCOVERABLE_DIRS} discovery never walks
+ * into. Neither is a verdict — it means the rule has nothing to say, and
+ * callers must fail open rather than treat it as invalid.
+ *
+ * The rule is pure, so one boundary the walker honors cannot be checked here:
+ * a nested `package.json` marks an independent unit whose tests belong to a
+ * different discovery pass, and detecting one needs a filesystem probe. A
+ * caller with filesystem access should apply that check itself and fail open
+ * when it fires — `@vitest-agent/cli`'s `agent check-test-path` does.
+ * @public
+ */
+declare function classifyTestPath(workspaces: ReadonlyArray<WorkspaceLike>, filePath: string): TestPathClassification | null;
+//#endregion
 //#region src/utils/validate-coverage-targets-shape.d.ts
 /** @public */
 interface CoverageTargetsShapeError {
@@ -5205,5 +5363,5 @@ declare function validateCoverageTargetsShape(input: unknown): CoverageTargetsSh
  */
 declare const CURRENT_SDK_VERSION: string;
 //#endregion
-export { AcceptanceMetrics, ActionSeverity, ActorType, Agent, AgentConsoleMode, AgentContext, AgentId, AgentNotFoundError, AgentPluginOptions, AgentReport, AgentReporterOptions, AgentTreeNode, AnsiOptions, ArtifactKind, AssembledHistoryRecord, AssociateRunSessionInput, BehaviorAbandonedEvent, BehaviorAddedEvent, BehaviorCompletedEvent, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorScopedEventType, BehaviorScopedEventTypes, BehaviorStartedEvent, BehaviorStatus, BehaviorsReadyEvent, BlockedEvent, CURRENT_SDK_VERSION, CacheManifest, CacheManifestEntry, CandidateContext, CellOptions, ChangeKind, ChannelEvent, ChatId, CiConsoleMode, CitedArtifact, CitedArtifactRow, CommitChangesEntry, ConfigLive, ConsoleFormatOptions, ConsoleLeakEntry, ConsoleLeakFile, ConsoleLeakTask, ConsoleLeaks, ConsoleLogLevel, ConsoleMode, ConsoleOutputMode, ConsoleOutputs, ConversationId, ConversationNotFoundError, CoverageBaselines, FileCoverageReport as CoverageFile, FileCoverageReport, CoverageGap, CoverageInput, CoverageLevel, CoverageLevelName, CoverageMetric, CoverageRenderState, CoverageReport, CoverageTargets, CoverageTargetsMetrics, CoverageTargetsShapeError, CoverageTargetsShapeInfo, CoverageTargetsShapeResult, CoverageTargetsShapeWarning, CoverageTotals, CreateBehaviorInput, CreateGoalInput, CurrentTddPhase, DATABASE_FILENAME, DataReader, DataReaderLive, DataStore, DataStoreError, DataStoreLive, DenialReason, DetailLevel, DetailResolver, DetailResolverLive, DiscoveryError, DiscoveryRegistry, DiscoveryRegistryLive, DispatchInputs, EndTddTaskInput, Environment, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, Executor, ExecutorResolver, ExecutorResolverLive, FailureRecord, FailureSignatureDetail, FailureSignatureInput, FailureSignatureWriteInput, FileCoverageInput, FileEditPayload, FileSystemAdapter, FlakyTest, FormatSelector, FormatSelectorLive, type FormatTriageOptions, type FormatWrapupOptions, Formatter, FormatterContext, FunctionBoundary, GfmFormatter, GitCommandError, GoalAbandonedEvent, GoalAddedEvent, GoalCompletedEvent, GoalDetail, GoalNotFoundError, GoalRow, GoalStartedEvent, GoalStatus, GoalsReadyEvent, HistoryRecord, HistoryRow, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, HistoryTrackerTestState, HookFirePayload, HookStatus, HookType, HostKind, HostMetadataParseError, HostMetadataResult, HumanConsoleMode, HypothesisDetail, HypothesisInput, HypothesisPayload, IDEMPOTENCY_ROOT_SENTINEL, IdempotencyHit, IdempotencyInput, IdempotentResponseInput, IllegalStatusTransitionEntity, IllegalStatusTransitionError, JsonFormatter, KnownProject, KnownProjectInput, LoggerLive, ManifestRow, MapSessionInput, MapSessionOutput, MarkdownFormatter, MetricThresholds, ModuleInput, ModuleListEntry, ModuleRecord, ModuleReport, NoteInput, NotePayload, NoteRow, Osc8Options, OutputFormat, OutputPipelineLive, OutputRenderer, OutputRendererLive, PackageManager, PathResolutionError, PathResolutionLive, PatternThresholds, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PersistentFailure, Phase, PhaseTransitionContext, PhaseTransitionEvent, PhaseTransitionResult, ProbeMissError, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityCandidate, ProjectIdentityCandidates, ProjectIdentityLive, ProjectIdentityNotResolvableError, ProjectIdentitySource, ProjectIdentityTest, ProjectKey, ProjectRunSummary, ProjectSummary, RegisterAgentInput, RegisterAgentResult, RegistrationConflictError, Remediation, RenderState, RenderTotals, RenderedOutput, ReportError, ReportSummary, ReporterKit, ReporterRenderInput, ResolveDataPathOptions, ResolveProjectIdentityOptions, ResolvedIdentity, ResolvedReporterConfig, ResolvedThresholds, RunChangedFile, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, RunEvent, RunEventByTag, RunHealth, RunInvocationMethod, RunOutcome, RunShape, SessionCompleteEvent, SessionContextRow, SessionDetail, SessionInput, SessionNotFoundError, SessionOutcome, SettingsInput, SettingsListEntry, SettingsRow, SidecarTimeoutError, SilentFormatter, StackFrameInput, SuggestedActionRecord, SuiteInput, SuiteListEntry, TagCountEntry, TagInventoryRow, TddArtifactDetail, TddArtifactRow, TddPhaseDetail, TddTaskAlreadyEndedError, TddTaskDetail, TddTaskEndOutcome, TddTaskId, TddTaskInput, TddTaskNotFoundError, TddTaskSummary, TerminalFormatter, TestCaseInput, TestClassification, TestError, TestErrorInput, TestFileEntry, TestHistory, TestListEntry, TestOutcome, TestRecord, TestReport, TestRun, TestRunInput, TestRunReason, TestState, ToolCallPayload, ToolResultPayload, Transport, TrendEntry, TrendRecord, TrendSummary, TurnInput, TurnPayload, TurnSearchOptions, TurnSummary, UpdateBehaviorInput, UpdateGoalInput, UserPromptPayload, ValidateHypothesisInput, VitestAgentConfig, VitestAgentConfigFile, VitestAgentConfigFileService, VitestAgentReporter, VitestAgentReporterFactory, VitestModuleDiagnostic, VitestModuleError, VitestParsedStack, VitestTestCase, VitestTestDiagnostic, VitestTestError, VitestTestModule, VitestTestResult, VitestTestSuite, type WrapupKind, WriteCommitInput, WriteRunChangedFilesInput, WriteTddArtifactInput, WriteTddPhaseInput, WriteTddPhaseOutput, _resetMigrationCacheForTesting, ansi, assembleHistoryRecord, assembleManifest, buildAgentReport, buildConsoleLeaks, buildEnvPrefix, canonicalizeGitUrl, ciAnnotationsFormatter, classifyTest, collectConsoleLeakEntries, compressLines, computeFailureSignature, computeTrend, countSuiteFailures, deriveIdempotencyKey, detectPackageManager, detectVitestScripts, determineTier, ensureMigrated, extractSqlReason, findFunctionBoundary, formatConsoleMarkdown, formatFatalError, formatGfm, formatTriageEffect, formatWrapupEffect, getRecentDirection, getRunCommand, getWorstMetric, gitUrlToProjectKey, hashTargets, historyKey, initialRenderState, isTimeoutError, isVitestInvocation, migration as migration0001, normalizeAssertionShape, normalizeWorkspaceKey, osc8, probeHostMetadataFromEnv, migration$1 as registryMigration0001, relativePath, requiredArtifactForTransition, resolveCoverageInput, resolveDataPath, resolveLogFile, resolveLogLevel, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveWorkspaceKey, rewriteBashCommand, safeFilename, migration$2 as sessionMapMigration0001, stripAnsi, transitionEnforcesBehaviorMatch, validateCoverageConfig, validateCoverageTargetsShape, validatePhaseTransition };
+export { AcceptanceMetrics, ActionSeverity, ActorType, Agent, AgentConsoleMode, AgentContext, AgentId, AgentNotFoundError, AgentPluginOptions, AgentReport, AgentReporterOptions, AgentTreeNode, AnsiOptions, ArtifactKind, AssembledHistoryRecord, AssociateRunSessionInput, BehaviorAbandonedEvent, BehaviorAddedEvent, BehaviorCompletedEvent, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorScopedEventType, BehaviorScopedEventTypes, BehaviorStartedEvent, BehaviorStatus, BehaviorsReadyEvent, BlockedEvent, CURRENT_SDK_VERSION, CacheManifest, CacheManifestEntry, CandidateContext, CellOptions, ChangeKind, ChannelEvent, ChatId, CiConsoleMode, CitedArtifact, CitedArtifactRow, CommitChangesEntry, ConfigLive, ConsoleFormatOptions, ConsoleLeakEntry, ConsoleLeakFile, ConsoleLeakTask, ConsoleLeaks, ConsoleLogLevel, ConsoleMode, ConsoleOutputMode, ConsoleOutputs, ConversationId, ConversationNotFoundError, CoverageBaselines, FileCoverageReport as CoverageFile, FileCoverageReport, CoverageGap, CoverageInput, CoverageLevel, CoverageLevelName, CoverageMetric, CoverageRenderState, CoverageReport, CoverageTargets, CoverageTargetsMetrics, CoverageTargetsShapeError, CoverageTargetsShapeInfo, CoverageTargetsShapeResult, CoverageTargetsShapeWarning, CoverageTotals, CreateBehaviorInput, CreateGoalInput, CurrentTddPhase, DATABASE_FILENAME, DataReader, DataReaderLive, DataStore, DataStoreError, DataStoreLive, DenialReason, DetailLevel, DetailResolver, DetailResolverLive, DiscoveryError, DiscoveryRegistry, DiscoveryRegistryLive, DispatchInputs, EndTddTaskInput, Environment, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, Executor, ExecutorResolver, ExecutorResolverLive, FailureRecord, FailureSignatureDetail, FailureSignatureInput, FailureSignatureWriteInput, FileCoverageInput, FileEditPayload, FileSystemAdapter, FlakyTest, FormatSelector, FormatSelectorLive, type FormatTriageOptions, type FormatWrapupOptions, Formatter, FormatterContext, FunctionBoundary, GfmFormatter, GitCommandError, GoalAbandonedEvent, GoalAddedEvent, GoalCompletedEvent, GoalDetail, GoalNotFoundError, GoalRow, GoalStartedEvent, GoalStatus, GoalsReadyEvent, HistoryRecord, HistoryRow, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, HistoryTrackerTestState, HookFirePayload, HookStatus, HookType, HostKind, HostMetadataParseError, HostMetadataResult, HumanConsoleMode, HypothesisDetail, HypothesisInput, HypothesisPayload, IDEMPOTENCY_ROOT_SENTINEL, IdempotencyHit, IdempotencyInput, IdempotentResponseInput, IllegalStatusTransitionEntity, IllegalStatusTransitionError, JsonFormatter, KnownProject, KnownProjectInput, LoggerLive, ManifestRow, MapSessionInput, MapSessionOutput, MarkdownFormatter, MetricThresholds, ModuleInput, ModuleListEntry, ModuleRecord, ModuleReport, NON_DISCOVERABLE_DIRS, NoteInput, NotePayload, NoteRow, Osc8Options, OutputFormat, OutputPipelineLive, OutputRenderer, OutputRendererLive, PackageManager, PathResolutionError, PathResolutionLive, PatternThresholds, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PersistentFailure, Phase, PhaseTransitionContext, PhaseTransitionEvent, PhaseTransitionResult, ProbeMissError, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityCandidate, ProjectIdentityCandidates, ProjectIdentityLive, ProjectIdentityNotResolvableError, ProjectIdentitySource, ProjectIdentityTest, ProjectKey, ProjectRunSummary, ProjectSummary, RegisterAgentInput, RegisterAgentResult, RegistrationConflictError, Remediation, RenderState, RenderTotals, RenderedOutput, ReportError, ReportSummary, ReporterKit, ReporterRenderInput, ResolveDataPathOptions, ResolveProjectIdentityOptions, ResolvedIdentity, ResolvedReporterConfig, ResolvedThresholds, RunChangedFile, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, RunEvent, RunEventByTag, RunHealth, RunInvocationMethod, RunOutcome, RunShape, SRC_DIR, SessionCompleteEvent, SessionContextRow, SessionDetail, SessionInput, SessionNotFoundError, SessionOutcome, SettingsInput, SettingsListEntry, SettingsRow, SidecarTimeoutError, SilentFormatter, StackFrameInput, SuggestedActionRecord, SuiteInput, SuiteListEntry, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, TagCountEntry, TagInventoryRow, TddArtifactDetail, TddArtifactRow, TddPhaseDetail, TddTaskAlreadyEndedError, TddTaskDetail, TddTaskEndOutcome, TddTaskId, TddTaskInput, TddTaskNotFoundError, TddTaskSummary, TerminalFormatter, TestCaseInput, TestClassification, TestError, TestErrorInput, TestFileEntry, TestHistory, TestListEntry, TestOutcome, TestPathClassification, TestPathVerdict, TestRecord, TestReport, TestRun, TestRunInput, TestRunReason, TestState, ToolCallPayload, ToolResultPayload, Transport, TrendEntry, TrendRecord, TrendSummary, TurnInput, TurnPayload, TurnSearchOptions, TurnSummary, UpdateBehaviorInput, UpdateGoalInput, UserPromptPayload, ValidateHypothesisInput, VitestAgentConfig, VitestAgentConfigFile, VitestAgentConfigFileService, VitestAgentReporter, VitestAgentReporterFactory, VitestModuleDiagnostic, VitestModuleError, VitestParsedStack, VitestTestCase, VitestTestDiagnostic, VitestTestError, VitestTestModule, VitestTestResult, VitestTestSuite, WorkspaceLike, type WrapupKind, WriteCommitInput, WriteRunChangedFilesInput, WriteTddArtifactInput, WriteTddPhaseInput, WriteTddPhaseOutput, _resetMigrationCacheForTesting, ansi, assembleHistoryRecord, assembleManifest, buildAgentReport, buildConsoleLeaks, buildEnvPrefix, canonicalizeGitUrl, ciAnnotationsFormatter, classifyTest, classifyTestPath, coerceErrorField, coerceErrorText, collectConsoleLeakEntries, compressLines, computeFailureSignature, computeTrend, countSuiteFailures, deriveIdempotencyKey, detectPackageManager, detectVitestScripts, determineTier, ensureMigrated, extractSqlReason, findFunctionBoundary, findOwningWorkspace, formatConsoleMarkdown, formatFatalError, formatGfm, formatTriageEffect, formatWrapupEffect, getRecentDirection, getRunCommand, getWorstMetric, gitUrlToProjectKey, hashTargets, historyKey, initialRenderState, isTestFileName, isTimeoutError, isVitestInvocation, migration as migration0001, normalizeAssertionShape, normalizeWorkspaceKey, osc8, probeHostMetadataFromEnv, migration$1 as registryMigration0001, relativePath, requiredArtifactForTransition, resolveCoverageInput, resolveDataPath, resolveLogFile, resolveLogLevel, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveWorkspaceKey, rewriteBashCommand, safeFilename, migration$2 as sessionMapMigration0001, stripAnsi, transitionEnforcesBehaviorMatch, validateCoverageConfig, validateCoverageTargetsShape, validatePhaseTransition };
 //# sourceMappingURL=index.d.ts.map

package/index.js

@@ -25,6 +25,7 @@ import { DataReaderLive } from "./layers/DataReaderLive.js";
 import { ActorType, AgentId, ChatId, ConversationId, HostKind, ProjectKey, TddTaskId } from "./schemas/Identity.js";
 import { Agent, IdempotencyHit } from "./schemas/Agent.js";
 import { DataStore } from "./services/DataStore.js";
+import { coerceErrorField, coerceErrorText } from "./utils/coerce-error-text.js";
 import { DataStoreLive } from "./layers/DataStoreLive.js";
 import { DetailResolver } from "./services/DetailResolver.js";
 import { DetailResolverLive } from "./layers/DetailResolverLive.js";
@@ -48,6 +49,7 @@ import { 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";
+import { NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, classifyTestPath, findOwningWorkspace, isTestFileName } from "./utils/test-location.js";
 import { ProjectDiscoveryLive } from "./layers/ProjectDiscoveryLive.js";
 import { ProjectDiscoveryTest } from "./layers/ProjectDiscoveryTest.js";
 import { canonicalizeGitUrl, gitUrlToProjectKey } from "./utils/canonicalize-git-url.js";
@@ -115,7 +117,7 @@ import { requiredArtifactForTransition, transitionEnforcesBehaviorMatch, validat
 * substituted yet. Exported for version introspection by downstream tooling.
 * @public
 */
-const CURRENT_SDK_VERSION = "2.0.16";
+const CURRENT_SDK_VERSION = "2.2.0";
 
 //#endregion
-export { ActionSeverity, ActorType, Agent, AgentConsoleMode, AgentContext, AgentId, AgentNotFoundError, AgentPluginOptions, AgentReport, AgentReporterOptions, BehaviorAbandonedEvent, BehaviorAddedEvent, BehaviorCompletedEvent, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorScopedEventTypes, BehaviorStartedEvent, BehaviorStatus, BehaviorsReadyEvent, BlockedEvent, CURRENT_SDK_VERSION, CacheManifest, CacheManifestEntry, ChannelEvent, ChatId, CiConsoleMode, ConfigLive, ConsoleLeakFile, ConsoleLeaks, ConsoleLogLevel, ConsoleMode, ConsoleOutputMode, ConsoleOutputs, ConversationId, ConversationNotFoundError, CoverageBaselines, FileCoverageReport as CoverageFile, CoverageGap, CoverageLevel, CoverageMetric, CoverageRenderState, CoverageReport, CoverageTargets, CoverageTargetsMetrics, CoverageTotals, DATABASE_FILENAME, DataReader, DataReaderLive, DataStore, DataStoreError, DataStoreLive, DetailLevel, DetailResolver, DetailResolverLive, DiscoveryError, DiscoveryRegistry, DiscoveryRegistryLive, Environment, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, Executor, ExecutorResolver, ExecutorResolverLive, FailureRecord, FileCoverageReport, FileEditPayload, FormatSelector, FormatSelectorLive, GfmFormatter, GitCommandError, GoalAbandonedEvent, GoalAddedEvent, GoalCompletedEvent, GoalDetail, GoalNotFoundError, GoalRow, GoalStartedEvent, GoalStatus, GoalsReadyEvent, HistoryRecord, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, HookFirePayload, HookStatus, HookType, HostKind, HostMetadataParseError, HumanConsoleMode, HypothesisPayload, IDEMPOTENCY_ROOT_SENTINEL, IdempotencyHit, IllegalStatusTransitionError, JsonFormatter, LoggerLive, MarkdownFormatter, MetricThresholds, ModuleRecord, ModuleReport, NotePayload, OutputFormat, OutputPipelineLive, OutputRenderer, OutputRendererLive, PackageManager, PathResolutionError, PathResolutionLive, PatternThresholds, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PhaseTransitionEvent, ProbeMissError, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityLive, ProjectIdentityNotResolvableError, ProjectIdentityTest, ProjectKey, RegistrationConflictError, RenderState, RenderTotals, ReportError, ReportSummary, ResolvedThresholds, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, RunEvent, SessionCompleteEvent, SessionNotFoundError, SessionOutcome, SidecarTimeoutError, SilentFormatter, SuggestedActionRecord, TagCountEntry, TddTaskAlreadyEndedError, TddTaskId, TddTaskNotFoundError, TerminalFormatter, TestClassification, TestHistory, TestRecord, TestReport, TestRun, TestRunReason, TestState, ToolCallPayload, ToolResultPayload, Transport, TrendEntry, TrendRecord, TurnPayload, UserPromptPayload, VitestAgentConfig, VitestAgentConfigFile, _resetMigrationCacheForTesting, ansi, assembleHistoryRecord, assembleManifest, buildAgentReport, buildConsoleLeaks, buildEnvPrefix, canonicalizeGitUrl, ciAnnotationsFormatter, classifyTest, collectConsoleLeakEntries, compressLines, computeFailureSignature, computeTrend, countSuiteFailures, deriveIdempotencyKey, detectPackageManager, detectVitestScripts, determineTier, ensureMigrated, extractSqlReason, findFunctionBoundary, formatConsoleMarkdown, formatFatalError, formatGfm, formatTriageEffect, formatWrapupEffect, getRecentDirection, getRunCommand, getWorstMetric, gitUrlToProjectKey, hashTargets, historyKey, initialRenderState, isTimeoutError, isVitestInvocation, migration as migration0001, normalizeAssertionShape, normalizeWorkspaceKey, osc8, probeHostMetadataFromEnv, migration$1 as registryMigration0001, relativePath, requiredArtifactForTransition, resolveCoverageInput, resolveDataPath, resolveLogFile, resolveLogLevel, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveWorkspaceKey, rewriteBashCommand, safeFilename, migration$2 as sessionMapMigration0001, stripAnsi, transitionEnforcesBehaviorMatch, validateCoverageConfig, validateCoverageTargetsShape, validatePhaseTransition };
+export { ActionSeverity, ActorType, Agent, AgentConsoleMode, AgentContext, AgentId, AgentNotFoundError, AgentPluginOptions, AgentReport, AgentReporterOptions, BehaviorAbandonedEvent, BehaviorAddedEvent, BehaviorCompletedEvent, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorScopedEventTypes, BehaviorStartedEvent, BehaviorStatus, BehaviorsReadyEvent, BlockedEvent, CURRENT_SDK_VERSION, CacheManifest, CacheManifestEntry, ChannelEvent, ChatId, CiConsoleMode, ConfigLive, ConsoleLeakFile, ConsoleLeaks, ConsoleLogLevel, ConsoleMode, ConsoleOutputMode, ConsoleOutputs, ConversationId, ConversationNotFoundError, CoverageBaselines, FileCoverageReport as CoverageFile, CoverageGap, CoverageLevel, CoverageMetric, CoverageRenderState, CoverageReport, CoverageTargets, CoverageTargetsMetrics, CoverageTotals, DATABASE_FILENAME, DataReader, DataReaderLive, DataStore, DataStoreError, DataStoreLive, DetailLevel, DetailResolver, DetailResolverLive, DiscoveryError, DiscoveryRegistry, DiscoveryRegistryLive, Environment, EnvironmentDetector, EnvironmentDetectorLive, EnvironmentDetectorTest, Executor, ExecutorResolver, ExecutorResolverLive, FailureRecord, FileCoverageReport, FileEditPayload, FormatSelector, FormatSelectorLive, GfmFormatter, GitCommandError, GoalAbandonedEvent, GoalAddedEvent, GoalCompletedEvent, GoalDetail, GoalNotFoundError, GoalRow, GoalStartedEvent, GoalStatus, GoalsReadyEvent, HistoryRecord, HistoryTracker, HistoryTrackerLive, HistoryTrackerTest, HookFirePayload, HookStatus, HookType, HostKind, HostMetadataParseError, HumanConsoleMode, HypothesisPayload, IDEMPOTENCY_ROOT_SENTINEL, IdempotencyHit, IllegalStatusTransitionError, JsonFormatter, LoggerLive, MarkdownFormatter, MetricThresholds, ModuleRecord, ModuleReport, NON_DISCOVERABLE_DIRS, NotePayload, OutputFormat, OutputPipelineLive, OutputRenderer, OutputRendererLive, PackageManager, PathResolutionError, PathResolutionLive, PatternThresholds, PerClientSessionMapReader, PerClientSessionMapReaderLive, PerClientSessionMapWriter, PerClientSessionMapWriterLive, PhaseTransitionEvent, ProbeMissError, ProjectDiscovery, ProjectDiscoveryLive, ProjectDiscoveryTest, ProjectIdentity, ProjectIdentityLive, ProjectIdentityNotResolvableError, ProjectIdentityTest, ProjectKey, RegistrationConflictError, RenderState, RenderTotals, ReportError, ReportSummary, ResolvedThresholds, RunContext, RunContext$, RunContextLive, RunContextService, RunContextTest, RunEvent, SRC_DIR, SessionCompleteEvent, SessionNotFoundError, SessionOutcome, SidecarTimeoutError, SilentFormatter, SuggestedActionRecord, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, TagCountEntry, TddTaskAlreadyEndedError, TddTaskId, TddTaskNotFoundError, TerminalFormatter, TestClassification, TestHistory, TestRecord, TestReport, TestRun, TestRunReason, TestState, ToolCallPayload, ToolResultPayload, Transport, TrendEntry, TrendRecord, TurnPayload, UserPromptPayload, VitestAgentConfig, VitestAgentConfigFile, _resetMigrationCacheForTesting, ansi, assembleHistoryRecord, assembleManifest, buildAgentReport, buildConsoleLeaks, buildEnvPrefix, canonicalizeGitUrl, ciAnnotationsFormatter, classifyTest, classifyTestPath, coerceErrorField, coerceErrorText, collectConsoleLeakEntries, compressLines, computeFailureSignature, computeTrend, countSuiteFailures, deriveIdempotencyKey, detectPackageManager, detectVitestScripts, determineTier, ensureMigrated, extractSqlReason, findFunctionBoundary, findOwningWorkspace, formatConsoleMarkdown, formatFatalError, formatGfm, formatTriageEffect, formatWrapupEffect, getRecentDirection, getRunCommand, getWorstMetric, gitUrlToProjectKey, hashTargets, historyKey, initialRenderState, isTestFileName, isTimeoutError, isVitestInvocation, migration as migration0001, normalizeAssertionShape, normalizeWorkspaceKey, osc8, probeHostMetadataFromEnv, migration$1 as registryMigration0001, relativePath, requiredArtifactForTransition, resolveCoverageInput, resolveDataPath, resolveLogFile, resolveLogLevel, resolveProjectIdentityFromCandidates, resolveProjectKeyFromCwd, resolveWorkspaceKey, rewriteBashCommand, safeFilename, migration$2 as sessionMapMigration0001, stripAnsi, transitionEnforcesBehaviorMatch, validateCoverageConfig, validateCoverageTargetsShape, validatePhaseTransition };

package/layers/DataStoreLive.js

@@ -3,6 +3,7 @@ import { DataStoreError, extractSqlReason } from "../errors/DataStoreError.js";
 import { BehaviorNotFoundError, GoalNotFoundError, IllegalStatusTransitionError, TddTaskAlreadyEndedError, TddTaskNotFoundError } from "../errors/TddErrors.js";
 import { Agent, IdempotencyHit } from "../schemas/Agent.js";
 import { DataStore } from "../services/DataStore.js";
+import { coerceErrorText } from "../utils/coerce-error-text.js";
 import { Effect, Layer, Option } from "effect";
 import { SqlClient } from "effect/unstable/sql/SqlClient";
 import { randomUUID } from "node:crypto";
@@ -134,16 +135,18 @@ const DataStoreLive = Layer.effect(DataStore, Effect.gen(function* () {
 			count: errors.length
 		}));
 		for (const err of errors) {
-			yield* sql`INSERT INTO test_errors (run_id, test_case_id, test_suite_id, module_id, scope, name, message, diff, actual, expected, stack, cause_error_id, signature_hash, ordinal) VALUES (${runId}, ${err.testCaseId ?? null}, ${err.testSuiteId ?? null}, ${err.moduleId ?? null}, ${err.scope}, ${err.name ?? null}, ${err.message}, ${err.diff ?? null}, ${err.actual ?? null}, ${err.expected ?? null}, ${err.stack ?? null}, ${err.causeErrorId ?? null}, ${err.signatureHash ?? null}, ${err.ordinal ?? 0})`;
+			const message = coerceErrorText(err.message) ?? "<missing message>";
+			yield* sql`INSERT INTO test_errors (run_id, test_case_id, test_suite_id, module_id, scope, name, message, diff, actual, expected, stack, cause_error_id, signature_hash, ordinal) VALUES (${runId}, ${err.testCaseId ?? null}, ${err.testSuiteId ?? null}, ${err.moduleId ?? null}, ${err.scope}, ${coerceErrorText(err.name) ?? null}, ${message}, ${coerceErrorText(err.diff) ?? null}, ${coerceErrorText(err.actual) ?? null}, ${coerceErrorText(err.expected) ?? null}, ${coerceErrorText(err.stack) ?? null}, ${err.causeErrorId ?? null}, ${err.signatureHash ?? null}, ${err.ordinal ?? 0})`;
+			const stackText = coerceErrorText(err.stack);
 			if (err.frames && err.frames.length > 0) {
 				const errorId = (yield* sql`SELECT last_insert_rowid() as id`)[0].id;
 				for (const frame of err.frames) {
 					const fileId = yield* ensureFile(frame.filePath);
 					yield* sql`INSERT INTO stack_frames (error_id, ordinal, method, file_id, line, col, source_mapped_line, function_boundary_line) VALUES (${errorId}, ${frame.ordinal}, ${frame.method}, ${fileId}, ${frame.line}, ${frame.col}, ${frame.sourceMappedLine ?? null}, ${frame.functionBoundaryLine ?? null})`;
 				}
-			} else if (err.stack) {
+			} else if (stackText) {
 				const errorId = (yield* sql`SELECT last_insert_rowid() as id`)[0].id;
-				const frames = [...err.stack.matchAll(/at\s+(?:(.+?)\s+)?\(?(.+?):(\d+):(\d+)\)?/g)];
+				const frames = [...stackText.matchAll(/at\s+(?:(.+?)\s+)?\(?(.+?):(\d+):(\d+)\)?/g)];
 				for (let frameOrdinal = 0; frameOrdinal < frames.length; frameOrdinal++) {
 					const m = frames[frameOrdinal];
 					const method = m[1] ?? null;

package/layers/ProjectDiscoveryLive.js

@@ -1,13 +1,12 @@
 import { DiscoveryError } from "../errors/DiscoveryError.js";
 import { ProjectDiscovery } from "../services/ProjectDiscovery.js";
+import { NON_DISCOVERABLE_DIRS } from "../utils/test-location.js";
 import { Effect, FileSystem, Layer } from "effect";
 
 //#region src/layers/ProjectDiscoveryLive.ts
 const TEST_FILE_PATTERN = /\.(test|spec)\.(ts|tsx)$/;
 const SKIP_DIRS = /* @__PURE__ */ new Set([
-	"node_modules",
-	".git",
-	"dist",
+	...NON_DISCOVERABLE_DIRS,
 	"coverage",
 	".turbo",
 	".vite"

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitest-agent/sdk",
-	"version": "2.0.16",
+	"version": "2.2.0",
 	"private": false,
 	"description": "Shared Effect Schema, SQLite migrations, data layer, services, formatters, and utilities for vitest-agent packages.",
 	"keywords": [

package/schemas/AgentReport.js

@@ -13,7 +13,15 @@ const ReportSummary = Schema.Struct({
 	passed: Schema.Number,
 	failed: Schema.Number,
 	skipped: Schema.Number,
-	duration: Schema.Number
+	duration: Schema.Number,
+	/**
+	* Count of every collected module, passing modules included.
+	* `AgentReport.failed` only carries failing modules, so a green run
+	* has no other way to know how many files actually ran — without
+	* this a fully-passing replay reads as "0 modules all-passed"
+	* (issue #204). Optional to keep existing fixtures/reports valid.
+	*/
+	modules: Schema.optional(Schema.Number)
 }).annotate({ identifier: "ReportSummary" });
 /**
 * An individual test case result.

package/schemas/RenderState.js

@@ -135,7 +135,17 @@ const RenderState = Schema.Struct({
 		runCount: Schema.Number
 	})),
 	failures: Schema.Array(FailureRecord),
-	suggestedActions: Schema.Array(SuggestedActionRecord)
+	suggestedActions: Schema.Array(SuggestedActionRecord),
+	/**
+	* Count of every collected module, passing modules included. Folded
+	* from `RunFinished.collectedModules`. Renderers prefer this over
+	* `moduleOrder.length` for "N modules all-passed" copy — a report
+	* replay only queues failing modules, so `moduleOrder` alone
+	* undercounts a green run to zero (issue #204). Optional — a run
+	* that never carried the field (older replay data, hand-built
+	* fixtures) falls back to `moduleOrder.length`.
+	*/
+	collectedModules: Schema.optional(Schema.Number)
 }).annotate({ identifier: "RenderState" });
 /**
 * Initial state — what the reducer returns when no events have been

package/schemas/RunEvent.js

@@ -194,7 +194,15 @@ const RunEvent = Schema.Union([
 		failCount: Schema.Number,
 		skipCount: Schema.Number,
 		durationMs: Schema.Number,
-		timeoutCount: Schema.optional(Schema.Number)
+		timeoutCount: Schema.optional(Schema.Number),
+		/**
+		* Count of every collected module, passing modules included.
+		* Carries `AgentReport.summary.modules` (or the live module
+		* count) into the event stream so the reducer can fold the
+		* true count into `RenderState` even when `moduleOrder` only
+		* tracks failing modules (report replay). See issue #204.
+		*/
+		collectedModules: Schema.optional(Schema.Number)
 	})
 ]).annotate({ identifier: "RunEvent" });
 

package/testing.d.ts

@@ -36,6 +36,14 @@ declare const AgentReport: Schema.Struct<{
     readonly failed: Schema.Number;
     readonly skipped: Schema.Number;
     readonly duration: Schema.Number;
+    /**
+     * Count of every collected module, passing modules included.
+     * `AgentReport.failed` only carries failing modules, so a green run
+     * has no other way to know how many files actually ran — without
+     * this a fully-passing replay reads as "0 modules all-passed"
+     * (issue #204). Optional to keep existing fixtures/reports valid.
+     */
+    readonly modules: Schema.optional<Schema.Number>;
   }>;
   readonly failed: Schema.$Array<Schema.Struct<{
     readonly file: Schema.String;

package/utils/build-report.js

@@ -1,3 +1,5 @@
+import { coerceErrorField } from "./coerce-error-text.js";
+
 //#region src/utils/build-report.ts
 /**
 * Format a single Vitest stack frame: strings pass through; ParsedStack
@@ -18,10 +20,23 @@ function formatStackFrame(frame) {
 function mapErrors(errors) {
 	if (!errors || errors.length === 0) return void 0;
 	return errors.map((e) => {
+		const diffText = coerceErrorField(e, "diff");
+		let stacks;
+		try {
+			stacks = e.stacks;
+		} catch {
+			stacks = void 0;
+		}
+		let stackText;
+		try {
+			stackText = stacks && stacks.length > 0 ? stacks.map(formatStackFrame).join("\n") : void 0;
+		} catch {
+			stackText = void 0;
+		}
 		return {
-			message: e.message,
-			..."diff" in e && e.diff != null ? { diff: e.diff } : {},
-			...e.stacks && e.stacks.length > 0 ? { stack: e.stacks.map(formatStackFrame).join("\n") } : {}
+			message: coerceErrorField(e, "message") ?? "<missing message>",
+			...diffText !== void 0 ? { diff: diffText } : {},
+			...stackText !== void 0 ? { stack: stackText } : {}
 		};
 	});
 }
@@ -68,6 +83,13 @@ function buildAgentReport(testModules, unhandledErrors, reason, options, project
 		totalDuration += moduleDuration;
 		const moduleState = normalizeState(testModule.state());
 		const moduleErrors = mapErrors(testModule.errors());
+		const suiteErrors = [];
+		let anySuiteFailed = false;
+		for (const suite of testModule.children.allSuites()) {
+			if (normalizeState(suite.state()) === "failed") anySuiteFailed = true;
+			const mapped = mapErrors(suite.errors?.());
+			if (mapped) suiteErrors.push(...mapped);
+		}
 		const allTests = [...testModule.children.allTests()];
 		const testReports = [];
 		let moduleHasFailure = false;
@@ -96,29 +118,33 @@ function buildAgentReport(testModules, unhandledErrors, reason, options, project
 				testReports.push(testReport);
 			}
 		}
-		if (moduleHasFailure || !moduleHasFailure && moduleState === "failed" && moduleErrors !== void 0) {
+		const combinedErrors = [...moduleErrors ?? [], ...suiteErrors];
+		const moduleLevelFailure = moduleState === "failed" || anySuiteFailed || combinedErrors.length > 0;
+		if (moduleHasFailure || moduleLevelFailure) {
 			failedFiles.push(testModule.relativeModuleId);
 			const moduleReport = {
 				file: testModule.relativeModuleId,
-				state: moduleState,
+				state: moduleLevelFailure ? "failed" : moduleState,
 				duration: moduleDuration,
 				tests: testReports,
-				...moduleErrors ? { errors: moduleErrors } : {}
+				...combinedErrors.length > 0 ? { errors: combinedErrors } : {}
 			};
 			failedModules.push(moduleReport);
 		}
 	}
 	const mappedUnhandled = mapErrors(unhandledErrors) ?? [];
+	const effectiveReason = reason === "passed" && (failedFiles.length > 0 || mappedUnhandled.length > 0) ? "failed" : reason;
 	return {
 		timestamp: (/* @__PURE__ */ new Date()).toISOString(),
 		...projectName != null ? { project: projectName } : {},
-		reason,
+		reason: effectiveReason,
 		summary: {
 			total: totalCount,
 			passed: passedCount,
 			failed: failedCount,
 			skipped: skippedCount,
-			duration: totalDuration
+			duration: totalDuration,
+			modules: testModules.length
 		},
 		failed: failedModules,
 		unhandledErrors: mappedUnhandled,

package/utils/coerce-error-text.js

@@ -0,0 +1,60 @@
+//#region src/utils/coerce-error-text.ts
+/**
+* Coerce an unknown error-field value (message, name, diff, stack, ...) into a
+* string that is safe to bind to a SQLite TEXT column or render to a terminal.
+*
+* Vitest failure values are typed as strings but are whatever the test threw:
+* `Effect.flip` on an unexpectedly-succeeding effect puts arbitrary success
+* values (routinely plain objects) into the error channel, and Effect's
+* `ConfigError.message` is a getter that throws when its cause lacks
+* `toString`. Every access here is exception-safe.
+*
+* - `undefined` / `null` → `undefined` (caller decides null vs sentinel)
+* - string → unchanged
+* - other primitives → `String(value)`
+* - objects → `JSON.stringify`, falling back to `String(value)`, falling back
+*   to `"<unserializable>"` when both throw
+*
+* @public
+*/
+const coerceErrorText = (value) => {
+	if (value === void 0 || value === null) return void 0;
+	if (typeof value === "string") return value;
+	if (typeof value === "number" || typeof value === "boolean" || typeof value === "bigint") return String(value);
+	try {
+		return JSON.stringify(value) ?? "<unserializable>";
+	} catch {
+		try {
+			return String(value);
+		} catch {
+			return "<unserializable>";
+		}
+	}
+};
+/**
+* Read a named field off an unknown error-like value and coerce it with
+* {@link coerceErrorText}, guarding the property access itself. A live getter
+* that throws (Effect's `ConfigError.message` when its cause lacks
+* `toString`) throws at the read site — `coerceErrorText(e.message)` never
+* reaches the helper's own exception handling. Route every read of an
+* untrusted error object's field through this helper instead.
+*
+* - `null` / non-object sources → `undefined`
+* - a getter that throws → `"<unreadable field>"`
+* - otherwise → `coerceErrorText(value)`
+*
+* @public
+*/
+const coerceErrorField = (source, key) => {
+	if (source === null || typeof source !== "object" && typeof source !== "function") return void 0;
+	let value;
+	try {
+		value = source[key];
+	} catch {
+		return "<unreadable field>";
+	}
+	return coerceErrorText(value);
+};
+
+//#endregion
+export { coerceErrorField, coerceErrorText };

package/utils/failure-signature.js

@@ -15,6 +15,7 @@ const typeTagOf = (literal) => {
 };
 /** @public */
 const normalizeAssertionShape = (message) => {
+	if (typeof message !== "string") return "";
 	const match = message.match(ASSERTION_REGEX);
 	if (match === null) return null;
 	const matcher = match[1];

package/utils/format-fatal-error.js

@@ -21,11 +21,23 @@ function formatFatalError(err) {
 		const cause = err[FIBER_FAILURE_CAUSE_KEY];
 		if (Cause.isCause(cause)) detail = Cause.pretty(cause);
 		else detail = String(err);
-	} else if (err instanceof Error) detail = err.stack ?? err.message;
+	} else if (err instanceof Error) try {
+		detail = err.stack ?? err.message;
+	} catch {
+		let ctorName = "Error";
+		try {
+			ctorName = err.constructor?.name ?? "Error";
+		} catch {}
+		detail = `<unserializable ${ctorName}>`;
+	}
 	else if (err !== null && typeof err === "object") try {
 		detail = JSON.stringify(err, null, 2);
 	} catch {
-		detail = String(err);
+		try {
+			detail = String(err);
+		} catch {
+			detail = "<unserializable error>";
+		}
 	}
 	else detail = String(err);
 	return `${detail}\nPlease report at ${ISSUE_URL}`;

package/utils/test-location.js

@@ -0,0 +1,114 @@
+import { basename, join, relative, sep } from "node:path";
+
+//#region src/utils/test-location.ts
+/** Directory holding source files whose tests may be co-located. @public */
+const SRC_DIR = "src";
+/** Canonical test directory name. @public */
+const TEST_DIR = "__test__";
+/** Support directories under `__test__/` that are never collected. @public */
+const TEST_HELPER_DIRS = [
+	"fixtures",
+	"snapshots",
+	"utils"
+];
+/** Glob suffix matching every discoverable test file. @public */
+const TEST_FILE_GLOB_SUFFIX = "*.{test,spec}.{ts,tsx,js,jsx}";
+/**
+* Directory names discovery never walks into, and therefore never collects
+* tests from. Keep in step with {@link TEST_FILE_GLOB_SUFFIX}'s extension set:
+* this is the directory half of the same "what can Vitest reach" question.
+*
+* `@vitest-agent/plugin`'s test-file walker and its cache-signature walk both
+* prune these before recursing (pruning first, rather than filtering results
+* after, is what keeps a symlinked `node_modules` from dragging in the whole
+* pnpm store), and {@link classifyTestPath} declines to render a verdict for
+* any path that crosses one.
+* @public
+*/
+const NON_DISCOVERABLE_DIRS = /* @__PURE__ */ new Set([
+	"node_modules",
+	".git",
+	"dist"
+]);
+/**
+* Regex form of {@link TEST_FILE_GLOB_SUFFIX}. Anchored at the end of the
+* string, so it matches a bare basename and a full path alike. Change both
+* together — they are the same extension set in two notations.
+*/
+const TEST_FILE_SUFFIX_RE = /\.(?:test|spec)\.(?:ts|tsx|js|jsx)$/;
+/**
+* True when `filePathOrName` ends in a discoverable test-file suffix — the
+* extension half of the layout rule, matching {@link TEST_FILE_GLOB_SUFFIX}.
+*
+* Says nothing about location: a `node_modules` copy of `foo.test.ts` passes
+* this and is still not discoverable. Pair it with {@link classifyTestPath}
+* when the answer has to account for where the file sits.
+* @public
+*/
+function isTestFileName(filePathOrName) {
+	return TEST_FILE_SUFFIX_RE.test(filePathOrName);
+}
+/** True when `child` is `parent` or sits beneath it on a segment boundary. */
+function contains(parent, child) {
+	return child === parent || child.startsWith(parent.endsWith(sep) ? parent : `${parent}${sep}`);
+}
+/**
+* Returns the deepest supplied workspace containing `filePath`, or `null` when
+* none does. Attribution is by longest matching path, so a nested package wins
+* over the repository root that contains it.
+*
+* Exported because callers that need to reason about a path relative to its
+* owning package — walking down from the package root, for instance — must
+* agree with {@link classifyTestPath} about which package owns it.
+* @public
+*/
+function findOwningWorkspace(workspaces, filePath) {
+	let owner = null;
+	for (const ws of workspaces) {
+		if (!contains(ws.path, filePath)) continue;
+		if (owner === null || ws.path.length > owner.path.length) owner = ws;
+	}
+	return owner;
+}
+/**
+* Classifies `filePath` against the test-layout rule: a test file is
+* discoverable only at `<workspace>/src/**` or `<workspace>/__test__/**`,
+* excluding the helper directories under `__test__/`.
+*
+* Returns `null` when no supplied workspace contains the path, and when the
+* path crosses one of the {@link NON_DISCOVERABLE_DIRS} discovery never walks
+* into. Neither is a verdict — it means the rule has nothing to say, and
+* callers must fail open rather than treat it as invalid.
+*
+* The rule is pure, so one boundary the walker honors cannot be checked here:
+* a nested `package.json` marks an independent unit whose tests belong to a
+* different discovery pass, and detecting one needs a filesystem probe. A
+* caller with filesystem access should apply that check itself and fail open
+* when it fires — `@vitest-agent/cli`'s `agent check-test-path` does.
+* @public
+*/
+function classifyTestPath(workspaces, filePath) {
+	const owner = findOwningWorkspace(workspaces, filePath);
+	if (owner === null) return null;
+	const segments = relative(owner.path, filePath).split(sep);
+	if (segments.some((s) => NON_DISCOVERABLE_DIRS.has(s))) return null;
+	const [head] = segments;
+	if (head === "src") return {
+		verdict: "valid",
+		workspace: owner.name,
+		suggestedPath: null
+	};
+	if (head === "__test__") return {
+		verdict: segments.slice(1, -1).some((s) => TEST_HELPER_DIRS.includes(s)) ? "excluded" : "valid",
+		workspace: owner.name,
+		suggestedPath: null
+	};
+	return {
+		verdict: "invalid",
+		workspace: owner.name,
+		suggestedPath: join(owner.path, TEST_DIR, basename(filePath))
+	};
+}
+
+//#endregion
+export { NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, classifyTestPath, findOwningWorkspace, isTestFileName };