Skip to content

Conversation

@Hexeong
Copy link
Contributor

@Hexeong Hexeong commented Nov 15, 2025

관련 이슈

작업 내용

  1. 진짜 마지막 입니다... 잦은 pr 생성 죄송합니다.
  2. fork repo에서 테스트를 거쳤을때 owner와 permission-packages 파라미터로 넣은 토큰이 정상적으로 발행이 되었고, 해당 토큰이 성공적으로 이미지를 pull 하는 것을 확인했습니다.
  3. 아래 스크린샷을 첨부하겠습니다!
image

특이 사항

리뷰 요구사항 (선택)

@coderabbitai
Copy link

coderabbitai bot commented Nov 15, 2025

Walkthrough

GitHub Actions 워크플로우 두 파일에서 GitHub App 토큰 생성 단계의 권한 필드명이 변경되었습니다. 다음과 같이 정리됩니다:

  1. dev-cd.yml 파일 수정

    • permission-organization-packages: "read" → permission-packages: "read"로 변경
  2. prod-cd.yml 파일 수정

    • permission-organization-packages: "read" → permission-packages: "read"로 변경

두 워크플로우 모두 동일한 권한 필드명 변경이 적용되었으며, 워크플로우의 제어 흐름이나 동작 로직에는 변화가 없습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • 변경 사항이 순수한 필드명 변경으로 매우 제한적임
  • 두 파일에서만 동일한 패턴으로 반복되는 변경
  • 로직이나 워크플로우 동작에 영향을 주지 않는 구성 업데이트

Suggested reviewers

  • wibaek
  • whqtker
  • JAEHEE25
  • lsy1307
  • sukangpunch
  • Gyuhyeok99

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 깃허브 앱 토큰 권한 오류 수정이라는 PR의 주요 변경사항을 명확하게 요약하고 있습니다.
Description check ✅ Passed PR 설명은 관련 이슈 링크, 작업 내용, 테스트 결과 스크린샷을 포함하여 템플릿의 필수 항목들을 충실히 따르고 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항(permission-organization-packages를 permission-packages로 수정)이 #564의 CD 프로세스에서 깃허브 앱 권한 명시 오류 수정 목표를 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항은 두 개의 워크플로우 파일에서의 권한 필드 수정으로, 연결된 이슈 #564의 범위 내에 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 66e2255 and fb4e700.

📒 Files selected for processing (2)
  • .github/workflows/dev-cd.yml (1 hunks)
  • .github/workflows/prod-cd.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (2)
.github/workflows/dev-cd.yml (1)

70-77: 제목: GitHub App 토큰 권한 필드명 수정 - 올바른 매개변수 이름으로 변경됨

다음과 같은 개선사항이 적용되었습니다:

  1. 권한 매개변수명 정정: permission-organization-packages: "read"에서 permission-packages: "read"로 변경
    • 기존 필드명은 actions/create-github-app-token@v2 액션의 공식 문서에 지원되지 않는 이름이었습니다
    • 수정된 필드명은 GitHub Actions 공식 문서의 권한 필드 명명 규칙(permission-<scope>)과 일치합니다
    • 테스트 확인: PR 작성자가 포크 저장소에서 검증했으며, 토큰이 정상 생성되고 컨테이너 이미지를 성공적으로 풀링했습니다

이 변경은 워크플로우 CD 프로세스에서 GitHub App 권한 명시 오류를 직접적으로 해결합니다.

.github/workflows/prod-cd.yml (1)

68-75: 제목: 프로덕션 워크플로우에도 동일한 권한 필드명 수정 일관성 적용

다음과 같은 개선사항이 적용되었습니다:

  1. 일관된 권한 매개변수명 수정: 개발 워크플로우와 동일하게 permission-organization-packages: "read"에서 permission-packages: "read"로 변경됨
    • DEV/PROD 환경 모두에서 GitHub App 토큰 생성 로직의 일관성이 유지됩니다
    • 두 워크플로우가 동일한 올바른 매개변수명을 사용하므로 유지보수성이 향상됩니다

수정사항은 프로덕션 CD 프로세스의 안정성을 보장하면서 개발 환경과의 설정 편차를 제거합니다.


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 and usage tips.

@Hexeong Hexeong added the 버그 Something isn't working label Nov 16, 2025
@Hexeong Hexeong merged commit 76b25da into solid-connection:develop Nov 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: CD process에서 Github App에 대한 권한 명시 오류 수정

2 participants