Skip to content

fix(platform): 스택 안내 코멘트에 pull-requests 쓰기 권한을 준다 (#2068) - #2070

Merged
OnlyReadOneBook merged 1 commit into
developfrom
fix/2068-stack-notify-permission
Sep 13, 2026
Merged

OnlyReadOneBook merged 1 commit into
developfrom
fix/2068-stack-notify-permission

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • stack-integrity-notify.yml 의 notify-affected-pull-requests job 권한에서 pull-requests 를 read 에서 write 로 올립니다. 이 job 이 코멘트를 남기는 대상은 언제나 PR 이고, GitHub 은 issues/{n}/comments 의 필요 스코프를 대상 타입으로 가르기 때문에 read 로는 목록 GET 만 통과하고 첫 POST 에서 403 이 납니다.
  • merge-order-guard-policy.test.mjs 의 stack-integrity 권한 단언에 pull-requests: write 를 더합니다. 그 테스트는 issues: write 만 계약으로 박아 두고 pull-requests 칸을 비워 뒀고, 그 빈칸이 read 로 머문 채 머지된 경로입니다.
  • 최상단 permissions: {} 와 job 의 issues: write 는 건드리지 않았습니다. 전자는 narrow bridge 불변식입니다.

검증

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "변경이 .github/workflows/stack-integrity-notify.yml 의 job permissions 한 줄과 그것을 잠그는 .github/scripts 정책 테스트 단언뿐이다. 앱 소스, AndroidManifest, navigation 패키지, MainActivity, 계측 하네스, 전역 Gradle 파일을 건드리지 않아 계측 경계가 움직이지 않는다.",
    "tests": []
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

  • 저장소 안 대조군을 근거로 삼았습니다. screenshot.yml 은 issues 스코프 없이 pull-requests: write 만으로 PR 에 코멘트를 성공시키고, conflict-label.yml 과 merge-queue-dequeue.yml 은 둘 다 쓰기를 갖습니다. issues: write + pull-requests: read 조합은 auto-close-issue.yml 뿐인데 그쪽은 PR 을 명시적으로 건너뜁니다.
  • issues: write 를 좁히지 않은 이유는 같은 테스트 파일이 그것을 단언하고 있기 때문입니다. 좁히려면 그 단언을 함께 고쳐야 하는데, 이 job 이 이슈에 쓸 일이 생길 여지를 남겨 두는 쪽을 택했습니다.

notify-affected-pull-requests job 이 코멘트를 남기는 대상은 언제나 PR 이다.
GitHub 은 issues/{n}/comments 의 필요 스코프를 대상 타입으로 가르므로 PR 에
쓰려면 pull-requests 쓰기가 필요하다. read 로 두면 목록 GET 은 통과하고 첫
POST 에서 403 이 나 스택 상위 PR 안내가 통째로 막힌다.

merge-order-guard-policy.test.mjs 는 issues 쓰기만 계약으로 박아 두고
pull-requests 칸은 비워 뒀다. 그 빈칸이 read 로 머문 채 머지된 경로라
같은 자리에 단언을 더한다. 권한을 read 로 되돌리면 그 테스트가 실패한다.
@OnlyReadOneBook
OnlyReadOneBook merged commit 7d87aa3 into develop Sep 13, 2026
22 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the fix/2068-stack-notify-permission branch September 13, 2026 12:20
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.

fix(platform): 스택 안내 코멘트가 pull-requests 쓰기 권한 부족으로 403 에 막힌다

1 participant