불필요한 의존성 삭제

[Bangdayeon]

관련 이슈

• close 필요없는 의존성 삭제 및 파일 정리 #255

PR 설명

• @storybook/nextjs-vite, @storybook/nextjs가 혼용되고 있어, @storybook/next로 통일
  • Next.js 환경을 그대로 반영하는 공식 프레임워크 (nextjs-vite는 nextjs를 vite 기반으로 실행하는 프레임워크로, 실험적구성으로 인해 제약이 발생할 수 있음)
• playwright 제거
  • 실제 브라우저를 띄워 테스트를 자동화하는 테스트 도구
  • 현재 테스트 진행 계획이 없어 제거 (추후 필요할 시 조사 및 논의 후 사용)
• @vitest/coverage-v8 제거
  • Vitest 테스트 실행 시 코드 커버리지를 V8 엔진 기준으로 수집하는 플러그인
  • 위와 같음
• @emotion/* 제거
  • tailwind + css만 사용하고 있으므로 제거

[github-actions]

🚀 storybook url: https://689bfce92a7bd08f5aa1edbe-cabdlhrcoq.chromatic.com/

[coderabbitai]

Warning

Rate limit exceeded

@Bangdayeon has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 3 minutes and 54 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

이 풀 리퀘스트는 Storybook 패키지 마이그레이션과 함께 여러 관련 변경사항을 포함합니다. @storybook/nextjs-vite에서 @storybook/nextjs로 30개 이상의 스토리 파일에서 타입 임포트를 변경하고, .storybook/preview.ts에 nextjs.appDirectory 설정을 추가합니다. 동시에 package.json에서 불필요한 의존성(Emotion 관련 패키지, Playwright, Vitest coverage)을 제거하고, 여러 파일에서 SafeFetch 및 에러 타입 임포트 경로를 서버측에서 클라이언트측으로 변경합니다.

Possibly related PRs

• Storybook 8 -> 10 업데이트 #253: 동일한 Storybook 타입 임포트와 .storybook/preview.ts 구성 변경사항을 다루며, 패키지 마이그레이션과 직접적인 코드 수준의 연관성이 있습니다.
• 회원가입 기능 추가, safeFetch 정리 #337: SafeFetch 및 FetchError, TimeoutError, ParseError 타입의 임포트 경로와 사용법을 수정하여 동일한 fetch/에러 리팩토링에 관련됩니다.
• Chat API 구현 및 관련 소켓 모듈 추가 #280: 클라이언트측 SafeFetch 및 에러 유틸리티를 다루며, 본 PR의 API 모듈 임포트 경로 변경과 코드 수준에서 연관됩니다.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	불필요한 의존성 제거 목표는 충족되었으나, 연결된 이슈 #255에서 요구하는 tsconfig.json의 jsx 설정 변경이 누락되었습니다.	tsconfig.json의 jsx 설정을 'react-jsx'에서 'preserve'로 변경하는 작업을 추가하여 이슈 #255의 모든 요구사항을 완료하세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용을 명확하게 요약하고 있습니다.
Description check	✅ Passed	PR 설명이 템플릿을 따르고 각 의존성 제거 이유를 상세히 설명하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 불필요한 의존성 제거 및 Storybook 임포트 경로 통일과 관련이 있으며, 범위 내에 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch task/#255-remove-unused-dependencies

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.storybook/preview.ts:
- Around line 16-23: The Storybook nextjs config currently sets
nextjs.appDirectory: true but places route data under nextjs.router, which App
Router ignores; change the key nextjs.router to nextjs.navigation and move the
routing properties (pathname, query, asPath) under nextjs.navigation so the App
Router environment is correctly simulated for Storybook.

🧹 Nitpick comments (3)

src/stories/ProgressNotification.stories.tsx (1)

3-3: 공식 Storybook 권장사항과의 불일치 확인 필요.

Storybook 공식 문서에 따르면 대부분의 Next.js 프로젝트에 @storybook/nextjs-vite를 권장하고 있습니다. @storybook/nextjs(Webpack 기반)는 custom Webpack/Babel 설정이 필수적인 경우에만 사용하도록 안내하고 있습니다.

현재 코드베이스의 모든 story 파일(28개 이상)이 @storybook/nextjs로 일괄 마이그레이션되었으며, PR #255 "Chore: remove unused-dependencies"에서 진행된 것으로 보입니다. Meta와 StoryObj 타입은 두 패키지 모두에서 정상 지원되므로 기능적 문제는 없습니다.

다만, 공식 권장사항과 현재 상태가 맞지 않으므로 다음을 확인해 주세요:

• 이 마이그레이션이 의도적으로 진행된 것인지, 특정 의존성 이슈가 있었는지
• 혹은 @storybook/nextjs-vite로 복원하는 것이 나을지

src/stories/ReportModal.stories.tsx (1)

4-4: import type을 사용하여 다른 스토리 파일과 일관성을 유지하세요.

다른 스토리 파일들(Tab.stories.tsx, Tooltip.stories.tsx, Label.stories.tsx, Dropdown.stories.tsx 등)은 모두 import type { Meta, StoryObj }를 사용하고 있지만, 이 파일만 import { Meta, StoryObj }로 되어 있습니다. Meta와 StoryObj는 타입 전용이므로 import type을 사용하는 것이 적절합니다.

♻️ 수정 제안

-import { Meta, StoryObj } from '@storybook/nextjs';
+import type { Meta, StoryObj } from '@storybook/nextjs';

src/stories/Popover.stories.tsx (1)

5-5: type 키워드 없이 타입만 임포트하고 있습니다.

Meta와 StoryObj는 타입으로만 사용되므로, Footer.stories.tsx처럼 import type을 사용하는 것이 일관성과 번들 최적화 측면에서 좋습니다.

-import { Meta, StoryObj } from '@storybook/nextjs';
+import type { Meta, StoryObj } from '@storybook/nextjs';

[github-actions]

🚀 storybook url: https://689bfce92a7bd08f5aa1edbe-gvzdetfgmc.chromatic.com/