@open-mercato/core 0.6.3-develop.3586.1.d5a44c810f → 0.6.3-develop.3593.1.8eda8943fa

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
-[build:core] found 2553 entry points
+[build:core] found 2554 entry points
 [build:core] built successfully
 [build:core:generated] found 172 entry points
 [build:core:generated] built successfully

package/dist/modules/customers/lib/personCompanyLinkTable.js

@@ -1,35 +1,9 @@
-import { sql } from "@mikro-orm/postgresql";
-const PERSON_COMPANY_LINKS_TABLE = "customer_person_company_links";
-const PERSON_COMPANY_LINKS_DELETED_AT_COLUMN = "deleted_at";
-let supportsDeletedAtColumnPromise = null;
-let warnedAboutMissingDeletedAtColumn = false;
-async function customerPersonCompanyLinksSupportDeletedAt(em) {
-  if (typeof em.getKysely !== "function") {
-    return true;
-  }
-  if (!supportsDeletedAtColumnPromise) {
-    const db = em.getKysely();
-    const probe = db.selectFrom("information_schema.columns").select(["column_name"]).where(sql`table_schema = current_schema()`).where("table_name", "=", PERSON_COMPANY_LINKS_TABLE).where("column_name", "=", PERSON_COMPANY_LINKS_DELETED_AT_COLUMN).executeTakeFirst().then((row) => !!row).catch(() => false);
-    supportsDeletedAtColumnPromise = probe;
-    return probe;
-  }
-  return supportsDeletedAtColumnPromise;
+async function customerPersonCompanyLinksSupportDeletedAt(_em) {
+  return true;
 }
-function warnMissingCustomerPersonCompanyLinksDeletedAt(source) {
-  if (warnedAboutMissingDeletedAtColumn) {
-    return;
-  }
-  warnedAboutMissingDeletedAtColumn = true;
-  console.warn(
-    `[${source}] missing ${PERSON_COMPANY_LINKS_TABLE}.${PERSON_COMPANY_LINKS_DELETED_AT_COLUMN}; continuing without link soft-delete filtering. Run yarn db:migrate.`
-  );
+function warnMissingCustomerPersonCompanyLinksDeletedAt(_source) {
 }
-async function withActiveCustomerPersonCompanyLinkFilter(em, where, source) {
-  const supportsDeletedAt = await customerPersonCompanyLinksSupportDeletedAt(em);
-  if (!supportsDeletedAt) {
-    warnMissingCustomerPersonCompanyLinksDeletedAt(source);
-    return { ...where };
-  }
+async function withActiveCustomerPersonCompanyLinkFilter(_em, where, _source) {
   return { ...where, deletedAt: null };
 }
 function filterActivePersonCompanyLinks(links) {

package/dist/modules/customers/lib/personCompanyLinkTable.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../../src/modules/customers/lib/personCompanyLinkTable.ts"],
-  "sourcesContent": ["import type { EntityManager } from '@mikro-orm/postgresql'\nimport { sql } from '@mikro-orm/postgresql'\n\nconst PERSON_COMPANY_LINKS_TABLE = 'customer_person_company_links'\nconst PERSON_COMPANY_LINKS_DELETED_AT_COLUMN = 'deleted_at'\n\nlet supportsDeletedAtColumnPromise: Promise<boolean> | null = null\nlet warnedAboutMissingDeletedAtColumn = false\n\nexport async function customerPersonCompanyLinksSupportDeletedAt(em: EntityManager): Promise<boolean> {\n  if (typeof (em as { getKysely?: unknown }).getKysely !== 'function') {\n    return true\n  }\n  if (!supportsDeletedAtColumnPromise) {\n    const db = em.getKysely<any>() as any\n    const probe: Promise<boolean> = db\n      .selectFrom('information_schema.columns')\n      .select(['column_name'])\n      .where(sql<boolean>`table_schema = current_schema()`)\n      .where('table_name', '=', PERSON_COMPANY_LINKS_TABLE)\n      .where('column_name', '=', PERSON_COMPANY_LINKS_DELETED_AT_COLUMN)\n      .executeTakeFirst()\n      .then((row: unknown) => !!row)\n      .catch(() => false)\n    supportsDeletedAtColumnPromise = probe\n    return probe\n  }\n  return supportsDeletedAtColumnPromise\n}\n\nexport function warnMissingCustomerPersonCompanyLinksDeletedAt(source: string): void {\n  if (warnedAboutMissingDeletedAtColumn) {\n    return\n  }\n  warnedAboutMissingDeletedAtColumn = true\n  console.warn(\n    `[${source}] missing ${PERSON_COMPANY_LINKS_TABLE}.${PERSON_COMPANY_LINKS_DELETED_AT_COLUMN}; ` +\n      'continuing without link soft-delete filtering. Run yarn db:migrate.',\n  )\n}\n\nexport async function withActiveCustomerPersonCompanyLinkFilter<T extends Record<string, unknown>>(\n  em: EntityManager,\n  where: T,\n  source: string,\n): Promise<T & { deletedAt?: null }> {\n  const supportsDeletedAt = await customerPersonCompanyLinksSupportDeletedAt(em)\n  if (!supportsDeletedAt) {\n    warnMissingCustomerPersonCompanyLinksDeletedAt(source)\n    return { ...where }\n  }\n  return { ...where, deletedAt: null }\n}\n\n/**\n * Drop soft-deleted link rows from a result set as a defense-in-depth fallback.\n * MikroORM has historically dropped `deletedAt: null` from the WHERE clause for\n * nullable date columns under certain configurations, so callers SHOULD apply this\n * after `findWithDecryption(...)` until the upstream query filter is verified to\n * fully cover all callers.\n */\nexport function filterActivePersonCompanyLinks<T extends { deletedAt?: Date | string | null | undefined }>(\n  links: T[] | null | undefined,\n): T[] {\n  if (!Array.isArray(links)) return []\n  return links.filter((entry) => entry?.deletedAt == null)\n}\n"],
-  "mappings": "AACA,SAAS,WAAW;AAEpB,MAAM,6BAA6B;AACnC,MAAM,yCAAyC;AAE/C,IAAI,iCAA0D;AAC9D,IAAI,oCAAoC;AAExC,eAAsB,2CAA2C,IAAqC;AACpG,MAAI,OAAQ,GAA+B,cAAc,YAAY;AACnE,WAAO;AAAA,EACT;AACA,MAAI,CAAC,gCAAgC;AACnC,UAAM,KAAK,GAAG,UAAe;AAC7B,UAAM,QAA0B,GAC7B,WAAW,4BAA4B,EACvC,OAAO,CAAC,aAAa,CAAC,EACtB,MAAM,oCAA6C,EACnD,MAAM,cAAc,KAAK,0BAA0B,EACnD,MAAM,eAAe,KAAK,sCAAsC,EAChE,iBAAiB,EACjB,KAAK,CAAC,QAAiB,CAAC,CAAC,GAAG,EAC5B,MAAM,MAAM,KAAK;AACpB,qCAAiC;AACjC,WAAO;AAAA,EACT;AACA,SAAO;AACT;AAEO,SAAS,+CAA+C,QAAsB;AACnF,MAAI,mCAAmC;AACrC;AAAA,EACF;AACA,sCAAoC;AACpC,UAAQ;AAAA,IACN,IAAI,MAAM,aAAa,0BAA0B,IAAI,sCAAsC;AAAA,EAE7F;AACF;AAEA,eAAsB,0CACpB,IACA,OACA,QACmC;AACnC,QAAM,oBAAoB,MAAM,2CAA2C,EAAE;AAC7E,MAAI,CAAC,mBAAmB;AACtB,mDAA+C,MAAM;AACrD,WAAO,EAAE,GAAG,MAAM;AAAA,EACpB;AACA,SAAO,EAAE,GAAG,OAAO,WAAW,KAAK;AACrC;AASO,SAAS,+BACd,OACK;AACL,MAAI,CAAC,MAAM,QAAQ,KAAK,EAAG,QAAO,CAAC;AACnC,SAAO,MAAM,OAAO,CAAC,UAAU,OAAO,aAAa,IAAI;AACzD;",
+  "sourcesContent": ["import type { EntityManager } from '@mikro-orm/postgresql'\n\n// The previous implementation probed `information_schema.columns` on every\n// first call via `em.getKysely()` to decide whether to add a `deletedAt: null`\n// filter. When that probe ran inside an already-active MikroORM transaction it\n// shared the transaction's connection \u2014 and any failure (including transient\n// connection-pool state, an aborted TX from a sibling query, or pg returning\n// 25P02) propagated back into the host transaction, poisoning every following\n// query with `current transaction is aborted, commands ignored until end of\n// transaction block`. That is what produced the standalone-CI failure burst\n// (loadPersonCompanyLinks \u2192 25P02 \u2192 customers.people.create \u2192 500) and the\n// sync_excel worker `failedCount: 1` regression.\n//\n// The `deleted_at` column on `customer_person_company_links` has been part of\n// the committed module snapshot since `Migration20260415095203` (April 2026)\n// and is referenced by `Migration20260417140000` (partial unique index uses\n// `where deleted_at is null`). Any environment running these migrations\n// already has the column. So the probe is no longer load-bearing \u2014 it only\n// adds risk. We now unconditionally include `deletedAt: null` in the filter.\n// If a downstream DB is genuinely un-migrated, the underlying `em.find` will\n// surface the authentic `column \"deleted_at\" of relation\n// \"customer_person_company_links\" does not exist` (pg 42703) pointing the\n// operator at the real fix (`yarn db:migrate`).\n//\n// `customerPersonCompanyLinksSupportDeletedAt` and\n// `warnMissingCustomerPersonCompanyLinksDeletedAt` are kept as exported\n// no-op-equivalents so external callers (and the published @open-mercato/core\n// API surface) don't break.\n\nexport async function customerPersonCompanyLinksSupportDeletedAt(_em: EntityManager): Promise<boolean> {\n  return true\n}\n\nexport function warnMissingCustomerPersonCompanyLinksDeletedAt(_source: string): void {\n  // No-op: column is guaranteed present by Migration20260415095203 and the\n  // module snapshot. Kept for backward-compatible exports.\n}\n\nexport async function withActiveCustomerPersonCompanyLinkFilter<T extends Record<string, unknown>>(\n  _em: EntityManager,\n  where: T,\n  _source: string,\n): Promise<T & { deletedAt?: null }> {\n  return { ...where, deletedAt: null }\n}\n\n/**\n * Drop soft-deleted link rows from a result set as a defense-in-depth fallback.\n * MikroORM has historically dropped `deletedAt: null` from the WHERE clause for\n * nullable date columns under certain configurations, so callers SHOULD apply this\n * after `findWithDecryption(...)` until the upstream query filter is verified to\n * fully cover all callers.\n */\nexport function filterActivePersonCompanyLinks<T extends { deletedAt?: Date | string | null | undefined }>(\n  links: T[] | null | undefined,\n): T[] {\n  if (!Array.isArray(links)) return []\n  return links.filter((entry) => entry?.deletedAt == null)\n}\n"],
+  "mappings": "AA6BA,eAAsB,2CAA2C,KAAsC;AACrG,SAAO;AACT;AAEO,SAAS,+CAA+C,SAAuB;AAGtF;AAEA,eAAsB,0CACpB,KACA,OACA,SACmC;AACnC,SAAO,EAAE,GAAG,OAAO,WAAW,KAAK;AACrC;AASO,SAAS,+BACd,OACK;AACL,MAAI,CAAC,MAAM,QAAQ,KAAK,EAAG,QAAO,CAAC;AACnC,SAAO,MAAM,OAAO,CAAC,UAAU,OAAO,aAAa,IAAI;AACzD;",
   "names": []
 }

package/dist/modules/customers/migrations/Migration20260521175027.js

@@ -0,0 +1,12 @@
+import { Migration } from "@mikro-orm/migrations";
+class Migration20260521175027 extends Migration {
+  async up() {
+    this.addSql(`alter table "customer_person_company_links" add column if not exists "deleted_at" timestamptz null;`);
+  }
+  async down() {
+  }
+}
+export {
+  Migration20260521175027
+};
+//# sourceMappingURL=Migration20260521175027.js.map

package/dist/modules/customers/migrations/Migration20260521175027.js.map

@@ -0,0 +1,7 @@
+{
+  "version": 3,
+  "sources": ["../../../../src/modules/customers/migrations/Migration20260521175027.ts"],
+  "sourcesContent": ["import { Migration } from '@mikro-orm/migrations';\n\n/**\n * Idempotent backstop for `customer_person_company_links.deleted_at`.\n *\n * The column was originally added by `Migration20260415095203` and is\n * referenced by `Migration20260417140000`'s partial unique index\n * (`where \"deleted_at\" is null`). It is part of the committed\n * `migrations/.snapshot-open-mercato.json`.\n *\n * However, the runtime defensive probe that lived in\n * `packages/core/src/modules/customers/lib/personCompanyLinkTable.ts`\n * historically existed because an aborted-transaction state on a\n * shared connection could (very rarely) make `loadPersonCompanyLinks`\n * fail in a way that *looked* like the column was missing \u2014 and the\n * cached fallback then silently dropped the `deletedAt: null` filter.\n * That probe is removed in the same change as this migration (no\n * Kysely query is ever issued from inside an active MikroORM TX);\n * this migration is the matching belt-and-suspenders guarantee that\n * the column is physically present so the unconditional `deletedAt:\n * null` filter cannot 42703.\n *\n * Postgres-native `add column if not exists` is used so this re-applies\n * cleanly on already-migrated databases without raising.\n */\nexport class Migration20260521175027 extends Migration {\n\n  override async up(): Promise<void> {\n    this.addSql(`alter table \"customer_person_company_links\" add column if not exists \"deleted_at\" timestamptz null;`);\n  }\n\n  override async down(): Promise<void> {\n    // No-op on the way down: dropping the column would break\n    // `Migration20260417140000`'s partial unique index, which is the\n    // authoritative owner of the column going forward. The original\n    // `Migration20260415095203` down still drops it for full rollback.\n  }\n\n}\n"],
+  "mappings": "AAAA,SAAS,iBAAiB;AAyBnB,MAAM,gCAAgC,UAAU;AAAA,EAErD,MAAe,KAAoB;AACjC,SAAK,OAAO,qGAAqG;AAAA,EACnH;AAAA,EAEA,MAAe,OAAsB;AAAA,EAKrC;AAEF;",
+  "names": []
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-mercato/core",
-  "version": "0.6.3-develop.3586.1.d5a44c810f",
+  "version": "0.6.3-develop.3593.1.8eda8943fa",
   "type": "module",
   "main": "./dist/index.js",
   "scripts": {
@@ -242,16 +242,16 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
-    "@open-mercato/ai-assistant": "0.6.3-develop.3586.1.d5a44c810f",
-    "@open-mercato/shared": "0.6.3-develop.3586.1.d5a44c810f",
-    "@open-mercato/ui": "0.6.3-develop.3586.1.d5a44c810f",
+    "@open-mercato/ai-assistant": "0.6.3-develop.3593.1.8eda8943fa",
+    "@open-mercato/shared": "0.6.3-develop.3593.1.8eda8943fa",
+    "@open-mercato/ui": "0.6.3-develop.3593.1.8eda8943fa",
     "react": "^19.0.0",
     "react-dom": "^19.0.0"
   },
   "devDependencies": {
-    "@open-mercato/ai-assistant": "0.6.3-develop.3586.1.d5a44c810f",
-    "@open-mercato/shared": "0.6.3-develop.3586.1.d5a44c810f",
-    "@open-mercato/ui": "0.6.3-develop.3586.1.d5a44c810f",
+    "@open-mercato/ai-assistant": "0.6.3-develop.3593.1.8eda8943fa",
+    "@open-mercato/shared": "0.6.3-develop.3593.1.8eda8943fa",
+    "@open-mercato/ui": "0.6.3-develop.3593.1.8eda8943fa",
     "@testing-library/dom": "^10.4.1",
     "@testing-library/jest-dom": "^6.9.1",
     "@testing-library/react": "^16.3.1",

package/src/modules/customers/lib/personCompanyLinkTable.ts

@@ -1,54 +1,46 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
-import { sql } from '@mikro-orm/postgresql'
 
-const PERSON_COMPANY_LINKS_TABLE = 'customer_person_company_links'
-const PERSON_COMPANY_LINKS_DELETED_AT_COLUMN = 'deleted_at'
+// The previous implementation probed `information_schema.columns` on every
+// first call via `em.getKysely()` to decide whether to add a `deletedAt: null`
+// filter. When that probe ran inside an already-active MikroORM transaction it
+// shared the transaction's connection — and any failure (including transient
+// connection-pool state, an aborted TX from a sibling query, or pg returning
+// 25P02) propagated back into the host transaction, poisoning every following
+// query with `current transaction is aborted, commands ignored until end of
+// transaction block`. That is what produced the standalone-CI failure burst
+// (loadPersonCompanyLinks → 25P02 → customers.people.create → 500) and the
+// sync_excel worker `failedCount: 1` regression.
+//
+// The `deleted_at` column on `customer_person_company_links` has been part of
+// the committed module snapshot since `Migration20260415095203` (April 2026)
+// and is referenced by `Migration20260417140000` (partial unique index uses
+// `where deleted_at is null`). Any environment running these migrations
+// already has the column. So the probe is no longer load-bearing — it only
+// adds risk. We now unconditionally include `deletedAt: null` in the filter.
+// If a downstream DB is genuinely un-migrated, the underlying `em.find` will
+// surface the authentic `column "deleted_at" of relation
+// "customer_person_company_links" does not exist` (pg 42703) pointing the
+// operator at the real fix (`yarn db:migrate`).
+//
+// `customerPersonCompanyLinksSupportDeletedAt` and
+// `warnMissingCustomerPersonCompanyLinksDeletedAt` are kept as exported
+// no-op-equivalents so external callers (and the published @open-mercato/core
+// API surface) don't break.
 
-let supportsDeletedAtColumnPromise: Promise<boolean> | null = null
-let warnedAboutMissingDeletedAtColumn = false
-
-export async function customerPersonCompanyLinksSupportDeletedAt(em: EntityManager): Promise<boolean> {
-  if (typeof (em as { getKysely?: unknown }).getKysely !== 'function') {
-    return true
-  }
-  if (!supportsDeletedAtColumnPromise) {
-    const db = em.getKysely<any>() as any
-    const probe: Promise<boolean> = db
-      .selectFrom('information_schema.columns')
-      .select(['column_name'])
-      .where(sql<boolean>`table_schema = current_schema()`)
-      .where('table_name', '=', PERSON_COMPANY_LINKS_TABLE)
-      .where('column_name', '=', PERSON_COMPANY_LINKS_DELETED_AT_COLUMN)
-      .executeTakeFirst()
-      .then((row: unknown) => !!row)
-      .catch(() => false)
-    supportsDeletedAtColumnPromise = probe
-    return probe
-  }
-  return supportsDeletedAtColumnPromise
+export async function customerPersonCompanyLinksSupportDeletedAt(_em: EntityManager): Promise<boolean> {
+  return true
 }
 
-export function warnMissingCustomerPersonCompanyLinksDeletedAt(source: string): void {
-  if (warnedAboutMissingDeletedAtColumn) {
-    return
-  }
-  warnedAboutMissingDeletedAtColumn = true
-  console.warn(
-    `[${source}] missing ${PERSON_COMPANY_LINKS_TABLE}.${PERSON_COMPANY_LINKS_DELETED_AT_COLUMN}; ` +
-      'continuing without link soft-delete filtering. Run yarn db:migrate.',
-  )
+export function warnMissingCustomerPersonCompanyLinksDeletedAt(_source: string): void {
+  // No-op: column is guaranteed present by Migration20260415095203 and the
+  // module snapshot. Kept for backward-compatible exports.
 }
 
 export async function withActiveCustomerPersonCompanyLinkFilter<T extends Record<string, unknown>>(
-  em: EntityManager,
+  _em: EntityManager,
   where: T,
-  source: string,
+  _source: string,
 ): Promise<T & { deletedAt?: null }> {
-  const supportsDeletedAt = await customerPersonCompanyLinksSupportDeletedAt(em)
-  if (!supportsDeletedAt) {
-    warnMissingCustomerPersonCompanyLinksDeletedAt(source)
-    return { ...where }
-  }
   return { ...where, deletedAt: null }
 }
 

package/src/modules/customers/migrations/Migration20260521175027.ts

@@ -0,0 +1,39 @@
+import { Migration } from '@mikro-orm/migrations';
+
+/**
+ * Idempotent backstop for `customer_person_company_links.deleted_at`.
+ *
+ * The column was originally added by `Migration20260415095203` and is
+ * referenced by `Migration20260417140000`'s partial unique index
+ * (`where "deleted_at" is null`). It is part of the committed
+ * `migrations/.snapshot-open-mercato.json`.
+ *
+ * However, the runtime defensive probe that lived in
+ * `packages/core/src/modules/customers/lib/personCompanyLinkTable.ts`
+ * historically existed because an aborted-transaction state on a
+ * shared connection could (very rarely) make `loadPersonCompanyLinks`
+ * fail in a way that *looked* like the column was missing — and the
+ * cached fallback then silently dropped the `deletedAt: null` filter.
+ * That probe is removed in the same change as this migration (no
+ * Kysely query is ever issued from inside an active MikroORM TX);
+ * this migration is the matching belt-and-suspenders guarantee that
+ * the column is physically present so the unconditional `deletedAt:
+ * null` filter cannot 42703.
+ *
+ * Postgres-native `add column if not exists` is used so this re-applies
+ * cleanly on already-migrated databases without raising.
+ */
+export class Migration20260521175027 extends Migration {
+
+  override async up(): Promise<void> {
+    this.addSql(`alter table "customer_person_company_links" add column if not exists "deleted_at" timestamptz null;`);
+  }
+
+  override async down(): Promise<void> {
+    // No-op on the way down: dropping the column would break
+    // `Migration20260417140000`'s partial unique index, which is the
+    // authoritative owner of the column going forward. The original
+    // `Migration20260415095203` down still drops it for full rollback.
+  }
+
+}