Skip to content

refactor(atelier): trim git skill to policy + hooks, drop mechanical guides#797

Merged
kys0213 merged 1 commit into
mainfrom
refactor/trim-git-skill-references
Jun 14, 2026
Merged

refactor(atelier): trim git skill to policy + hooks, drop mechanical guides#797
kys0213 merged 1 commit into
mainfrom
refactor/trim-git-skill-references

Conversation

@kys0213

@kys0213 kys0213 commented Jun 13, 2026

Copy link
Copy Markdown
Owner

  • git skill이 모델이 이미 아는 mechanical git 사용법(sync bash, 충돌 해결 명령)과 모델이 잘하는 판단 가이드(리뷰 정리·이슈 우선순위)를 잔뜩 들고 있었어요.
  • 방금 머지한 doctrine — "skill = 코드로 알 수 없는 암묵지, mechanical은 모델이 직접" — 의 자연스러운 적용 대상이에요.
  • 추가로 atelier git sync를 CLI인 척 부르는데 그 CLI는 존재하지 않는 깨진 참조였어요.

무엇을

  • 삭제 (370줄): sync-strategy.md(89줄 bash 재구현), review-followup.md·issue-prioritization.md(판단 가이드).
  • 슬림화: conflict-resolution.md 96→27줄 — mechanical 제거, ours/theirs 반전 gotcha + 분할정복 정책만 남김.
  • 수정: 깨진 atelier git sync 참조 → git checkout <base> && git pull. cli-reference의 review-followup dangling ref 정리.
  • 유지: cli-reference.md(커스텀 CLI 계약), SKILL.md 본문(정책 + Default Branch Guard hook).
  • extraction-invariants 2개 제거(의도적 삭제 콘텐츠).

어떻게

  • 정책 + hook 만 남긴다: 결정적 안전(commit/branch guard)은 hook이, 판단은 모델이, 결정적 데이터는 atelier git CLI가 담당. skill은 모델이 모르는 것(CLI 계약·반직관 gotcha·이 프로젝트 정책)만.
  • 신규 커맨드 안 만듦: 리뷰 정리·우선순위는 /atelier:git <자연어> 트리거 + atelier git reviews CLI로 이미 닿음 — Epic 2(epic: atelier skill extraction — Fat Controller 해체 + 호출 표면 관심사화 (Epic 2) #766)의 관심사 수렴을 되돌리지 않음.
  • conflict-resolution은 pr-merger·merge-coordinator가 "충돌 전략 단일 출처"로 위임하므로 삭제 않고 전략만 슬림 유지.

확인 방법

  • make validate-ci → 0 failed (extraction-invariant 포함)
  • git/references/cli-reference.md + conflict-resolution.md만 남음
  • 삭제 파일 dangling ref 없음 (sync-strategy·review-followup·issue-prioritization·atelier git sync)
  • conflict-resolution 외부 소비자(pr-merger·merge-coordinator) 참조 여전히 유효

⚠️ 의도적으로 드롭한 load-bearing 토큰

  • closedByPullRequests (이슈 우선순위 PR 상태 GraphQL 필터), auto-stash before git-sync (sync stash 메시지) — 모델이 직접 처리로 대체. PR-aware 우선순위나 sync stash 안전장치를 견고히 원하면 추후 atelier git CLI로 승격하는 게 맞는 altitude예요.

🤖 Generated with Claude Code

…guides

doctrine(skill=암묵지, mechanical git 은 모델이 직접) 적용 — git skill 을 정책+hook 중심으로 슬림화.

- 삭제: references/sync-strategy.md (89줄 bash 재구현 — 모델이 직접 sync), review-followup.md·issue-prioritization.md (판단이라 모델이 잘 처리, 가이드 불필요). 자연어 트리거 + atelier git reviews CLI 로 여전히 닿음 (Epic 2 수렴 유지, 신규 커맨드 안 만듦)
- 슬림화: conflict-resolution.md 96→27줄 — mechanical(checkout --ours/--theirs·case 처리·마커 문법) 제거, ours/theirs 반전 gotcha + 분할정복 정책만. pr-merger·merge-coordinator 의 '충돌 전략 단일 출처' 역할 유지
- 수정: 깨진 'atelier git sync (CLI)' 참조 (CLI 에 sync 없음) → git checkout+pull. cli-reference 의 review-followup dangling ref 정리
- extraction-invariants: git/sync·git/prioritize 토큰 제거 (의도적 삭제 콘텐츠)

Dropped load-bearing tokens (의도적): 'closedByPullRequests'(이슈 우선순위 PR 필터 GraphQL), 'auto-stash before git-sync'(sync stash 메시지) — 모델이 직접 처리. PR-aware 우선순위나 sync stash 안전장치를 견고히 원하면 추후 CLI 로 승격.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
@kys0213 kys0213 merged commit 426c8b7 into main Jun 14, 2026
1 check passed
@kys0213 kys0213 deleted the refactor/trim-git-skill-references branch June 14, 2026 03:19
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