Skip to content

docs: close issue 84 coverage-truth documentation - #417

Merged
seonghobae merged 2 commits into
mainfrom
docs/issue-84-coverage-truth
Aug 17, 2026
Merged

docs: close issue 84 coverage-truth documentation#417
seonghobae merged 2 commits into
mainfrom
docs/issue-84-coverage-truth

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Purpose

Close the remaining canonical-documentation portion of issue #84 after #71 integrated the Test Strategy, gap audit, and traceability graph.

Verified defect

Fresh protected-main inspection found one surviving contradiction in CLAUDE.md: contributor guidance still said /* v8 ignore */ markers in src/index.ts were deliberate even though issue #84 source repair removed the broad credential/security exclusions and the newly protected docs/TEST_STRATEGY.md makes their reintroduction a regression.

Test-first repair

  • RED contract 3df14a9378a748e8e58fe4c4d9fedf05bfc0b063 adds a documentation test that rejects the stale statement and requires contributor guidance to link to docs/TEST_STRATEGY.md and state the surviving broad-exclusion invariant. Its first PR-triggered runs were cancelled when the branch immediately advanced; cancellation is not counted as passing or as CI-proven RED.
  • GREEN candidate c0006b50066f6f3ba883138a95cfbb476cecc86e replaces only the stale CLAUDE coverage sentence: broad credential/security V8 exclusions are regressions and the canonical exact-coverage policy lives in docs/TEST_STRATEGY.md.

CHANGELOG.md was inspected separately and did not contain the stale v8 ignore claim, so no churn was added there merely to satisfy the issue checkbox.

Evidence boundary

Current protected base: a9f491f5ba4cd8249456504fefb071d0cb33bc17.

Current exact head: c0006b50066f6f3ba883138a95cfbb476cecc86e.

Fresh application CI, reviewer-ci, and eligible central Security Scan are queued on this exact head. They are non-passing until terminal success. The PR remains Draft until exact-head evidence is green and review findings, if any, are addressed.

Summary by CodeRabbit

  • 문서

    • 테스트 안내를 운영 코드 100% 커버리지 정책과 예외 근거 중심으로 업데이트했습니다.
    • Credential 및 보안 관련 광범위한 V8 제외가 의도된 동작으로 안내되지 않도록 관련 지침을 명확히 했습니다.
  • 테스트

    • 테스트 안내 문서가 올바른 정책과 참조를 포함하는지 자동으로 검증합니다.
    • 향후 부적절한 커버리지 제외 안내가 다시 추가되는 것을 방지합니다.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 973d8ce7-64a5-4c20-9842-9a81ccd7a63c

📥 Commits

Reviewing files that changed from the base of the PR and between a9f491f and c0006b5.

📒 Files selected for processing (2)
  • CLAUDE.md
  • test/documentation-active-work-contract.test.ts

📝 Walkthrough

Walkthrough

CLAUDE.md의 테스트 커버리지 안내가 docs/TEST_STRATEGY.md 정책 참조로 변경되었습니다. 문서 계약 테스트는 credential/security 관련 광범위한 V8 제외 안내가 다시 추가되지 않는지 검증합니다.

Changes

테스트 커버리지 정책

Layer / File(s) Summary
커버리지 정책 안내 및 회귀 검증
CLAUDE.md, test/documentation-active-work-contract.test.ts
CLAUDE.md가 100% 운영 코드 커버리지 정책과 제한적인 근거 기반 예외를 참조합니다. 문서 계약 테스트가 docs/TEST_STRATEGY.md 참조와 광범위한 credential/security V8 제외 안내의 부재를 검증합니다.

Estimated code review effort: 2 (간단) | ~10분

✨ 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 docs/issue-84-coverage-truth

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.

@seonghobae
seonghobae marked this pull request as ready for review August 17, 2026 19:59
@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@seonghobae
seonghobae merged commit 38d2b2d into main Aug 17, 2026
15 of 16 checks passed
@seonghobae
seonghobae deleted the docs/issue-84-coverage-truth branch August 17, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant