Skip to content

refactor(skill): handdrawn-diagram 색 위계·분기 도형 규칙 개선#22

Merged
Hyeongseob91 merged 1 commit into
mainfrom
refactor/handdrawn-diagram
Jun 26, 2026
Merged

refactor(skill): handdrawn-diagram 색 위계·분기 도형 규칙 개선#22
Hyeongseob91 merged 1 commit into
mainfrom
refactor/handdrawn-diagram

Conversation

@HyeonsangKim

Copy link
Copy Markdown
Contributor
  • 의미 기반 3색(60-30-10): 메인 파이프라인만 강조, 나머지 중립색
  • 분기는 마름모 {..} + Yes/No 조건 라벨 필수 규칙 추가
  • 예시 템플릿을 main/decision/muted/alert 4종으로 재구성
  • 한글+영문 혼합 라벨 끝글자 클리핑 gotcha 보강 (렌더 검증으로 확인)

Quality Score: 91/100

Summary

Changes

Type

  • feat — new agent / command / skill / capability
  • fix — bug fix
  • docs — documentation only
  • refactor / chore

Checklist

  • 새 agent/command/skill을 plugin.json에 등록했다 (해당 시)
  • 카운트를 모두 동기화했다 — plugin.json, marketplace.json, README.{md,ko,cn} 배지/표, CLAUDE.md (해당 시)
  • python3 .github/scripts/validate_plugin.py 통과
  • 버전 변경 시 marketplace.jsonplugin.json ↔ README/CLAUDE.md 일치
  • 사용자 노출 콘텐츠는 한국어, 코드/커밋은 영어

Related issue

- 의미 기반 3색(60-30-10): 메인 파이프라인만 강조, 나머지 중립색
- 분기는 마름모 {..} + Yes/No 조건 라벨 필수 규칙 추가
- 예시 템플릿을 main/decision/muted/alert 4종으로 재구성
- 한글+영문 혼합 라벨 끝글자 클리핑 gotcha 보강 (렌더 검증으로 확인)

Quality Score: 91/100
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Hyeongseob91 Hyeongseob91 self-requested a review June 26, 2026 03:16
@Hyeongseob91

Copy link
Copy Markdown
Contributor

✅ Review: APPROVE

문서 1개(SKILL.md)만 변경, 동작 변경 없음. 위험도 낮음.

항목 평가
60-30-10 의미색 규칙 (muted 60% / main 30% / semantic 10%) ✅ 명확하고 실용적
분기 = 마름모 {} + Yes/No 라벨 강제 ✅ "fork without diamond reads as a straight line" 설명이 정확
한글+영문 끝글자 클리핑 gotcha ✅ 실제 PNG 렌더 검증 기반, <br/> 분리 해법 구체적
예시 템플릿 4종(main/decision/muted/alert) 재구성 ✅ 일관

실전 경험칙("A single trailing space is often NOT enough — verify the PNG")이 잘 녹아 있어 신뢰도 높음. 바로 머지 가능.

🤖 Generated with Claude Code

@Hyeongseob91 Hyeongseob91 merged commit f1e995f into main Jun 26, 2026
1 check passed
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.

2 participants