Skip to content

docs(gaps): refresh exact-head delivery evidence - #1161

Open
seonghobae wants to merge 2 commits into
mainfrom
docs/exact-head-gap-refresh-20261004
Open

seonghobae wants to merge 2 commits into
mainfrom
docs/exact-head-gap-refresh-20261004

Conversation

@seonghobae

@seonghobae seonghobae commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Refresh the gap baseline from the fetched main head and preserve the unique dated evidence from the existing baseline PR lane. The snapshot separates ADR/research authority, protected implementation, open candidate heads, and synthetic/runtime acceptance. It records exact heads for the 16 ready main-targeted PRs, checks, review state, integration conflicts, and unverified protection status.

The Voice gap section assesses current candidate #1153 without describing it as delivered. It records the independently authorized derivation-Post boundary, retains truth/cutoff and PROV-O constraints, and links synthetic desktop/mobile Storybook screenshots. The inventory wording distinguishes hidden evidence from authorized evidence outside the bounded traversal.

Validation: documentation hygiene 5 passed; git diff --check passed. Candidate #1153 local focused suites passed (77 backend, 50 frontend); full frontend lint, 564 tests, and production build passed. Its authenticated PostgreSQL API test stopped at token setup with HTTP 400 before database/API assertions, so authenticated runtime acceptance remains unavailable.

Summary by CodeRabbit

  • 문서
    • 근거 게시물이 없거나 접근 권한이 없는 경우, 파생 Voice가 보기 및 내보내기에서 제외되며 대체 게시물로 표시되지 않는다고 명시했습니다.
    • CSV 내보내기에서 carrying Post와 derivation Post를 구분하고 기록된 유효 기간을 보존하는 점, JSON-LD 내보내기에서 해당 Voice를 생략하는 점을 문서화했습니다.
    • 좁은 화면에서 근거 작업에 접근하는 방법과 권한이 있는 근거 게시물의 참조 범위를 명확히 했습니다.

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

🧰 Additional context used
📚 Code guidelines (2)
docs/adr/README.md — auto-discovered
docs/storybook-inventory.md — auto-discovered

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d8d2a254-ecb7-4862-8623-81230393fc76
📥 Commits

Reviewing files that changed from the base of the PR and between a67c5b0 and 4677412.

⛔ Files ignored due to path filters (2)
  • docs/screenshots/ontology-voice-export-desktop.png is excluded by !**/*.png
  • docs/screenshots/ontology-voice-export-mobile.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • docs/product-technical-gap-baseline.md
  • docs/storybook-inventory.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

문서는 별도 evidence Post가 없거나 권한이 없을 때 파생 Voice를 보기와 내보내기에서 제외한다고 설명합니다. CSV·JSON-LD export의 감사 정보와 390 CSS 픽셀 화면에서의 스크롤 동작도 기록합니다.

Changes

Evidence 및 export 문서

Layer / File(s) Summary
Evidence 및 export 안내 갱신
docs/storybook-inventory.md
별도 evidence Post가 없거나 권한이 없으면 파생 Voice를 보기와 내보내기에서 제외하고 carrying Post로 대체하지 않는다고 명시했습니다. CSV의 carrying Post와 derivation Post 구분, validity interval 보존, JSON-LD의 Voice 생략 및 독립적으로 권한이 부여된 evidence Post의 traversal 외부 참조를 설명합니다. 390 CSS 픽셀 너비에서 페이지 오버플로 없이 영역을 스크롤하는 동작도 기록했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 46774

This documentation change does not alter runtime behavior, and its evidence visibility and export descriptions match the existing implementation. The narrow-screen statement has no demonstrated contradiction, so no concrete merge blocker is established.

Architecture Summary

Architecture risk: 🔵 Low · up to 46774

The change affects 1 system.

Changed systems: docs

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in docs/storybook-inventory.md: SeparateVoiceEvidence 설명이 evidence Post 부재 시의 필터링·내보내기 동작에서, evidence가 없거나 권한이 없는 경우 파생 Voice를 보기와 내보내기에서 제외하고 carrying Post를 대체로 사용하지 않는 동작으로 변경되었습니다.
  • observed — Modified behavior in docs/storybook-inventory.md: CSV의 carrying Post와 derivation Post 구분 및 validity interval 보존, 390 CSS 픽셀에서의 스크롤 동작, JSON-LD에서 evidence가 없거나 권한이 없는 추가 Voice의 생략, 독립적으로 권한이 부여된 evidence Post의 traversal 외부 참조를 감사 설명에 추가했습니다.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 main 기준의 갭 현황을 갱신하고 exact-head 전달 근거를 정리하는 주요 변경 사항을 명확히 설명합니다.
Docstring Coverage ✅ Passed 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 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

This branch has not been deployed

No deployments
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