@fragno-dev/db 0.1.14 → 0.2.0

package/src/adapters/kysely/migration/kysely-migrator.test.ts

@@ -1,261 +0,0 @@
-import { Kysely, PostgresDialect, SqliteDialect } from "kysely";
-import { describe, expect, beforeAll, test } from "vitest";
-import { KyselyAdapter } from "../kysely-adapter";
-import { column, idColumn, referenceColumn, schema } from "../../../schema/create";
-
-describe("KyselyMigrator", () => {
-  const testSchema = schema((s) => {
-    return s
-      .addTable("users", (t) => {
-        return t.addColumn("id", idColumn()).addColumn("name", column("string"));
-      })
-      .alterTable("users", (t) => {
-        return t
-          .addColumn("age", column("integer").nullable())
-          .createIndex("name_idx", ["name"])
-          .createIndex("age_idx", ["age"]);
-      })
-      .addTable("posts", (t) => {
-        return t
-          .addColumn("id", idColumn())
-          .addColumn("title", column("string"))
-          .addColumn("user_id", column("string"));
-      })
-      .addReference("author", {
-        type: "one",
-        from: { table: "posts", column: "user_id" },
-        to: { table: "users", column: "id" },
-      });
-  });
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  let db: Kysely<any>;
-  let adapter: KyselyAdapter;
-
-  beforeAll(async () => {
-    // Create a Kysely instance with a PostgresDialect, but not actually connected to a database
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    db = new Kysely({ dialect: new PostgresDialect({} as any) });
-    adapter = new KyselyAdapter({ db, provider: "postgresql" });
-  });
-
-  test("generate SQL for migration 0 -> 1 (create users table)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(1, {
-      updateSettings: true,
-      fromVersion: 0,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toMatchInlineSnapshot(`
-      "create table "users_test_namespace" ("id" varchar(30) not null unique, "name" text not null, "_internalId" bigserial not null primary key, "_version" integer default 0 not null);
-
-      insert into "fragno_db_settings" ("id", "key", "value") values ('jprP_43K5uMwxAFiepbbrQ', 'test_namespace.schema_version', '1');"
-    `);
-  });
-
-  test("generate SQL for migration 0 -> 2 (create users table and add age)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(2, {
-      updateSettings: true,
-      fromVersion: 0,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toMatchInlineSnapshot(`
-      "create table "users_test_namespace" ("id" varchar(30) not null unique, "name" text not null, "_internalId" bigserial not null primary key, "_version" integer default 0 not null);
-
-      alter table "users_test_namespace" add column "age" integer;
-
-      create index "name_idx" on "users_test_namespace" ("name");
-
-      create index "age_idx" on "users_test_namespace" ("age");
-
-      insert into "fragno_db_settings" ("id", "key", "value") values ('jprP_43K5uMwxAFiepbbrQ', 'test_namespace.schema_version', '2');"
-    `);
-  });
-
-  test("generate SQL for migration 1 -> 2 (add age column and indexes)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(2, {
-      updateSettings: true,
-      fromVersion: 1,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toMatchInlineSnapshot(`
-      "alter table "users_test_namespace" add column "age" integer;
-
-      create index "name_idx" on "users_test_namespace" ("name");
-
-      create index "age_idx" on "users_test_namespace" ("age");
-
-      update "fragno_db_settings" set "value" = '2' where "key" = 'test_namespace.schema_version';"
-    `);
-  });
-
-  test("generate SQL for migration 0 -> 3 (full schema)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(3, {
-      updateSettings: true,
-      fromVersion: 0,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toMatchInlineSnapshot(`
-      "create table "users_test_namespace" ("id" varchar(30) not null unique, "name" text not null, "_internalId" bigserial not null primary key, "_version" integer default 0 not null);
-
-      alter table "users_test_namespace" add column "age" integer;
-
-      create index "name_idx" on "users_test_namespace" ("name");
-
-      create index "age_idx" on "users_test_namespace" ("age");
-
-      create table "posts_test_namespace" ("id" varchar(30) not null unique, "title" text not null, "user_id" text not null, "_internalId" bigserial not null primary key, "_version" integer default 0 not null);
-
-      insert into "fragno_db_settings" ("id", "key", "value") values ('jprP_43K5uMwxAFiepbbrQ', 'test_namespace.schema_version', '3');"
-    `);
-  });
-
-  test("generate SQL for migration 2 -> 3 (add posts table)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(3, {
-      updateSettings: true,
-      fromVersion: 2,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toMatchInlineSnapshot(`
-      "create table "posts_test_namespace" ("id" varchar(30) not null unique, "title" text not null, "user_id" text not null, "_internalId" bigserial not null primary key, "_version" integer default 0 not null);
-
-      update "fragno_db_settings" set "value" = '3' where "key" = 'test_namespace.schema_version';"
-    `);
-  });
-
-  test("generate empty SQL for migration 2 -> 2 (no changes)", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-    const preparedMigration = await migrator.prepareMigrationTo(2, {
-      updateSettings: true,
-      fromVersion: 2,
-    });
-
-    expect(preparedMigration.operations.length).toBe(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-    expect(sql).toBe("");
-  });
-
-  test("throw error for backward migration", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-
-    await expect(
-      migrator.prepareMigrationTo(1, {
-        updateSettings: true,
-        fromVersion: 2,
-      }),
-    ).rejects.toThrow("Cannot migrate backwards");
-  });
-
-  test("throw error for version beyond schema", async () => {
-    const migrator = adapter.createMigrationEngine(testSchema, "test_namespace");
-
-    await expect(
-      migrator.prepareMigrationTo(10, {
-        updateSettings: true,
-        fromVersion: 0,
-      }),
-    ).rejects.toThrow("schema only has version 4");
-  });
-});
-
-describe("KyselyMigrator - SQLite Foreign Key Merging", () => {
-  // Test the user's exact example schema
-  const userExampleSchema = schema((s) => {
-    return s
-      .addTable("users", (t) => {
-        return t.addColumn("id", idColumn());
-      })
-      .addTable("posts", (t) => {
-        return t.addColumn("id", idColumn()).addColumn("authorId", referenceColumn());
-      })
-      .addReference("author", {
-        type: "one",
-        from: { table: "posts", column: "authorId" },
-        to: { table: "users", column: "id" },
-      });
-  });
-
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  let db: Kysely<any>;
-  let adapter: KyselyAdapter;
-
-  beforeAll(async () => {
-    // Create a Kysely instance with SQLite dialect
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    db = new Kysely({ dialect: new SqliteDialect({} as any) });
-    adapter = new KyselyAdapter({ db, provider: "sqlite" });
-  });
-
-  test("SQLite should merge foreign keys into create-table operations", async () => {
-    const migrator = adapter.createMigrationEngine(userExampleSchema, "test");
-
-    // Migrate from 0 -> 3 (all tables + FK in one batch)
-    const preparedMigration = await migrator.prepareMigrationTo(3, {
-      updateSettings: true,
-      fromVersion: 0,
-    });
-
-    expect(preparedMigration.operations.length).toBeGreaterThan(0);
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-
-    // The SQL should have PRAGMA defer_foreign_keys
-    expect(sql).toContain("PRAGMA defer_foreign_keys = ON");
-
-    // Should create users table
-    expect(sql).toContain('create table "users_test"');
-
-    // Should create posts table WITH inline foreign key constraint
-    expect(sql).toContain('create table "posts_test"');
-    expect(sql).toContain("foreign key");
-    expect(sql).toContain("authorId");
-    expect(sql).toContain('references "users_test"');
-
-    // Should NOT have a separate ALTER TABLE ADD FOREIGN KEY
-    // (SQLite doesn't support it, and we've merged it into create-table)
-    expect(sql).not.toMatch(/alter table.*add.*foreign key/i);
-  });
-
-  test("SQLite FK merging full schema verification", async () => {
-    const migrator = adapter.createMigrationEngine(userExampleSchema, "test");
-
-    const preparedMigration = await migrator.prepareMigrationTo(3, {
-      updateSettings: true,
-      fromVersion: 0,
-    });
-
-    const sql = preparedMigration.getSQL?.();
-    expect(sql).toBeDefined();
-
-    // Verify the complete SQL snapshot
-    expect(sql).toMatchInlineSnapshot(`
-      "PRAGMA defer_foreign_keys = ON;
-
-      create table "users_test" ("id" text not null unique, "_internalId" integer not null primary key autoincrement, "_version" integer default 0 not null);
-
-      create table "posts_test" ("id" text not null unique, "authorId" integer not null, "_internalId" integer not null primary key autoincrement, "_version" integer default 0 not null, constraint "posts_users_author_fk" foreign key ("authorId") references "users_test" ("_internalId") on delete restrict on update restrict);
-
-      insert into "fragno_db_settings" ("id", "key", "value") values ('BflimUWc1NbCMMDD9SM3gQ', 'test.schema_version', '3');"
-    `);
-  });
-});

package/src/bind-services.test.ts

@@ -1,214 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { bindServicesToContext } from "./bind-services";
-import { withUnitOfWork, type DatabaseRequestThisContext } from "./fragment";
-import type { IUnitOfWorkBase } from "./query/unit-of-work";
-import { schema, idColumn } from "./schema/create";
-
-// Create a simple test schema for tests
-const testSchema = schema((s) => {
-  return s.addTable("test", (t) => {
-    return t.addColumn("id", idColumn());
-  });
-});
-
-describe("bindServicesToContext", () => {
-  it("should bind simple function to context", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      testMethod: function (this: DatabaseRequestThisContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    const result = await withUnitOfWork(mockUow, () => bound.testMethod());
-
-    expect(result).toBe(mockSchemaView);
-  });
-
-  it("should bind multiple functions", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      method1: function (this: DatabaseRequestThisContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-      method2: function (this: DatabaseRequestThisContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    await withUnitOfWork(mockUow, () => {
-      expect(bound.method1()).toBe(mockSchemaView);
-      expect(bound.method2()).toBe(mockSchemaView);
-    });
-  });
-
-  it("should bind nested service objects", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      nested: {
-        method: function (this: DatabaseRequestThisContext) {
-          // eslint-disable-next-line @typescript-eslint/no-explicit-any
-          return this.getUnitOfWork(testSchema as any);
-        },
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    const result = await withUnitOfWork(mockUow, () => bound.nested.method());
-
-    expect(result).toBe(mockSchemaView);
-  });
-
-  it("should preserve non-function properties", () => {
-    const services = {
-      method: function () {
-        return "test";
-      },
-      constant: "value",
-      number: 42,
-    };
-
-    const bound = bindServicesToContext(services);
-
-    expect(bound.constant).toBe("value");
-    expect(bound.number).toBe(42);
-    expect(bound.method()).toBe("test");
-  });
-
-  it("should handle functions with parameters", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      testMethod: function (this: DatabaseRequestThisContext, param1: string, param2: number) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        const uow = this.getUnitOfWork(testSchema as any);
-        return { uow, param1, param2 };
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    const result = await withUnitOfWork(mockUow, () => bound.testMethod("hello", 123));
-
-    expect(result.uow).toBe(mockSchemaView);
-    expect(result.param1).toBe("hello");
-    expect(result.param2).toBe(123);
-  });
-
-  it("should handle async functions", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      asyncMethod: async function (this: DatabaseRequestThisContext) {
-        await new Promise((resolve) => setTimeout(resolve, 10));
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    const result = await withUnitOfWork(mockUow, async () => await bound.asyncMethod());
-
-    expect(result).toBe(mockSchemaView);
-  });
-
-  it("should not bind arrays", () => {
-    const services = {
-      array: [1, 2, 3],
-      method: function () {
-        return "test";
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    expect(bound.array).toEqual([1, 2, 3]);
-    expect(Array.isArray(bound.array)).toBe(true);
-  });
-
-  it("should handle deeply nested objects", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      level1: {
-        level2: {
-          level3: {
-            method: function (this: DatabaseRequestThisContext) {
-              // eslint-disable-next-line @typescript-eslint/no-explicit-any
-              return this.getUnitOfWork(testSchema as any);
-            },
-          },
-        },
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    const result = await withUnitOfWork(mockUow, () => bound.level1.level2.level3.method());
-
-    expect(result).toBe(mockSchemaView);
-  });
-
-  it("should allow bound services to access UOW independently", async () => {
-    const mockSchemaView = { test: "schema-view" };
-    const mockUow = {
-      test: "uow",
-      forSchema: () => mockSchemaView,
-    } as unknown as IUnitOfWorkBase;
-
-    const services = {
-      getUow: function (this: DatabaseRequestThisContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-      callOther: function (this: DatabaseRequestThisContext) {
-        // Both methods can access UOW via their own `this` context
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return this.getUnitOfWork(testSchema as any);
-      },
-    };
-
-    const bound = bindServicesToContext(services);
-
-    await withUnitOfWork(mockUow, () => {
-      expect(bound.getUow()).toBe(mockSchemaView);
-      expect(bound.callOther()).toBe(mockSchemaView);
-    });
-  });
-});

package/src/bind-services.ts

@@ -1,37 +0,0 @@
-import { serviceContext } from "./fragment";
-
-// Type helper to remove 'this' parameter from functions
-type OmitThisParameter<T> = T extends (this: infer _This, ...args: infer A) => infer R
-  ? (...args: A) => R
-  : T;
-
-// Recursively remove 'this' parameter from all functions in an object
-export type BoundServices<T> = {
-  [K in keyof T]: T[K] extends (...args: never[]) => unknown
-    ? OmitThisParameter<T[K]>
-    : T[K] extends Record<string, unknown>
-      ? BoundServices<T[K]>
-      : T[K];
-};
-
-export function bindServicesToContext<T extends Record<string, unknown>>(
-  services: T,
-): BoundServices<T> {
-  const bound = {} as BoundServices<T>;
-
-  for (const [key, value] of Object.entries(services)) {
-    if (typeof value === "function") {
-      // Bind function to serviceContext
-      bound[key as keyof T] = value.bind(serviceContext) as BoundServices<T>[keyof T];
-    } else if (value && typeof value === "object" && !Array.isArray(value)) {
-      // Recursively bind nested service objects
-      bound[key as keyof T] = bindServicesToContext(
-        value as Record<string, unknown>,
-      ) as BoundServices<T>[keyof T];
-    } else {
-      bound[key as keyof T] = value as BoundServices<T>[keyof T];
-    }
-  }
-
-  return bound;
-}