메뉴
Multi-persona branch review for ai-defense-cli before opening or merging a PR. Runs five CLI-focused personas, writes a report, fixes BLOCKER/MAJOR issues, re-reviews. Use when the user asks for a code review or branch readiness check.
기본적으로 소스를 먼저 확인하는 Prompt가 선택됩니다. 직접 명령으로 전환하거나 로컬 사본을 다운로드할 수도 있습니다.
설치 여부를 결정하기 전에 SKILL.md와 SkillsMP에 표시된 보조 파일을 읽어 보세요.
Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.
직접 명령은 검토 Prompt를 거치지 않습니다. 실행하기 전에 소스를 확인하세요.
npx skills add https://github.com/cisco-ai-defense/ai-defense-cli --skill pr-review명령은 한 줄로 유지됩니다. 복사하기 전에 가로로 스크롤해 전체 내용을 확인하세요.
로컬 사본을 원하시나요? SkillsMP에서 현재 제공할 수 있는 파일을 다운로드하세요.
SKILL.md 표시 중
SOC 직업 분류 기준
| name | pr-review |
| description | Multi-persona branch review for ai-defense-cli before opening or merging a PR. Runs five CLI-focused personas, writes a report, fixes BLOCKER/MAJOR issues, re-reviews. Use when the user asks for a code review or branch readiness check. |
Review a local branch against a target (default main) before merge.
GitHub PRs: .github/workflows/pr-persona-review.yml posts the same persona
synthesis as an advisory PR comment only (read-only — no auto-commits). Requires
repo secret PERSONA_REVIEW_API_KEY.
Read .github/prompts/system.md plus each persona file:
| Persona | File | When to include |
|---|---|---|
| Engineer | persona-engineer.md | Go code changed |
| Security | persona-security.md | Go code, workflows, Dockerfile |
| CLI Contract | persona-cli-contract.md | Commands, flags, exit codes, output |
| Release | persona-release.md | Workflows, Makefile, Dockerfile, install docs |
| Docs | persona-docs.md | docs/, README, CHANGELOG, specs |
Omit personas with nothing relevant in the diff.
1. SCOPE → git diff target...branch --stat
2. REVIEW → Run personas (parallel batches of 3–5)
3. SYNTHESIZE → Apply .github/prompts/synthesizer.md
4. REPORT → Write reviews/{branch-short}-review.md
5. FIX → BLOCKER and MAJOR findings
6. RE-REVIEW → Re-run affected personas; overwrite report
7. PUSH → When PRS ≥ 70 and no BLOCKERs (user asks to push)
git diff <target>...<branch> --stat
git log <target>...<branch> --oneline
git diff <target>...<branch> --name-only
| Finding | Action |
|---|---|
| Code bug, missing test, wrong exit mapping | Fix in code |
| Secret leak, missing timeout | Fix in code |
| Doc/spec drift | Fix docs |
| Release wiring | Fix workflow/Makefile |
| Architecture / external API change | Defer — list under "Human follow-up" |
After fixes:
go build ./... && go vet ./... && go test ./...
Commit only when the user asks:
git commit -m "fix(review): address PR review findings"
reviews/{branch-short-name}-review.md (gitignored — local only, do not commit)
Optional: render a canvas if the user wants a visual report (see Cursor canvas skill).
- [ ] Diff scoped
- [ ] Personas run on relevant files
- [ ] Report written with PRS
- [ ] BLOCKER/MAJOR fixed or deferred with rationale
- [ ] Tests pass