@cursor/july 0.1.39 → 0.1.41

package/dist/storage-backends/postgres-kv.d.ts

@@ -1,82 +0,0 @@
-/**
- * Postgres `defineStorage` helper on the agent-serve tables
- * (backend/packages/prisma-bugbot):
- *
- *   agent_serve_kv(deployment_id, key, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, key)
- *   agent_serve_eval_runs(deployment_id, agent, run_id, run JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, run_id)
- *   agent_serve_ab(deployment_id, agent, kind, ref, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, kind, ref)
- *
- * For Cursor-managed hosting prefer `cursorHostedStorage` from
- * `@cursor/july/storage/cursor-hosted` — same tables via a control-plane
- * HTTP proxy (no pod-visible DSN). Use this helper when you supply
- * `BUGBOTDB_URL` / `AGENT_SERVE_DEPLOYMENT_ID` yourself (self-host).
- *
- * `postgresStorage` implements the whole `defineStorage` surface as plain
- * SQL: the four KV methods against the last-write-wins `agent_serve_kv`
- * table (sessions, event chunks, continuations, reminders), the `evals`
- * table against `agent_serve_eval_runs` (one row per playground eval
- * batch), and the `abs` table against `agent_serve_ab` (`kind = 'sample'`
- * rows per metric sample plus one `kind = 'snapshot'` row holding the
- * latest aggregate).
- *
- * - KV `put` upserts on the primary key (LWW matches the framework contract).
- * - KV `list` is a prefix range scan on the PK btree (`key >= p AND key < p||…`
- *   spelled as LIKE with escaped wildcards, ordered by key).
- * - `get` returning no row is a definitive miss (`undefined`); a connection
- *   error throws, so lazy restore fails the follow-up instead of forking
- *   the session.
- *
- * Connection comes from `BUGBOTDB_URL` — the repo-wide convention for the
- * bugbot Postgres database, whose Prisma package owns these tables' schema
- * and migrations; locally it falls back to the dev bugbot DB. Non-local
- * URLs enable TLS (`ssl.rejectUnauthorized`) because PlanetScale requires
- * SSL even when the vault URL omits `sslmode`.
- * `deploymentId` scopes rows per deployment (`AGENT_SERVE_DEPLOYMENT_ID`);
- * the local default keeps dev runs of different checkouts from colliding
- * only by agent name.
- *
- * Callers must pass `Pool` from their own `import { Pool } from "pg"`.
- * This module must not import `pg` itself: Node resolves modules from
- * *this* file's directory tree, so a `pg` dep declared only on the
- * consuming agent package would never be found after a hosted install.
- */
-import type { StorageABTable, StorageConfig, StorageEvalsTable } from "../storage.js";
-/** Minimal `pg.Pool` surface used here (avoids importing `pg` from this path). */
-export interface PostgresQueryable {
-    query<T extends Record<string, unknown> = Record<string, unknown>>(text: string, values?: readonly unknown[]): Promise<{
-        rows: T[];
-    }>;
-}
-/** `pg.Pool` constructor supplied by the consuming example package. */
-export type PostgresPoolConstructor = new (config?: {
-    connectionString?: string;
-    max?: number;
-    ssl?: boolean | {
-        rejectUnauthorized?: boolean;
-    };
-}) => PostgresQueryable;
-export interface PostgresStorageOptions {
-    /**
-     * Pass `Pool` from `import { Pool } from "pg"` in the example that declares
-     * the `pg` dependency (hosted installs put it in that package's
-     * `node_modules` only).
-     */
-    Pool: PostgresPoolConstructor;
-    /** Postgres connection string (default: env, then local bugbot dev DB). */
-    databaseUrl?: string;
-    /** Value for the tables' deployment_id column (default: env or "local-dev"). */
-    deploymentId?: string;
-}
-type PostgresStorageMethods = Required<Pick<StorageConfig, "put" | "get" | "delete" | "list">> & {
-    evals: StorageEvalsTable;
-    abs: StorageABTable;
-};
-/** True for local bugbot/dev URLs that do not speak TLS. */
-export declare function isLocalPostgresUrl(connectionString: string): boolean;
-/** Postgres-backed `defineStorage` methods, ready to spread into `defineStorage({...})`. */
-export declare function postgresStorage(options: PostgresStorageOptions): PostgresStorageMethods;
-export {};
-//# sourceMappingURL=postgres-kv.d.ts.map

package/dist/storage-backends/postgres-kv.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"postgres-kv.d.ts","sourceRoot":"","sources":["../../src/storage-backends/postgres-kv.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4CG;AAIH,OAAO,KAAK,EACV,cAAc,EACd,aAAa,EAEb,iBAAiB,EAClB,MAAM,eAAe,CAAC;AAGvB,kFAAkF;AAClF,MAAM,WAAW,iBAAiB;IAChC,KAAK,CAAC,CAAC,SAAS,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC/D,IAAI,EAAE,MAAM,EACZ,MAAM,CAAC,EAAE,SAAS,OAAO,EAAE,GAC1B,OAAO,CAAC;QAAE,IAAI,EAAE,CAAC,EAAE,CAAA;KAAE,CAAC,CAAC;CAC3B;AAED,uEAAuE;AACvE,MAAM,MAAM,uBAAuB,GAAG,KAAK,MAAM,CAAC,EAAE;IAClD,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,OAAO,GAAG;QAAE,kBAAkB,CAAC,EAAE,OAAO,CAAA;KAAE,CAAC;CAClD,KAAK,iBAAiB,CAAC;AAExB,MAAM,WAAW,sBAAsB;IACrC;;;;OAIG;IACH,IAAI,EAAE,uBAAuB,CAAC;IAC9B,2EAA2E;IAC3E,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,gFAAgF;IAChF,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,KAAK,sBAAsB,GAAG,QAAQ,CACpC,IAAI,CAAC,aAAa,EAAE,KAAK,GAAG,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC,CACvD,GAAG;IACF,KAAK,EAAE,iBAAiB,CAAC;IACzB,GAAG,EAAE,cAAc,CAAC;CACrB,CAAC;AAEF,4DAA4D;AAC5D,wBAAgB,kBAAkB,CAAC,gBAAgB,EAAE,MAAM,GAAG,OAAO,CAYpE;AAED,4FAA4F;AAC5F,wBAAgB,eAAe,CAC7B,OAAO,EAAE,sBAAsB,GAC9B,sBAAsB,CA4HxB"}

package/dist/storage-backends/postgres-kv.js

@@ -1,185 +0,0 @@
-/**
- * Postgres `defineStorage` helper on the agent-serve tables
- * (backend/packages/prisma-bugbot):
- *
- *   agent_serve_kv(deployment_id, key, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, key)
- *   agent_serve_eval_runs(deployment_id, agent, run_id, run JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, run_id)
- *   agent_serve_ab(deployment_id, agent, kind, ref, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, kind, ref)
- *
- * For Cursor-managed hosting prefer `cursorHostedStorage` from
- * `@cursor/july/storage/cursor-hosted` — same tables via a control-plane
- * HTTP proxy (no pod-visible DSN). Use this helper when you supply
- * `BUGBOTDB_URL` / `AGENT_SERVE_DEPLOYMENT_ID` yourself (self-host).
- *
- * `postgresStorage` implements the whole `defineStorage` surface as plain
- * SQL: the four KV methods against the last-write-wins `agent_serve_kv`
- * table (sessions, event chunks, continuations, reminders), the `evals`
- * table against `agent_serve_eval_runs` (one row per playground eval
- * batch), and the `abs` table against `agent_serve_ab` (`kind = 'sample'`
- * rows per metric sample plus one `kind = 'snapshot'` row holding the
- * latest aggregate).
- *
- * - KV `put` upserts on the primary key (LWW matches the framework contract).
- * - KV `list` is a prefix range scan on the PK btree (`key >= p AND key < p||…`
- *   spelled as LIKE with escaped wildcards, ordered by key).
- * - `get` returning no row is a definitive miss (`undefined`); a connection
- *   error throws, so lazy restore fails the follow-up instead of forking
- *   the session.
- *
- * Connection comes from `BUGBOTDB_URL` — the repo-wide convention for the
- * bugbot Postgres database, whose Prisma package owns these tables' schema
- * and migrations; locally it falls back to the dev bugbot DB. Non-local
- * URLs enable TLS (`ssl.rejectUnauthorized`) because PlanetScale requires
- * SSL even when the vault URL omits `sslmode`.
- * `deploymentId` scopes rows per deployment (`AGENT_SERVE_DEPLOYMENT_ID`);
- * the local default keeps dev runs of different checkouts from colliding
- * only by agent name.
- *
- * Callers must pass `Pool` from their own `import { Pool } from "pg"`.
- * This module must not import `pg` itself: Node resolves modules from
- * *this* file's directory tree, so a `pg` dep declared only on the
- * consuming agent package would never be found after a hosted install.
- */
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-/** True for local bugbot/dev URLs that do not speak TLS. */
-export function isLocalPostgresUrl(connectionString) {
-    try {
-        const host = new URL(connectionString).hostname.toLowerCase();
-        return (host === "localhost" ||
-            host === "127.0.0.1" ||
-            host === "::1" ||
-            host === "[::1]");
-    }
-    catch (_a) {
-        return /@(localhost|127\.0\.0\.1|\[::1\])([:/]|$)/i.test(connectionString);
-    }
-}
-/** Postgres-backed `defineStorage` methods, ready to spread into `defineStorage({...})`. */
-export function postgresStorage(options) {
-    var _a, _b, _c, _d;
-    const connectionString = (_b = (_a = options.databaseUrl) !== null && _a !== void 0 ? _a : process.env.BUGBOTDB_URL) !== null && _b !== void 0 ? _b : "postgresql://prisma:prisma@localhost:5432/bugbot";
-    // Prod Bugbot is PlanetScale Postgres: TLS is required and the vault
-    // `BUGBOTDB_URL` often omits `sslmode`. Without an explicit `ssl` option,
-    // `pg` opens plaintext and PlanetScale rejects with "SSL connection is
-    // required" — sessions stay in-memory. Skip TLS only for local hosts.
-    const pool = new options.Pool(Object.assign({ connectionString, 
-        // Small fixed pool per agent, mirroring the backend's connection_limit
-        // clamp on BUGBOTDB_URL consumers (backend/server/src/prisma.ts).
-        max: 3 }, (isLocalPostgresUrl(connectionString)
-        ? {}
-        : { ssl: { rejectUnauthorized: true } })));
-    const deploymentId = (_d = (_c = options.deploymentId) !== null && _c !== void 0 ? _c : process.env.AGENT_SERVE_DEPLOYMENT_ID) !== null && _d !== void 0 ? _d : "local-dev";
-    return {
-        put(key, value) {
-            return __awaiter(this, void 0, void 0, function* () {
-                yield pool.query(`INSERT INTO agent_serve_kv (deployment_id, key, value, updated_at)
-         VALUES ($1, $2, $3::jsonb, now())
-         ON CONFLICT ON CONSTRAINT pk_agent_serve_kv
-         DO UPDATE SET value = EXCLUDED.value, updated_at = now()`, [deploymentId, key, JSON.stringify(value)]);
-            });
-        },
-        get(key) {
-            return __awaiter(this, void 0, void 0, function* () {
-                var _a;
-                const result = yield pool.query(`SELECT value FROM agent_serve_kv
-         WHERE deployment_id = $1 AND key = $2`, [deploymentId, key]);
-                return (_a = result.rows[0]) === null || _a === void 0 ? void 0 : _a.value; // no row ⇒ definitive miss
-            });
-        },
-        delete(key) {
-            return __awaiter(this, void 0, void 0, function* () {
-                yield pool.query(`DELETE FROM agent_serve_kv WHERE deployment_id = $1 AND key = $2`, [deploymentId, key]);
-            });
-        },
-        list(prefix) {
-            return __awaiter(this, void 0, void 0, function* () {
-                // Escape LIKE wildcards in the prefix so keys containing % or _
-                // (URI-encoded segments cannot, but be safe) match literally.
-                const escaped = prefix.replace(/([\\%_])/g, "\\$1");
-                const result = yield pool.query(`SELECT key, value FROM agent_serve_kv
-         WHERE deployment_id = $1 AND key LIKE $2 ESCAPE '\\'
-         ORDER BY key ASC`, [deploymentId, `${escaped}%`]);
-                return result.rows;
-            });
-        },
-        // Playground eval batches → agent_serve_eval_runs (one row per run).
-        evals: {
-            put(run, ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    yield pool.query(`INSERT INTO agent_serve_eval_runs (deployment_id, agent, run_id, run, updated_at)
-           VALUES ($1, $2, $3, $4::jsonb, now())
-           ON CONFLICT ON CONSTRAINT pk_agent_serve_eval_runs
-           DO UPDATE SET run = EXCLUDED.run, updated_at = now()`, [deploymentId, ctx.agentName, run.runId, JSON.stringify(run)]);
-                });
-            },
-            delete(runId, ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    yield pool.query(`DELETE FROM agent_serve_eval_runs
-           WHERE deployment_id = $1 AND agent = $2 AND run_id = $3`, [deploymentId, ctx.agentName, runId]);
-                });
-            },
-            list(ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    const result = yield pool.query(`SELECT run FROM agent_serve_eval_runs
-           WHERE deployment_id = $1 AND agent = $2`, [deploymentId, ctx.agentName]);
-                    return result.rows.map((row) => row.run);
-                });
-            },
-        },
-        // A/B metrics → agent_serve_ab: sample rows plus one snapshot row.
-        abs: {
-            putSample(sample, ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    yield upsertAbRow(pool, {
-                        deploymentId,
-                        agent: ctx.agentName,
-                        kind: "sample",
-                        // One boundary event emits a sample per enrolled experiment with
-                        // the same sessionId/at — the experiment must be in the row key
-                        // or those samples overwrite each other.
-                        ref: `${sample.experiment}/${sample.sessionId}/${sample.at}`,
-                        value: sample,
-                    });
-                });
-            },
-            putSnapshot(snapshot, ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    yield upsertAbRow(pool, {
-                        deploymentId,
-                        agent: ctx.agentName,
-                        kind: "snapshot",
-                        ref: "",
-                        value: snapshot,
-                    });
-                });
-            },
-            getSnapshot(ctx) {
-                return __awaiter(this, void 0, void 0, function* () {
-                    var _a;
-                    const result = yield pool.query(`SELECT value FROM agent_serve_ab
-           WHERE deployment_id = $1 AND agent = $2 AND kind = 'snapshot' AND ref = ''`, [deploymentId, ctx.agentName]);
-                    return (_a = result.rows[0]) === null || _a === void 0 ? void 0 : _a.value;
-                });
-            },
-        },
-    };
-}
-function upsertAbRow(pool, row) {
-    return __awaiter(this, void 0, void 0, function* () {
-        yield pool.query(`INSERT INTO agent_serve_ab (deployment_id, agent, kind, ref, value, updated_at)
-     VALUES ($1, $2, $3, $4, $5::jsonb, now())
-     ON CONFLICT ON CONSTRAINT pk_agent_serve_ab
-     DO UPDATE SET value = EXCLUDED.value, updated_at = now()`, [row.deploymentId, row.agent, row.kind, row.ref, JSON.stringify(row.value)]);
-    });
-}

package/src/storage-backends/postgres-kv.ts

@@ -1,253 +0,0 @@
-/**
- * Postgres `defineStorage` helper on the agent-serve tables
- * (backend/packages/prisma-bugbot):
- *
- *   agent_serve_kv(deployment_id, key, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, key)
- *   agent_serve_eval_runs(deployment_id, agent, run_id, run JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, run_id)
- *   agent_serve_ab(deployment_id, agent, kind, ref, value JSONB, updated_at)
- *     PRIMARY KEY (deployment_id, agent, kind, ref)
- *
- * For Cursor-managed hosting prefer `cursorHostedStorage` from
- * `@cursor/july/storage/cursor-hosted` — same tables via a control-plane
- * HTTP proxy (no pod-visible DSN). Use this helper when you supply
- * `BUGBOTDB_URL` / `AGENT_SERVE_DEPLOYMENT_ID` yourself (self-host).
- *
- * `postgresStorage` implements the whole `defineStorage` surface as plain
- * SQL: the four KV methods against the last-write-wins `agent_serve_kv`
- * table (sessions, event chunks, continuations, reminders), the `evals`
- * table against `agent_serve_eval_runs` (one row per playground eval
- * batch), and the `abs` table against `agent_serve_ab` (`kind = 'sample'`
- * rows per metric sample plus one `kind = 'snapshot'` row holding the
- * latest aggregate).
- *
- * - KV `put` upserts on the primary key (LWW matches the framework contract).
- * - KV `list` is a prefix range scan on the PK btree (`key >= p AND key < p||…`
- *   spelled as LIKE with escaped wildcards, ordered by key).
- * - `get` returning no row is a definitive miss (`undefined`); a connection
- *   error throws, so lazy restore fails the follow-up instead of forking
- *   the session.
- *
- * Connection comes from `BUGBOTDB_URL` — the repo-wide convention for the
- * bugbot Postgres database, whose Prisma package owns these tables' schema
- * and migrations; locally it falls back to the dev bugbot DB. Non-local
- * URLs enable TLS (`ssl.rejectUnauthorized`) because PlanetScale requires
- * SSL even when the vault URL omits `sslmode`.
- * `deploymentId` scopes rows per deployment (`AGENT_SERVE_DEPLOYMENT_ID`);
- * the local default keeps dev runs of different checkouts from colliding
- * only by agent name.
- *
- * Callers must pass `Pool` from their own `import { Pool } from "pg"`.
- * This module must not import `pg` itself: Node resolves modules from
- * *this* file's directory tree, so a `pg` dep declared only on the
- * consuming agent package would never be found after a hosted install.
- */
-
-import type { ABSnapshot } from "../ab.js";
-import type { EvalRunSnapshot } from "../evals.js";
-import type {
-  StorageABTable,
-  StorageConfig,
-  StorageEntry,
-  StorageEvalsTable,
-} from "../storage.js";
-import type { JsonValue } from "../types.js";
-
-/** Minimal `pg.Pool` surface used here (avoids importing `pg` from this path). */
-export interface PostgresQueryable {
-  query<T extends Record<string, unknown> = Record<string, unknown>>(
-    text: string,
-    values?: readonly unknown[]
-  ): Promise<{ rows: T[] }>;
-}
-
-/** `pg.Pool` constructor supplied by the consuming example package. */
-export type PostgresPoolConstructor = new (config?: {
-  connectionString?: string;
-  max?: number;
-  ssl?: boolean | { rejectUnauthorized?: boolean };
-}) => PostgresQueryable;
-
-export interface PostgresStorageOptions {
-  /**
-   * Pass `Pool` from `import { Pool } from "pg"` in the example that declares
-   * the `pg` dependency (hosted installs put it in that package's
-   * `node_modules` only).
-   */
-  Pool: PostgresPoolConstructor;
-  /** Postgres connection string (default: env, then local bugbot dev DB). */
-  databaseUrl?: string;
-  /** Value for the tables' deployment_id column (default: env or "local-dev"). */
-  deploymentId?: string;
-}
-
-type PostgresStorageMethods = Required<
-  Pick<StorageConfig, "put" | "get" | "delete" | "list">
-> & {
-  evals: StorageEvalsTable;
-  abs: StorageABTable;
-};
-
-/** True for local bugbot/dev URLs that do not speak TLS. */
-export function isLocalPostgresUrl(connectionString: string): boolean {
-  try {
-    const host = new URL(connectionString).hostname.toLowerCase();
-    return (
-      host === "localhost" ||
-      host === "127.0.0.1" ||
-      host === "::1" ||
-      host === "[::1]"
-    );
-  } catch {
-    return /@(localhost|127\.0\.0\.1|\[::1\])([:/]|$)/i.test(connectionString);
-  }
-}
-
-/** Postgres-backed `defineStorage` methods, ready to spread into `defineStorage({...})`. */
-export function postgresStorage(
-  options: PostgresStorageOptions
-): PostgresStorageMethods {
-  const connectionString =
-    options.databaseUrl ??
-    process.env.BUGBOTDB_URL ??
-    "postgresql://prisma:prisma@localhost:5432/bugbot";
-  // Prod Bugbot is PlanetScale Postgres: TLS is required and the vault
-  // `BUGBOTDB_URL` often omits `sslmode`. Without an explicit `ssl` option,
-  // `pg` opens plaintext and PlanetScale rejects with "SSL connection is
-  // required" — sessions stay in-memory. Skip TLS only for local hosts.
-  const pool = new options.Pool({
-    connectionString,
-    // Small fixed pool per agent, mirroring the backend's connection_limit
-    // clamp on BUGBOTDB_URL consumers (backend/server/src/prisma.ts).
-    max: 3,
-    ...(isLocalPostgresUrl(connectionString)
-      ? {}
-      : { ssl: { rejectUnauthorized: true } }),
-  });
-  const deploymentId =
-    options.deploymentId ??
-    process.env.AGENT_SERVE_DEPLOYMENT_ID ??
-    "local-dev";
-
-  return {
-    async put(key, value): Promise<void> {
-      await pool.query(
-        `INSERT INTO agent_serve_kv (deployment_id, key, value, updated_at)
-         VALUES ($1, $2, $3::jsonb, now())
-         ON CONFLICT ON CONSTRAINT pk_agent_serve_kv
-         DO UPDATE SET value = EXCLUDED.value, updated_at = now()`,
-        [deploymentId, key, JSON.stringify(value)]
-      );
-    },
-
-    async get(key): Promise<JsonValue | undefined> {
-      const result = await pool.query<{ value: JsonValue }>(
-        `SELECT value FROM agent_serve_kv
-         WHERE deployment_id = $1 AND key = $2`,
-        [deploymentId, key]
-      );
-      return result.rows[0]?.value; // no row ⇒ definitive miss
-    },
-
-    async delete(key): Promise<void> {
-      await pool.query(
-        `DELETE FROM agent_serve_kv WHERE deployment_id = $1 AND key = $2`,
-        [deploymentId, key]
-      );
-    },
-
-    async list(prefix): Promise<StorageEntry[]> {
-      // Escape LIKE wildcards in the prefix so keys containing % or _
-      // (URI-encoded segments cannot, but be safe) match literally.
-      const escaped = prefix.replace(/([\\%_])/g, "\\$1");
-      const result = await pool.query<{ key: string; value: JsonValue }>(
-        `SELECT key, value FROM agent_serve_kv
-         WHERE deployment_id = $1 AND key LIKE $2 ESCAPE '\\'
-         ORDER BY key ASC`,
-        [deploymentId, `${escaped}%`]
-      );
-      return result.rows;
-    },
-
-    // Playground eval batches → agent_serve_eval_runs (one row per run).
-    evals: {
-      async put(run, ctx): Promise<void> {
-        await pool.query(
-          `INSERT INTO agent_serve_eval_runs (deployment_id, agent, run_id, run, updated_at)
-           VALUES ($1, $2, $3, $4::jsonb, now())
-           ON CONFLICT ON CONSTRAINT pk_agent_serve_eval_runs
-           DO UPDATE SET run = EXCLUDED.run, updated_at = now()`,
-          [deploymentId, ctx.agentName, run.runId, JSON.stringify(run)]
-        );
-      },
-      async delete(runId, ctx): Promise<void> {
-        await pool.query(
-          `DELETE FROM agent_serve_eval_runs
-           WHERE deployment_id = $1 AND agent = $2 AND run_id = $3`,
-          [deploymentId, ctx.agentName, runId]
-        );
-      },
-      async list(ctx): Promise<EvalRunSnapshot[]> {
-        const result = await pool.query<{ run: EvalRunSnapshot }>(
-          `SELECT run FROM agent_serve_eval_runs
-           WHERE deployment_id = $1 AND agent = $2`,
-          [deploymentId, ctx.agentName]
-        );
-        return result.rows.map((row) => row.run);
-      },
-    },
-
-    // A/B metrics → agent_serve_ab: sample rows plus one snapshot row.
-    abs: {
-      async putSample(sample, ctx): Promise<void> {
-        await upsertAbRow(pool, {
-          deploymentId,
-          agent: ctx.agentName,
-          kind: "sample",
-          // One boundary event emits a sample per enrolled experiment with
-          // the same sessionId/at — the experiment must be in the row key
-          // or those samples overwrite each other.
-          ref: `${sample.experiment}/${sample.sessionId}/${sample.at}`,
-          value: sample,
-        });
-      },
-      async putSnapshot(snapshot, ctx): Promise<void> {
-        await upsertAbRow(pool, {
-          deploymentId,
-          agent: ctx.agentName,
-          kind: "snapshot",
-          ref: "",
-          value: snapshot,
-        });
-      },
-      async getSnapshot(ctx): Promise<ABSnapshot | undefined> {
-        const result = await pool.query<{ value: JsonValue }>(
-          `SELECT value FROM agent_serve_ab
-           WHERE deployment_id = $1 AND agent = $2 AND kind = 'snapshot' AND ref = ''`,
-          [deploymentId, ctx.agentName]
-        );
-        return result.rows[0]?.value as unknown as ABSnapshot | undefined;
-      },
-    },
-  };
-}
-
-async function upsertAbRow(
-  pool: PostgresQueryable,
-  row: {
-    deploymentId: string;
-    agent: string;
-    kind: "sample" | "snapshot";
-    ref: string;
-    value: unknown;
-  }
-): Promise<void> {
-  await pool.query(
-    `INSERT INTO agent_serve_ab (deployment_id, agent, kind, ref, value, updated_at)
-     VALUES ($1, $2, $3, $4, $5::jsonb, now())
-     ON CONFLICT ON CONSTRAINT pk_agent_serve_ab
-     DO UPDATE SET value = EXCLUDED.value, updated_at = now()`,
-    [row.deploymentId, row.agent, row.kind, row.ref, JSON.stringify(row.value)]
-  );
-}