[m-a-king]

한눈에

사람이 승인한 자동 수정 패치만 PR 로 올리는 공통 액션 추가

• 클라우드 에이전트가 매일 추출 실패를 조사해 비공개 repo 에 패치를 남김
• 이 액션이 renderer·extractor 에 패치마다 승인 이슈를 만듦
• 사람이 이슈의 링크로 패치를 보고 체크박스를 누르면 그 패치로 PR 이 열림
  • 승인 전에는 브랜치도 PR 도 없음
• 워크플로 변경·숨은 글·URL 노출 같은 위험한 패치는 PR 전에 거부
• 머지해도 바로 일어나는 일 없음
  • renderer·extractor 가 이 액션을 부르는 워크플로를 따로 추가해야 동작

Situation

이슈

• ops 패치를 PR 로 여는 공통 액션 추가 #139

추가

• 첫 구현은 패치 글자를 정규식으로 검사
  • 결과: 대화 맥락 없는 리뷰가 Ubuntu 에서 우회 9가지 재현
  • 8진 이스케이프 경로·모드 앞 0·NUL 바이트·파이프 SIGPIPE·userinfo 로 위장한 URL 등
• 패치가 .gitattributes 를 함께 넣으면 바이너리 판정이 풀리고 리뷰 화면의 diff 가 접힘 (재현)
• ops 의 .gitattributes 와 LFS 설정이 체크아웃 때 러너의 외부 요청을 일으킴 (재현)
• 렌더링된 pr.md 는 HTML 주석 같은 숨은 글을 보여 주지 않음
  • 그 글은 PR 본문과 커밋 메시지로 공개됨
• 사람이 확인한 뒤 올리기 전에 에이전트가 같은 id 의 패치를 다시 쓸 수 있음
• environment 승인은 Free 플랜에서 공개 repo 만 됨 (GitHub 문서)
• TeamPiKi 조직은 배포 키 생성이 꺼져 있음
  • 원인: 새 조직의 기본값

Task

이슈

• ops 패치를 PR 로 여는 공통 액션 추가 #139
• 범위: 매일 draft PR 을 자동으로 여는 계획에서 사람이 승인 이슈를 체크해야 PR 이 열리는 방식으로 바뀜

Action

승인 이슈 체크박스로 통일

• 예약 실행(request)이 아직 요청하지 않은 항목마다 그 repo 에 ops 패치 <id> 승인 이슈를 만듦
  • 커밋 하나로 만들어지고 그 뒤로 바뀌지 않은 항목만. 만든 지 2일 안. 미래 날짜는 건너뜀
  • 한 실행에 5건까지. 이슈 조회는 전체 페이지
• 이슈에는 그 커밋의 pr.md 원문 링크와 patch.diff 링크
  • 링크의 커밋이 곧 PR 이 되는 커밋이라 sha 를 맞춰 볼 필요 없음
• 체크 편집(approve)을 쓰기 권한자의 체크로 확인한 뒤 PR 을 열고 이슈를 닫음
  • 루틴은 renderer·extractor 에 접근하지 못해 대신 체크할 수 없음
• 거절은 이슈 닫기. 한 번 이슈가 생긴 항목은 다시 요청하지 않음
• ops 의 폴더 이름은 patches
  • 대안: outbox
    • 기각: 무엇이 들어 있는지 이름으로 드러나지 않음

선택지	장점	비용
승인 이슈 체크박스 (채택)	두 repo 같은 방식·입력 없음·승인 전 공개 없음	각 repo 이슈 목록에 승인 이슈
renderer 브랜치 링크·extractor environment 승인	이슈 없음	repo 마다 방식이 다름
사람이 id·커밋 sha 를 넣어 실행	방식 하나	패치마다 입력 두 개
매일 draft PR 자동 개시	사람 손이 들지 않음	확인 전 PR 이 섞임·공개 repo 는 열리는 순간 공개

git 이 해석한 결과로 판정

• 패치를 base 위에 적용한 뒤 git diff --cached 의 경로·모드·바이너리 표시로 판정
• 작업 트리의 속성 파일은 판정과 ops 체크아웃에서 읽지 않음
• 속성 파일(.gitattributes·.lfsconfig·.gitmodules)을 바꾸는 패치 거부
• 파일 20개 초과·바뀐 줄 800줄 초과 패치 거부
  • 원인: 큰 diff 는 리뷰 화면이 접음
• 코드 표시 밖에 HTML 이 있는 pr.md 와 세 파일 밖의 파일이 든 항목 거부

선택지	장점	비용
적용 뒤 git 출력으로 판정 (채택)	git 과 해석이 갈릴 수 없음	checkout 에 패치를 한 번 적용
패치 글자를 정규식으로 검사	적용 없이 판정	우회 9가지 재현

공개 repo 의 노출 검사

• 공개 여부는 입력값이 아니라 repo 설정에서 읽음
  • 원인: 입력값 실수로 검사가 꺼지는 경로 차단
• 잡는 형태
  • scheme 이 있는 주소 전부. userinfo 가 있으면 허용 도메인이어도 거부
  • scheme 없이 도메인 뒤에 /·?·#·:포트 가 오는 형태
  • IP 뒤에 경로·쿼리가 오는 형태
  • 이메일 주소
  • 예외: 경로 없는 도메인 단독은 통과. 사용자 데이터가 아니고 파일명(Foo.java)과 구분되지 않음
• 금지어는 호출하는 repo 의 시크릿으로 받음. 단어 목록이 공개 코드에 남지 않음
  • 공개 repo 에서 금지어 시크릿이 비면 실행 실패
• 한글 경로는 이스케이프 없이 검사
  • 원인: git 기본 출력은 8진 이스케이프라 금지어가 빠져나감

CI 실행 승인

• action 이 연 PR 의 CI 는 push 한 커밋·이 repo·pull_request 이벤트가 모두 맞는 실행만 승인
  • 원인: 같은 이름의 fork 브랜치 실행까지 승인할 수 있음
• 승인한 실행이 0건이면 실패 처리. 이슈 댓글로 PR 화면에서 직접 승인하라고 알림

그 밖의 차단

• ops 를 받을 때 심볼릭 링크를 일반 파일로 받음
  • 원인: 항목 파일이 checkout 의 .git/config 토큰을 가리킬 수 있음
• 끝나면 checkout 을 base 로 되돌림
  • 원인: 같은 job 의 뒤 step 이 패치된 트리를 실행하지 않게 함

Result

셀프 테스트

• fetch·scan·request·gate·check·open 셀프 테스트를 CI block-test 에 추가
• 리뷰 세 차례가 재현하거나 짚은 우회를 케이스로 넣음
• 속성 파일 무시 설정은 negative control 로 확인
  • 설정을 뺀 사본에서 해당 케이스 2건이 실패
• macOS 와 Ubuntu 24.04(git 2.43) 컨테이너 양쪽에서 확인

위험

• 승인은 루틴이 renderer·extractor 에 접근하지 못한다는 설정에 기댐
  • Claude App 을 ops 에만 설치. 설치 뒤 범위를 API 로 확인
• 일부러 인코딩한 URL(퍼센트 인코딩·base64)은 검사가 잡지 못함
  • 승인 전에 사람이 원문을 봄

후속

• renderer·extractor 호출 워크플로 추가. 이 PR 머지 뒤
• 조직 설정에서 배포 키 허용·Actions 의 PR 생성 허용

---

연관 이슈

• close ops 패치를 PR 로 여는 공통 액션 추가 #139

[coderabbitai]

Important

• 🔍 Trigger review

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

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: adfc5a1e-33e6-4c5f-a0e1-43c860d00541

• 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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}