refactor(guard-clause): extract

Author: TorinAsakura

examples/guard-clause/src/index.ts

@@ -1,4 +1,6 @@
-import { Against, Guard, GuardErrors } from '@atls/guard-clause'
+import { Against }     from '@atls/guard-clause'
+import { Guard }       from '@atls/guard-clause'
+import { GuardErrors } from '@atls/guard-clause'
 
 type SignUpPayload = {
   email: string
@@ -10,13 +12,14 @@ type SignUpPayload = {
 export class UserOnboardingService {
   @Guard()
   register(
-    @Against('email').Empty().NotStringLengthBetween(5, 128)
+    @(Against('email').Empty())
+    @(Against('email').NotStringLengthBetween(5, 128))
     email: string,
-    @Against('age').NotNumberBetween(16, 120)
+    @(Against('age').NotNumberBetween(16, 120))
     age: number,
-    @Against('roles').Optional.Each.NotOneOf(['root', 'god-mode'])
+    @(Against('roles').Optional.Each.NotOneOf(['root', 'god-mode']))
     roles: Array<string> = [],
-    @Against('profileId').Optional.NotUUID('4')
+    @(Against('profileId').Optional.NotUUID('4'))
     profileId?: string
   ): SignUpPayload {
     return { email, age, roles, profileId }
@@ -28,9 +31,11 @@ const onboarding = new UserOnboardingService()
 try {
   onboarding.register('', 14, ['root'], 'not-a-uuid')
 } catch (error) {
-  if (error instanceof GuardErrors) {
-    const messages = error.errors.map((issue) => issue.message).join('\n')
-
-    process.stderr.write(`Не пускаем пользователя:\n${messages}\n`)
+  if (!(error instanceof GuardErrors)) {
+    throw error
   }
+
+  const messages = error.errors.map((issue) => issue.message).join('\n')
+
+  process.stderr.write(`Не пускаем пользователя:\n${messages}\n`)
 }

examples/service-entrypoint/README.md

@@ -2,8 +2,8 @@
 
 Этот пример нужен как эталон запуска NestJS-сервиса внутри наших библиотек. Тут показываем, как собирать `@atls/logger` в боевом окружении, как правильно держать hot-reload через `import.meta.hot` и где врубать graceful shutdown. Никаких «магических» пакетов — только базовые зависимости из Nest + наш логгер.
 
-* Поднимаем один экземпляр логгера и прокидываем его в модуль, чтобы не плодить инстансы.
-* Прячем старт приложения за функцией `bootstrap`, чтобы было куда подвесить ошибки запуска и выключение.
-* Демонстрируем, как закрывать Nest-приложение, когда Vite/HMR даёт сигнал на перезагрузку.
+- Поднимаем один экземпляр логгера и прокидываем его в модуль, чтобы не плодить инстансы.
+- Прячем старт приложения за функцией `bootstrap`, чтобы было куда подвесить ошибки запуска и выключение.
+- Демонстрируем, как закрывать Nest-приложение, когда Vite/HMR даёт сигнал на перезагрузку.
 
 Короче, это рельсовый шаблон старта сервиса: берёшь и копируешь, чтобы не наступать на грабли с ESM-импортами и хаотичными логгерами.

packages/core-errors/src/index.ts

@@ -1,48 +1 @@
-import { DomainError } from '@atls/core-errors'
-
-type PaymentGateway = 'stripe' | 'cloud-payments'
-
-type PaymentIntent = {
-  id: string
-  gateway: PaymentGateway
-  amountMinor: number
-  currency: string
-}
-
-export class PaymentRejectedError extends DomainError {
-  public readonly intent: PaymentIntent
-
-  private constructor(intent: PaymentIntent, reason: string) {
-    super(`Платёж ${intent.id} улетел в трубу: ${reason}`)
-
-    this.intent = intent
-  }
-
-  static becauseGatewayRejected(intent: PaymentIntent, reason: string): PaymentRejectedError {
-    return new PaymentRejectedError(intent, reason)
-  }
-}
-
-export const ensureCaptured = (intent: PaymentIntent & { captured: boolean }): PaymentIntent => {
-  if (!intent.captured) {
-    throw PaymentRejectedError.becauseGatewayRejected(intent, 'гейт закрыл транзакцию')
-  }
-
-  return intent
-}
-
-try {
-  ensureCaptured({
-    id: 'pay_123',
-    gateway: 'stripe',
-    amountMinor: 1990,
-    currency: 'RUB',
-    captured: false,
-  })
-} catch (error) {
-  if (error instanceof PaymentRejectedError) {
-    const { amountMinor, currency } = error.intent
-
-    process.stderr.write(`Надо раскатать рефанд на ${amountMinor} ${currency}\n`)
-  }
-}
+export { DomainError } from './domain.error.js'

packages/guard-clause/src/decorators/against/index.ts

@@ -27,13 +27,28 @@ const factory = (name: string, options?: AbstractGuardExtensionFactoryOptions['o
   NotBigInt: NotBigIntDecoratorFactory(name, options),
 })
 
-export const Against = (
-  name: string
-): ReturnType<typeof factory> & {
-  Optional: ReturnType<typeof factory>
-  Each: ReturnType<typeof factory>
-} => ({
-  ...factory(name),
-  Optional: factory(name, { optional: true }),
-  Each: factory(name, { each: true }),
-})
+type GuardDecoratorBuilder = ReturnType<typeof factory> & {
+  Optional: GuardDecoratorBuilder
+  Each: GuardDecoratorBuilder
+}
+
+const createBuilder = (
+  name: string,
+  options?: AbstractGuardExtensionFactoryOptions['options']
+): GuardDecoratorBuilder =>
+  Object.defineProperties(factory(name, options), {
+    Optional: {
+      enumerable: true,
+      get(): GuardDecoratorBuilder {
+        return createBuilder(name, { ...options, optional: true })
+      },
+    },
+    Each: {
+      enumerable: true,
+      get(): GuardDecoratorBuilder {
+        return createBuilder(name, { ...options, each: true })
+      },
+    },
+  }) as GuardDecoratorBuilder
+
+export const Against = (name: string): GuardDecoratorBuilder => createBuilder(name)

packages/guard-clause/src/index.ts

@@ -1,36 +1,4 @@
-import { Against, Guard, GuardErrors } from '@atls/guard-clause'
-
-type SignUpPayload = {
-  email: string
-  age: number
-  roles: Array<string>
-  profileId?: string
-}
-
-export class UserOnboardingService {
-  @Guard()
-  register(
-    @Against('email').Empty().NotStringLengthBetween(5, 128)
-    email: string,
-    @Against('age').NotNumberBetween(16, 120)
-    age: number,
-    @Against('roles').Optional.Each.NotOneOf(['root', 'god-mode'])
-    roles: Array<string> = [],
-    @Against('profileId').Optional.NotUUID('4')
-    profileId?: string
-  ): SignUpPayload {
-    return { email, age, roles, profileId }
-  }
-}
-
-const onboarding = new UserOnboardingService()
-
-try {
-  onboarding.register('', 14, ['root'], 'not-a-uuid')
-} catch (error) {
-  if (error instanceof GuardErrors) {
-    const messages = error.errors.map((issue) => issue.message).join('\n')
-
-    process.stderr.write(`Не пускаем пользователя:\n${messages}\n`)
-  }
-}
+export * from './decorators/index.js'
+export * from './errors/index.js'
+export * from './extensions/index.js'
+export * from './factory/index.js'

packages/logger/src/sonic-boom.utils.ts

@@ -41,7 +41,6 @@ function autoEnd(stream: any, eventName: string) {
 }
 
 export const build = () => {
-  // @ts-expect-error -- sonic-boom typings miss the construct signature in nodenext mode
   const stream = new SonicBoom({ fd: process.stdout.fd || 1 })
 
   stream.on('error', filterBrokenPipe)