Skip to content

[chore] Vite 전환 이후 남아있던 Webpack 패키지 정리#1098

Merged
oesnuj merged 4 commits intodevelop-fefrom
chore/#1097-remove-webpack-deps-MOA-560
Jan 25, 2026
Merged

[chore] Vite 전환 이후 남아있던 Webpack 패키지 정리#1098
oesnuj merged 4 commits intodevelop-fefrom
chore/#1097-remove-webpack-deps-MOA-560

Conversation

@oesnuj
Copy link
Member

@oesnuj oesnuj commented Jan 25, 2026

#️⃣연관된 이슈

#1097

📝작업 내용

Vite 마이그레이션 완료 후 불필요하게 남아있던 webpack 관련 파일 및 의존성을 제거했습니다.

삭제된 파일

  • config/webpack.common.ts
  • config/webpack.dev.ts
  • config/webpack.prod.ts

제거된 스크립트

  • webpack:dev
  • webpack:build:dev
  • webpack:build:prod

제거된 dependencies (2개)

  • @sentry/webpack-plugin
  • dotenv-webpack

제거된 devDependencies (18개)

  • @pmmmwh/react-refresh-webpack-plugin
  • @types/html-webpack-plugin
  • @types/react-refresh
  • @types/webpack
  • @types/webpack-dev-server
  • copy-webpack-plugin
  • css-loader
  • css-minimizer-webpack-plugin
  • detect-port
  • esbuild-loader
  • fork-ts-checker-webpack-plugin
  • html-webpack-plugin
  • mini-css-extract-plugin
  • react-refresh
  • style-loader
  • webpack
  • webpack-cli
  • webpack-dev-server
  • webpack-merge

스크립트 정리

  • build 스크립트 추가 - 기본 빌드 명령어 통일
  • preview 스크립트 추가 - 빌드 결과 로컬에서 미리보기 가능
  • build:dev, build:prod는 현재 동일한 명령어로 중복되어 안전하게 추후 제거하기 위해 build 명령어를 추가했습니다

중점적으로 리뷰받고 싶은 부분(선택)

  • 제거한 의존성 중 다른 곳에서 사용되고 있는 것이 있는지 확인 부탁드립니다.

🫡 참고사항

  • tsc --noEmit 실행 시 12개의 타입 에러가 발견됩니다. 이는 기존에 타입 체크 없이 개발되면서 누적된 에러로, 별도 PR에서 수정 예정입니다.

Summary by CodeRabbit

  • Chores
    • 빌드 프로세스를 위한 개발 도구 업데이트
    • 새로운 빌드 및 미리보기 스크립트 추가
    • 개발 의존성 업데이트

✏️ Tip: You can customize this high-level summary in your review settings.

Vite 마이그레이션 완료로 더 이상 사용하지 않는 webpack 설정 파일 제거
- config/webpack.common.ts
- config/webpack.dev.ts
- config/webpack.prod.ts
- webpack 관련 스크립트 제거 (webpack:dev, webpack:build:dev, webpack:build:prod)
- webpack 관련 dependencies 제거 (@sentry/webpack-plugin, dotenv-webpack)
- webpack 관련 devDependencies 18개 제거
- typecheck 스크립트 추가 (tsc --noEmit)
- npm run build 추가 (vite build)
- npm run preview 추가 (빌드 결과 미리보기)
- build:dev, build:prod는 추후 제거 예정
@vercel
Copy link

vercel bot commented Jan 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
moadong Ready Ready Preview, Comment Jan 25, 2026 3:22am

@oesnuj oesnuj added ⚙ Setting 개발 환경 세팅 💻 FE Frontend labels Jan 25, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 25, 2026

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

Webpack 설정 파일 세 개를 삭제하고 package.json에서 Webpack 관련 패키지 및 스크립트를 제거하여 Vite 마이그레이션을 완료합니다. 빌드 및 프리뷰 스크립트를 추가합니다.

Changes

응집군 / 파일 변경 내용
Webpack 설정 파일 제거
frontend/config/webpack.common.ts, frontend/config/webpack.dev.ts, frontend/config/webpack.prod.ts
공통, 개발, 프로덕션 Webpack 설정 파일 완전 삭제 (캐시 구성, 모듈 규칙, 플러그인 포함)
Package 의존성 정리
frontend/package.json
Webpack 관련 devDependencies 24개 제거 (@pmmmwh/react-refresh-webpack-plugin, webpack, webpack-cli, webpack-dev-server, css-loader, mini-css-extract-plugin 등), 런타임 의존성 2개 제거 (@sentry/webpack-plugin, dotenv-webpack), build 및 preview 스크립트 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 분

Possibly related issues

Possibly related PRs

  • [chore] 빌드 툴 Webpack to Vite 전환 #844: Vite 설정 추가 및 프론트엔드 package.json의 Vite로의 마이그레이션을 진행한 PR로, 이 PR의 Webpack 설정 제거와 직접적으로 관련됩니다.
  • [release] v1.1.10 #1006: Storybook과 package.json을 Vite로 전환한 PR로, 동일한 Webpack 제거 및 Vite 마이그레이션의 일부입니다.

Suggested labels

🚗 Chore

Suggested reviewers

  • lepitaaar
  • seongwon030
  • suhyun113
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: removing unnecessary Webpack packages after Vite migration, which aligns with the file deletions and dependency removals.
Linked Issues check ✅ Passed The PR addresses all coding requirements from MOA-560: removes Webpack config files, updates package.json scripts, removes Webpack dependencies, and updates lock file.
Out of Scope Changes check ✅ Passed All changes are directly related to Webpack removal and Vite migration as defined in MOA-560; no unrelated modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings

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

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

Copy link
Member

@seongwon030 seongwon030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

깔끔하네요 굳굳

Copy link
Contributor

@lepitaaar lepitaaar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

라이브러리 정리 좋습니다 수고하셨어요

@oesnuj oesnuj merged commit e220fa3 into develop-fe Jan 25, 2026
3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Jan 25, 2026
@oesnuj oesnuj deleted the chore/#1097-remove-webpack-deps-MOA-560 branch January 25, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend ⚙ Setting 개발 환경 세팅

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments