docs: keep product gap baseline free of mutable lifecycle snapshots - #384
Conversation
📝 WalkthroughWalkthrough제품 기술 기준선을 날짜 기반 운영 스냅샷에서 보호된 정적 기준선으로 변경했다. 실시간 GitHub 증거 조회 규칙, 제품 경계, P0–P2 수용 기준, 결함 조사와 릴리스 종료 조건을 추가했다. 최신성 계약을 검증하는 Vitest 테스트도 추가했다. Changes제품 기술 기준선
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: 🔵 Low · up to The PR removes mutable lifecycle snapshots from the product gap baseline and adds freshness checks. It is mergeable with owner awareness, but the canonical documentation should be aligned and the freshness test strengthened so stale claims cannot bypass validation. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
@opencode-agent @cwl-noema-review Review-only request for exact current head Please submit only a formal GitHub review bound to that exact head after independently verifying the live diff and current repository contracts. Current review threads are resolved, but the Reviews API has zero If the head/base moves, treat this request as stale and require fresh exact-head/live-base evidence. |
|
Agreed on the causal defect: embedding mutable lifecycle snapshots in a protected static document makes staleness a publish-time property. The freshness-contract test is the right enforcement boundary. Merging via the organization admin path; the release CHANGELOG consolidation will rebase onto this structure. |
Causal defect
The product/technical gap baseline merged from #372 embedded mutable GitHub lifecycle snapshots in a protected static document. That lets a document become stale in the same lifecycle transition that publishes it and can mislead later merge, release, ownership, or readiness decisions.
Protected
mainis the shipped implementation authority. This PR changes only the static documentation/freshness-contract boundary; it does not make this PR body, a workflow summary, or any recorded SHA/run ID into lifecycle authority.TDD / repair lineage
dcbd74eaee1c771f7f4bcf2d6724e54fca01b535introduced the baseline-freshness contract. Its hosted run was superseded before terminal completion, so no terminal RED claim is made from that run.f21bd3ca75d0803823ba83cc88661f8e5b00d584strengthened the contract after review: it rejects formatted/multiline protected-main SHA and PR-status snapshots, requires the decision-time live-refetch rule, and requires the canonical index to describe the baseline as static rather than live status. Against the then-unchanged index, the new index assertion was intentionally RED.78d3f447315f85830b26c4bf81bdb5c376f1fcdfaligneddocs/README.mdwith the same static-baseline/live-refetch contract. Exact-head repository CI subsequently proved typecheck, exact coverage, package consumers, demo build, Office matrix, and cross-engine checks on that source generation.Historical or predecessor workflow/review results are lineage only and never transfer to a later head.
Live acceptance rule
Do not infer current merge/release readiness from this body. Immediately before any merge, release, readiness, closure, or ownership decision, independently refetch at least:
mainand the exact PR head/live base plus ancestry/mergeability;Pending, queued, skipped, cancelled, absent, neutral, failed, stale, predecessor, wrong-checkout, synthetic-source-only, status-only, or model-only evidence is non-passing. Automated comments/reviews are technical input, not qualifying independent approval. Any head/base/ruleset movement invalidates the corresponding decision evidence.
Ownership
This follows the merged #372 documentation owner and is limited to:
docs/product-technical-gap-baseline.md;docs/README.mdcanonical-index wording needed to keep that contract coherent; andIt does not move transport, authentication, tenancy, durable persistence, credentials, migration, retention, deployment, durable audit, Yjs authority, model policy, durable PDF authority, or release authority into Inkspan. If another active writer begins touching this exact documentation boundary before integration, reconcile ownership before further source mutation rather than racing it.