Fix IO FAT selected-IED context, grid layout, and evidence safety - #126
Fix IO FAT selected-IED context, grid layout, and evidence safety#126masarray wants to merge 32 commits into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3f27b25254
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| var preparation = await engineeringWindow.PrepareIoTestIedForFatAsync( | ||
| Project, | ||
| selectedIed, | ||
| progress); |
There was a problem hiding this comment.
Exclude completed rows before connection preparation
When a mixed IED contains a completed signal that is no longer available in the live model, this preparation call still includes it because completed points remain enabled until immediately before Session.Start. PrepareIoTestIedForFatAsync snapshots every enabled point (MainWindow.IoTesting.AutoConnect.cs:24-26) and fails unless all requested points become live (MainWindow.IoTesting.AutoConnect.cs:203-213), so the method returns before pending, otherwise-valid rows can resume. Apply the protected-row scope during preflight and preparation as well as during Session.Start.
Useful? React with 👍 / 👎.
Ringkasan
Perbaikan workspace IO List FAT berdasarkan audit layout multi-IED:
✔ PASSpada card IED ketika seluruh test point aktif sudah PASS.COMPLETEDyang ambigu dari toolbar workspace.Safety behavior
TestEnabledyang sementara diubah selalu dikembalikan melaluifinally.Validation
Regression contract tests ditambahkan untuk selected-IED context, lokasi preview toggle, balanced grid widths, centered relay headers, PASS card badge, dan evidence protection flow.