Skip to content

chore(skill): SKILL.md 를 라우터 + references 로 분할 - #105

Merged
jon890 merged 2 commits into
mainfrom
chore/split-skill-references
Jul 14, 2026
Merged

chore(skill): SKILL.md 를 라우터 + references 로 분할#105
jon890 merged 2 commits into
mainfrom
chore/split-skill-references

Conversation

@jon890

@jon890 jon890 commented Jul 14, 2026

Copy link
Copy Markdown
Owner

개요

공개 스킬 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·체이닝 시나리오

검증

  • 본문 verbatim 이동 — 원본 39개 섹션 전부 references 에 존재 (누락 0)
  • 의도-매핑 마스터 표 70행 그대로 보존
  • 내부 참조(ADR/Issue/task) 0건, 개인·사내 식별자 0건

🤖 Generated with Claude Code

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>
@jon890
jon890 force-pushed the chore/split-skill-references branch from 566d6be to 22604ca Compare July 14, 2026 10:15
@jon890

jon890 commented Jul 14, 2026

Copy link
Copy Markdown
Owner Author

/review

Comment thread skills/dooray-cli/references/wiki.md Outdated

```bash
# --json 으로 구조화 출력 → jq 로 성공/실패 분리
RESULT=$(dooray post file download-all <project> <number> -o ./ --json)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 문제: 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 로 이동.

Comment thread skills/dooray-cli/references/post.md Outdated

---

---

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 문제: 133-135 라인에 --- 이 연달아 두 번 (섹션 사이 빈 구분선 중첩). 렌더링에 지장은 없지만 시각적으로 지저분.

수정: 세 곳 모두 정리 — 133-135, 177-179, 193-196 라인. 한 섹션 끝의 --- 하나만 남기고 다음 섹션 헤딩 앞의 여분 --- 삭제.

Comment thread skills/dooray-cli/references/common.md Outdated

---

---

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 문제: 83-85 라인에 --- 이 연달아 두 번 나옴 (---\n\n---). 원본 SKILL.md 를 잘라 붙일 때 상하 섹션 각각의 끝-시작 구분선이 겹친 것으로 보임.

수정: 83 또는 85 중 하나만 남기고 삭제.

@claude

claude Bot commented Jul 14, 2026

Copy link
Copy Markdown

코드 리뷰 chore(skill): SKILL.md 를 라우터 + references 로 분할

문서 전용 refactor — SKILL.md 698줄 → 46줄 라우터 + 7개 references 로 progressive disclosure 적용. 코드 변경 없음. 개인 식별 정보·내부 참조 번호 노출 없음(검증 통과). 마이너한 마크다운 정돈 몇 곳만 남았고 머지 차단 사유는 없음.


🟡 개선 권장

skills/dooray-cli/references/wiki.md 22-30
문제: wiki.md 인데 예시 코드가 dooray post file download-all 을 씀. 사용자가 라우팅 표에서 "위키 첨부" 로 이 파일에 왔을 때 post file 예시가 나오면 어느 명령을 쓸지 혼동.
수정: dooray wiki page file download-all <project> <page-id> -o ./ --json 로 교체(스키마 mirror 라 로직 그대로). post 관련 예시가 필요하면 workflow.md 나 별도 파일로 분리.

skills/dooray-cli/references/post.md 133-135, 177-179, 193-196
문제: 세 곳에서 --- 수평선이 연달아 두 번 나옴 (---\n\n--- 형태). 원본 SKILL.md 를 분할하며 상하 섹션의 끝-시작 구분선이 겹친 것으로 보임.
수정: 각 구간에서 --- 하나만 남긴다.

skills/dooray-cli/references/common.md 83-85
문제: 위와 동일 — 에러 핸들링 표 아래에 --- 이 두 번 연속.
수정: 하나만 남기고 삭제.

🟢 잘 된 점

  • 단일 스킬 유지 + progressive disclosure 방향 (nhncloud-cli 참고) — 트리거 하나 유지하면서 컨텍스트 부담을 줄이는 절충안이 실용적이고 PR body 에 그 판단 근거가 잘 정리됨
  • CLAUDE.md 규칙 준수 확인: SKILL.md 및 references/ 전 파일에서 ADR/Issue/task 내부 참조 번호 0건 (grep 통과)
  • PII 검증 통과: 사내 도메인·실제 19+자리 numeric ID·사내 이메일 노출 없음 — 모두 <placeholder> 또는 CLAUDE.md 허용 dummy(1234567890123456789) 사용
  • SKILL.md 46 줄 라우터가 "먼저 읽을 것 표 → 공통 우선 규칙 → 빠른 시작 → 안전한 탐색 순서" 로 명확히 구성되어 라우팅 목적에 부합
  • 라우팅 표 7행 ↔ references 7파일 정확히 매치, 각 카테고리 설명도 실제 파일 범위와 일치

🤖 Reviewed with Claude Code (single opus reviewer: type · conventions · security · architecture)
💬 인라인 코멘트는 Files changed 탭에서 확인하세요

- 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>
@jon890

jon890 commented Jul 14, 2026

Copy link
Copy Markdown
Owner Author

리뷰 반영 결과 (커밋 51a0175):

🟡 항목 1 (wiki.md 의 post file 예시): ✅ 반영 — 첨부파일 일괄 다운로드 후 실패 분리 시나리오를 wiki.md 에서 post.md 로 이동. 명령 교체 대신 재배치로 처리 (verbatim 보존 — 실제 post file 명령이므로).

🟡 항목 2 (post.md 중복 구분선): ✅ 반영 — 연속 --- 3곳 각각 하나만 남김.

🟡 항목 3 (common.md 중복 구분선): ✅ 반영 — 에러 표 아래 중복 --- 제거.

@jon890
jon890 merged commit 90ed842 into main Jul 14, 2026
1 check passed
@jon890
jon890 deleted the chore/split-skill-references branch July 14, 2026 10:35
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