Skip to content

Fix atomic one-click control confirmation - #22

Merged
masarray merged 18 commits into
mainfrom
fix/atomic-control-confirmation
Jul 14, 2026
Merged

Fix atomic one-click control confirmation#22
masarray merged 18 commits into
mainfrom
fix/atomic-control-confirmation

Conversation

@masarray

@masarray masarray commented Jul 14, 2026

Copy link
Copy Markdown
Owner

P0 control fix

  • split background control inspection from operator command ownership
  • claim a pending confirmation atomically before hiding the confirmation UI
  • guard each control object with a sequence-scoped single-command owner
  • keep Confirm enabled while a background ctlModel/status inspection is finishing
  • prevent late successful or failed inspection results from overwriting an active command result
  • replace the two virtualized Confirm Open / Confirm Close buttons with one stable Confirm button
  • log every stage and confirmation transition: received, staged, accepted/rejected, dispatch ownership, and MMS submission
  • tie stable feedback cleanup to command ownership instead of generic background-busy state
  • release ownership only with the matching command sequence

Expected diagnostic sequence

Control confirmation stage click receivedControl confirmation stagedConfirm click receivedConfirm acceptedDispatch ownership acquiredMMS command submitted

Rejected clicks now include an explicit reason instead of returning silently. Background ctlModel/status inspection no longer disables Confirm and cannot overwrite the command result after ownership is acquired.

Release

Application, assembly, file, workflow and portable package versions are aligned at 1.6.7.

Validation

GitHub Actions Build ArIED 61850 #135 completed successfully:

  • source, website and license-boundary verification
  • ARIEC61850 API dependency verification
  • .NET restore
  • Release build
  • portable Windows x64 publish
  • portable artifact upload

Final artifact: ArIED61850-win-x64

SHA-256: 37b7077dfa96f4131933804f0b0a22d4e04e191dbff2d18deb88a3022de2b21c

The final PR diff contains nine production files only; all temporary patch and workflow files were removed.

@masarray
masarray marked this pull request as ready for review July 14, 2026 07:17
@masarray
masarray merged commit 3229851 into main Jul 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant