feat: infra 자신에도 개발 스킬을 설치한다 - #33
Conversation
- self 모드가 스킬 설치를 건너뛰던 가드를 풀었다. "infra 는 생산자라 소비자가 아니다"던 전제가 실제 작업과 어긋났다. infra 에서도 커밋·PR 이 일어나며, infra#32 를 스킬 없이 수동 STAR 로 올린 것이 계기 - untracked 노이즈 우려는 .gitignore 에 .claude/commands/ 를 더해 해소 (worktrees 와 같은 방식) - 규약 문서(testing-principles)는 소비 repo 한정 유지. infra 엔 import 할 CLAUDE.md 가 없고 JVM 원칙이라 대상이 아니다 - self 모드 get() 은 working tree 를 읽으므로 스킬 정본 수정이 자기 세션에 즉시 반영되는 드라이푸딩 효과가 따라온다
|
Warning Review limit reached
Next review available in: 52 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. 📝 WalkthroughWalkthrough
ChangesSelf-mode 개발 스킬 설치
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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 `@install.sh`:
- Around line 203-206: README.md의 self 모드 설명을 찾아 개발 스킬 설치를 건너뛴다는 내용을 제거하고,
install.sh의 현재 정책에 맞게 commit, gc, coderabbit, pr, issue, session-check,
session-close 스킬이 self 모드에도 설치된다고 명시하세요.
- Around line 210-216: Keep the canonical skills files and their install_asset
entries synchronized so deleted skills are removed from the installed command
set. Prefer generating the installation list from the current skills/ files, or
otherwise enforce removing the corresponding install_asset call whenever a skill
file is deleted; add a regression test covering removal of the stale installed
command after fetch or validation failure.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 397b8bd7-e396-48fe-b738-8bc342fc8cb0
📒 Files selected for processing (2)
.gitignoreinstall.sh
- 개발 스킬 SSOT 화 항목의 "self 모드는 스킵" 서술이 #33 의 정책 전환(self 모드에도 설치)과 어긋나 결정 로그 형식대로 전환 사실을 병기 - CodeRabbit 리뷰 수용
- main 의 #33(infra 자신에도 스킬 설치)이 진행 상태 불릿을 수정해 삭제와 충돌했다. 로그 삭제는 유지하고, 바뀐 사실만 유지되는 구조 트리 주석에 흡수
* docs: README 진행 상태 로그 삭제, 구조 트리 최신화 - 완료 트랙의 날짜별 작업 로그(진행 상태 섹션)를 삭제. 기록은 git 이력과 PR 이 소유한다 - 구조 트리에 observability.md·run_container·alloy 블록을 반영하고 SSH 단일 transport 확정을 본문에 반영 (로컬 체크아웃에 커밋되지 않은 채 남아 있던 문서 개선을 흡수) - 전체 시스템 구성은 core README 를 참조하도록 한 줄 추가 * docs: blocks 주석에 관측 설정 포함을 명시하고 install.sh 주석을 규약 갈래로 한정 - blocks/ 를 순수 bash 로 서술했지만 alloy/ 의 config.alloy 는 관측 설정이라 불일치 (CodeRabbit 지적 수용) - install.sh 가 배포 갈래(blocks·contracts)까지 설치한다는 오독이 나와, 설치 대상이 개발 규약 갈래뿐임을 주석에 명시. 배포 자산은 각 서비스 deploy 가 원격 fetch 로 소비한다 * docs: 스킬 설치 대상에 infra 자신 포함을 반영 - main 의 #33(infra 자신에도 스킬 설치)이 진행 상태 불릿을 수정해 삭제와 충돌했다. 로그 삭제는 유지하고, 바뀐 사실만 유지되는 구조 트리 주석에 흡수
1 participant
Situation
.claude/commands에 설치하면서, infra 자신(self 모드)은 의도적으로 건너뛰었다. 명시된 근거는 둘: 버전 영역에 untracked 파일이 남는다, 그리고 "스킬은 소비 repo 를 위한 자산이고 infra 는 그 생산자다".Task
Action
[ "$self" = 0 ]가드를 걷어 self 모드에서도 설치한다. self 모드의get()은 이미 working tree 를 읽도록 되어 있어 추가 구현 없이 동작하고, 스킬 정본을 고치면 자기 세션에 즉시 반영되는 드라이푸딩 효과가 따라온다..gitignore에.claude/commands/를 추가했다 (기존.claude/worktrees/와 같은 방식). 정본은skills/가 유지하고 설치본은 버전 관리 밖에 둔다.self = 0가드를 남겼다.Result
.claude/commands에 설치되는 것,git status에 설치본이 나타나지 않는 것(ignore 동작), 설치본이skills/정본과 내용 일치하는 것을 확인했다.is_managed_path가 이미$repo_root/.claude/commands/를 관리 경로로 포함하고 있어, 정본에서 스킬을 지우면 infra 설치본도 다음 실행에서 함께 걷힌다.연관 이슈
Summary by CodeRabbit
새로운 기능
개선 사항
.claude/commands/관련 파일이 불필요하게 추적되지 않도록 관리됩니다.