Skip to content

chore: 설정 기준선에 비공개 취약점 보고를 더한다 - #387

Merged
minjunkim-dev merged 1 commit into
mainfrom
chore/private-vulnerability-reporting
Nov 8, 2026
Merged

minjunkim-dev merged 1 commit into
mainfrom
chore/private-vulnerability-reporting

Conversation

@minjunkim-dev

@minjunkim-dev minjunkim-dev commented Nov 8, 2026 •

Copy link
Copy Markdown
Owner

Author: Claude Code / Opus 5.5

Summary

repo-baseline c5e994e(#89)부터 GitHub 설정 기준선에 private_vulnerability_reporting이 필수 항목이 됐다. 이 저장소의 .github/repository-settings.json에는 이 키가 없어서, 최신 repo-baseline의 설정 검사가 쓰기 전에 멈춘다.

  • 전: expected에 private_vulnerability_reporting 없음
  • 후: immutable_releases 다음에 "private_vulnerability_reporting": {"enabled": true}를 넣었다. 템플릿과 같은 자리다.

이 저장소는 공개 저장소이고 이 기능은 이미 켜져 있다. 그래서 apply를 실행해도 바뀌는 설정은 없다.

Evidence

  • gh api repos/minjunkim-dev/smallnext/private-vulnerability-reporting → {"enabled":true}
  • gh repo view → visibility: PUBLIC
  • 바꾼 파일을 json.loads로 파싱했고, expected의 키를 repo-baseline main의 templates/github/repository-settings.json과 비교했다. 빠진 키는 없다. 템플릿에 없는 키는 codeql_workflow 하나인데, 검사기가 허용하는 선택 키다.
  • 확인하지 않은 것: apply_github_settings.py의 계획 출력. 이 단계는 소유자가 자기 기기에서 직접 실행한다.

Merge Danger

Door: 되돌리기 쉬움
영향 범위: 설정 검사에서 기대하는 값 하나. GitHub 설정은 바꾸지 않는다.
되돌리는 법: 이 PR을 revert한다.
신뢰 경계: .github/repository-settings.json

Review

리뷰어(구현과 다른 컨텍스트, 도구/모델): 없음. JSON 세 줄을 템플릿 그대로 옮겨 넣은 것이다.
고친 것: 없음
남은 지적: 없음

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Private vulnerability reporting is now available for the repository, allowing security issues to be submitted confidentially. This provides a private channel for reporting vulnerabilities rather than requiring public disclosure. Existing repository settings are unchanged.

repo-baseline c5e994e(#89)에서 private_vulnerability_reporting이 필수 항목이 됐다.
공개 저장소이고 이미 켜져 있어서 기대값만 맞춘다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The repository settings add private_vulnerability_reporting and set it to true. Existing settings remain unchanged.

Changes: Private Vulnerability Reporting

Layer / File(s) Summary
Enable private vulnerability reporting
.github/repository-settings.json
The repository settings enable private vulnerability reporting. Existing settings remain unchanged.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: Minimal · up to bdcd0

The baseline enables private vulnerability reporting; no actionable merge-blocking risk is evident.

Architecture risk: Low · up to bdcd0

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/repository-settings.json: Adds the private_vulnerability_reporting setting with enabled set to true.
Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed 제목은 설정 기준선에 비공개 취약점 보고 설정을 추가한다는 PR의 주요 변경을 설명한다. 변경 내용은 .github/repository-settings.json에 private_vulnerability_reporting 키 3줄을 추가하는 것이다. 제목은 chore 접두사를 포함하고 짧은 단일 문장이며, 실제 변경과 관련이 있다.
Description check Passed 설명은 템플릿의 주요 섹션을 모두 채웠다. Author, Summary, Evidence, Merge Danger, Review 섹션이 있다. Summary는 변경 이유와 전후 상태를 설명한다. Evidence는 실행한 gh api 명령, JSON 파싱, 키 비교 결과와 확인하지 않은 범위(apply 스크립트의 계획 출력)를 적었다. Merge Danger…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

설명은 템플릿의 주요 섹션을 모두 채웠다. Author, Summary, Evidence, Merge Danger, Review 섹션이 있다. Summary는 변경 이유와 전후 상태를 설명한다. Evidence는 실행한 gh api 명령, JSON 파싱, 키 비교 결과와 확인하지 않은 범위(apply 스크립트의 계획 출력)를 적었다. Merge Danger는 되돌리기 쉬움, 영향 범위, 되돌리는 법, 신뢰 경계를 적었다. Review 섹션에는 리뷰어 정보가 '없음'으로 적혀 있으며 변경 규모에 맞는 내용이다. 템플릿의 요구 정보가 대부분 채워져 있어 통과로 판단한다.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

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: 3cc0de88-5164-4572-993e-4b644debfb38
Commits

Reviewing files that changed from the base of the PR and between 256a18c and bdcd02a.

Files selected for processing: 1

  • .github/repository-settings.json

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.

@minjunkim-dev
minjunkim-dev merged commit 51b5ac2 into main Nov 8, 2026
14 checks passed
@minjunkim-dev
minjunkim-dev deleted the chore/private-vulnerability-reporting branch November 8, 2026 13:59
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.

1 participant