Skip to content

[codex] clarify spec evidence and coverage boundaries#188

Merged
sungjunlee merged 1 commit into
mainfrom
codex/spec-evidence-boundaries
Jun 3, 2026
Merged

[codex] clarify spec evidence and coverage boundaries#188
sungjunlee merged 1 commit into
mainfrom
codex/spec-evidence-boundaries

Conversation

@sungjunlee
Copy link
Copy Markdown
Owner

@sungjunlee sungjunlee commented Jun 3, 2026

Summary

  • Clarify that task AC, sprint execution context, and relay Done Criteria stay outside durable spec/* files.
  • Add Markdown-first repo evidence guidance for spec-system-map and multi-evidence admission guidance for spec-grill.
  • Mark component-lint and capabilities-doctor as structural-only checks and expose coverage-not-assessed fields/counts.

Validation

  • node --test skills/dev-backlog/scripts/*.test.js skills/spec-charter/scripts/*.test.js skills/spec-grill/scripts/*.test.js skills/backlog-triage/scripts/*.test.js
  • git diff --check HEAD~1..HEAD

Closes #183
Closes #184
Closes #185
Closes #186
Closes #187

Summary by CodeRabbit

릴리스 노트

  • Documentation

    • 작업 아티팩트와 스펙 문서 간의 경계 규칙을 명시하여 증거 사용 원칙을 강화했습니다.
    • 역량 승인 기준에 여러 증거 클래스 요구 사항을 추가했습니다.
    • 스펙 시스템 문서의 책임 범위와 사용 원칙을 상세화했습니다.
  • Chores

    • 스펙 진단 및 컴포넌트 검증 도구의 구조 평가 로직을 개선했습니다.
    • 스프린트 라우팅 처리 기능을 추가하여 도구 기능을 확장했습니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

이 PR은 spec 시스템의 원본 소유권 경계를 명확히 하고, 작업 수용 기준과 내구 규칙 간의 복제를 금지하며, 스펙 수용 요건을 강화하고, 구조 검사가 커버리지를 평가하지 않음을 명시합니다.

Changes

Spec 경계 및 문서 정책

Layer / File(s) Summary
핵심 spec 경계 정책 문서화
spec/README.md, docs/spec-system-design.md, skills/dev-backlog/references/integration-contract.md, skills/spec-grill/templates/capabilities.md
spec/* 문서가 작업 AC/Done Criteria를 참고할 수 있으나 복제 금지, GitHub Issues/dev-relay/backlog/sprints/ 소유권을 표로 정의, 아티팩트 경계와 증거 재진술 규칙을 명시합니다.

Capability 수용 및 증거 요구사항

Layer / File(s) Summary
다중 증거 클래스 수용 요구사항
skills/spec-grill/SKILL.md, skills/spec-grill/references/capabilities.md
Brownfield capability는 최소 2개 증거 클래스(예: system-map 경계+스크립트, 소스+테스트)가 필요하며, 사용자 override 시 누락 증거를 리포트에 명시하도록 규정합니다. 원시 신호 vs 코드-이해 증거를 구분하고 grill 리포트 형식을 강화합니다.

System Map Repo 증거 Pass

Layer / File(s) Summary
Repo 증거 Pass 및 품질 검사
skills/spec-system-map/SKILL.md
Create 모드 이전에 엔트리포인트, 설정, 테스트, 상태/외부 시스템을 검사하는 Repo Evidence Pass를 추가하고, brownfield 경계를 깊이 있게 평가하며, 완료 시 Evidence Read/Missing을 보고합니다.

린트 및 닥터 구조 전용 표시

Layer / File(s) Summary
Capabilities-doctor 구조 전용 표시
skills/dev-backlog/scripts/capabilities-doctor.js, skills/dev-backlog/scripts/capabilities-doctor.test.js
모든 경우에 structuralOnly: true, coverage: "not_assessed"를 반환하고, 출력 문구를 스펙 위생 검사만 수행함을 강조하며, 테스트에서 새 필드와 변경된 문구를 검증합니다.
Component-lint 라우팅 핸들 카운팅 및 구조 표시
skills/dev-backlog/scripts/component-lint.js, skills/dev-backlog/scripts/component-lint.test.js
parseSprintStatus, countSprintRouting 헬퍼를 내보내고 라우팅/활성 상태 통계를 계산하며, 구조 전용 플래그 및 라우팅 카운트를 결과에 포함시키고, 인간 가독 리포트와 테스트 단언을 갱신합니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • sungjunlee/dev-backlog#100: 두 PR 모두 docs/spec-system-design.md의 spec 시스템 규칙을 수정하며 직접 관련(이전 PR이 계층화 spec 아키텍처를 정의, 본 PR이 작업 수용 경계와 재사용 제약을 추가).

Poem

🐰 어둠 속 경계를 그으리니,
작업은 이곳, 규칙은 저곳에.
증거 깊게, 명세 얕게,
다중 클래스 문이 열리거라 ✨
구조만 확인, 커버리지는 아직,
투명한 선언이 신뢰를 쌓는다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 PR의 주요 변경 내용인 spec 증거 및 커버리지 경계 명확화를 정확하게 요약하고 있습니다.
Linked Issues check ✅ Passed PR의 변경 사항들이 모든 연결된 이슈(#183-#187)의 요구 사항을 충족합니다. spec 소유권 경계 명확화, 증거 패스 강화, 다중 증거 규칙 추가, 깊은 조사 원칙 문서화, 구조 검증 표시 등이 구현되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 명시된 이슈 목표(#183-#187)의 범위 내에 있습니다. 문서 개선, 스크립트 출력 개선, 테스트 강화가 모두 spec 증거/경계 명확화 목표와 직접적으로 관련됩니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/spec-evidence-boundaries

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sungjunlee sungjunlee marked this pull request as ready for review June 3, 2026 13:54
@sungjunlee sungjunlee merged commit 7e1bbb1 into main Jun 3, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment