tui: upgrade database migration system to drizzle migrator

Replaces custom migration system with drizzle-orm's built-in migrator, bundling migrations at build-time instead of runtime generation. This reduces bundle complexity and provides better integration with drizzle's migration tracking.

Author: thdxr

AGENTS.md

@@ -1,80 +1,93 @@
-- To regenerate the JavaScript SDK, run `./packages/sdk/js/script/build.ts`.
-- ALWAYS USE PARALLEL TOOLS WHEN APPLICABLE.
-- The default branch in this repo is `dev`.
+# Agent Instructions
+
+## Quick Reference
+
+- Default branch: `dev`
+- Regenerate JS SDK: `./packages/sdk/js/script/build.ts`
+- Always use parallel tools when applicable
 
 ## Style Guide
 
+### General Principles
+
 - Keep things in one function unless composable or reusable
-- Avoid unnecessary destructuring. Instead of `const { a, b } = obj`, use `obj.a` and `obj.b` to preserve context
 - Avoid `try`/`catch` where possible
 - Avoid using the `any` type
-- Prefer single word variable names where possible
-- Use Bun APIs when possible, like `Bun.file()`
+- Use Bun APIs when possible (e.g., `Bun.file()` instead of `fs.existsSync()`)
+- For sync file reads, use `readFileSync` from `fs` (Bun.file is async-only)
+- Avoid generated file artifacts - prefer build-time `define` globals for bundled data
 
-### Avoid let statements
+### Naming
 
-We don't like `let` statements, especially combined with if/else statements.
-Prefer `const`.
+Prefer single word variable names. Only use multiple words if necessary.
+
+```ts
+// Good
+const foo = 1
+const bar = 2
+
+// Bad
+const fooBar = 1
+const barBaz = 2
+```
 
-Good:
+Reduce total variable count by inlining when a value is only used once.
 
 ```ts
-const foo = condition ? 1 : 2
+// Good
+const journal = await Bun.file(path.join(dir, "journal.json")).json()
+
+// Bad
+const journalPath = path.join(dir, "journal.json")
+const journal = await Bun.file(journalPath).json()
 ```
 
-Bad:
+### Destructuring
+
+Avoid unnecessary destructuring. Use dot notation to preserve context.
 
 ```ts
-let foo
+// Good
+obj.a
+obj.b
+
+// Bad
+const { a, b } = obj
+```
+
+### Variables
+
+Prefer `const` over `let`. Use ternaries or early returns instead of reassignment.
 
+```ts
+// Good
+const foo = condition ? 1 : 2
+
+// Bad
+let foo
 if (condition) foo = 1
 else foo = 2
 ```
 
-### Avoid else statements
-
-Prefer early returns or using an `iife` to avoid else statements.
+### Control Flow
 
-Good:
+Avoid `else` statements. Prefer early returns.
 
 ```ts
+// Good
 function foo() {
   if (condition) return 1
   return 2
 }
-```
-
-Bad:
 
-```ts
+// Bad
 function foo() {
   if (condition) return 1
   else return 2
 }
 ```
 
-### Prefer single word naming
-
-Try your best to find a single word name for your variables, functions, etc.
-Only use multiple words if you cannot.
-
-Good:
-
-```ts
-const foo = 1
-const bar = 2
-const baz = 3
-```
-
-Bad:
-
-```ts
-const fooBar = 1
-const barBaz = 2
-const bazFoo = 3
-```
-
 ## Testing
 
-You MUST avoid using `mocks` as much as possible.
-Tests MUST test actual implementation, do not duplicate logic into a test.
+- Avoid mocks as much as possible
+- Test actual implementation, do not duplicate logic into tests

bun.lock

@@ -15,7 +15,8 @@
     "lint": "echo 'Running lint checks...' && bun test --coverage",
     "format": "echo 'Formatting code...' && bun run --prettier --write src/**/*.ts",
     "docs": "echo 'Generating documentation...' && find src -name '*.ts' -exec echo 'Processing: {}' \\;",
-    "deploy": "echo 'Deploying application...' && bun run build && echo 'Deployment completed successfully'"
+    "deploy": "echo 'Deploying application...' && bun run build && echo 'Deployment completed successfully'",
+    "db": "bun drizzle-kit"
   },
   "bin": {
     "opencode": "./bin/opencode"
@@ -25,7 +26,7 @@
   },
   "devDependencies": {
     "@babel/core": "7.28.4",
-    "drizzle-kit": "0.31.0",
+    "drizzle-kit": "1.0.0-beta.12-a5629fb",
     "@octokit/webhooks-types": "7.6.1",
     "@opencode-ai/script": "workspace:*",
     "@parcel/watcher-darwin-arm64": "2.5.1",
@@ -98,7 +99,7 @@
     "clipboardy": "4.0.0",
     "decimal.js": "10.5.0",
     "diff": "catalog:",
-    "drizzle-orm": "0.44.2",
+    "drizzle-orm": "1.0.0-beta.12-a5629fb",
     "fuzzysort": "3.1.0",
     "gray-matter": "4.0.3",
     "hono": "catalog:",

packages/opencode/package.json

@@ -25,6 +25,18 @@ await Bun.write(
 )
 console.log("Generated models-snapshot.ts")
 
+// Load migrations from journal
+const journal = (await Bun.file(path.join(dir, "migration/meta/_journal.json")).json()) as {
+  entries: { tag: string; when: number }[]
+}
+const migrations = await Promise.all(
+  journal.entries.map(async (entry) => {
+    const sql = await Bun.file(path.join(dir, `migration/${entry.tag}.sql`)).text()
+    return { sql, timestamp: entry.when }
+  }),
+)
+console.log(`Loaded ${migrations.length} migrations`)
+
 const singleFlag = process.argv.includes("--single")
 const baselineFlag = process.argv.includes("--baseline")
 const skipInstall = process.argv.includes("--skip-install")
@@ -156,6 +168,7 @@ for (const item of targets) {
     entrypoints: ["./src/index.ts", parserWorker, workerPath],
     define: {
       OPENCODE_VERSION: `'${Script.version}'`,
+      OPENCODE_MIGRATIONS: JSON.stringify(migrations),
       OTUI_TREE_SITTER_WORKER_PATH: bunfsRoot + workerRelativePath,
       OPENCODE_WORKER_PATH: workerPath,
       OPENCODE_CHANNEL: `'${Script.channel}'`,

packages/opencode/script/build.ts

@@ -129,9 +129,6 @@ const ExportCommand = cmd({
         stats.shares++
       }
 
-      // Create migration marker so this can be imported back
-      await Bun.write(path.join(outDir, "migration"), Date.now().toString())
-
       UI.println(`Exported to ${outDir}:`)
       UI.println(`  ${stats.projects} projects`)
       UI.println(`  ${stats.sessions} sessions`)