Skip to content

chore: 웹 PR 리뷰어 자동 지정 워크플로 추가 - #90

Open
ParkSungju01 wants to merge 1 commit into
developfrom
chore/89-pick-reviewer-workflow
Open

ParkSungju01 wants to merge 1 commit into
developfrom
chore/89-pick-reviewer-workflow

Conversation

@ParkSungju01

Copy link
Copy Markdown
Collaborator

What is this PR? 🔍

Changes 📝

  • KOIN_WEB_RECODE에서 쓰던 PICK_REVIEWER.yml을 이 저장소로 이전
  • BCSDLab 조직 공용 워크플로(BCSDLab/.github/.github/workflows/pick-reviewer.yml@v1) 호출
    • 리뷰어 명단은 조직 .github 레포의 config/frontend/reviewer.json 사용 (KOIN_WEB_RECODE와 공유)
    • PR 작성자를 뺀 2명을 무작위로 리뷰어 지정
  • apps/web/** 변경이 있는 PR에만 동작하도록 경로 조건 추가 (API·b-bot·출근부 PR에는 프론트엔드 리뷰어 미지정)

Test CheckList ✅

  • 머지 후 apps/web을 변경한 PR을 열었을 때 리뷰어 2명이 자동 지정되는지 확인
  • apps/web 변경이 없는 PR에서는 워크플로가 실행되지 않는지 확인

Precaution

KOIN_WEB_RECODE의 PICK_REVIEWER 워크플로 이전
apps/web 변경이 있는 PR에만 동작하도록 경로 조건 추가

Closes #89
@ParkSungju01 ParkSungju01 self-assigned this Nov 14, 2026
@coderabbitai

coderabbitai Bot commented Nov 14, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ff11c81d-5591-4efb-ad62-0835914e95db
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
koin-web bad5c95 Nov 14 2026, 08:56 AM

@bcsd-united-error-publisher

bcsd-united-error-publisher Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

자동 리뷰 결과 (claude-opus-5-5, 독립 검증)

  • 리뷰 head: bad5c9583f227de9835406c21ae66d2de8dd7c9c
  • 발행 head: bad5c9583f227de9835406c21ae66d2de8dd7c9c
  • base(develop): 4d32208b818794bfc8ad906ae04e11938222f32c
  • manifest sha256: ad301433f67ec7108b54797ac5d7a61df8c92f62fa52ae631da3f49731576ac3
  • 판정: approved (P1 0 / P2 0 / P3 1)

발견 사항

  • P3 .github/workflows/PICK_REVIEWER.yml:23 — `secrets: inherit` passes every repository secret to an outside reusable workflow pinned to the movable tag @v1. Passing only the required Slack secret explicitly, or pinning to a commit SHA, would limit what is exposed. The fix belongs in a protected workflow file, so this edit is blocked for the reviewer.

적용된 수정

  • 없음 (리뷰 head 유지)

독립 검증 통과 항목

  • pr-head-pinned
  • native-fixture-tests
  • typecheck
  • autofix-security-tests
  • lint
  • build
  • runtime-route-smoke

주의

  • 리뷰 판정은 GitHub approval이 아닙니다. merge 여부는 아래 자동 merge 정책으로 별도 결정하며 main 승격과 배포는 수동입니다. PR 제목/본문/diff와 모델 출력은 신뢰할 수 없는 입력으로 취급했습니다.
  • 수정 커밋은 리뷰 head를 부모로 하는 fast-forward만 적용했으며 사람의 변경을 덮어쓰지 않습니다.

자동 merge

  • skipped: 보호된 설정, 인증/결제/세션 또는 운영 경로가 변경되었습니다.

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.

[공통] 웹 PR 리뷰어 자동 지정 워크플로 추가

1 participant