From 3608dec30d29551601fb894f35dd58e2083b8c3a Mon Sep 17 00:00:00 2001 From: Matt Jones <47545907+SoundMatt@users.noreply.github.com> Date: Fri, 21 Aug 2026 08:09:16 -0700 Subject: [PATCH] ci: rename "RELAY interop gate" step to name what it runs (INTEROP-07) The step only drives `relay interop --protocol LIN` against RELAY's own embedded LIN golden vectors -- self-consistency against RELAY's own reference conversion, not third-party interoperability against an independent LIN peer. Per SoundMatt/RELAY docs/INTEROP-ARCHITECTURE.md's naming discipline, `*-interop` is reserved for a check against a peer this repo did not build; a step invoking only RELAY's own CLI should say so. Not previously flagged: the design doc's own current-state inventory only called out rust-RCP and rust-LIN as naming violators (at the job level) and missed this repo's step-level mislabeling entirely -- re-verified directly against this repo's real ci.yml rather than trusting that inventory. The enclosing job (`relay-conform`) was already correctly named; only this one step's display name was misleading. Pure rename, no behavior change: the underlying `run:` command is unchanged. Manual duplicate-job-ID scan (9 job ids, 9 unique) since `yaml.safe_load` alone can't be trusted to catch that class of bug. Verified locally against the pinned relay v2.0.4 build this job installs: built cpp-lin-cli from a clean cmake+ninja configure, then confirmed `relay conform --strict` and `relay interop --protocol LIN` both still PASS. SoundMatt/RELAY THEME-K (#125), docs/INTEROP-ARCHITECTURE.md shipping order item 2. Companion fix: SoundMatt/go-LIN#77. Sign-off: DCO Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com> --- .github/workflows/ci.yml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 40d3909..c851169 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -153,7 +153,14 @@ jobs: - name: RELAY conformance gate run: relay conform --strict ./build/cli/cpp-lin-cli - - name: RELAY interop gate (§20 Continuous Conformance) + # This step only drives RELAY's own CLI against RELAY's own embedded + # LIN golden vectors -- self-consistency against RELAY's own + # reference conversion, not third-party interoperability against an + # independent LIN peer. Named to say what it runs rather than + # "interop gate", per SoundMatt/RELAY docs/INTEROP-ARCHITECTURE.md's + # naming discipline (THEME-K #125, INTEROP-07): `*-interop` is + # reserved for a check against a peer this repo did not build. + - name: relay interop --protocol LIN (RELAY conformance gate, §20 Continuous Conformance) run: relay interop --protocol LIN ./build/cli/cpp-lin-cli sanitizers: