メニュー
기존 server-exporter 코드를 4축 (구조 / 품질 / 보안 / 벤더 경계)으로 리뷰. PR 제출 전, 의심 영역 점검, 신규 작업자 온보딩 시. 사용자가 "코드 리뷰해줘", "이 파일 봐줘", "리뷰" 등 요청 시. - 코드 리뷰 일반 / PR 머지 전 / 4축 점검 (rule 22 fragment + rule 12 boundary + rule 60 security + rule 95 quality)
Codex または Claude でインストール この Prompt をコピーして Codex、Claude、または他のアシスタントに貼り付けると、Skill ページを確認してインストールできます。
SOC 職業分類に基づく
| name | review-existing-code |
| description | 기존 server-exporter 코드를 4축 (구조 / 품질 / 보안 / 벤더 경계)으로 리뷰. PR 제출 전, 의심 영역 점검, 신규 작업자 온보딩 시. 사용자가 "코드 리뷰해줘", "이 파일 봐줘", "리뷰" 등 요청 시. - 코드 리뷰 일반 / PR 머지 전 / 4축 점검 (rule 22 fragment + rule 12 boundary + rule 60 security + rule 95 quality) |
server-exporter 코드를 4 축 매트릭스로 리뷰 (.claude/policy/review-matrix.yaml 정본). PR 제출 전 또는 의심 발견 시 호출.
is none / is undefined / length == 0 혼동int() cast 미방어.claude/templates/REVIEW_SUMMARY.template.md 형식)## 4축 리뷰 결과 — <대상>
| 축 | 결과 | 발견 |
|---|---|---|
| Structure | PASS | — |
| Quality | WARN | int cast 미방어 1건 |
| Security | PASS | — |
| Vendor Boundary | FAIL | redfish-gather/library/ 안에 "Dell" 하드코딩 |
### 심각도별 list
- [CRIT] Vendor Boundary: redfish-gather/library/redfish_gather.py:152 "Dell" 하드코딩
- [WARN] Quality: redfish-gather/tasks/collect_storage.yml:88 int(capacity) 미방어
### 권고
1. CRIT 즉시 수정 PR
2. WARN은 다음 정리 PR (CONVENTION_DRIFT.md 기록)
### 회귀 검토 필요
- 영향 vendor: Dell (변경 시 baseline 회귀)
verify_vendor_boundary.py 결과 통합본인이 작성한 코드를 본인이 리뷰/승인하지 말 것. 별도 reviewer agent 호출.
.claude/policy/review-matrix.yamlvalidate-fragment-philosophy, verify-adapter-boundary, verify-json-outputcode-reviewer, adapter-boundary-reviewer, vendor-boundary-guardian, output-schema-reviewer, naming-consistency-reviewer, integration-impact-reviewer (cycle-011: security-reviewer 제거).claude/templates/REVIEW_SUMMARY.template.md/review-guideschema/sections.yml + schema/field_dictionary.yml + schema/baseline_v1/ 3종 정합 갱신 (rule 13 R1). schema 변경 후 호출. 사용자가 "schema 정합 갱신", "field dictionary 동기화", "ERD 갱신" (server-exporter 어휘로는 schema doc) 등 요청 시 또는 schema 5개 이상 누적 변경 시 자동. - schema/sections.yml or field_dictionary.yml 수정 후 / 새 섹션 / 새 필드 / Round 검증 후 baseline 변경
구조화된 server-exporter 기능 명세서 작성. 사용자가 "명세서 써줘", "스펙 정리" 등 요청 시. - 구조화된 spec 필요 / PR 본문 / 문서화
server-exporter에 새 벤더(예: Huawei iBMC, Inspur, Ampere)를 3단계로 추가하는 절차 안내. vendor_aliases.yml + adapters/{channel}/{vendor}_*.yml + (선택) OEM tasks + vault + baseline + ai-context + docs 동시 갱신. 사용자가 "Huawei vendor 추가", "새 벤더 추가", "Huawei 지원하자" 등 요청 시 자동 호출. - 새 vendor가 호출자 시스템에서 요청됨 / 펌웨어 프로파일링 후 정식 지원 / 기존 generic fallback 한계 시
사용자 사이트 사고 환경 (HPE Gen12 / Lenovo XCC3 / Dell iDRAC8 / Cisco CIMC 등) 의 fixture를 캡처해 회귀 회피. cycle 2026-05-01 학습 — 사이트 사고 fixture 부재로 다음 reverse regression 위험. 사이트 envelope / probe 결과 / 펌웨어 응답을 tests/fixtures/redfish/{vendor}_{firmware}/ 에 저장 + tests/evidence/<날짜>-<vendor>.md 작성. 사용자가 "사이트 fixture 캡처", "사고 재현 입력 저장" 요청 시.
lab 부재 vendor 추가 (web sources 의무). cycle 2026-05-06 학습 — 4 신규 vendor (Huawei/Inspur/Fujitsu/Quanta) + Superdome Flex 의 lab 부재 web sources 패턴 라이브러리화. 사용자가 "Huawei iBMC 추가", "lab 없는 vendor 어떻게?", "vendor 추가" 등 요청 시 + lab 부재 명시 시. - rule 50 R2 9단계 + rule 96 R1-A web sources 의무 + vault SKIP 사용자 명시 승인
다중 worker cycle 자동화 (Phase 0 ticket 작성 → Phase 1 분석 → Phase 2 결정 → Phase 3 구현 → Phase 4 회귀 → Phase 5 종료). cycle 2026-05-06 학습 — 24 ticket × 5 worker 패턴 라이브러리화. 사용자가 "cycle 시작", "사용자 N 작업 묶음 ticket 으로 분해해줘", "cycle 자동화" 등 요청 시. - 3 항목 이상 작업 묶음 / 다중 worker 동시 진행 / 호환성 / 큰 리팩토링