Revert "Merge pull request #284 from THIP-TextHip/feat/influencer"

[ljh130334]

변경 사항

feat/influencer 브랜치의 변경사항을 일시적으로 제외합니다.

이유

• 배포 일정 조정을 위해 feat/influencer 작업을 제외한 나머지 변경사항을 먼저 배포합니다.
• feat/influencer 작업은 추후 별도로 배포 예정입니다.

제외된 내용

• 피드 추천 영역 (지금 뜨는 추천 글 섹션)
• Embla Carousel 기반 카드 가로 스크롤
• 공식 인플루언서 관련 UI
• 관련 목업 데이터 및 아이콘

다음 단계

1. 이 PR이 머지되면 develop → main PR 생성
2. main 배포 완료 후, 필요 시 feat/influencer 재머지

Summary by CodeRabbit

릴리즈 노트

• Style

  • 피드 포스트 하단 테두리 레이아웃 미세 조정

• Chores

  • 캐러셀 라이브러리 의존성 제거
  • 추천 피드 섹션 및 관련 컴포넌트 제거
  • 모의 데이터 정리

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
thip	Ready	Preview	Comment	Nov 11, 2025 8:47am

[coderabbitai]

Walkthrough

Embla carousel 라이브러리 및 관련 추천 피드 기능을 제거합니다. RecommendedFeedCard, RecommendedFeedSection 컴포넌트와 mock 데이터를 삭제하고, TotalFeed에서 10개 항목마다 추천 피드를 삽입하는 로직을 제거하며 isLast prop을 추가합니다.

Changes

Cohort / File(s)	변경 요약
패키지 의존성 제거 package.json	embla-carousel 및 embla-carousel-react 의존성 제거
추천 피드 기능 삭제 src/components/feed/RecommendedFeedCard.tsx, src/components/feed/RecommendedFeedSection.tsx, src/mocks/recommendedFeeds.mock.ts	RecommendedFeedCard 컴포넌트, RecommendedFeedSection 컴포넌트 및 관련 Embla carousel 설정, allMockRecommendedFeeds mock 데이터 전체 파일 삭제
피드 렌더링 로직 업데이트 src/components/feed/TotalFeed.tsx	Fragment 및 RecommendedFeedSection 임포트 제거, isLast prop 추가(기본값 false), 10개 항목 단위 추천 섹션 삽입 로직 제거, FeedPost에 isLast prop 전달
스타일 미세 조정 src/components/feed/FeedPost.tsx	BorderBottom 너비를 100%에서 94.8%로 조정

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• 주의 사항:
  • TotalFeed.tsx의 로직 변경이 기존 렌더링 동작을 올바르게 유지하는지 확인 필요 (isLast prop 전파 및 post 배열 매핑)
  • RecommendedFeedCard, RecommendedFeedSection, recommendedFeeds.mock.ts 삭제로 인한 미사용 임포트 및 타입 참조 확인
  • FeedPost의 너비 변경(94.8%)이 의도된 디자인 변경인지, UI 레이아웃에 미치는 영향 확인
  • 삭제된 컴포넌트/파일이 다른 부분에서 임포트되고 있지 않은지 확인

Possibly related PRs

• [FEAT] 인플루언서 추천 피드 - 지현 #284: 이 PR은 Embla 의존성, RecommendedFeedCard, RecommendedFeedSection 및 recommendedFeeds.mock.ts를 추가한 PR이고, 현재 PR이 이를 직접 제거하고 TotalFeed의 10개 항목 단위 삽입 로직을 역으로 변경합니다.
• fix: QA 3차 #178: src/components/feed/FeedPost.tsx를 수정하며, 현재 PR은 스타일 너비 조정, 검색된 PR은 onSaveToggle prop 추가 변경을 합니다.
• feat: add followlist page, followerlist page #48: src/components/feed/TotalFeed.tsx를 수정하며, 현재 PR은 isLast prop 추가 및 RecommendedFeedSection 제거, 검색된 PR은 빈 상태 렌더링을 추가합니다.

Suggested reviewers

• ho0010
• heeeeyong

Poem

🐰 회전목마 덤벼드는 Embla 안녕히~
추천 카드 몇 장 작별을 하고,
Feed는 더 깔끔해졌어, 참 좋은 일이야!
의존성 빼니 가벼워진 기분 들어,
94.8%의 경계도 예쁘네! ✨

Pre-merge checks and finishing touches

✅ 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 clearly and specifically describes the main change: reverting a previous merge of the influencer feature, which is exactly what all the file changes accomplish.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/revert-influencer

---

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: 0

🧹 Nitpick comments (1)

src/components/feed/TotalFeed.tsx (1)

14-22: 렌더링 로직 단순화가 올바르게 적용되었습니다

추천 피드 삽입 로직을 제거하고 posts를 직접 매핑하는 것으로 단순화한 것이 revert의 목적에 부합합니다.

특히 Line 19의 isLast && index === posts.length - 1 로직은:

• 상위 컴포넌트의 isLast가 true일 때만
• 배열의 마지막 항목에만 isLast를 전달

위 조건을 정확히 구현하고 있어 적절합니다.

참고: Line 16의 key에 index를 포함하는 것은 일반적으로 권장되지 않지만, 피드와 같이 append-only 특성을 가진 리스트에서는 허용 가능합니다. 향후 정렬이나 필터링 기능이 추가될 경우 feedId만 사용하는 것을 고려해주세요.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between cad5ced and 2ad171b.

⛔ Files ignored due to path filters (2)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
• src/assets/feed/lookmore-influencer.svg is excluded by !**/*.svg

📒 Files selected for processing (6)

• package.json (0 hunks)
• src/components/feed/FeedPost.tsx (1 hunks)
• src/components/feed/RecommendedFeedCard.tsx (0 hunks)
• src/components/feed/RecommendedFeedSection.tsx (0 hunks)
• src/components/feed/TotalFeed.tsx (1 hunks)
• src/mocks/recommendedFeeds.mock.ts (0 hunks)

💤 Files with no reviewable changes (4)

• package.json
• src/components/feed/RecommendedFeedSection.tsx
• src/mocks/recommendedFeeds.mock.ts
• src/components/feed/RecommendedFeedCard.tsx

🧰 Additional context used

🧬 Code graph analysis (1)

src/components/feed/TotalFeed.tsx (1)

src/types/post.ts (1)

• FeedListProps (21-27)

🔇 Additional comments (2)

src/components/feed/TotalFeed.tsx (1)

7-7: isLast prop 추가가 적절합니다

isLast prop에 기본값 false를 추가한 것은 좋은 설계입니다. 이를 통해 TotalFeed를 사용하는 상위 컴포넌트에서 마지막 항목 여부를 제어할 수 있습니다.

src/components/feed/FeedPost.tsx (1)

19-27: 일관된 디자인 패턴 검증됨 - 검토 필요

rg 검색 결과에서 94.8% 너비 값이 코드베이스의 여러 파일에서 일관되게 사용되고 있는 것으로 확인되었습니다:

• src/components/feed/FeedPost.tsx:20
• src/components/feed/FeedDetailPost.tsx:20
• src/pages/mypage/SavePage.tsx:357
• src/pages/feed/FollowerListPage.tsx:189

이는 임의의 값이 아니라 의도적인 디자인 패턴임을 시사합니다. 모든 컨테이너가 padding: 0 20px를 가지고 있다는 점을 고려하면, 94.8%는 패딩을 고려한 계산된 값으로 보입니다.

확인 필요 사항:

• 이 값이 디자인 시스템의 의도된 부분인지 개발자가 확인해주세요
• 레이아웃이 의도대로 동작하는지 시각적으로 검증해주세요
• 필요시 calc(100% - 5.2%) 등으로 계산 의도를 명확히 할 수 있습니다