refactor(better-auth): improve field type normalization and migration logic

- Enhanced `normalizeBetterAuthFieldType` function for clearer type handling.
- Updated `planMigration` function to use a more structured approach for determining field types.
- Adjusted comments for better clarity on type normalization logic.

Author: eluce2

packages/better-auth/src/migrate.ts

@@ -8,8 +8,12 @@ import type { createRawFetch } from "./odata";
 type BetterAuthSchema = Record<string, { fields: Record<string, DBFieldAttribute>; order: number }>;
 
 function normalizeBetterAuthFieldType(fieldType: unknown): string {
-  if (typeof fieldType === "string") return fieldType;
-  if (Array.isArray(fieldType)) return fieldType.map(String).join("|");
+  if (typeof fieldType === "string") {
+    return fieldType;
+  }
+  if (Array.isArray(fieldType)) {
+    return fieldType.map(String).join("|");
+  }
   return String(fieldType);
 }
 
@@ -106,8 +110,12 @@ export async function planMigration(
       // Normalize it to a string so our FM mapping logic remains stable.
       // Use .includes() for all checks to handle array types like ["boolean", "null"] → "boolean|null"
       const t = normalizeBetterAuthFieldType(field.type);
-      const type: "varchar" | "numeric" | "timestamp" =
-        t.includes("boolean") || t.includes("number") ? "numeric" : t.includes("date") ? "timestamp" : "varchar";
+      let type: "varchar" | "numeric" | "timestamp" = "varchar";
+      if (t.includes("boolean") || t.includes("number")) {
+        type = "numeric";
+      } else if (t.includes("date")) {
+        type = "timestamp";
+      }
       return {
         name: field.fieldName ?? key,
         type,

packages/fmodata/tests/e2e/e2e.test.ts

@@ -16,10 +16,10 @@ import {
 } from "@proofkit/fmodata";
 import { afterEach, assert, describe, expect, expectTypeOf, it } from "vitest";
 import { z } from "zod/v4";
-import { apiKey, contacts, contactsTOWithIds, database, password, serverUrl, username, users } from "./setup";
 import { mockResponses } from "../fixtures/responses";
 import { jsonCodec } from "../utils/helpers";
 import { createMockFetch, simpleMock } from "../utils/mock-fetch";
+import { apiKey, contacts, contactsTOWithIds, database, password, serverUrl, username, users } from "./setup";
 
 if (!serverUrl) {
   throw new Error("FMODATA_SERVER_URL environment variable is required");

packages/typegen/vitest.config.ts

@@ -7,11 +7,10 @@ import { defineConfig } from "vitest/config";
 
 export default defineConfig({
   test: {
-    testTimeout: 15_000, // 15 seconds for unit tests
-    exclude: [
-      "**/node_modules/**",
-      "**/dist/**",
-      "tests/e2e/**", // E2E tests require live FM server, run separately with test:e2e
-    ],
+    testTimeout: 15_000, // 15 seconds, since we're making a network call to FM
+    setupFiles: ["./tests/setupEnv.ts"], // Add setup file
+    // Exclude E2E tests from default test runs
+    // Run E2E tests with: pnpm test:e2e
+    exclude: ["**/node_modules/**", "**/dist/**", "tests/e2e/**"],
   },
 });

scripts/ralph-once.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 claude --dangerously-skip-permissions "\
-1. Run 'bd ready' to find available work. \
+1. Run 'bv --robot-triage' to find available work. \
 2. Pick ONE task and run 'bd update <id> --status=in_progress'. \
 3. Run 'bd show <id>' to understand the task. \
 4. Implement the task. \