| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244 |
- import { describe, expect, test } from "bun:test"
- import { $ } from "bun"
- import { fileURLToPath } from "url"
- import path from "path"
- import { SqliteClient } from "@effect/sql-sqlite-bun"
- import { EffectDrizzleSqlite } from "@opencode-ai/effect-drizzle-sqlite"
- import { Effect, Layer } from "effect"
- import { sql } from "drizzle-orm"
- import { DatabaseMigration } from "@opencode-ai/core/database/migration"
- import { migrations } from "@opencode-ai/core/database/migration.gen"
- import { Database } from "@opencode-ai/core/database/database"
- import { tmpdir } from "./fixture/tmpdir"
- import type { SqlClient } from "effect/unstable/sql/SqlClient"
- import { importLegacyCredentials } from "@opencode-ai/core/database/migration/20260805200742_import_legacy_credentials"
- const run = <A, E>(effect: Effect.Effect<A, E, SqlClient>) =>
- Effect.runPromise(
- effect.pipe(Effect.provide(SqliteClient.layer({ filename: ":memory:", disableWAL: true })), Effect.scoped),
- )
- const makeDb = EffectDrizzleSqlite.makeWithDefaults()
- describe("DatabaseMigration", () => {
- test("serializes concurrent embedded initialization for one database path", async () => {
- await using tmp = await tmpdir()
- const filename = path.join(tmp.path, "embedded.sqlite")
- await Effect.runPromise(
- Effect.all(
- [Database.layer({ path: filename }), Database.layer({ path: filename })].map((layer) =>
- Effect.scoped(Layer.build(layer)),
- ),
- { concurrency: "unbounded" },
- ),
- )
- })
- if (process.platform === "linux") {
- test("declared schema has no ungenerated migrations", async () => {
- const result = await $`bun ${fileURLToPath(new URL("../script/migration.ts", import.meta.url))} --check`
- .quiet()
- .nothrow()
- expect(result.exitCode, result.stderr.toString()).toBe(0)
- expect(result.stdout.toString()).toContain("No schema changes, nothing to migrate")
- }, 30_000)
- }
- test("bootstraps the current schema and records the migration registry", async () => {
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* DatabaseMigration.apply(db)
- expect(yield* db.get(sql`SELECT name FROM sqlite_master WHERE type = 'table' AND name = 'session_v2'`)).toEqual(
- {
- name: "session_v2",
- },
- )
- expect(
- yield* db.get(sql`SELECT name FROM sqlite_master WHERE type = 'table' AND name = 'session_pending'`),
- ).toEqual({ name: "session_pending" })
- expect(yield* db.get(sql`SELECT count(*) AS count FROM migration`)).toEqual({ count: migrations.length })
- }),
- )
- })
- test("rejects a non-empty database without a session table", async () => {
- await expect(
- run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* db.run(sql`CREATE TABLE unrelated (id text PRIMARY KEY)`)
- yield* DatabaseMigration.apply(db)
- }),
- ),
- ).rejects.toThrow("Database is not empty and has no session table")
- })
- test("applies generic migrations once and records their order", async () => {
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* db.run(sql`CREATE TABLE session (id text PRIMARY KEY)`)
- const input = [
- {
- id: "first",
- up: (tx: Parameters<Parameters<typeof db.transaction>[0]>[0]) =>
- tx.run(sql`CREATE TABLE applied (id text PRIMARY KEY)`),
- },
- {
- id: "second",
- up: (tx: Parameters<Parameters<typeof db.transaction>[0]>[0]) =>
- tx.run(sql`INSERT INTO applied (id) VALUES ('second')`),
- },
- ]
- yield* DatabaseMigration.applyOnly(db, input)
- yield* DatabaseMigration.applyOnly(db, input)
- expect(yield* db.all(sql`SELECT id FROM applied`)).toEqual([{ id: "second" }])
- expect(yield* db.all(sql`SELECT id FROM migration ORDER BY time_completed, id`)).toEqual([
- { id: "first" },
- { id: "second" },
- ])
- }),
- )
- })
- test("imports legacy JSON credentials without changing the source file or existing credentials", async () => {
- await using tmp = await tmpdir()
- const source = path.join(tmp.path, "auth.json")
- const content = JSON.stringify({
- openai: { type: "oauth", refresh: "refresh", access: "access", expires: 123, accountId: "account" },
- anthropic: { type: "api", key: "legacy-key", metadata: { region: "us" } },
- "https://example.com/": { type: "wellknown", key: "TOKEN", token: "wellknown-key" },
- invalid: { type: "unknown" },
- })
- await Bun.write(source, content)
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* DatabaseMigration.apply(db)
- const now = Date.now()
- yield* db.run(sql`
- INSERT INTO credential (id, integration_id, label, value, time_created, time_updated)
- VALUES ('existing', 'anthropic', 'Existing', ${JSON.stringify({ type: "key", key: "current-key" })}, ${now}, ${now})
- `)
- yield* db.transaction((tx) => importLegacyCredentials(tx, source))
- expect(yield* db.all(sql`SELECT integration_id, label, value FROM credential ORDER BY integration_id`)).toEqual(
- [
- {
- integration_id: "anthropic",
- label: "Existing",
- value: JSON.stringify({ type: "key", key: "current-key" }),
- },
- {
- integration_id: "https://example.com",
- label: "default",
- value: JSON.stringify({ type: "key", key: "wellknown-key" }),
- },
- {
- integration_id: "openai",
- label: "default",
- value: JSON.stringify({
- type: "oauth",
- methodID: "chatgpt-browser",
- refresh: "refresh",
- access: "access",
- expires: 123,
- metadata: { accountID: "account" },
- }),
- },
- ],
- )
- expect(yield* db.get(sql`SELECT value FROM kv WHERE key = 'wellknown:sources'`)).toEqual({
- value: JSON.stringify(["https://example.com"]),
- })
- }),
- )
- expect(await Bun.file(source).text()).toBe(content)
- })
- test("rolls back a failed migration without recording it", async () => {
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* db.run(sql`CREATE TABLE session (id text PRIMARY KEY)`)
- const migration = {
- id: "failing",
- up: (tx: Parameters<Parameters<typeof db.transaction>[0]>[0]) =>
- Effect.gen(function* () {
- yield* tx.run(sql`CREATE TABLE rolled_back (id text PRIMARY KEY)`)
- yield* Effect.fail(new Error("stop"))
- }),
- }
- expect((yield* Effect.exit(DatabaseMigration.applyOnly(db, [migration])))._tag).toBe("Failure")
- expect(
- yield* db.get(sql`SELECT name FROM sqlite_master WHERE type = 'table' AND name = 'rolled_back'`),
- ).toBeUndefined()
- expect(yield* db.get(sql`SELECT id FROM migration WHERE id = 'failing'`)).toBeUndefined()
- }),
- )
- })
- test("suspends foreign keys outside migrations that rebuild referenced tables", async () => {
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* db.run(sql`PRAGMA foreign_keys = ON`)
- yield* db.run(sql`CREATE TABLE session (id text PRIMARY KEY, title text NOT NULL)`)
- yield* db.run(
- sql`CREATE TABLE message (id text PRIMARY KEY, session_id text NOT NULL REFERENCES session(id) ON DELETE CASCADE)`,
- )
- yield* db.run(sql`INSERT INTO session VALUES ('session', 'title')`)
- yield* db.run(sql`INSERT INTO message VALUES ('message', 'session')`)
- yield* DatabaseMigration.applyOnly(db, [
- {
- id: "rebuild",
- foreignKeys: false,
- up: (tx) =>
- Effect.gen(function* () {
- yield* tx.run(sql`CREATE TABLE next_session (id text PRIMARY KEY, title text)`)
- yield* tx.run(sql`INSERT INTO next_session SELECT * FROM session`)
- yield* tx.run(sql`DROP TABLE session`)
- yield* tx.run(sql`ALTER TABLE next_session RENAME TO session`)
- }),
- },
- ])
- expect(yield* db.get(sql`SELECT id FROM message`)).toEqual({ id: "message" })
- expect(yield* db.get<{ foreign_keys: number }>(sql`PRAGMA foreign_keys`)).toEqual({ foreign_keys: 1 })
- }),
- )
- })
- test("imports an existing Drizzle migration journal once", async () => {
- await run(
- Effect.gen(function* () {
- const db = yield* makeDb
- yield* db.run(
- sql`CREATE TABLE __drizzle_migrations (id INTEGER PRIMARY KEY, hash text NOT NULL, created_at numeric, name text, applied_at TEXT)`,
- )
- yield* db.run(sql`
- INSERT INTO __drizzle_migrations (hash, created_at, name, applied_at)
- VALUES ('hash', 1, 'legacy', ${new Date().toISOString()})
- `)
- yield* DatabaseMigration.applyOnly(db, [])
- expect(yield* db.all(sql`SELECT id FROM migration`)).toEqual([{ id: "legacy" }])
- yield* db.run(sql`INSERT INTO migration (id, time_completed) VALUES ('existing', 1)`)
- yield* db.run(sql`UPDATE __drizzle_migrations SET name = 'ignored'`)
- yield* DatabaseMigration.applyOnly(db, [])
- expect(yield* db.all(sql`SELECT id FROM migration ORDER BY id`)).toEqual([{ id: "existing" }, { id: "legacy" }])
- }),
- )
- })
- })
|