chore: docs 기본 차단 (로컬 산출물 유출 방지) - #15
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe ChangesRepository ignore rules
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.gitignore:
- Around line 5-7: Update the docs ignore rule in .gitignore from ignoring the
directory itself to ignoring its contents, then explicitly re-include the
approved docs/conventions and docs/maps directories and their files so those
public documents can be committed while other docs remain ignored.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
디렉토리(`/docs/`)를 무시하면 git 이 그 안으로 내려가지 않아, 이후 `!/docs/공개문서/` 같은 예외를 붙여도 무력해진다. 내용물(`/docs/*`)을 무시하는 형태로 바꿔 공개할 문서를 선택적으로 허용할 수 있게 한다. CodeRabbit 지적 반영. 다만 제안된 `!/docs/conventions/`·`!/docs/maps/` 예외는 rutter 의 디렉토리 구조라 pilot 에는 없는 경로여서 넣지 않는다 — 공개할 문서가 생길 때 그 경로로 예외를 추가한다.
|
CodeRabbit 지적 반영했습니다 (67194c1). 진단은 정확합니다. 제안된 예외 경로는 넣지 않았습니다. |
docs/차단이.git/info/exclude(로컬 전용, 커밋되지 않음)에만 있어서 다른 사람이 clone하면 보호가 사라진다. 세션 의도 기록 같은 로컬 산출물이 공개 저장소에 올라갈 수 있어.gitignore에 명시한다.변경
.gitignore에/docs/추가 — 기본 차단, 공개할 문서가 생기면 그때 명시적으로 허용(!/docs/<dir>/).참고
rutter도 같은 원칙으로 정리(followingseas/rutter#16). wake는 이미
.gitignore에docs/가 있어 안전.🤖 Generated with Claude Code
Summary by CodeRabbit
/docs/*.