@vitest-agent/sdk 2.5.1 → 3.1.0

package/services/DataStore.js

@@ -1,8 +1,15 @@
 import { Context } from "effect";
 
 //#region src/services/DataStore.ts
+/**
+* Ceiling on an inline attachment body. Vitest has already copied file
+* attachments into `.vitest/attachments/`, so a large body is referenced
+* by `path` rather than duplicated into `data.db`.
+* @public
+*/
+const INLINE_ATTACHMENT_BODY_CAP_BYTES = 65536;
 /** @public */
 var DataStore = class extends Context.Service()("vitest-agent/DataStore") {};
 
 //#endregion
-export { DataStore };
+export { DataStore, INLINE_ATTACHMENT_BODY_CAP_BYTES };

package/testing/layers.js

@@ -1,6 +1,6 @@
 import { DataReaderLive } from "../layers/DataReaderLive.js";
 import { DataStoreLive } from "../layers/DataStoreLive.js";
-import migration from "../migrations/0001_initial.js";
+import { PROJECT_MIGRATIONS } from "../migrations/index.js";
 import { Layer } from "effect";
 import * as NodeServices from "@effect/platform-node/NodeServices";
 import { layer } from "@effect/sql-sqlite-node/SqliteClient";
@@ -11,7 +11,7 @@ import * as SqliteMigrator from "@effect/sql-sqlite-node/SqliteMigrator";
 function makeTestLayer(filename) {
 	const SqliteLayer = layer({ filename });
 	const PlatformLayer = NodeServices.layer;
-	const MigratorLayer = SqliteMigrator.layer({ loader: SqliteMigrator.fromRecord({ "0001_initial": migration }) }).pipe(Layer.provide(Layer.merge(SqliteLayer, PlatformLayer)));
+	const MigratorLayer = SqliteMigrator.layer({ loader: SqliteMigrator.fromRecord(PROJECT_MIGRATIONS) }).pipe(Layer.provide(Layer.merge(SqliteLayer, PlatformLayer)));
 	return Layer.mergeAll(DataStoreLive.pipe(Layer.provide(SqliteLayer)), DataReaderLive.pipe(Layer.provide(SqliteLayer)), MigratorLayer, SqliteLayer, PlatformLayer);
 }
 /** @public */

package/testing.d.ts

@@ -100,6 +100,12 @@ export declare const AgentReport: Schema.Struct<{
         readonly functions: Schema.optional<Schema.Number>;
         readonly branches: Schema.optional<Schema.Number>;
         readonly statements: Schema.optional<Schema.Number>;
+        readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+          readonly lines: Schema.optional<Schema.Number>;
+          readonly functions: Schema.optional<Schema.Number>;
+          readonly branches: Schema.optional<Schema.Number>;
+          readonly statements: Schema.optional<Schema.Number>;
+        }>]>>;
       }>]>>, never>;
     }>;
     readonly targets: Schema.optional<Schema.Struct<{
@@ -114,6 +120,12 @@ export declare const AgentReport: Schema.Struct<{
         readonly functions: Schema.optional<Schema.Number>;
         readonly branches: Schema.optional<Schema.Number>;
         readonly statements: Schema.optional<Schema.Number>;
+        readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+          readonly lines: Schema.optional<Schema.Number>;
+          readonly functions: Schema.optional<Schema.Number>;
+          readonly branches: Schema.optional<Schema.Number>;
+          readonly statements: Schema.optional<Schema.Number>;
+        }>]>>;
       }>]>>, never>;
     }>>;
     readonly baselines: Schema.optional<Schema.Struct<{
@@ -128,6 +140,12 @@ export declare const AgentReport: Schema.Struct<{
         readonly functions: Schema.optional<Schema.Number>;
         readonly branches: Schema.optional<Schema.Number>;
         readonly statements: Schema.optional<Schema.Number>;
+        readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+          readonly lines: Schema.optional<Schema.Number>;
+          readonly functions: Schema.optional<Schema.Number>;
+          readonly branches: Schema.optional<Schema.Number>;
+          readonly statements: Schema.optional<Schema.Number>;
+        }>]>>;
       }>]>>, never>;
     }>>;
     readonly scoped: Schema.withDecodingDefaultKey<Schema.Boolean, never>;
@@ -198,6 +216,12 @@ export declare const CoverageBaselines: Schema.Struct<{
     readonly functions: Schema.optional<Schema.Number>;
     readonly branches: Schema.optional<Schema.Number>;
     readonly statements: Schema.optional<Schema.Number>;
+    readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+      readonly lines: Schema.optional<Schema.Number>;
+      readonly functions: Schema.optional<Schema.Number>;
+      readonly branches: Schema.optional<Schema.Number>;
+      readonly statements: Schema.optional<Schema.Number>;
+    }>]>>;
   }>]>>, never>;
 }>;
 /** @public */
@@ -262,6 +286,12 @@ export declare const CoverageReport: Schema.Struct<{
       readonly functions: Schema.optional<Schema.Number>;
       readonly branches: Schema.optional<Schema.Number>;
       readonly statements: Schema.optional<Schema.Number>;
+      readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+        readonly lines: Schema.optional<Schema.Number>;
+        readonly functions: Schema.optional<Schema.Number>;
+        readonly branches: Schema.optional<Schema.Number>;
+        readonly statements: Schema.optional<Schema.Number>;
+      }>]>>;
     }>]>>, never>;
   }>;
   readonly targets: Schema.optional<Schema.Struct<{
@@ -276,6 +306,12 @@ export declare const CoverageReport: Schema.Struct<{
       readonly functions: Schema.optional<Schema.Number>;
       readonly branches: Schema.optional<Schema.Number>;
       readonly statements: Schema.optional<Schema.Number>;
+      readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+        readonly lines: Schema.optional<Schema.Number>;
+        readonly functions: Schema.optional<Schema.Number>;
+        readonly branches: Schema.optional<Schema.Number>;
+        readonly statements: Schema.optional<Schema.Number>;
+      }>]>>;
     }>]>>, never>;
   }>>;
   readonly baselines: Schema.optional<Schema.Struct<{
@@ -290,6 +326,12 @@ export declare const CoverageReport: Schema.Struct<{
       readonly functions: Schema.optional<Schema.Number>;
       readonly branches: Schema.optional<Schema.Number>;
       readonly statements: Schema.optional<Schema.Number>;
+      readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+        readonly lines: Schema.optional<Schema.Number>;
+        readonly functions: Schema.optional<Schema.Number>;
+        readonly branches: Schema.optional<Schema.Number>;
+        readonly statements: Schema.optional<Schema.Number>;
+      }>]>>;
     }>]>>, never>;
   }>>;
   readonly scoped: Schema.withDecodingDefaultKey<Schema.Boolean, never>;
@@ -698,12 +740,23 @@ declare const ResolvedThresholds: Schema.Struct<{
     readonly branches: Schema.optional<Schema.Number>;
     readonly statements: Schema.optional<Schema.Number>;
   }>;
-  readonly perFile: Schema.withDecodingDefaultKey<Schema.Boolean, never>;
+  readonly perFile: Schema.withDecodingDefaultKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+    readonly lines: Schema.optional<Schema.Number>;
+    readonly functions: Schema.optional<Schema.Number>;
+    readonly branches: Schema.optional<Schema.Number>;
+    readonly statements: Schema.optional<Schema.Number>;
+  }>]>, never>;
   readonly patterns: Schema.withDecodingDefaultKey<Schema.$Array<Schema.Tuple<readonly [Schema.String, Schema.Struct<{
     readonly lines: Schema.optional<Schema.Number>;
     readonly functions: Schema.optional<Schema.Number>;
     readonly branches: Schema.optional<Schema.Number>;
     readonly statements: Schema.optional<Schema.Number>;
+    readonly perFile: Schema.optionalKey<Schema.Union<readonly [Schema.Boolean, Schema.Struct<{
+      readonly lines: Schema.optional<Schema.Number>;
+      readonly functions: Schema.optional<Schema.Number>;
+      readonly branches: Schema.optional<Schema.Number>;
+      readonly statements: Schema.optional<Schema.Number>;
+    }>]>>;
   }>]>>, never>;
 }>;
 /** @public */
@@ -877,6 +930,41 @@ interface TestErrorInput {
   readonly ordinal?: number;
 }
 /** @public */
+interface TestAttachmentInput {
+  readonly contentType?: string;
+  /** Post-resolution path under `.vitest/attachments/`, or an http(s) URL. */
+  readonly path?: string;
+  /** Inline body; persisted only when `byteSize` is under the 64 KiB cap. */
+  readonly body?: string;
+  /** How to read `body`. Vitest treats a string body as base64 unless `"utf-8"`. */
+  readonly bodyEncoding?: "base64" | "utf-8";
+  readonly byteSize: number;
+}
+/** @public */
+interface TestAnnotationInput {
+  readonly testCaseId: number;
+  /** Arbitrary Vitest annotation type -- not an enum. */
+  readonly type: string;
+  readonly message: string;
+  readonly locationFile?: string;
+  readonly locationLine?: number;
+  readonly locationColumn?: number;
+  readonly attachments?: ReadonlyArray<TestAttachmentInput>;
+}
+/** @public */
+interface TestArtifactInput {
+  readonly testCaseId: number;
+  /** `pkg:name`; `internal:` types are Vitest's own and are never written. */
+  readonly type: string;
+  readonly message?: string;
+  /** JSON of the artifact's custom fields, minus attachments and location. */
+  readonly data?: string;
+  readonly locationFile?: string;
+  readonly locationLine?: number;
+  readonly locationColumn?: number;
+  readonly attachments?: ReadonlyArray<TestAttachmentInput>;
+}
+/** @public */
 interface FileCoverageInput {
   readonly fileId: number;
   readonly statements: number;
@@ -1090,6 +1178,17 @@ declare const DataStore_base: Context.ServiceClass<DataStore, "vitest-agent/Data
   readonly writeSuites: (moduleId: number, suites: ReadonlyArray<SuiteInput>) => Effect.Effect<ReadonlyArray<number>, DataStoreError>;
   readonly writeTestCases: (moduleId: number, tests: ReadonlyArray<TestCaseInput>) => Effect.Effect<ReadonlyArray<number>, DataStoreError>;
   readonly writeErrors: (runId: number, errors: ReadonlyArray<TestErrorInput>) => Effect.Effect<void, DataStoreError>;
+  /**
+   * Persist `context.annotate` notes and their attachment descriptors.
+   * `runId` is carried for log correlation only -- `test_annotations`
+   * rows are reachable through `test_cases`.
+   */
+  readonly writeAnnotations: (runId: number, annotations: ReadonlyArray<TestAnnotationInput>) => Effect.Effect<void, DataStoreError>;
+  /**
+   * Persist `recordArtifact` payloads and their attachment
+   * descriptors. `runId` is carried for log correlation only.
+   */
+  readonly writeArtifacts: (runId: number, artifacts: ReadonlyArray<TestArtifactInput>) => Effect.Effect<void, DataStoreError>;
   readonly writeCoverage: (runId: number, coverage: ReadonlyArray<FileCoverageInput>) => Effect.Effect<void, DataStoreError>;
   readonly writeHistory: (project: string, fullName: string, modulePath: string, runId: number, timestamp: string, state: string, duration: number | null, flaky: boolean, retryCount: number, errorMessage: string | null) => Effect.Effect<void, DataStoreError>;
   readonly writeBaselines: (baselines: CoverageBaselines) => Effect.Effect<void, DataStoreError>;
@@ -1515,6 +1614,45 @@ interface TagInventoryRow {
    */
   readonly testCount: number;
 }
+/** @public */
+interface TestArtifactQueryOptions {
+  /** Disambiguates a fullName that exists in more than one module. */
+  readonly modulePath?: string;
+}
+/** @public */
+interface PersistedAttachment {
+  readonly contentType?: string;
+  readonly path?: string;
+  readonly body?: string;
+  /** How to read `body`. Absent when nothing was stored inline. */
+  readonly bodyEncoding?: "base64" | "utf-8";
+  readonly byteSize: number | null;
+}
+/** @public */
+interface TestAnnotationRow {
+  readonly id: number;
+  readonly type: string;
+  readonly message: string;
+  readonly location?: {
+    readonly file: string;
+    readonly line: number;
+    readonly column: number;
+  };
+  readonly attachments: ReadonlyArray<PersistedAttachment>;
+}
+/** @public */
+interface TestArtifactRow {
+  readonly id: number;
+  readonly type: string;
+  readonly message: string | null;
+  readonly data: string | null;
+  readonly location?: {
+    readonly file: string;
+    readonly line: number;
+    readonly column: number;
+  };
+  readonly attachments: ReadonlyArray<PersistedAttachment>;
+}
 declare const DataReader_base: Context.ServiceClass<DataReader, "vitest-agent/DataReader", {
   readonly getLatestRun: (project: string) => Effect.Effect<Option.Option<AgentReport>, DataStoreError>;
   readonly getRunsByProject: () => Effect.Effect<ReadonlyArray<ProjectRunSummary>, DataStoreError>;
@@ -1527,6 +1665,10 @@ declare const DataReader_base: Context.ServiceClass<DataReader, "vitest-agent/Da
   readonly getCoverage: (project: string) => Effect.Effect<Option.Option<CoverageReport>, DataStoreError>;
   readonly getTestsForFile: (filePath: string) => Effect.Effect<ReadonlyArray<string>, DataStoreError>;
   readonly getErrors: (project: string, errorName?: string) => Effect.Effect<ReadonlyArray<TestError>, DataStoreError>;
+  /** Annotations recorded for a test in the project's latest run. */
+  readonly getAnnotationsForTest: (project: string, fullName: string, options?: TestArtifactQueryOptions) => Effect.Effect<ReadonlyArray<TestAnnotationRow>, DataStoreError>;
+  /** Artifacts recorded for a test in the project's latest run. */
+  readonly getArtifactsForTest: (project: string, fullName: string, options?: TestArtifactQueryOptions) => Effect.Effect<ReadonlyArray<TestArtifactRow>, DataStoreError>;
   readonly getNotes: (scope?: string, project?: string, testFullName?: string) => Effect.Effect<ReadonlyArray<NoteRow>, DataStoreError>;
   readonly getNoteById: (id: number) => Effect.Effect<Option.Option<NoteRow>, DataStoreError>;
   readonly searchNotes: (query: string) => Effect.Effect<ReadonlyArray<NoteRow>, DataStoreError>;
@@ -1677,6 +1819,27 @@ declare const DataReader_base: Context.ServiceClass<DataReader, "vitest-agent/Da
 /** @public */
 export declare class DataReader extends DataReader_base {}
 //#endregion
+//#region src/migrations/index.d.ts
+/**
+ * Every migration that must run against a per-project `data.db`, keyed by
+ * migration id in application order.
+ *
+ * This is the single source of truth for the project database's migration
+ * set: `ensureMigrated`, the sdk testing layer, `ReporterLive`, `McpLive`,
+ * `CliLive` and `SidecarLive` all pass it straight to
+ * `SqliteMigrator.fromRecord`, so adding a migration file and registering it
+ * here is enough to reach every process that opens a project database.
+ *
+ * The session-map and discovery-registry databases are separate schemas with
+ * their own migrations and are deliberately not listed here.
+ *
+ * @public
+ */
+export declare const PROJECT_MIGRATIONS: {
+  readonly "0001_initial": import("effect/Effect").Effect<void, import("effect/unstable/sql/SqlError").SqlError, import("effect/unstable/sql/SqlClient").SqlClient>;
+  readonly "0002_test_artifacts": import("effect/Effect").Effect<void, import("effect/unstable/sql/SqlError").SqlError, import("effect/unstable/sql/SqlClient").SqlClient>;
+};
+//#endregion
 //#region src/testing/layers.d.ts
 /** @public */
 export declare function makeTestLayer(filename: string): Layer.Layer<DataReader | DataStore | import("effect/unstable/sql/SqlClient").SqlClient | import("@effect/sql-sqlite-node/SqliteClient").SqliteClient | NodeServices.NodeServices, SqliteMigrator.MigrationError | import("effect/unstable/sql/SqlError").SqlError, never>;
@@ -1695,5 +1858,5 @@ export declare function flaky(filename: string): Layer.Layer<DataReader | DataSt
 /** @public */
 export declare function withTddTask(filename: string): Layer.Layer<DataReader | DataStore | import("effect/unstable/sql/SqlClient").SqlClient | import("@effect/sql-sqlite-node/SqliteClient").SqliteClient | import("@effect/platform-node/NodeServices").NodeServices, BehaviorNotFoundError | DataStoreError | GoalNotFoundError | IllegalStatusTransitionError | import("@effect/sql-sqlite-node/SqliteMigrator").MigrationError | import("effect/unstable/sql/SqlError").SqlError | TddTaskAlreadyEndedError | TddTaskNotFoundError, never>;
 //#endregion
-export type { AcceptanceMetrics, ArtifactKind, ArtifactSuite, AssociateRunSessionInput, ChangeKind, CitedArtifact, CitedArtifactRow, ClassificationQueryOptions, CommitChangesEntry, CreateBehaviorInput, CreateGoalInput, CurrentTddPhase, EndTddTaskInput, FailureSignatureDetail, FailureSignatureWriteInput, FileCoverageInput, FlakyTest, HistoryQueryOptions, HypothesisDetail, HypothesisInput, IdempotentResponseInput, IllegalStatusTransitionEntity, ModuleInput, ModuleListEntry, NoteInput, NoteRow, PersistentFailure, Phase, ProjectRunSummary, RegisterAgentInput, ResolvedThresholds, RunChangedFile, RunInvocationMethod, SessionDetail, SessionInput, SettingsInput, SettingsListEntry, SettingsRow, StackFrameInput, SuiteInput, SuiteListEntry, TagInventoryRow, TddArtifactDetail, TddArtifactRow, TddPhaseDetail, TddTaskDetail, TddTaskEndOutcome, TddTaskInput, TddTaskSummary, TestCaseInput, TestError, TestErrorInput, TestListEntry, TestLookupOptions, TestRunInput, TurnInput, TurnSearchOptions, TurnSummary, UpdateBehaviorInput, UpdateGoalInput, ValidateHypothesisInput, WriteCommitInput, WriteRunChangedFilesInput, WriteTddArtifactInput, WriteTddPhaseInput, WriteTddPhaseOutput };
+export type { AcceptanceMetrics, ArtifactKind, ArtifactSuite, AssociateRunSessionInput, ChangeKind, CitedArtifact, CitedArtifactRow, ClassificationQueryOptions, CommitChangesEntry, CreateBehaviorInput, CreateGoalInput, CurrentTddPhase, EndTddTaskInput, FailureSignatureDetail, FailureSignatureWriteInput, FileCoverageInput, FlakyTest, HistoryQueryOptions, HypothesisDetail, HypothesisInput, IdempotentResponseInput, IllegalStatusTransitionEntity, ModuleInput, ModuleListEntry, NoteInput, NoteRow, PersistedAttachment, PersistentFailure, Phase, ProjectRunSummary, RegisterAgentInput, ResolvedThresholds, RunChangedFile, RunInvocationMethod, SessionDetail, SessionInput, SettingsInput, SettingsListEntry, SettingsRow, StackFrameInput, SuiteInput, SuiteListEntry, TagInventoryRow, TddArtifactDetail, TddArtifactRow, TddPhaseDetail, TddTaskDetail, TddTaskEndOutcome, TddTaskInput, TddTaskSummary, TestAnnotationInput, TestAnnotationRow, TestArtifactInput, TestArtifactQueryOptions, TestArtifactRow, TestAttachmentInput, TestCaseInput, TestError, TestErrorInput, TestListEntry, TestLookupOptions, TestRunInput, TurnInput, TurnSearchOptions, TurnSummary, UpdateBehaviorInput, UpdateGoalInput, ValidateHypothesisInput, WriteCommitInput, WriteRunChangedFilesInput, WriteTddArtifactInput, WriteTddPhaseInput, WriteTddPhaseOutput };
 //# sourceMappingURL=testing.d.ts.map

package/testing.js

@@ -5,6 +5,7 @@ import { DataReader } from "./services/DataReader.js";
 import { AgentId, ChatId } from "./schemas/Identity.js";
 import { Agent, IdempotencyHit } from "./schemas/Agent.js";
 import { DataStore } from "./services/DataStore.js";
+import { PROJECT_MIGRATIONS } from "./migrations/index.js";
 import { CoverageReport, FileCoverageReport } from "./schemas/Coverage.js";
 import { AgentReport } from "./schemas/AgentReport.js";
 import { CoverageBaselines } from "./schemas/Baselines.js";
@@ -26,7 +27,7 @@ function singlePassingRun(filename) {
 	const seed = Effect.gen(function* () {
 		const store = yield* DataStore;
 		yield* store.writeSettings("hash-preset-spr", {
-			vitestVersion: "4.1.5",
+			vitestVersion: "5.0.0",
 			pool: "forks"
 		}, {});
 		const runId = yield* store.writeRun({
@@ -80,7 +81,7 @@ function withFailures(filename) {
 	const seed = Effect.gen(function* () {
 		const store = yield* DataStore;
 		yield* store.writeSettings("hash-preset-wf", {
-			vitestVersion: "4.1.5",
+			vitestVersion: "5.0.0",
 			pool: "forks"
 		}, {});
 		const runId = yield* store.writeRun({
@@ -140,7 +141,7 @@ function flaky(filename) {
 	const seed = Effect.gen(function* () {
 		const store = yield* DataStore;
 		yield* store.writeSettings("hash-preset-flaky", {
-			vitestVersion: "4.1.5",
+			vitestVersion: "5.0.0",
 			pool: "forks"
 		}, {});
 		const runId1 = yield* store.writeRun({
@@ -250,4 +251,4 @@ function withTddTask(filename) {
 }
 
 //#endregion
-export { Agent, AgentId, AgentNotFoundError, AgentReport, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorStatus, CacheManifest, ChatId, CoverageBaselines, CoverageReport, DataReader, DataStore, DataStoreError, DataStoreTestLayer, FileCoverageReport, GoalDetail, GoalNotFoundError, GoalRow, GoalStatus, HistoryRecord, IdempotencyHit, IllegalStatusTransitionError, RegistrationConflictError, TddTaskAlreadyEndedError, TddTaskNotFoundError, TrendEntry, TrendRecord, empty, flaky, makeTestLayer, singlePassingRun, withFailures, withTddTask };
+export { Agent, AgentId, AgentNotFoundError, AgentReport, BehaviorDetail, BehaviorNotFoundError, BehaviorRow, BehaviorStatus, CacheManifest, ChatId, CoverageBaselines, CoverageReport, DataReader, DataStore, DataStoreError, DataStoreTestLayer, FileCoverageReport, GoalDetail, GoalNotFoundError, GoalRow, GoalStatus, HistoryRecord, IdempotencyHit, IllegalStatusTransitionError, PROJECT_MIGRATIONS, RegistrationConflictError, TddTaskAlreadyEndedError, TddTaskNotFoundError, TrendEntry, TrendRecord, empty, flaky, makeTestLayer, singlePassingRun, withFailures, withTddTask };

package/utils/ensure-migrated.js

@@ -1,5 +1,5 @@
 import { LoggerLive } from "../layers/LoggerLive.js";
-import migration from "../migrations/0001_initial.js";
+import { PROJECT_MIGRATIONS } from "../migrations/index.js";
 import { Effect, Layer } from "effect";
 import { SqlClient } from "effect/unstable/sql/SqlClient";
 import * as NodeServices from "@effect/platform-node/NodeServices";
@@ -30,7 +30,7 @@ function ensureMigrated(dbPath, logLevel, logFile) {
 	if (cached) return cached;
 	const SqliteLayer = layer({ filename: dbPath });
 	const PlatformLayer = NodeServices.layer;
-	const MigratorLayer = SqliteMigrator.layer({ loader: SqliteMigrator.fromRecord({ "0001_initial": migration }) }).pipe(Layer.provide(Layer.merge(SqliteLayer, PlatformLayer)));
+	const MigratorLayer = SqliteMigrator.layer({ loader: SqliteMigrator.fromRecord(PROJECT_MIGRATIONS) }).pipe(Layer.provide(Layer.merge(SqliteLayer, PlatformLayer)));
 	const program = Effect.gen(function* () {
 		yield* SqlClient;
 	}).pipe(Effect.provide(MigratorLayer), Effect.provide(Layer.merge(SqliteLayer, PlatformLayer)), Effect.provide(LoggerLive(logLevel, logFile)));

package/utils/test-location.js

@@ -14,6 +14,16 @@ const TEST_HELPER_DIRS = [
 /** Glob suffix matching every discoverable test file. @public */
 const TEST_FILE_GLOB_SUFFIX = "*.{test,spec}.{ts,tsx,js,jsx}";
 /**
+* Glob suffix matching every Bats shell test file. Bats tests are run by
+* `bats`, not Vitest — a package whose `__test__/` directory holds only
+* `.bats` files (plus any of {@link TEST_HELPER_DIRS}) is a fully supported,
+* intentional layout, not an orphaned test directory. Paired with
+* {@link TEST_FILE_GLOB_SUFFIX} by `@vitest-agent/plugin`'s
+* `isTestShapedPackage` to decide whether a package without a discoverable
+* Vitest project still warrants a warning. @public
+*/
+const BATS_FILE_GLOB_SUFFIX = "*.bats";
+/**
 * 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.
@@ -114,4 +124,4 @@ function classifyTestPath(workspaces, filePath) {
 }
 
 //#endregion
-export { NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, classifyTestPath, findOwningWorkspace, isTestFileName };
+export { BATS_FILE_GLOB_SUFFIX, NON_DISCOVERABLE_DIRS, SRC_DIR, TEST_DIR, TEST_FILE_GLOB_SUFFIX, TEST_HELPER_DIRS, classifyTestPath, findOwningWorkspace, isTestFileName };

package/utils/validate-coverage-targets-shape.js

@@ -24,9 +24,15 @@ function checkBooleanShortcut(key, path, errors) {
 		message: `Coverage target at "${path}" cannot be true. The "true" shorthand is only valid at key "100" (meaning 100% across all metrics).`
 	});
 }
-/** Checks all values inside a nested per-metric object (glob-pattern entry). */
+/**
+* Checks all values inside a nested per-metric object (glob-pattern entry).
+* `perFile` is a legal key inside a glob entry under Vitest 5 — a glob entry
+* no longer inherits the top-level setting — so it is skipped here rather
+* than run through the boolean-shortcut check.
+*/
 function checkNestedMetrics(nested, prefix, errors) {
 	for (const [metricKey, metricValue] of Object.entries(nested)) {
+		if (metricKey === "perFile") continue;
 		const path = `${prefix}.${metricKey}`;
 		if (typeof metricValue === "number") checkNumericValue(metricValue, path, errors);
 		else if (metricValue === true) checkBooleanShortcut(metricKey, path, errors);
@@ -41,8 +47,10 @@ function checkNestedMetrics(nested, prefix, errors) {
 *   OR a literal `true` appears at any key other than `"100"`. The
 *   offending `path` is included (e.g. `"lines"`, `"src/**.ts.lines"`,
 *   or `"statements"` when misused as a boolean shortcut).
-* - `PERFILE_ON_TARGETS` — the `perFile` key appears inside coverageTargets;
-*   it should be set on `coverage.thresholds.perFile` instead.
+* - `PERFILE_ON_TARGETS` — a top-level `perFile` key appears in
+*   coverageTargets; the top-level setting belongs on
+*   `coverage.thresholds.perFile`. A `perFile` inside a glob-pattern entry
+*   is legal and is not reported.
 *
 * @param input - The raw value passed as coverageTargets.
 * @returns A result object with errors, warnings, and info arrays.
@@ -62,7 +70,7 @@ function validateCoverageTargetsShape(input) {
 			warnings.push({
 				code: "PERFILE_ON_TARGETS",
 				path: key,
-				message: "The \"perFile\" key is not valid inside coverageTargets. Set coverage.thresholds.perFile instead."
+				message: "A top-level \"perFile\" key is not valid inside coverageTargets. Set coverage.thresholds.perFile instead, or move it inside a glob-pattern entry — under Vitest 5 a glob-pattern entry carries its own perFile and no longer inherits the top-level one."
 			});
 			continue;
 		}