Skip to content

Commit 1d4c5d1

Browse files
Claudehotlong
andauthored
fix: add type annotations to resolve TypeScript errors in auth-manager
Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/93e8e1d7-431c-457a-b084-4e5cc358b097 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent 48e5ea8 commit 1d4c5d1

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

packages/plugins/plugin-auth/src/auth-manager.ts

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,11 @@ import type { Auth, BetterAuthOptions } from 'better-auth';
55
import { organization } from 'better-auth/plugins/organization';
66
import { twoFactor } from 'better-auth/plugins/two-factor';
77
import { magicLink } from 'better-auth/plugins/magic-link';
8-
import type { AuthConfig } from '@objectstack/spec/system';
8+
import type {
9+
AuthConfig,
10+
EmailAndPasswordConfig,
11+
AuthPluginConfig,
12+
} from '@objectstack/spec/system';
913
import type { IDataEngine } from '@objectstack/core';
1014
import { createObjectQLAdapterFactory } from './objectql-adapter.js';
1115
import {
@@ -374,20 +378,20 @@ export class AuthManager {
374378
}
375379

376380
// Extract email/password config (safe fields only)
377-
const emailPasswordConfig = this.config.emailAndPassword || {};
381+
const emailPasswordConfig: Partial<EmailAndPasswordConfig> = this.config.emailAndPassword ?? {};
378382
const emailPassword = {
379383
enabled: emailPasswordConfig.enabled !== false, // Default to true
380-
disableSignUp: emailPasswordConfig.disableSignUp,
381-
requireEmailVerification: emailPasswordConfig.requireEmailVerification,
384+
disableSignUp: emailPasswordConfig.disableSignUp ?? false,
385+
requireEmailVerification: emailPasswordConfig.requireEmailVerification ?? false,
382386
};
383387

384388
// Extract enabled features
385-
const pluginConfig = this.config.plugins || {};
389+
const pluginConfig: Partial<AuthPluginConfig> = this.config.plugins ?? {};
386390
const features = {
387-
twoFactor: pluginConfig.twoFactor || false,
388-
passkeys: pluginConfig.passkeys || false,
389-
magicLink: pluginConfig.magicLink || false,
390-
organization: pluginConfig.organization || false,
391+
twoFactor: pluginConfig.twoFactor ?? false,
392+
passkeys: pluginConfig.passkeys ?? false,
393+
magicLink: pluginConfig.magicLink ?? false,
394+
organization: pluginConfig.organization ?? false,
391395
};
392396

393397
return {

0 commit comments

Comments
 (0)