[OnlyReadOneBook]

QA 회차 문서와 커밋별 런타임 증거를 Git으로 추적하는 저장소 정본으로 모읍니다. 최신 develop의 빌드 셋업과 개발 규칙은 보존하고, 길어진 배포·스크린샷 안내는 전용 문서로 분리했습니다.

무엇이 바뀌나

• docs/qa/round-1.md, round-2.md, status.md에 회차 이력·현재 커버리지·3차 QA 진행 상태를 정리했습니다.
• 로컬 QA 원본 34건을 docs/qa/evidence/<full-head-sha>.json 34건과 manifest.json으로 1:1 이관했습니다.
• .github/scripts/qa-evidence.mjs가 원본 필드 정규화, 민감정보 비식별화, 동일 HEAD 덮어쓰기 방지, manifest 해시 생성·검증을 담당합니다.
• 계정·수신자 ID·인증값·표시명·기기 serial/AVD·로컬 경로·비공개 URL은 제거하고, 공개 GitHub 이슈/PR 링크와 QA 의미는 보존합니다.
• qa-device-profile.sh --json은 공개 증거에 넣을 수 있도록 로컬 serial과 AVD 이름을 출력하지 않습니다.
• README는 모듈·문서 진입점과 최신 빌드 셋업·개발 규칙만 유지하고, 배포와 스크린샷 절차는 docs/release/distribution.md, docs/testing/screenshot.md로 분리했습니다.

이관 검증

항목	결과
원본 ↔ 공개 파일 ↔ manifest	34 ↔ 34 ↔ 34, HEAD 중복 0
필드 대조	head_sha·result·scenario·not_covered 34/34, 나머지 공개 가능한 관찰은 details에 보존
원본 스키마	schema 1 32건 · schema 2 1건 · legacy 1건
민감값 독립 감사	후보 111개 공개 재사용 0건
내장 JSON	원본에서 파싱 가능한 7개 모두 공개본에서도 파싱, 구조 손실·bare marker 0건
의미 보존	앱 잠금 PIN 라벨 6/6 및 문장부호 placeholder 보존, 실제 인증값 6종 잔존 0건

기존 리뷰 반영

• 3차 QA는 미착수가 아니라 2026-08-25에 예비·핵심 8개 패스를 수행한 진행 중 상태로 바로잡았습니다. findings 50건과 not_covered 6건이 있어 전 기능 순회 완료로 보지는 않습니다.
• 관리자 승인·수신자 API 응답은 확인했지만 받은 기록함이 서버 목록 API 를 쓰지 않고 로컬 stub 으로 동작 — record-boxes 미연동 #607 때문에 승인 이후 FE 화면의 종단 간 검증은 남았다고 명시했습니다.
• assumptions.md, 증거 대장, 기기 baseline을 README·status에서 연결했습니다.
• 최신 develop의 빌드 셋업과 개발 규칙을 보존하면서, 기존 release keystore 인계·WIF 권한 경계·origin/main 검증·Play runbook 및 스크린샷 validate/generate/apply 절차를 전용 문서로 복원했습니다.

검증

• node --test .github/scripts/*.test.mjs — 295/295 통과
• node .github/scripts/qa-evidence.mjs verify <원본 34건> docs/qa/evidence — 34/34 통과
• 문서 분류·release·WIF·QA evidence 정책 테스트 — 18/18 통과
• scripts/repository-quality.sh — 통과
• scripts/test-qa-device-profile.sh + ShellCheck — 통과
• ./gradlew :build-logic:test — 통과
• Gitleaks(공개 증거 및 staged diff) — 탐지 0건
• Markdown 내부 링크와 git diff --check — 통과

QA Metadata

{
  "scope": "covered-by-ci",
  "exclusionReason": "Compose 화면과 앱 사용자 흐름은 바꾸지 않고 QA 증거 변환기, 문서, 공개용 기기 JSON 출력만 변경한다.",
  "androidTest": {
    "required": false,
    "reason": "Android 런타임 경계를 변경하지 않으며 동일 입력과 출력 경계를 Node 및 셸 회귀 테스트로 직접 검증한다. 모든 동일 저장소 PR에 적용되는 기기 회귀 실행은 별도로 유지한다."
  },
  "evidence": [
    {
      "kind": "test",
      "ref": ".github/scripts/qa-evidence.test.mjs",
      "assertion": "비식별화, 1:1 manifest, 덮어쓰기 방지, 내장 JSON 구조와 공개 corpus 34건을 검증한다.",
      "input": "민감정보와 JSON-like 문장을 포함한 fixture 및 공개 QA 증거 34건",
      "boundary": "원본 정규화부터 redaction, 파일 생성, manifest 해시, 공개 스키마 검증까지",
      "observation": "QA evidence 테스트와 전체 Node 테스트가 통과했다."
    },
    {
      "kind": "test",
      "ref": "scripts/test-qa-device-profile.sh",
      "assertion": "공개 JSON에는 serial과 AVD 이름이 없고 화면 재현 정보는 유지된다.",
      "input": "단일·복수 에뮬레이터와 화면 override를 모사한 adb fixture",
      "boundary": "기기 선택과 status --json 직렬화 출력",
      "observation": "셸 회귀 테스트와 ShellCheck가 통과했다."
    }
  ]
}

병합 뒤 정리

• 이관 전 기록되지 않은 과거 QA는 사후 복원하지 않습니다.
• 로컬 민감 원본은 이 PR이 실제로 병합된 것을 확인한 뒤 .codex/qa-evidence만 삭제합니다.

Closes #1242

[Sadturtleman]

지적

1. [blocking] 옮긴 docs/testing/screenshot.md 가 존재하지 않는 태스크를 그대로 들고 갑니다 — #1082 이 지금 고치는 바로 그 줄입니다.

   $ git show origin/docs/1088-qa-records:docs/testing/screenshot.md | sed -n '14,17p'
     ./gradlew :core:ui:updateScreenshotTest \
               :app:updateScreenshotTest \          ← app 모듈엔 screenshotTest 소스가 없다
               :feature:onboarding:presentation:updateScreenshotTest \
               :feature:afternote:presentation:updateScreenshotTest
   

   :feature:home:presentation · :feature:receiver:presentation 두 모듈이 빠져 있는 것도 그대로입니다 (screenshot.yml 은 5개 모듈을 검증). 문서를 옮기는 PR 이 낡은 내용을 새 파일로 이관하면, #1082 이 머지된 뒤 이 PR 이 들어갈 때 고쳐진 README 는 삭제되고 낡은 사본만 남습니다.

   두 PR 이 같은 내용을 서로 다른 파일에서 다루니, 이 PR 에서 옮긴 문서 쪽에 #1082 의 수정(죽은 :app 2건 제거 + 누락 2모듈 추가 + 「정본은 screenshot.yml」 명시)을 반영해 주세요. 저는 #1082 를 방금 승인했습니다.

   제가 오늘 이 문서대로 실행해 본 결과이기도 합니다 — :feature:receiver:presentation · :feature:home:presentation baseline 을 갱신해야 했는데 문서의 명령 목록에는 둘 다 없었습니다.

2. [normal] docs/qa/assumptions.md(#897) 와의 관계가 문서 지도에 없습니다.

   docs/qa/ 에 status.md·round-1.md·round-2.md 가 들어오면서 기존 assumptions.md 와 같은 디렉터리를 쓰게 됩니다. 파일은 그대로 두신 걸 확인했지만, README 의 문서 지도와 status.md 어디에도 언급이 없어 «QA 기록은 여기» 를 읽은 사람이 그 파일을 못 찾습니다. 한 줄 링크면 됩니다 — 그 파일은 «명세가 침묵해 추측으로 진행한 항목» 을 모으는 곳이라 QA 현황과 같이 읽혀야 합니다.

나머지는 이견 없습니다

• 노션 원본을 지우지 않고 링크만 남긴 판단, README 를 304→111줄로 줄이고 절차 문서를 docs/ 로 뺀 구성 모두 좋습니다.
• 1·2차 회차를 결함 건수까지 옮겨 «어디까지 했는지» 를 셀 수 있게 만든 것이 이 PR 의 핵심 값이라 봅니다.

[github-actions]

이 PR 은 develop 와 충돌 상태이고, 그래서 CI 가 돌지 않는다.

pull_request 워크플로는 base 와 head 를 합친 가상 머지 커밋(refs/pull/<N>/merge)에서
실행되는데, 충돌이면 GitHub 이 그 ref 를 만들지 못해 워크플로가 트리거조차 되지 않는다.
체크가 실패하는 것이 아니라 아예 생성되지 않으므로, 새 커밋을 올려도 no checks reported
인 채 최신 커밋이 한 번도 검증되지 않은 상태로 남는다 (#1028).

develop 를 병합해 충돌을 풀면 검증이 다시 돈다. 해소되면 이 라벨은 자동으로 떨어진다.

[OnlyReadOneBook]

두 건 다 이미 닫혀 있습니다 — 근거만 적습니다

1. [blocking] docs/testing/screenshot.md 의 죽은 태스크

리뷰 시점(efe5c625c)에는 지적하신 그대로였습니다.

$ git show efe5c625c:docs/testing/screenshot.md
  ./gradlew :core:ui:updateScreenshotTest \
            :app:updateScreenshotTest \
            :feature:onboarding:presentation:updateScreenshotTest \
            :feature:afternote:presentation:updateScreenshotTest

그 뒤 #1082 가 develop 에 들어왔고, 이 브랜치가 develop 을 물리면서(76677f912) 이관처 문서에 그 수정이 반영됐습니다. 지금은 이렇습니다.

  ./gradlew :core:ui:updateScreenshotTest \
            :feature:home:presentation:updateScreenshotTest \
            :feature:receiver:presentation:updateScreenshotTest \
            :feature:onboarding:presentation:updateScreenshotTest \
            :feature:afternote:presentation:updateScreenshotTest

죽은 :app 2건이 빠지고 home · receiver 가 들어왔습니다. 「정본은 screenshot.yml」 도 함께 있습니다.

대상 모듈 목록의 정본은 .github/workflows/screenshot.yml 이다. 모듈을 추가·이전했다면 워크플로와 이 문서를 함께 갱신한다.

말씀하신 「고쳐진 README 는 삭제되고 낡은 사본만 남는」 순서 문제는 이걸로 사라졌습니다.

2. [normal] assumptions.md 와의 관계

이건 리뷰 시점에도 이미 있었습니다. 보신 브랜치가 origin/docs/1088-qa-records 라 하셨는데 이 PR 의 head 는 docs/qa-status-and-readme 라, 그쪽에는 아직 없던 게 아닐까 싶습니다.

efe5c625c 기준 README 문서 지도:

| [docs/qa/assumptions.md](docs/qa/assumptions.md) | 시안 · 명세에 없어 판단으로 정한 것 |

docs/qa/status.md 는 두 곳입니다 — 머리말과 문서 지도 표.

> [assumptions.md](assumptions.md), 런타임 증거는 `.codex/qa-evidence/emulator/<sha>.json`(로컬 전용).
| 시안·명세에 없어 판단으로 정한 것 | `docs/qa/assumptions.md` |

혹시 다른 자리를 뜻하신 것이면 알려주세요.

[koongmai]

인라인 지적 사항을 확인해 주세요.

[OnlyReadOneBook]

/review-response

두 변경 요청을 최신 develop 기준으로 반영했습니다.

• docs/testing/screenshot.md에서 존재하지 않는 :app 태스크를 제거하고 core:ui·home·receiver·onboarding·afternote·mindrecord 6개 모듈을 모두 넣었습니다. 검증·생성·적용 3개 workflow가 대상 목록의 정본임도 명시했습니다.
• 최신 README의 빌드 셋업과 개발 규칙은 유지하고, 배포·스크린샷 절차만 docs/release/distribution.md와 docs/testing/screenshot.md로 분리했습니다. 배포 문서의 자동 산출 범위는 Closes·Fixes·Resolves 이슈로 한정하고, 공용 앱 설정 secret과 보호 Environment 자격의 보관 범위도 실제 workflow에 맞췄습니다.
• README.md와 docs/qa/status.md 문서 지도에서 docs/qa/assumptions.md를 연결했습니다.
• 3차 QA는 2026-08-25 착수·8개 패스·findings 50건·not_covered 6건으로 문서 전체 표현을 통일했습니다. 받은 기록함이 서버 목록 API 를 쓰지 않고 로컬 stub 으로 동작 — record-boxes 미연동 #607 때문에 승인 이후 FE 화면 E2E가 미검증이라는 경계도 명시했습니다.
• 로컬 런타임 기록 34개는 민감정보를 제거한 공개 스키마와 manifest로 34/34 이관했고, 원본 SHA·결과·시나리오·미검증 범위를 대조 검증했습니다.

현재 HEAD 31201803d에서 전체 Node 테스트 296/296, 증거 대조 34/34와 저장소 품질 검사를 다시 통과했습니다. 재검토 부탁드립니다.

@Sadturtleman의 더 늦은 승인(review #5040325166)을 최종 판정으로 사용합니다. 저장소 정책은 PR 전체에서 가장 최근의 APPROVED/CHANGES_REQUESTED 판정을 우선합니다.