chore(skill): SKILL.md 를 라우터 + references 로 분할 - #105
Conversation
704줄 단일 SKILL.md 가 매 트리거마다 전량 로드돼 토큰을 낭비하던 문제 해소. nhncloud-cli progressive disclosure 모델 적용 — 단일 스킬 유지, 관심사별 on-demand 로드. - SKILL.md: 48줄 라우터 (라우팅 표 + 공통 우선 규칙 + 빠른 시작 + 안전한 탐색 순서) - references/: common / intent-map / post / comment / wiki / mention-link / workflow 7개 - 본문 verbatim 이동 (43개 섹션 무손실, wiki tree·위키 링크 문서 포함) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
566d6be to
22604ca
Compare
|
/review |
|
|
||
| ```bash | ||
| # --json 으로 구조화 출력 → jq 로 성공/실패 분리 | ||
| RESULT=$(dooray post file download-all <project> <number> -o ./ --json) |
There was a problem hiding this comment.
🟡 문제: wiki.md 파일임에도 예시가 dooray post file download-all 을 씀. 사용자는 라우팅 표를 보고 위키 첨부를 찾아 이 파일로 왔는데, 예시는 post file 다운로드라 위키 첨부에 실제로 어떻게 적용해야 하는지가 불명확.
수정: 이 스니펫을 dooray wiki page file download-all <project> <page-id> -o ./ --json 로 교체하면 wiki.md 범위와 일치. (--json 스키마는 ADR-031 mirror 라 예시 로직 그대로 유지) 원래 post 예시가 필요하면 workflow.md 나 (신설 시) post-file.md 로 이동.
|
|
||
| --- | ||
|
|
||
| --- |
There was a problem hiding this comment.
🟡 문제: 133-135 라인에 --- 이 연달아 두 번 (섹션 사이 빈 구분선 중첩). 렌더링에 지장은 없지만 시각적으로 지저분.
수정: 세 곳 모두 정리 — 133-135, 177-179, 193-196 라인. 한 섹션 끝의 --- 하나만 남기고 다음 섹션 헤딩 앞의 여분 --- 삭제.
|
|
||
| --- | ||
|
|
||
| --- |
There was a problem hiding this comment.
🟡 문제: 83-85 라인에 --- 이 연달아 두 번 나옴 (---\n\n---). 원본 SKILL.md 를 잘라 붙일 때 상하 섹션 각각의 끝-시작 구분선이 겹친 것으로 보임.
수정: 83 또는 85 중 하나만 남기고 삭제.
코드 리뷰 chore(skill): SKILL.md 를 라우터 + references 로 분할문서 전용 refactor — SKILL.md 698줄 → 46줄 라우터 + 7개 references 로 progressive disclosure 적용. 코드 변경 없음. 개인 식별 정보·내부 참조 번호 노출 없음(검증 통과). 마이너한 마크다운 정돈 몇 곳만 남았고 머지 차단 사유는 없음. 🟡 개선 권장skills/dooray-cli/references/wiki.md 22-30 skills/dooray-cli/references/post.md 133-135, 177-179, 193-196 skills/dooray-cli/references/common.md 83-85 🟢 잘 된 점
🤖 Reviewed with Claude Code (single opus reviewer: type · conventions · security · architecture) |
- post.md/common.md 의 연속 `---` 수평선 중복 제거 (분할 시 섹션 경계 겹침) - wiki.md 의 `post file download-all` 시나리오를 post.md 로 이동 (위키 파일이 아니라 post 파일 명령) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
리뷰 반영 결과 (커밋 51a0175): 🟡 항목 1 (wiki.md 의 post file 예시): ✅ 반영 — 🟡 항목 2 (post.md 중복 구분선): ✅ 반영 — 연속 🟡 항목 3 (common.md 중복 구분선): ✅ 반영 — 에러 표 아래 중복 |
개요
공개 스킬
skills/dooray-cli/SKILL.md가 698줄로 커져 매 트리거마다 전량 컨텍스트에 로드되던 문제를 해소한다.nhncloud-cli의 progressive disclosure 모델을 적용해 단일 스킬은 유지하되 관심사별로 on-demand 로드하도록 분할했다.왜 단일 스킬 + references 인가 (멀티 스킬 아님)
dooray-cli 는 CLI 1개·config 1개·install 1개라 런타임이 하나다.
멀티 top-level 스킬로 쪼개면 공통 섹션(설치·설정·출력·제약·링크) 중복으로 drift 가 나고, 크로스 관심사 체이닝(업무 찾아 → 댓글 → 위키 첨부)이 스킬 경계로 파편화되며, 트리거가 모호해진다.
단일 스킬 + references 는 트리거 1개를 유지하면서 상세만 필요할 때 읽는다.
변경
SKILL.md: 46줄 라우터 (라우팅 표 → 공통 우선 규칙 → 빠른 시작 → 안전한 탐색 순서)references/7개 신설common.md— 설치·초기 설정·출력 모드·제약사항·피드백·에러 핸들링·캐시intent-map.md— 자연어 의도 → 커맨드 마스터 표post.md— 업무 식별·생성·수정·cc/to·부모·태그·본문 수정comment.md— 댓글 추가·목록·단일 조회wiki.md— 위키 페이지·첨부·댓글 시나리오mention-link.md— 멘션·cc 동선·마크다운 링크 형식workflow.md— 워크플로우 판단·정형 task·체이닝 시나리오검증
🤖 Generated with Claude Code