@wopr-network/platform-core 1.0.2 → 1.0.3

package/dist/auth/better-auth.d.ts

@@ -11,6 +11,7 @@ import { betterAuth } from "better-auth";
 import type { Pool } from "pg";
 import type { PlatformDb } from "../db/index.js";
 import { PgEmailVerifier } from "../email/verification.js";
+import { type IUserCreator } from "./user-creator.js";
 /** OAuth provider credentials. */
 export interface OAuthProvider {
     clientId: string;
@@ -63,6 +64,7 @@ export interface BetterAuthConfig {
     /** Called after a new user signs up (e.g., create personal tenant). */
     onUserCreated?: (userId: string, userName: string, email: string) => Promise<void>;
 }
+export declare function getUserCreator(): Promise<IUserCreator>;
 /** The type of a better-auth instance. */
 export type Auth = ReturnType<typeof betterAuth>;
 /** Initialize Better Auth with the given config. Must be called before getAuth(). */

package/dist/auth/better-auth.js

@@ -24,15 +24,20 @@ const DEFAULT_RATE_LIMIT_RULES = {
 let _config = null;
 let _userCreator = null;
 let _userCreatorPromise = null;
-async function getUserCreator() {
+export async function getUserCreator() {
     if (_userCreator)
         return _userCreator;
     if (!_userCreatorPromise) {
         if (!_config)
             throw new Error("BetterAuth not initialized — call initBetterAuth() first");
-        _userCreatorPromise = createUserCreator(new RoleStore(_config.db)).then((creator) => {
+        _userCreatorPromise = createUserCreator(new RoleStore(_config.db))
+            .then((creator) => {
             _userCreator = creator;
             return creator;
+        })
+            .catch((err) => {
+            _userCreatorPromise = null;
+            throw err;
         });
     }
     return _userCreatorPromise;

package/dist/auth/better-auth.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/auth/better-auth.test.js

@@ -0,0 +1,35 @@
+import { afterEach, describe, expect, it, vi } from "vitest";
+const mockCreateUserCreator = vi.fn();
+vi.mock("./user-creator.js", () => ({
+    createUserCreator: (...args) => mockCreateUserCreator(...args),
+}));
+vi.mock("../admin/role-store.js", () => ({
+    RoleStore: vi.fn(),
+}));
+const { getUserCreator, initBetterAuth, resetUserCreator } = await import("./better-auth.js");
+const fakeConfig = { pool: {}, db: {} };
+describe("getUserCreator", () => {
+    afterEach(() => {
+        resetUserCreator();
+        mockCreateUserCreator.mockReset();
+    });
+    it("caches the resolved creator on success", async () => {
+        initBetterAuth(fakeConfig);
+        const fakeCreator = { createUser: vi.fn() };
+        mockCreateUserCreator.mockResolvedValueOnce(fakeCreator);
+        const first = await getUserCreator();
+        const second = await getUserCreator();
+        expect(first).toBe(second);
+        expect(mockCreateUserCreator).toHaveBeenCalledOnce();
+    });
+    it("clears cached promise on rejection so next call retries", async () => {
+        initBetterAuth(fakeConfig);
+        const fakeCreator = { createUser: vi.fn() };
+        mockCreateUserCreator.mockRejectedValueOnce(new Error("DB unavailable"));
+        mockCreateUserCreator.mockResolvedValueOnce(fakeCreator);
+        await expect(getUserCreator()).rejects.toThrow("DB unavailable");
+        const creator = await getUserCreator();
+        expect(creator).toBe(fakeCreator);
+        expect(mockCreateUserCreator).toHaveBeenCalledTimes(2);
+    });
+});

package/dist/auth/index.d.ts

@@ -187,7 +187,7 @@ export declare function validateTenantAccess(userId: string, requestedTenantId:
 export type { IApiKeyRepository } from "./api-key-repository.js";
 export { DrizzleApiKeyRepository } from "./api-key-repository.js";
 export type { Auth, AuthRateLimitRule, BetterAuthConfig, OAuthProvider, } from "./better-auth.js";
-export { getAuth, getEmailVerifier, initBetterAuth, resetAuth, resetUserCreator, runAuthMigrations, setAuth, } from "./better-auth.js";
+export { getAuth, getEmailVerifier, getUserCreator, initBetterAuth, resetAuth, resetUserCreator, runAuthMigrations, setAuth, } from "./better-auth.js";
 export type { ILoginHistoryRepository, LoginHistoryEntry } from "./login-history-repository.js";
 export { BetterAuthLoginHistoryRepository } from "./login-history-repository.js";
 export type { SessionAuthEnv } from "./middleware.js";

package/dist/auth/index.js

@@ -422,7 +422,7 @@ export async function validateTenantAccess(userId, requestedTenantId, orgMemberR
 }
 export { DrizzleApiKeyRepository } from "./api-key-repository.js";
 // Test utilities — do not call in production code
-export { getAuth, getEmailVerifier, initBetterAuth, resetAuth, resetUserCreator, runAuthMigrations, setAuth, } from "./better-auth.js";
+export { getAuth, getEmailVerifier, getUserCreator, initBetterAuth, resetAuth, resetUserCreator, runAuthMigrations, setAuth, } from "./better-auth.js";
 export { BetterAuthLoginHistoryRepository } from "./login-history-repository.js";
 export { dualAuth, sessionAuth } from "./middleware.js";
 export { createUserCreator } from "./user-creator.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wopr-network/platform-core",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/auth/better-auth.test.ts

@@ -0,0 +1,47 @@
+import { afterEach, describe, expect, it, vi } from "vitest";
+
+const mockCreateUserCreator = vi.fn();
+vi.mock("./user-creator.js", () => ({
+  createUserCreator: (...args: unknown[]) => mockCreateUserCreator(...args),
+}));
+
+vi.mock("../admin/role-store.js", () => ({
+  RoleStore: vi.fn(),
+}));
+
+const { getUserCreator, initBetterAuth, resetUserCreator } = await import("./better-auth.js");
+
+const fakeConfig = { pool: {}, db: {} } as Parameters<typeof initBetterAuth>[0];
+
+describe("getUserCreator", () => {
+  afterEach(() => {
+    resetUserCreator();
+    mockCreateUserCreator.mockReset();
+  });
+
+  it("caches the resolved creator on success", async () => {
+    initBetterAuth(fakeConfig);
+    const fakeCreator = { createUser: vi.fn() };
+    mockCreateUserCreator.mockResolvedValueOnce(fakeCreator);
+
+    const first = await getUserCreator();
+    const second = await getUserCreator();
+
+    expect(first).toBe(second);
+    expect(mockCreateUserCreator).toHaveBeenCalledOnce();
+  });
+
+  it("clears cached promise on rejection so next call retries", async () => {
+    initBetterAuth(fakeConfig);
+    const fakeCreator = { createUser: vi.fn() };
+
+    mockCreateUserCreator.mockRejectedValueOnce(new Error("DB unavailable"));
+    mockCreateUserCreator.mockResolvedValueOnce(fakeCreator);
+
+    await expect(getUserCreator()).rejects.toThrow("DB unavailable");
+
+    const creator = await getUserCreator();
+    expect(creator).toBe(fakeCreator);
+    expect(mockCreateUserCreator).toHaveBeenCalledTimes(2);
+  });
+});

package/src/auth/better-auth.ts

@@ -94,14 +94,19 @@ let _config: BetterAuthConfig | null = null;
 let _userCreator: IUserCreator | null = null;
 let _userCreatorPromise: Promise<IUserCreator> | null = null;
 
-async function getUserCreator(): Promise<IUserCreator> {
+export async function getUserCreator(): Promise<IUserCreator> {
   if (_userCreator) return _userCreator;
   if (!_userCreatorPromise) {
     if (!_config) throw new Error("BetterAuth not initialized — call initBetterAuth() first");
-    _userCreatorPromise = createUserCreator(new RoleStore(_config.db)).then((creator) => {
-      _userCreator = creator;
-      return creator;
-    });
+    _userCreatorPromise = createUserCreator(new RoleStore(_config.db))
+      .then((creator) => {
+        _userCreator = creator;
+        return creator;
+      })
+      .catch((err) => {
+        _userCreatorPromise = null;
+        throw err;
+      });
   }
   return _userCreatorPromise;
 }

package/src/auth/index.ts

@@ -535,6 +535,7 @@ export type {
 export {
   getAuth,
   getEmailVerifier,
+  getUserCreator,
   initBetterAuth,
   resetAuth,
   resetUserCreator,

package/vitest.config.ts

@@ -3,6 +3,7 @@ import { defineConfig } from "vitest/config";
 export default defineConfig({
   test: {
     testTimeout: 30000,
+    hookTimeout: 30000,
     include: ["src/**/*.test.ts"],
     coverage: {
       provider: "v8",