@owlmeans/postgres-resource 0.1.18-rc.41 → 0.1.18-rc.42

package/src/utils/types.ts

@@ -0,0 +1,19 @@
+import type { PgRuntimeTable, TableSpec } from '../types.js'
+import type { MigrationReport } from '@owlmeans/resource'
+import type { BasicContext } from '@owlmeans/context'
+
+export interface TableInit {
+  spec: TableSpec
+  /** The Drizzle table CRUD is built against. */
+  entity: PgRuntimeTable
+  reports: MigrationReport[]
+}
+
+/**
+ * The context a placeholder resolves through.
+ *
+ * Resolution reads exactly one thing — `context.resource(alias).table` — so the config a context
+ * carries is irrelevant here, and pinning one would force every caller holding a server context
+ * to cast its way in. Deliberately open, and named so callers pass what they already have.
+ */
+export type PlaceholderContext = BasicContext<any>

package/tests/diff.spec.ts

@@ -109,3 +109,27 @@ describe('@owlmeans/postgres-resource — comparing a declared definition with t
     }
   })
 })
+
+describe('@owlmeans/postgres-resource — NOT NULL constraints of Postgres 18', () => {
+  test('a not-null constraint row is never planned for a drop', () => {
+    // From 18 every NOT NULL is a pg_constraint row named `<table>_<column>_not_null`. The platform
+    // names its own constraints after the table too, so it took these for its own leftovers and sent
+    // `DROP CONSTRAINT …_id_not_null` — which Postgres refuses on a primary key column.
+    const live: LiveTable = {
+      exists: true,
+      columns: [
+        columnOf({ name: 'id', type: 'text' }),
+        columnOf({ name: 'title', type: 'text', notNull: false }),
+      ],
+      indexes: [],
+      constraints: [
+        { name: 'tasks_id_not_null', type: 'n', definition: 'NOT NULL id' },
+        { name: 'task_entry_id_not_null', type: 'n', definition: 'NOT NULL id' },
+      ],
+    }
+
+    const dropped = planSync(specOf('string'), live).statements.filter(statement => statement.kind === 'drop-constraint')
+    expect(dropped).toEqual([])
+  })
+})
+