Fix PR review comment links and metadata

Author: hwisu

README.ko.md

@@ -242,7 +242,7 @@ opensession cleanup run --apply
 - artifact branch TTL: 30일
 - GitHub/GitLab 설정 시 PR/MR 갱신마다 session artifact branch를 갱신하고 리뷰 코멘트를 남기는 session-review 자동화 템플릿도 함께 생성됩니다.
 - 기본값은 ephemeral PR/MR artifact branch이며 리뷰가 닫히면 삭제됩니다. `--session-archive-branch <branch>`를 설정하면 `pr/sessions` 같은 전용 archive branch에 immutable snapshot을 계속 보관합니다.
-- session-review 코멘트에는 `Reviewer Quick Digest` 블록이 포함되며, Q&A 발췌(질문/응답), 수정 파일, 추가/수정 테스트가 함께 표시됩니다.
+- session-review 코멘트에는 `Reviewer Quick Digest` 블록이 포함되며, 모바일 친화적인 Q&A 줄글 요약, 수정 파일, 추가/수정 테스트가 함께 표시됩니다.
 
 민감한 저장소는 즉시 정리 모드를 권장합니다.
 

README.md

@@ -243,7 +243,7 @@ Defaults:
 - artifact branch TTL: 30 days
 - GitHub/GitLab setup also writes PR/MR session-review automation that updates a session artifact branch and posts a review comment on PR/MR updates.
 - By default PR/MR artifact branches are ephemeral and are deleted when the review closes; set `--session-archive-branch <branch>` to keep immutable review snapshots on a dedicated archive branch such as `pr/sessions`.
-- Session review comments now include a `Reviewer Quick Digest` block with Q&A excerpts (question/answer rows), modified files, and added/updated tests.
+- Session review comments now include a `Reviewer Quick Digest` block with mobile-friendly Q&A prose, modified files, and added/updated tests.
 
 Sensitive repositories can force immediate cleanup:
 

docs.ko.md

@@ -216,7 +216,7 @@ opensession cleanup init --provider auto --session-archive-branch pr/sessions --
 - GitHub: `.github/workflows/opensession-cleanup.yml`와 `.github/workflows/opensession-session-review.yml`을 생성합니다. 기본값은 ephemeral `opensession/pr-<number>-sessions` 브랜치를 PR 동안만 유지하고 PR close 시 삭제합니다. `--session-archive-branch <branch>`를 설정하면 `pr/sessions` 같은 전용 archive branch에 immutable snapshot을 저장합니다.
 - GitLab: `.gitlab/opensession-cleanup.yml`와 `.gitlab/opensession-session-review.yml`을 생성합니다. `.gitlab-ci.yml`은 OpenSession 관리 마커 블록이 있을 때만(또는 새 파일일 때만) 갱신합니다. MR 파이프라인은 `opensession/mr-<iid>-sessions`를 게시/갱신하거나, `--session-archive-branch`가 설정된 경우 해당 archive branch를 사용합니다.
 - Generic git: cron/system scheduler 연동용 `.opensession/cleanup/cron.example`를 생성합니다.
-- session-review 코멘트에는 `Reviewer Quick Digest`가 포함되며, Q&A 발췌(`Question | Answer` 행), 수정 파일 요약, 추가/수정 테스트가 함께 표시됩니다.
+- session-review 코멘트에는 `Reviewer Quick Digest`가 포함되며, 모바일 친화적인 Q&A 줄글 요약, 수정 파일 요약, 추가/수정 테스트가 함께 표시됩니다.
 
 ## 개발 및 검증
 
@@ -346,7 +346,7 @@ opensession view main..feature/my-branch
 
 로컬 `view` 대상은 등록된 git credential이 필요하지 않습니다.
 로컬 git object / 로컬 source byte를 사용해 local review bundle을 만들기 때문입니다.
-commit-linked local review page는 Q&A 내용 발췌, 수정 파일, 추가/수정 테스트를 포함하는 `Reviewer Quick Digest` 패널을 노출합니다.
+commit-linked local review page는 모바일 친화적인 Q&A 내용 발췌, 수정 파일, 추가/수정 테스트를 포함하는 `Reviewer Quick Digest` 패널을 노출합니다.
 
 ## Handoff
 

docs.md

@@ -215,7 +215,7 @@ Provider matrix:
 - GitHub: `.github/workflows/opensession-cleanup.yml` plus `.github/workflows/opensession-session-review.yml` are generated. By default PR updates publish ephemeral `opensession/pr-<number>-sessions` branches and delete them when the PR closes; set `--session-archive-branch <branch>` to keep immutable review snapshots on a dedicated archive branch such as `pr/sessions`.
 - GitLab: `.gitlab/opensession-cleanup.yml` plus `.gitlab/opensession-session-review.yml` are generated; `.gitlab-ci.yml` is updated only when an OpenSession managed marker block exists (or file is newly created). MR pipelines publish/refresh `opensession/mr-<iid>-sessions` and post an MR note, or use the configured archive branch when `--session-archive-branch` is set.
 - Generic git: `.opensession/cleanup/cron.example` is generated for cron/system scheduler wiring.
-- Session-review comments include `Reviewer Quick Digest` with Q&A excerpts (`Question | Answer` rows), modified file summary, and added/updated tests.
+- Session-review comments include `Reviewer Quick Digest` with mobile-friendly Q&A prose, modified file summary, and added/updated tests.
 
 ## Development & Validation
 
@@ -344,7 +344,7 @@ Default mode is web. Use `--no-open` to print URL only.
 
 Local `view` targets do not require registered git credentials.
 They use local git objects / local source bytes and generate a local review bundle.
-Commit-linked local review pages expose a `Reviewer Quick Digest` panel that renders Q&A content excerpts (not just counts), modified files, and added/updated tests.
+Commit-linked local review pages expose a `Reviewer Quick Digest` panel that renders mobile-friendly Q&A content excerpts (not just counts), modified files, and added/updated tests.
 
 ## Handoff
 

docs/development-validation-flow.md

@@ -153,7 +153,7 @@ Desktop dry-run reliability workflow:
 Reviewer block highlights:
 
 - `Reviewer Quick Digest`
-- Q&A excerpts rendered as `Question | Answer` table rows (content, not counts)
+- Q&A excerpts rendered as mobile-friendly Markdown prose with session/commit context (content, not counts)
 - modified file summary
 - added/updated test file summary
-- direct local review deep-link (`/review/local/:id`) and commit/session trail
+- local replay command (`opensession review <github-pr-url>`), review ID, and commit/session trail

scripts/check-session-review-workflow.mjs

@@ -43,20 +43,23 @@ function main() {
   if (!reportScript.includes("'<!-- opensession-session-review-final -->'")) {
     fail('Final marker is missing in scripts/pr_session_report.mjs.');
   }
-  if (!reportScript.includes('| Metric | Value |')) {
-    fail('Report must include an overview metric table.');
+  if (reportScript.includes('| Metric | Value |')) {
+    fail('Report must not use markdown tables for overview metrics.');
   }
   if (!reportScript.includes('Files changed](') || !reportScript.includes('Commits](')) {
     fail('Report must include PR quick links to files/commits.');
   }
-  if (!reportScript.includes('Open in UI') || !reportScript.includes('ops review')) {
-    fail('Report must include local review deep-link.');
+  if (reportScript.includes('127.0.0.1:8788/review/local/')) {
+    fail('Report must not embed localhost review links in PR comments.');
   }
-  if (!reportScript.includes('Artifact branch |')) {
-    fail('Report must include artifact branch summary link.');
+  if (!reportScript.includes('**Local replay:**') || !reportScript.includes('localReviewCommand(')) {
+    fail('Report must include a local replay command instead of localhost links.');
   }
-  if (!reportScript.includes('| Session ID | Tool | Files | Commits | Open | OpenSession | JSONL | Meta | Title |')) {
-    fail('Report must include session metadata columns for per-session navigation.');
+  if (!reportScript.includes('**Artifact storage:**')) {
+    fail('Report must include artifact storage metadata.');
+  }
+  if (reportScript.includes('| Session ID | Tool | Files | Commits | Open | OpenSession | JSONL | Meta | Title |')) {
+    fail('Report must not use markdown tables for per-session navigation.');
   }
   if (!reportScript.includes('opensessionSourceLink(')) {
     fail('Report must build opensession.io source links for web review.');
@@ -85,14 +88,20 @@ function main() {
   if (!reportScript.includes('#### Reviewer Quick Digest')) {
     fail('Report must include Reviewer Quick Digest block for high-signal review context.');
   }
-  if (!reportScript.includes('| Q&A | Areas | Files | Tests | Sessions / Commit |')) {
-    fail('Report must summarize review KPIs in a digest table.');
+  if (!reportScript.includes('**Comment type:**') || !reportScript.includes('**Review ID:**')) {
+    fail('Report must include review metadata for comment type and review id.');
+  }
+  if (!reportScript.includes('**Coverage:**')) {
+    fail('Report must summarize review KPIs in digest prose.');
+  }
+  if (!reportScript.includes('**Top areas:**')) {
+    fail('Report must summarize changed areas in the quick digest.');
   }
-  if (!reportScript.includes('#### Area Summary')) {
-    fail('Report must summarize changed areas.');
+  if (reportScript.includes('| Session | Commit | Question | Answer |')) {
+    fail('Report must not use markdown tables for Q&A digest rows.');
   }
-  if (!reportScript.includes('| Session | Commit | Question | Answer |')) {
-    fail('Report must render Q&A digest with session and commit context.');
+  if (!reportScript.includes('**Question:**') || !reportScript.includes('**Answer:**')) {
+    fail('Report must render Q&A digest as prose with session and commit context.');
   }
   if (!reportScript.includes('primary only (auxiliary filtered)')) {
     fail('Report must describe primary-session filtering.');
@@ -106,7 +115,7 @@ function main() {
   if (!reportScript.includes('collectAreaSummary')) {
     fail('Report script must derive area summary rows from changed files.');
   }
-  if (!reportScript.includes('Updated at (UTC)')) {
+  if (!reportScript.includes('**Updated at (UTC):**')) {
     fail('Report must include update timestamp for per-run freshness.');
   }