Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 31 additions & 4 deletions data/outstanding-issues-snapshot.json
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
{
"version": "outstanding-issues-snapshot-v2",
"ledger_revision": {
"committed_at": "2026-10-16"
"committed_at": "2026-10-17"
},
"counts": {
"open": 108,
"open": 111,
"p1": 15,
"p2": 58,
"p3": 35,
"p2": 59,
"p3": 37,
"queued": 6,
"pending": 0,
"resolved": 796
Expand Down Expand Up @@ -1052,6 +1052,33 @@
"detail": "The registrar_training_milestones table and RLS policies exist, but no application code reads or writes it, the Training page still hard-codes not-recorded, and its milestone reader queries a different table. Implement owner-scoped storage and sync and replace the hard-coded state before closing.",
"source": "PR #3570 review; src/app/(search-app)/cme/training/page.tsx; src/lib/cme/training-repository.ts",
"added": "2026-10-16"
},
{
"id": "#8Y9169",
"priority": "P3",
"type": "task",
"summary": "Remove the remaining PR 3538 leftovers after 13 Nov 2026 (split from #08216B, which closed only the calculator fixes)",
"detail": "#08216B was archived on 17 Oct with an outcome covering only the clinical-math calculators. Still open: src/app/api/proxy/route.ts (nothing calls it), scripts/recalculate-answer-support-labels.ts (no production caller; reassess it as a deletion candidate) and its test (covered by the normal test suite; do not treat it as unused), and the developer tool pages under src/app/mockups/development (unreachable duplicates of /admin/developer-hub) - remove after the 30-day dead-code window (13 Nov 2026) via npm run check:dead-code-candidate. The safe-number editor and clinical note editor are unwired: wire or remove each; the note editor needs a privacy review before any use. The 16 Oct clinical-math calculator changes (IBW guidance, lithium boundaries, dose rounding) need clinician verification before any clinical use.",
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
"source": "PR #3609 review (gitar-bot, chatgpt-codex-connector), 17 Oct 2026",
"added": "2026-10-17"
},
{
"id": "#WWTRM5",
"priority": "P2",
"type": "issue",
"summary": "Doc/instruction drift items left open when #WMGPSG was archived: ledger arrow, cloud-profile worktree rule, Caring Contacts decision, renewals wording",
"detail": "#WMGPSG was archived on 17 Oct with an outcome covering items 3 and 6 and the encoding repair in item 1 only. Still open: (1) docs/outstanding-issues.md:15 corrupted arrow (canonical ledger; fix only through reconciliation). (2) .claude/cloud-profile/skills/prompt/references/coding-workflow.md:21 pre-hardening worktree rule (owner decision: owned versus vendored). (4) docs/decisions/2026-09-26-remove-ward-flow-and-caring-contacts.md says Caring Contacts was removed but src/lib/caring-contacts/clock.ts is live with MHA, First Nations and On Call importers; add a dated superseding decision, do not delete the file. (5) Admin renewals page test dropped its checked-wording assertion while admin-renewed-sheet still forbids the word; record the decision.",
"source": "PR #3609 review (gitar-bot, chatgpt-codex-connector), 17 Oct 2026",
"added": "2026-10-17"
},
{
"id": "#C4Q3T7",
"priority": "P3",
"type": "task",
"summary": "Possible dead code: older calculator mockups (audit M5-2), split from #1WAA0Z",
"detail": "#1WAA0Z was archived on 17 Oct after confirming only the On Call Education, Compliance and Logistics screens are live (audit L4D-05). The older calculator mockups under src/app/mockups (calculators-*) from audit M5-2 were not checked. Delete nothing without npm run check:dead-code-candidate (see docs/agents/dead-code-deletion.md).",
"source": "PR #3609 review (gitar-bot), 17 Oct 2026",
"added": "2026-10-17"
}
],
"pending": []
Expand Down
5 changes: 5 additions & 0 deletions data/repo-awareness-snapshot.json
Original file line number Diff line number Diff line change
Expand Up @@ -3969,6 +3969,11 @@
"section": "decisions",
"catalogued": true
},
{
"path": "docs/decisions/2026-10-17-wfydv6-stays-open-until-sign-off.md",
"section": "decisions",
"catalogued": true
},
{
"path": "docs/agents/native-startup/external-skill-precedence.md",
"section": "agents",
Expand Down
14 changes: 14 additions & 0 deletions docs/decisions/2026-10-17-wfydv6-stays-open-until-sign-off.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Decision: The WA MHA Form 6A/6B statutory-text issue stays open until legal and clinical sign-off

- **Status:** decided 2026-10-17
- **Source:** `docs/outstanding-issues-inbox/a5c25ca4-4fa3-421f-a944-35621169e55e.json`, find "Owner decision 17 Oct 2026 21:37 AWST"

PR #3609 archived ledger issue #WFYDV6 (the unwired `src/components/forms/wa-mha-form-print.tsx` with
unverified statutory text) without the legal and clinical sign-off the issue required. The owner decided on
17 Oct 2026 at 21:37 AWST that the issue stays open and the form stays unwired until that sign-off is
recorded. The ledger tooling has no reopen action, so the issue is tracked again as a new open P1 row
queued through the ledger inbox, and the archived #WFYDV6 row's outcome records the reopen. A reconciliation
must not archive that row again without the recorded sign-off, or the deletion of the file after the
30-day dead-code window (13 Nov 2026).

If this summary and the source ever differ, the source wins.
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
"version": 2,
"id": "15a11955-9ee3-4c86-ad8f-faa0af2a342c",
"createdOn": "2026-10-17",
"action": "add",
"payload": {
"pri": "P3",
"type": "task",
"summary": "Ledger correction: PR #3570 prlanded record names the wrong branch",
"detail": "docs/branch-review-records/a7be2760026da4bc040fa90bf847382bdcded2a8a7510ceaf7be4ac49c042211.record.md (added by PR #3609) records branch claude/reconcile-inbox-20261016 for PR #3570, but gh shows #3570's head branch was claude/reconcile-ledger-20261016 (merge fa526775c6, head d8eefbb91e). The SHAs and merge state are correct. Correct the branch name through the sanctioned branch-review-ledger tooling only; never hand-edit the record.",
"source": "PR #3609 sweep proof verification, 17 Oct 2026",
"issueUlid": "01M551BST62X90V6VK3DEP4C7S"
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
"version": 2,
"id": "177b7474-70f8-47a9-8299-8fdaf811c419",
"createdOn": "2026-10-17",
"action": "add",
"payload": {
"pri": "P3",
"type": "task",
"summary": "Ledger tooling: add a reopen action that moves an archived issue back to the open table with its original ID and ULID",
"detail": "There is no sanctioned reopen in scripts/ledger-inbox.mjs (actions: add, done, update, amend-outcome, queue, cancel). Reopening #WFYDV6 by owner decision on 17 Oct 2026 therefore created a new open P1 row and amended the archived row, splitting the issue's history across two identities. Add a reopen request kind that the reconciler applies by moving the archive row back to the open table with its original ID and ULID, then use it to fold the new P1 row back into #WFYDV6.",
"source": "PR #3616 review (chatgpt-codex-connector), 17 Oct 2026",
"issueUlid": "01M552JNXFGYR7Q31XTTJSNQZD"
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
"version": 2,
"id": "a5c25ca4-4fa3-421f-a944-35621169e55e",
"createdOn": "2026-10-17",
"action": "add",
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
"payload": {
"pri": "P1",
"type": "issue",
"summary": "Reopened #WFYDV6: unwired WA MHA Form 6A/6B print component carries unverified statutory text; keep unwired until legal and clinical sign-off",
"detail": "Owner decision 17 Oct 2026 21:37 AWST: #WFYDV6 must stay open; PR #3609 archived it without the required sign-off. src/components/forms/wa-mha-form-print.tsx (PR #3538) was reworded on 16 Oct (Form 6B now a report to the Chief Psychiatrist; Non-Statutory Study Layout header), but section lists, the 7-day pre-expiry examination and copy recipients remain unverified and no legal or clinical sign-off is recorded. Nothing imports or renders it; it must stay unwired. Delete it once the 30-day dead-code window passes (after 13 Nov 2026) unless legal and clinical sign-off is obtained first.",
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
"source": "PR 3538 review, 15 Oct 2026; owner decision on PR #3609, 17 Oct 2026",
"issueUlid": "01M551BSHWZ12CBC9F0K21CPEN"
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"version": 2,
"id": "c455b372-522a-4894-b87c-4fd7827b61bb",
"createdOn": "2026-10-17",
"action": "update",
"payload": {
"id": "#8Y9169",
"detail": "#08216B was archived on 17 Oct with an outcome covering only the clinical-math calculators. Still open: src/app/api/proxy/route.ts (nothing calls it), scripts/recalculate-answer-support-labels.ts (no production caller; reassess it as a deletion candidate) and its test (covered by the normal test suite; do not treat it as unused), and the developer tool pages under src/app/mockups/development (unreachable duplicates of /admin/developer-hub) - remove after the 30-day dead-code window (13 Nov 2026) via npm run check:dead-code-candidate. The safe-number editor and clinical note editor are unwired: wire or remove each; the note editor needs a privacy review before any use (the attendance roster and QR check-in helper are wired through organise-sheets.tsx and teaching-code-sheet.tsx). The 16 Oct clinical-math calculator changes (IBW guidance, lithium boundaries, dose rounding) need clinician verification before any clinical use.",
"baseRowFingerprint": "556db211f1a3e8c8cad29cb6d400fe4b85d2f1058dc98d465eae5d8be84370ce",
"targetUlid": "01M54YEVKA8Y91694B291WKTQS"
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
"version": 2,
"id": "f8628e25-fe07-442f-9221-119b40e34213",
"createdOn": "2026-10-17",
"action": "amend-outcome",
"payload": {
"id": "#WFYDV6",
"outcome": "Reopened 17 Oct 2026 by owner decision (21:37 AWST): archived by PR #3609 without the required legal and clinical sign-off; tracked again as an open P1 row. The form stays unwired until sign-off.",
"baseRowFingerprint": "7557eae4330cfdd49d1530a684e58d5668cd895b9f73047e23be2dc8a3fb7531",
"allowArchived": true,
"targetTable": "archive",
"targetUlid": "01M4ZM793BWFYDV65NNBHBP53K"
}
}
3 changes: 3 additions & 0 deletions docs/outstanding-issues.md
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,9 @@ removed after current-main verification; it is not missing recommended work.
| #9E87MG <!-- issue-ulid:01M51JV9M29E87MGRFV0QFE7BA --> | P2 | task | Clinical KB: provision PROXY_AUTH dedicated secret on the PsychSift Railway service (follow-up to #789SD7) | Route validation for PROXY_AUTH exists and fails closed with 503, but docs/supabase-remediation-closeout-notes.md still records PROXY_AUTH as not on Railway by name. The owner must authorise and confirm the variable on the correct PsychSift Railway service in a quiet change window; no agent sets production secrets. | PR #3570 review; docs/supabase-remediation-closeout-notes.md; src/lib/supabase/proxy-auth-crypto.ts | 2026-10-16 |
| #AD9JC6 <!-- issue-ulid:01M51JV9CNAD9JC6FPMWA8RZZ0 --> | P2 | task | Teaching Assessments: wire durable authenticated assessment storage and supervisor bookings (follow-up to #4KQGXZ) | The schema, RLS policies and migration for registrar assessments exist, but the dual-sign-off route still uses in-process memory and returns 503 in production, and no supervisor-booking endpoints or availability storage exist. Wire durable authenticated storage into the route and add the booking endpoints before closing. | PR #3570 review; src/app/api/teaching/assessments/dual-sign-off/route.ts; supabase/migrations/20261013160000_teaching_assessments_and_milestones.sql | 2026-10-16 |
| #XNAHY2 <!-- issue-ulid:01M51JV9EEXNAHY22XVMGMAD14 --> | P2 | task | CPD: connect the Training page to registrar training milestone storage (follow-up to #T983N9) | The registrar_training_milestones table and RLS policies exist, but no application code reads or writes it, the Training page still hard-codes not-recorded, and its milestone reader queries a different table. Implement owner-scoped storage and sync and replace the hard-coded state before closing. | PR #3570 review; src/app/(search-app)/cme/training/page.tsx; src/lib/cme/training-repository.ts | 2026-10-16 |
| #8Y9169 <!-- issue-ulid:01M54YEVKA8Y91694B291WKTQS --> | P3 | task | Remove the remaining PR 3538 leftovers after 13 Nov 2026 (split from #08216B, which closed only the calculator fixes) | #08216B was archived on 17 Oct with an outcome covering only the clinical-math calculators. Still open: src/app/api/proxy/route.ts (nothing calls it), scripts/recalculate-answer-support-labels.ts (no production caller; reassess it as a deletion candidate) and its test (covered by the normal test suite; do not treat it as unused), and the developer tool pages under src/app/mockups/development (unreachable duplicates of /admin/developer-hub) - remove after the 30-day dead-code window (13 Nov 2026) via npm run check:dead-code-candidate. The safe-number editor and clinical note editor are unwired: wire or remove each; the note editor needs a privacy review before any use. The 16 Oct clinical-math calculator changes (IBW guidance, lithium boundaries, dose rounding) need clinician verification before any clinical use. | PR #3609 review (gitar-bot, chatgpt-codex-connector), 17 Oct 2026 | 2026-10-17 |
| #WWTRM5 <!-- issue-ulid:01M54YEVQ8WWTRM5RPCGA0FEVP --> | P2 | issue | Doc/instruction drift items left open when #WMGPSG was archived: ledger arrow, cloud-profile worktree rule, Caring Contacts decision, renewals wording | #WMGPSG was archived on 17 Oct with an outcome covering items 3 and 6 and the encoding repair in item 1 only. Still open: (1) docs/outstanding-issues.md:15 corrupted arrow (canonical ledger; fix only through reconciliation). (2) .claude/cloud-profile/skills/prompt/references/coding-workflow.md:21 pre-hardening worktree rule (owner decision: owned versus vendored). (4) docs/decisions/2026-09-26-remove-ward-flow-and-caring-contacts.md says Caring Contacts was removed but src/lib/caring-contacts/clock.ts is live with MHA, First Nations and On Call importers; add a dated superseding decision, do not delete the file. (5) Admin renewals page test dropped its checked-wording assertion while admin-renewed-sheet still forbids the word; record the decision. | PR #3609 review (gitar-bot, chatgpt-codex-connector), 17 Oct 2026 | 2026-10-17 |
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
Comment thread
BigSimmo marked this conversation as resolved.
Resolved
| #C4Q3T7 <!-- issue-ulid:01M54YEVVBC4Q3T76N15KMZVWV --> | P3 | task | Possible dead code: older calculator mockups (audit M5-2), split from #1WAA0Z | #1WAA0Z was archived on 17 Oct after confirming only the On Call Education, Compliance and Logistics screens are live (audit L4D-05). The older calculator mockups under src/app/mockups (calculators-*) from audit M5-2 were not checked. Delete nothing without npm run check:dead-code-candidate (see docs/agents/dead-code-deletion.md). | PR #3609 review (gitar-bot), 17 Oct 2026 | 2026-10-17 |

## Resolved / archive

Expand Down
Loading