From c4ec904ad8fe5198ddeeb72cf1cb149d205c1b3f Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 18 Aug 2026 19:19:32 +0000 Subject: [PATCH] docs(issues): re-queue the Therapy sign-off tooling task MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR #2145 was closed in favour of #2150, which supersedes the Therapy exposure change but does not carry this follow-up, so the request would have been lost with the closed branch. Therapy now ships with its review state disclosed rather than hidden, so sign-off is the remaining clinical work — and nothing currently supports it. `reviewStatus` is a bare string: a record can be flipped to `reviewed` with all seven `reviewChecklist` booleans still false and nothing detects it. There is no `reviewedBy` or `reviewedAt` anywhere in the 44 record fields, so a sign-off cannot record who signed or when. And 205 records x 7 checks is 1435 clinical judgements with no workflow. Completing sign-off is also what retires the new catalogue notice, which reads `THERAPY_CATALOGUE_SUMMARY.needsReviewCount` and disappears at zero. Carries the stop rule that an assistant must never tick the five clinical checks; `proofread` and `australianEnglishChecked` are non-clinical and may be done with attribution. Co-Authored-By: Claude Opus 5 --- .../aaca6636-f820-4ef5-881c-752440dda09d.json | 13 +++++++++++++ 1 file changed, 13 insertions(+) create mode 100644 docs/outstanding-issues-inbox/aaca6636-f820-4ef5-881c-752440dda09d.json diff --git a/docs/outstanding-issues-inbox/aaca6636-f820-4ef5-881c-752440dda09d.json b/docs/outstanding-issues-inbox/aaca6636-f820-4ef5-881c-752440dda09d.json new file mode 100644 index 0000000000..e5f784d905 --- /dev/null +++ b/docs/outstanding-issues-inbox/aaca6636-f820-4ef5-881c-752440dda09d.json @@ -0,0 +1,13 @@ +{ + "version": 2, + "id": "aaca6636-f820-4ef5-881c-752440dda09d", + "createdOn": "2026-08-18", + "action": "add", + "payload": { + "pri": "P2", + "type": "task", + "summary": "Therapy sign-off has no tooling: nothing stops reviewStatus reviewed being set with an empty checklist, and there is no reviewer attribution", + "detail": "Re-queued 2026-08-18 after PR #2145 was closed in favour of #2150, which supersedes the exposure change but does not carry this follow-up. Therapy now ships in production with its review state disclosed rather than hidden, so sign-off is the remaining clinical work. Three gaps. (1) reviewStatus is a bare string in src/data/therapies-source.json; a record can be flipped to reviewed with all seven reviewChecklist booleans still false and nothing detects it. Needs a script that refuses the flip unless the checklist is complete, plus a contract test pinning reviewed implies full checklist. (2) No attribution: none of the 44 record fields carries reviewedBy or reviewedAt, so a sign-off cannot record who signed or when - the same defect #318 flags against the medication interaction lexicon. (3) No review workflow: 205 records x 7 checks is 1435 clinical judgements by hand; a CLI that walks records, shows the fields each check covers, and writes the decision with attribution would make it tractable. State at re-queue: 205 records, all reviewStatus needs_review, all seven checklist booleans false, reviewCompleteness 57-71 with zero records complete. The catalogue notice #2150 adds reads from THERAPY_CATALOGUE_SUMMARY.needsReviewCount and disappears when that reaches zero, so completing sign-off is what retires it. Stop rule: an assistant must never tick clinicalAccuracyReviewed, sourceChecked, evidenceAppraised, safetyCautionsChecked or patientExplanationChecked - those are qualified-clinician attestations. proofread and australianEnglishChecked are non-clinical and may be done with attribution.", + "issueUlid": "01M0B4YZFPSBKXZ7BGFC4YJ52G" + } +}