김경종
43 lines
1.8 KiB
Markdown
43 lines
1.8 KiB
Markdown
---
|
|
name: harness-review
|
|
description: "Use when reviewing this Harness repository: local changes, generated phase files, step outputs, implementation diffs, missing tests, build readiness, or compliance with AGENTS.md, docs/ARCHITECTURE.md, docs/ADR.md, and Harness acceptance criteria."
|
|
---
|
|
|
|
# Harness Review
|
|
|
|
## Overview
|
|
|
|
Use this skill to review Harness work against the repository's persistent rules, architecture docs, and executable verification requirements. Prioritize bugs, regressions, missing tests, and rule violations.
|
|
|
|
## Review Process
|
|
|
|
1. Read `/AGENTS.md`, `/docs/ARCHITECTURE.md`, and `/docs/ADR.md`.
|
|
2. Inspect the changed files with `git status --short` and `git diff`.
|
|
3. Check architecture, stack choices, tests, critical rules, and build readiness.
|
|
4. Run relevant verification commands when feasible. If a command cannot be run, report that as residual risk.
|
|
5. Lead with actionable findings. Keep summaries secondary.
|
|
|
|
## Checklist
|
|
|
|
| Item | Question |
|
|
| --- | --- |
|
|
| Architecture | Does the change follow `docs/ARCHITECTURE.md` directory and module boundaries? |
|
|
| Stack | Does the change stay within choices documented in `docs/ADR.md`? |
|
|
| Tests | Are new or changed behaviors covered by tests? |
|
|
| Critical Rules | Does the change violate any `AGENTS.md` CRITICAL rule? |
|
|
| Build | Do relevant build/test/lint commands pass? |
|
|
|
|
## Output Format
|
|
|
|
If there are findings, list them first in severity order with file and line references when possible. Then include this table:
|
|
|
|
| 항목 | 결과 | 비고 |
|
|
| --- | --- | --- |
|
|
| 아키텍처 준수 | PASS/FAIL | {상세} |
|
|
| 기술 스택 준수 | PASS/FAIL | {상세} |
|
|
| 테스트 존재 | PASS/FAIL | {상세} |
|
|
| CRITICAL 규칙 | PASS/FAIL | {상세} |
|
|
| 빌드 가능 | PASS/FAIL | {상세} |
|
|
|
|
When there are no findings, say that clearly, then mention any commands not run or remaining risk.
|