Skip to content

Use exact read-only RCB evidence in selected CID export - #79

Merged
masarray merged 1 commit into
mainfrom
fix/file-transfer-rcb-option-fidelity
Jul 20, 2026
Merged

Use exact read-only RCB evidence in selected CID export#79
masarray merged 1 commit into
mainfrom
fix/file-transfer-rcb-option-fidelity

Conversation

@masarray

Copy link
Copy Markdown
Owner

Summary

Completes the application side of the field fixes in masarray/ARIEC61850#36.

For live selected-RCB CID export, ARSAS now merges the exact configuration values collected by the existing read-only availability audit into the cloned export model before filtering to one RCB. This preserves the IED's real TrgOps, OptFlds, BufTm, IntgPd, RptID and ConfRev evidence instead of relying on incomplete broad-discovery text.

The engine change also fixes the real FileDirectory layout that caused FRA00019.cfg and FRA00019.dat to be collapsed into one CFG / Missing DAT / size unknown row. No additional file-transfer UI workaround is required; ARSAS consumes the corrected engine from main.

Expected field result

  • the fault-record catalog shows the CFG and DAT companion files independently, with the returned paths and sizes
  • Download selected opens and streams the actual remote paths
  • a GI-only RCB remains GI-only in the generated CID
  • a buffer-overflow-only OptFlds value remains buffer-overflow-only
  • all availability reads remain read-only; no RCB reservation or write is introduced

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@masarray
masarray merged commit f25ab6a into main Jul 20, 2026
2 checks passed
masarray added a commit that referenced this pull request Aug 16, 2026
masarray added a commit that referenced this pull request Aug 16, 2026
Merge the CI-green ARSAS consumer fix, pinning ARIEC61850 to merged PR #79, preserving member-centric inventory invariants, and retaining compatibility normalization for legacy structured stVal report values.
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