refactor: 기획 변경에 따른 홈화면 아티클 영역 제거 (#130)#131
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Walkthrough홈 화면 페이지에서 StoreStory와 RecentlySupportedStores 컴포넌트의 임포트 및 JSX 사용을 제거했습니다. 나머지 레이아웃과 흐름은 변경되지 않았습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Assessment against linked issues
Poem
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
🎨 Storybook Preview: https://685a32a1c0bbd269fdb67af4-nqascgwjeb.chromatic.com/ |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/app/(home)/page.tsx (1)
17-21: 여백 과잉 및 단일 자식 VStack 제거 제안컴포넌트 제거 이후에도 Spacer가 2개(12, 32) 연속이고, VStack은 자식이 1개뿐이라 불필요합니다. 섹션 간 여백을 1개로 유지하고, VStack을 제거해 DOM 복잡도를 줄이는 것을 권장합니다.
아래처럼 단순화할 수 있습니다:
- <Spacer size={12} /> - <Spacer size={32} /> - <VStack gap={40}> - <RecentCheers /> - </VStack> + <Spacer size={32} /> + <RecentCheers />추가로, 위 변경을 적용한다면 상단 import의 VStack도 함께 제거하세요(라인 5 인근):
import { Bleed } from "@/components/ui/Bleed"; import { Spacer } from "@/components/ui/Spacer"; // import { VStack } from "@/components/ui/Stack"; // 제거
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/app/(home)/page.tsx(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
src/**/*.{ts,tsx}
📄 CodeRabbit Inference Engine (.cursor/rules/nextjs-folder-structure.mdc)
All files must use TypeScript.
Files:
src/app/(home)/page.tsx
⏰ 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). (1)
- GitHub Check: test
🔇 Additional comments (2)
src/app/(home)/page.tsx (2)
7-7: 임포트 정리 LGTM — 제거 대상 컴포넌트 반영 완료StoreStory/RecentlySupportedStores 제거 의도대로 임포트가 정리되었습니다. 남은 사용 컴포넌트만 명시되어 명확합니다.
7-21: PR 설명과 구현 상태 불일치 가능성 확인 요청PR 설명에는 “이번주 인기 가게 컴포넌트 제거 (not completed)”로 표기되어 있으나, 본 파일 기준으로는 해당 영역이 보이지 않습니다. 실제로 전역에서 완전히 제거되었는지, 아니면 다른 화면에 잔존하는지 확인 후 PR 설명/체크리스트를 최신 상태로 맞춰 주세요.
| StoreStory, | ||
| Story, | ||
| } from "./_components"; | ||
| import { RecentCheers, RegisterPopup, Story } from "./_components"; |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
제거된 컴포넌트의 사용처/재-export/파일 잔존 여부 점검 권장
홈 페이지에서는 제거되었지만, 레포 전반에서 StoreStory/RecentlySupportedStores 관련 잔존 코드(다른 페이지 사용, _components/index 재-export, 파일 자체)가 있는지 확인해 주세요. 아래 스크립트로 자동 점검 가능합니다. 필요 시 잔여 파일/라인 제거도 도와드릴게요.
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "1) 코드베이스에서 제거된 컴포넌트/카피 검색"
rg -n -S 'StoreStory|RecentlySupportedStores|이번주\s*인기' || true
echo
echo "2) home 하위 _components/index.ts(x) 재-export 잔존 여부 확인"
fd -a -t f --glob 'src/app/(home)/**/_components/**/index.{ts,tsx}' | while read -r idx; do
echo " - $idx"
rg -n 'export\s+\{[^}]*\b(StoreStory|RecentlySupportedStores)\b' "$idx" || true
done
echo
echo "3) 컴포넌트 파일 자체가 남아있는지 확인"
fd -a -t f '(StoreStory|RecentlySupportedStores)\.(ts|tsx)' src || trueLength of output: 2498
StoreStory 및 RecentlySupportedStores 관련 잔존 코드 삭제 필요
아래 경로에서 홈 페이지에서 제거된 두 컴포넌트가 여전히 re-export 또는 파일 형태로 남아있습니다. 사용되지 않는다면 관련 코드를 삭제해주세요.
- src/app/(home)/_components/index.ts
• 2행: export { RecentlySupportedStores } from "./RecentlySupportStories";
• 4행: export { StoreStory } from "./StoreStory"; - src/app/(home)/_components/RecentlySupportStories/RecentlySupportStories.tsx
• RecentlySupportedStores 컴포넌트 구현 및 연관 파일 잔존 - src/app/(home)/_components/StoreStory/StoreStory.tsx
• StoreStory 컴포넌트 구현 및 연관 파일 잔존
위 파일 및 re-export 문 제거 후 빌드 및 기능에 영향이 없는지 확인 부탁드립니다.
🤖 Prompt for AI Agents
In src/app/(home)/page.tsx around line 7, the import still references components
that were removed from the homepage; remove the unused import { RecentCheers,
RegisterPopup, Story } (or delete the specific removed names if others remain)
and then delete the stale re-exports and component files: remove the export
lines for RecentlySupportedStores and StoreStory from
src/app/(home)/_components/index.ts and delete the folders/files
src/app/(home)/_components/RecentlySupportStories/RecentlySupportStories.tsx and
src/app/(home)/_components/StoreStory/StoreStory.tsx (and any related
assets/tests). After removal, run the build and run the app to confirm no
remaining imports or references cause errors.
2 participants
✅ 이슈 번호
close #130
🪄 작업 내용 (변경 사항)
📸 스크린샷
💡 설명
🗣️ 리뷰어에게 전달 사항
📍 트러블 슈팅
Summary by CodeRabbit