From c84eb5aeefb6c1267d77e90e17dcd2f43d6f7ddc Mon Sep 17 00:00:00 2001 From: Joseph Doherty Date: Sun, 7 Jun 2026 10:06:58 -0400 Subject: [PATCH] docs(cli): note intentional omission of HiLo per-setpoint priorities/deadbands/messages (review fix) --- src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs b/src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs index f451a42f..8fa6eca3 100644 --- a/src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs +++ b/src/ZB.MOM.WW.ScadaBridge.CLI/AlarmTriggerConfigJson.cs @@ -56,6 +56,9 @@ internal static class AlarmTriggerConfigJson w.WriteString("direction", NormalizeDirection(direction)); break; case "hilo": + // Only the four setpoints are exposed as flags. The codec also accepts + // per-setpoint priorities/deadbands/messages — intentionally omitted here; + // use raw --trigger-config for those (see the YAGNI scope guard in the plan). if (loLo.HasValue) w.WriteNumber("loLo", loLo.Value); if (lo.HasValue) w.WriteNumber("lo", lo.Value); if (hi.HasValue) w.WriteNumber("hi", hi.Value);