chore: introduce prettier (#27)

Author: yshrsmz

Diff for: .eslintrc.cjs

@@ -4,38 +4,38 @@ module.exports = {
   extends: [
     'eslint:recommended',
     'plugin:@typescript-eslint/recommended',
-    'plugin:import/typescript'
+    'plugin:import/typescript',
+    'plugin:prettier/recommended',
   ],
   parser: '@typescript-eslint/parser',
   plugins: ['@typescript-eslint'],
   rules: {
-    indent: ['error', 2],
     'no-multiple-empty-lines': 'error',
-    quotes: ['error', 'single', { 'allowTemplateLiterals': true }],
+    quotes: ['error', 'single', { allowTemplateLiterals: true }],
     semi: 'off',
     '@typescript-eslint/semi': ['error', 'never'],
     '@typescript-eslint/member-delimiter-style': [
       'error',
       {
-        'multiline': {
-          'delimiter': 'none',
-          'requireLast': true
+        multiline: {
+          delimiter: 'none',
+          requireLast: true,
         },
-        'singleline': {
-          'delimiter': 'semi',
-          'requireLast': false
+        singleline: {
+          delimiter: 'semi',
+          requireLast: false,
         },
-        'multilineDetection': 'brackets'
-      }
+        multilineDetection: 'brackets',
+      },
     ],
     '@typescript-eslint/no-unused-vars': [
       'warn',
       {
-        'argsIgnorePattern': '^_',
-        'varsIgnorePattern': '^_',
-        'caughtErrorsIgnorePattern': '^_',
-        'destructuredArrayIgnorePattern': '^_'
-      }
-    ]
-  }
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+        caughtErrorsIgnorePattern: '^_',
+        destructuredArrayIgnorePattern: '^_',
+      },
+    ],
+  },
 }

Diff for: .prettierrc

@@ -0,0 +1,5 @@
+{
+  "semi": false,
+  "singleQuote": true,
+  "trailingComma": "all"
+}

Diff for: package.json

@@ -30,9 +30,12 @@
     "@typescript-eslint/parser": "5.54.0",
     "cross-fetch": "3.1.5",
     "eslint": "8.35.0",
+    "eslint-config-prettier": "^8.7.0",
     "eslint-plugin-import": "2.27.5",
+    "eslint-plugin-prettier": "^4.2.1",
     "happy-dom": "8.9.0",
     "msw": "1.1.0",
+    "prettier": "^2.8.4",
     "typescript": "4.9.5",
     "vite": "4.1.4",
     "vitest": "0.29.2"

Diff for: pnpm-lock.yaml

@@ -44,8 +44,7 @@ export const defineBKTConfig = (config: RawBKTConfig): BKTConfig => {
 
   if (!result.apiKey) throw new Error('apiKey is required')
   if (!result.apiEndpoint) throw new Error('apiEndpoint is required')
-  if (!isValidUrl(result.apiEndpoint))
-    throw new Error('apiEndpoint is invalid')
+  if (!isValidUrl(result.apiEndpoint)) throw new Error('apiEndpoint is invalid')
   if (!result.featureTag) throw new Error('featureTag is required')
   if (!result.appVersion) throw new Error('appVersion is required')
 

Diff for: src/BKTConfig.ts

@@ -5,5 +5,11 @@ export interface BKTEvaluation {
   readonly userId: string
   readonly variationId: string
   readonly variationValue: string
-  readonly reason: 'TARGET' | 'RULE' | 'DEFAULT' | 'CLIENT' | 'OFF_VARIATION' | 'PREREQUISITE'
+  readonly reason:
+    | 'TARGET'
+    | 'RULE'
+    | 'DEFAULT'
+    | 'CLIENT'
+    | 'OFF_VARIATION'
+    | 'PREREQUISITE'
 }

Diff for: src/BKTEvaluation.ts

@@ -1,4 +1,7 @@
-import { ErrorMetricsEventType, MetricsEventType } from './internal/model/MetricsEventData'
+import {
+  ErrorMetricsEventType,
+  MetricsEventType,
+} from './internal/model/MetricsEventData'
 
 abstract class BKTBaseException extends Error {
   type?: ErrorMetricsEventType = undefined
@@ -39,7 +42,6 @@ export class ServiceUnavailableException extends BKTBaseException {
   type?: ErrorMetricsEventType = MetricsEventType.ServiceUnavailableError
 }
 
-
 // network errors
 export class TimeoutException extends BKTBaseException {
   type?: ErrorMetricsEventType = MetricsEventType.TimeoutError
@@ -55,4 +57,16 @@ export class IllegalStateException extends BKTBaseException {}
 // unknown errors
 export class UnknownException extends BKTBaseException {}
 
-export type BKTException = BadRequestException | UnauthorizedException | ForbiddenException | NotFoundException | ClientClosedRequestException | InternalServerErrorException | ServiceUnavailableException | TimeoutException | NetworkException | IllegalArgumentException | IllegalStateException | UnknownException
+export type BKTException =
+  | BadRequestException
+  | UnauthorizedException
+  | ForbiddenException
+  | NotFoundException
+  | ClientClosedRequestException
+  | InternalServerErrorException
+  | ServiceUnavailableException
+  | TimeoutException
+  | NetworkException
+  | IllegalArgumentException
+  | IllegalStateException
+  | UnknownException

Diff for: src/BKTExceptions.ts

@@ -8,7 +8,6 @@ export interface BKTUser extends RawBKTUser {
   readonly attributes: Record<string, string>
 }
 
-
 export const defineBKTUser = (user: RawBKTUser): BKTUser => {
   if (!user.id) {
     throw new Error('user id is required')
@@ -17,7 +16,7 @@ export const defineBKTUser = (user: RawBKTUser): BKTUser => {
   return {
     id: user.id,
     attributes: {
-      ...user.customAttributes
-    }
+      ...user.customAttributes,
+    },
   }
 }

Diff for: src/BKTUser.ts

@@ -1,4 +1,3 @@
-
 export interface IdGenerator {
   newId(): string
 }

Diff for: src/internal/IdGenerator.ts

@@ -5,9 +5,7 @@ export class UserHolder {
   private user: User
   userId: string
 
-  constructor(
-    user: User,
-  ) {
+  constructor(user: User) {
     this.user = user
     this.userId = this.user.id
   }
@@ -16,7 +14,9 @@ export class UserHolder {
     return this.user
   }
 
-  updateAttributes(updater: (previous: Record<string, string>) => Record<string, string>): void {
+  updateAttributes(
+    updater: (previous: Record<string, string>) => Record<string, string>,
+  ): void {
     this.user = {
       ...this.user,
       data: updater(this.user.data ?? {}),

Diff for: src/internal/UserHolder.ts

@@ -33,7 +33,7 @@ export class DefaultComponent implements Component {
       this._evaluationInteractor = this.interactorModule.evaluationInteractor(
         this.dataModule.apiClient(),
         this.dataModule.evaluationStorage(),
-        this.dataModule.idGenerator()
+        this.dataModule.idGenerator(),
       )
     }
     return this._evaluationInteractor

Diff for: src/internal/di/Component.ts

@@ -1,6 +1,9 @@
 import { BKTConfig } from '../../BKTConfig'
 import { Clock, DefaultClock } from '../Clock'
-import { EvaluationStorage, EvaluationStorageImpl } from '../evaluation/EvaluationStorage'
+import {
+  EvaluationStorage,
+  EvaluationStorageImpl,
+} from '../evaluation/EvaluationStorage'
 import { EventStorage, EventStorageImpl } from '../event/EventStorage'
 import { DefaultIdGenerator, IdGenerator } from '../IdGenerator'
 import { User } from '../model/User'
@@ -17,10 +20,7 @@ export class DataModule {
   private _evaluationStorage?: EvaluationStorage
   private _eventStorage?: EventStorage
 
-  constructor(
-    user: User,
-    config: BKTConfig,
-  ) {
+  constructor(user: User, config: BKTConfig) {
     this._config = config
     this._userHolder = new UserHolder(user)
   }
@@ -64,7 +64,7 @@ export class DataModule {
     if (!this._evaluationStorage) {
       this._evaluationStorage = new EvaluationStorageImpl(
         this.userHolder().userId,
-        new DefaultStorage(`${this.config().storageKeyPrefix}_bkt_evaluations`)
+        new DefaultStorage(`${this.config().storageKeyPrefix}_bkt_evaluations`),
       )
     }
     return this._evaluationStorage
@@ -74,7 +74,7 @@ export class DataModule {
     if (!this._eventStorage) {
       this._eventStorage = new EventStorageImpl(
         this.userHolder().userId,
-        new DefaultStorage(`${this.config().storageKeyPrefix}_bkt_events`)
+        new DefaultStorage(`${this.config().storageKeyPrefix}_bkt_events`),
       )
     }
     return this._eventStorage

Diff for: src/internal/di/DataModule.ts

@@ -12,11 +12,7 @@ export class InteractorModule {
     evaluationStorage: EvaluationStorage,
     idGenerator: IdGenerator,
   ): EvaluationInteractor {
-    return new EvaluationInteractor(
-      apiClient,
-      evaluationStorage,
-      idGenerator,
-    )
+    return new EvaluationInteractor(apiClient, evaluationStorage, idGenerator)
   }
 
   eventInteractor(

Diff for: src/internal/di/InteractorModule.ts

@@ -10,15 +10,23 @@ export class EvaluationInteractor {
     private apiClient: ApiClient,
     private evaluationStorage: EvaluationStorage,
     private idGenerator: IdGenerator,
-  ) { }
+  ) {}
 
   // visible for testing. should only be accessed from test code
   updateListeners: Record<string, () => void> = {}
 
-  async fetch(user: User, timeoutMillis?: number): Promise<GetEvaluationsResult> {
-    const currentEvaluationsId = this.evaluationStorage.getCurrentEvaluationsId() ?? ''
+  async fetch(
+    user: User,
+    timeoutMillis?: number,
+  ): Promise<GetEvaluationsResult> {
+    const currentEvaluationsId =
+      this.evaluationStorage.getCurrentEvaluationsId() ?? ''
 
-    const result = await this.apiClient.getEvaluations(user, currentEvaluationsId, timeoutMillis)
+    const result = await this.apiClient.getEvaluations(
+      user,
+      currentEvaluationsId,
+      timeoutMillis,
+    )
 
     if (result.type === 'success') {
       const response = result.value
@@ -31,10 +39,10 @@ export class EvaluationInteractor {
 
       this.evaluationStorage.deleteAllAndInsert(
         response.userEvaluationsId,
-        response.evaluations.evaluations ?? []
+        response.evaluations.evaluations ?? [],
       )
 
-      Object.values(this.updateListeners).forEach(listener => listener())
+      Object.values(this.updateListeners).forEach((listener) => listener())
     }
 
     return result