refactor: remove dead code and align with effect best practices

AGENTS.md

@@ -116,7 +116,7 @@ Schema.String.pipe(
 | File | Purpose |
 |------|---------|
 | `defaultFetcher.ts` | HTTP client — Effect.gen, retry 3x exponential backoff, Match |
-| `effectErrorHandler.ts` | `runSafePromise`, `runSafeSync`, `unwrapCause` |
+| `effectErrorHandler.ts` | `runSafePromise`, `unwrapCause` |
 | `authenticator.ts` | HMAC-SHA256 auth header |
 | `stringifyQuery.ts` | URL query string builder (array handling) |
 | `fileToBase64.ts` | File/URL → Base64 |

CLAUDE.md

@@ -49,7 +49,7 @@ pnpm docs             # Generate TypeDoc documentation
 - 에러: `Data.TaggedError` + environment-aware `toString()`
 - 비동기: `Effect.gen` + `Effect.tryPromise`
 - 검증: Effect Schema (`Schema.filter`, `Schema.transform`)
-- Promise 변환: `runSafePromise()` / `runSafeSync()`
+- Promise 변환: `runSafePromise()`
 
 ### Path Aliases
 ```

src/lib/defaultFetcher.ts

@@ -14,13 +14,31 @@ type DefaultRequest = {
   method: string;
 };
 
+const ERROR_MESSAGE_PREVIEW_LENGTH = 200;
+const RETRYABLE_ERROR_KEYWORDS = [
+  'aborted',
+  'refused',
+  'reset',
+  'econn',
+] as const;
+
 class RetryableError extends Data.TaggedError('RetryableError')<{
   readonly error?: unknown;
 }> {}
 
 const toMessage = (e: unknown): string =>
   e instanceof Error ? e.message : String(e);
 
+const isRetryableNetworkError = (error: Error): boolean => {
+  const cause = error.cause;
+  const causeCode =
+    cause && typeof cause === 'object' && 'code' in cause
+      ? String(cause.code)
+      : '';
+  const message = `${error.message} ${causeCode}`.toLowerCase();
+  return RETRYABLE_ERROR_KEYWORDS.some(keyword => message.includes(keyword));
+};
+
 const makeParseError = (res: Response, message: string) =>
   new DefaultError({
     errorCode: 'ParseError',
@@ -62,7 +80,9 @@ const handleClientErrorResponse = (res: Response) =>
     Effect.flatMap(text => {
       const genericError = new ClientError({
         errorCode: `HTTP_${res.status}`,
-        errorMessage: text.substring(0, 200) || 'Client error occurred',
+        errorMessage:
+          text.substring(0, ERROR_MESSAGE_PREVIEW_LENGTH) ||
+          'Client error occurred',
         httpStatus: res.status,
         url: res.url,
       });
@@ -148,7 +168,8 @@ const handleServerErrorResponse = (res: Response) =>
 
       const genericError = makeError(
         `HTTP_${res.status}`,
-        text.substring(0, 200) || 'Server error occurred',
+        text.substring(0, ERROR_MESSAGE_PREVIEW_LENGTH) ||
+          'Server error occurred',
       );
 
       return parseServerErrorBody(text, genericError, makeError);
@@ -191,18 +212,7 @@ export function defaultFetcherEffect<T, R>(
         }),
       catch: (error: unknown) => {
         if (error instanceof Error) {
-          const cause = error.cause;
-          const causeCode =
-            cause && typeof cause === 'object' && 'code' in cause
-              ? String(cause.code)
-              : '';
-          const message = (error.message + ' ' + causeCode).toLowerCase();
-          const isRetryable =
-            message.includes('aborted') ||
-            message.includes('refused') ||
-            message.includes('reset') ||
-            message.includes('econn');
-          if (isRetryable) {
+          if (isRetryableNetworkError(error)) {
             return new RetryableError({error});
           }
           return new NetworkError({

src/lib/effectErrorHandler.ts

@@ -76,16 +76,6 @@ const unwrapCause = (cause: Cause.Cause<unknown>): unknown => {
   return new UnhandledExitError({message});
 };
 
-export const runSafeSync = <E, A>(effect: Effect.Effect<A, E>): A => {
-  const exit = Effect.runSyncExit(effect);
-  return Exit.match(exit, {
-    onFailure: cause => {
-      throw unwrapCause(cause);
-    },
-    onSuccess: value => value,
-  });
-};
-
 export const runSafePromise = <E, A>(
   effect: Effect.Effect<A, E>,
 ): Promise<A> => {

src/lib/schemaUtils.ts

@@ -1,11 +1,13 @@
-import {Schema} from 'effect';
+import {ParseResult, Schema} from 'effect';
 import * as Effect from 'effect/Effect';
 import {BadRequestError, InvalidDateError} from '../errors/defaultError';
 import stringDateTransfer, {formatWithTransfer} from './stringDateTransfer';
 
 /**
  * Schema 디코딩 + BadRequestError 변환을 결합한 Effect 헬퍼.
  * 서비스 레이어에서 반복되는 검증 패턴을 통일합니다.
+ * Effect 공식 ParseResult 포맷터(TreeFormatter/ArrayFormatter)로
+ * 에러 경로를 구조화하여 디버깅 가능성을 높입니다.
  */
 export const decodeWithBadRequest = <A, I>(
   schema: Schema.Schema<A, I>,
@@ -15,7 +17,11 @@ export const decodeWithBadRequest = <A, I>(
     Schema.decodeUnknown(schema)(data),
     error =>
       new BadRequestError({
-        message: error.message,
+        message: ParseResult.TreeFormatter.formatErrorSync(error),
+        validationErrors: ParseResult.ArrayFormatter.formatErrorSync(error).map(
+          issue =>
+            `${issue.path.length > 0 ? issue.path.join('.') : '(root)'}: ${issue.message}`,
+        ),
       }),
   );
 

src/models/base/kakao/kakaoAlimtalkTemplate.ts

@@ -1,7 +1,7 @@
+import {type InvalidDateError} from '@errors/defaultError';
 import {safeDateTransfer} from '@lib/schemaUtils';
 import {Schema} from 'effect';
 import * as Effect from 'effect/Effect';
-import {type InvalidDateError} from '@/errors/defaultError';
 import {kakaoAlimtalkTemplateQuickReplySchema} from './kakaoAlimtalkTemplateQuickReply';
 import {kakaoButtonSchema} from './kakaoButton';
 import {type KakaoChannelCategory} from './kakaoChannel';

src/models/base/kakao/kakaoChannel.ts

@@ -1,7 +1,7 @@
+import {type InvalidDateError} from '@errors/defaultError';
 import {safeDateTransfer} from '@lib/schemaUtils';
 import {Schema} from 'effect';
 import * as Effect from 'effect/Effect';
-import {type InvalidDateError} from '@/errors/defaultError';
 
 /**
  * @description 카카오 채널 카테고리 타입

src/models/base/messages/message.ts

@@ -1,8 +1,8 @@
 import {baseKakaoOptionSchema} from '@models/base/kakao/kakaoOption';
 import {naverOptionSchema} from '@models/base/naver/naverOption';
 import {rcsOptionSchema} from '@models/base/rcs/rcsOption';
+import {voiceOptionSchema} from '@models/requests/voice/voiceOption';
 import {Schema} from 'effect';
-import {voiceOptionSchema} from '@/models/requests/voice/voiceOption';
 
 export const messageTypeSchema = Schema.Literal(
   'SMS',

src/models/base/naver/naverOption.ts

@@ -1,6 +1,5 @@
 import {Schema} from 'effect';
 
-// 네이버 스마트 알림 naverOptions 버튼 스키마
 const naverOptionButtonSchema = Schema.Struct({
   buttonName: Schema.String,
   buttonType: Schema.String,
@@ -10,7 +9,6 @@ const naverOptionButtonSchema = Schema.Struct({
   linkIos: Schema.optional(Schema.String),
 });
 
-// naverOptions 최상위 스키마
 export const naverOptionSchema = Schema.Struct({
   talkId: Schema.String,
   templateId: Schema.String,

src/models/index.ts

@@ -1,7 +1,5 @@
-// Base Models - Kakao BMS
 export * from './base/kakao/bms';
 
-// Base Models - Kakao
 export {
   decodeKakaoAlimtalkTemplate,
   type KakaoAlimtalkTemplate,
@@ -67,12 +65,10 @@ export {
   messageSchema,
   messageTypeSchema,
 } from './base/messages/message';
-// Base Models - Naver
 export {
   type NaverOptionSchema,
   naverOptionSchema,
 } from './base/naver/naverOption';
-// Base Models - RCS
 export {
   type RcsButton,
   type RcsButtonSchema,
@@ -88,8 +84,5 @@ export {
   rcsOptionSchema,
 } from './base/rcs/rcsOption';
 
-// Requests
 export * from './requests/index';
-
-// Responses
 export * from './responses/index';

src/models/requests/messages/getMessagesRequest.ts

@@ -20,7 +20,6 @@ const baseGetMessagesRequestSchema = Schema.Struct({
   endDate: Schema.optional(Schema.Union(Schema.String, Schema.DateFromSelf)),
 });
 
-// dateType은 startDate 또는 endDate가 함께 제공될 때만 유효
 export const getMessagesRequestSchema = baseGetMessagesRequestSchema.pipe(
   Schema.filter(data => {
     const hasDate = data.startDate != null || data.endDate != null;
@@ -59,7 +58,6 @@ export type GetMessagesRequest =
   | GetMessagesRequestWithStartDate
   | GetMessagesRequestWithEndDate;
 
-// 스키마 디코딩 결과 타입 (런타임 검증 후 내부에서 사용)
 type GetMessagesRequestDecoded = Schema.Schema.Type<
   typeof getMessagesRequestSchema
 >;

src/services/messages/groupService.ts

@@ -15,6 +15,10 @@ import {
   ScheduledDateSendingRequest,
 } from '@models/requests/messages/groupMessageRequest';
 import {osPlatform, sdkVersion} from '@models/requests/messages/requestConfig';
+import {
+  type RequestSendMessagesSchema,
+  requestSendMessageSchema,
+} from '@models/requests/messages/sendMessage';
 import {
   AddMessageResponse,
   GetGroupsResponse,
@@ -23,10 +27,6 @@ import {
   RemoveGroupMessagesResponse,
 } from '@models/responses/messageResponses';
 import * as Effect from 'effect/Effect';
-import {
-  type RequestSendMessagesSchema,
-  requestSendMessageSchema,
-} from '@/models/requests/messages/sendMessage';
 import DefaultService from '../defaultService';
 
 /**

src/types/commonTypes.ts

@@ -1,7 +1,5 @@
 import {Schema} from 'effect';
 
-// --- Count & Charge Types ---
-
 export const countSchema = Schema.Struct({
   total: Schema.Number,
   sentTotal: Schema.Number,
@@ -62,8 +60,6 @@ export type MessageTypeRecord = Schema.Schema.Type<
   typeof messageTypeRecordSchema
 >;
 
-// --- App & Log ---
-
 export const appSchema = Schema.Struct({
   profit: messageTypeRecordSchema,
   appId: Schema.NullishOr(Schema.String),
@@ -75,8 +71,6 @@ export const logSchema = Schema.Array(
 );
 export type Log = Schema.Schema.Type<typeof logSchema>;
 
-// --- Group ---
-
 export const groupIdSchema = Schema.String;
 export type GroupId = Schema.Schema.Type<typeof groupIdSchema>;
 
@@ -101,13 +95,9 @@ export const groupSchema = Schema.Struct({
 });
 export type Group = Schema.Schema.Type<typeof groupSchema>;
 
-// --- Handle Key ---
-
 export const handleKeySchema = Schema.String;
 export type HandleKey = Schema.Schema.Type<typeof handleKeySchema>;
 
-// --- Black (080 수신거부) ---
-
 export const blackSchema = Schema.Struct({
   handleKey: handleKeySchema,
   type: Schema.Literal('DENIAL'),
@@ -118,8 +108,6 @@ export const blackSchema = Schema.Struct({
 });
 export type Black = Schema.Schema.Type<typeof blackSchema>;
 
-// --- Block Group ---
-
 export const blockGroupSchema = Schema.Struct({
   blockGroupId: Schema.String,
   accountId: Schema.String,
@@ -132,8 +120,6 @@ export const blockGroupSchema = Schema.Struct({
 });
 export type BlockGroup = Schema.Schema.Type<typeof blockGroupSchema>;
 
-// --- Block Number ---
-
 export const blockNumberSchema = Schema.Struct({
   blockNumberId: Schema.String,
   accountId: Schema.String,