@open-mercato/shared 0.6.6-develop.5586.1.c9ed1d68a8 → 0.6.6-develop.5594.1.30cd738303

package/AGENTS.md

@@ -41,6 +41,7 @@ yarn workspace @open-mercato/shared build
 | `crud/` | When building CRUD routes | `@open-mercato/shared/lib/crud` |
 | `custom-fields/` | When handling custom field payloads | `@open-mercato/shared/lib/custom-fields` |
 | `data/` | When you need `DataEngine` or `QueryEngine` types | `@open-mercato/shared/lib/data/engine` |
+| `db/` | When resolving the ORM/connection-pool config (`resolvePoolConfig`, pool/timeout env knobs) | `@open-mercato/shared/lib/db/mikro` |
 | `di/` | When setting up dependency injection (Awilix) | `@open-mercato/shared/lib/di` |
 | `encryption/` | When querying encrypted entities (MUST use instead of raw `em.find`) | `@open-mercato/shared/lib/encryption/find` |
 | `i18n/` | When translating strings — `useT()` client-side, `resolveTranslations()` server-side | `@open-mercato/shared/lib/i18n/context` or `/server` |
@@ -66,6 +67,17 @@ When you need shared type definitions, import from these:
 
 ## Key Patterns
 
+### Database Connection Pool — MUST keep total demand under `max_connections`
+
+The MikroORM pool is configured from env via `resolvePoolConfig(process.env)` in
+`@open-mercato/shared/lib/db/mikro` (pure and unit-testable). Each process gets
+its own pool (`DB_POOL_MAX`, default 20). Because background worker jobs each run
+in their own request container (one pooled connection per in-flight job), the
+peak connection demand of all processes against one database is additive.
+
+- **Invariant:** `web_pool_max + worker_pool_max + scheduler/overhead ≤ Postgres max_connections` (with headroom). Violating it lets background work starve the request/onboarding path — see `packages/queue/AGENTS.md` → Connection Budget.
+- Tune per process with `DB_POOL_MAX` (and the opt-in `DB_STATEMENT_TIMEOUT_MS` / `DB_LOCK_TIMEOUT_MS`; `idle_in_transaction` defaults to a finite 120s). Bound the worker's job concurrency with `OM_WORKERS_DB_CONNECTION_BUDGET`.
+
 ### Encryption — MUST use instead of raw ORM queries
 
 ```typescript

package/dist/lib/indexers/status-log.js

@@ -29,6 +29,10 @@ function pickDb(deps) {
   }
   return null;
 }
+function isInactiveTransactionError(error) {
+  const message = error instanceof Error ? error.message : String(error ?? "");
+  return /transaction is already (committed|rolled back)/i.test(message);
+}
 async function pruneExcessLogs(db, source) {
   const rows = await db.selectFrom("indexer_status_logs").select("id").where("source", "=", source).orderBy("occurred_at", "desc").orderBy("id", "desc").offset(MAX_LOGS_PER_SOURCE).limit(MAX_DELETE_BATCH).execute();
   if (!rows.length) return;
@@ -63,13 +67,17 @@ async function recordIndexerLog(deps, input) {
       occurred_at: occurredAt
     }).execute();
   } catch (error) {
-    console.error("[indexers] Failed to persist indexer log", error);
+    if (!isInactiveTransactionError(error)) {
+      console.error("[indexers] Failed to persist indexer log", error);
+    }
     return;
   }
   try {
     await pruneExcessLogs(db, input.source);
   } catch (pruneError) {
-    console.warn("[indexers] Failed to prune indexer logs", pruneError);
+    if (!isInactiveTransactionError(pruneError)) {
+      console.warn("[indexers] Failed to prune indexer logs", pruneError);
+    }
   }
 }
 export {

package/dist/lib/indexers/status-log.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/lib/indexers/status-log.ts"],
-  "sourcesContent": ["import type { EntityManager } from '@mikro-orm/postgresql'\nimport { type Kysely, sql } from 'kysely'\nimport type { IndexerErrorSource } from './error-log'\n\nexport type IndexerLogLevel = 'info' | 'warn'\n\nexport type RecordIndexerLogInput = {\n  source: IndexerErrorSource\n  handler: string\n  message: string\n  level?: IndexerLogLevel\n  entityType?: string | null\n  recordId?: string | null\n  tenantId?: string | null\n  organizationId?: string | null\n  details?: unknown\n}\n\ntype RecordIndexerLogDeps = {\n  em?: EntityManager\n  db?: Kysely<any>\n}\n\nconst MAX_MESSAGE_LENGTH = 4_096\nconst MAX_DELETE_BATCH = 5_000\nconst MAX_LOGS_PER_SOURCE = 10_000\n\nfunction truncate(input: string | null | undefined, limit: number): string | null {\n  if (!input) return null\n  return input.length > limit ? `${input.slice(0, limit - 3)}...` : input\n}\n\nfunction safeJson(value: unknown): unknown {\n  if (value === undefined) return null\n  try {\n    return JSON.parse(JSON.stringify(value))\n  } catch {\n    if (value == null) return null\n    if (typeof value === 'object') {\n      return { note: 'unserializable', asString: String(value) }\n    }\n    return value\n  }\n}\n\nfunction pickDb(deps: RecordIndexerLogDeps): Kysely<any> | null {\n  if (deps.db) return deps.db\n  if (deps.em) {\n    try {\n      return deps.em.getKysely<any>()\n    } catch {\n      return null\n    }\n  }\n  return null\n}\n\nasync function pruneExcessLogs(db: Kysely<any>, source: IndexerErrorSource): Promise<void> {\n  const rows = await db\n    .selectFrom('indexer_status_logs' as any)\n    .select('id' as any)\n    .where('source' as any, '=', source)\n    .orderBy('occurred_at' as any, 'desc')\n    .orderBy('id' as any, 'desc')\n    .offset(MAX_LOGS_PER_SOURCE)\n    .limit(MAX_DELETE_BATCH)\n    .execute()\n\n  if (!rows.length) return\n  const ids = rows.map((row: any) => row.id).filter(Boolean)\n  if (!ids.length) return\n  await db\n    .deleteFrom('indexer_status_logs' as any)\n    .where('id' as any, 'in', ids)\n    .execute()\n}\n\nexport async function recordIndexerLog(\n  deps: RecordIndexerLogDeps,\n  input: RecordIndexerLogInput,\n): Promise<void> {\n  const db = pickDb(deps)\n  if (!db) {\n    console.warn('[indexers] Unable to record indexer log (missing db connection)', {\n      source: input.source,\n      handler: input.handler,\n    })\n    return\n  }\n\n  const level: IndexerLogLevel = input.level === 'warn' ? 'warn' : 'info'\n  const message = truncate(input.message, MAX_MESSAGE_LENGTH) ?? '\u2014'\n  const details = safeJson(input.details)\n  const occurredAt = new Date()\n\n  try {\n    await db\n      .insertInto('indexer_status_logs' as any)\n      .values({\n        source: input.source,\n        handler: input.handler,\n        level,\n        entity_type: input.entityType ?? null,\n        record_id: input.recordId ?? null,\n        tenant_id: input.tenantId ?? null,\n        organization_id: input.organizationId ?? null,\n        message,\n        details: details === null ? null : sql`${JSON.stringify(details)}::jsonb`,\n        occurred_at: occurredAt,\n      } as any)\n      .execute()\n  } catch (error) {\n    console.error('[indexers] Failed to persist indexer log', error)\n    return\n  }\n\n  try {\n    await pruneExcessLogs(db, input.source)\n  } catch (pruneError) {\n    console.warn('[indexers] Failed to prune indexer logs', pruneError)\n  }\n}\n"],
-  "mappings": "AACA,SAAsB,WAAW;AAsBjC,MAAM,qBAAqB;AAC3B,MAAM,mBAAmB;AACzB,MAAM,sBAAsB;AAE5B,SAAS,SAAS,OAAkC,OAA8B;AAChF,MAAI,CAAC,MAAO,QAAO;AACnB,SAAO,MAAM,SAAS,QAAQ,GAAG,MAAM,MAAM,GAAG,QAAQ,CAAC,CAAC,QAAQ;AACpE;AAEA,SAAS,SAAS,OAAyB;AACzC,MAAI,UAAU,OAAW,QAAO;AAChC,MAAI;AACF,WAAO,KAAK,MAAM,KAAK,UAAU,KAAK,CAAC;AAAA,EACzC,QAAQ;AACN,QAAI,SAAS,KAAM,QAAO;AAC1B,QAAI,OAAO,UAAU,UAAU;AAC7B,aAAO,EAAE,MAAM,kBAAkB,UAAU,OAAO,KAAK,EAAE;AAAA,IAC3D;AACA,WAAO;AAAA,EACT;AACF;AAEA,SAAS,OAAO,MAAgD;AAC9D,MAAI,KAAK,GAAI,QAAO,KAAK;AACzB,MAAI,KAAK,IAAI;AACX,QAAI;AACF,aAAO,KAAK,GAAG,UAAe;AAAA,IAChC,QAAQ;AACN,aAAO;AAAA,IACT;AAAA,EACF;AACA,SAAO;AACT;AAEA,eAAe,gBAAgB,IAAiB,QAA2C;AACzF,QAAM,OAAO,MAAM,GAChB,WAAW,qBAA4B,EACvC,OAAO,IAAW,EAClB,MAAM,UAAiB,KAAK,MAAM,EAClC,QAAQ,eAAsB,MAAM,EACpC,QAAQ,MAAa,MAAM,EAC3B,OAAO,mBAAmB,EAC1B,MAAM,gBAAgB,EACtB,QAAQ;AAEX,MAAI,CAAC,KAAK,OAAQ;AAClB,QAAM,MAAM,KAAK,IAAI,CAAC,QAAa,IAAI,EAAE,EAAE,OAAO,OAAO;AACzD,MAAI,CAAC,IAAI,OAAQ;AACjB,QAAM,GACH,WAAW,qBAA4B,EACvC,MAAM,MAAa,MAAM,GAAG,EAC5B,QAAQ;AACb;AAEA,eAAsB,iBACpB,MACA,OACe;AACf,QAAM,KAAK,OAAO,IAAI;AACtB,MAAI,CAAC,IAAI;AACP,YAAQ,KAAK,mEAAmE;AAAA,MAC9E,QAAQ,MAAM;AAAA,MACd,SAAS,MAAM;AAAA,IACjB,CAAC;AACD;AAAA,EACF;AAEA,QAAM,QAAyB,MAAM,UAAU,SAAS,SAAS;AACjE,QAAM,UAAU,SAAS,MAAM,SAAS,kBAAkB,KAAK;AAC/D,QAAM,UAAU,SAAS,MAAM,OAAO;AACtC,QAAM,aAAa,oBAAI,KAAK;AAE5B,MAAI;AACF,UAAM,GACH,WAAW,qBAA4B,EACvC,OAAO;AAAA,MACN,QAAQ,MAAM;AAAA,MACd,SAAS,MAAM;AAAA,MACf;AAAA,MACA,aAAa,MAAM,cAAc;AAAA,MACjC,WAAW,MAAM,YAAY;AAAA,MAC7B,WAAW,MAAM,YAAY;AAAA,MAC7B,iBAAiB,MAAM,kBAAkB;AAAA,MACzC;AAAA,MACA,SAAS,YAAY,OAAO,OAAO,MAAM,KAAK,UAAU,OAAO,CAAC;AAAA,MAChE,aAAa;AAAA,IACf,CAAQ,EACP,QAAQ;AAAA,EACb,SAAS,OAAO;AACd,YAAQ,MAAM,4CAA4C,KAAK;AAC/D;AAAA,EACF;AAEA,MAAI;AACF,UAAM,gBAAgB,IAAI,MAAM,MAAM;AAAA,EACxC,SAAS,YAAY;AACnB,YAAQ,KAAK,2CAA2C,UAAU;AAAA,EACpE;AACF;",
+  "sourcesContent": ["import type { EntityManager } from '@mikro-orm/postgresql'\nimport { type Kysely, sql } from 'kysely'\nimport type { IndexerErrorSource } from './error-log'\n\nexport type IndexerLogLevel = 'info' | 'warn'\n\nexport type RecordIndexerLogInput = {\n  source: IndexerErrorSource\n  handler: string\n  message: string\n  level?: IndexerLogLevel\n  entityType?: string | null\n  recordId?: string | null\n  tenantId?: string | null\n  organizationId?: string | null\n  details?: unknown\n}\n\ntype RecordIndexerLogDeps = {\n  em?: EntityManager\n  db?: Kysely<any>\n}\n\nconst MAX_MESSAGE_LENGTH = 4_096\nconst MAX_DELETE_BATCH = 5_000\nconst MAX_LOGS_PER_SOURCE = 10_000\n\nfunction truncate(input: string | null | undefined, limit: number): string | null {\n  if (!input) return null\n  return input.length > limit ? `${input.slice(0, limit - 3)}...` : input\n}\n\nfunction safeJson(value: unknown): unknown {\n  if (value === undefined) return null\n  try {\n    return JSON.parse(JSON.stringify(value))\n  } catch {\n    if (value == null) return null\n    if (typeof value === 'object') {\n      return { note: 'unserializable', asString: String(value) }\n    }\n    return value\n  }\n}\n\nfunction pickDb(deps: RecordIndexerLogDeps): Kysely<any> | null {\n  if (deps.db) return deps.db\n  if (deps.em) {\n    try {\n      return deps.em.getKysely<any>()\n    } catch {\n      return null\n    }\n  }\n  return null\n}\n\n/**\n * Indexer status logging is best-effort observability and must never ride on \u2014\n * or be noisy about \u2014 the caller's transaction lifecycle. When index maintenance\n * runs inline on a request `em` (e.g. an inline force reindex that emits the\n * vector-purge subscriber), the captured Kysely can be a transaction handle that\n * has already committed/rolled back, so a follow-up read/write throws\n * \"Transaction is already committed\". Treat that class of error as a quiet skip\n * rather than an error the operator must triage. A fresh-EM path (the events\n * worker) never hits this; this guard only de-noises the inline path.\n */\nfunction isInactiveTransactionError(error: unknown): boolean {\n  const message = error instanceof Error ? error.message : String(error ?? '')\n  return /transaction is already (committed|rolled back)/i.test(message)\n}\n\nasync function pruneExcessLogs(db: Kysely<any>, source: IndexerErrorSource): Promise<void> {\n  const rows = await db\n    .selectFrom('indexer_status_logs' as any)\n    .select('id' as any)\n    .where('source' as any, '=', source)\n    .orderBy('occurred_at' as any, 'desc')\n    .orderBy('id' as any, 'desc')\n    .offset(MAX_LOGS_PER_SOURCE)\n    .limit(MAX_DELETE_BATCH)\n    .execute()\n\n  if (!rows.length) return\n  const ids = rows.map((row: any) => row.id).filter(Boolean)\n  if (!ids.length) return\n  await db\n    .deleteFrom('indexer_status_logs' as any)\n    .where('id' as any, 'in', ids)\n    .execute()\n}\n\nexport async function recordIndexerLog(\n  deps: RecordIndexerLogDeps,\n  input: RecordIndexerLogInput,\n): Promise<void> {\n  const db = pickDb(deps)\n  if (!db) {\n    console.warn('[indexers] Unable to record indexer log (missing db connection)', {\n      source: input.source,\n      handler: input.handler,\n    })\n    return\n  }\n\n  const level: IndexerLogLevel = input.level === 'warn' ? 'warn' : 'info'\n  const message = truncate(input.message, MAX_MESSAGE_LENGTH) ?? '\u2014'\n  const details = safeJson(input.details)\n  const occurredAt = new Date()\n\n  try {\n    await db\n      .insertInto('indexer_status_logs' as any)\n      .values({\n        source: input.source,\n        handler: input.handler,\n        level,\n        entity_type: input.entityType ?? null,\n        record_id: input.recordId ?? null,\n        tenant_id: input.tenantId ?? null,\n        organization_id: input.organizationId ?? null,\n        message,\n        details: details === null ? null : sql`${JSON.stringify(details)}::jsonb`,\n        occurred_at: occurredAt,\n      } as any)\n      .execute()\n  } catch (error) {\n    // A committed/rolled-back caller transaction is an expected, harmless\n    // condition for best-effort logging \u2014 skip quietly instead of surfacing it.\n    if (!isInactiveTransactionError(error)) {\n      console.error('[indexers] Failed to persist indexer log', error)\n    }\n    return\n  }\n\n  try {\n    await pruneExcessLogs(db, input.source)\n  } catch (pruneError) {\n    if (!isInactiveTransactionError(pruneError)) {\n      console.warn('[indexers] Failed to prune indexer logs', pruneError)\n    }\n  }\n}\n"],
+  "mappings": "AACA,SAAsB,WAAW;AAsBjC,MAAM,qBAAqB;AAC3B,MAAM,mBAAmB;AACzB,MAAM,sBAAsB;AAE5B,SAAS,SAAS,OAAkC,OAA8B;AAChF,MAAI,CAAC,MAAO,QAAO;AACnB,SAAO,MAAM,SAAS,QAAQ,GAAG,MAAM,MAAM,GAAG,QAAQ,CAAC,CAAC,QAAQ;AACpE;AAEA,SAAS,SAAS,OAAyB;AACzC,MAAI,UAAU,OAAW,QAAO;AAChC,MAAI;AACF,WAAO,KAAK,MAAM,KAAK,UAAU,KAAK,CAAC;AAAA,EACzC,QAAQ;AACN,QAAI,SAAS,KAAM,QAAO;AAC1B,QAAI,OAAO,UAAU,UAAU;AAC7B,aAAO,EAAE,MAAM,kBAAkB,UAAU,OAAO,KAAK,EAAE;AAAA,IAC3D;AACA,WAAO;AAAA,EACT;AACF;AAEA,SAAS,OAAO,MAAgD;AAC9D,MAAI,KAAK,GAAI,QAAO,KAAK;AACzB,MAAI,KAAK,IAAI;AACX,QAAI;AACF,aAAO,KAAK,GAAG,UAAe;AAAA,IAChC,QAAQ;AACN,aAAO;AAAA,IACT;AAAA,EACF;AACA,SAAO;AACT;AAYA,SAAS,2BAA2B,OAAyB;AAC3D,QAAM,UAAU,iBAAiB,QAAQ,MAAM,UAAU,OAAO,SAAS,EAAE;AAC3E,SAAO,kDAAkD,KAAK,OAAO;AACvE;AAEA,eAAe,gBAAgB,IAAiB,QAA2C;AACzF,QAAM,OAAO,MAAM,GAChB,WAAW,qBAA4B,EACvC,OAAO,IAAW,EAClB,MAAM,UAAiB,KAAK,MAAM,EAClC,QAAQ,eAAsB,MAAM,EACpC,QAAQ,MAAa,MAAM,EAC3B,OAAO,mBAAmB,EAC1B,MAAM,gBAAgB,EACtB,QAAQ;AAEX,MAAI,CAAC,KAAK,OAAQ;AAClB,QAAM,MAAM,KAAK,IAAI,CAAC,QAAa,IAAI,EAAE,EAAE,OAAO,OAAO;AACzD,MAAI,CAAC,IAAI,OAAQ;AACjB,QAAM,GACH,WAAW,qBAA4B,EACvC,MAAM,MAAa,MAAM,GAAG,EAC5B,QAAQ;AACb;AAEA,eAAsB,iBACpB,MACA,OACe;AACf,QAAM,KAAK,OAAO,IAAI;AACtB,MAAI,CAAC,IAAI;AACP,YAAQ,KAAK,mEAAmE;AAAA,MAC9E,QAAQ,MAAM;AAAA,MACd,SAAS,MAAM;AAAA,IACjB,CAAC;AACD;AAAA,EACF;AAEA,QAAM,QAAyB,MAAM,UAAU,SAAS,SAAS;AACjE,QAAM,UAAU,SAAS,MAAM,SAAS,kBAAkB,KAAK;AAC/D,QAAM,UAAU,SAAS,MAAM,OAAO;AACtC,QAAM,aAAa,oBAAI,KAAK;AAE5B,MAAI;AACF,UAAM,GACH,WAAW,qBAA4B,EACvC,OAAO;AAAA,MACN,QAAQ,MAAM;AAAA,MACd,SAAS,MAAM;AAAA,MACf;AAAA,MACA,aAAa,MAAM,cAAc;AAAA,MACjC,WAAW,MAAM,YAAY;AAAA,MAC7B,WAAW,MAAM,YAAY;AAAA,MAC7B,iBAAiB,MAAM,kBAAkB;AAAA,MACzC;AAAA,MACA,SAAS,YAAY,OAAO,OAAO,MAAM,KAAK,UAAU,OAAO,CAAC;AAAA,MAChE,aAAa;AAAA,IACf,CAAQ,EACP,QAAQ;AAAA,EACb,SAAS,OAAO;AAGd,QAAI,CAAC,2BAA2B,KAAK,GAAG;AACtC,cAAQ,MAAM,4CAA4C,KAAK;AAAA,IACjE;AACA;AAAA,EACF;AAEA,MAAI;AACF,UAAM,gBAAgB,IAAI,MAAM,MAAM;AAAA,EACxC,SAAS,YAAY;AACnB,QAAI,CAAC,2BAA2B,UAAU,GAAG;AAC3C,cAAQ,KAAK,2CAA2C,UAAU;AAAA,IACpE;AAAA,EACF;AACF;",
   "names": []
 }

package/dist/lib/version.js

@@ -1,4 +1,4 @@
-const APP_VERSION = "0.6.6-develop.5586.1.c9ed1d68a8";
+const APP_VERSION = "0.6.6-develop.5594.1.30cd738303";
 const appVersion = APP_VERSION;
 export {
   APP_VERSION,

package/dist/lib/version.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../src/lib/version.ts"],
-  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.6-develop.5586.1.c9ed1d68a8'\nexport const appVersion = APP_VERSION\n"],
+  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.6-develop.5594.1.30cd738303'\nexport const appVersion = APP_VERSION\n"],
   "mappings": "AACO,MAAM,cAAc;AACpB,MAAM,aAAa;",
   "names": []
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-mercato/shared",
-  "version": "0.6.6-develop.5586.1.c9ed1d68a8",
+  "version": "0.6.6-develop.5594.1.30cd738303",
   "type": "module",
   "main": "./dist/index.js",
   "scripts": {
@@ -92,7 +92,7 @@
     "@mikro-orm/core": "^7.1.4",
     "@mikro-orm/decorators": "^7.1.4",
     "@mikro-orm/postgresql": "^7.1.4",
-    "@open-mercato/cache": "0.6.6-develop.5586.1.c9ed1d68a8",
+    "@open-mercato/cache": "0.6.6-develop.5594.1.30cd738303",
     "dotenv": "^17.4.2",
     "rate-limiter-flexible": "^11.2.0",
     "re2js": "2.8.3",

package/src/lib/indexers/__tests__/status-log.test.ts

@@ -0,0 +1,94 @@
+import { recordIndexerLog } from '../status-log'
+
+type Behaviors = {
+  insert?: () => unknown
+  select?: () => unknown
+  delete?: () => unknown
+}
+
+function makeBuilder(op: keyof Behaviors, behaviors: Behaviors): any {
+  const builder: any = {}
+  for (const method of ['values', 'select', 'where', 'orderBy', 'offset', 'limit']) {
+    builder[method] = () => builder
+  }
+  builder.execute = async () => {
+    const behavior = behaviors[op]
+    return behavior ? behavior() : []
+  }
+  return builder
+}
+
+function makeDb(behaviors: Behaviors): any {
+  return {
+    insertInto: () => makeBuilder('insert', behaviors),
+    selectFrom: () => makeBuilder('select', behaviors),
+    deleteFrom: () => makeBuilder('delete', behaviors),
+  }
+}
+
+const INPUT = { source: 'vector' as const, handler: 'event:test', message: 'hi' }
+
+describe('recordIndexerLog — inactive-transaction de-noising', () => {
+  let errorSpy: jest.SpyInstance
+  let warnSpy: jest.SpyInstance
+
+  beforeEach(() => {
+    errorSpy = jest.spyOn(console, 'error').mockImplementation(() => {})
+    warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => {})
+  })
+
+  afterEach(() => {
+    errorSpy.mockRestore()
+    warnSpy.mockRestore()
+  })
+
+  it('skips quietly when the insert hits an already-committed transaction', async () => {
+    const db = makeDb({
+      insert: () => {
+        throw new Error('Transaction is already committed')
+      },
+    })
+
+    await recordIndexerLog({ db }, INPUT)
+
+    expect(errorSpy).not.toHaveBeenCalled()
+  })
+
+  it('still logs an unexpected insert failure', async () => {
+    const db = makeDb({
+      insert: () => {
+        throw new Error('connection refused')
+      },
+    })
+
+    await recordIndexerLog({ db }, INPUT)
+
+    expect(errorSpy).toHaveBeenCalledTimes(1)
+  })
+
+  it('skips quietly when the prune hits a rolled-back transaction', async () => {
+    const db = makeDb({
+      insert: () => undefined,
+      select: () => {
+        throw new Error('Transaction is already rolled back')
+      },
+    })
+
+    await recordIndexerLog({ db }, INPUT)
+
+    expect(warnSpy).not.toHaveBeenCalled()
+  })
+
+  it('still warns on an unexpected prune failure', async () => {
+    const db = makeDb({
+      insert: () => undefined,
+      select: () => {
+        throw new Error('deadlock detected')
+      },
+    })
+
+    await recordIndexerLog({ db }, INPUT)
+
+    expect(warnSpy).toHaveBeenCalledTimes(1)
+  })
+})

package/src/lib/indexers/status-log.ts

@@ -55,6 +55,21 @@ function pickDb(deps: RecordIndexerLogDeps): Kysely<any> | null {
   return null
 }
 
+/**
+ * Indexer status logging is best-effort observability and must never ride on —
+ * or be noisy about — the caller's transaction lifecycle. When index maintenance
+ * runs inline on a request `em` (e.g. an inline force reindex that emits the
+ * vector-purge subscriber), the captured Kysely can be a transaction handle that
+ * has already committed/rolled back, so a follow-up read/write throws
+ * "Transaction is already committed". Treat that class of error as a quiet skip
+ * rather than an error the operator must triage. A fresh-EM path (the events
+ * worker) never hits this; this guard only de-noises the inline path.
+ */
+function isInactiveTransactionError(error: unknown): boolean {
+  const message = error instanceof Error ? error.message : String(error ?? '')
+  return /transaction is already (committed|rolled back)/i.test(message)
+}
+
 async function pruneExcessLogs(db: Kysely<any>, source: IndexerErrorSource): Promise<void> {
   const rows = await db
     .selectFrom('indexer_status_logs' as any)
@@ -110,13 +125,19 @@ export async function recordIndexerLog(
       } as any)
       .execute()
   } catch (error) {
-    console.error('[indexers] Failed to persist indexer log', error)
+    // A committed/rolled-back caller transaction is an expected, harmless
+    // condition for best-effort logging — skip quietly instead of surfacing it.
+    if (!isInactiveTransactionError(error)) {
+      console.error('[indexers] Failed to persist indexer log', error)
+    }
     return
   }
 
   try {
     await pruneExcessLogs(db, input.source)
   } catch (pruneError) {
-    console.warn('[indexers] Failed to prune indexer logs', pruneError)
+    if (!isInactiveTransactionError(pruneError)) {
+      console.warn('[indexers] Failed to prune indexer logs', pruneError)
+    }
   }
 }