Fix#388 webpack 설정에서 SVG 파일 처리 규칙 제거

[toothlessdev]

✅ Linked Issue

• [✨ 기능 요청] 학교대항전 이벤트 페이지 구현 #388

Summary by CodeRabbit

• Chores
  • Updated asset handling configuration by removing a custom SVG processing rule in the build setup.
  • No user-visible changes are expected; performance and functionality remain the same for typical usage.
  • In rare cases, SVG assets may render differently depending on the environment—please report any discrepancies.
  • No action required for end-users.

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
mosu-client	Ready	Preview	Comment	Sep 11, 2025 3:41pm

[coderabbitai]

Walkthrough

Removed the custom webpack rule in Next.js config that applied @svgr/webpack to .svg files. No other configuration or exports were changed.

Changes

Cohort / File(s)	Summary
Build config cleanup mosu-app/next.config.ts	Deleted the webpack rule that added { test: /\.svg$/i, use: ["@svgr/webpack"] } to config.module.rules. No other edits.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

도메인 : 공통

Suggested reviewers

• kimgho

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "Fix#388 webpack 설정에서 SVG 파일 처리 규칙 제거" directly and accurately summarizes the primary change in this PR — removal of the SVG file handling rule from the webpack configuration (next.config.ts); it is concise, specific, and not vague.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

Poem

A nibble of config, a hop and a skip,
SVGs unyoked from the webpack clip.
I twitch my whiskers, tidy and neat—
Less is more in this garden of feat.
Thump-thump! The build still runs sweet. 🐇✨

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 07a1653 and df150c9.

📒 Files selected for processing (1)

• mosu-app/next.config.ts (0 hunks)

💤 Files with no reviewable changes (1)

• mosu-app/next.config.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Run DangerJS
• GitHub Check: Run Unit & Integration Tests

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature#388

---

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.}

[github-actions]

	Warnings
⚠️	PR에 Reviewers가 지정되어 있지 않습니다. 리뷰어를 지정해주세요.
⚠️	PR에 라벨이 지정되어 있지 않습니다. 적절한 라벨을 추가해주세요.
⚠️	PR에 Assignees가 지정되어 있지 않습니다. 적절한 Assignee를 추가해주세요.

	Messages
📖	✅ PR 제목에 이슈 번호가 포함되어 있습니다.
📖	✅ package.json에 변경사항이 없습니다.
📖	✅ 브랜치 이름 'feature#388'이 컨벤션을 따릅니다.
📖	✅ TypeScript 컴파일이 성공적으로 완료되었습니다.
📖	✅ ESLint 검사 결과 문제가 없습니다.

📝 추가 및 변경된 파일

총 1개 파일 변경

└── 📂 mosu-app/
    └── 📘 next.config.ts

Generated by 🚫 dangerJS against df150c9

[github-actions]

📚 Storybook이 Chromatic에 배포되었습니다!

• 🔗 Storybook Preview: https://www.chromatic.com/build?appId=68627ee20725077171386a15&number=234
• 🚀 Chromatic Build: https://www.chromatic.com/build?appId=68627ee20725077171386a15&number=234