From 9a28e46d6ee1f345f66abb2999a61cc7a017acda Mon Sep 17 00:00:00 2001 From: Joseph Doherty Date: Tue, 16 Jun 2026 18:41:30 -0400 Subject: [PATCH] chore(phase7): mark tasks complete (shipped 1122dcdc) --- ...phase-7-client-alarm-buttons.md.tasks.json | 27 +++++++++++-------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/docs/plans/2026-06-16-stillpending-phase-7-client-alarm-buttons.md.tasks.json b/docs/plans/2026-06-16-stillpending-phase-7-client-alarm-buttons.md.tasks.json index 21313095..b1f411f2 100644 --- a/docs/plans/2026-06-16-stillpending-phase-7-client-alarm-buttons.md.tasks.json +++ b/docs/plans/2026-06-16-stillpending-phase-7-client-alarm-buttons.md.tasks.json @@ -1,19 +1,24 @@ { "planPath": "docs/plans/2026-06-16-stillpending-phase-7-client-alarm-buttons.md", - "branch": "feat/stillpending-phase-7-client-alarm-buttons", + "branch": "feat/stillpending-phase-7-client-alarm-buttons (merged to master 1122dcdc, deleted)", "baseMaster": "ad3ec9d9", "designCommit": "573728b5", - "executionState": "PLANNED — not started", + "executionState": "COMPLETE — shipped + pushed to master 1122dcdc. Final integration review = SHIP. Build clean; Client.UI.Tests 122/122, Client.CLI.Tests 93/93. Live: CLI connected to the rig OPC UA server (shared service proven); GUI click-through operator-gated (Avalonia not agent-drivable).", "tasks": [ - {"id": 468, "subject": "P7 Task 1: Extend FakeOpcUaClientService (Shelve/Confirm tracking)", "status": "pending"}, - {"id": 469, "subject": "P7 Task 2: AlarmsViewModel Shelve/Confirm + CanShelve/CanConfirm + tests", "status": "pending", "blockedBy": [468]}, - {"id": 470, "subject": "P7 Task 3: ShelveAlarmWindow dialog (kind + duration)", "status": "pending", "blockedBy": [469]}, - {"id": 471, "subject": "P7 Task 4: ConfirmAlarmWindow dialog (comment)", "status": "pending", "blockedBy": [469]}, - {"id": 472, "subject": "P7 Task 5: AlarmsView context-menu (3 items + per-item enablement)", "status": "pending", "blockedBy": [470, 471]}, - {"id": 473, "subject": "P7 Task 6: Docs + bookkeeping", "status": "pending", "blockedBy": [472]}, - {"id": 474, "subject": "P7 Task 7: Full build + test + final integration review", "status": "pending", "blockedBy": [468, 469, 470, 471, 472, 473]}, - {"id": 475, "subject": "P7 Task 8: Live launch/connect smoke (operator-gated click-through)", "status": "pending", "blockedBy": [474]} + {"id": 468, "subject": "P7 Task 1: Extend FakeOpcUaClientService (Shelve/Confirm tracking)", "status": "completed", "commit": "8980adce"}, + {"id": 469, "subject": "P7 Task 2: AlarmsViewModel Shelve/Confirm + CanShelve/CanConfirm + tests", "status": "completed", "commit": "4f55d894", "blockedBy": [468]}, + {"id": 470, "subject": "P7 Task 3: ShelveAlarmWindow dialog (kind + duration)", "status": "completed", "commit": "60da9224", "blockedBy": [469]}, + {"id": 471, "subject": "P7 Task 4: ConfirmAlarmWindow dialog (comment)", "status": "completed", "commit": "a774bb08", "blockedBy": [469]}, + {"id": 472, "subject": "P7 Task 5: AlarmsView context-menu (3 items + per-item enablement)", "status": "completed", "commit": "6a61f35a", "blockedBy": [470, 471]}, + {"id": 473, "subject": "P7 Task 6: Docs + bookkeeping", "status": "completed", "commit": "33687999", "blockedBy": [472]}, + {"id": 474, "subject": "P7 Task 7: Full build + test + final integration review", "status": "completed", "commit": "4675bd61", "blockedBy": [468, 469, 470, 471, 472, 473]}, + {"id": 475, "subject": "P7 Task 8: Live launch/connect smoke (operator-gated click-through)", "status": "completed", "blockedBy": [474]} ], - "notes": "UI surface = extend context menu; Confirm gating = simple (AckedState, no ConfirmedState tracking). No IOpcUaClientService/CLI/Commons/EF change. Avalonia GUI not agent-drivable → live click-through operator-gated.", + "reviewFixCommits": ["fd986c3b (Unshelve button relabel)", "4675bd61 (3 review test-gaps)", "1122dcdc (Shelve success-message polish)"], + "finalIntegrationReview": "SHIP (no Critical/Important). Follow-ups (non-blocking): #1 Shelve success copy FIXED; #2 fake lacks LastAcknowledgeCall (pre-existing); #3 no CanAcknowledge unit tests (pre-existing).", + "decisions": { + "uiSurface": "extend the existing runtime context menu (not inline buttons / toolbar)", + "confirmGating": "simple — CanConfirm = AckedState + ids; no ConfirmedState event-parsing" + }, "lastUpdated": "2026-06-16" }