이랑이
da85b92a6b
Sprint 006 전체 구현 — F3/F2 의 QA 측면 완성:
Schema:
- src/qa/schema.ts — QaTemplate, QaChecklistResult, QaArtifact Zod 스키마
- Zod + DodCheck 재사용
Templates (6종 YAML):
- qa-templates/scaffold-v1.yaml — README/LICENSE/gitignore/lockfile/strict
- qa-templates/feature-v1.yaml — tests/typecheck/no-console/no-any/tests-added
- qa-templates/bugfix-v1.yaml — regression-test/root-cause/no-scope-creep
- qa-templates/refactor-v1.yaml — tests/typecheck/no-behavior-change
- qa-templates/migration-v1.yaml — rollback/dry-run/data-loss/backup (critical)
- qa-templates/infra-v1.yaml — config-validated/secrets/rollback
Core:
- src/qa/template.ts — YAML loader, extends 체인 resolution, 프로젝트별 extras
- src/qa/verdict.ts — verdict 규칙 (critical/major → REQUEST_CHANGES,
minor/recommendation 만 → APPROVE_WITH_NITS, 절대 REQUEST_CHANGES 안 됨)
- src/qa/runtime.ts — runQaTemplate: Contract check handlers 재사용
manual 체크는 resolver 주입 가능 (없으면 SKIP 기본값)
CLI:
- rails qa run <type> [-s sprint-id] [-w workdir]
- rails qa show <artifact-id>
- rails qa templates (목록)
Tests (19 신규, 101 total pass):
- computeVerdict 7가지 시나리오 (APPROVE / APPROVE_WITH_NITS / REQUEST_CHANGES / ABORT)
- minor-only 는 절대 REQUEST_CHANGES 안 된다는 rule 명시 테스트
- Template loader + listTemplates + extends merge
- runtime: file_exists pass/fail + manual resolver 주입 + artifact 저장
- scaffold-v1 실파일 로드 확인
검증: tsc --noEmit ✓ | vitest 101/101 ✓ | build ✓
rails qa templates → 6개 전부 출력 ✓
사용자 메모리 feedback_qa_thorough.md 준수:
- 체크 항목 수 제한 없음
- 각 템플릿이 타입별로 세분화됨
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
51 lines
1.3 KiB
YAML
51 lines
1.3 KiB
YAML
template: refactor-v1
|
|
version: v1
|
|
appliesTo: [refactor]
|
|
|
|
requiredChecks:
|
|
- id: tests-pass
|
|
description: 리팩터 후 모든 테스트 pass
|
|
kind: command_success
|
|
spec:
|
|
command: pnpm test
|
|
timeoutMs: 120000
|
|
expectExitCode: 0
|
|
blocking: true
|
|
severity: major
|
|
|
|
- id: typecheck
|
|
description: 타입 체크 pass
|
|
kind: command_success
|
|
spec:
|
|
command: pnpm tsc --noEmit
|
|
timeoutMs: 60000
|
|
expectExitCode: 0
|
|
blocking: true
|
|
severity: major
|
|
|
|
- id: no-behavior-change
|
|
description: 외부 동작 변경 없음 (순수 리팩터)
|
|
kind: manual
|
|
spec:
|
|
question: 사용자 관찰 가능한 동작이 변경되지 않았는가?
|
|
guidance: 만약 변경되었다면 feature 로 재분류되어야 함.
|
|
blocking: true
|
|
severity: major
|
|
|
|
- id: tests-still-cover
|
|
description: 기존 테스트 커버리지 유지
|
|
kind: manual
|
|
spec:
|
|
question: 리팩터로 인해 테스트가 삭제되거나 우회되지 않았는가?
|
|
blocking: true
|
|
severity: major
|
|
|
|
- id: public-api-compatible
|
|
description: 공개 API 하위 호환
|
|
kind: manual
|
|
spec:
|
|
question: 공개 export 의 시그니처가 변경되지 않았는가?
|
|
guidance: 변경되었다면 breaking-change flag 필요.
|
|
blocking: false
|
|
severity: minor
|