Skip to content

P3: gate dynamic reports by ARIEC association capability - #213

Merged
masarray merged 8 commits into
mainfrom
fix/p3-association-report-capability
Aug 18, 2026
Merged

P3: gate dynamic reports by ARIEC association capability#213
masarray merged 8 commits into
mainfrom
fix/p3-association-report-capability

Conversation

@masarray

Copy link
Copy Markdown
Owner

P3 — Consume association capability before dynamic reporting

Pins ARSAS to ARIEC61850 PR #85 (5149a28d7d8ab5a2126595a6ccaa130c46807dfe) and makes the engine's capability-aware planner authoritative at both decision points:

  1. initial hybrid acquisition planning
  2. fresh RCB revalidation immediately before activation

ARSAS does not duplicate MMS/RCB capability logic. It passes current association evidence to ARIEC and consumes the returned acquisition plan.

Effective capability gate

ARIEC P3 evaluates negotiated MMS Write / DefineNamedVariableList / DeleteNamedVariableList service bits plus fresh per-RCB DatSet, RptEna, TrgOps, OptFlds, GI, IntgPd, ownership/reservation and DataSet evidence. Unknown negotiation evidence is not treated as unsupported; explicit unsupported service bits are honored.

Critical activation fix consumed here

The P3 engine preserves fresh probed RCB Attributes and restores them onto the selected ReportControlCandidate before activation. This prevents a dynamic RCB that exposed TrgOps during availability probing from later being rejected merely because that evidence was lost between planning and StartPersistentReportMonitorAsync.

Diagnostics and safety

  • planning summary includes the ARIEC association-capability summary
  • capability warnings are surfaced through the existing planning warnings path
  • pre-start revalidation uses the same capability-aware planner with fresh availability
  • MMS polling remains the residual fallback when capability or exact RCB evidence is insufficient

Regression coverage ensures ARSAS uses the capability-aware planner twice (initial + revalidation), never calls the bare hybrid planner from this bridge, and pins the reviewed P3 engine/provenance.

@masarray
masarray marked this pull request as ready for review August 18, 2026 02:34
@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 388a765 into main Aug 18, 2026
4 checks 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