Skip to content

fix(reporting): expose source-neutral configured static activation - #137

Closed
masarray wants to merge 3 commits into
mainfrom
fix/canonical-static-report-activation
Closed

masarray wants to merge 3 commits into
mainfrom
fix/canonical-static-report-activation

Conversation

@masarray

Copy link
Copy Markdown
Owner

Purpose

Make the proven configured-static RCB activation sequence reusable by both canonical live-discovery models and opened SCL models without duplicating wire behavior.

Scope

  • adds StartConfiguredStaticReportMonitorAsync as the source-neutral entry point;
  • keeps StartStaticSclReportMonitorAsync as a backward-compatible wrapper;
  • preserves the existing minimal static activation sequence and fail-closed behavior;
  • adds an API regression test.

No discovery algorithm, DataSet decoding, dynamic DataSet behavior, or polling policy is changed.

@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 commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

Coordination note: the source-neutral StartConfiguredStaticReportMonitorAsync concept from this PR has now been carried forward onto current main in #145 / issue #144, together with the type-safe RCB mutation work. I am not changing this branch further so parallel history stays intact and we avoid two competing implementations of the same activation API.

Copy link
Copy Markdown
Owner Author

Staging reconciliation 2026-10-22: this old standalone PR's intended functional scope exists in the newer staging implementation, but the old branch still diverges from the newer code. Do NOT cherry-pick the historic file snapshots. Original HEAD 375543da77ea515757407e7587e08a259df9653a is not claimed an ancestor of staging.

Code and test evidence: Staging retains StartConfiguredStaticReportMonitorAsync plus backward-compatible StartStaticSclReportMonitorAsync wrapper. The SclStaticReportActivationPolicyTests.cs Git blob is byte-identical to the original branch; current implementation also retains later transactional/JIT proof hardening.

This source scope passed Windows PR #192 synthetic merge CI #39782663272: 1,262/1,262 tests, zero warnings/errors, exact staging SHA 44ac7554f43f2ba06763f3895242cb569b2e6a1c. Closing old PR as functionally superseded (without merge); Git branch/commits retained, and staging→main #192 stays DRAFT pending consumer and physical acceptance. Reopen if a specific missing invariant is documented.

@masarray masarray closed this Oct 22, 2026
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