Skip to content

refactor(daily): 일일 브리핑 생성 구조 정리#28

Merged
stdiodh merged 3 commits into
mainfrom
develop/refactor-daily-generator
Jul 21, 2026
Merged

refactor(daily): 일일 브리핑 생성 구조 정리#28
stdiodh merged 3 commits into
mainfrom
develop/refactor-daily-generator

Conversation

@stdiodh

@stdiodh stdiodh commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Title

Refactor: 일일 브리핑 생성 구조 정리

Description

What

  • 일일 브리핑의 완료 상태와 노출 범위를 회귀 테스트로 고정했습니다.
  • PS 문제 ID와 VERIFIED lesson 판별의 중복 로직을 정리했습니다.
  • 백엔드 실무, PS, OSS 준비, CS 지식 렌더링 책임을 섹션 단위로 분리했습니다.
  • CLI와 생성되는 Markdown 형식은 유지했습니다.

Why

  • generate_backend_daily.py에 데이터 선택과 네 개 섹션 렌더링 책임이 집중되어 변경 영향 범위를 파악하기 어려웠습니다.
  • 브리핑 섹션별 수정과 테스트가 쉽도록 내부 구조를 정리할 필요가 있었습니다.

Closes #

Key Code (Before & After)

Before

  • 하나의 큰 렌더링 함수가 모든 브리핑 섹션을 함께 구성했습니다.
# Render all report sections in one function

After

  • 섹션별 렌더링 함수가 각 목록을 만들고 최상위 함수가 조합합니다.
# Compose independently rendered sections
  • 출력 형식과 항목 선택 동작은 기존과 동일하게 유지됩니다.

Reason for Change

  • 렌더링과 선택 로직의 결합도를 낮춰 회귀 위험을 줄입니다.
  • 각 브리핑 섹션을 독립적으로 이해하고 테스트하기 쉽게 만듭니다.
  • 새로운 외부 의존성이나 별도 모듈 없이 기존 구조만 정리합니다.

To Reviewer

  • 기존 Markdown 출력과 섹션 순서가 유지되는지 확인해 주세요.
  • 검증되지 않은 lesson이 노출되지 않는지 확인해 주세요.
  • OSS 계약 오류가 다른 브리핑 섹션에 영향을 주지 않는지 확인해 주세요.
  • ./scripts/validate.sh 전체 검증 결과를 확인해 주세요.

@stdiodh
stdiodh marked this pull request as ready for review July 21, 2026 17:39
@stdiodh
stdiodh merged commit 3e6641a into main Jul 21, 2026
1 check passed
@stdiodh
stdiodh deleted the develop/refactor-daily-generator branch July 21, 2026 17:40
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.

1 participant