Skip to content

요약 생성 중 버튼 노출 및 재생성 안내 문구 수정 - #601

Merged
goder-0 merged 2 commits into
mainfrom
feature/#600-summary-loading-ui
Oct 6, 2026
Merged

goder-0 merged 2 commits into
mainfrom
feature/#600-summary-loading-ui

Conversation

@goder-0

@goder-0 goder-0 commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

요약 생성 중 상세 패널에 재생성·복사 버튼이 노출되고, 재생성 모달에 progress notifications가 표시되는 문제를 수정합니다.

요약 내용이 표시 가능한 상태에서만 버튼 그룹을 렌더링하고, 재생성 진행 문구를 요약 재생성 중...으로 지정했습니다. 오류 상태의 다시 시도 동작은 유지합니다.

image

@goder-0 goder-0 changed the title Fix: 요약 생성 중 버튼 및 안내 문구 수정 (#600) 요약 생성 중 버튼 노출 및 재생성 안내 문구 수정 Sep 22, 2026
@goder-0
goder-0 marked this pull request as ready for review September 22, 2026 09:03
@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 99dde98e-3f45-4ac7-bcef-740228a54612

📥 Commits

Reviewing files that changed from the base of the PR and between d895261 and e0d3568.

📒 Files selected for processing (1)
  • CHANGELOG.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

SummarySection은 요약이 표시되지 않거나 작성 중이면 재생성·복사 버튼을 숨깁니다. 오류 상태의 기존 동작은 유지됩니다. ReSummaryModal은 로딩 중 ProgressNotification에 요약 재생성 중... 라벨을 표시합니다. CHANGELOG.md에 두 수정 사항을 기록합니다.

Priority: ⬇️ Low

Severity of issue fixed: Low

Merge Risk: ⚪ Minimal · up to e0d35

요약 상태에 따른 작업 버튼 표시와 재생성 진행 문구가 요청된 동작에 부합하며, 오류 상태의 다시 시도 동작도 유지됩니다. 확인된 변경으로 인한 추가 병합 위험은 없습니다.

Architecture Summary

Architecture risk: 🔵 Low · up to e0d35

The change affects 2 systems.

Changed systems: src, CHANGELOG.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — src (ui) was modified; 2 changed files map to changed impact.
  • observed — CHANGELOG.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in src/components/wrappers/LinkCardDetailPanel/Sections/SummarySection.tsx: renderActions의 조기 반환 조건이 summaryState === 'error'에서 !shouldShowSummary || summaryState === 'writing'로 변경되어, 오류 상태뿐 아니라 요약이 표시되지 않는 상태(빈 요약 또는 로딩/대기 상태)와 작성 중 상태에서도 작업 버튼이 숨겨지게 됩니다.
  • observed — Modified behavior in src/components/wrappers/ReSummaryModal/ReSummaryModal.tsx: ProgressNotification에 label="요약 재생성 중..." 라벨 prop이 추가되었습니다. isLoading 상태 표시 중 사용자에게 진행 문구가 나타납니다.
  • observed — Modified behavior in CHANGELOG.md: [Unreleased]에 수정 항목 두 개를 추가했습니다. 요약 생성 중 상세 패널에 재생성·복사 버튼이 표시되던 문제와 재생성 중 안내 문구가 요약 재생성 중...으로 표시되지 않던 문제를 수정했다고 기록합니다.
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 요약 생성 중 버튼 노출 문제와 재생성 안내 문구 수정이라는 주요 변경 사항을 간결하게 설명합니다.
Description check ✅ Passed 설명은 템플릿의 관련 이슈와 PR 설명 섹션을 포함합니다. 변경 원인, 수정 내용, 오류 상태 동작 유지 사항을 구체적으로 설명합니다.
Linked Issues check ✅ Passed 이슈 #600의 코딩 요구사항을 충족합니다. SummarySection.tsx는 shouldShowSummary가 거짓이거나 summaryState === 'writing'이면 재생성·복사 버튼 그룹을 렌더링하지 않습니다. 표시 가능한 요약 내용이 있으면 버튼 그룹을 표시합니다. 오류 상태의 다시 시도 동작은 유지됩니다. `ReSummaryM…
Out of Scope Changes check ✅ Passed 변경은 이슈 #600에 직접 연결된 두 컴포넌트의 표시 조건과 진행 문구 수정으로 제한됩니다. CHANGELOG.md 변경은 이 수정 사항을 기록하는 지원 문서입니다. 관련 없는 기능 변경은 확인되지 않습니다.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@goder-0
goder-0 requested a review from Bangdayeon September 22, 2026 10:02
@goder-0
goder-0 requested a review from Seong-Myeong October 1, 2026 06:47
@goder-0
goder-0 merged commit bc32ef1 into main Oct 6, 2026
3 checks passed
@goder-0
goder-0 deleted the feature/#600-summary-loading-ui branch October 6, 2026 06:22

This branch was successfully deployed

1 active deployment
Preview — e0d3568b Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

요약 생성 중 상세 패널 버튼 및 재생성 안내 문구 수정

2 participants