[25.06.19 / TASK-207] Refactor - 프로젝트 전반 리팩토링 (#43)

Author: six-standard

.env.sample

@@ -1,11 +1,7 @@
 '.env.development (개발 환경), .env.production (배포 환경)'
 
 NEXT_PUBLIC_BASE_URL=<'server url here'>
-NEXT_PUBLIC_VELOG_URL=https://velog.io
-NEXT_PUBLIC_ABORT_MS=<'abort time(ms) for fetch here'>
-NEXT_PUBLIC_SENTRY_AUTH_TOKEN=<'sentry auth token here'>
 NEXT_PUBLIC_CHANNELTALK_PLUGIN_KEY=<'channelTalk plugin key here'>
 NEXT_PUBLIC_GA_ID=<'Google Analytics ID here'>
-NEXT_PUBLIC_EVENT_LOG=<'Whether to send an event log here (true | false)'>
-NEXT_PUBLIC_SENTRY_DSN=<'sentry dsn here'>
-NEXT_PUBLIC_ARCADE_URL=<'how-to-use url here'>
+NEXT_PUBLIC_SENTRY_AUTH_TOKEN=<'sentry auth token here'>
+NEXT_PUBLIC_SENTRY_DSN=<'sentry dsn here'>

.eslintrc.js

@@ -28,7 +28,10 @@ module.exports = {
   ],
   rules: {
     'prettier/prettier': ['error', { printWidth: 100 }],
-    'import/order': ['error', { groups: ['builtin', 'external', 'internal'] }],
+    'import/order': [
+      'error',
+      { groups: ['builtin', 'external', 'internal'], alphabetize: { order: 'asc' } },
+    ],
     'no-restricted-imports': ['warn', { patterns: ['../../*'] }],
     'react/react-in-jsx-scope': 'off',
     'testing-library/no-container': 'warn',

sentry.client.config.ts

@@ -3,10 +3,11 @@
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
 import * as Sentry from '@sentry/nextjs';
-import { env } from '@/constants';
+
+import { ENVS } from '@/constants';
 
 Sentry.init({
-  dsn: env.SENTRY_DSN,
+  dsn: ENVS.SENTRY_DSN,
   release: 'production',
 
   // Add optional integrations for additional features
@@ -25,5 +26,5 @@ Sentry.init({
 
   // Setting this option to true will print useful information to the console while you're setting up Sentry.
   debug: false,
-  enabled: process.env.NODE_ENV === 'production',
+  enabled: ENVS.NODE_ENV === 'production',
 });

sentry.edge.config.ts

@@ -4,16 +4,16 @@
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
 import * as Sentry from '@sentry/nextjs';
-import { env } from '@/constants';
+import { ENVS } from '@/constants';
 
 Sentry.init({
-  dsn: env.SENTRY_DSN,
+  dsn: ENVS.SENTRY_DSN,
   release: 'production',
 
   // Define how likely traces are sampled. Adjust this value in production, or use tracesSampler for greater control.
   tracesSampleRate: 0.05,
 
   // Setting this option to true will print useful information to the console while you're setting up Sentry.
   debug: false,
-  enabled: process.env.NODE_ENV === 'production',
+  enabled: ENVS.NODE_ENV === 'production',
 });

sentry.server.config.ts

@@ -3,16 +3,16 @@
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/
 
 import * as Sentry from '@sentry/nextjs';
-import { env } from '@/constants';
+import { ENVS } from '@/constants';
 
 Sentry.init({
-  dsn: env.SENTRY_DSN,
+  dsn: ENVS.SENTRY_DSN,
   release: 'production',
 
   // Define how likely traces are sampled. Adjust this value in production, or use tracesSampler for greater control.
   tracesSampleRate: 0.1,
 
   // Setting this option to true will print useful information to the console while you're setting up Sentry.
   debug: false,
-  enabled: process.env.NODE_ENV === 'production',
+  enabled: ENVS.NODE_ENV === 'production',
 });

src/utils/componentUtil.tsx src/__test__/instance.test.tsxsrc/utils/componentUtil.tsx renamed to src/__test__/instance.test.tsx

@@ -1,6 +1,6 @@
 import { render, RenderResult } from '@testing-library/react';
 import { ReactElement } from 'react';
-import { QueryProvider } from '@/components';
+import { QueryProvider } from '@/app/components/Provider/QueryProvider';
 
 export const renderWithQueryClient = (element: ReactElement): RenderResult =>
   render(<QueryProvider>{element}</QueryProvider>);

src/__test__/login.test.tsx

@@ -1,9 +1,9 @@
-import { userEvent } from '@testing-library/user-event';
 import { act, screen } from '@testing-library/react';
-import { ToastContainer } from 'react-toastify';
+import { userEvent } from '@testing-library/user-event';
 import { useRouter } from 'next/navigation';
-import { renderWithQueryClient } from '@/utils/componentUtil';
+import { ToastContainer } from 'react-toastify';
 import { default as Login } from '@/app/(login)/page';
+import { renderWithQueryClient } from './instance.test';
 
 jest.mock('next/navigation', () => ({
   useRouter: jest.fn(),

src/__test__/main.test.tsx

@@ -1,7 +1,7 @@
 import { screen, waitFor } from '@testing-library/react';
-import { renderWithQueryClient } from '@/utils/componentUtil';
 import { Content } from '@/app/(auth-required)/main/Content';
-import { Header } from '@/app/(auth-required)/components/header';
+import { Header } from '@/app/components/Header';
+import { renderWithQueryClient } from './instance.test';
 
 jest.mock('next/navigation', () => ({
   useSearchParams: () => ({

src/apis/dashboard.request.ts

@@ -1,5 +1,5 @@
-import { PostDetailDto, PostListDto, PostSummaryDto, TotalStatsDto } from '@/types';
 import { PATHS, SidebarIdType } from '@/constants';
+import { PostDetailDto, PostListDto, PostSummaryDto, TotalStatsDto } from '@/types';
 
 import { instance } from './instance.request';
 

src/apis/instance.request.ts

@@ -1,8 +1,9 @@
-import returnFetch, { FetchArgs } from 'return-fetch';
-
 import { captureException, setContext } from '@sentry/nextjs';
+import returnFetch, { FetchArgs } from 'return-fetch';
+import { ENVS } from '@/constants';
 import { ServerNotRespondingError } from '@/errors';
-import { env } from '@/constants';
+
+const ABORT_MS = 10000;
 
 type ErrorType = {
   code: string;
@@ -27,7 +28,7 @@ const abortPolyfill = (ms: number) => {
 };
 
 const fetch = returnFetch({
-  baseUrl: env.BASE_URL,
+  baseUrl: ENVS.BASE_URL,
   headers: {
     Accept: 'application/json',
     'Content-Type': 'application/json',
@@ -66,8 +67,8 @@ export const instance = async <I, R>(
         : init?.headers,
       body: init?.body ? JSON.stringify(init.body) : undefined,
       signal: AbortSignal.timeout
-        ? AbortSignal.timeout(Number(env.ABORT_MS))
-        : abortPolyfill(Number(env.ABORT_MS)),
+        ? AbortSignal.timeout(Number(ABORT_MS))
+        : abortPolyfill(Number(ABORT_MS)),
       credentials: 'include',
       cache: 'no-store',
     });