Skip to content

Preserve concrete live RCB names without duplicate 01 suffix#44

Merged
masarray merged 5 commits into
mainfrom
fix/standard-live-rcb-instance-name
Jul 21, 2026
Merged

Preserve concrete live RCB names without duplicate 01 suffix#44
masarray merged 5 commits into
mainfrom
fix/standard-live-rcb-instance-name

Conversation

@masarray

Copy link
Copy Markdown
Owner

Field defect

The live model already exposes the concrete MMS RCB object A_BRCB_1201. The generated SCL omitted ReportControl@indexed, whose SCL default is true, while also emitting RptEnabled max=1. A legacy importer therefore instantiated the already-concrete name again and addressed the non-existent A_BRCB_120101.

Standard-aligned correction

  • treat names obtained from live MMS discovery as concrete runtime instances
  • export each exact live name with indexed="false"
  • retain exactly one instance using RptEnabled max="1"
  • remove ClientLN index assignments from concrete live instances
  • align ConfReportControl max with the number of exported concrete RCBs
  • validate every live RCB name after normalization

Regression

The BCU7SL case now verifies that A_BRCB_1201 remains exactly A_BRCB_1201 and that A_BRCB_120101 cannot be produced.

@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 05e46c5 into main Jul 21, 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