primitive-review

# Asset Review Skill 정적 `validate`(kebab id·순환·참조 무결성)가 잡지 못하는 **품질·판단** 문제를 LLM 관점으로 검토한다. CLI `solosquad <manager> review` 를 대체하는 대화형 표면 — review 는 비결정적 출력이라 CLI/CI 게이트가 아니라 Chief 와의 대화에서 수행한다(v1.3.2 conversational-first 결정). ## 입력 - **kind**: skill | agent | workflow | goal | cron - **id/경로**: 검토 대상 (예: `product/product-manager`, `workflows/market-research/workflow.yaml`) - (선택) 정적 validate 결과 — 이미 알려진 finding 은 반복하지 않는다. ## 절차 1. 대상 정의를 읽는다(`solosquad agent show <id>` / `workflow show` / 파일 직접). 2. 먼저 `solosquad <manager> validate` 로 정적 검증을 돌려 그 결과를 컨텍스트��� 둔다. 3. kind 별 렌즈로 **판단 문제**만 짚는다(정적 규칙이 잡는 건 제외): - **skill** — description 이 3인칭·��체적이라 라우팅 가능한가? process 가 실행 가능한가? 단일 목적인가? - **agent** — 팀 내 다른 actor 와 역할이 구별되나? 명확한 escalation/handoff 경로가 있나? collaborator 가 과배선 아닌가? - **workflow** — 각 stage 의 exit_criteria 가 실제로 측정 가능한가? handoff 순서가 합리적인가? fixed/agentic 선택이 맞나? - **goal** — success metric 이 객관적·판정 가능한가? 가드레일(budget/cycles)이 적정한가? modifiable_paths 가 ��게 잡혔나? - **cron** — cadence 가 prompt 의도와 맞나? channel 이 kind 에 맞나? prompt 가 무인 실행에 자족적인가? 4. 제안을 severity 로 분류해 제시: **blocker**(고쳐야 함) / **improvement**(권장) / **nit**(사소). ## 출력 - 한 줄 요약 + severity 태그된 actionable 제안 목록. - anti-sycophancy: 문제가 없으면 "문제 없음"이라고 분명히 말한다 — 억지 제안 금지. ## 비범위 - 정적 규칙으로 잡히는 것(id 형식·순환·깨진 참조)은 `validate` 의 몫. 여기서 반복하지 않는다. - 자동 수정은 하지 않는다 — 제안만. 적용은 사용자/author 루프가 결정.