Skip to content

GitHub Actions summary에 전체 merge risk report를 출력한다 - #65

Merged
opficdev merged 3 commits into
developfrom
feat/#51
Jul 19, 2026
Merged

GitHub Actions summary에 전체 merge risk report를 출력한다#65
opficdev merged 3 commits into
developfrom
feat/#51

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • Discord 설정이나 전송 결과와 관계없이 GitHub Actions 화면에서 전체 merge risk report를 확인할 수 있는 전달 경로 구성

📝 작업 내용

📌 요약

  • 전체 Markdown report를 GITHUB_STEP_SUMMARY에 append하는 workflow 전용 deliver() 추가
  • Actions summary 또는 local stdout을 기본 출력으로 사용하고 Discord를 추가 전송하는 경로 구성
  • summary 기록 실패 시 stdout fallback을 시도하고, 적용 가능한 전달 경로를 마친 뒤 오류를 순서대로 집계하는 처리 추가
  • mergeRiskWatch가 report 전달을 마친 뒤 집계된 실패를 workflow 오류로 전파하도록 연결
  • sendMergeRiskReport, send(), ReportChannel* 공개 계약 유지

🔍 상세

  • src/reportChannels/reportChannel.ts
    • 값이 있는 GITHUB_STEP_SUMMARY 경로에 전체 Markdown report 추가 기록
    • summary 경로가 없으면 stdout 출력, Discord 설정이 있으면 별도 추가 전송
    • summary 기록 실패 시 stdout fallback 후 Discord 전송 지속
    • summary, stdout, Discord 실패를 전달 순서대로 집계
  • src/workflows/mergeRiskWatch.ts
    • report 전달 진입점을 workflow 전용 deliverMergeRiskReport로 변경
    • 모든 전달 시도가 끝난 뒤 집계 오류를 throw하도록 연결
  • tests/reportChannels/reportChannel.test.ts
    • GITHUB_STEP_SUMMARY 환경 변수 경로, Actions summary와 Discord 동시 전달, local stdout fallback 검증
    • summary·stdout·Discord 실패 격리와 오류 집계 순서 검증
  • tests/workflows/mergeRiskWatch.test.ts
    • 전체 report의 summary 기록과 summary 실패 후 fallback·Discord 전달·workflow 오류 전파 검증
  • 검증
    • npm run build 통과
    • report-channel 대상 컴파일 테스트 22개 통과
    • npm test 전체 175개 통과
    • git diff --check 통과

📸 영상 / 이미지 (Optional)

  • 해당 없음

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new deliver function in reportChannel.ts to support writing reports to GitHub Actions step summaries, with fallback to stdout and Discord webhook delivery. It also updates the mergeRiskWatch workflow and adds comprehensive tests. The review feedback suggests refactoring the duplicate sendStdout logic in deliver for better readability and trimming the Discord webhook URL to prevent empty string issues.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/reportChannels/reportChannel.ts
Comment thread src/reportChannels/reportChannel.ts
@opficdev opficdev changed the title [#51] GitHub Actions summary에 전체 merge risk report 출력 GitHub Actions summary에 전체 merge risk report를 출력한다 Jul 19, 2026
@opficdev
opficdev marked this pull request as ready for review July 19, 2026 04:38
@opficdev
opficdev merged commit ffe532a into develop Jul 19, 2026
3 checks passed
@opficdev
opficdev deleted the feat/#51 branch July 19, 2026 04:39
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.

GitHub Actions summary에 전체 merge risk report를 출력한다

1 participant