Skip to content

chore(maintenance): consolidate runtime boundary and deterministic CI - #416

Merged
masarray merged 1 commit into
mainfrom
maintenance/combined-ci-validation-413-415-20260926
Sep 26, 2026
Merged

masarray merged 1 commit into
mainfrom
maintenance/combined-ci-validation-413-415-20260926

Conversation

@masarray

@masarray masarray commented Sep 26, 2026 •

Copy link
Copy Markdown
Owner

Purpose — consolidated first-pass maintenance candidate

This draft brings the exact file changes from #413, #414, and #415 together on one commit and validates the combined GitHub PR candidate. It is intended to reduce review/integration ambiguity, not create new implementation paths. Do not merge until the contributor's affirmative CLA acceptance is recorded and the final integration state is rechecked. The source PRs remain open as traceability and should only be closed as superseded after the chosen integration is actually merged.

Exact baseline and provenance

Combined-head validation — PASS on signed-off exact head

9/9 GitHub Actions workflows succeeded on f7d1f65163eca3731c33056bcbe6aa8cbff5572b after the contributor's DCO amend. The tree remains identical to the previously validated 8-file candidate.

  • Build ARSAS: source revision, solution build, 1,315 tests passed, 0 failed, 0 skipped, portable single-EXE publish and upload succeeded.
  • Smart Discovery Merge Execution Guard: 1,315 tests passed, 0 failed, 0 skipped.
  • Smart Discovery Field Capture Build: succeeded.
  • Legacy SAS RCB export guard: succeeded on the pinned engine.
  • Golden Budget Lock, Golden Provenance, Repeat-Run Stability, Mainline Readiness, and Production Promotion Guard: all succeeded.
  • The previous pre-amend head c37bec2c20f99077d8b6b4b2d38ebcc55896e2c9 is historical evidence only; the exact-head rerun above is authoritative.

No-regression boundary

No change to the accepted v1.6.40 Discovery/SCL production path, canonical model, static DataSet membership, RCB/GI/report-only authority, dynamic mutation/polling policy, engine or ArdIrec lock, release workflow, tag, or published packages. No physical IED operation was performed for this source-only maintenance change.

Merge requirements — not yet satisfied

  • DCO commit trailer is present on the current head: Signed-off-by: ARI SULISTIONO <ari.sulistiono@gmail.com>. This is distinct from a cryptographic Git commit signature, which is not required by DCO.
  • Affirmative CLA acceptance has not been observed in the PR comments. The repository's unprotected main has no formal approval-review requirement; technical source/blob and CI review is recorded in the PR discussion. No submitted GitHub approval is implied, and no legal agreement is asserted on the contributor's behalf.
  • The current signed-off exact-head rerun completed 9/9 green. After the CLA statement is recorded, recheck latest main, the final head and checks before any deliberate merge; no automatic merge.
  • If this consolidated PR becomes the accepted integration vehicle, close chore(maintenance): clarify runtime device contract and documentation index #413–ci(maintenance): use pinned ARIEC61850 in RCB export guard #415 as superseded only after it is merged, not before.

Tracks #380.

Integration result — verified

Copy link
Copy Markdown
Owner Author

Integration blocker (verified on 2026-09-26): this candidate is 9/9 CI green, but has no review submission and its commit c37bec2c20f99077d8b6b4b2d38ebcc55896e2c9 contains no Signed-off-by trailer. CONTRIBUTING.md requires every contribution commit to satisfy DCO.txt, plus affirmative agreement to CONTRIBUTOR-LICENSE-AGREEMENT.md.

Required authorized contributor/maintainer action before merge:

  1. Personally read the DCO/CLA, confirm legal rights and any required employer authorization, and affirm agreement to the CLA in this PR discussion.
  2. Create/approve a legitimate signed-off contribution commit using the contributor's own identity (do not substitute an AI-generated signature). Since SHA changes, rerun checks on the exact final combined head.
  3. Review the final eight-file diff and approve the integration under the repository's workflow.

I have not signed on anyone's behalf, marked the PR ready, or merged it. The dependent pure-formatter refactoring is in draft #417, explicitly stacked on this branch and not eligible for main integration until this gate is satisfied.

Signed-off-by: ARI SULISTIONO <ari.sulistiono@gmail.com>
@masarray
masarray force-pushed the maintenance/combined-ci-validation-413-415-20260926 branch from c37bec2 to f7d1f65 Compare September 26, 2026 09:36

Copy link
Copy Markdown
Owner Author

Technical integration audit (2026-09-26) — source and CI ready, CLA statement still outstanding.

Still needed before merge: contributor posts an affirmative CLA acceptance in this PR's Conversation (per CONTRIBUTING.md and CONTRIBUTOR-LICENSE-AGREEMENT.md). Current issue comments contain only the old blocker note; no new acceptance is visible. Do not post the legal declaration on the contributor's behalf. After it appears, recheck main, commit and exact-head CI, choose an intentional merge method, then verify the resulting main before changing #417's base.

Copy link
Copy Markdown
Owner Author

Pernyataan pribadi kontributor, diposting melalui ChatGPT berdasarkan izin eksplisit yang diberikan dalam percakapan:

Saya telah membaca dan menyetujui CLA ARSAS untuk kontribusi saya pada PR #416 dan #417. Saya memiliki hak yang diperlukan untuk menyumbangkan kontribusi tersebut, termasuk izin pemberi kerja jika diperlukan. Saya mengizinkan ChatGPT memposting pernyataan ini di kedua PR melalui akun GitHub saya.

@masarray
masarray marked this pull request as ready for review September 26, 2026 10:09
@masarray
masarray merged commit e7b4350 into main Sep 26, 2026
9 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