docs: record completed PWA setup review in branch review ledger

[BigSimmo]

Summary

• Append one row to docs/branch-review-ledger.md recording the completed PWA setup review of main at e5caaa46cad5fb9a937f1dc43312723799b98abb (the ledger append permitted during a pure review by docs/codex-review-protocol.md).
• Review outcome: the PWA surface is byte-identical to the already-reviewed codex/pwa-privacy-safe-20260717 merge (35fa8c9); privacy contract confirmed sound; no P0–P2 findings. Recorded P3 items: no committed retirement/kill-switch worker despite docs/pwa.md rule 6, and offline.html is install-time precached with revalidation dependent on a manual CACHE_VERSION bump (a Vitest guard tying the file hash to the version is suggested). Minor/P4 observations also recorded.

Verification

This PR is a docs-only, single-line ledger append — no source, config, or test files changed.

Verification not run: verify:pr-local, verify:ui, and verify:release are not applicable to this docs-only, single-line ledger append.

• npm run verify:pr-local — not run (docs-only ledger append)
• npm run verify:ui — not applicable (no UI/browser behavior changed)
• npm run verify:release — not applicable

Checks that were run:

• npx prettier --check docs/branch-review-ledger.md — passes.
• As review evidence (not diff verification): focused Vitest via npm run test -- tests/pwa-service-worker.test.ts tests/pwa-manifest.test.ts tests/pwa-lifecycle.dom.test.tsx — 45/45 passed (38 node + 7 jsdom).

Clinical Governance Preflight

Not applicable — this change touches no ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical output behavior; it appends a review record to a docs file. No provider-backed checks were run.

Notes

• Full findings and check details are in the appended ledger row of docs/branch-review-ledger.md.
• Merge commit 17c1257d resolved the ledger-tail conflict with main, keeping all rows from both sides (verified: main's 583 lines + this branch's 1 review row = 584).

🤖 Generated with Claude Code

https://claude.ai/code/session_01EXsJcLrbZUXwnBeG91cVo9

---

Generated by Claude Code

[supabase]

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[coderabbitai]

📝 Walkthrough

Walkthrough

The branch review ledger gains a 2026-07-17 entry documenting a completed PWA setup review, its privacy contract, P3 findings, test commands, CI expectations, and checks not run.

Changes

PWA review ledger

Layer / File(s)	Summary
Append PWA setup review record docs/branch-review-ledger.md	Adds the completed review entry with confirmed PWA behavior, no P0–P2 findings, two P3 items, test commands, CI/playwright expectations, and provider-backed or local browser checks marked as not run.

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

Possibly related PRs

• BigSimmo/Database#702: Directly related PWA review documentation in the branch review ledger.
• BigSimmo/Database#719: Also appends a review record to docs/branch-review-ledger.md.

Suggested reviewers: claude

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check	✅ Passed	The title clearly matches the docs-only change: recording a completed PWA review in the branch review ledger.
Description check	✅ Passed	The description is complete enough and follows the template structure, with summary, verification, preflight, and notes provided.

✨ 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 claude/clinical-kb-pwa-review-asi3wb

---

_{Comment @coderabbitai help to get the list of available commands.}

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[BigSimmo]

@copilot resolve the merge conflicts on this branch.

[Copilot]

@copilot resolve the merge conflicts on this branch.

Resolved. The only conflict was in docs/branch-review-ledger.md — both this branch and main had appended rows to the same tail of the file. Fixed by keeping all rows from both sides (the PWA review row from this branch followed by the rows added to main). Merge commit: 17c1257d.